Skip to content

Refactor skill command UI boundary - #217

Merged
codeaholicguy merged 1 commit into
mainfrom
feature-skill-service-ui-boundary
Sep 9, 2026
Merged

Refactor skill command UI boundary#217
codeaholicguy merged 1 commit into
mainfrom
feature-skill-service-ui-boundary

Conversation

@codeaholicguy

Copy link
Copy Markdown
Owner

Summary

  • Move skill command implementation into commands/skill with an index entry point and render helper.
  • Remove terminal UI, spinners, and environment prompts from services/skill.
  • Return structured install/remove/rebuild results from skill services and render them in the command layer.
  • Update install and skill tests to assert service reports instead of service-side console output.

Validation

  • npx tsc -p packages/cli/tsconfig.json --noEmit
  • npm --workspace packages/cli test -- skill
  • npm --workspace packages/cli test -- install.service
  • npm run build
  • npm --workspace packages/cli test
  • pre-commit hook: repo lint + repo tests passed

Risk

  • Low: CLI output is now rendered from service result objects, so command behavior should remain equivalent while service tests no longer depend on terminal output.

@codeaholicguy
codeaholicguy merged commit dafaedb into main Sep 9, 2026
7 checks passed
@codeaholicguy
codeaholicguy deleted the feature-skill-service-ui-boundary branch September 9, 2026 13:13
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