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.
🟡 Changes recommended
The recommended verification command omits its required box argument and only displays help.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Updates agent guidance to accurately describe silent no-ops when incorrect IDs are passed to posting commands.
Changes:
- Documents affected commands and verification guidance.
- Clarifies which commands still return
not_found.
[!TIP]
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or rungh pr ready --undo.
Click "Ready for review" or rungh pr readyto reengage.
File summaries
| File | Description |
|---|---|
skills/hey/SKILL.md |
Corrects ID-error behavior guidance. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 04c6e4f83f
ℹ️ 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".
There was a problem hiding this comment.
🟡 Changes recommended
The verification recipe can still misclassify capped listings and ID collisions.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Balanced
HEY's seen, unseen and moves endpoints scope the requested ids to the identity's postings and answer success whatever matched, and there is no per-posting read to check against, so the CLI cannot answer not_found for a wrong-kind id there. The skill said it does, which is what let an agent trust a week of no-op marks (hey-cli#396). Say what actually happens and how to confirm, until HEY reports the non-match.
Refs #396 (does not close it — the fix that issue asks for is on HEY's side)
What is wrong
The skill says a
topic_idpassed tohey moveanswersnot_found, and that passing the wrong kind of id to any posting command answersnot_found. Neither is true forhey seen,hey unseenorhey move: HEY's/postings/seen,/postings/unseenand/postings/movesendpoints scope the requested ids to the identity's postings and answer success whatever matched, and there is no per-posting read the CLI could check against.hey trash,hey spamand the label/folder commands do 404 when nothing matches, which is where the general claim came from. That false promise is what let an agent trust several days of silently no-op'd marks in #396.Change
Documentation only: the two ID notes in
skills/hey/SKILL.mdnow say which commands cannot report a non-match yet and how to confirm one (hey box view --json). No client-side pre-check is added — that is a server change (a404when none of the ids belong to the identity, and a per-id result for a partial batch), recorded against the API separately; once it lands the CLI surfaces it asnot_foundunchanged.Summary by cubic
Updates
skills/hey/SKILL.mdto stop promisingnot_foundfor atopic_idpassed tohey seen,hey unseen, orhey move. Those commands ignore unmatched ids, act on any that do match, and answer success regardless, so a mixed batch is a partial success reported as whole. Docs now show confirming a mark withhey box view <box> --json --all, reading every page (including the cursor and notice) so a posting past the first page isn't mistaken for a non-match. Docs only, no behavior change; cleans up the misleading promise that let an agent trust no-op'd marks (refs #396, does not close it).Written for commit 5c50e3d. Summary will update on new commits.