Skip to content

docs: document PATCH /v1/sessions/:id (pinned sessions)#4058

Merged
aegis-gh-agent[bot] merged 1 commit into
developfrom
docs/pinned-session-patch-4057
May 23, 2026
Merged

docs: document PATCH /v1/sessions/:id (pinned sessions)#4058
aegis-gh-agent[bot] merged 1 commit into
developfrom
docs/pinned-session-patch-4057

Conversation

@OneStepAt4time
Copy link
Copy Markdown
Owner

Summary

Documents the new PATCH /v1/sessions/:id endpoint introduced in #4057 (pinned-session reaper, closes #4027).

CC v2.1.147 added pinned sessions (Ctrl+T). Aegis reapers now respect this β€” pinned sessions are never killed by stale or zombie reapers.

Changes

  • api-reference.md: new "Update Session Metadata" section β€” endpoint, RBAC (admin/operator), request body (isPinned), curl examples, response, error codes (400 INVALID_UPDATE, 404 NOT_FOUND)
  • api-quick-ref.md: added PATCH /v1/sessions/{id} to session-by-ID table

Verification

  • Endpoint matches src/routes/sessions.ts (registerWithLegacy patch, isPinned boolean check, updateSessionMetadata call)
  • RBAC matches code (withOwnership, no explicit requireRole β†’ inherits session ownership rules)

PR #4057 introduced PATCH /v1/sessions/:id for pinning sessions
to protect them from stale/zombie reapers (CC v2.1.147 pinned sessions).

- api-reference.md: new Update Session Metadata section with endpoint
  details, request/response, RBAC (admin/operator), and error codes
- api-quick-ref.md: add PATCH to session-by-ID table
@OneStepAt4time OneStepAt4time added the documentation Improvements or additions to documentation label May 23, 2026
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.

Review: PR #4058 β€” Document PATCH /v1/sessions/:id

Verdict: βœ… Approved

Docs-only, +44/-0, 2 files. Good catch on the gap from #4057.

Verified against code:

  • Endpoint, request body, error codes (400 INVALID_UPDATE, 404 NOT_FOUND) β€” all match src/routes/sessions.ts
  • RBAC table consistent with other mutating session endpoints
  • Quick-ref entry added
  • curl examples correct

9 Merge Gates β€” All Pass

  1. βœ… Review completed
  2. βœ… No conflicts β€” MERGEABLE
  3. βœ… CI green
  4. βœ… No regressions β€” docs-only
  5. βœ… N/A β€” docs-only
  6. βœ… E2E pass
  7. βœ… This IS the docs
  8. βœ… Security clean
  9. βœ… Targets develop

@aegis-gh-agent aegis-gh-agent Bot merged commit 8282195 into develop May 23, 2026
18 checks passed
@aegis-gh-agent aegis-gh-agent Bot deleted the docs/pinned-session-patch-4057 branch May 23, 2026 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant