Skip to content

Refactor skill installation helpers - #968

Open
jana-selva wants to merge 3 commits into
feature/963-multi-skill-supportfrom
feature/967-refactor-skill-installation
Open

jana-selva wants to merge 3 commits into
feature/963-multi-skill-supportfrom
feature/967-refactor-skill-installation

Conversation

@jana-selva

Copy link
Copy Markdown
Contributor

Closes #967

Stacked on PR #966 (feature/963-multi-skill-support).

This follow-up incorporates the remaining review context from PR #966 and the api-contract-audit skill review:

Validation:

  • 38 focused skill tests passed.
  • skills:check passed.
  • Project-template skill integration test passed on retry.
  • Ruff and git diff --check passed.

@ArBridgeman
ArBridgeman force-pushed the feature/967-refactor-skill-installation branch from 79f1c22 to ded951f Compare October 5, 2026 11:10
@sonarqubecloud

sonarqubecloud Bot commented Oct 5, 2026

Copy link
Copy Markdown

This branch is waiting to be deployed

1 waiting deployment
manual-approval — ded951ff Waiting Oct 5, 2026 by ArBridgeman via Merge Gate / Extension / Approve Running Slow Tests? #2848
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.

Refactor install_skill into reusable filesystem helpers

2 participants