Skip to content

Commit dc46982

Browse files
committed
refactor: split oversized files and remove dead code
- Split brew.go → brew_install.go (parallel install logic extracted) - Split snapshot.go → snapshot_import.go + snapshot_publish.go - Split selector.go → selector_view.go (rendering separated from model) - Remove unused testutil helpers (MockExecCommand, CommandResult, AssertCommandSuccess/Failure, GetInstalledBrewPackages) - Remove dead stubBrewBinary from shell_extra_test.go - Fix root_test.go: update cfg → installCfg after prior refactor - Drop stale project.go reference from config package doc
1 parent 0739892 commit dc46982

17 files changed

Lines changed: 1717 additions & 1769 deletions

CLAUDE.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,7 @@ cmd/openboot/ # main.go → cli.Execute()
4242
internal/
4343
auth/ # OAuth-like login, token in ~/.openboot/auth.json (0600)
4444
brew/ # Homebrew ops, parallel workers (4 max), retry, uninstall
45-
cleaner/ # Diff current vs desired, remove extras
46-
cli/ # Cobra cmds: install, snapshot, push, pull, config, login/logout, version, deprecated
45+
cli/ # Cobra cmds: install, snapshot, login, logout, version
4746
config/ # Package catalog + presets + remote fetch (embed fallback in data/)
4847
diff/ # Pure-logic system-vs-config comparison
4948
dotfiles/ # Clone + stow with .openboot.bak backup

README.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -166,10 +166,6 @@ openboot snapshot --local # Save to ~/.openboot/snapshot.json
166166
openboot snapshot --publish # Upload to openboot.dev
167167
openboot snapshot --import FILE # Restore from a snapshot file
168168

169-
openboot config list # List your openboot.dev configs
170-
openboot config edit # Open a config in the browser
171-
openboot config delete SLUG # Remove a config
172-
173169
openboot login / logout # openboot.dev auth
174170
openboot version # Print version
175171
```

docs/SPEC.md

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ Two directions, two verbs:
1919
- **`install`** — add only. Never uninstalls.
2020
- **`snapshot`** — output only. Captures current state.
2121

22-
Everything else is cloud-config CRUD (`config list/edit/delete`) or auth/version.
22+
Everything else is auth/version infrastructure.
2323

2424
---
2525

@@ -32,22 +32,14 @@ Everything else is cloud-config CRUD (`config list/edit/delete`) or auth/version
3232
| `openboot install [source]` | Add packages / settings from source to this Mac. |
3333
| `openboot snapshot` | Capture current state (local, cloud, or stdout). |
3434

35-
### Cloud config CRUD
36-
37-
| Command | Purpose |
38-
|---------|---------|
39-
| `openboot config list` | List your configs on openboot.dev. |
40-
| `openboot config edit [slug]` | Open a config in the browser for editing. |
41-
| `openboot config delete <slug>` | Delete a config. |
42-
4335
### Infrastructure
4436

4537
| Command | Purpose |
4638
|---------|---------|
4739
| `openboot login` / `logout` | Authenticate with openboot.dev. |
4840
| `openboot version` | Print version. Self-update happens on launch automatically. |
4941

50-
**Total: 8 commands.** (plus `help`, `completion` from Cobra)
42+
**Total: 5 commands.** (plus `help`, `completion` from Cobra)
5143

5244
---
5345

@@ -163,9 +155,9 @@ preset ← Config-only
163155
| `clean` | **No replacement.** OpenBoot no longer manages removals. |
164156
| `log` | **No replacement.** Version history dropped. |
165157
| `restore` | **No replacement.** Version history dropped. |
166-
| `list` | `config list` |
167-
| `edit` | `config edit` |
168-
| `delete` | `config delete` |
158+
| `list` | **No replacement.** Manage configs at openboot.dev directly. |
159+
| `edit` | **No replacement.** Manage configs at openboot.dev directly. |
160+
| `delete` | **No replacement.** Manage configs at openboot.dev directly. |
169161
| `init` | **No replacement.** Project deps are each ecosystem's own job (npm/pip/go/cargo). |
170162
| `setup-agent` | **No replacement.** Existed only to service `init`. |
171163
| `doctor` | **No replacement.** Use `brew doctor` and `git config --list` directly. |

0 commit comments

Comments
 (0)