Skip to content

AD-901: Add permission policy#47

Open
alph-cmky wants to merge 1 commit into
feat/ad-802-patch-proposal-servicefrom
feat/ad-901-permission-policy
Open

AD-901: Add permission policy#47
alph-cmky wants to merge 1 commit into
feat/ad-802-patch-proposal-servicefrom
feat/ad-901-permission-policy

Conversation

@alph-cmky

Copy link
Copy Markdown
Owner

Summary

  • Add a centralized server permission policy for read, write, safe commands, arbitrary commands, network, dependency installs, git commit, and git push.
  • Default agent policy allows read and safe commands, while dangerous permissions are denied and marked as requiring approval.
  • Add command evaluation that maps safe commands to safeCommands and unsafe commands to arbitraryCommands.
  • Export permission policy APIs from the server package and update the AD-901 backlog checklist.

Testing

  • TDD red check: pnpm test apps/server/src/security/permission-policy.test.ts initially failed because permission-policy.js did not exist.
  • pnpm test apps/server/src/security/permission-policy.test.ts
  • pnpm lint && pnpm format && pnpm typecheck && pnpm test && pnpm build

Stacked on #46.

Closes #22

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