Skip to content

feat: default breakout autorouter to fanout - #761

Merged
seveibar merged 1 commit into
mainfrom
agent/default-breakout-autorouter
Jul 28, 2026
Merged

feat: default breakout autorouter to fanout#761
seveibar merged 1 commit into
mainfrom
agent/default-breakout-autorouter

Conversation

@seveibar

@seveibar seveibar commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • make the autorouter contract explicit on BreakoutProps
  • default the shared breakout/fanout schema to autorouter="fanout"
  • preserve explicit autorouter overrides such as single_layer_fanout
  • update generated component documentation

<breakout> and <fanout> both use BreakoutProps, so this establishes the default once at the props boundary instead of duplicating it in core.

Validation

  • bun test — 388 passing, 0 failing
  • bunx tsc --noEmit
  • bun run build
  • git diff --check
  • all GitHub test, type-check, and format checks passing

@seveibar
seveibar marked this pull request as ready for review July 28, 2026 15:48
@seveibar
seveibar merged commit 7b0f4fe into main Jul 28, 2026
4 checks passed
@seveibar
seveibar deleted the agent/default-breakout-autorouter branch July 28, 2026 15:48
Comment thread tests/breakout.test.ts
Comment on lines +20 to +25
test("breakout and fanout elements default to the fanout autorouter", () => {
expect(breakoutProps.parse({}).autorouter).toBe("fanout")
expect(
breakoutProps.parse({ autorouter: "single_layer_fanout" }).autorouter,
).toBe("single_layer_fanout")
})

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A *.test.ts file may have AT MOST one test(...) call. After this modification, breakout.test.ts contains at least two test(...) blocks (the pre-existing ones plus the newly added "breakout and fanout elements default to the fanout autorouter" test). The file should be split into multiple numbered files, e.g. breakout1.test.ts, breakout2.test.ts, breakout3.test.ts, with each file containing exactly one test(...).

Spotted by Graphite (based on custom rule: Custom rule)

Fix in Graphite


Is this helpful? React 👍 or 👎 to let us know.

@tscircuitbot

Copy link
Copy Markdown
Contributor

Thank you for your contribution! 🎉

PR Rating: ⭐⭐
Impact: Minor

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


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.

2 participants