umputun / revdiff / 24640363081
90%

DEFAULT BRANCH: master
Ran
Jobs 1
Files 58
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 89.063%. Remained the same
24640363081

push

github

web-flow 
feat: launcher override chain for Claude plugins (#126)

* docs: add launcher override-chain implementation plan

Plan to adopt the cc-thingz override-chain pattern for the two
Claude-side launcher scripts (launch-revdiff.sh and
launch-plan-review.sh) so users can drop their own launcher at
.claude/<plugin-namespace>/scripts/ or ${CLAUDE_PLUGIN_DATA}/scripts/
without forking the plugin. Resolves action item from discussion #121.

* feat: add override-chain resolver to revdiff diff-review skill

* feat: add launcher override chain to revdiff-planning plugin

* chore(plugins): bump revdiff to 0.8.0 and revdiff-planning to 0.3.0

* docs: sync launcher override chain notes across top-level docs

* docs: archive launcher override-chain plan as completed

* fix: address code review findings

* fix: address iteration 2 critical review findings

* fix: quote env vars in resolver call to handle spaces in paths

* fix(security): drop project-layer override to prevent untrusted-repo code execution

The revdiff-planning hook fires automatically on every ExitPlanMode in
any repo. A project-layer executable override would let an untrusted
repo run arbitrary code on routine Claude actions without per-repo
opt-in. Drop the project layer from both resolvers (the diff-review
skill keeps the same shape for symmetry, even though it's user-invoked)
and reduce the chain to user (${CLAUDE_PLUGIN_DATA}/scripts/) +
bundled. Flagged by codex external review.

* fix: harden resolver against directory entries and document addendum

The -x test alone matches directories (search permission), so a user-data
directory named launch-revdiff.sh / launch-plan-review.sh would win over
the bundled launcher and then fail at exec. Combine with -f to require a
regular file. Also append a scope-change addendum to the archived plan
documenting the project-layer removal and this fix. Flagged by codex.

* fix: quote plugin-root path in hook command and clean up archived plan

- hooks.json: quote... (continued)

7549 of 8476 relevant lines covered (89.06%)

474.86 hits per line

Jobs
ID Job ID Ran Files Coverage
1 24640363081.1 58
89.06
 GitHub Action Run
Source Files on build 24640363081