From f2ca2ed86fdb4bc21f8fc07dc309e0ded3be2d01 Mon Sep 17 00:00:00 2001 From: Saravana Achu Mac Date: Mon, 4 May 2026 16:02:18 -0700 Subject: [PATCH] docs(audit): mark tracker warning cleanup What changed: - Recorded tracker-web cleanup commit 04d2398. - Updated W7 checklist items, warning delta, and verification commands. - Captured the serialized post-push typecheck rerun after a parallel build collision. Warning impact: - Workspace lint warning count recorded as 95 -> 90. Verification: - pnpm --filter @bytelyst/tracker-web typecheck - pnpm --filter @bytelyst/tracker-web test - pnpm --filter @bytelyst/tracker-web exec eslint . --ext .ts,.tsx - pnpm --filter @bytelyst/tracker-web build - pnpm lint --- docs/AUDIT_WARNING_COMPLETION_CHECKLIST.md | 49 ++++++++++++++-------- 1 file changed, 32 insertions(+), 17 deletions(-) diff --git a/docs/AUDIT_WARNING_COMPLETION_CHECKLIST.md b/docs/AUDIT_WARNING_COMPLETION_CHECKLIST.md index 96352f46..464f3b7a 100644 --- a/docs/AUDIT_WARNING_COMPLETION_CHECKLIST.md +++ b/docs/AUDIT_WARNING_COMPLETION_CHECKLIST.md @@ -19,7 +19,7 @@ As of the latest local sweep: | `pnpm install -r --prefer-offline` | Pass | Run with `source ~/.zshrc && export GITEA_NPM_TOKEN`. | | `pnpm typecheck` | Pass | All workspace TypeScript compiles. | | `pnpm test` | Pass with known load sensitivity | Full suite passes; isolated reruns cleared prior transient platform/cowork timeouts. | -| `pnpm lint` | Pass | `0` errors. Latest summarized warning count: `189` warnings. | +| `pnpm lint` | Pass | `0` errors. Latest summarized warning count: `90` warnings after Agent C tracker-web cleanup. | | Known untouched WIP | Present | Do not touch `docker-compose.ecosystem.yml`, `products/nomgap/product.json`, `services/platform-service/src/modules/flags/seed.ts`, or incidental `pnpm-lock.yaml` churn unless explicitly assigned. | Recent warning-cleanup commits: @@ -32,6 +32,7 @@ Recent warning-cleanup commits: | `9625999` | `@lysnrai/platform-service` | Removed stale eslint-disable comments. | | `6978ddb` | `@lysnrai/cowork-service` | Typed test doubles for IPC/Fastify mocks. | | `db4257f` | `@bytelyst/admin-web` | Cleared feedback page React warnings. | +| `04d2398` | `@bytelyst/tracker-web` | Removed roadmap console/module warnings. | ## Agent Execution Protocol @@ -143,11 +144,11 @@ and push small commits quickly. If two agents need the same files, serialize. ### Active / Claim Table -| Stream | Status | Owner | File ownership | Current target | Last commit | -| ------ | ------ | ----- | ---------------------------------------------------------- | ------------------------------------------------- | ----------- | -| A | Open | — | `dashboards/admin-web/**` | W2 admin web React warnings | — | -| B | Open | — | `services/platform-service/src/**` only | W3/W4 service runtime console + predictive typing | — | -| C | Open | — | lint policy, CLI/script files, tracker module config, docs | W1/W5/W6/W7/W8/W9/W10 | — | +| Stream | Status | Owner | File ownership | Current target | Last commit | +| ------ | ------ | ------- | ---------------------------------------------------------- | --------------------------------------- | ----------- | +| A | Done | Agent A | `dashboards/admin-web/**` | W2 feedback page React warnings | `db4257f` | +| B | Done | Agent B | `services/platform-service/src/**` only | W4 predictive analytics typing | `021f053` | +| C | Done | Agent C | lint policy, CLI/script files, tracker module config, docs | W7 tracker-web console + module warning | `04d2398` | ### One-Line Prompts For Parallel Agents @@ -380,16 +381,25 @@ behavior, CLI output, or noise. Goal: finish tracker warnings without hiding real runtime errors. -- [ ] `dashboards/tracker-web/src/app/roadmap/page.tsx`: replace or route - `console.error` calls through an app logger if available. -- [ ] Decide whether client-side console errors are acceptable for public - roadmap actions. +- [x] `dashboards/tracker-web/src/app/roadmap/page.tsx`: replace or route + `console.error` calls through an app logger if available. Done in + `04d2398`; warning delta `-2`; verified with tracker-web + typecheck/test/build/lint and workspace lint. +- [x] Decide whether client-side console errors are acceptable for public + roadmap actions. Done in `04d2398`; public roadmap load/vote failures now + surface in page error state instead of browser console output. - [ ] Add `"type": "module"` to `dashboards/tracker-web/package.json` only if it is safe for Next/Vitest/Husky scripts. -- [ ] Alternatively rename/configure `eslint.config.js` to avoid Node's module - type warning. -- [ ] Run tracker-web typecheck/test/lint after any package.json or config - change. +- [x] Alternatively rename/configure `eslint.config.js` to avoid Node's module + type warning. Done in `04d2398`; removed stale `eslint.config.js` because + `eslint.config.mjs` is already the active Next ESLint config. +- [x] Run tracker-web typecheck/test/lint after any package.json or config + change. Done after `04d2398`; verified with + `pnpm --filter @bytelyst/tracker-web typecheck`, + `pnpm --filter @bytelyst/tracker-web test`, + `pnpm --filter @bytelyst/tracker-web exec eslint . --ext .ts,.tsx`, and + `pnpm --filter @bytelyst/tracker-web build`. A parallel post-push + typecheck collided with Next build `.next/types`; serialized rerun passed. ### W8. Lockfile + Repo State @@ -430,9 +440,12 @@ Goal: keep future agents aligned with the true baseline. - [ ] Update `docs/HANDOVER.md` when the current gate story changes. Include known flakes/load sensitivity if still relevant. - [ ] Keep `docs/CODEX_RESUME_PROMPT.md` aligned with the latest workflow. -- [ ] Record warning counts after each batch in this checklist. Use the latest - `/tmp/lint-*.log` path and warning count. -- [ ] Include commit hashes in this checklist for every completed batch. +- [x] Record warning counts after each batch in this checklist. Use the latest + `/tmp/lint-*.log` path and warning count. Agent C recorded + `/tmp/lint-before-agent-c.log` at `95` warnings and + `/tmp/lint-after-agent-c.log` at `90` warnings. +- [x] Include commit hashes in this checklist for every completed batch. Agent C + recorded tracker-web cleanup commit `04d2398`. - [ ] Keep "do not touch nomgap WIP" instructions visible until those files are resolved by their owner. @@ -473,4 +486,6 @@ Goal: keep future agents aligned with the true baseline. | 2026-05-04 | `b6da94d` | Typed tracker-web proxy test request helper. | Removed 1 `no-explicit-any` warning. | | 2026-05-04 | `9625999` | Removed stale platform-service lint disables. | Removed 2 stale-disable warnings. | | 2026-05-04 | `6978ddb` | Typed cowork-service test doubles. | Removed cowork-service test `any` cluster. | +| 2026-05-04 | `db4257f` | Cleared admin feedback page React warnings. | Removed 7 W2 warnings; lint remains 0 errors. | | 2026-05-04 | `021f053` | Typed predictive campaign event dispatch. | Removed 5 `no-explicit-any` warnings; workspace lint `189` -> `184`. | +| 2026-05-04 | `04d2398` | Cleared tracker-web roadmap lint warnings. | Removed 5 workspace warnings; tracker scoped lint is 0 warnings. |