Skip to content

Commit bf7b669

Browse files
trevorturkclaude
andcommitted
Add Zeitwerk eager loading check to CI
Adds validation that Zeitwerk can eager load all files without errors. This catches autoloading issues like inflector bugs and missing conditional checks before they reach production. - Added check after linter step for fast fail - Runs on all Ruby/Rails matrix combinations - ~2 second check prevents production crashes ## Blocked This PR depends on crmne#504 (ActiveRecord dependency fix) to pass. Without it, the eager loading check fails with NameError. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 46bc870 commit bf7b669

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

.github/workflows/cicd.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,9 @@ jobs:
6868
- name: Check code format
6969
run: bundle exec rubocop
7070

71+
- name: Check Zeitwerk eager loading
72+
run: bundle exec ruby -e "require 'ruby_llm'; Zeitwerk::Loader.eager_load_all"
73+
7174
- name: Run tests
7275
if: matrix.ruby-version != '3.4' || matrix.rails-version != 'rails-8.0'
7376
run: bundle exec appraisal ${{ matrix.rails-version }} rspec --tag '~generator'

0 commit comments

Comments
 (0)