feat(cli): replace --skill flag with skill command#618
Merged
Conversation
… flag Promote running a single context-mill skill from a hidden-ish `--skill=<id>` flag on the default command to a first-class `wizard skill <skill-name>` subcommand. The skill id is still fetched from context-mill's release at runtime (same `runSkillMode` engine), so no context-mill change is required. `--ci` still runs it headlessly: `wizard skill <name> --ci`. Removes the `--skill` flag, its `check` rules, and its handler branches from the default command; updates the CLI parsing tests accordingly. Generated-By: PostHog Code Task-Id: 4debb1ba-a98a-494f-ba44-29df9fd1d87b
🧙 Wizard CIRun the Wizard CI and test your changes against wizard-workbench example apps by replying with a GitHub comment using one of the following commands: Test all apps:
Test all apps in a directory: Test an individual app: Results will be posted here when complete. |
… bridge Addresses two review gaps on the `wizard skill` command: - Re-add the empty-id guard the old `--skill` flag had: `wizard skill ""` now fails with a friendly "skill needs a skill name" instead of running a skill with no id. - Add handler-level coverage that the `<skill-name>` positional is bridged onto runSkillMode's skill id (the parse tests stubbed the handler, so this logic was untested), plus the whitespace-trim behavior. Generated-By: PostHog Code Task-Id: 4debb1ba-a98a-494f-ba44-29df9fd1d87b
sarahxsanders
added a commit
that referenced
this pull request
Jun 12, 2026
Resolves seven conflicts from main, which shipped #618 (`wizard skill` command) and #630 (source-maps → upload-source-maps rename) while this branch reworked the audit family and retired audit-3000. Resolutions: - skill.ts: hybrid. Keeps main's `wizard skill <skill-name>` contract (positional + runSkillMode) and adds `wizard skill list` as a child command that fetches the live skill-menu.json so users have an in-CLI way to discover skill ids. Drops the search subcommand from our pre-merge version — easy to add later if needed. - bin.ts, basic-integration/index.ts: take main's side. The `--skill` flag is gone for good (it's a real command now). - upload-sourcemaps.ts + error-tracking-upload-source-maps/index.ts: take main's renamed file and the matching `command: 'upload-source-maps'` config update. Drops our orphan `src/commands/source-maps.ts`. - audit-3000 modified files: kept deleted. We retired the program. - programs-cli.test.ts: combined imports from both sides. All tests from both sides pass after the merge. - wizard.test.ts: chased the rename through to its sourceMapsCommand import. Verification: pnpm typecheck green; pnpm jest 837/837 passing across 54 suites.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
instead of running a single skill using
--skill=<id>flag, promote it to awizard skill <skill-name>subcommandchanges
wizard skill <skill-name>command: a thin wrapper over the existingrunSkillModeengine, so behavior is unchanged--skillflag from the default command, along with itscheckrules and handler branches
--skillflag tests, addedcoverage for the new command'
testing
pnpm build && pnpm test && pnpm lintall green. Manually confirmedwizard skill <name>registers in--help