Skip to content

feat(installer): support explicit-only skill invocation - #174

Merged
abretonc7s merged 2 commits into
mainfrom
feat/explicit-skill-invocation
Sep 23, 2026
Merged

abretonc7s merged 2 commits into
mainfrom
feat/explicit-skill-invocation

Conversation

@abretonc7s

@abretonc7s abretonc7s commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Add explicit-only workflow metadata to the skill installer and emit each supported runner's native invocation controls. Preserve repository overlays and custom files during install and reinstall.

  • Contains installer/schema changes, installation tests and documentation.
  • Perps review content is isolated in feat(perps): distribute generated review checklists #163, which is stacked on this branch.
  • Validation: explicit-invocation tests passed on this branch; all 75 repository tests passed on the stacked Perps branch.
  • The published installer is still 0.3.1. Adoption requires a new installer release and consumer dependency updates.

@abretonc7s
abretonc7s marked this pull request as ready for review September 21, 2026 13:09
@abretonc7s
abretonc7s enabled auto-merge (squash) September 23, 2026 00:10
@abretonc7s
abretonc7s merged commit 20978da into main Sep 23, 2026
27 of 28 checks passed
@abretonc7s
abretonc7s deleted the feat/explicit-skill-invocation branch September 23, 2026 01:15
abretonc7s added a commit that referenced this pull request Sep 23, 2026
Distribute the generated Perps static review checklist for Mobile,
Extension and Core.

- This PR contains only Perps review content, its generator, parity
tests and documentation.
- It is independent of installer PR #174. The skill can be invoked
directly from the checked out source; installer explicit-only controls
are a separate follow-up in #174.
- `--analyzer-out <path> --client <client>` optionally exports inlined
context for an existing PR risk analyzer. It does not enable an analyzer
integration.
- Validation: the repository test suite passes, including per-client
parity and install/reinstall checks.

Farmslot validation ran the Core template with Astra medium, registered
all 30 child checklist rows, saved three actionable findings plus line
comments, and posted the selected review to MetaMask/core#10294 after
the human gate.
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.

2 participants