Skip to content

feat: switch to slash command syntax and add skill execution#235

Merged
avoidwork merged 5 commits into
mainfrom
feat/slash-command-syntax
Jun 14, 2026
Merged

feat: switch to slash command syntax and add skill execution#235
avoidwork merged 5 commits into
mainfrom
feat/slash-command-syntax

Conversation

@avoidwork

@avoidwork avoidwork commented Jun 14, 2026

Copy link
Copy Markdown
Owner

Description

Switch all harness commands from : to / prefix (:help → /help) and add skill execution via /skillName [args]. The command parser now tries registered commands first, then falls back to the skill registry for unmatched slash-prefixed inputs. Skills load their SKILL.md body and stream through the provider with full tool-call and todo-status rendering.

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactor (no functional change)
  • Performance improvement
  • CI / build / tooling

Testing

  • All 1126 tests pass
  • Updated all existing command parser tests from : to / syntax
  • Added new skill execution tests:
    • Dispatches to _executeSkill when skill is in _skillList
    • Returns error when skill not in _skillList
    • Returns error when skill found but no _executeSkill
    • Registered commands take priority over skills
    • Skills shown in /help output

Coverage

  • 100% line coverage maintained

Checklist

  • npm run lint passes
  • Tests pass with 100% line coverage
  • No forbidden patterns used
  • Conventional Commit style applied

- Update commandParser to use '/' instead of ':' for command syntax
- Add skill execution via /skillName [args] — dispatches to skill registry
- Update banner help text to show '/' commands
- Update all tests to use new '/' syntax
- Add skill execution tests
- Exclude internal commands from listCommands()
@avoidwork avoidwork self-assigned this Jun 14, 2026
- Add skillBody field to skill execution result for clean separation
- Trigger dispatchProvider with skill body so agent sees the instructions
- Mirror handleChat streaming flow for skill responses
- Suppress status bar and chat echo for skill results
- Resolve config.test.js merge conflict
- Add Cron.uninstall() to cron_sync test setup
@avoidwork avoidwork changed the title feat: change harness commands from ':' to '/' prefix feat: switch to slash command syntax and add skill execution Jun 14, 2026
@avoidwork avoidwork merged commit e8a369c into main Jun 14, 2026
2 checks passed
@avoidwork avoidwork deleted the feat/slash-command-syntax branch June 14, 2026 19:46
@avoidwork avoidwork mentioned this pull request Jun 14, 2026
4 tasks
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.

1 participant