Fix CI and CodeRabbit findings on mono PR #14446
completedSlava explicitly requested CI and CodeRabbit remediation for https://github.com/TextureHQ/mono/pull/14446. Inspect every current failing CI log and unresolved CodeRabbit thread; make only valid scope-contained changes directly on the existing PR branch in a dedicated worktree. Required: git identity, commit Co-authored-by trailer, relevant local lint/typecheck/tests before every push, then monitor CI + CodeRabbit through green. Do not merge.
Event Timeline
created
progress
Fixed the sole current CI failure (Knip) and pushed f2f83e5 to slava/site-live-commongrid. Removed the unused CommonGrid barrel file/type exports and made the internally used territory resolver private. Local package checks were blocked by the Fleet GitHub App token lacking private npm package download permission (HTTP 403); GitHub Actions is the authoritative verification. CodeRabbit had no review threads yet and was still processing.
progress
A concurrent scope-contained push advanced the PR branch while the first CI run was in flight, adding the supergraph schema (e55e6de). It carried forward the first Knip cleanup but left the CommonGrid barrel itself unused. Removed that barrel at 496e514. New CI watch active; CodeRabbit still has zero review threads.
progress
Addressed the five CodeRabbit findings at 509a9d2: normalized meta/pagination cursors, removed silent program-page truncation and added terminal logging on scan failure, introduced shared point cache + 5-concurrent lookup cap + bounded Retry-After/backoff handling, used the sibling relative import, and verified the gateway schema already exactly matches the canonical site schema. Local lint and production build pass. Targeted Jest is blocked by the existing local jest-prisma interactiveTransactions configuration mismatch; CI will exercise it.
progress
CodeRabbit’s re-review surfaced two valid edge cases. Fixed at 1e6e187: program pagination now fails fast if a continuing cursor is null/repeated, and the shared territory cache now deletes expired entries and uses 1,000-entry LRU eviction. Local lint and build pass; new CI/re-review watch active.
status_change
in_progress → completed