Skip to content

v7#21240

Merged
NullVoxPopuli merged 5 commits intomainfrom
nvp/v7-test
Mar 29, 2026
Merged

v7#21240
NullVoxPopuli merged 5 commits intomainfrom
nvp/v7-test

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 24, 2026

📊 Package size report   -70.3%↓

File Before (Size / Brotli) After (Size / Brotli)
dist/dev/packages/@ember/-internals/deprecations/index.js 5 kB / 1.5 kB -5.11%↓4.7 kB / -2.16%↓1.5 kB
dist/ember-template-compiler.js 766.2 kB / 113.9 kB
dist/ember-template-compiler.js.map 1.5 MB / 259.8 kB
dist/ember-testing.js 12.5 kB / 2.8 kB
dist/ember-testing.js.map 12.7 kB / 3.3 kB
dist/ember.debug.js 2.1 MB / 315.9 kB
dist/ember.debug.js.map 3.4 MB / 626.1 kB
dist/ember.prod.js 1.9 MB / 287.1 kB
dist/ember.prod.js.map 3.2 MB / 599.4 kB
dist/prod/packages/@ember/-internals/deprecations/index.js 4.7 kB / 1.4 kB -5.47%↓4.4 kB / -2.49%↓1.4 kB
lib/index.js 15.2 kB / 2.7 kB -25%↓11.4 kB / -36%↓1.7 kB
types/stable/@ember/-internals/deprecations/index.d.ts 1.1 kB / 297 B -15.8%↓941 B / -9.43%↓269 B
types/stable/ember-template-compiler/index.d.ts 224 B / 107 B -53.6%↓104 B / -22.4%↓83 B
types/stable/loader/lib/index.d.ts 129 B / 82 B
types/stable/require.d.ts 129 B / 82 B
Total (Includes all files) 18.3 MB / 3.5 MB -70.3%↓5.4 MB / -63.2%↓1.3 MB
Tarball size 4 MB -69.7%↓1.2 MB

🤖 This report was automatically generated by pkg-size-action

@NullVoxPopuli-ai-agent
Copy link
Copy Markdown

Upstream PRs needed for v7 compatibility

The following ecosystem packages access absolutePaths.templateCompiler or paths from the ember-source addon, which are removed in v7. PRs have been opened to handle the missing properties gracefully:

Package PR Issue
ember-cli ember-cli/ember-cli#10971 paths.debug in _initVendorFiles
@embroider/compat embroider-build/embroider#2695 / embroider-build/embroider#2696 vendor file creation
ember-cli-htmlbars ember-cli/ember-cli-htmlbars#793 absolutePaths.templateCompiler in templateCompilerPath()
ember-auto-import embroider-build/ember-auto-import#708 absolutePaths.templateCompiler in cleanBabelConfig()

All four are patched via pnpm in #21262 (merged). Once the upstream PRs are released, the patches can be removed.

Additionally, #21265 improves those patches to use require.resolve('ember-source/ember-template-compiler/index.js') instead of hardcoded dist paths.

Add pnpm patches for ember-cli and @embroider/compat paths fix

Patches ember-cli@6.11.0 (ember-cli/ember-cli#10971) and
@embroider/compat@4.1.16 (embroider-build/embroider#2695) to handle
ember-source no longer exporting `paths` in v7.

- ember-cli: guard `_initVendorFiles` against missing `paths`
- @embroider/compat: skip legacy vendor file creation when `paths` absent

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Add ember-cli-htmlbars patch, fix sourcemap test

- Patch ember-cli-htmlbars@7.0.0 to handle missing absolutePaths in
  v7 ember-source, falling back to the ESM template compiler path.
- Update sourcemap-test.js for v7's ESM-only dist structure
  (dist/ember.debug.js no longer exists).
- Include template-compiler-test.js fix for ESM imports.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Fix ember-cli-htmlbars patch: use dist/dev/ path for v7

The v7 build outputs to dist/dev/packages/ and dist/prod/packages/,
not dist/packages/. The template compiler fallback path must use the
dev directory.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Patch ember-auto-import to handle missing absolutePaths

ember-auto-import also accesses emberSource.absolutePaths.templateCompiler
for babel template compilation config. Add the same fallback to the
dist/dev/ ESM template compiler path for v7.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Use require.resolve instead of hardcoded dist path in patches

Replace hardcoded `dist/dev/packages/ember-template-compiler/index.js`
with `require.resolve('ember-source/ember-template-compiler/index.js')`
which resolves through the package exports map, making the patches
resilient to dist layout changes.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Remove pnpm patches for ember-cli-htmlbars, ember-auto-import, @embroider/compat

These patches were temporary shims for v7 compatibility. The upstream
PRs have landed:
- ember-cli-htmlbars: ember-cli/ember-cli-htmlbars#793
- ember-auto-import: embroider-build/ember-auto-import#708
- @embroider/compat: embroider-build/embroider#2695

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Lockfile
@NullVoxPopuli NullVoxPopuli marked this pull request as ready for review March 29, 2026 20:53
kategengler
kategengler previously approved these changes Mar 29, 2026
kategengler
kategengler previously approved these changes Mar 29, 2026
@NullVoxPopuli NullVoxPopuli merged commit 8352f23 into main Mar 29, 2026
38 checks passed
@NullVoxPopuli NullVoxPopuli deleted the nvp/v7-test branch March 29, 2026 23:43
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.

3 participants