docs: fix broken links, align standards with codebase conventions

Fix broken lifecycle.md links in contributing/concepts/ that pointed to
a non-existent file (should reference docs/concepts/lifecycle.md). Update
CLAUDE.md test file glob to match actual colocated test pattern, add
type-fest to tech stack. Align testing.md integration test directory
example with real codebase (test/integration/ not __tests__/). Add
undocumented Crust color to diagrams.md palette table and clarify
classDef vs inline style guidance for subgraphs.

Update changesets to cover output/prompts removal, utils fs export,
and CLI command refactoring.

Co-Authored-By: Claude <noreply@anthropic.com>

Author: zrosenbauer

.changeset/cli-refactor-commands.md

@@ -0,0 +1,5 @@
+---
+'@kidd-cli/cli': patch
+---
+
+Refactor CLI commands to use shared config helpers and validation utilities

.changeset/oauth-pkce-device-code.md

@@ -21,3 +21,7 @@ After:
 ```
 
 New `device-code` resolver added for headless/browserless environments (RFC 8628).
+
+**Breaking change:** Remove `lib/output` and `lib/prompts` sub-exports. The `Spinner` interface is now inlined in `context/types.ts` and prompts use `@clack/prompts` directly. Consumers importing from `@kidd-cli/core/lib/output` or `@kidd-cli/core/lib/prompts` must update to use `@clack/prompts` directly.
+
+**Breaking change:** Export `MiddlewareEnv` type from main entry point.

.changeset/utils-fs-export.md

@@ -0,0 +1,5 @@
+---
+'@kidd-cli/utils': patch
+---
+
+Add `fileExists` utility for checking file existence without throwing

CLAUDE.md

@@ -29,6 +29,7 @@ You are a strict functional programmer. You write pure, immutable, declarative T
 | [tsdown](https://tsdown.dev)                           | Bundler                 | [llms.txt](https://tsdown.dev/llms.txt) \| [llms-full.txt](https://tsdown.dev/llms-full.txt) |
 | [OXC](https://oxc.rs) (oxlint)                         | Linting                 | [llms.txt](https://oxc.rs/llms.txt)                                                          |
 | [Vitest](https://vitest.dev)                           | Testing                 | [GitHub](https://github.com/vitest-dev/vitest)                                               |
+| [type-fest](https://github.com/sindresorhus/type-fest) | Type utilities           | [GitHub](https://github.com/sindresorhus/type-fest)                                          |
 | [Changesets](https://github.com/changesets/changesets) | Versioning & publishing | [GitHub](https://github.com/changesets/changesets)                                           |
 
 ## Commands
@@ -55,7 +56,7 @@ pnpm test           # Run package tests
 - ESM only (`"type": "module"`)
 - Built with `tsdown` (`dts: true`, `format: 'esm'`, `clean: true`, `outDir: 'dist'`)
 - TypeScript: `target: ES2022`, `module: ESNext`, `moduleResolution: bundler`, `strict: true`, `isolatedDeclarations: true`
-- `vitest` for tests (test files in `test/**/*.test.ts`)
+- `vitest` for tests (unit tests colocated as `src/**/*.test.ts`, integration tests in `test/integration/*.test.ts`)
 - All public properties `readonly`
 - Config validated with Zod at boundaries
 - Explicit return types on all exported functions (required by `isolatedDeclarations`)

contributing/concepts/architecture.md

@@ -254,7 +254,7 @@ middleware(async (ctx, next) => {
 })
 ```
 
-Middleware follows an onion model: root middleware (from `cli()`) wraps command middleware (from `command()`), which wraps the handler. Each middleware calls `next()` to pass control inward. Data flows between middleware and handlers via `ctx.store`. See [Lifecycle](./lifecycle.md) for the full execution model.
+Middleware follows an onion model: root middleware (from `cli()`) wraps command middleware (from `command()`), which wraps the handler. Each middleware calls `next()` to pass control inward. Data flows between middleware and handlers via `ctx.store`. See [Lifecycle](../../docs/concepts/lifecycle.md) for the full execution model.
 
 ## Autoloader
 
@@ -323,7 +323,7 @@ if (error) return [error, null]
 ## References
 
 - [CLI](./cli.md)
-- [Lifecycle](./lifecycle.md)
+- [Lifecycle](../../docs/concepts/lifecycle.md)
 - [Coding Style](../standards/typescript/coding-style.md)
 - [Design Patterns](../standards/typescript/design-patterns.md)
 - [Errors](../standards/typescript/errors.md)

contributing/concepts/cli.md

@@ -192,7 +192,7 @@ root middleware end
 - A middleware can short-circuit by not calling `next()`
 - Data flows between middleware and handlers via `ctx.store`
 
-See [Lifecycle](./lifecycle.md) for the full execution model.
+See [Lifecycle](../../docs/concepts/lifecycle.md) for the full execution model.
 
 ## Autoloading
 
@@ -272,6 +272,6 @@ See the [Adding a CLI Command](../guides/adding-a-cli-command.md) guide for a st
 ## References
 
 - [Architecture](./architecture.md)
-- [Lifecycle](./lifecycle.md)
+- [Lifecycle](../../docs/concepts/lifecycle.md)
 - [Errors](../standards/typescript/errors.md)
 - [Design Patterns](../standards/typescript/design-patterns.md)

contributing/standards/documentation/diagrams.md

@@ -31,6 +31,7 @@ Based on [Catppuccin Mocha](https://catppuccin.com/palette).
 
 | Name     | Hex       | Usage                                  |
 | -------- | --------- | -------------------------------------- |
+| Crust    | `#181825` | Subgraph fill (app boundary, groups)   |
 | Base     | `#1e1e2e` | Background, cluster backgrounds        |
 | Surface0 | `#313244` | Node fill (primary)                    |
 | Surface1 | `#45475a` | Node fill (secondary), cluster borders |
@@ -136,7 +137,7 @@ Apply with the `class` directive -- never use `classDef default`.
 
 - Max 10-15 nodes per diagram
 - Prefer LR (left-to-right) or TB for vertical flows
-- Use `classDef` + `class` for styling -- never inline `style` on individual nodes
+- Use `classDef` + `class` for styling nodes -- never inline `style` on individual nodes (subgraphs require inline `style` since `classDef` does not apply to them)
 - Use semantic class names (`external`, `core`, `agent`) -- never `classDef default`
 - Group related nodes with subgraphs
 - Add a legend when using both solid and dashed lines

contributing/standards/typescript/testing.md

@@ -26,8 +26,9 @@ src/
 ├── runtime/
 │   ├── executor.ts
 │   └── executor.test.ts
-└── __tests__/
-    └── integration.test.ts
+test/
+└── integration/
+    └── auth-oauth-pkce.test.ts
 ```
 
 ### Write Clear Test Cases

package.json

@@ -14,7 +14,7 @@
     "build": "turbo run build --filter='./packages/*'",
     "dev": "turbo run dev",
     "test": "turbo run test --filter='./packages/*'",
-    "test:coverage": "turbo run build --filter='./packages/*' && vitest run --coverage",
+    "test:coverage": "turbo run test:coverage --filter='./packages/*'",
     "test:watch": "turbo run build --filter='./packages/*' && vitest",
     "typecheck": "turbo run typecheck --filter='./packages/*'",
     "check": "turbo run typecheck --filter='./packages/*' && pnpm lint && pnpm format",

packages/bundler/package.json

@@ -24,6 +24,7 @@
     "lint": "oxlint --ignore-pattern node_modules",
     "lint:fix": "oxlint --fix --ignore-pattern node_modules",
     "test": "vitest run",
+    "test:coverage": "vitest run --coverage",
     "test:watch": "vitest"
   },
   "dependencies": {