Skip to content

[9/11] test(repo): enforce architecture and compatibility gates - #1484

Closed
aidenybai wants to merge 1 commit into
codex/internal-architecture-08-cli-facade-migrationfrom
codex/internal-architecture-09-compatibility-guardrails
Closed

[9/11] test(repo): enforce architecture and compatibility gates#1484
aidenybai wants to merge 1 commit into
codex/internal-architecture-08-cli-facade-migrationfrom
codex/internal-architecture-09-compatibility-guardrails

Conversation

@aidenybai

@aidenybai aidenybai commented Jul 29, 2026

Copy link
Copy Markdown
Member

Part 9 of 11. Review this PR against codex/internal-architecture-08-cli-facade-migration.

What changed

  • Adds source-layer architecture checks and cycle detection.
  • Adds packed public-package, CLI-help, build-policy, and dependency compatibility gates.
  • Centralizes build constants and version reads.
  • Adds skill/guidance validation and the corrected lockfile.

Why

The architecture rewrite needs executable constraints so future changes cannot silently collapse the new boundaries or alter published contracts.

Impact

Repository validation only. Compatibility snapshots record zero active public deltas.

Validation

  • Frozen install
  • 2,177 source files, zero architecture violations
  • Five public packages match snapshots; zero active deltas
  • Build-policy, CLI-help, published-dependency, and node-support tests

Stack

Review only each PR's diff against its base branch. Merge in order.

  1. #1463 — Git service decomposition
  2. #1478 — Core execution pipeline
  3. #1479 — Workspace package graph
  4. #1480 — Package-aware rules
  5. #1487 — Private evaluator parity
  6. #1482 — API/LSP adapter facades
  7. #1483 — CLI workflow decomposition
  8. #1481 — CLI facade import migration
  9. #1484 — Architecture and compatibility gates
  10. #1485 — Shared deslop cycle traversal
  11. #1486 — Architecture docs and skill sync

@pkg-pr-new

pkg-pr-new Bot commented Jul 29, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/eslint-plugin-react-doctor@1484
npm i https://pkg.pr.new/oxlint-plugin-react-doctor@1484
npm i https://pkg.pr.new/react-doctor@1484

commit: 0b2a2b3

@github-actions

Copy link
Copy Markdown
Contributor

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit 0b2a2b3.

@aidenybai

Copy link
Copy Markdown
Member Author

Closing this architecture and compatibility gates slice as part of the regression-risk reduction. The original architecture stack has been replaced by four independent correctness/performance PRs that each target main. The full rewrite remains preserved on codex/internal-architecture-rewrite-full if we want to revisit it separately.

@aidenybai aidenybai closed this Jul 30, 2026
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