chore: address maintainer review feedback (round 2)

- README: fix stale [tests] link pointing at the never-existing
  `src/__tests__/index.js` — should be `test/index.js`.
- .gitignore: add `.DS_Store` (macOS noise).
- scripts/benchmark-jison.ts: dynamic imports → static imports, as
  the maintainer asked. Reviewer noted dynamic imports aren't
  needed once Node 20 compat is dropped.
- tsconfig: extract a shared tsconfig.base.json; tsconfig.json (lint
  / IDE, noEmit) and tsconfig.build.json (emit dist/) now `extends`
  it, removing ~10 lines of duplicated compiler options across the
  two children.

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

Author: dkryaklin

.gitignore

@@ -1,4 +1,5 @@
 node_modules
+.DS_Store
 test/fixtures/*.actual.css
 src/parser.js
 yarn-error.log

README.md

@@ -258,5 +258,5 @@ npm test
 [PostCSS]: https://github.com/postcss
 [PostCSS Calc]: https://github.com/postcss/postcss-calc
 [PostCSS Custom Properties]: https://github.com/postcss/postcss-custom-properties
-[tests]: src/__tests__/index.js
+[tests]: test/index.js
 [W3C calc() implementation]: https://www.w3.org/TR/css3-values/#calc-notation

scripts/benchmark-jison.ts

@@ -5,17 +5,16 @@ import { join, dirname } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import postcss, { type AcceptedPlugin } from 'postcss';
 
+import jisonPlugin from '../src/index.js';
+import prattPlugin from '../src/pratt/src/plugin/plugin.ts';
+
 interface BenchResult {
   name: string;
   perRun: number;
   total: number;
 }
 
 async function main(): Promise<void> {
-  // Dynamic imports so CJS/ESM interop works under tsx.
-  const jisonPlugin = (await import('../src/index.js')).default as () => AcceptedPlugin;
-  const prattPlugin = (await import('../src/pratt/src/plugin/plugin.ts')).default as () => AcceptedPlugin;
-
   const HERE = dirname(fileURLToPath(import.meta.url));
   const CORPUS_DIR = join(HERE, '..', 'src', 'pratt', 'test', 'corpus');
 
@@ -48,8 +47,8 @@ async function main(): Promise<void> {
   console.log(`CSS size: ${css.length} bytes, ${calcs.length} declarations`);
   console.log(`Iterations: ${ITERATIONS} each\n`);
 
-  const jison = await bench('jison', jisonPlugin);
-  const pratt = await bench('pratt', prattPlugin);
+  const jison = await bench('jison', jisonPlugin as () => AcceptedPlugin);
+  const pratt = await bench('pratt', prattPlugin as () => AcceptedPlugin);
 
   console.log(`  ${jison.name}:  ${jison.perRun.toFixed(2)}ms / run  (${jison.total.toFixed(0)}ms total)`);
   console.log(`  ${pratt.name}:  ${pratt.perRun.toFixed(2)}ms / run  (${pratt.total.toFixed(0)}ms total)`);

tsconfig.base.json

@@ -0,0 +1,12 @@
+{
+  "compilerOptions": {
+    "target": "es2022",
+    "lib": ["es2022"],
+    "strict": true,
+    "noUncheckedIndexedAccess": true,
+    "noImplicitOverride": true,
+    "allowImportingTsExtensions": true,
+    "skipLibCheck": true,
+    "types": ["node"]
+  }
+}

tsconfig.build.json

@@ -1,20 +1,14 @@
+// Build config. Emits dist/ with declarations from the shipped sources only.
 {
+  "extends": "./tsconfig.base.json",
   "compilerOptions": {
-    "target": "es2022",
-    "lib": ["es2022"],
     "module": "nodenext",
     "moduleResolution": "nodenext",
-    "strict": true,
-    "noUncheckedIndexedAccess": true,
-    "noImplicitOverride": true,
     "esModuleInterop": true,
-    "allowImportingTsExtensions": true,
     "rewriteRelativeImportExtensions": true,
     "declaration": true,
-    "skipLibCheck": true,
     "outDir": "dist",
-    "rootDir": ".",
-    "types": ["node"]
+    "rootDir": "."
   },
   "include": ["src/index.ts", "src/pratt/src/**/*.ts"],
   "exclude": ["src/pratt/src/plugin/plugin-csstools.ts"]

tsconfig.json

@@ -1,17 +1,11 @@
+// Lint / IDE / typecheck config. Covers src + scripts + tests, emits nothing.
 {
+  "extends": "./tsconfig.base.json",
   "compilerOptions": {
-    "target": "es2022",
-    "lib": ["es2022"],
     "module": "esnext",
     "moduleResolution": "bundler",
-    "strict": true,
-    "noUncheckedIndexedAccess": true,
-    "noImplicitOverride": true,
-    "allowImportingTsExtensions": true,
     "verbatimModuleSyntax": true,
-    "noEmit": true,
-    "skipLibCheck": true,
-    "types": ["node"]
+    "noEmit": true
   },
   "include": ["src/index.ts", "src/pratt/**/*.ts", "scripts/**/*.ts"]
 }