docs(plugin): teach session refresh lifecycle - #758
Conversation
|
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. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7c1e06ba19
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
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".
| using the same selectors and returned handle. Use the authoritative | ||
| selectors provided by the host/runtime; do not reconstruct refresh identity | ||
| from chat text or a filesystem path. | ||
| host or user lifecycle intent before calling `tracedecay_session_refresh_begin`. |
There was a problem hiding this comment.
Route profile refreshes through the compatible lifecycle
When tracedecay_message_search runs against an authorized profile root and returns refresh_required, this instruction sends the agent to tracedecay_session_refresh_begin, but the split begin/status/cancel schemas require a project identity and are mounted only under project-open admission. They cannot accept the authoritative profile selectors that the compatibility tracedecay_session_refresh tool supports, so the documented lifecycle is unusable for stale profile-scoped reads; make this routing scope-aware and retain the compatibility lifecycle for profile refreshes.
AGENTS.md reference: AGENTS.md:L9-L12
Useful? React with 👍 / 👎.
1e6ed08
into
ScriptedAlchemy:codex/tracedecay-total-redesign-plan-reopened
Summary
Why
PR #707 splits session refresh into lifecycle tools, but the packaged guidance exposed status without teaching agents how to safely start, track, and cancel the operation.
Verification
Supports #707.