only tsc once

Author: mbostock

bundle.js

@@ -1,2 +1,2 @@
 export {version} from "./package.json";
-export * from "./src/index.js";
+export * from "./build/index.js";

package.json

@@ -32,7 +32,7 @@
     "test:mocha": "mkdir -p test/output && mocha --conditions=mocha 'test/**/*-test.*' 'test/plot.js'",
     "test:lint": "eslint src test",
     "test:typecheck": "tsc --noEmit",
-    "prepublishOnly": "rm -rf build dist && rollup -c && tsc && tsx scripts/readme-to-jsdoc.ts",
+    "prepublishOnly": "rm -rf build dist && tsc && tsx scripts/readme-to-jsdoc.ts && rollup -c",
     "postpublish": "git push && git push --tags",
     "dev": "vite"
   },
@@ -45,7 +45,6 @@
     "@rollup/plugin-commonjs": "22",
     "@rollup/plugin-json": "4",
     "@rollup/plugin-node-resolve": "13",
-    "@rollup/plugin-typescript": "^8.3.2",
     "@types/d3": "^7.4.0",
     "@types/expect": "^24.3.0",
     "@types/mocha": "^9.1.1",

rollup.config.js

@@ -4,7 +4,6 @@ import commonjs from "@rollup/plugin-commonjs";
 import json from "@rollup/plugin-json";
 import node from "@rollup/plugin-node-resolve";
 import * as meta from "./package.json";
-import typescript from "@rollup/plugin-typescript";
 
 const filename = meta.name.split("/").pop();
 
@@ -27,7 +26,7 @@ const config = {
     indent: false,
     banner: `// ${meta.name} v${meta.version} Copyright ${copyrights.join(", ")}`
   },
-  plugins: [typescript(), commonjs(), json(), node()]
+  plugins: [commonjs(), json(), node()]
 };
 
 export default [
@@ -38,7 +37,7 @@ export default [
       name: "Plot",
       format: "umd",
       extend: true,
-      file: `build/${filename}.umd.js`,
+      file: `dist/${filename}.umd.js`,
       globals: {d3: "d3"},
       paths: {d3: d3Path}
     }

scripts/readme-to-jsdoc.ts

@@ -58,9 +58,8 @@ for (const file of glob.sync("build/**/*.js")) {
       const [, name] = match;
       const docs = docmap.get(name);
       if (!docs) throw new Error(`missing @jsdoc definition: ${name}`);
-      if (file === "build/plot.umd.js") "ignore";
-      else if (!unused.has(name)) throw new Error(`duplicate @jsdoc reference: ${name}`);
-      else unused.delete(name);
+      if (!unused.has(name)) throw new Error(`duplicate @jsdoc reference: ${name}`);
+      unused.delete(name);
       ++count;
       lines[i] = docs
         .map((line, i, lines) => (i === 0 ? `/** ${line}` : i === lines.length - 1 ? ` * ${line}\n */` : ` * ${line}`))

yarn.lock

@@ -225,14 +225,6 @@
     is-module "^1.0.0"
     resolve "^1.19.0"
 
-"@rollup/plugin-typescript@^8.3.2":
-  version "8.5.0"
-  resolved "https://registry.yarnpkg.com/@rollup/plugin-typescript/-/plugin-typescript-8.5.0.tgz#7ea11599a15b0a30fa7ea69ce3b791d41b862515"
-  integrity sha512-wMv1/scv0m/rXx21wD2IsBbJFba8wGF3ErJIr6IKRfRj49S85Lszbxb4DCo8iILpluTjk2GAAu9CoZt4G3ppgQ==
-  dependencies:
-    "@rollup/pluginutils" "^3.1.0"
-    resolve "^1.17.0"
-
 "@rollup/pluginutils@^3.0.8", "@rollup/pluginutils@^3.1.0":
   version "3.1.0"
   resolved "https://registry.yarnpkg.com/@rollup/pluginutils/-/pluginutils-3.1.0.tgz#706b4524ee6dc8b103b3c995533e5ad680c02b9b"