fix(ci): read version for merge commit message from .bitmap instead of npm

[davidfirst]

The merge commit message often showed the version of the previous release.

The script read the version from the npm registry. The registry data can stay old for some minutes after a publish. Because of this delay, the script got the old version.

Now the script reads the version of teambit.harmony/bit from the .bitmap file. At commit time, this file already contains the new version. Thus the message always agrees with the committed data, and no network call is necessary. If the file read fails, the script uses the fallback message as before.

[qodo-free-for-open-source-projects]

PR Summary by Qodo

Use .bitmap version in CI merge commit messages

🐞 Bug fix 🕐 Less than 10 minutes

AI Description

• Read release versions from committed .bitmap instead of the eventually consistent npm registry.
• Preserve fallback commit messaging when .bitmap parsing or version lookup fails.

Diagram

graph TD
  CI["bit ci merge"] --> Script["Message script"] --> Read["Read .bitmap"] --> Valid{"Valid bit version?"}
  Valid -- Yes --> Versioned["Versioned message"]
  Valid -- No --> Fallback["Fallback message"]

Loading

High-Level Assessment

The PR's approach is appropriate because .bitmap is already committed with the release version when the merge message is generated, avoiding npm registry propagation delays and a network dependency. Continuing to query npm was rejected due to eventual consistency, while retaining the existing fallback safely handles malformed or unavailable local metadata.

Files changed (1) +11 / -4

Bug fix (1) +11 / -4

generate-merge-commit-message.jsRead merge commit version from .bitmap +11/-4

Read merge commit version from .bitmap

• Replaces the npm registry query with local '.bitmap' parsing and reads the 'bit.version' field after removing block comments. It validates the version before generating the release message and retains the existing fallback for read, parse, or lookup failures.

scripts/generate-merge-commit-message.js

[qodo-free-for-open-source-projects]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Tip of the day

💡 Did you know, you can copy the agent prompt from any finding and feed it to your IDE agent

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗