test(wtr): windows compat, kinda

.gitattributes

@@ -0,0 +1 @@
+* text=auto eol=lf

.prettierrc

@@ -2,5 +2,6 @@
     "printWidth": 100,
     "singleQuote": true,
     "tabWidth": 4,
-    "trailingComma": "es5"
+    "trailingComma": "es5",
+    "endOfLine": "lf"
 }

package.json

@@ -104,5 +104,6 @@
         "http-cache-semantics": "4.2.0",
         "@types/estree": "^1.0.8"
     },
-    "dependencies": {}
+    "dependencies": {},
+    "packageManager": "yarn@1.22.22"
 }

packages/@lwc/integration-not-karma/configs/plugins/serve-hydration.js

@@ -12,7 +12,7 @@ const LWC_SSR = readFileSync(
     'utf8'
 );
 
-const ROOT_DIR = path.join(import.meta.dirname, '../..');
+const ROOT_DIR = path.join(import.meta.dirname, '..', '..');
 const COMPONENT_NAME = 'x-main';
 const COMPONENT_ENTRYPOINT = 'x/main/main.js';
 
@@ -25,7 +25,9 @@ async function compileModule(input, targetSSR, format) {
                 targetSSR,
                 modules: [{ dir: modulesDir }],
                 experimentalDynamicComponent: {
-                    loader: fileURLToPath(new URL('../../helpers/loader.js', import.meta.url)),
+                    loader: fileURLToPath(
+                        new URL('../../helpers/loader.js', import.meta.url)
+                    ).replaceAll(path.sep, path.posix.sep),
                     strict: true,
                 },
                 enableSyntheticElementInternals: true,
@@ -108,8 +110,8 @@ async function getSsrMarkup(componentEntrypoint, configPath) {
  * This function wraps those configs in the test code to be executed.
  */
 async function wrapHydrationTest(configPath) {
-    const suiteDir = path.dirname(configPath);
-    const componentEntrypoint = path.join(suiteDir, COMPONENT_ENTRYPOINT);
+    const suiteDir = path.posix.dirname(configPath);
+    const componentEntrypoint = path.posix.join(suiteDir, COMPONENT_ENTRYPOINT);
     const ssrOutput = await getSsrMarkup(componentEntrypoint, configPath);
 
     return `

packages/@lwc/integration-not-karma/configs/plugins/test-hydration.js

@@ -1,6 +1,7 @@
 import { spyOn } from '@vitest/spy';
 import * as LWC from 'lwc';
 import { setHooks } from '../../helpers/hooks';
+import { resetDOM, resetAlreadyLoggedMessages } from '../../helpers/reset';
 
 /*
  * Because these tests are written in JS, the type defs below are not enforced. They are provided
@@ -132,6 +133,9 @@ export function runTest(configPath, componentPath, ssrRendered) {
 
     afterEach(() => {
         consoleSpy.reset();
+        // in case of retries
+        resetDOM();
+        resetAlreadyLoggedMessages();
     });
 
     afterAll(() => {

packages/@lwc/rollup-plugin/src/index.ts

@@ -79,9 +79,9 @@ export interface RollupLwcOptions {
 
 const PLUGIN_NAME = 'rollup-plugin-lwc-compiler';
 
-const IMPLICIT_DEFAULT_HTML_PATH = '@lwc/resources/empty_html.js';
+const IMPLICIT_DEFAULT_HTML_PATH = ['@lwc', 'resources', 'empty_html.js'].join(path.sep);
 const EMPTY_IMPLICIT_HTML_CONTENT = 'export default void 0';
-const IMPLICIT_DEFAULT_CSS_PATH = '@lwc/resources/empty_css.css';
+const IMPLICIT_DEFAULT_CSS_PATH = ['@lwc', 'resources', 'empty_css.css'].join(path.sep);
 const EMPTY_IMPLICIT_CSS_CONTENT = '';
 const SCRIPT_FILE_EXTENSIONS = ['.js', '.mjs', '.jsx', '.ts', '.mts', '.tsx'];
 
@@ -352,18 +352,19 @@ export default function lwc(pluginOptions: RollupLwcOptions = {}): Plugin {
             // Specifier will only exist for modules with alias paths.
             // Otherwise, use the file directory structure to resolve namespace and name.
             const [namespace, name] =
-                // Note we do not need to use path.sep here because this filename contains
-                // a '/' regardless of Windows vs Unix, since it comes from the Rollup `id`
-                specifier?.split('/') ?? path.dirname(filename).split('/').slice(-2);
+                specifier?.split('/') ?? path.dirname(filename).split(path.sep).slice(-2);
 
             /* v8 ignore next */
             if (!namespace || !name) {
                 // TODO [#4824]: Make this an error rather than a warning
                 this.warn(
-                    'The component namespace and name could not be determined from the specifier ' +
-                        JSON.stringify(specifier) +
-                        ' or filename ' +
-                        JSON.stringify(filename)
+                    `The component namespace and name (${JSON.stringify(
+                        namespace
+                    )} and ${JSON.stringify(
+                        name
+                    )}) could not be determined from the specifier ${JSON.stringify(
+                        specifier
+                    )} or filename ${JSON.stringify(path.dirname(filename))}`
                 );
             }
 

playground/package.json

@@ -6,12 +6,13 @@
     "description": "Playground project to experiment with LWC.",
     "scripts": {
         "dev": "rollup -c --watch",
-        "build": "NODE_ENV=production rollup -c"
+        "build": "cross-env NODE_ENV=production rollup -c"
     },
     "devDependencies": {
         "@lwc/rollup-plugin": "8.24.0",
         "@rollup/plugin-replace": "^6.0.3",
         "@rollup/plugin-typescript": "^12.3.0",
+        "cross-env": "^10.1.0",
         "lwc": "8.24.0",
         "rollup": "^4.52.5",
         "rollup-plugin-livereload": "^2.0.5",

scripts/tasks/check-and-rewrite-package-json.js

@@ -40,7 +40,7 @@ const expectedPkgJsons = [];
 
 for (const dir of globSync('./packages/@lwc/*')) {
     const filename = path.join('./', dir, 'package.json');
-    const actual = fs.readFileSync(filename, 'utf-8');
+    const actual = fs.readFileSync(filename, 'utf-8').replace(/\r\n/g, '\n'); // Windows compat
     const pkg = JSON.parse(actual);
     // Skip private packages
     if (pkg.private) {

scripts/tasks/generate-license-files.mjs

@@ -98,7 +98,8 @@ const formattedLicense = await prettier.format(newLicense, {
 
 // Check against current top-level license for changes
 const shouldWarnChanges =
-    process.argv.includes('--test') && formattedLicense !== (await readFile('LICENSE.md', 'utf-8'));
+    process.argv.includes('--test') &&
+    formattedLicense !== (await readFile('LICENSE.md', 'utf-8')).replace(/\r\n/g, '\n');
 
 // Top level license
 await writeFile('LICENSE.md', formattedLicense, 'utf-8');

scripts/tasks/verify-treeshakable.mjs

@@ -6,19 +6,16 @@
  */
 
 // Inspired from: https://github.com/Rich-Harris/agadoo
-import path from 'node:path';
 import { rollup } from 'rollup';
 import pluginVirtual from '../rollup/plugin-virtual.mjs';
 
 async function check(input) {
-    const resolved = path.resolve(input);
-
     // Tell rollup to bundle a fake file whose content is just `import "path from CLI arg"`
     // If the imported file is tree-shakeable (pure imports/exports, no side effects),
     // then the bundled code will be an empty file
     const bundle = await rollup({
         input: '__virtual__',
-        plugins: [pluginVirtual(`import "${resolved}";`)],
+        plugins: [pluginVirtual(`import "${input}";`)],
         onwarn: (warning, handle) => {
             if (warning.code !== 'EMPTY_BUNDLE') handle(warning);
         },
@@ -28,26 +25,20 @@ async function check(input) {
         format: 'esm',
     });
 
-    const [chunk] = res.output;
+    const [{ code }] = res.output;
 
     return {
-        code: chunk.code,
-        isTreeShakable: chunk.code.trim().length === 0,
+        code,
+        isTreeShakable: code.trim().length === 0,
     };
 }
 
 const input = process.argv[2];
-check(input)
-    .then((res) => {
-        if (res.isTreeShakable === false) {
-            console.error(
-                `${res.code}\n❗️ Failed to fully treeshake ${input}; see remaining code above.`
-            );
-        }
+const result = await check(input);
+if (!result.isTreeShakable) {
+    console.error(
+        `${result.code}\n❗️ Failed to fully treeshake ${input}; see remaining code above.`
+    );
+}
 
-        process.exit(res.isTreeShakable ? 0 : 1);
-    })
-    .catch((err) => {
-        console.error(err);
-        process.exit(1);
-    });
+process.exitCode = result.isTreeShakable ? 0 : 1;

yarn.lock

@@ -1320,6 +1320,11 @@
   dependencies:
     tslib "^2.4.0"
 
+"@epic-web/invariant@^1.0.0":
+  version "1.0.0"
+  resolved "https://registry.yarnpkg.com/@epic-web/invariant/-/invariant-1.0.0.tgz#1073e5dee6dd540410784990eb73e4acd25c9813"
+  integrity sha512-lrTPqgvfFQtR/eY/qkIzp98OGdNJu0m5ji3q/nJI8v3SXkRKEnWiOxMmbvcSoAIzv/cGiuvRy57k4suKQSAdwA==
+
 "@esbuild/aix-ppc64@0.25.12":
   version "0.25.12"
   resolved "https://registry.yarnpkg.com/@esbuild/aix-ppc64/-/aix-ppc64-0.25.12.tgz#80fcbe36130e58b7670511e888b8e88a259ed76c"
@@ -5432,6 +5437,14 @@ create-wdio@9.18.2:
     type-fest "^4.41.0"
     yargs "^17.7.2"
 
+cross-env@^10.1.0:
+  version "10.1.0"
+  resolved "https://registry.yarnpkg.com/cross-env/-/cross-env-10.1.0.tgz#cfd2a6200df9ed75bfb9cb3d7ce609c13ea21783"
+  integrity sha512-GsYosgnACZTADcmEyJctkJIoqAhHjttw7RsFrVoJNXbsWWqaq6Ym+7kZjq6mS45O0jij6vtiReppKQEtqWy6Dw==
+  dependencies:
+    "@epic-web/invariant" "^1.0.0"
+    cross-spawn "^7.0.6"
+
 cross-fetch@3.1.5:
   version "3.1.5"
   resolved "https://registry.yarnpkg.com/cross-fetch/-/cross-fetch-3.1.5.tgz#e1389f44d9e7ba767907f7af8454787952ab534f"