Skip to content

ci: add generated code staleness check#97

Merged
bdougie merged 2 commits intomainfrom
fix/ci-ent-generate-check
Feb 15, 2026
Merged

ci: add generated code staleness check#97
bdougie merged 2 commits intomainfrom
fix/ci-ent-generate-check

Conversation

@bdougie
Copy link
Contributor

@bdougie bdougie commented Feb 14, 2026

Summary

  • Found this while working on feat: add project context to sessions #96, CI doesn't generate a new build.
  • Adds a check-generate CI job that runs go generate for ent schemas and diffs the output against committed files
  • Catches stale codegen early in the pipeline instead of surfacing as a runtime panic in smoke tests
  • New CheckGenerate Dagger function copies ent dir, regenerates, and diffs to detect drift

Test plan

  • Verify check-generate job passes on a clean branch
  • Verify it fails when ent schema is modified without running make generate

Continue Tasks: ✅ 3 no changes — View all

Runs go generate for ent schemas and diffs the output against the
committed files. Catches stale codegen before it surfaces as a
runtime panic in the smoke tests.
@bdougie bdougie requested a review from jpmcb February 15, 2026 16:16
Co-authored-by: John McBride <john@papercompute.com>
@bdougie bdougie merged commit 0594a9c into main Feb 15, 2026
12 checks passed
@bdougie bdougie deleted the fix/ci-ent-generate-check branch February 15, 2026 16:27
@continue
Copy link

continue bot commented Feb 15, 2026

No docs update needed. This PR adds an internal CI check that verifies generated code (ent schemas) stays in sync with committed files. It's a developer workflow improvement that:

  • Adds a check-generate CI job to catch stale codegen
  • Adds a CheckGenerate Dagger function for the check

This doesn't change any user-facing behavior, configuration options, or CLI commands. Contributors who need to run go generate already know the workflow from the AGENTS.md file in the repo.

PR #97 was merged: ci: add generated code staleness check
PR URL: #97
Merged by: bdougie

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