fix(npm): preserve -v passthrough

Summary:
- Stop treating -v as an npm wrapper version fallback.
- Keep wrapper fallback for --version and -V.
- Add a Node regression test for wrapper version flag detection.

Test plan:
- npm test from npm/deepseek-tui
- git diff --check origin/main...HEAD

Supersedes #959.
This commit is contained in:
Hunter Bown
2026-05-07 03:37:22 -05:00
committed by GitHub
parent dd822edcfc
commit 593ed2f900
3 changed files with 17 additions and 4 deletions
+3 -1
View File
@@ -30,7 +30,8 @@
"release:check": "node scripts/verify-release-assets.js",
"postinstall": "node scripts/install.js",
"prepublishOnly": "node scripts/verify-release-assets.js",
"prepack": "node scripts/install.js"
"prepack": "node scripts/install.js",
"test": "node --test test/*.test.js"
},
"engines": {
"node": ">=18"
@@ -42,6 +43,7 @@
"files": [
"bin/*.js",
"scripts/*.js",
"test/*.js",
"README.md",
"package.json"
]
+3 -3
View File
@@ -3,9 +3,8 @@ const { getBinaryPath } = require("./install");
const pkg = require("../package.json");
function isVersionFlag() {
const args = process.argv.slice(2);
return args.includes("--version") || args.includes("-v") || args.includes("-V");
function isVersionFlag(args = process.argv.slice(2)) {
return args.includes("--version") || args.includes("-V");
}
function handleVersionFallback(binaryName) {
@@ -46,6 +45,7 @@ module.exports = {
run,
runDeepseek,
runDeepseekTui,
_internal: { isVersionFlag },
};
if (require.main === module) {
+11
View File
@@ -0,0 +1,11 @@
const assert = require("node:assert/strict");
const test = require("node:test");
const { _internal } = require("../scripts/run");
test("version fallback handles only version flags", () => {
assert.equal(_internal.isVersionFlag(["--version"]), true);
assert.equal(_internal.isVersionFlag(["-V"]), true);
assert.equal(_internal.isVersionFlag(["-v"]), false);
assert.equal(_internal.isVersionFlag(["--verbose"]), false);
});