chore(release): v3.4.1 - #180
Conversation
The bump re-serializes composer.json/package.json with JSON.stringify, whose array wrapping differs from Biome and fails `bun run lint`. Run `bun run format` at the end so the release stays lint-clean without a manual step.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (11)
📒 Files selected for processing (4)
📝 WalkthroughWalkthroughPackage versions in the Composer root manifest, workspace manifest, and countries package manifest are bumped from Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Release v3.4.1 updating published artifacts to include the recently merged country-name alias support in getCountryCode(name), along with the corresponding data regeneration and version bumps across the JS + Composer distributions.
Changes:
- Bumps versions to 3.4.1 across workspace/package artifacts (
package.json,composer.json,dist/package.json, etc.) - Rebuilds and commits
dist/outputs so aliases and the Myanmar name cleanup are reflected in shipped JSON/CSV/SQL and JS bundles - Updates the version-bump script to run
bun run formatafter writing JSON + updating the lockfile to avoid Biome formatting drift
Reviewed changes
Copilot reviewed 4 out of 15 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| packages/scripts/version.ts | Runs bun run format after version/lock updates to normalize JSON formatting for Biome/lint. |
| packages/countries/package.json | Bumps package version to 3.4.1. |
| package.json | Bumps workspace version to 3.4.1. |
| composer.json | Bumps Composer package version to 3.4.1. |
| dist/package.json | Bumps dist package version to 3.4.1. |
| dist/mjs/index.js | Rebuilt ESM bundle including alias-aware getCountryCode logic and updated dataset. |
| dist/mjs/currencies.js | Rebuilt ESM currencies bundle header version bump. |
| dist/cjs/currencies.js | Rebuilt CJS currencies bundle header version bump. |
| dist/index.d.ts | Updates ICountry typings to include optional alias?: string[]. |
| dist/countries.min.json | Rebuilt minified countries dataset including alias fields and Myanmar rename. |
| dist/minimal/countries.en.min.json | Updates minimal EN names dataset (Myanmar cleanup). |
| dist/countries.csv | Updates CSV export for Myanmar name cleanup. |
| dist/data.sql | Updates SQL export for Myanmar name cleanup. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Release v3.4.1 — bundles the country-name alias work merged since v3.4.0 (#178 + #179).
User-facing
getCountryCode(name)now resolves a curated set of alternative and former names, fixing name→code lookups that broke on ISO renames:Czech Republic,Cape Verde,Turkey,Swaziland,Macedonia,Burma.UK,Britain,US,USA,America,UAE,Holland.Persia,Ceylon,Siam,Rhodesia,Abyssinia,Formosa,Zaire, …Byelorussia,Rumania,Bahrein, … and diacritic-freeCote d'Ivoire,Turkiye.Also cleans the
Myanmar (Burma)name hack → name is nowMyanmarwithBurma/Myanmar (Burma)as aliases. New optionalalias?: string[]onICountry; a guard test prevents any ambiguous alias. Adds ~1.2 KB tocountries.min.json; non-breaking.Release mechanics
composer.json, root +packages/countriespackage.json,dist/package.json, and the lockfile.dist/rebuilt and committed (the release exception to the no-dist-in-PRs rule) — aliases baked intocountries.min.json, CSV, SQL, and the JS bundles.bun run formatat the end, so the JSON.stringify vs Biome array-wrapping mismatch no longer failsbun run lint(hit during this release).After merge
gh release create v3.4.1creates the tag and triggers the npm publish workflow.Verified:
bun run cigreen — lint, 3 typecheck targets, build, 22 tests.