Skip to content

refactor: migrate to ESM and consolidate actions into shared build pipeline#97

Merged
xavpaice merged 23 commits intomainfrom
gerard/sc-133970/dependabot
Mar 3, 2026
Merged

refactor: migrate to ESM and consolidate actions into shared build pipeline#97
xavpaice merged 23 commits intomainfrom
gerard/sc-133970/dependabot

Conversation

@nvanthao
Copy link
Member

@nvanthao nvanthao commented Feb 13, 2026

Why

replicated-libs used by replicated-actions recently get updated for CVEs fixes and contain ESM code, the current replicated-actions ncc can't bundle mixed code of CommonJS and ESM.

What

  • Migrate from CommonJS to ESM: Converted the entire codebase to ES modules to
    support ESM-only dependencies like updated versions of @actions/core, @actions/exec, and @actions/github.
  • Replace ncc with rollup for bundling: @vercel/ncc does not support ESM output, so it was replaced with rollup
  • Consolidate promote-release, get-customer-instances, report-compatibility-result, and upgrade-cluster into the shared package: These four actions previously had their own separate package.json and were built independently. Their source files have been moved into src/ and they now share the root build pipeline, matching the pattern used by the other 11 actions.
  • Bump dependencies to ESM-compatible versions: @actions/core v3, @actions/exec v3, @actions/github v9, replicated-lib beta.28, and updated dev dependencies.

Test plan

  • make package-all completes without errors
  • All CI checks pass

@nvanthao nvanthao changed the title chore: update replicated-lib refactor: migrate to ESM and consolidate actions into shared build pipeline Feb 19, 2026
@nvanthao nvanthao requested a review from xavpaice February 19, 2026 23:09
@xavpaice xavpaice merged commit a84fef7 into main Mar 3, 2026
16 of 18 checks passed
@xavpaice xavpaice deleted the gerard/sc-133970/dependabot branch March 3, 2026 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants