Skip to content

Add workflow permission hook#805

Open
hccnm wants to merge 1 commit into
agentclientprotocol:mainfrom
hccnm:workflow-permission-hook
Open

Add workflow permission hook#805
hccnm wants to merge 1 commit into
agentclientprotocol:mainfrom
hccnm:workflow-permission-hook

Conversation

@hccnm

@hccnm hccnm commented Jun 23, 2026

Copy link
Copy Markdown

Summary

  • Adds first-class Workflow tool display/result parsing and session-level workflow run tracking.
  • Routes Workflow tool launches through the existing ACP requestPermission surface with run_once, view_script, and deny options.
  • Applies the same permission hook to workflow slash command dispatch before the prompt is sent to the SDK, with a single-turn grant to avoid duplicate approval when the SDK invokes the matching Workflow tool.
  • Exposes workflow state through _claude/workflows/list, _claude/workflows/get, _claude/workflows/stop, and workflow update notifications.

Notes

  • ACP does not make business approval decisions. It only exposes the workflow launch interception point and relays the client choice.
  • /workflows remains local-only and does not enter the model.
  • bypassPermissions continues to allow Workflow tool use without prompting.

Validation

  • npm ci
  • npm run format:check
  • npm run lint
  • npm run build
  • npm run test:run

@hccnm hccnm marked this pull request as ready for review June 23, 2026 07:45
@hccnm hccnm force-pushed the workflow-permission-hook branch from e65bdc0 to 79013b7 Compare July 2, 2026 02:04
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