Skip to content

Add bbox config init command#57

Merged
JAORMX merged 2 commits intomainfrom
config-init
Mar 18, 2026
Merged

Add bbox config init command#57
JAORMX merged 2 commits intomainfrom
config-init

Conversation

@JAORMX
Copy link
Copy Markdown
Contributor

@JAORMX JAORMX commented Mar 17, 2026

Summary

  • Add bbox config init subcommand that generates ~/.config/broodbox/config.yaml with every configuration option documented as YAML comments
  • All values are commented out so the file is safe to ship as-is and parses to a zero-value config
  • Respects --config for a custom path and --force to overwrite an existing file
  • Update CLAUDE.md with guidance to keep the template in sync when config fields change

Test plan

  • Unit tests cover: file creation, parent directory creation, existing-file guard, --force overwrite, template parses to zero-value Config, all config sections present
  • Manual: task build && ./bin/bbox config init writes the file, inspect contents
  • Manual: re-run without --force → errors; re-run with --force → overwrites

🤖 Generated with Claude Code

jhrozek
jhrozek previously approved these changes Mar 17, 2026
Copy link
Copy Markdown
Contributor

@jhrozek jhrozek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

two nits inline non-blocking plus #62 filed for a follow-up

JAORMX and others added 2 commits March 18, 2026 09:47
Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>
- Add os.Chmod after WriteFile for consistent 0600 on --force overwrite
- Update memory descriptions for ByteSize type ("4g", "512m")
- Narrow per-agent MCP template to enabled + authz only (PR #63)
- Add authz/config subsection checks to template test

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@JAORMX JAORMX merged commit 67c88d1 into main Mar 18, 2026
7 checks passed
@JAORMX JAORMX deleted the config-init branch March 18, 2026 12:47
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.

2 participants