fix(agent-org-ui): align recovery, plan approval, and task state#381
Closed
Harry19081 wants to merge 3 commits into
Closed
fix(agent-org-ui): align recovery, plan approval, and task state#381Harry19081 wants to merge 3 commits into
Harry19081 wants to merge 3 commits into
Conversation
Split the TypeScript contracts, Group Chat projections, plan approval controls, task outcome handling, settings, and UI unit coverage from the combined #373 change so the frontend can be reviewed on top of the backend contract. Verification: - npx vitest run Agent Org frontend tests (25 passed) - pnpm run lint - pnpm run check:circular (0 cycles) - pnpm run typecheck (only unchanged develop error at ContextInfoButton.tsx:468) - git diff --cached --check Pre-commit hook ran. Total eslint: 0, total circular: 0
Keep the rendered Group Chat, recovery, plan approval, pause/resume, task-state, and settings scenarios in a dedicated frontend test commit for the stacked #373 split. Verification: - node --check on all four changed Agent Org E2E modules - git diff --cached --check - Original combined branch completed live Agent Org manual verification Pre-commit hook ran. Total eslint: 0, total circular: 0
Keep the CLI parity, planner approval, and final UI audit decisions with the frontend half of the #373 split so reviewers retain the original design rationale. Verification: - git diff --cached --check Pre-commit hook ran. Total eslint: 0, total circular: 0
This was referenced Jul 15, 2026
Collaborator
|
Closed only to correct the PR creator identity. Replacement Frontend PR: #383. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #272 after the backend stack lands.
This is the frontend half of #373. It projects the durable Agent Org backend state introduced by #380 into Group Chat, plan approval, task/Kanban views, settings, structured tool guidance, and rendered E2E coverage.
Depends on: #380
Combined reference: #373
What changed
Review boundaries
Relative to #380, this PR contains:
src/**src-tauri/**filesThe stacked frontend tree plus #380 is byte-for-byte identical to the combined #373 snapshot
78338c38a.Merge order
develop.fix/issue-272-agent-org-recovery-backendtodevelop.developnormally if needed; do not rebase or force-push.Verification
pnpm run lint— passedpnpm run check:circular— no circular dependenciesnode --check— passed for all 4 changed E2E modulessrc/**, Agent Org browser E2E, and frontend audit files78338c38a— passedKnown target-branch baseline
Full
pnpm run typecheckreports one unchangeddeveloperror:That file is outside this PR diff.