@dizzlkheinz/ynab-mcpb 0.16.0 → 0.17.0

package/docs/plans/2025-11-21-fix-transaction-cached-property.md

@@ -1,362 +0,0 @@
-# Fix Missing `cached` Property in Large Transaction Responses
-
-> **For Claude:** REQUIRED SUB-SKILL: Use superpowers:executing-plans to implement this plan task-by-task.
-
-**Goal:** Fix GitHub Action test failure by adding the missing `cached` property to large transaction list responses.
-
-**Architecture:** The `handleListTransactions` function in `transactionTools.ts` has two response paths: normal (lines 815+) and large response (lines 788-812). The large response path is missing the `cached` and `cache_info` properties that the normal path includes, causing test assertion failures when transaction data exceeds 90KB.
-
-**Tech Stack:** TypeScript, Vitest, YNAB API integration tests
-
----
-
-## Background
-
-**Current Issue:**
-- GitHub Action failing: `src/tools/__tests__/accountTools.delta.integration.test.ts:94`
-- Error: `expected undefined to be false // Object.is equality`
-- Test code: `expect(firstPayload.cached).toBe(false);`
-
-**Root Cause:**
-File `src/tools/transactionTools.ts` has two response code paths:
-1. **Large response path** (lines 788-812): When transactions > 90KB, returns preview + summary
-2. **Normal path** (lines 815+): Returns full transaction list
-
-The large response path returns an object WITHOUT the `cached` property.
-The normal path returns an object WITH `cached: cacheHit` and `cache_info`.
-
-**Test accounts with many transactions trigger the large response path, causing `cached` to be undefined.**
-
----
-
-## Task 1: Add Unit Test Coverage for Large Response Path
-
-**Files:**
-- Read: `src/tools/__tests__/transactionTools.test.ts`
-- Modify: `src/tools/__tests__/transactionTools.test.ts` (add test after existing tests)
-
-**Step 1: Read the existing test file to understand patterns**
-
-```bash
-cat src/tools/__tests__/transactionTools.test.ts | head -100
-```
-
-Expected: See test structure, mocking patterns, imports
-
-**Step 2: Write failing test for large response cached property**
-
-Add this test to `src/tools/__tests__/transactionTools.test.ts` in the appropriate describe block:
-
-```typescript
-it('should include cached property in large response path', async () => {
-  // Create large transaction list (> 90KB)
-  const largeTransactionList: ynab.TransactionDetail[] = [];
-  for (let i = 0; i < 5000; i++) {
-    largeTransactionList.push({
-      id: `transaction-${i}`,
-      date: '2025-01-01',
-      amount: -10000,
-      memo: 'Test transaction with long memo to increase size '.repeat(10),
-      cleared: 'cleared',
-      approved: true,
-      flag_color: null,
-      account_id: 'test-account',
-      payee_id: null,
-      category_id: null,
-      transfer_account_id: null,
-      transfer_transaction_id: null,
-      matched_transaction_id: null,
-      import_id: null,
-      import_payee_name: null,
-      import_payee_name_original: null,
-      debt_transaction_type: null,
-      deleted: false,
-      account_name: 'Test Account',
-      payee_name: 'Test Payee',
-      category_name: 'Test Category',
-      subtransactions: [],
-    } as ynab.TransactionDetail);
-  }
-
-  const mockDeltaFetcher = {
-    fetchTransactionsByAccount: vi.fn().mockResolvedValue({
-      data: largeTransactionList,
-      wasCached: false,
-      usedDelta: false,
-    }),
-  } as unknown as DeltaFetcher;
-
-  const result = await handleListTransactions(mockYnabAPI, mockDeltaFetcher, {
-    budget_id: 'test-budget',
-    account_id: 'test-account',
-  });
-
-  const content = result.content?.[0];
-  expect(content).toBeDefined();
-  expect(content?.type).toBe('text');
-
-  const parsedResponse = JSON.parse(content!.text);
-
-  // Should have cached property even in large response path
-  expect(parsedResponse.cached).toBeDefined();
-  expect(parsedResponse.cached).toBe(false);
-  expect(parsedResponse.cache_info).toBeDefined();
-});
-```
-
-**Step 3: Run the test to verify it fails**
-
-```bash
-npm run test:unit -- src/tools/__tests__/transactionTools.test.ts -t "should include cached property in large response path"
-```
-
-Expected: FAIL with error about `cached` being undefined
-
-**Step 4: Commit the failing test**
-
-```bash
-git add src/tools/__tests__/transactionTools.test.ts
-git commit -m "test: add failing test for cached property in large transaction responses"
-```
-
----
-
-## Task 2: Fix Large Response Path to Include Cached Property
-
-**Files:**
-- Modify: `src/tools/transactionTools.ts:788-812`
-
-**Step 1: Read the current large response code**
-
-```bash
-cat src/tools/transactionTools.ts | sed -n '788,812p'
-```
-
-Expected: See the current implementation missing `cached` and `cache_info`
-
-**Step 2: Add cached properties to large response**
-
-In `src/tools/transactionTools.ts`, replace lines 788-812 with:
-
-```typescript
-      if (estimatedSize > sizeLimit) {
-        // Return summary and suggest export
-        const preview = transactions.slice(0, 50);
-        return {
-          content: [
-            {
-              type: 'text',
-              text: responseFormatter.format({
-                message: `Found ${transactions.length} transactions (${Math.round(estimatedSize / 1024)}KB). Too large to display all.`,
-                suggestion: "Use 'export_transactions' tool to save all transactions to a file.",
-                showing: `First ${preview.length} transactions:`,
-                total_count: transactions.length,
-                estimated_size_kb: Math.round(estimatedSize / 1024),
-                cached: cacheHit,
-                cache_info: cacheHit
-                  ? `Data retrieved from cache for improved performance${usedDelta ? ' (delta merge applied)' : ''}`
-                  : 'Fresh data retrieved from YNAB API',
-                preview_transactions: preview.map((transaction) => ({
-                  id: transaction.id,
-                  date: transaction.date,
-                  amount: milliunitsToAmount(transaction.amount),
-                  memo: transaction.memo,
-                  payee_name: transaction.payee_name,
-                  category_name: transaction.category_name,
-                })),
-              }),
-            },
-          ],
-        };
-      }
-```
-
-**Changes:**
-- Added `cached: cacheHit,` after `estimated_size_kb`
-- Added `cache_info` with same pattern as normal response path
-
-**Step 3: Run unit tests to verify fix**
-
-```bash
-npm run test:unit -- src/tools/__tests__/transactionTools.test.ts -t "should include cached property in large response path"
-```
-
-Expected: PASS
-
-**Step 4: Run all transaction tool tests**
-
-```bash
-npm run test:unit -- src/tools/__tests__/transactionTools.test.ts
-```
-
-Expected: All tests PASS
-
-**Step 5: Commit the fix**
-
-```bash
-git add src/tools/transactionTools.ts
-git commit -m "fix: add cached property to large transaction response path
-
-- Large responses (>90KB) now include cached and cache_info properties
-- Maintains consistency with normal response path
-- Fixes test failure in delta integration tests"
-```
-
----
-
-## Task 3: Verify Integration Test Now Passes
-
-**Files:**
-- Test: `src/tools/__tests__/accountTools.delta.integration.test.ts`
-
-**Step 1: Run the failing integration test locally**
-
-```bash
-npm run test:integration -- src/tools/__tests__/accountTools.delta.integration.test.ts -t "reports delta usage for list_transactions after a change"
-```
-
-Expected: PASS (requires YNAB_ACCESS_TOKEN environment variable)
-
-Note: If you don't have a YNAB token or want to skip, this is acceptable - the GitHub Action will verify.
-
-**Step 2: Run type checking**
-
-```bash
-npm run type-check
-```
-
-Expected: No TypeScript errors
-
-**Step 3: Run all unit tests to ensure no regressions**
-
-```bash
-npm run test:unit
-```
-
-Expected: All tests PASS
-
-**Step 4: Commit verification checkpoint**
-
-```bash
-git add -A
-git commit -m "test: verify integration test passes with cached property fix"
-```
-
----
-
-## Task 4: Update CHANGELOG and Documentation
-
-**Files:**
-- Modify: `CHANGELOG.md` (add entry at top of Unreleased section)
-
-**Step 1: Add CHANGELOG entry**
-
-Add this entry to the `## [Unreleased]` section in `CHANGELOG.md`:
-
-```markdown
-### Fixed
-- Fixed missing `cached` property in large transaction list responses (>90KB)
-  - Large response path now includes `cached` and `cache_info` properties
-  - Maintains consistency with normal response path
-  - Resolves integration test failures when accounts have many transactions
-```
-
-**Step 2: Commit documentation**
-
-```bash
-git add CHANGELOG.md
-git commit -m "docs: add CHANGELOG entry for cached property fix"
-```
-
----
-
-## Task 5: Push and Verify GitHub Action
-
-**Files:**
-- Remote: GitHub Actions CI
-
-**Step 1: Push all commits to remote**
-
-```bash
-git push origin HEAD
-```
-
-Expected: Push successful
-
-**Step 2: Monitor GitHub Action**
-
-```bash
-gh run watch
-```
-
-Expected:
-- Job `unit-tests` should PASS
-- Job `integration-core` should PASS (was previously failing at accountTools.delta.integration.test.ts)
-
-**Step 3: Verify specific test that was failing**
-
-Check GitHub Action logs for:
-```
-✓ src/tools/__tests__/accountTools.delta.integration.test.ts > Delta-backed account tool handlers > reports delta usage for list_transactions after a change
-```
-
-Expected: Green checkmark, no assertion errors
-
-**Step 4: Create completion summary**
-
-Document verification results:
-```markdown
-## Verification Complete
-
-- ✅ Unit tests passing locally
-- ✅ Integration tests passing locally (if run)
-- ✅ Type checking passing
-- ✅ GitHub Actions CI passing
-- ✅ Specific failing test now passes
-
-The `cached` property is now consistently included in all transaction list responses.
-```
-
----
-
-## Testing Strategy
-
-**Unit Tests:**
-- New test verifies large response path includes `cached` property
-- Existing tests verify normal response path unchanged
-
-**Integration Tests:**
-- `accountTools.delta.integration.test.ts` verifies delta fetcher integration
-- Test creates real transaction, expects `cached: false` on first call
-- Was failing because large accounts returned `cached: undefined`
-
-**Manual Verification:**
-- GitHub Action will run full integration suite with real YNAB API
-- Throttled test runner prevents rate limit issues
-- Sequential execution ensures reliable results
-
----
-
-## Rate Limiting Context (For Reference)
-
-**Note:** The GitHub Action failure was NOT a rate limiting issue. The codebase already has excellent rate limiting:
-
-**Existing Rate Limit Infrastructure:**
-- `scripts/run-throttled-integration-tests.js` - Sequential test execution with request tracking
-- Client-side throttling (200 req/hour with 20 req buffer)
-- Request history pruning (60-minute sliding window)
-- Intelligent wait logic with min/max bounds
-- Per-test estimated API call counts
-
-**No changes needed to rate limiting.** The issue was purely a missing property in the response object.
-
----
-
-## Success Criteria
-
-- ✅ Unit test passes for large response cached property
-- ✅ Integration test `accountTools.delta.integration.test.ts:94` passes
-- ✅ GitHub Action CI pipeline passes completely
-- ✅ No TypeScript errors
-- ✅ CHANGELOG updated
-- ✅ All commits follow conventional commit format

package/docs/plans/2025-11-21-reconciliation-error-handling.md

@@ -1,90 +0,0 @@
-# Reconciliation Error Handling Implementation Plan
-
-> **For Claude:** REQUIRED SUB-SKILL: Use superpowers:executing-plans to implement this plan task-by-task.
-
-**Goal:** Fix reconciliation integration failures by properly surfacing YNAB API errors (invalid accounts, rate limits) so tests skip or fail appropriately instead of silently returning zero creations.
-
-**Architecture:** Add a small error-normalization layer inside the reconciliation executor to interpret YNAB SDK error payloads, propagate fatal conditions (429/invalid account) as errors, and include actionable reasons in action logs for rate-limit detection. Keep bulk/sequential creation flow intact while improving error transparency.
-
-**Tech Stack:** TypeScript, Vitest, YNAB SDK, Node 22+
-
-### Task 1: Normalize YNAB API errors
-
-**Files:**
-- Modify: `src/tools/reconciliation/executor.ts`
-- Test: `src/tools/reconciliation/__tests__/executor.test.ts`
-
-**Step 1: Add error normalization utilities**
-
-```ts
-// executor.ts (near helper section)
-interface NormalizedYnabError { status?: number; name?: string; message: string; detail?: string }
-function normalizeYnabError(err: unknown): NormalizedYnabError { /* parse err.error.id/detail/status, strings, Error */ }
-function shouldPropagateYnabError(err: NormalizedYnabError): boolean { return [401, 403, 404, 429, 500].includes(err.status ?? 0); }
-function attachStatus(err: NormalizedYnabError): Error { const e = new Error(err.message || err.detail || 'YNAB API error'); if (err.status) (e as any).status = err.status; if (err.name) e.name = err.name; return e; }
-```
-
-**Step 2: Use normalized errors in bulk chunk handling**
-
-```ts
-// executor.ts processBulkChunk catch
-const ynabErr = normalizeYnabError(error);
-if (shouldPropagateYnabError(ynabErr)) throw attachStatus(ynabErr);
-const reason = ynabErr.message;
-bulkOperationDetails.bulk_chunk_failures += 1;
-actions_taken.push({ type: 'bulk_create_fallback', reason: `Bulk chunk #${chunkIndex} failed (${reason})...` });
-```
-
-Expected: rate-limit or invalid-account errors now bubble; other bulk failures still fall back.
-
-### Task 2: Propagate fatal errors during sequential creation
-
-**Files:**
-- Modify: `src/tools/reconciliation/executor.ts`
-- Test: `src/tools/reconciliation/__tests__/executor.integration.test.ts`
-
-**Step 1: Update sequential catch block**
-
-```ts
-const ynabErr = normalizeYnabError(error);
-const failureReason = ynabErr.message;
-actions_taken.push({ type: 'create_transaction_failed', reason: ...failureReason... });
-if (shouldPropagateYnabError(ynabErr)) throw attachStatus(ynabErr);
-```
-
-Include status-aware message so `containsRateLimitFailure` sees 429 text.
-
-**Step 2: Ensure transaction failure counters reflect thrown errors**
-
-If fatal error occurs, increment `transaction_failures` before throw to preserve metrics.
-
-### Task 3: Cover new error handling with tests
-
-**Files:**
-- Modify: `src/tools/reconciliation/__tests__/executor.test.ts`
-- Modify: `src/tools/reconciliation/__tests__/executor.integration.test.ts` (if needed for assertions/fixtures)
-
-**Step 1: Add unit tests for non-Error YNAB payload**
-
-```ts
-it('propagates rate-limit error objects with status codes', async () => {
-  mockCreateTransactions.rejects({ error: { id: '429', detail: 'Too many requests' } });
-  await expect(executeReconciliation(...)).rejects.toMatchObject({ status: 429 });
-});
-```
-
-**Step 2: Add unit test for invalid account error propagation**
-
-Mock 404 payload and expect rejection; verify action reason contains detail when not thrown.
-
-**Step 3: Adjust integration expectation if fixtures rely on new messaging**
-
-Ensure `containsRateLimitFailure` continues to match updated reason text (no code changes anticipated).
-
-### Task 4: Verify fixes locally
-
-**Commands:**
-- `npx vitest run --project unit --runInBand src/tools/reconciliation/__tests__/executor.test.ts`
-- `npm run test:integration:core -- --testNamePattern="Reconciliation Executor - Bulk Create Integration"` (rerun the failing suite)
-
-Expected: unit tests pass; integration suite either passes or rate-limit skips instead of failing counts.

package/docs/plans/2025-11-21-v014-hardening.md

@@ -1,153 +0,0 @@
-# YNAB MCP v0.14 Hardening Implementation Plan
-
-> **For Claude:** REQUIRED SUB-SKILL: Use superpowers:executing-plans to implement this plan task-by-task.
-
-**Goal:** Ship v0.14 with tighter schema fidelity, cache/knowledge resets, resilient config+token handling, clearer reconciliation errors, and strongly-typed dry-run payloads.
-
-**Architecture:** Focused, TDD-driven changes inside existing server/tool layers: enforce ToolRegistry metadata/schema parity, route cache clears through DeltaCache, inject per-instance config with robust token validation, add error normalization in reconciliation executor, and replace untyped dry-run request blobs with explicit Zod schemas mirrored by handlers.
-
-**Tech Stack:** TypeScript 5, Node 18+, MCP SDK, Zod, Vitest (unit/integration/e2e), Prettier/ESLint, docs in `docs/reference`.
-
-### Task 1: Enforce tool metadata schema parity
-
-**Files:**
-- Modify: `src/server/toolRegistry.ts`
-- Modify: `src/server/__tests__/toolRegistry.test.ts`
-
-**Step 1: Write failing unit test**
-Add a case that registers a tool whose `metadata.inputJsonSchema` differs from the generated schema and expects an error: `metadata input schema must match generated schema for tool "budget-sync"`.
-
-**Step 2: Run test to see it fail**
-`npm run test:unit -- --runInBand src/server/__tests__/toolRegistry.test.ts --testNamePattern "metadata input schema"`
-
-**Step 3: Implement enforcement**
-In `register`, call a new helper (e.g., `ensureMetadataSchemaMatches`) that generates the JSON schema from `inputSchema`; if `metadata.inputJsonSchema` exists and differs, throw; otherwise populate it. Remove redundant schema regeneration in `listTools` when metadata is present.
-
-**Step 4: Re-run unit test**
-`npm run test:unit -- --runInBand src/server/__tests__/toolRegistry.test.ts --testNamePattern "metadata input schema"`
-
-**Step 5: Commit**
-`git add src/server/toolRegistry.ts src/server/__tests__/toolRegistry.test.ts && git commit -m "fix: validate tool metadata schemas at registration"`
-
-### Task 2: Make clear_cache reset delta knowledge
-
-**Files:**
-- Modify: `src/server/YNABMCPServer.ts`
-- Modify: `src/server/__tests__/YNABMCPServer.integration.test.ts`
-- Modify: `docs/reference/TOOLS.md`
-
-**Step 1: Add failing integration test**
-In `YNABMCPServer.integration.test.ts`, assert that after `list_accounts` populates caches, `clear_cache` drives knowledge entries to zero via `diagnostic_info` with `include_delta: true`.
-
-**Step 2: Run test to confirm failure**
-`npm run test:integration:domain -- --testNamePattern "clear cache runs"`
-
-**Step 3: Implement reset**
-In `setupToolRegistry`, change the `clear_cache` handler to call `this.deltaCache.forceFullRefresh()` (not just `cacheManager.clear()`), ensuring both cache and `ServerKnowledgeStore` reset.
-
-**Step 4: Update docs**
-Document in `docs/reference/TOOLS.md` that `clear_cache` now clears cache *and* delta knowledge.
-
-**Step 5: Re-run integration slice**
-`npm run test:integration:domain -- --testNamePattern "clear cache runs"`
-
-**Step 6: Commit**
-`git add src/server/YNABMCPServer.ts src/server/__tests__/YNABMCPServer.integration.test.ts docs/reference/TOOLS.md && git commit -m "feat: reset delta knowledge when cache is cleared"`
-
-### Task 3: Reloadable config, per-instance server config, and resilient token validation
-
-**Files:**
-- Modify: `src/server/config.ts`
-- Modify: `src/server/YNABMCPServer.ts`
-- Modify: `src/server/__tests__/config.test.ts`
-- Modify: `src/server/__tests__/YNABMCPServer.test.ts`
-- Modify: `src/server/__tests__/server-startup.integration.test.ts`
-- Modify: `scripts/run-throttled-integration-tests.js`
-
-**Step 1: Verify/extend config reload tests**
-Ensure `config.test.ts` covers re-parsing on successive `loadConfig()` calls and explicit env overrides. Add a case asserting per-call env changes are observed.
-
-**Step 2: Add failing per-instance server test**
-In `YNABMCPServer.test.ts`, instantiate two servers after mutating `process.env.YNAB_ACCESS_TOKEN` and assert each uses its instantiation-time config (via diagnostic/info accessor).
-
-**Step 3: Make config injectable per instance**
-In `YNABMCPServer.ts`, replace module-level `config` usage with `this.serverConfig = loadConfig()` inside the constructor; route YNAB API creation and tool setup through `this.serverConfig`.
-
-**Step 4: Harden token validation**
-Wrap `validateToken` to catch `SyntaxError` / malformed JSON or HTML responses from YNAB and throw `AuthenticationError` with guidance; keep 401/403 handling; rethrow others.
-
-**Step 5: Windows-safe throttled runner**
-In `scripts/run-throttled-integration-tests.js`, spawn Vitest via a platform-safe path (use `.cmd` on win32, otherwise the binary) and avoid EINVAL issues.
-
-**Step 6: Run focused tests**
-`npm run test:unit -- src/server/__tests__/config.test.ts src/server/__tests__/YNABMCPServer.test.ts`
-`npm run test:integration:core -- --testNamePattern "server-startup"`
-`node scripts/run-throttled-integration-tests.js --help` (manual sanity on Windows)
-
-**Step 7: Commit**
-`git add src/server/config.ts src/server/YNABMCPServer.ts src/server/__tests__/*.ts scripts/run-throttled-integration-tests.js && git commit -m "feat: reloadable config and robust token validation"`
-
-### Task 4: Propagate fatal YNAB errors during reconciliation
-
-**Files:**
-- Modify: `src/tools/reconciliation/executor.ts`
-- Modify: `src/tools/reconciliation/__tests__/executor.test.ts`
-- Modify: `src/tools/reconciliation/__tests__/executor.integration.test.ts`
-
-**Step 1: Add failing unit tests for error normalization**
-Cover `normalizeYnabError`, `shouldPropagateYnabError`, and status/message shaping for 429/404 plus generic errors.
-
-**Step 2: Implement normalization helpers**
-Add helpers in `executor.ts` to parse YNAB SDK error objects, standard Errors, and strings; decide propagation for 401/403/404/429/500/503; attach status to thrown Error.
-
-**Step 3: Add failing integration test for bulk rate-limit**
-Simulate bulk create rejecting with `id: '429'` and assert `executeReconciliation` rejects with status 429 instead of silent fallback.
-
-**Step 4: Wire helpers into bulk + sequential paths**
-In bulk chunk catch blocks, propagate fatal errors, otherwise log and fall back; in sequential creation, log failure reasons with status and propagate fatal errors after logging.
-
-**Step 5: Run reconciliation suites**
-`npm run test:unit -- src/tools/reconciliation/__tests__/executor.test.ts`
-`npm run test:integration:reconciliation` (or project pattern)
-
-**Step 6: Commit**
-`git add src/tools/reconciliation/executor.ts src/tools/reconciliation/__tests__/executor*.test.ts && git commit -m "fix: propagate fatal YNAB errors during reconciliation"`
-
-### Task 5: Strongly type dry-run transaction outputs
-
-**Files:**
-- Modify: `src/tools/schemas/outputs/transactionMutationOutputs.ts`
-- Modify: `src/tools/schemas/outputs/__tests__/transactionMutationSchemas.test.ts`
-- Modify: `src/tools/transactionTools.ts`
-- Modify: `src/tools/__tests__/transactionTools.test.ts`
-- (Optional) Modify: `src/tools/__tests__/transactionTools.integration.test.ts`, `src/__tests__/workflows.e2e.test.ts`
-
-**Step 1: Add failing schema tests**
-Assert dry-run `request` for create/update/delete matches explicit schemas (no `budget_id`/`dry_run`, required ids present) and rejects arbitrary `z.record` payloads.
-
-**Step 2: Define typed dry-run request schemas**
-In `transactionMutationOutputs.ts`, introduce `CreateTransactionDryRunRequestSchema`, `UpdateTransactionDryRunRequestSchema`, and `DeleteTransactionDryRunRequestSchema` (omit `budget_id`/`dry_run`), export inferred types, and replace existing `z.record` uses in dry-run branches.
-
-**Step 3: Update handler dry-run branches**
-In `transactionTools.ts`, build `request` objects that include only allowed fields (exclude `budget_id`/`dry_run`) for create/update/delete dry-runs.
-
-**Step 4: Update unit tests**
-Adjust/create tests in `transactionTools.test.ts` to assert dry-run outputs match the new schemas and make no API calls.
-
-**Step 5: (Optional) Integration/E2E checks**
-Add integration/E2E assertions that dry-run outputs validate against updated schemas.
-
-**Step 6: Run targeted suites**
-`npm run test:unit -- src/tools/schemas/outputs/__tests__/transactionMutationSchemas.test.ts src/tools/__tests__/transactionTools.test.ts`
-`npm run test:integration -- --testPathPattern=transactionTools` (if present)
-
-**Step 7: Commit**
-`git add src/tools/schemas/outputs/transactionMutationOutputs.ts src/tools/schemas/outputs/__tests__/transactionMutationSchemas.test.ts src/tools/transactionTools.ts src/tools/__tests__/transactionTools.test.ts && git commit -m "feat: type-safe dry-run transaction outputs"`
-
-### Final verification (after all tasks)
-
-- `npm run lint`
-- `npm run type-check`
-- `npm test`
-- `npm run build` (or `npm run build:prod` if cutting release)
-- Update `CHANGELOG.md` and `.pr-description.md` with changes and test commands.