@wirecat/cli-template 0.13.2 → 0.15.0

package/src/program.test.ts

@@ -1,10 +1,13 @@
 import { chmodSync, mkdirSync, readFileSync, rmSync, statSync, writeFileSync } from "node:fs"
 import { join } from "node:path"
+import { annotate } from "@wirecat/cli-base/commands"
+import { REMOTE_READ, run as runShared } from "@wirecat/cli-core/cli"
+import { Command } from "commander"
 import { expect, it, vi } from "vitest"
 import { doctorReport } from "./commands/doctor.js"
 import { configuration } from "./config.js"
 import { IDENTITY } from "./identity.js"
-import { run } from "./program.js"
+import { definition, run } from "./program.js"
 
 const ran = async (argv: string[]) => {
   const out: string[] = []
@@ -177,3 +180,85 @@ it("a failed run keeps the words typed, and record_input false masks them", asyn
     expect((await ran(["personal", "config", "unset", "record_input"])).code).toBe(0)
   }
 })
+
+it("runs list --compact prints one plain line per run and the next page as a # line, not JSON", async () => {
+  for (const _ of [1, 2]) expect((await ran(["--no-such-flag"])).code).toBe(2)
+  const { code, out } = await ran(["runs", "list", "--limit", "1", "--compact"])
+  expect(code).toBe(0)
+  expect(out.trim().split("\n")).toEqual([
+    expect.stringContaining("failed"),
+    "# more: clidev runs list --page 2 --limit 1 --compact",
+  ])
+  expect(() => JSON.parse(out)).toThrow()
+})
+
+it("config migrate writes the missing defaults, keeps the owner's value, and repeats as a no-op", async () => {
+  const file = join(process.env.CLIDEV_CONFIG_DIR ?? "", "config.json")
+  mkdirSync(join(file, ".."), { recursive: true })
+  writeFileSync(file, '{ "defaults": { "limit": 5 } }')
+  try {
+    const first = await ran(["config", "migrate", "--json"])
+    expect(first.code).toBe(0)
+    expect(json(first.out).item.added).toContain("defaults.record")
+    expect(JSON.parse(readFileSync(file, "utf8")).defaults).toMatchObject({ limit: 5, record: expect.any(Boolean) })
+    const again = await ran(["config", "migrate", "--json"])
+    expect(json(again.out).item.added).toEqual([])
+  } finally {
+    rmSync(file)
+  }
+})
+
+it("config repair --dry-run changes nothing, and config repair fixes the JSON with a backup", async () => {
+  const dir = process.env.CLIDEV_CONFIG_DIR ?? ""
+  const file = join(dir, "config.json")
+  const broken = '{ "defaults": { "limit": 5, }, }'
+  mkdirSync(dir, { recursive: true })
+  writeFileSync(file, broken)
+  let backup = ""
+  try {
+    const preview = await ran(["config", "repair", "--dry-run", "--json"])
+    expect(preview.code).toBe(0)
+    expect(json(preview.out).item).toMatchObject({ dry_run: true, repaired: false, backup: null })
+    expect(readFileSync(file, "utf8")).toBe(broken)
+
+    const fixed = await ran(["config", "repair", "--json"])
+    expect(fixed.code).toBe(0)
+    const item = json(fixed.out).item
+    expect(item).toMatchObject({
+      repaired: true,
+      changes: expect.arrayContaining([{ kind: "trailing_comma", line: 1 }]),
+    })
+    backup = String(item.backup)
+    expect(readFileSync(backup, "utf8")).toBe(broken)
+    expect(JSON.parse(readFileSync(file, "utf8"))).toEqual({ defaults: { limit: 5 } })
+  } finally {
+    rmSync(file)
+    if (backup) rmSync(backup)
+  }
+})
+
+it("a deny in the profile stops a read command before its action runs", async () => {
+  const reached: string[] = []
+  const probe = annotate(new Command("probe"), { effects: REMOTE_READ, offline: false }).action(() => {
+    reached.push("probe")
+  })
+  const withProbe = { ...definition(), commands: () => [...definition().commands(), probe] }
+  const dir = process.env.CLIDEV_CONFIG_DIR ?? ""
+  mkdirSync(dir, { recursive: true })
+  writeFileSync(
+    join(dir, "config.json"),
+    JSON.stringify({ profiles: { personal: { permissions: { probe: "deny" } } } }),
+  )
+  const err: string[] = []
+  try {
+    const code = await runShared(["personal", "probe", "--json"], withProbe, {
+      streams: { data: () => {}, diagnostic: (text) => err.push(text) },
+      tty: false,
+    })
+    expect(code).not.toBe(0)
+    expect(err.join("\n")).toContain("permission_error")
+    expect(reached).toEqual([])
+  } finally {
+    rmSync(join(dir, "config.json"))
+  }
+})

package/src/program.ts

@@ -33,7 +33,8 @@ const update = appUpdate({
 
 export const definition = (): ProgramDefinition => ({
   app: IDENTITY,
-  configuration: { resolveSettings: configuration.resolveSettings },
+  // The whole object: cli-core refuses a command by its key only when it gets the permission table too.
+  configuration,
   commands: () => [
     commandsCommand(IDENTITY),
     doctorCommand(),

package/src/version.ts

@@ -1 +1 @@
-export const VERSION = "0.13.2"
+export const VERSION = "0.15.0"

package/dist/bundle/chunk-IDWW4TU2.js

@@ -1,18 +0,0 @@
-import { createRequire as __bundleRequire } from "node:module";
-const require = __bundleRequire(import.meta.url);
-import {
-  resolvePaths
-} from "./chunk-EFHLQXG6.js";
-import {
-  __name
-} from "./chunk-ZVVMGPBJ.js";
-
-// node_modules/.pnpm/@wirecat+cli-core@0.495.0_@wirecat+cli-base@0.65.0_commander@15.0.0_esbuild@0.28.2_type_24cca5e5c8b0984f2f26e7d1ba6b6d47/node_modules/@wirecat/cli-core/dist/store/path.js
-import { join } from "node:path";
-var wirecatDirectory = /* @__PURE__ */ __name((env = process.env, platform) => resolvePaths({ appName: "WireCat", prefix: "WIRECAT", env, ...platform ? { platform } : {} }).state, "wirecatDirectory");
-var storePath = /* @__PURE__ */ __name((env = process.env) => env.MESSAGING_STORE ?? join(wirecatDirectory(env), "wirecat.db"), "storePath");
-
-export {
-  wirecatDirectory,
-  storePath
-};