Skip to content

docs: drop the pinned version from the port's preamble #929

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
max-sixty merged 1 commit into from
Aug 9, 2026
+11 −6
Merged

docs: drop the pinned version from the port's preamble #929

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 10 additions & 5 deletions TODO.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -280,11 +280,16 @@ The same gap covers `tend-outage`; #816 tracks it from that side, and the
## Decide whether the built-in `/code-review` replaces the vendored port

`plugins/tend-ci-runner/skills/code-review/` is a copy of Claude Code's
built-in `/code-review`, read out of the 2.1.220 binary `claude/action.yaml`
pins. The two share their method almost verbatim — the same ten angles, the
same three-verdict verify with "PLAUSIBLE by default", the same sweep — so the
copy buys nothing on method and costs the usual: it drifts silently, and
nothing re-checks it against the binary.
built-in `/code-review`, taken from a 2.1.220 binary. The two share their
method almost verbatim — the same ten angles, the same three-verdict verify
with "PLAUSIBLE by default", the same sweep — so the copy buys nothing on
method and costs the usual: it drifts silently, and nothing re-checks it
against the binary.

That drift is no longer hypothetical. `claude/action.yaml` now pins 2.1.226,
which restructured the built-in without touching those texts, so the copy is
already a version behind the binary CI runs — and the thing that changed is
exactly what decides this question.

Reaching the built-in is possible. It carries `disable-model-invocation`,
which the `Skill` tool waives for a turn whose own user message names the
Expand Down
2 changes: 1 addition & 1 deletion plugins/tend-ci-runner/skills/code-review/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ metadata:

# Code review

A structured pass over a diff that returns ranked findings. Ported from Claude Code's built-in `/code-review`, read out of the 2.1.220 binary `claude/action.yaml` pins. The Codex harness has no built-in equivalent, and on Claude the built-in carries `disable-model-invocation`, which no tend prompt lifts. This copy is tend-owned: reachable from the model on either harness.
A structured pass over a diff that returns ranked findings. Ported from Claude Code's built-in `/code-review`. The Codex harness has no built-in equivalent, and on Claude the built-in carries `disable-model-invocation`, which no tend prompt lifts. This copy is tend-owned: reachable from the model on either harness.

**Return findings; don't act on them.** No review, comment, commit, or artifact from this skill — the caller folds the findings into its own single review.

Expand Down
Loading