Repository navigation
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Refs #185. Certify the skill-only
darrow-explanationartifact against the native-platform plugin baseline without adding an unnecessary Python package or runtime dependency.What changed
Verification
bun test evals/runner/checks.test.ts evals/runner/explanation-eval-checks.test.ts scripts/check-docs.test.ts: 39 passed.bun scripts/check-explanation-install.ts: both native host installations passed on macOS, including installed skill bytes and Claude discovery.inspect-skill,bun run check:docs,bun run lint,bun run lint:ts,bun run lint:shell,bun run typecheck, andbun run check:decisions: passed.gpt-5.6-terraand Claudeclaude-sonnet-5, medium effort, one trial per invocation, and an 80% per-case threshold. CLI versions: Codex 0.154.0 and Claude Code 2.1.223. Final Claude results: 9/9 task and activation checks passed. Across the first final-candidate trial of each case, Codex passed 8/9 task checks and 8/9 activation checks, with failures in different cases. An additional unchanged algorithm diagnostic passed both; the complete live evidence is not uniformly green.explain-visually, failing the activation exclusion. This distinct false activation is retained as a failed check, not counted as a task failure or a pass.505488c6d951e560ed332a84717eab56ca9e4ef7, native installation and assertion CI passed on all three platforms, including native Windows after resolving the npm host entrypoints directly. Documentation CI and Python inventory/aggregate CI also passed; unrelated Python package jobs were skipped by change scope.Review notes
Claude activation variability is documented as a nonblocking limitation, as requested by the maintainer. The final description improves the observed pressure-case activation, but the sample is small and earlier activation misses remain visible. The Codex wording inconsistency likewise limits any reliability claim; this is bounded portability evidence rather than an assertion that every live trial passed.
Authenticated native Linux and Windows invocation remains unverified because no such environment or CI credentials are available. The maintainer accepted working with the existing environments. The credential-free matrix establishes installation and assertion portability; macOS provides the live model evidence. No live cross-platform certification is claimed.
Checklist
CONTRIBUTING.md, including the contributionlicensing terms.
this change does not affect a capability invariant.
behavior.
change does not affect plugin content.
bun run check:python, or this change does not affect registeredPython packages or their repository quality infrastructure.