From 197e1df3e0adfc6a9a0a9876fa8514516d2b1176 Mon Sep 17 00:00:00 2001 From: SuperCLI Dev Date: Sun, 12 Jul 2026 15:14:18 +0000 Subject: [PATCH 1/3] test: add plugin routing tests for bundled notion-cli plugin Covers the notion namespace wrappers (version, search, db list, page list) and the binary dependency doctor check for the notion-cli plugin introduced for #298, guarding against regressions in its plugin.json command routing. --- __tests__/notion-cli-plugin.test.js | 98 +++++++++++++++++++++++++++++ 1 file changed, 98 insertions(+) create mode 100644 __tests__/notion-cli-plugin.test.js diff --git a/__tests__/notion-cli-plugin.test.js b/__tests__/notion-cli-plugin.test.js new file mode 100644 index 000000000..c1a3ce295 --- /dev/null +++ b/__tests__/notion-cli-plugin.test.js @@ -0,0 +1,98 @@ +const fs = require("fs") +const os = require("os") +const path = require("path") +const { execSync } = require("child_process") + +const CLI = path.join(__dirname, "..", "cli", "supercli.js") + +function runNoServer(args, options = {}) { + try { + const env = { ...process.env } + delete env.SUPERCLI_SERVER + const out = execSync(`node ${CLI} ${args}`, { + encoding: "utf-8", + timeout: 15000, + env: { ...env, ...(options.env || {}) } + }) + return { ok: true, output: out.trim(), code: 0 } + } catch (err) { + return { + ok: false, + output: (err.stdout || "").trim(), + stderr: (err.stderr || "").trim(), + code: err.status + } + } +} + +function writeFakeNotionBinary(dir) { + const bin = path.join(dir, "notion") + fs.writeFileSync(bin, [ + "#!/usr/bin/env node", + "const args = process.argv.slice(2);", + "if (args[0] === '--version') { console.log('notion 0.1.0-test'); process.exit(0); }", + "if (args[0] === 'search') { console.log(JSON.stringify({ results: [{ id: 'p1', title: 'Hello' }] })); process.exit(0); }", + "if (args[0] === 'db' && args[1] === 'list') { console.log(JSON.stringify({ databases: [{ id: 'db1', title: 'Tasks' }] })); process.exit(0); }", + "if (args[0] === 'page' && args[1] === 'list') { console.log(JSON.stringify({ pages: [{ id: 'pg1', title: 'Note' }] })); process.exit(0); }", + "console.log(JSON.stringify({ ok: true, args }));" + ].join("\n"), "utf-8") + fs.chmodSync(bin, 0o755) + return bin +} + +describe("notion-cli plugin", () => { + const fakeDir = fs.mkdtempSync(path.join(os.tmpdir(), "dcli-notion-")) + const tempHome = fs.mkdtempSync(path.join(os.tmpdir(), "dcli-home-notion-")) + writeFakeNotionBinary(fakeDir) + const env = { ...process.env, PATH: `${fakeDir}:${process.env.PATH || ""}`, SUPERCLI_HOME: tempHome } + + beforeAll(() => { + runNoServer("plugins install ./plugins/notion-cli --on-conflict replace --json", { env }) + }) + + afterAll(() => { + runNoServer("plugins remove notion-cli --json", { env }) + fs.rmSync(fakeDir, { recursive: true, force: true }) + fs.rmSync(tempHome, { recursive: true, force: true }) + }) + + test("routes version command as raw text", () => { + const r = runNoServer("notion self version --json", { env }) + expect(r.ok).toBe(true) + const data = JSON.parse(r.output) + expect(data.command).toBe("notion.self.version") + expect(data.data.raw).toBe("notion 0.1.0-test") + }) + + test("routes search with positional query", () => { + const r = runNoServer('notion search run "Hello" --json', { env }) + expect(r.ok).toBe(true) + const data = JSON.parse(r.output) + expect(data.command).toBe("notion.search.run") + expect(data.data.results[0].id).toBe("p1") + }) + + test("routes db list as parsed json", () => { + const r = runNoServer("notion db list --json", { env }) + expect(r.ok).toBe(true) + const data = JSON.parse(r.output) + expect(data.command).toBe("notion.db.list") + expect(data.data.databases[0].title).toBe("Tasks") + }) + + test("routes page list as parsed json", () => { + const r = runNoServer("notion page list --json", { env }) + expect(r.ok).toBe(true) + const data = JSON.parse(r.output) + expect(data.command).toBe("notion.page.list") + expect(data.data.pages[0].id).toBe("pg1") + }) + + test("doctor reports notion binary dependency as healthy", () => { + const r = runNoServer("plugins doctor notion-cli --json", { env }) + expect(r.ok).toBe(true) + const data = JSON.parse(r.output) + expect(data.ok).toBe(true) + expect(data.checks.some(c => c.type === "binary" && c.binary === "notion" && c.ok === true)).toBe(true) + }) +}) From 4633de67240d1f292f90e42aba4f4843c410f7b5 Mon Sep 17 00:00:00 2001 From: SuperCLI Dev Date: Sun, 12 Jul 2026 15:16:10 +0000 Subject: [PATCH 2/3] test: add plugin routing tests for bundled slim plugin Covers slim namespace wrappers (version, image xray, dockerfile lint) and the binary dependency doctor check, guarding the slim plugin.json command routing against regressions. --- __tests__/slim-plugin.test.js | 91 +++++++++++++++++++++++++++++++++++ 1 file changed, 91 insertions(+) create mode 100644 __tests__/slim-plugin.test.js diff --git a/__tests__/slim-plugin.test.js b/__tests__/slim-plugin.test.js new file mode 100644 index 000000000..c022dc775 --- /dev/null +++ b/__tests__/slim-plugin.test.js @@ -0,0 +1,91 @@ +const fs = require("fs") +const os = require("os") +const path = require("path") +const { execSync } = require("child_process") + +const CLI = path.join(__dirname, "..", "cli", "supercli.js") + +function runNoServer(args, options = {}) { + try { + const env = { ...process.env } + delete env.SUPERCLI_SERVER + const out = execSync(`node ${CLI} ${args}`, { + encoding: "utf-8", + timeout: 15000, + env: { ...env, ...(options.env || {}) } + }) + return { ok: true, output: out.trim(), code: 0 } + } catch (err) { + return { + ok: false, + output: (err.stdout || "").trim(), + stderr: (err.stderr || "").trim(), + code: err.status + } + } +} + +function writeFakeSlimBinary(dir) { + const bin = path.join(dir, "slim") + fs.writeFileSync(bin, [ + "#!/usr/bin/env node", + "const args = process.argv.slice(2);", + "if (args[0] === 'version') { console.log('slim version 1.40.0-test'); process.exit(0); }", + "if (args[0] === 'xray') { console.log(JSON.stringify({ target: args[1], layers: [{ index: 0, size: 1024 }] })); process.exit(0); }", + "if (args[0] === 'lint') { console.log(JSON.stringify({ target: args[1], findings: [{ rule: 'ID.SHELL', level: 'info' }] })); process.exit(0); }", + "console.log(JSON.stringify({ ok: true, args }));" + ].join("\n"), "utf-8") + fs.chmodSync(bin, 0o755) + return bin +} + +describe("slim plugin", () => { + const fakeDir = fs.mkdtempSync(path.join(os.tmpdir(), "dcli-slim-")) + const tempHome = fs.mkdtempSync(path.join(os.tmpdir(), "dcli-home-slim-")) + writeFakeSlimBinary(fakeDir) + const env = { ...process.env, PATH: `${fakeDir}:${process.env.PATH || ""}`, SUPERCLI_HOME: tempHome } + + beforeAll(() => { + runNoServer("plugins install ./plugins/slim --on-conflict replace --json", { env }) + }) + + afterAll(() => { + runNoServer("plugins remove slim --json", { env }) + fs.rmSync(fakeDir, { recursive: true, force: true }) + fs.rmSync(tempHome, { recursive: true, force: true }) + }) + + test("routes version command as raw text", () => { + const r = runNoServer("slim self version --json", { env }) + expect(r.ok).toBe(true) + const data = JSON.parse(r.output) + expect(data.command).toBe("slim.self.version") + expect(data.data.raw).toBe("slim version 1.40.0-test") + }) + + test("routes xray with positional target", () => { + const r = runNoServer("slim image xray nginx:latest --json", { env }) + expect(r.ok).toBe(true) + const data = JSON.parse(r.output) + expect(data.command).toBe("slim.image.xray") + // xray declares parseJson:false, so the tool output is returned verbatim as raw + expect(data.data.raw).toContain("nginx:latest") + expect(JSON.parse(data.data.raw).layers[0].size).toBe(1024) + }) + + test("routes dockerfile lint with positional target", () => { + const r = runNoServer("slim dockerfile lint ./Dockerfile --json", { env }) + expect(r.ok).toBe(true) + const data = JSON.parse(r.output) + expect(data.command).toBe("slim.dockerfile.lint") + expect(data.data.raw).toContain("ID.SHELL") + }) + + test("doctor reports slim binary dependency as healthy", () => { + const r = runNoServer("plugins doctor slim --json", { env }) + expect(r.ok).toBe(true) + const data = JSON.parse(r.output) + expect(data.ok).toBe(true) + expect(data.checks.some(c => c.type === "binary" && c.binary === "slim" && c.ok === true)).toBe(true) + }) +}) From d635738c1f2303528cd87ce207b1b4f137723dab Mon Sep 17 00:00:00 2001 From: SuperCLI Dev Date: Sun, 12 Jul 2026 15:16:43 +0000 Subject: [PATCH 3/3] test: add plugin routing tests for bundled diun plugin Covers diun namespace wrappers (version, check-config) and the binary dependency doctor check, guarding the diun plugin.json command routing against regressions. --- __tests__/diun-plugin.test.js | 80 +++++++++++++++++++++++++++++++++++ 1 file changed, 80 insertions(+) create mode 100644 __tests__/diun-plugin.test.js diff --git a/__tests__/diun-plugin.test.js b/__tests__/diun-plugin.test.js new file mode 100644 index 000000000..38c541469 --- /dev/null +++ b/__tests__/diun-plugin.test.js @@ -0,0 +1,80 @@ +const fs = require("fs") +const os = require("os") +const path = require("path") +const { execSync } = require("child_process") + +const CLI = path.join(__dirname, "..", "cli", "supercli.js") + +function runNoServer(args, options = {}) { + try { + const env = { ...process.env } + delete env.SUPERCLI_SERVER + const out = execSync(`node ${CLI} ${args}`, { + encoding: "utf-8", + timeout: 15000, + env: { ...env, ...(options.env || {}) } + }) + return { ok: true, output: out.trim(), code: 0 } + } catch (err) { + return { + ok: false, + output: (err.stdout || "").trim(), + stderr: (err.stderr || "").trim(), + code: err.status + } + } +} + +function writeFakeDiunBinary(dir) { + const bin = path.join(dir, "diun") + fs.writeFileSync(bin, [ + "#!/usr/bin/env node", + "const args = process.argv.slice(2);", + "if (args[0] === 'version') { console.log('diun version 4.28.0-test'); process.exit(0); }", + "if (args[0] === 'check-config') { console.log('configuration is valid'); process.exit(0); }", + "console.log(JSON.stringify({ ok: true, args }));" + ].join("\n"), "utf-8") + fs.chmodSync(bin, 0o755) + return bin +} + +describe("diun plugin", () => { + const fakeDir = fs.mkdtempSync(path.join(os.tmpdir(), "dcli-diun-")) + const tempHome = fs.mkdtempSync(path.join(os.tmpdir(), "dcli-home-diun-")) + writeFakeDiunBinary(fakeDir) + const env = { ...process.env, PATH: `${fakeDir}:${process.env.PATH || ""}`, SUPERCLI_HOME: tempHome } + + beforeAll(() => { + runNoServer("plugins install ./plugins/diun --on-conflict replace --json", { env }) + }) + + afterAll(() => { + runNoServer("plugins remove diun --json", { env }) + fs.rmSync(fakeDir, { recursive: true, force: true }) + fs.rmSync(tempHome, { recursive: true, force: true }) + }) + + test("routes version command as raw text", () => { + const r = runNoServer("diun self version --json", { env }) + expect(r.ok).toBe(true) + const data = JSON.parse(r.output) + expect(data.command).toBe("diun.self.version") + expect(data.data.raw).toBe("diun version 4.28.0-test") + }) + + test("routes check-config command as raw text", () => { + const r = runNoServer("diun self check-config --json", { env }) + expect(r.ok).toBe(true) + const data = JSON.parse(r.output) + expect(data.command).toBe("diun.self.check-config") + expect(data.data.raw).toBe("configuration is valid") + }) + + test("doctor reports diun binary dependency as healthy", () => { + const r = runNoServer("plugins doctor diun --json", { env }) + expect(r.ok).toBe(true) + const data = JSON.parse(r.output) + expect(data.ok).toBe(true) + expect(data.checks.some(c => c.type === "binary" && c.binary === "diun" && c.ok === true)).toBe(true) + }) +})