Skip to content

fix(mixin): conditionally apply Baritone mixins - #6677

Merged
Wide-Cat merged 1 commit into
MeteorDevelopment:masterfrom
Big-Iron-Cheems:fix/baritone-mixin-loading
Sep 27, 2026
Merged

Wide-Cat merged 1 commit into
MeteorDevelopment:masterfrom
Big-Iron-Cheems:fix/baritone-mixin-loading

Conversation

@Big-Iron-Cheems

Copy link
Copy Markdown
Collaborator

Type of change

  • Bug fix
  • New feature

Description

Conditionally apply Baritone mixins only when a supported Baritone implementation is installed.

Meteor's Baritone integration is optional at runtime, but the Baritone mixin configuration is currently processed even when Baritone is absent.
This causes Mixin to log a missing target warning for baritone.command.defaults.ComeCommand.

Related issues

Closes #6676

How Has This Been Tested?

  • Meteor without Baritone: startup produces no missing baritone.command.defaults.ComeCommand Mixin warning.
  • Meteor with upstream Baritone (baritone): Baritone integration still works.
  • Meteor with MeteorDevelopment Baritone (baritone-meteor): Baritone integration still works.

Checklist:

  • My code follows the style guidelines of this project.
  • I have added comments to my code in more complex areas.
  • I have tested the code in both development and production environments.

@Wide-Cat
Wide-Cat merged commit c3d6cbb into MeteorDevelopment:master Sep 27, 2026
1 check passed
@Big-Iron-Cheems
Big-Iron-Cheems deleted the fix/baritone-mixin-loading branch September 27, 2026 09:03
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.

Unhandled ClassNotFoundException in ComeCommandMixin when Baritone is absent[Bug]

2 participants