Skip to content
This repository was archived by the owner on May 15, 2026. It is now read-only.

feat: expose skills as slash commands with skill fallback execution - #11834

Merged
hannesrudolph merged 2 commits into
mainfrom
feat/skill-slash-command-fallback
Mar 2, 2026
Merged

feat: expose skills as slash commands with skill fallback execution#11834
hannesrudolph merged 2 commits into
mainfrom
feat/skill-slash-command-fallback

Conversation

@hannesrudolph

@hannesrudolphhannesrudolph commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Added autogenerated discovery of skill-backed slash commands so available skills are exposed in slash command flows.
  • Implemented invocation fallback behavior in mention/tool execution paths so skill commands can resolve and execute when direct slash command handlers are absent.
  • Enforced command precedence so real slash commands continue to win before skill fallback execution is attempted.

Validation

  • cd src && pnpm -s check-types ✅ PASS
  • cd src && npx vitest run __tests__/command-mentions.spec.ts core/mentions/__tests__/index.spec.ts core/mentions/__tests__/processUserContentMentions.spec.ts core/tools/__tests__/runSlashCommandTool.spec.ts core/webview/__tests__/webviewMessageHandler.spec.ts ✅ PASS

Interactively review PR in Roo Code Cloud

@dosubotdosubotBot added size:XL This PR changes 500-999 lines, ignoring generated files. Enhancement New feature or request labels Mar 2, 2026
@hannesrudolph

Copy link
Copy Markdown
ContributorAuthor

@roomote run the pr review on this

@ghost

ghost commented Mar 2, 2026

Copy link
Copy Markdown

FixarooClockSee task

Reviewed the PR and addressed two issues: added missing unit tests for skillInvocation.ts, fixed inconsistent optional chaining on getSkillsManager in RunSlashCommandTool.ts, and added getSkillsManager to the default test mock so command-not-found tests work without relying on ?. method calls. All checks pass (type-check, lint, 89 tests).

View commit | Revert commit

@dosubotdosubotBot added the lgtm This PR has been approved by a maintainer label Mar 2, 2026

@hannesrudolphhannesrudolph left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Completed review of this PR. No actionable issues were identified within the changed scope.

@hannesrudolph
hannesrudolph merged commit af1e12c into mainMar 2, 2026
17 of 20 checks passed
@hannesrudolph
hannesrudolph deleted the feat/skill-slash-command-fallback branch March 2, 2026 22:10
@github-project-automationgithub-project-automationBot moved this from New to Done in Roo Code RoadmapMar 2, 2026
@github-actionsgithub-actionsBot mentioned this pull request Mar 4, 2026
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

EnhancementNew feature or requestlgtmThis PR has been approved by a maintainersize:XLThis PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@hannesrudolph@daniel-lxs@roomote