From 70c61df3c98c6b21fe2235faa9f546d18a567412 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Tue, 2 Jun 2026 22:49:29 +0000 Subject: [PATCH 1/4] Bump commander from 12.1.0 to 15.0.0 Bumps [commander](https://github.com/tj/commander.js) from 12.1.0 to 15.0.0. - [Release notes](https://github.com/tj/commander.js/releases) - [Changelog](https://github.com/tj/commander.js/blob/master/CHANGELOG.md) - [Commits](https://github.com/tj/commander.js/compare/v12.1.0...v15.0.0) --- updated-dependencies: - dependency-name: commander dependency-version: 15.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] --- package-lock.json | 16 ++++++++-------- package.json | 2 +- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/package-lock.json b/package-lock.json index f591d65..7e61003 100644 --- a/package-lock.json +++ b/package-lock.json @@ -9,7 +9,7 @@ "version": "0.2.0", "license": "MIT", "dependencies": { - "commander": "^12.0.0", + "commander": "^15.0.0", "node-fetch": "^2.6.13" }, "bin": { @@ -548,11 +548,11 @@ } }, "node_modules/commander": { - "version": "12.1.0", - "resolved": "https://registry.npmjs.org/commander/-/commander-12.1.0.tgz", - "integrity": "sha512-Vw8qHK3bZM9y/P10u3Vib8o/DdkvA2OtPtZvD871QKjy74Wj1WSKFILMPRPSdUSx5RFK1arlJzEtA4PkFgnbuA==", + "version": "15.0.0", + "resolved": "https://registry.npmjs.org/commander/-/commander-15.0.0.tgz", + "integrity": "sha512-z67u4ZhzCL/Tydu1lJARtEZYWbWaN7oYLHbsuzocr6y4N6WZAagG3RQ4FW61V1/0+jImpj293XfrcYnd1qxtPg==", "engines": { - "node": ">=18" + "node": ">=22.12.0" } }, "node_modules/concat-map": { @@ -2094,9 +2094,9 @@ } }, "commander": { - "version": "12.1.0", - "resolved": "https://registry.npmjs.org/commander/-/commander-12.1.0.tgz", - "integrity": "sha512-Vw8qHK3bZM9y/P10u3Vib8o/DdkvA2OtPtZvD871QKjy74Wj1WSKFILMPRPSdUSx5RFK1arlJzEtA4PkFgnbuA==" + "version": "15.0.0", + "resolved": "https://registry.npmjs.org/commander/-/commander-15.0.0.tgz", + "integrity": "sha512-z67u4ZhzCL/Tydu1lJARtEZYWbWaN7oYLHbsuzocr6y4N6WZAagG3RQ4FW61V1/0+jImpj293XfrcYnd1qxtPg==" }, "concat-map": { "version": "0.0.1", diff --git a/package.json b/package.json index f7da7c3..6616d82 100644 --- a/package.json +++ b/package.json @@ -27,7 +27,7 @@ "shielded": "./dist/bin.js" }, "dependencies": { - "commander": "^12.0.0", + "commander": "^15.0.0", "node-fetch": "^2.6.13" } } From 26a66a4f5def2dc960880231450adec491f8057f Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 3 Jun 2026 11:27:47 +0000 Subject: [PATCH 2/4] Add commander flag parsing compatibility tests --- package.json | 3 ++- src/flags.test.ts | 48 +++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 50 insertions(+), 1 deletion(-) create mode 100644 src/flags.test.ts diff --git a/package.json b/package.json index 6616d82..8916a87 100644 --- a/package.json +++ b/package.json @@ -6,7 +6,8 @@ "scripts": { "test": "echo \"Error: no test specified\" && exit 1", "build": "tsc && chmod +x dist/bin.js", - "lint": "npx eslint ." + "lint": "npx eslint .", + "test:flags": "npm run build && node --test dist/flags.test.js" }, "author": { "name": "Jesse G. Donat", diff --git a/src/flags.test.ts b/src/flags.test.ts new file mode 100644 index 0000000..d31e7d0 --- /dev/null +++ b/src/flags.test.ts @@ -0,0 +1,48 @@ +import assert from 'node:assert/strict'; +import { spawnSync } from 'node:child_process'; +import path from 'node:path'; +import test from 'node:test'; + +const binPath = path.resolve(__dirname, 'bin.js'); +const parseErrorPattern = /error:\s+(unknown option|too many arguments|missing required argument)/i; + +const runCli = (args: string[]) => spawnSync(process.execPath, [binPath, ...args], { +encoding: 'utf8', +env: { +...process.env, +SHIELDED_TOKEN: 'test-token' +}, +timeout: 5000 +}); + +void test('help output includes all defined flags', () => { +const result = runCli(['--help']); + +assert.equal(result.status, 0); +assert.ok(result.stdout.includes('-e, --endpoint ')); +assert.ok(result.stdout.includes('-c, --color ')); +assert.ok(result.stdout.includes('-T, --title ')); +assert.ok(result.stdout.includes('-x, --text <text>')); +}); + +void test('long-form flags parse without commander errors', () => { +const result = runCli([ +'--endpoint', 'http://127.0.0.1:9', +'--color', '00AA33', +'--title', 'Last Build', +'--text', 'ok' +]); + +assert.doesNotMatch(`${result.stdout}\n${result.stderr}`, parseErrorPattern); +}); + +void test('short-form flags parse without commander errors', () => { +const result = runCli([ +'-e', 'http://127.0.0.1:9', +'-c', '00AA33', +'-T', 'Last Build', +'-x', 'ok' +]); + +assert.doesNotMatch(`${result.stdout}\n${result.stderr}`, parseErrorPattern); +}); From ea4f2a68f0a934d29634e31e21c489237349f1f2 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 3 Jun 2026 11:28:59 +0000 Subject: [PATCH 3/4] Run flag parsing tests via npm test --- package.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package.json b/package.json index 8916a87..43b4464 100644 --- a/package.json +++ b/package.json @@ -4,7 +4,7 @@ "description": "Shielded.dev JS SDK", "main": "dist/sdk.js", "scripts": { - "test": "echo \"Error: no test specified\" && exit 1", + "test": "npm run test:flags", "build": "tsc && chmod +x dist/bin.js", "lint": "npx eslint .", "test:flags": "npm run build && node --test dist/flags.test.js" From 8740555ef08bfaeac3ce5788624a6c8d3a6a63f9 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 3 Jun 2026 11:29:55 +0000 Subject: [PATCH 4/4] Refine commander flag test harness --- src/flags.test.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/flags.test.ts b/src/flags.test.ts index d31e7d0..4f05a3c 100644 --- a/src/flags.test.ts +++ b/src/flags.test.ts @@ -4,6 +4,7 @@ import path from 'node:path'; import test from 'node:test'; const binPath = path.resolve(__dirname, 'bin.js'); +const CLI_TEST_TIMEOUT = 5000; const parseErrorPattern = /error:\s+(unknown option|too many arguments|missing required argument)/i; const runCli = (args: string[]) => spawnSync(process.execPath, [binPath, ...args], { @@ -12,7 +13,7 @@ env: { ...process.env, SHIELDED_TOKEN: 'test-token' }, -timeout: 5000 +timeout: CLI_TEST_TIMEOUT }); void test('help output includes all defined flags', () => {