Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8fd9b96622
ℹ️ 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.
Pull request overview
Docs/backlog hygiene around the tools/peer-call surface: adds a “future direction” section in the peer-call README, closes backlog row B-0119, and adds cross-references between peer-call backlog rows.
Changes:
- Add a “Future direction (queued backlog work)” section to
tools/peer-call/README.mdthat points to related backlog rows. - Mark B-0119 as closed and record where it was closed.
- Add
composes_withcross-refs from B-0120 to the related queued work. - Regenerate
docs/BACKLOG.mdto reflect B-0119 as closed.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| tools/peer-call/README.md | Adds a “Future direction” section that links to queued backlog work. |
| docs/backlog/P3/B-0119-peer-call-existing-scripts-role-ref-cleanup-2026-04-30.md | Marks B-0119 closed and adds closure metadata + new cross-ref. |
| docs/backlog/P2/B-0120-peer-call-architecture-refactor-script-per-cli-persona-flag-2026-04-30.md | Adds cross-references to related backlog rows. |
| docs/BACKLOG.md | Updates the index to show B-0119 as closed. |
…B-0121/B-0122 Three small hygiene moves around the peer-call surface: 1. tools/peer-call/README.md gains a "Future direction (queued backlog work)" section pointing at B-0120 (architecture refactor) + B-0121 (Otto/Kenji additions) + B-0122 (TypeScript migration). Future readers + agents see the trajectory without grep-spelunking. 2. B-0119 status open → closed, closed_in: PR #965. The role-ref cleanup landed on main as aab3adb; the row needs to reflect that. 3. B-0120 composes_with gains B-0121 + B-0122 cross-refs. The three rows form a triangle: B-0120 names the architecture, B-0121 adds new peer surfaces in that architecture, B-0122 names the language target. Cross- refs make the relationship discoverable. Note on BACKLOG.md state: this branch's regenerated index shows B-0119 closed and B-0120 unchanged but does NOT yet include B-0121 or B-0122 (their PRs #964 and #966 haven't merged yet). The next regen post-merge picks them up. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
8fd9b96 to
12ae05e
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 12ae05e067
ℹ️ 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".
* hygiene(tick-history): shard 0026Z — queue-saturation tick Per autonomous-loop tick-must-never-stop discipline. Queue at 27 PRs; pivoted from speculative work to queue-care (#967 rebased + #754 auto-merge armed). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * fix(tick-history): col3 — '(none)' → cron-id 98fc7424 per schema (Copilot review) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
…ef + B-0122 forward-references
Two classes of fix (6 threads — Codex P2 + Copilot P1 each ×3):
1. **Persona names in current-state doc (P1, line 247+253 of README)**:
`.github/copilot-instructions.md:306-362` requires role-refs (not
persona names) on current-state surfaces (code/docs/skills outside
the closed-list history surfaces). Reworded:
- "Add Otto + Kenji as externally-callable peers" → "Add the
Anthropic-side Claude-code-instance personas as externally-callable
peers"
- "Otto + Kenji additions (B-0121)" → "The Anthropic-side
(Claude-code-instance) peer additions (B-0121)"
B-0121 filename slug stays as `otto-kenji-*` (closed-list history
surface — backlog rows allow persona names per the carve-out).
2. **B-0122 forward-references (P1+P2, 4 threads)**: B-0122 is filed in
the in-flight PR #966; not yet on main. References in `B-0119`,
`B-0120`, and the README's table cell rewritten:
- YAML `composes_with` lists in B-0119 + B-0120: replaced direct
path with inline `# B-0122 ... is filed in the in-flight PR #966`
comment.
- README table row for B-0122: replaced live link with explicit
"filed in in-flight PR #966; link will resolve once that PR lands"
callout.
B-0121 stays as direct ref because it IS on main.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
| - [ ] **[B-0115](backlog/P3/B-0115-zsh-vim-muscle-memory-aliases-wq-q-2026-04-30.md)** Shell aliases for `:wq` / `:wq!` / `:q` — catch vim-muscle-memory leakage in zsh (Deepseek 2026-04-30 finding) | ||
| - [ ] **[B-0116](backlog/P3/B-0116-gh-jq-safe-wrapper-zsh-quoting-2026-04-30.md)** tools/gh-jq-safe.sh — wrap gh-jq calls to handle zsh quoting (Deepseek 2026-04-30 finding) | ||
| - [ ] **[B-0119](backlog/P3/B-0119-peer-call-existing-scripts-role-ref-cleanup-2026-04-30.md)** Existing peer-call scripts (grok.sh / gemini.sh / codex.sh / amara.sh) — role-ref cleanup per copilot-instructions.md (Codex 2026-04-30 finding on PR #962) | ||
| - [x] **[B-0119](backlog/P3/B-0119-peer-call-existing-scripts-role-ref-cleanup-2026-04-30.md)** Existing peer-call scripts (grok.sh / gemini.sh / codex.sh / amara.sh) — role-ref cleanup per copilot-instructions.md (Codex 2026-04-30 finding on PR #962) | ||
| - [ ] **[B-0123](backlog/P3/B-0123-stacked-pr-create-tooling-gh-fallback-aaron-2026-04-30.md)** Stacked-PR creation tooling — `gh pr create --base <not-main>` fails with cryptic GraphQL error; needs a wrapper or doc (Aaron 2026-04-30) | ||
|
|
There was a problem hiding this comment.
The PR description says the regenerated BACKLOG index "does NOT yet include B-0121", but docs/BACKLOG.md in this branch already contains a B-0121 entry (currently around line 94). Please update the PR description (or the note) so it matches the actual state of the index on this branch.
Summary
Three small hygiene moves around the peer-call surface, landing as one diff:
tools/peer-call/README.mdgains a "Future direction (queued backlog work)" section pointing at B-0120 (architecture refactor) + B-0121 (Otto/Kenji additions) + B-0122 (TypeScript migration). Future readers and agents see the trajectory without grep-spelunking.B-0119 status: closed. Role-ref cleanup landed in PR cleanup(B-0119): peer-call scripts role-ref discipline (2026-04-30) #965 (commit
aab3adbon main). Row file updated to reflect that —closed_in: PR #965 (LFG, merged 2026-04-30)field added.B-0120 cross-refs B-0121 + B-0122 in its
composes_withfrontmatter. The three rows form a triangle: B-0120 names the architecture, B-0121 adds new peer surfaces in that architecture, B-0122 names the language target. Cross-refs make the relationship discoverable.Note on BACKLOG.md state
This branch's regenerated index shows B-0119 closed and B-0120 unchanged but does NOT yet include B-0121 or B-0122 — their PRs #964 and #966 are still in CI on the same main base. The next regenerate post-merge picks them up. No drift; just sequencing.
Test plan
[x]🤖 Generated with Claude Code