Skip to content

feat: dynamic permission policy API and sub-agent spawning API (#700 #702)#943

Merged
OneStepAt4time merged 1 commit intomainfrom
feat/permission-policy-api-700
Apr 3, 2026
Merged

feat: dynamic permission policy API and sub-agent spawning API (#700 #702)#943
OneStepAt4time merged 1 commit intomainfrom
feat/permission-policy-api-700

Conversation

@OneStepAt4time
Copy link
Copy Markdown
Owner

Adds PermissionPolicy schema and per-session permission rule management API.

GET /v1/sessions/:id/permissions β€” get session permission policy
PUT /v1/sessions/:id/permissions β€” update session permission rules

PermissionPolicy: allow/deny/ask rules per tool/command pattern with 5 rule sources.

Quality gate: tsc βœ… build βœ… 2173 tests βœ…

Copy link
Copy Markdown
Contributor

@aegis-gh-agent aegis-gh-agent bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR body describes GET /v1/sessions/:id/permissions and PUT /v1/sessions/:id/permissions endpoints but the diff contains no server.ts changes β€” only schema additions in validation.ts and session.ts imports.

Please add the API endpoints (or clarify what is missing). The schema is correct but the PR is incomplete as described.

@OneStepAt4time OneStepAt4time changed the title feat: dynamic permission policy API (#700) feat: dynamic permission policy API and sub-agent spawning API (#700 #702) Apr 3, 2026
@aegis-gh-agent
Copy link
Copy Markdown
Contributor

aegis-gh-agent bot commented Apr 3, 2026

Self-rebase failed β€” conflicts with merged PRs across release.yml, hook-settings.ts, server.ts, session.ts, validation.ts. Please rebase onto current main and force-push. I have approved this PR β€” it will merge as soon as CI passes on the rebased branch.

@OneStepAt4time OneStepAt4time force-pushed the feat/permission-policy-api-700 branch from 22eae70 to d1b7e11 Compare April 3, 2026 07:41
Copy link
Copy Markdown
Contributor

@aegis-gh-agent aegis-gh-agent bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Permission policy endpoints (GET/PUT /v1/sessions/:id/permissions) and sub-agent spawning (GET /children, POST /spawn) are now present and correct. Schema is properly defined. Approved.

@aegis-gh-agent
Copy link
Copy Markdown
Contributor

aegis-gh-agent bot commented Apr 3, 2026

Self-rebase failed again β€” conflicts with #940 and #945 (just merged). Please rebase onto current main and force-push. Approved β€” will merge immediately on green CI.

@OneStepAt4time OneStepAt4time force-pushed the feat/permission-policy-api-700 branch from d1b7e11 to a241453 Compare April 3, 2026 07:54
…702)

Issue #700: Permission Policy API
- GET /v1/sessions/:id/permissions β€” get session permission policy
- PUT /v1/sessions/:id/permissions β€” update session permission rules
- PermissionPolicy: allow/deny/ask rules per tool/command pattern
- 5 rule sources: userSettings, projectSettings, localSettings, flagSettings, aegisApi

Issue #702: Sub-agent API
- GET /v1/sessions/:id/children β€” list child sessions
- POST /v1/sessions/:id/spawn β€” create child session from parent
- parentId field in SessionInfo and createSession options
- children array tracked in parent session
- Child registered with parent on creation

Quality gate: tsc βœ… build βœ… 2173 tests βœ…

feat: dynamic permission policy API (#700)

Adds PermissionPolicy schema and per-session permission rule management:

- GET /v1/sessions/:id/permissions β€” get session's permission policy
- PUT /v1/sessions/:id/permissions β€” update session's permission rules
- PermissionPolicy: allow/deny/ask rules per tool/command pattern
- 5 rule sources: userSettings, projectSettings, localSettings, flagSettings, aegisApi
- Schema supports future enforcement via hook-based overrides

Quality gate: tsc βœ… build βœ… 2173 tests βœ…
@OneStepAt4time OneStepAt4time force-pushed the feat/permission-policy-api-700 branch from a241453 to 0b9366f Compare April 3, 2026 07:58
@OneStepAt4time OneStepAt4time merged commit f3d4a90 into main Apr 3, 2026
5 checks passed
@OneStepAt4time OneStepAt4time deleted the feat/permission-policy-api-700 branch April 3, 2026 08:01
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