Skip to content

Commit f2ac776

Browse files
NullVoxPopuliclaude
andcommitted
Fix lints, compile @embroider/macros in ember-source and @warp-drive
- Add configFile: false to eslint babel parser to prevent auto-discovery of babel.config.mjs (which has async plugins) - Rename config/environment.js to .cjs for type=module compat - Fix blueprint glob patterns in eslint config - Remove @percy/ember imports from tests (v1 addon, removed) - Remove unused parseNestedValue function - Add hbs() plugin for .hbs template compilation - Add emberSourceResolver plugin to resolve @ember/* and @glimmer/* virtual modules to ember-source/dist/packages/ - Configure WarpDrive global config for @embroider/macros compilation - Two babel passes: full pass (excluding @warp-drive) + macros-only for @warp-drive - All @embroider/macros calls now compiled away (0 runtime fallbacks) Test results: 205 tests, 67 pass, 138 fail (failures expected - removed ember-styleguide components, percy, etc.) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 2c50e58 commit f2ac776

54 files changed

Lines changed: 174 additions & 238 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.eslintrc.cjs

Lines changed: 24 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,24 @@
1-
'use strict';
1+
"use strict";
22

33
module.exports = {
44
root: true,
5-
parser: '@babel/eslint-parser',
5+
parser: "@babel/eslint-parser",
66
parserOptions: {
7-
ecmaVersion: 'latest',
8-
sourceType: 'module',
7+
ecmaVersion: "latest",
8+
sourceType: "module",
99
requireConfigFile: false,
1010
babelOptions: {
11+
configFile: false,
1112
plugins: [
12-
['@babel/plugin-proposal-decorators', { decoratorsBeforeExport: true }],
13+
["@babel/plugin-proposal-decorators", { decoratorsBeforeExport: true }],
1314
],
1415
},
1516
},
16-
plugins: ['ember'],
17+
plugins: ["ember"],
1718
extends: [
18-
'eslint:recommended',
19-
'plugin:ember/recommended',
20-
'plugin:prettier/recommended',
19+
"eslint:recommended",
20+
"plugin:ember/recommended",
21+
"plugin:prettier/recommended",
2122
],
2223
env: {
2324
browser: true,
@@ -27,31 +28,31 @@ module.exports = {
2728
// node files
2829
{
2930
files: [
30-
'./.eslintrc.cjs',
31-
'./.prettierrc.cjs',
32-
'./.stylelintrc.cjs',
33-
'./.template-lintrc.cjs',
34-
'./testem.cjs',
35-
'./blueprints/*/index.js',
36-
'./config/**/*.js',
37-
'./lib/*/index.js',
38-
'./scripts/**/*.mjs',
31+
"./.eslintrc.cjs",
32+
"./.prettierrc.cjs",
33+
"./.stylelintrc.cjs",
34+
"./.template-lintrc.cjs",
35+
"./testem.cjs",
36+
"./blueprints/*/index.cjs",
37+
"./config/**/*.js",
38+
"./lib/*/index.js",
39+
"./scripts/**/*.mjs",
3940
],
4041
parserOptions: {
41-
sourceType: 'script',
42+
sourceType: "script",
4243
},
4344
env: {
4445
browser: false,
4546
node: true,
4647
},
47-
extends: ['plugin:n/recommended'],
48+
extends: ["plugin:n/recommended"],
4849
},
4950
{
5051
// test files
51-
files: ['tests/**/*-test.{js,ts}'],
52-
extends: ['plugin:qunit/recommended'],
52+
files: ["tests/**/*-test.{js,ts}"],
53+
extends: ["plugin:qunit/recommended"],
5354
rules: {
54-
'qunit/require-expect': 'off',
55+
"qunit/require-expect": "off",
5556
},
5657
},
5758
],

.prettierrc.cjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
1-
'use strict';
1+
"use strict";
22

33
module.exports = {
44
overrides: [
55
{
6-
files: '*.{js,ts}',
6+
files: "*.{js,ts}",
77
options: {
88
singleQuote: true,
99
},

.stylelintrc.cjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
'use strict';
1+
"use strict";
22

33
module.exports = {
4-
extends: ['stylelint-config-standard', 'stylelint-prettier/recommended'],
4+
extends: ["stylelint-config-standard", "stylelint-prettier/recommended"],
55
};

.template-lintrc.cjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
'use strict';
1+
"use strict";
22

33
module.exports = {
4-
extends: 'recommended',
4+
extends: "recommended",
55
};

babel.config.mjs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,13 @@ const macros = buildMacros({
55
config.setGlobalConfig(import.meta.filename, '@embroider/core', {
66
active: true,
77
});
8+
config.setGlobalConfig(import.meta.filename, 'WarpDrive', {
9+
env: { DEBUG: false, TESTING: false },
10+
debug: {},
11+
activeLogging: {},
12+
deprecations: {},
13+
features: {},
14+
});
815
},
916
});
1017

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
/* eslint-env node */
22
module.exports = {
3-
description: 'Simple Blueprint to add an Initiative Sponsor',
3+
description: "Simple Blueprint to add an Initiative Sponsor",
44
};

blueprints/meetup/index.cjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
/* eslint-env node */
22
module.exports = {
3-
description: 'Simple Blueprint to add a meetup',
3+
description: "Simple Blueprint to add a meetup",
44
};

blueprints/showcase/index.cjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
/* eslint-env node */
22
module.exports = {
3-
description: 'Simple Blueprint to add a showcase',
3+
description: "Simple Blueprint to add a showcase",
44
};

blueprints/sponsor/index.cjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/* eslint-env node */
22
module.exports = {
3-
description: 'Simple Blueprint to add a Sponsor',
3+
description: "Simple Blueprint to add a Sponsor",
44

55
locals() {
66
return {

blueprints/team-member/index.cjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/* eslint-env node */
22
module.exports = {
3-
description: 'Simple Blueprint to add an Ember Team-Member',
3+
description: "Simple Blueprint to add an Ember Team-Member",
44

55
locals() {
66
return {

0 commit comments

Comments
 (0)