Skip to content

Migrate to Script API v2 & more architectural changes#393

Merged
JaylyDev merged 8 commits intomainfrom
repo-update-2026
May 4, 2026
Merged

Migrate to Script API v2 & more architectural changes#393
JaylyDev merged 8 commits intomainfrom
repo-update-2026

Conversation

@JaylyDev
Copy link
Copy Markdown
Owner

@JaylyDev JaylyDev commented Apr 29, 2026

This pull request includes several dependency updates, workflow cleanup, and minor code improvements across multiple scripts. The most significant changes are grouped below.

Dependency Updates:

  • Updated various @minecraft package dependencies in package.json and packages/minecraft-editor/package.json to newer versions, including switching to caret (^) versions and adding new dependencies like glob, @minecraft/common, @minecraft/gameplay-utilities, and ts-morph. [1] [2]
  • Updated @types/node and @minecraft/math to newer versions in package.json.

Workflow and Automation Changes:

  • Removed the .github/workflows/dependency-update.yml workflow, which previously handled automated dependency updates and pull requests.
  • In .github/workflows/cl.yml, changed the format check step to use node ./dist/index.js instead of node ./dist/tools/index.js.

Code Quality and Robustness Improvements:

  • Added null checks for inventory and enchantable components in several scripts (e.g., anti-32k-event, anti-32k) to prevent runtime errors. [1] [2] [3] [4]
  • Added // @ts-nocheck to multiple scripts to suppress TypeScript errors and facilitate smoother development. [1] [2] [3] [4] [5]

Script and Comment Cleanup:

Minor Fixes and Refactoring:

  • Fixed import paths in test files and scripts to use relative imports and updated usage of certain APIs (e.g., switching from runCommandAsync to runCommand). [1] [2]
  • Refactored the chat emoji replacements script to use a custom command registration instead of a chat event subscription.
  • Added missing property initializations in class definitions (e.g., AssertionError, IncompatibleEnchantmentAlertEvent). [1] [2]

These changes collectively improve dependency management, code robustness, and maintainability across the project.

@JaylyDev JaylyDev requested a review from Copilot May 4, 2026 20:25
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@JaylyDev JaylyDev marked this pull request as ready for review May 4, 2026 20:27
@JaylyDev JaylyDev merged commit f16e4c9 into main May 4, 2026
1 check passed
@JaylyDev JaylyDev deleted the repo-update-2026 branch May 4, 2026 22:17
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