Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 4, 2026

Remove addon-shim from ember-repl

This task successfully removes the @embroider/addon-shim dependency from the ember-repl package, as it's no longer needed for modern Ember v2 addons.

Completed:

  • Explore the repository structure and understand the codebase
  • Identify how addon-shim is used in ember-repl
  • Create initial plan
  • Remove the config/addon-main.cjs file that uses addon-shim
  • Update package.json to remove @embroider/addon-shim dependency
  • Update package.json to remove the main field from ember-addon config
  • Update package.json to remove addon-main.cjs and config from files array
  • Update package.json to remove the export for addon-main.js
  • Run pnpm install to update lock file
  • Build the package to ensure it still works
  • Revert unrelated formatting changes per feedback

Changes Made:

  1. Removed @embroider/addon-shim from dependencies in package.json
  2. Removed the config/addon-main.cjs file that was using the addon-shim
  3. Removed the main field from the ember-addon configuration
  4. Removed references to addon-main.cjs and config from the files array
  5. Removed the ./addon-main.js export from package.json
  6. Updated pnpm-lock.yaml to reflect the dependency removal

Package builds successfully. This follows the same pattern used by the repl-sdk package in this monorepo.

Original prompt

This repo uses pnpm. Try to remove addon-shim from ember-repl


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copilot AI and others added 2 commits February 4, 2026 13:25
Co-authored-by: NullVoxPopuli <199018+NullVoxPopuli@users.noreply.github.com>
Co-authored-by: NullVoxPopuli <199018+NullVoxPopuli@users.noreply.github.com>
@NullVoxPopuli
Copy link
Owner

@copilot undo your linting changes. You did it wrong. Please stick to package.json scripts

Copilot AI changed the title [WIP] Remove addon-shim from ember-repl Remove @embroider/addon-shim from ember-repl Feb 4, 2026
Copilot AI requested a review from NullVoxPopuli February 4, 2026 13:32
Co-authored-by: NullVoxPopuli <199018+NullVoxPopuli@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 4, 2026

@copilot undo your linting changes. You did it wrong. Please stick to package.json scripts

Reverted the formatting changes in commit 1fcbc0a. The PR now only contains the addon-shim removal changes from package.json and the deletion of config/addon-main.cjs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants