Repository navigation
feat: agent skill package, harness installer, and HD logo render - #1
Merged
Merged
Conversation
Packaging toolscan for the agent ecosystem: a harness-agnostic skill (SKILL.md + composition reference + preflight example) teaching the real CLI surface and the fail-closed exit-code contract; an installer that deploys it to the four verified harness conventions with the same UX discipline as the Ix installer (dry-run, --json report, [id] labels, refusal guard, --help, preview/real exit parity); and a 2048px render of the radar mark for HD use.
Alot1z
marked this pull request as ready for review
September 6, 2026 00:14
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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.
Summary
Package toolscan for the agent ecosystem: a harness-agnostic agent skill, an installer that deploys it to every agent harness found on the machine, and a 2048px render of the radar mark for HD use.
What this ships
skills/toolscan/— the skill package:SKILL.md(the real CLI surface and the fail-closed exit-code contract: exit2= truncated scan = no verdict, never "not found"),references/composition.md(how agents compose with it), andexamples/agent-preflight.sh(a runnable preflight recipe). Machine-generic by construction — no personal paths, no machine-specific config.scripts/install-skill.sh— installer with the same UX discipline as the Ix project's installer:--dry-run(previews, writes nothing),--json(machine-readable per-host report),[id]labels, unknown-id/unknown-option refusal with--help, and a refusal guard that never silently overwrites a foreign same-name skill without--force. Dry-run and real mode agree on exit codes (preview ≡ real parity).assets/logo-2048.png— HD render of the radar mark.Tests
tests/skill-package.test.ts(new) pins the contracts doctor-style with hermetic HOME fixtures — the suite never touches real harness directories:--helplists all four harness ids; unknown id exits 1 listing them; dry-run writes nothing (byte-level directory comparison); refusal guard exits 1 in BOTH modes and leaves the foreign skill byte-identical;--forceoverwrites; real install lands in each present harness (cursor'sskills-cursorconvention pinned); re-install over its own output needs no--force;--jsonreport is a single parseable document with honest per-host records (skip =dest: null).Full suite: 68 passed / 3 skipped (71). Harness presence is tested through config-dir fixtures, never through
command -v, so the suite is hermetic on any machine.Validation
npm test— 5 files, 68 passed / 3 skipped.--forcepaths.Notes
skills-cursorpath) mirror the Ix repo's verified harness registry; kept deliberately minimal until more harnesses are verified.TOOLSCAN_DRY_RUN(namespaced, not borrowed).