Skip to content

Add external action permission scopes#121

Draft
FanyinLiu wants to merge 1 commit into
codex/notification-card-quick-actionsfrom
codex/external-action-permission-scopes
Draft

Add external action permission scopes#121
FanyinLiu wants to merge 1 commit into
codex/notification-card-quick-actionsfrom
codex/external-action-permission-scopes

Conversation

@FanyinLiu

Copy link
Copy Markdown
Owner

Summary

  • add once, current-session, and persistent grant scopes for main-process external action confirmations
  • keep session grants in memory while persistent grants promote the integration to auto mode
  • record grant scope metadata without logging message text, commands, target IDs, or tool arguments

Verification

  • npm test -- tests/external-action-policy.test.ts tests/external-action-audit.test.ts
  • npx tsc -b --pretty false
  • npm run error-redaction:audit
  • npm run lint
  • git diff --check
  • npm run verify:pr

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