feature: settings manager

.changeset/cute-ideas-show.md

@@ -0,0 +1,5 @@
+---
+"wtc": minor
+---
+
+feature: adds a state manager and config manager

.github/workflows/ci.yml

@@ -4,6 +4,9 @@ on:
   pull_request:
     branches: [main]
 
+permissions:
+  contents: read
+
 jobs:
   quality:
     name: Lint, Typecheck & Test

AGENTS.md

@@ -1,45 +1,37 @@
-# wtc — AI Agent Instructions
-
 ## Overview
 
-`wtc` is a CLI/TUI tool for managing GitHub repos, AWS Amplify projects, and Teamwork tasks. Built with Bun + OpenTUI (core functional API, no JSX framework).
+`wtc` is a CLI/TUI tool for managing GitHub repos, AWS Amplify projects, and Teamwork tasks. Built with Bun + OpenTUI Solid.
 
 ## Important Files
 
 Before making changes, read these files for context:
 
 - **`plans/PLAN.md`** — Full project roadmap, architecture, conventions, all phases
-- **`plans/MVP.md`** — Current MVP scope and deliverables
-- **`package.json`** — Scripts and dependencies
-- **`tsconfig.json`** — TypeScript configuration (strict mode)
 
 ## Tech Stack
 
-| Concern           | Choice                            |
-| ----------------- | --------------------------------- |
-| Runtime           | Bun                               |
-| TUI               | `@opentui/solid` + `solid-js`     |
-| CLI parser        | yargs                             |
-| Linter            | oxlint                            |
-| Formatter         | oxfmt                             |
-| Test runner       | bun test                          |
-| Pre-commit        | husky + lint-staged               |
-| Release versions  | Changesets                        |
-| Encryption        | Web Crypto (AES-256-GCM + PBKDF2) |
-| Config validation | zod                               |
+| Concern           | Choice                        |
+| ----------------- | ----------------------------- |
+| Runtime           | Bun                           |
+| TUI               | `@opentui/solid` + `solid-js` |
+| CLI parser        | yargs                         |
+| Linter            | oxlint                        |
+| Formatter         | oxfmt                         |
+| Test runner       | bun test                      |
+| Pre-commit        | husky + lint-staged           |
+| Release versions  | Changesets                    |
+| Config validation | zod                           |
 
 ## Conventions
 
-### Branching
-
-```
-(feature|fix|chore)/TASK-XXXXX
-```
-
 ### Commits
 
 Conventional commits: `feat:`, `fix:`, `chore:`, `docs:`, `test:`, `refactor:`
 
+- Always check-in with user first before making the commit so they can check diff and ask for changes
+- Make small commits so it's easy to see progress and revert changes
+- Stop at a breakpoint before every commit and before continuing to the next main step. The user will inspect the diff themselves; do not commit or continue until they explicitly approve.
+
 ### TypeScript
 
 - strict mode, no `any`, no `as` casts where avoidable
@@ -50,39 +42,50 @@ Conventional commits: `feat:`, `fix:`, `chore:`, `docs:`, `test:`, `refactor:`
 - Functions: `camelCase`
 - Tests: `*.test.ts` in `tests/` mirroring `src/` structure
 
+### File Organization / Helper Scope
+
+- Keep helpers scoped to the smallest place that needs them.
+- Put values/functions in `consts.ts` only when they are shared across modules or own environment-variable behavior.
+- Keep filenames, local paths, and one-module constants inside the module that uses them.
+- Do not export helpers only for tests unless they represent meaningful domain behavior.
+- Do not create helper functions for one-use expressions.
+- Prefer inline local constants over private one-line functions.
+- Manager modules should contain domain behavior, not generic wrappers around simple file reads/writes.
+- If a helper exists only to make a test easier, reconsider the test or test higher-level behavior instead.
+
+Examples:
+
+- `getCacheDir()` belongs in `src/state/consts.ts` because it is shared and owns `WTC_CACHE_DIR`.
+- `getUserConfigDir()` belongs in `src/config/consts.ts` because it owns `WTC_CONFIG_DIR`.
+- `STATE_FILE = "tui-state.json"` belongs in `src/state/manager.ts` because only the state manager uses it.
+- `getStatePath()` should not exist if it only appends `STATE_FILE` to `getCacheDir()` in one module.
+- `formatUserConfig()` is valid because Bun's YAML parser does not preserve comments, so config saves need explicit commented formatting.
+
 ## Commands
 
 ```bash
-bun run dev          # Watch mode (src/index.ts)
 bun run lint         # oxlint
 bun run fmt          # oxfmt (write)
 bun run fmt:check    # oxfmt (check only)
 bun run check        # tsc --noEmit
 bun test             # Run all tests
 bun run build        # Build standalone binary
-bun run changeset    # Add a release changeset
 ```
 
 ## Code Quality Rules
 
-1. All TypeScript must compile with `tsc --noEmit` (strict mode)
-2. All code must pass `oxlint` with no errors
-3. All code must pass `oxfmt --check` (formatting)
-4. All tests must pass (`bun test`)
-5. Pre-commit hook runs lint-staged on staged `.ts` files
+- [ ] `bun run fmt` passes
+- [ ] `bun run lint` passes
+- [ ] `bun run check` passes (tsc)
+- [ ] `bun test` passes
+- [ ] `bun run build` passes
+- [ ] No `any` types, no default exports
+
+**NEVER** run the binary compiled files.
 
 ## Testing Philosophy
 
 - **Test logic, not layout.** Tests should cover pure functions, business logic, API clients, and utility modules. Do not test TUI rendering (box position, text content, styling against OpenTUI components) — that's the framework's job.
 - **No mocks of the OpenTUI renderer.** Module-mocking `@opentui/core` (Box, Text, createCliRenderer, t) adds fragility and tests the mock, not the real behavior. If a function delegates to OpenTUI, trust the function; test what it computes, not how it renders.
 - **Prefer real calls over mocks.** For utilities like `checkForUpdate`, test with a real (or near-real) API surface — mock at the HTTP layer only if necessary.
 - **Integration tests for CLI flows** are acceptable only when they invoke the binary as a subprocess (e.g., `wtc --version`), but are deferred until Phase 2+.
-
-## Pull Request Checklist
-
-- [ ] `bun run lint` passes
-- [ ] `bun run fmt:check` passes
-- [ ] `bun run check` passes (tsc)
-- [ ] `bun test` passes
-- [ ] New code has tests
-- [ ] No `any` types, no default exports

CONTRIBUTING.md

@@ -162,3 +162,13 @@ Maintainers create releases by merging the Changesets version PR. Tag creation,
 ## Before Committing
 
 Pre-commit hooks will automatically run linting and formatting on staged files.
+
+## Development Notes
+
+### Solid JSX at dev time
+
+`bun run dev` uses `--preload=@opentui/solid/preload` to register a Bun runtime plugin that transforms Solid JSX on-the-fly. Do **not** move this preload to `bunfig.toml` — compiled binaries also read that config and crash because `@opentui/solid/preload` calls `registerBunPlugin()`, a Bun runtime API unavailable in standalone executables.
+
+### `--watch` is broken in Bun 1.3.x
+
+The `--watch` (and `--hot`) flags do not restart the process on file changes in this Bun release. This is a known Bun bug, not caused by the preload. Manual restart is required until a fix lands.

README.md

@@ -35,14 +35,14 @@ See [CONTRIBUTING.md](CONTRIBUTING.md) for development setup instructions.
 ## Project Status
 
 - ✅ **Phase 1 (MVP):** Hello World dashboard with tooling, CI/CD, and distribution
-- **Phase 2:** Incorporate SolidJS
+- ✅ **Phase 2:** Incorporate SolidJS
 - **Phase 3:** GitHub repo creation
 - **Phase 4:** AWS Amplify project bootstrapping
 - **Phase 5:** Teamwork API integration
 - **Phase 6:** Full TUI dashboard with navigation
 - **Phase 7:** Distribution polish
 
-See [plans/PLAN.md](plans/PLAN.md) and [plans/MVP.md](plans/MVP.md) for the full roadmap.
+See [plans/PLAN.md](plans/PLAN.md) for the full roadmap.
 
 ## License