Repository navigation
feat(first-mate-skill): add a skill kit for the paseo-first-mate skill - #2
Merged
Merged
Conversation
Add a hybrid/v1 mixin that drops the paseo-first-mate Agent Skill (SKILL.md plus a change detector and a wait loop) into ~/.agents/skills. The kit has files[] records only: no commands, egress, agent block, or ports. The skill is a one-time snapshot of the author's working copy (skill version 1.2.0), with owners set to @mogul and a dangling pattern-router reference removed; the ADR records why and how to update it. scripts/verify is an offline gate. CI now runs it and validates both kit specs with the pinned patterns validator.
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.
Context
Adds
first-mate-skill/, ahybrid/v1skill kit that drops thepaseo-first-mateAgent Skill (SKILL.mdplus a change detector and a wait loop) into~/.agents/skills/paseo-first-mate/in an acq sandbox. It lets a personal skill ride along with--kitorACQ_EXTRA_KITSwithout touching the playbook kit or the image.Plan
spec.yamlwith threefiles[]records only (nocommands,caps.network,agent:, or ports), the skill payload underfiles/home/.agents/skills/paseo-first-mate/, README, TROUBLESHOOTING, an ADR, and an offlinescripts/verify.ownersis@mogul, and a danglingpattern-routerreference was removed. The ADR records both and how to update the skill.README.md: adds a skill-kit entry, a kits-table row, and a layout entry..github/workflows/ci.yml: runsfirst-mate-skill/scripts/verify, and the patterns-validator step now checks bothpi-coding-agentandfirst-mate-skill(registrykits.yamlis built in a loop).Verification
first-mate-skill/scripts/verify: 20 passed, 0 failed (spec andfiles/agree, modes and executable bits, skill frontmatter, script syntax,shellcheck --severity=warning, empty default watch list, credential-pattern scan,acq kit validate).acq kit validate first-mate-skill: OK.a290d5d): both kits valid under--strict.acq runsandbox. The kit only drops files, so the offline gate covers what the kit decides.Rollback
Delete
first-mate-skill/, the README entries, and the two ci.yml changes (or revert this PR). Nothing else depends on the kit.Security impact
No credentials, egress, startup commands, or ports are added. The payload is plain text that steers an agent, so review
SKILL.mdbefore pinning a new SHA. Files land under/home/agentand are agent-writable (not a trust anchor).scripts/verifyscans the kit for token and key patterns. The scripts needpython3,gh, andpaseoin the sandbox; the kit installs none of them, and missing tools are reported rather than hidden.AI assistance
AI-assisted (OpenCode, claude-sonnet-5.5), reviewed by the author. Requires human review before merge; it was not self-approved.