Skip to content

Document remaining-routing preflight and its 50-connection threshold - #1019

Merged
techmannih merged 2 commits into
mainfrom
techmannih/document-remaining-routing-preflight
Nov 11, 2026
Merged

techmannih merged 2 commits into
mainfrom
techmannih/document-remaining-routing-preflight

Conversation

@techmannih

@techmannih techmannih commented Nov 10, 2026 •

Copy link
Copy Markdown
Member

The board reference previously described omitting routeRemaining and explicitly setting it to true as equivalent. Under the default preflight, the implicit routing phase runs only when 50 or fewer connections remain after explicit phases finish.

Board configuration What core does
50 remaining, routeRemaining omitted Routes all 50.
51 remaining, routeRemaining omitted Skips the entire implicit phase and reports a routing error.
51 remaining, routeRemaining={true} Routes all 51.
52 total, 1 routed explicitly, default settings Keeps the 1 routed connection and skips the remaining 51.
52 total, 2 routed explicitly, default settings Routes the remaining 50, completing all 52.

This PR updates the documentation for existing core behavior. The regression also covers explicit overrides and nested subcircuits. Core regression · Preview the corrected board properties reference

To reproduce in a core checkout, run bun test tests/features/board-route-remaining-preflight.test.tsx.

@vercel

vercel Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Nov 10, 2026 6:55am UTC

Request Review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 10, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-10T05:16:18.935955Z b30148b PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@techmannih techmannih left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 50/51 threshold is correct. One sentence overstates the scope of the policy.

Comment thread docs/elements/board.mdx Resolved
@techmannih

Copy link
Copy Markdown
Member Author

The limit applies to connections still waiting for the implicit routing phase. With the default settings, 50 remaining connections route. With 51, the whole implicit phase is skipped and core reports a routing error. Setting <board routeRemaining={true}> opts in to all 51.

You can also see the boundary with 52 total connections. Route one explicitly and 51 remain, so only the first routes. Route two explicitly and the remaining 50 also route. This PR documents existing core behavior. Executable cases

@techmannih
techmannih merged commit ab0abf5 into main Nov 11, 2026
5 checks passed
@tscircuitbot

Copy link
Copy Markdown

Thank you for your contribution! 🎉

PR Rating: ⭐
Impact: Tiny

Track your contributions and see the leaderboard at: tscircuit Contribution Tracker


This branch was successfully deployed

1 active deployment
Preview — 469ee655 Deployed Nov 10, 2026 by vercel[bot]
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.

3 participants