feat(ship-check): tool-definition reviewer agent, skill, and surface-diff script - #33

Merged

aliasunder merged 29 commits into

mainfrom

feat/tool-definition-reviewer

Oct 4, 2026

Commits on Oct 3, 2026

feat(ship-check): surface-diff script for comparing MCP tool lists

aliasunder
and
claude
committed
7ee64cf
feat(ship-check): surface-diff prints a tool's definition as readable text

aliasunder
and
claude
committed
ccdf911
feat(ship-check): tool-definition-reviewer agent and tool-definition-review skill

aliasunder
and
claude
committed
275c103
docs: list the tool-definition reviewer and the bundled-script convention

aliasunder
and
claude
committed
84e2d88
refactor(ship-check): surface-diff runs on Bun only

aliasunder
and
claude
committed
8f84eb0
fix(ship-check): the tool-definition reviewer stops at eight tools and never reports untraced as complete

aliasunder
and
claude
committed
131b208
fix(ship-check): the reviewer's scope line cannot be mistaken for the intended list

aliasunder
and
claude
committed
6ca1364
fix(ship-check): review findings on the surface-diff script and workflows

aliasunder
and
claude
committed
cd348fa
fix(ship-check): duplication threshold applies to the trimmed overlap; validate job keeps no token

aliasunder
and
claude
committed
cee7553
docs: root README names all eight skills, the Bun prerequisite, and how to drop the MCP servers

aliasunder
and
claude
committed
b1f6e97
fix(ship-check): the tool-definition reviewer fails when its script cannot run, and five unclear passages are reworded

aliasunder
and
claude
committed
013a6fb
style: clarify comments in surface-diff, workflow path shapes, and README rubric naming

aliasunder
and
claude
committed
4404242
docs(ship-check): the reviewer's usage example defines its prompt lines, and AGENTS.md says where tests leave the release archives

aliasunder
and
claude
committed
4b93729
test(ship-check): cover every InputError path, CLI mode, and schema branch in surface-diff

aliasunder
and
claude
committed
f6a2418
fix(ship-check): the reviewer lists each traced failure as listed or MISSING, and --names is rejected with --show and --variants

aliasunder
and
claude
committed
cec0249
docs(ship-check): two skill sentences a test reader could take two ways

aliasunder
and
claude
committed
fdcb8ef
fix(ship-check): surface-diff reports a removed duplicate line and trims half characters from an overlap

aliasunder
and
claude
committed
df18884
fix(ship-check): surface-diff rejects a tool name that could carry shell syntax

aliasunder
and
claude
committed
3f7bd43
docs: the two READMEs and the marketplace description use plain punctuation in place of em dashes

aliasunder
and
claude
committed
023e243
fix(ship-check): the reviewer marks a schema-shadowed throw unreachable and names the files it searched for a size rule

aliasunder
and
claude
committed
6354e56
fix(ship-check): --show quotes an unknown tool name the same way the name check does

aliasunder
and
claude
committed
efcc51a
style: replace remaining em dash in plan-check marketplace description

aliasunder
and
claude
committed
dc1b650
ci: one release run at a time in each release workflow

aliasunder
and
claude
committed
1f1e39f
docs: the remaining public copy uses plain punctuation in place of em dashes

aliasunder
and
claude
committed
a1fbac0
fix(ci): check out current main in manual release, not the dispatch-time commit

aliasunder
and
claude
committed
963f400
fix(ci): a queued manual release checks out the branch tip, and the tag workflow keeps its original scheduling

aliasunder
and
claude
committed
5b8f2a3
docs: restore the original punctuation in the READMEs, manifests, and SECURITY.md

aliasunder
and
claude
committed
efd214f
docs: restore the last three README lines the punctuation rewrite changed

aliasunder
and
claude
committed
315985c
docs: the first adapting step in the root README is two plain sentences again

aliasunder
and
claude
committed
3a74463

Navigation Menu

feat(ship-check): tool-definition reviewer agent, skill, and surface-diff script - #33

feat(ship-check): tool-definition reviewer agent, skill, and surface-diff script#33
aliasunder merged 29 commits into
mainfrom
feat/tool-definition-reviewer

Commits on Oct 3, 2026

feat(ship-check): surface-diff script for comparing MCP tool lists

feat(ship-check): surface-diff prints a tool's definition as readable text

feat(ship-check): tool-definition-reviewer agent and tool-definition-review skill

docs: list the tool-definition reviewer and the bundled-script convention

refactor(ship-check): surface-diff runs on Bun only

fix(ship-check): the tool-definition reviewer stops at eight tools and never reports untraced as complete

fix(ship-check): the reviewer's scope line cannot be mistaken for the intended list

fix(ship-check): review findings on the surface-diff script and workflows

fix(ship-check): duplication threshold applies to the trimmed overlap; validate job keeps no token

docs: root README names all eight skills, the Bun prerequisite, and how to drop the MCP servers

fix(ship-check): the tool-definition reviewer fails when its script cannot run, and five unclear passages are reworded

style: clarify comments in surface-diff, workflow path shapes, and README rubric naming

docs(ship-check): the reviewer's usage example defines its prompt lines, and AGENTS.md says where tests leave the release archives

test(ship-check): cover every InputError path, CLI mode, and schema branch in surface-diff

fix(ship-check): the reviewer lists each traced failure as listed or MISSING, and --names is rejected with --show and --variants

docs(ship-check): two skill sentences a test reader could take two ways

fix(ship-check): surface-diff reports a removed duplicate line and trims half characters from an overlap

fix(ship-check): surface-diff rejects a tool name that could carry shell syntax

docs: the two READMEs and the marketplace description use plain punctuation in place of em dashes

fix(ship-check): the reviewer marks a schema-shadowed throw unreachable and names the files it searched for a size rule

fix(ship-check): --show quotes an unknown tool name the same way the name check does

style: replace remaining em dash in plan-check marketplace description

ci: one release run at a time in each release workflow

docs: the remaining public copy uses plain punctuation in place of em dashes

fix(ci): check out current main in manual release, not the dispatch-time commit

fix(ci): a queued manual release checks out the branch tip, and the tag workflow keeps its original scheduling

docs: restore the original punctuation in the READMEs, manifests, and SECURITY.md

docs: restore the last three README lines the punctuation rewrite changed

docs: the first adapting step in the root README is two plain sentences again