Skip to content

Commit 8d5d025

Browse files
trevorturkclaude
andcommitted
Add Zeitwerk eager loading check to CI
Adds a validation step that runs Zeitwerk::Loader.eager_load_all after dependency installation to catch autoloading issues early in the CI pipeline. This check helps prevent production crashes by detecting: - Inflector configuration errors - Constant definition issues - Missing conditional checks for optional dependencies The check runs quickly (~2 seconds) before the test matrix, providing rapid feedback on structural issues. Note: This check currently fails due to upstream ruby_llm ActiveRecord dependency issues. See: - crmne/ruby_llm#504 - crmne/ruby_llm#505 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 3b08a4a commit 8d5d025

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
@@ -30,6 +30,9 @@ jobs:
3030
run: |
3131
bundle install
3232
33+
- name: Check Zeitwerk eager loading
34+
run: bundle exec ruby -e "require 'ruby_llm/mcp'; Zeitwerk::Loader.eager_load_all"
35+
3336
- name: Install Bun dependencies for test fixtures
3437
run: |
3538
cd spec/fixtures/typescript-mcp && bun install

0 commit comments

Comments
 (0)