docs: fix stale counts and PR number coupling in documentation

- Remove hardcoded "22" rule count from task 5.4 (single-source-of-truth
  is eslint.config.mjs, not a prose count that goes stale)
- Update Phase 3 summary table to reflect PR 5 submission
- Replace "PR 20" reference in eslint.config.mjs comment with
  "type refinement phase" to avoid stale PR number coupling
- Remove specific "5 stale" directive count from task 5.7
  (exact count is unnecessary and was inaccurate — it was 6)

Author: kdaviduik

docs/tasks/prd-typescript-migration.md

@@ -293,7 +293,7 @@ These are key checkpoints where the executor should perform manual verification
 |-------|-------------|-----|--------|
 | 1 | TypeScript Infrastructure Setup | 1 (PR 1) | Complete |
 | 2 | Type Definitions | 1 (PR 2) | Complete |
-| 3 | Tooling Modernization | 7 (PRs 3-8, with 7 split into 7a/7b) | In progress (PRs 3-4 submitted) |
+| 3 | Tooling Modernization | 7 (PRs 3-8, with 7 split into 7a/7b) | In progress (PRs 3-5 submitted) |
 | 4 | TypeScript File Conversion | 13 (PRs 9-21) | Not started |
 | 5 | Test File Conversion | 6 (PRs 22-26, with 25 split into 25a/25b) | Not started |
 

docs/tasks/tasks-prd-typescript-migration.md

@@ -246,13 +246,13 @@ Use Graphite (gt) commands for managing stacked branches:
 
   - [x] 5.3. Install `eslint` 9.x, `typescript-eslint` v8 (unified package), `@eslint/js` 9.x, and `globals`
 
-  - [x] 5.4. Create `eslint.config.mjs` (flat config format) with 22 TypeScript rules, JS/TS file scoping, and `src/types/` override for known `any`/`Function` debt. See `eslint.config.mjs` for the authoritative rule list.
+  - [x] 5.4. Create `eslint.config.mjs` (flat config format) with TypeScript rules, JS/TS file scoping, and `src/types/` override for known `any`/`Function` debt. See `eslint.config.mjs` for the authoritative rule list.
 
   - [x] 5.5. Delete `.eslintrc`, `test/.eslintrc`, and `.eslintignore`
 
   - [x] 5.6. Update `package.json`: lint script (remove `-c .eslintrc`, use `src/` for recursive linting), test script (remove redundant `pnpm run lint` — CI runs lint separately)
 
-  - [x] 5.7. Fix lint errors in newly-linted files: `hasOwnProperty` → `Object.hasOwn()`, removed unused `isObject` function, removed unused `element` param, removed unused `err` binding, cleaned up 5 stale eslint-disable directives
+  - [x] 5.7. Fix lint errors in newly-linted files: `hasOwnProperty` → `Object.hasOwn()`, removed unused `isObject` function, removed unused `element` param, removed unused `err` binding, cleaned up stale eslint-disable directives
 
   - [x] 5.8. Verify: `pnpm run lint` passes (0 errors, 0 warnings), `pnpm run testem` passes (794/794), `pnpm run type-check` passes, `pnpm run build` passes
 

eslint.config.mjs

@@ -74,7 +74,7 @@ export default tseslint.config(
   },
 
   // TECH DEBT: src/types/ contains legacy type definitions with `any`, `Function`, and `{}`.
-  // These suppressions are temporary — removed in PR 20 (type refinement).
+  // These suppressions are temporary — removed during the type refinement phase.
   {
     files: ['src/types/**/*.ts'],
     rules: {