Skip to content

Add support for platform DRC-disable flags#2003

Merged
seveibar merged 1 commit intomainfrom
codex/add-platformconfig-flags-support
Mar 4, 2026
Merged

Add support for platform DRC-disable flags#2003
seveibar merged 1 commit intomainfrom
codex/add-platformconfig-flags-support

Conversation

@seveibar
Copy link
Contributor

@seveibar seveibar commented Mar 4, 2026

Motivation

  • Allow platform-level toggles to disable specific PCB DRC categories (netlist, placement, routing) so CI/host environments can selectively skip checks.

Description

  • Read new platform flags netlistDrcChecksDisabled, placementDrcChecksDisabled, and routingDrcChecksDisabled and incorporate them into DRC gating logic in lib/components/normal-components/Board.ts so shouldRunNetlistChecks, shouldRunPlacementChecks, and shouldRunRoutingChecks are computed from both existing pcbDisabled/routingDisabled and the new flags.
  • Bumped @tscircuit/props to ^0.0.487 to obtain the updated PlatformConfig typings that include these flags and updated package.json accordingly.
  • Added tests tests/root-circuit/platform-drc-checks-disabled.test.tsx to verify netlistDrcChecksDisabled and placementDrcChecksDisabled suppress the corresponding DRC errors.

Testing

  • Ran bun test tests/root-circuit/platform-drc-checks-disabled.test.tsx and both tests passed.
  • Ran bun test tests/root-circuit/platform-routing-disabled.test.tsx and it passed.
  • Ran bunx tsc --noEmit for full type-checking and it passed with no errors.
  • Ran bun run format to format code changes.

Codex Task

@vercel
Copy link

vercel bot commented Mar 4, 2026

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

Project Deployment Actions Updated (UTC)
tscircuit-core-benchmarks Ready Ready Preview, Comment Mar 4, 2026 0:28am

Request Review

@seveibar seveibar merged commit f3444be into main Mar 4, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant