Repository navigation
Add read-only CLI session inspection - #56
Conversation
|
@codex review |
|
Warning Review limit reached
Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing. Next included review available in 11 minutes. View limit detailsLimit details: You’ve used all 2 included reviews currently available. Your 66 included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour. The included review limit has been reached and this organization has disabled usage-based review continuation. Wait for reviews to reset or ask a billing admin to change After included review limits. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (6)
Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. Keep it up! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Change
Add
polymorfa session list,view, andstatususing the existing typed Platform SDK. An organization API key withsessions:readis required.listcan filter by project;viewandstatususe a team-unique session ID and reject--projectbefore transport.statusreturns only stored status and optional reason, without claiming a live probe.The API does not expose organization-key mint or rotate through this Platform surface, and the SDK has no typed methods for them. Existing
key list/revoke,auth,project, andlinkcommands are retained without invented credential mutations.Public documentation is in monorepo PR #398, which must merge before this CLI PR.
Five-part assessment
GET /platform/sessionsandGET /platform/sessions/{id}requiresessions:read. Organization-key mint/rotate remains blocked on a safe API contract and owner decision.@polymorfa/sdk@0.1.0-dev.20260926005918provides typedsessions.list/retrieve. Key mint/rotate is absent from its typed resource.Verification
On Node 22: format, lint, typecheck, build, and full suite passed (916 tests, 4 skipped). New tests cover team/project list, view/status projection, credential authority, unsupported project filter, and missing-session failure.
Need help on this PR? Tag
@codesmithwith what you need. Autofix is disabled.