Remove transitional command/parameter rename code#4771
Conversation
| }, | ||
| "Integration | commands | invalidate-realm-urls": { | ||
| "delta_mb": -9.9 | ||
| }, |
There was a problem hiding this comment.
Does this make sense to include in this PR, @lukemelia? I imagine renamed/deleted modules might linger forever.
There was a problem hiding this comment.
Pull request overview
Removes the temporary “URL-flavored” command/module-path compatibility shims and the associated rewrite scripts that were introduced to bridge the url → identifier rename during downstream rollout. This simplifies the host command surface back to a single canonical set of command module paths and removes now-unneeded post-clone mutation steps for catalog/skills contents.
Changes:
- Deleted legacy alias rewrite scripts in
packages/skills-realmandpackages/catalog, and removed their invocation from package scripts. - Removed deprecated host command alias modules (
*-urls/get-realm-of-url) and the correspondingvirtualNetwork.shimModule()registrations. - Pruned legacy command entries from the host memory baseline.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| packages/skills-realm/scripts/rewrite-legacy-aliases.mjs | Removes the skills contents rewrite shim now that downstream imports are updated. |
| packages/skills-realm/package.json | Drops the rewrite step from skills:* scripts. |
| packages/host/memory-baseline.json | Removes baseline entries for deleted legacy command integration cases. |
| packages/host/app/commands/invalidate-realm-urls.ts | Deletes deprecated alias module (re-export). |
| packages/host/app/commands/index.ts | Removes legacy alias imports and shimModule() registrations. |
| packages/host/app/commands/get-realm-of-url.ts | Deletes deprecated alias module (re-export). |
| packages/host/app/commands/get-catalog-realm-urls.ts | Deletes deprecated alias module (re-export). |
| packages/host/app/commands/get-available-realm-urls.ts | Deletes deprecated alias module (re-export). |
| packages/catalog/scripts/rewrite-legacy-aliases.mjs | Removes the catalog contents rewrite shim now that downstream sources are updated. |
| packages/catalog/package.json | Drops the rewrite step from catalog:* scripts. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
#4627 renamed commands and their parameters to not assume realm and resource URLs. #4730 restored the old imports for the transitional period before the
boxel-skillsPR was deployed in staging and production. Now that everywhere has the correct imports and parameter names, we don’t need these shims and rewrite scripts.