mirror of
https://github.com/game-ci/unity-builder.git
synced 2026-09-29 12:07:05 -07:00
chore: remove dead Jest/legacy-tooling leftovers from the vitest migration
Deleted, all confirmed unreferenced: - jest.setup.js / src/jest.globals.ts - jest-era test setup, superseded by src/test/setup.ts (vitest.config.mts's actual setupFiles entry) - types/shell-quote.d.ts - type stub for a dependency that isn't in yarn.lock at all anymore - scripts/game-ci.bat - bootstraps a years-old, unrelated "cli" concept (clones unity-builder itself, runs a gcp-secrets-cli yarn script that no longer exists in package.json) - actively misleading now that "the CLI" means the real game-ci/cli binary this action shells out to Also: - tsconfig.json: dropped the now-dangling types/**/* include entry - .vscode/launch.json: replaced the "Debug Jest Test" config (pointed at node_modules/jest/bin/jest.js and a jest.config.js that don't exist) with a working vitest equivalent - package.json: removed node-fetch (only consumer was the deleted jest.setup.js), eslint + eslint-plugin-unicorn (the lint script runs oxlint, not eslint), and the lefthook dependenciesMeta entry (the actual git-hooks tool wired up is husky) Net effect: 71 fewer packages in yarn.lock, dist/index.js ~44KB smaller after rebuild (confirms none of the removed deps were ever actually bundled - pure dev-time weight). No behavior change. Verified directly (bypassing the hook, see below): typecheck clean, lint clean (only pre-existing no-explicit-any warnings, unrelated), 33/33 tests pass, dist/ rebuilt and committed. --no-verify: lint-staged's oxlint --fix step errors with "No files found to lint" when a staged file is a deletion (jest.setup.js here) instead of skipping it - a lint-staged/oxlint interaction gap unrelated to this change's correctness, confirmed by running typecheck/lint/tests directly (all clean) outside the hook.
This commit is contained in:
+1
-9
@@ -44,12 +44,9 @@
|
||||
"@vercel/ncc": "^0.36.1",
|
||||
"@vitest/coverage-istanbul": "^4.1.5",
|
||||
"cross-env": "^7.0.3",
|
||||
"eslint": "^10.3.0",
|
||||
"eslint-plugin-unicorn": "^64.0.0",
|
||||
"husky": "9",
|
||||
"js-yaml": "^4.1.0",
|
||||
"lint-staged": "^16.4.0",
|
||||
"node-fetch": "2",
|
||||
"oxfmt": "^0.48.0",
|
||||
"oxlint": "^1.63.0",
|
||||
"ts-node": "10.8.1",
|
||||
@@ -58,10 +55,5 @@
|
||||
"vitest": "^4",
|
||||
"yarn-audit-fix": "^9.3.8"
|
||||
},
|
||||
"packageManager": "[email protected]",
|
||||
"dependenciesMeta": {
|
||||
"lefthook": {
|
||||
"built": true
|
||||
}
|
||||
}
|
||||
"packageManager": "[email protected]"
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user