Skip to content

feat(wizard): --edit tools/endpoints sections (#26) - #255

Merged
Destynova2 merged 1 commit into
mainfrom
feat/wizard-edit-sections
Apr 21, 2026
Merged

feat(wizard): --edit tools/endpoints sections (#26)#255
Destynova2 merged 1 commit into
mainfrom
feat/wizard-edit-sections

Conversation

@Destynova2

Copy link
Copy Markdown
Contributor

Summary

  • Adds the `tools` section to `grob setup --edit`: re-runs the tool picker and rewrites `[presets].active` based on the new selection.
  • Updates the unknown-section error message and the startup tip to list the full set: providers, auth, budget, compliance, fallback, endpoints, tools.
  • CLI `--help` entry for `--edit` now shows the full list (endpoints was already implemented but missing from docs).

Test plan

  • Existing wizard unit tests still pass
  • `cargo clippy --lib --all-features -- -D warnings` clean
  • Manual: `grob setup --edit tools` on a fresh config

Audit item #26 (routine, N=3).

🤖 Generated with Claude Code

Fills the gaps in `grob setup --edit <section>`:

* Adds the `tools` section — re-runs `screen_tools` and rewrites
  `[presets].active` based on the new tool mix. A follow-up note
  reminds the user to reconfigure providers if the preset changed.
* Updates the unknown-section error message and the startup tip to
  list the full set: providers, auth, budget, compliance, fallback,
  endpoints, tools.
* CLI help (`grob setup --edit --help`) now shows the full list.

The `endpoints` section was already implemented but was missing
from the help text — that's fixed here.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Destynova2
Destynova2 enabled auto-merge (squash) April 21, 2026 21:27
@Destynova2
Destynova2 merged commit be1ac3e into main Apr 21, 2026
42 checks passed
@Destynova2
Destynova2 deleted the feat/wizard-edit-sections branch April 21, 2026 21:43
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