Fix pkg name inconsistencies

webpro · webpro · commit 544c3e68d72d · 2026-04-25T14:46:52.000+02:00
diff --git a/packages/knip/fixtures/catalog-pnpm/packages/app/package.json b/packages/knip/fixtures/catalog-pnpm/packages/app/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "app",
+  "name": "@fixtures/catalog-pnpm__app",
   "dependencies": {
     "react": "catalog:"
   },
diff --git a/packages/knip/fixtures/catalog-yarn/packages/app/package.json b/packages/knip/fixtures/catalog-yarn/packages/app/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "app",
+  "name": "@fixtures/catalog-yarn__app",
   "dependencies": {
     "solid-js": "catalog:"
   },
diff --git a/packages/knip/fixtures/configuration-hints2/lib/package.json b/packages/knip/fixtures/configuration-hints2/lib/package.json
@@ -1,3 +1,3 @@
 {
-  "name": "@fixtures/configuration-hints__lib"
+  "name": "@fixtures/configuration-hints2__lib"
 }
diff --git a/packages/knip/fixtures/configuration-hints2/package.json b/packages/knip/fixtures/configuration-hints2/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/configuration-hints",
+  "name": "@fixtures/configuration-hints2",
   "workspaces": [
     "lib"
   ]
diff --git a/packages/knip/fixtures/export-spread/package.json b/packages/knip/fixtures/export-spread/package.json
@@ -1,4 +1,4 @@
 {
-  "name": "@fixed/export-spread",
+  "name": "@fixtures/export-spread",
   "type": "module"
 }
diff --git a/packages/knip/fixtures/imports-dynamic-access/package.json b/packages/knip/fixtures/imports-dynamic-access/package.json
@@ -1,4 +1,4 @@
 {
-  "name": "@fixtures/imports-dynamic",
+  "name": "@fixtures/imports-dynamic-access",
   "type": "module"
 }
diff --git a/packages/knip/fixtures/imports-namespace-jsx/package.json b/packages/knip/fixtures/imports-namespace-jsx/package.json
@@ -1,3 +1,3 @@
 {
-  "name": "imports-namespace-jsx"
+  "name": "@fixtures/imports-namespace-jsx"
 }
diff --git a/packages/knip/fixtures/package-entry-bare/package.json b/packages/knip/fixtures/package-entry-bare/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixture/package-entry-bare",
+  "name": "@fixtures/package-entry-bare",
   "main": "bare-dep/entry",
   "dependencies": {
     "bare-dep": "*"
diff --git a/packages/knip/fixtures/plugin-negated-entry-globs/package.json b/packages/knip/fixtures/plugin-negated-entry-globs/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/plugin-entry-globs",
+  "name": "@fixtures/plugin-negated-entry-globs",
   "type": "module",
   "dependencies": {
     "astro": "*"
diff --git a/packages/knip/fixtures/plugins/biome-workspace/packages/stub/package.json b/packages/knip/fixtures/plugins/biome-workspace/packages/stub/package.json
@@ -1,3 +1,3 @@
 {
-    "name": "stub-package"
-}
+  "name": "@plugins/biome-workspace__stub"
+}
diff --git a/packages/knip/fixtures/plugins/eslint5/package.json b/packages/knip/fixtures/plugins/eslint5/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@plugins/eslint2",
+  "name": "@plugins/eslint5",
   "devDependencies": {
     "@eslint/js": "*",
     "eslint-import-resolver-typescript": "*",
diff --git a/packages/knip/fixtures/plugins/expressive-code/package.json b/packages/knip/fixtures/plugins/expressive-code/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/expressive-code-nextjs",
+  "name": "@plugins/expressive-code",
   "scripts": {
     "dev": "next dev",
     "build": "next build"
diff --git a/packages/knip/fixtures/plugins/expressive-code2/package.json b/packages/knip/fixtures/plugins/expressive-code2/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/expressive-code-astro",
+  "name": "@plugins/expressive-code2",
   "type": "module",
   "scripts": {
     "dev": "astro dev",
diff --git a/packages/knip/fixtures/plugins/expressive-code3/package.json b/packages/knip/fixtures/plugins/expressive-code3/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/expressive-code-starlight",
+  "name": "@plugins/expressive-code3",
   "dependencies": {
     "@astrojs/starlight": "*",
     "@expressive-code/plugin-line-numbers": "*",
diff --git a/packages/knip/fixtures/plugins/lint-staged-comment/package.json b/packages/knip/fixtures/plugins/lint-staged-comment/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/lint-staged-comment",
+  "name": "@plugins/lint-staged-comment",
   "devDependencies": {
     "lint-staged": "*"
   }
diff --git a/packages/knip/fixtures/plugins/prettier-args/package.json b/packages/knip/fixtures/plugins/prettier-args/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "prettier-args-fixture",
+  "name": "@plugins/prettier-args",
   "devDependencies": {
     "prettier": "*"
   },
diff --git a/packages/knip/fixtures/plugins/prettier-json5/package.json b/packages/knip/fixtures/plugins/prettier-json5/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/prettier-json5",
+  "name": "@plugins/prettier-json5",
   "devDependencies": {
     "prettier": "*"
   }
diff --git a/packages/knip/fixtures/plugins/prettier-reexport/package.json b/packages/knip/fixtures/plugins/prettier-reexport/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/prettier-reexport",
+  "name": "@plugins/prettier-reexport",
   "type": "module",
   "scripts": {
     "format": "prettier"
diff --git a/packages/knip/fixtures/plugins/prisma2/package.json b/packages/knip/fixtures/plugins/prisma2/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@plugins/prisma-2",
+  "name": "@plugins/prisma2",
   "scripts": {
     "generate": "prisma generate"
   },
diff --git a/packages/knip/fixtures/plugins/raycast/package.json b/packages/knip/fixtures/plugins/raycast/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/raycast",
+  "name": "@plugins/raycast",
   "dependencies": {
     "@raycast/api": "^1.0.0"
   },
diff --git a/packages/knip/fixtures/plugins/react-router-with-server-entry/package.json b/packages/knip/fixtures/plugins/react-router-with-server-entry/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@plugins/react-router",
+  "name": "@plugins/react-router-with-server-entry",
   "devDependencies": {
     "@react-router/dev": "*"
   }
diff --git a/packages/knip/fixtures/plugins/storybook2/package.json b/packages/knip/fixtures/plugins/storybook2/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@plugins/storybook",
+  "name": "@plugins/storybook2",
   "scripts": {
     "storybook": "storybook"
   },
diff --git a/packages/knip/fixtures/plugins/stylelint3/package.json b/packages/knip/fixtures/plugins/stylelint3/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/stylelint3",
+  "name": "@plugins/stylelint3",
   "devDependencies": {
     "stylelint": "*",
     "stylelint-config-recommended": "*",
diff --git a/packages/knip/fixtures/plugins/svelte/package.json b/packages/knip/fixtures/plugins/svelte/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/svelte",
+  "name": "@plugins/svelte",
   "type": "module",
   "devDependencies": {
     "@sveltejs/adapter-auto": "*",
diff --git a/packages/knip/fixtures/plugins/sveltejs-package/package.json b/packages/knip/fixtures/plugins/sveltejs-package/package.json
@@ -1,10 +1,16 @@
 {
-  "name": "@fixtures/sveltejs-package",
+  "name": "@plugins/sveltejs-package",
   "type": "module",
   "exports": {
-    ".": { "svelte": "./dist/index.js" },
-    "./lib/*": { "svelte": "./dist/*.js" },
-    "./ui/*": { "svelte": "./dist/ui/*/index.js" }
+    ".": {
+      "svelte": "./dist/index.js"
+    },
+    "./lib/*": {
+      "svelte": "./dist/*.js"
+    },
+    "./ui/*": {
+      "svelte": "./dist/ui/*/index.js"
+    }
   },
   "scripts": {
     "build": "svelte-package"
diff --git a/packages/knip/fixtures/plugins/sveltejs-package2/package.json b/packages/knip/fixtures/plugins/sveltejs-package2/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/sveltejs-package2",
+  "name": "@plugins/sveltejs-package2",
   "type": "module",
   "main": "./build/index.js",
   "scripts": {
diff --git a/packages/knip/fixtures/plugins/sveltekit/package.json b/packages/knip/fixtures/plugins/sveltekit/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/sveltekit",
+  "name": "@plugins/sveltekit",
   "type": "module",
   "devDependencies": {
     "@sveltejs/adapter-auto": "*",
diff --git a/packages/knip/fixtures/plugins/sveltekit2/package.json b/packages/knip/fixtures/plugins/sveltekit2/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/sveltekit2",
+  "name": "@plugins/sveltekit2",
   "type": "module",
   "devDependencies": {
     "@sveltejs/kit": "*",
diff --git a/packages/knip/fixtures/plugins/tsgo/package.json b/packages/knip/fixtures/plugins/tsgo/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "tsgo",
+  "name": "@plugins/tsgo",
   "scripts": {
     "build": "tsgo"
   },
diff --git a/packages/knip/fixtures/plugins/vitest10/package.json b/packages/knip/fixtures/plugins/vitest10/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/vitest10",
+  "name": "@plugins/vitest10",
   "scripts": {
     "test": "vitest"
   },
diff --git a/packages/knip/fixtures/plugins/vitest9/package.json b/packages/knip/fixtures/plugins/vitest9/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/vitest9",
+  "name": "@plugins/vitest9",
   "workspaces": [
     "packages/*"
   ],
diff --git a/packages/knip/fixtures/plugins/vitest9/packages/client/package.json b/packages/knip/fixtures/plugins/vitest9/packages/client/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/vitest9-client",
+  "name": "@plugins/vitest9__client",
   "devDependencies": {
     "vitest": "*"
   }
diff --git a/packages/knip/fixtures/plugins/webpack-reexport/package.json b/packages/knip/fixtures/plugins/webpack-reexport/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/webpack-reexport",
+  "name": "@plugins/webpack-reexport",
   "scripts": {
     "build": "webpack"
   },
diff --git a/packages/knip/fixtures/session-re-exports/app/package.json b/packages/knip/fixtures/session-re-exports/app/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/session-re-exports-app",
+  "name": "@fixtures/session-re-exports__app",
   "dependencies": {
     "pkg": "file:../pkg"
   }
diff --git a/packages/knip/fixtures/session-re-exports/pkg/package.json b/packages/knip/fixtures/session-re-exports/pkg/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/session-re-exports-pkg",
+  "name": "@fixtures/session-re-exports__pkg",
   "exports": {
     "./public": "./public/barrel.ts",
     "./internal": "./internal/consumer-1.ts"
diff --git a/packages/knip/fixtures/tsconfig-include-dir/package.json b/packages/knip/fixtures/tsconfig-include-dir/package.json
@@ -1,3 +1,3 @@
 {
-  "name": "tsconfig-include-dir"
+  "name": "@fixtures/tsconfig-include-dir"
 }
diff --git a/packages/knip/fixtures/tsconfig-rootdirs-jsonc/package.json b/packages/knip/fixtures/tsconfig-rootdirs-jsonc/package.json
@@ -1,4 +1,4 @@
 {
-  "name": "tsconfig-rootdirs-jsonc",
+  "name": "@fixtures/tsconfig-rootdirs-jsonc",
   "private": true
 }
diff --git a/packages/knip/fixtures/workspaces-bun-test/package.json b/packages/knip/fixtures/workspaces-bun-test/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@knip/workspaces-bun-test",
+  "name": "@fixtures/workspaces-bun-test",
   "scripts": {
     "test": "bun test"
   },
diff --git a/packages/knip/fixtures/workspaces-bun-test/packages/lib/package.json b/packages/knip/fixtures/workspaces-bun-test/packages/lib/package.json
@@ -1,3 +1,3 @@
 {
-  "name": "@knip/workspaces-bun-test__lib"
+  "name": "@fixtures/workspaces-bun-test__lib"
 }
diff --git a/packages/knip/fixtures/workspaces-circular-symlinks/package.json b/packages/knip/fixtures/workspaces-circular-symlinks/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/workspaces-symlinks",
+  "name": "@fixtures/workspaces-circular-symlinks",
   "workspaces": [
     "packages/*"
   ]
diff --git a/packages/knip/fixtures/workspaces-circular-symlinks/packages/lib-a/package.json b/packages/knip/fixtures/workspaces-circular-symlinks/packages/lib-a/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/lib-a",
+  "name": "@fixtures/workspaces-circular-symlinks__lib-a",
   "version": "1.0.0",
   "main": "./index.ts",
   "exports": {
diff --git a/packages/knip/fixtures/workspaces-circular-symlinks/packages/lib-b/package.json b/packages/knip/fixtures/workspaces-circular-symlinks/packages/lib-b/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/lib-b",
+  "name": "@fixtures/workspaces-circular-symlinks__lib-b",
   "version": "1.0.0",
   "main": "./index.ts",
   "exports": {
diff --git a/packages/knip/fixtures/workspaces-cross-ref/package.json b/packages/knip/fixtures/workspaces-cross-ref/package.json
@@ -1,6 +1,8 @@
 {
-  "name": "@fixtures/ws-cross-ref",
-  "workspaces": ["scripts"],
+  "name": "@fixtures/workspaces-cross-ref",
+  "workspaces": [
+    "scripts"
+  ],
   "scripts": {
     "generate": "node scripts/generate.ts",
     "test": "node --test"
diff --git a/packages/knip/fixtures/workspaces-cross-ref/scripts/package.json b/packages/knip/fixtures/workspaces-cross-ref/scripts/package.json
@@ -1,4 +1,4 @@
 {
-  "name": "@fixtures/ws-cross-ref-scripts",
+  "name": "@fixtures/workspaces-cross-ref__scripts",
   "private": true
 }
diff --git a/packages/knip/fixtures/workspaces-node-test/package.json b/packages/knip/fixtures/workspaces-node-test/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@knip/workspaces-node-test",
+  "name": "@fixtures/workspaces-node-test",
   "scripts": {
     "test": "node --test"
   },
diff --git a/packages/knip/fixtures/workspaces-node-test/packages/lib/package.json b/packages/knip/fixtures/workspaces-node-test/packages/lib/package.json
@@ -1,3 +1,3 @@
 {
-  "name": "@knip/workspaces-node-test__lib"
+  "name": "@fixtures/workspaces-node-test__lib"
 }
diff --git a/packages/knip/fixtures/workspaces-root/app/package.json b/packages/knip/fixtures/workspaces-root/app/package.json
@@ -1,4 +1,4 @@
 {
-  "name": "@fixtures/workspaces__frontend",
+  "name": "@fixtures/workspaces-root__app",
   "main": "./index.js"
 }
diff --git a/packages/knip/fixtures/workspaces-root/package.json b/packages/knip/fixtures/workspaces-root/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@fixtures/workspaces",
+  "name": "@fixtures/workspaces-root",
   "workspaces": [
     "app"
   ],
diff --git a/packages/knip/scripts/verify-fixtures.ts b/packages/knip/scripts/verify-fixtures.ts
@@ -40,7 +40,12 @@ async function main(isFix = false) {
       else tree[parts.at(0)] = {};
     }
 
-    const pkg = JSON.parse(readFileSync(absPath, 'utf8'));
+    let pkg: { name?: string };
+    try {
+      pkg = JSON.parse(readFileSync(absPath, 'utf8').replace(/^﻿/, ''));
+    } catch {
+      continue;
+    }
     const name = pkg.name;
 
     if (name && !name.startsWith('x-') && !name.includes('/x-') && !name.includes('-x-') && name !== expectedName) {

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`{`
`2`		`- "name": "app",`
	`2`	`+ "name": "@fixtures/catalog-pnpm__app",`
`3`	`3`	`"dependencies": {`
`4`	`4`	`"react": "catalog:"`
`5`	`5`	`},`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`{`
`2`		`- "name": "app",`
	`2`	`+ "name": "@fixtures/catalog-yarn__app",`
`3`	`3`	`"dependencies": {`
`4`	`4`	`"solid-js": "catalog:"`
`5`	`5`	`},`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`	`1`	`{`
`2`		`- "name": "@fixtures/configuration-hints__lib"`
	`2`	`+ "name": "@fixtures/configuration-hints2__lib"`
`3`	`3`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`	`1`	`{`
`2`		`- "name": "@fixed/export-spread",`
	`2`	`+ "name": "@fixtures/export-spread",`
`3`	`3`	`"type": "module"`
`4`	`4`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`	`1`	`{`
`2`		`- "name": "@fixtures/imports-dynamic",`
	`2`	`+ "name": "@fixtures/imports-dynamic-access",`
`3`	`3`	`"type": "module"`
`4`	`4`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`	`1`	`{`
`2`		`- "name": "imports-namespace-jsx"`
	`2`	`+ "name": "@fixtures/imports-namespace-jsx"`
`3`	`3`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`{`
`2`		`- "name": "@fixture/package-entry-bare",`
	`2`	`+ "name": "@fixtures/package-entry-bare",`
`3`	`3`	`"main": "bare-dep/entry",`
`4`	`4`	`"dependencies": {`
`5`	`5`	`"bare-dep": "*"`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`{`
`2`		`- "name": "@fixtures/plugin-entry-globs",`
	`2`	`+ "name": "@fixtures/plugin-negated-entry-globs",`
`3`	`3`	`"type": "module",`
`4`	`4`	`"dependencies": {`
`5`	`5`	`"astro": "*"`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`	`1`	`{`
`2`		`- "name": "stub-package"`
`3`		`-}`
	`2`	`+ "name": "@plugins/biome-workspace__stub"`
	`3`	`+}`