AI Assist Inbox: Action button should immediately import group

[ryenski]

In the AI Assist inbox, currently the process is:

1. select a group,
2. review the transactions in the group,
3. click Looks Right Next group,
4. Go through all the groups
5. Approve selected entries.

The problem with this flow is that if the user clicks on a transaction in the middle of a group, and then goes back to the AIS's inbox, they have to start over at the beginning at the first group.

Instead, what should happen is the action button on each group should say approve and import.Clicking that button immediately imports the selected transactions in that group.The screen should stay on the inbox page and forward to the next group of entries.

[ryenski]

This was generated by AI during triage.

Agent Brief

Category: enhancement
Summary: AI Assist review approves one group at a time — each group's primary action immediately imports its checked entries (and its checked rules) instead of staging the whole pass for a final signing step.

Current behavior:
The AI Assist review is a step wizard over a completed pass: one step per ledger-account group, then a "Needs your eye" step, then a "Sign & approve" step. A group's primary button ("Looks right — next group") only marks the group reviewed in local component state; nothing touches the ledger until the final step, which approves the entire staged selection in one batch and closes the pass. Review state (current step, row/rule checkboxes) is component-local, so leaving the review mid-flow (e.g. clicking a row to edit it) and returning restarts at the first group. On the backend, batch approval validates entries against the pass, writes them, and unconditionally marks the whole pass approved.

Desired behavior:

• Group-level immediate approval. Each group step's primary button reads "Approve and import" and immediately approves that group's checked entries — each with its AI-suggested ledger account, payee, and narration — plus the group's checked proposed rules, as one batch. Snapshot-before-write, the ledger-validation gate, rollback on failure, and the "only the current pass can be approved" guard behave exactly as they do for today's final approval.
• Review advances in place. After a successful group approval the review stays open and moves to the next remaining group. Approved rows leave the pass, so their group disappears from the review.
• Needs-eye parity. The "Needs your eye" step gets the same kind of primary action: immediately import its checked rows (each with its validated suggestion) rather than proceeding to a signing summary.
• Signing step removed. The "Sign & approve" step no longer exists.
• Unchecked rows in an approved group drop out of the pass. Their suggestions are consumed; the rows stay pending in the regular Inbox for manual handling. The group does not reappear. (The existing "Unchecked rows stay in the Inbox — nothing is lost" promise still holds; they are just no longer part of the AI Assist review.)
• "Skip for now" leaves the group in the pass and moves to the next step; skipped groups remain in the review for later action.
• Pass lifecycle. The pass stays active while it still has groups with live suggestions. When a batch consumes the pass's last group, the backend marks the pass approved (leftover needs-eye suggestions close with the pass, their rows staying pending in the Inbox), and the UI exits the review and shows the regular Inbox list with whatever rows remain.
• Resumability falls out naturally. Because approved groups vanish from the pass, leaving the review mid-pass (e.g. to edit a row) and returning resumes at the first still-unapproved group — no starting over at group 1.

Key interfaces:

• ApproveAiAssistBatchInput — the input shape is unchanged; batches may now cover a subset of the pass's rows, and one pass may produce many batches.
• The backend batch-approval command — must support multiple batches per pass; transition the pass to approved only when a batch consumes the pass's last group (no remaining suggested-status suggestions); record each batch separately so each is individually revertible via its snapshot.
• The pass-state loader behind AiAssistPassState — must not surface suggestions whose Statement Rows are already accounted, nor proposed rules that a previous batch already created, so consumed groups disappear from a refreshed pass.
• useAiAssistLifecycle — the approve path must not unconditionally clear the pass: keep it and refresh pass state when the backend leaves it active; clear it only when the backend closes it.
• AiAssistReview — the group accept action becomes the immediate-approval action; local "reviewed/skipped" bookkeeping is replaced by the pass itself shrinking; the signing step is removed; Enter remains the primary-action keyboard shortcut.

Acceptance criteria:

• Each group card's primary button reads "Approve and import" (with the checked-entry count) and clicking it writes that group's checked entries to the ledger immediately as one batch, with snapshot, validation gate, and rollback semantics unchanged.
• The group's checked proposed rules are created in the same batch as its entries.
• After a successful group approval the review stays mounted and advances to the next remaining group; the approved group no longer appears in the review.
• Unchecked rows in an approved group are not written, no longer appear in the pass, and remain pending in the regular Inbox; the group does not reappear.
• "Skip for now" leaves the group in the pass and moves to the next step; skipped groups remain in the review.
• The needs-eye step has an immediate "Approve and import" action covering its checked rows; unchecked needs-eye rows remain pending in the regular Inbox.
• The "Sign & approve" step no longer exists anywhere in the review flow.
• When a batch consumes the pass's last group, the pass is marked approved, the review exits, and the regular Inbox list is shown with leftover pending rows (e.g. needs-eye rows never checked).
• Multiple batches per pass are each recorded with their own entries and rules and are individually revertible via snapshot restore.
• Leaving the review mid-pass (e.g. editing a row) and returning resumes at the first still-unapproved group; previously approved groups are gone.
• Approval actions are disabled while a batch is in flight.
• A batch that would leave the ledger invalid is rolled back, the pass stays open, and the group remains in the review.
• The "only the current pass can be approved" guard still rejects stale batch approvals.

Out of scope:

• Changing how single (non-AI-Assist) Inbox approvals or transfer approvals work.
• Changing AI Assist pass creation, chunking, retry of failed rows, or dismissal.
• Persisting review UI state (checkboxes, current step) across app restarts.
• Changing the snapshot mechanism, batch record format, or revert flow.
• New keyboard shortcuts beyond keeping Enter as the group primary action.