Skip to content

ci(dashboard): verify routeTree.gen.ts is regenerated#144

Merged
pratyush618 merged 1 commit into
masterfrom
ci/dashboard-route-tree-check
May 8, 2026
Merged

ci(dashboard): verify routeTree.gen.ts is regenerated#144
pratyush618 merged 1 commit into
masterfrom
ci/dashboard-route-tree-check

Conversation

@pratyush618
Copy link
Copy Markdown
Collaborator

Summary

Adds a guard step to the dashboard CI workflow that fails the build if dashboard/src/routeTree.gen.ts differs from the freshly-generated tree.

The composite dashboard-build action already runs pnpm exec tsr generate before typecheck/test/build, so a stale committed routeTree.gen.ts would silently be overwritten in CI. Without this check a developer can add a route, forget to regenerate, and have the build still pass — leaving the committed tree drifted from source.

The step runs only in the dashboard PR/push workflow (dashboard.yml), not in the publish workflow, so release builds aren't blocked by drift detection.

Addresses recommendation 11 from the 2026-05-08 audit (.claude/reports/audit-2026-05-08.md).

Test plan

  • Local: re-running pnpm exec tsr generate against the current source produces no diff against the committed file.
  • CI: this PR exercises the new step; expect green.
  • Manual: add a stub route, push without regenerating, observe the step fail with the helpful error message.

@github-actions github-actions Bot added the ci label May 8, 2026
@pratyush618 pratyush618 merged commit 346cb42 into master May 8, 2026
14 checks passed
@pratyush618 pratyush618 deleted the ci/dashboard-route-tree-check branch May 8, 2026 04:14
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