Repository navigation
Conversation
…swap scope-rename swaps @Codecademy/gamut* and eslint-plugin-gamut for their @SkillSoft names in every module position and in comments. package-json, mdx-imports, and yarnrc cover dependencies, MDX ESM blocks, and .yarnrc.yml lists. @codecademy/gamut-kit has no replacement, so package-json and mf-shared expand it into the packages it bundled. In Module Federation config that makes Gamut a real singleton, which the checklist calls out. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 20, 2026
…migrations Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The package-json warning already names each range that's newer than the fork point, so the extra next step only repeated it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
dreamwasp
marked this pull request as ready for review
October 20, 2026 15:37
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Overview
Breaking change: every
@codecademy/gamut*package moves to@skillsoft/*, and@codecademy/gamut-kitis removed with no replacement.Stack (split from #28, merge in order):
mainmainscope-renameswaps the old names in every module position: imports, exports,require,import(),jest.mock/vi.mock,declare module, andimport('x').T. It also covers exact package strings liketranspilePackages, warning once per file, and package names in comments. It runs last, so every other migration matches on@codecademynames.package-jsonrenames dependencies and sets their versions. It keeps any@skillsoft/*entry that's already listed (such as a preview URL) and keeps*peer ranges. It warns when a range is newer than the upstream release the fork matches (sourceVersions).gamut-kitremoval:package-jsonexpands it into the packages it bundled.mf-shareddoes the same in Module Federationsharedconfig. That makes Gamut a real singleton across host and remotes, and the checklist says to deploy them together.mdx-importsruns the source migrations over.mdxESM blocks.yarnrcrenames or drops old names in.yarnrc.ymllists.Video, deep imports, ESLint, and Next.js are separate PRs stacked on this one.
PR Checklist
Testing Instructions
yarn nx test gamut-codemods. Expect 65 tests to pass. That includes the fixtures for each migration and an end-to-end fixture that runs the whole preset.Build and run on a throwaway repo:
Check
git diff:doc.mdxuse@skillsoft/*package.jsonlists each package thatgamut-kitbundled, with nogamut-kitmf.tsshares each package individuallyCheck that the next steps include the
gamut-kitand Module Federation items.Rerun with
--force. Nothing should change.PR Links and Envs
🤖 Generated with Claude Code