Sync Smith skill during config init#46
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bb054b9b2b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2d08222f47
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9635cdc383
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Summary
smith.skillsync helper that links~/.agents/skills/smithto the bundled skill, preferring Homebrew's stableopt/smithpath when available.smith config initand addsmith skill sync/smith skill statusfor existing users.brew install faustodavid/tap/smithfollowed bysmith config init, with a concise upgrade note.Validation
uv run --extra dev pytest tests/unit/test_skill.py tests/contract/test_cli_contracts.py::test_handle_config_init_creates_empty_config_file tests/contract/test_cli_contracts.py::test_handle_config_init_rejects_existing_config_file tests/contract/test_cli_contracts.py::test_handle_skill_sync_reports_current_link tests/unit/test_config.py::test_load_config_rejects_reserved_top_level_remote_names tests/unit/test_onboarding.py::TestValidateRemoteName::test_reserved_name_rejected -quv run --extra dev ruff check src/smith/skill.py src/smith/cli/handlers.py src/smith/cli/parser.py src/smith/formatting.py tests/unit/test_skill.py tests/contract/test_cli_contracts.py tests/unit/test_config.py tests/unit/test_onboarding.pysmith config init,smith skill status, symlink verificationsmith config init --format jsonmake checkruby -c Formula/smith.rbbrew style ./Formula/smith.rbgit diff --checkbrew audit --formula --strictandbrew audit --formula --newRelease note
This updates source behavior. The public Homebrew install will get
smith config initskill syncing after the next Smith release/tag and tap formula bump. The formula helper change is ready for that bump and still keepssmith-install-skillas a compatibility path.