Skip to content

feat(skills): add double-blind-review skill + skill symlinking - #72

Merged
ulises-c merged 2 commits into
mainfrom
feat/double-blind-review-skill
Aug 12, 2026
Merged

ulises-c merged 2 commits into
mainfrom
feat/double-blind-review-skill

Conversation

@ulises-c

Copy link
Copy Markdown
Owner

Summary

Adds agentic-ai/skills/ as a home for cross-agent skills, wires it into the Claude install, and ships the first skill: double-blind-review — two reviewers with different training runs (a Claude subagent and OpenAI Codex) review the same change blind to each other, then cross-examine each other's findings before anything is acted on.

Two logical commits:

  1. feat(agentic-ai): symlink cross-agent skills into ~/.claude/skillsinstall.sh links each skill directory individually (never the parent, so plugin-installed and hand-authored skills survive), -n keeps re-runs from nesting links, and a pre-existing non-symlink is warned about and skipped.
  2. feat(skills): add double-blind-review orchestration skillSKILL.md (the four-round protocol: blind launch → hold → cross-examination → orchestrator verification), round-1/round-2 prompt templates, and a vendored scripts/codex-run.sh so the skill stands alone with no plugin dependency.

Design notes

  • Distilled from a live run, where cross-examination surfaced a bypass neither reviewer found alone; the round-2 questions each carry the observed outcome that justifies them.
  • No model name pinned--effort xhigh is the only knob the skill sets; model selection stays with codex's configured default (codex debug models for discovery).
  • Standalone by design — the wrapper is vendored from the codex-review plugin (0.4.0), which itself duplicates the file per-plugin for the same reason; the vendored copy's header says to re-vendor rather than hand-edit on divergence.
  • Read-only review path — the wrapper defaults to read-only sandbox, prompt text travels as a single argv element from a file (injection-inert), and the skill forbids auto-applying fixes.

Verification

  • shellcheck --severity=warning passes on codex-run.sh (also enforced by pre-commit).
  • Security review of the branch: zero findings — prompt content cannot be re-expanded by the shell, and the symlink loop cannot be steered outside ~/.claude/skills by crafted names or pre-planted links.
  • Privacy scan: no identifying info in the new files (the earlier laza-guidance plugin-path reference was removed before commit).

🤖 Generated with Claude Code

ulises-c and others added 2 commits August 12, 2026 15:23
Link each skill under agentic-ai/skills/ individually rather than the
whole directory, so plugin-installed and hand-authored skills in
~/.claude/skills survive. -n keeps re-runs from nesting links; a
pre-existing non-symlink is warned about and skipped, never clobbered.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Two reviewers with different training runs (a Claude subagent and OpenAI
Codex) review the same change blind to each other, then cross-examine
each other's findings; the orchestrator spot-verifies load-bearing
claims and reports a reconciled verdict. Distilled from a live run where
cross-examination surfaced a bypass neither reviewer found alone.

Standalone by design: scripts/codex-run.sh is vendored from the
codex-review plugin (0.4.0), which itself duplicates the wrapper
per-plugin for the same reason. No model name is pinned — effort is the
only knob the skill sets.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@ulises-c
ulises-c merged commit 156afbd into main Aug 12, 2026
7 checks passed
@ulises-c
ulises-c deleted the feat/double-blind-review-skill branch August 12, 2026 22:30
ulises-c added a commit that referenced this pull request Aug 31, 2026
Single content conflict in agentic-ai/Claude/validate.sh: branch added the
Codex hook registration section, upstream (#72/#73) added skills and
output-styles checks at the same insertion point. Both are disjoint intents;
kept both, Codex registration first to stay adjacent to the Hooks section.

install.sh and README.md auto-merged cleanly: upstream's skills/output-styles
deployment coexists with the branch's Codex hooks.json registration; re-read
both for prose contradictions and found none.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant