Skip to content

Fix Zeitwerk eager loading and GemfileHelper conflicts - #36

Merged
knu merged 2 commits into
masterfrom
fix/gemfile-helper-conflicts
Apr 5, 2026
Merged

Fix Zeitwerk eager loading and GemfileHelper conflicts#36
knu merged 2 commits into
masterfrom
fix/gemfile-helper-conflicts

Conversation

@knu

@knu knu commented Apr 5, 2026

Copy link
Copy Markdown
Member
  • Skip duplicate gems in GemfileHelper — Instead of aborting on gem version conflicts between the agent gemspec and huginn's Gemfile, silently skip gems that are already present. This avoids unnecessary failures when versions are compatible.
  • Require registered agent files directly — Replace setup_zeitwerk_loader with plain require. The Zeitwerk loader mapped the entire lib/<gem_name>/ directory to the Agents namespace, which caused version.rb to be eagerly loaded as Agents::Version and fail with NameError in CI (eager_load = true).

@knu
knu merged commit 72622a2 into master Apr 5, 2026
2 checks passed
@knu
knu deleted the fix/gemfile-helper-conflicts branch April 5, 2026 11:27
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