Skip to content

Report check-shorts progress - #4139

Draft
seveibar wants to merge 1 commit into
mainfrom
agent/report-check-shorts-progress
Draft

Report check-shorts progress#4139
seveibar wants to merge 1 commit into
mainfrom
agent/report-check-shorts-progress

Conversation

@seveibar

@seveibar seveibar commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • report input resolution and whether circuit JSON came from a prebuilt file, the current build cache, or a source render
  • surface named async effects while source circuits are rendering or routing
  • report checker loading, per-layer bitmap work, short-region detection, and debug-artifact generation
  • throttle connectivity-group progress to roughly 10% increments to avoid noisy output
  • add CLI coverage for high-level progress and structured checker events

Why

tsci check shorts can spend significant time rendering/routing a source circuit and rasterizing connectivity groups. With no intermediate output, the command looks like it has hung.

High-level phase messages work with the currently packaged fallback checker. Granular rasterization events use the optional progress API introduced by tscircuit/check-shorts#40.

Dependency sequencing

Merge and release tscircuit/check-shorts#40 so the CLI's dynamic latest import receives granular checker events. The CLI remains backward-compatible when the loaded checker does not emit progress.

Validation

  • bunx tsc --noEmit
  • bun test tests/cli/check — 18 tests passed
  • bun test tests/cli/check/check-shorts.test.ts — 5 tests passed
  • bun run format:check
  • bun run build

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant