fix: fix import type keywords and package subpath exports

- Copy browser-logs source from fix/node24 for correct `import type`
- Add ./test-helpers.js to dev-server-core exports map for nodenext
- Remove erroneous .js from bare package specifiers (@web/foo.js -> @web/foo)

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

Author: bennypowers

packages/browser-logs/src/browserScript.ts

@@ -4,17 +4,15 @@ import path from 'path';
 const REGEXP_SOURCE_MAP = /\/\/# sourceMappingURL=.*/;
 
 const serializeScript = fs
-  .readFileSync(path.resolve(__dirname, 'serialize.js'), 'utf-8')
+  .readFileSync(path.resolve(import.meta.dirname, 'serialize.js'), 'utf-8')
   .replace(REGEXP_SOURCE_MAP, '');
 const logUncaughtErrorsScript = fs
-  .readFileSync(path.resolve(__dirname, 'logUncaughtErrors.js'), 'utf-8')
+  .readFileSync(path.resolve(import.meta.dirname, 'logUncaughtErrors.js'), 'utf-8')
   .replace(REGEXP_SOURCE_MAP, '');
 
 /**
- * Create the browser script. This project is compiled as CJS because it also needs to run in node, so
- * we create a small wrapper.
- *
- * It can't be ESM anyway, because it should work on older browsers as well.
+ * Create the browser script as an IIFE wrapper.
+ * It can't be ESM because it should work on older browsers as well.
  */
 export const browserScript =
   '(function () { var module={};var exports={};\n' +

packages/browser-logs/src/deserialize.ts

@@ -1,4 +1,4 @@
-import { ParseStackTraceOptions, parseStackTrace } from './parseStackTrace.ts';
+import { type ParseStackTraceOptions, parseStackTrace } from './parseStackTrace.ts';
 
 const KEY_WTR_TYPE = '__WTR_TYPE__';
 const KEY_CONSTRUCTOR_NAME = '__WTR_CONSTRUCTOR_NAME__';

packages/browser-logs/src/serialize.ts

@@ -103,7 +103,7 @@ function createReplacer() {
     }
 
     if (type === 'symbol') {
-      return (value as Symbol).toString();
+      return (value as symbol).toString();
     }
 
     if (type === 'object') {

packages/dev-server-core/package.json

@@ -27,6 +27,10 @@
     "./test-helpers": {
       "types": "./test-helpers.d.ts",
       "default": "./dist/test-helpers.js"
+    },
+    "./test-helpers.js": {
+      "types": "./test-helpers.d.ts",
+      "default": "./dist/test-helpers.js"
     }
   },
   "engines": {

packages/dev-server-esbuild/test/ts.test.ts

@@ -2,7 +2,7 @@ import path from 'path';
 import { createTestServer } from '@web/dev-server-core/test-helpers.js';
 import { expectIncludes, expectNotIncludes } from '@web/dev-server-core/test-helpers.js';
 import type { Plugin as RollupPlugin } from 'rollup';
-import { fromRollup } from '@web/dev-server-rollup.js';
+import { fromRollup } from '@web/dev-server-rollup';
 
 import { esbuildPlugin } from '../src/index.js';
 

packages/dev-server-rollup/test/node/plugins/commonjs.test.ts

@@ -2,12 +2,12 @@ import { describe, it } from 'node:test';
 import rollupCommonjs from '@rollup/plugin-commonjs.ts';
 import { runTests } from '@web/test-runner-core/test-helpers.js';
 import { resolve } from 'path';
-import { chromeLauncher } from '@web/test-runner-chrome.js';
+import { chromeLauncher } from '@web/test-runner-chrome';
 
 import * as path from 'path';
 import { createTestServer, fetchText, expectIncludes } from '../test-helpers.ts';
 import { fromRollup } from '../../../src/index.ts';
-import { nodeResolvePlugin } from '@web/dev-server.js';
+import { nodeResolvePlugin } from '@web/dev-server';
 
 const commonjs = fromRollup(rollupCommonjs);
 

packages/dev-server-rollup/test/node/plugins/postcss.test.ts

@@ -1,6 +1,6 @@
 /// <reference types="../../../types/rollup-plugin-postcss" />
 import rollupPostcss from 'rollup-plugin-postcss';
-import { chromeLauncher } from '@web/test-runner-chrome.js';
+import { chromeLauncher } from '@web/test-runner-chrome';
 import { runTests } from '@web/test-runner-core/test-helpers.js';
 import { resolve } from 'path';
 

packages/dev-server-rollup/test/node/test-helpers.ts

@@ -5,7 +5,7 @@ import {
   fetchText,
   expectIncludes,
 } from '@web/dev-server-core/test-helpers';
-import type { DevServerCoreConfig, Logger } from '@web/dev-server-core.js';
+import type { DevServerCoreConfig, Logger } from '@web/dev-server-core';
 
 const __dirname = import.meta.dirname;
 

packages/rollup-plugin-polyfills-loader/test/src/rollupPluginPolyfillsLoader.test.ts

@@ -3,7 +3,7 @@ import { OutputChunk, rollup, OutputAsset, RollupOptions, OutputOptions } from '
 import { expect } from 'chai';
 import fs from 'fs';
 import path from 'path';
-import { rollupPluginHTML as html } from '@web/rollup-plugin-html.js';
+import { rollupPluginHTML as html } from '@web/rollup-plugin-html';
 import polyfillsLoader from '../../src/index.ts';
 
 type Output = (OutputChunk | OutputAsset)[];

packages/rollup-plugin-polyfills-loader/test/src/utils.test.ts

@@ -1,6 +1,6 @@
 import { describe, it } from 'node:test';
 import assert from 'node:assert/strict';
-import { fileTypes } from '@web/polyfills-loader.js';
+import { fileTypes } from '@web/polyfills-loader';
 import { shouldInjectLoader } from '../../src/utils.ts';
 
 describe('shouldInjectLoader', () => {