Skip to content

Extract permission response command#888

Closed
boudra wants to merge 1 commit into
mainfrom
refactor-permission-response-command
Closed

Extract permission response command#888
boudra wants to merge 1 commit into
mainfrom
refactor-permission-response-command

Conversation

@boudra

@boudra boudra commented May 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • extract shared permission-response orchestration into an agent-domain command
  • route both WebSocket Session and MCP respond_to_permission through the same command
  • preserve follow-up prompt handling so MCP now starts the same follow-up run as Session

Verification

  • npx vitest run packages/server/src/server/agent/permission-response.test.ts --bail=1
  • npx vitest run packages/server/src/server/agent/mcp-server.test.ts --bail=1
  • npm run format
  • npm run typecheck
  • npm run lint

@boudra

boudra commented May 10, 2026

Copy link
Copy Markdown
Collaborator Author

Closing in favor of the single integration PR #893, which contains the full Session/MCP command-boundary stack and is being held until after the stable release.

@boudra boudra closed this May 10, 2026
@boudra boudra deleted the refactor-permission-response-command branch May 10, 2026 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant