Public: anyone can read this; nobody can change it.

src/web/pages/flow.ts drops the unused live and readQueues imports #39

Merged opened this view on origin

Check · Review · Land done · Live — Landed; nothing deploys from this branch, so this is the end of its path.

Merged

Closes forge tracker issue #2171 (https://forge-trevato-apps.open-platform.sh/trevato/forge/issues/2171). Pillar (2) Cut the noise.

shipped merged

opened this view on origin

Closes forge tracker issue #2171 (https://forge-trevato-apps.open-platform.sh/trevato/forge/issues/2171). Pillar (2) Cut the noise.

After #2139 the flow page reads duplicates through systemFor and the queue through queuesFor, so live (from system/live.ts) and the value readQueues (from flow/queue.ts) were imported and never used. tsc doesn't flag them because noUnusedLocals is off. This drops the live import and keeps only import type { QueueRow }. Both modules still load through system/telemetry.ts, so nothing changes at import time. No behaviour change.

Tests:

  • bun scripts/gate.ts: exit 0
  • ./node_modules/typescript/bin/tsc --noEmit: clean
  • bun test --timeout 60000 on flow-pipeline, flow-throughput, flow-stage, flow-card, merge-queue, crew-held, system-telemetry and check-list: 79 pass, 0 fail across 8 files

The full gate wasn't run locally.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs