Skip to content

Compare launcher version against package.json instead of a literal - #5

Merged
chenmingtang830 merged 1 commit into
mainfrom
fix-version-test
Jul 24, 2026
Merged

Compare launcher version against package.json instead of a literal#5
chenmingtang830 merged 1 commit into
mainfrom
fix-version-test

Conversation

@chenmingtang830

Copy link
Copy Markdown
Owner

Summary

  • Run 30081775689 failed because tests/npm.test.js hardcoded "0.1.0-alpha.1"; the alpha.2 bump missed it
  • Read the version from package.json so the launcher/engine equality check survives future bumps

Test plan

  • node --test tests/npm.test.js passes locally (3/3)
  • After merge: re-dispatch npm publish (tag next)

🤖 Generated with Claude Code

The hardcoded version string broke the suite on every bump; reading
package.json keeps the launcher/engine equality check without the trap.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@chenmingtang830
chenmingtang830 merged commit 5ffbef3 into main Jul 24, 2026
@chenmingtang830
chenmingtang830 deleted the fix-version-test branch July 26, 2026 14:09
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