@wirecat/cli-template 0.1.5 → 0.1.7

package/src/program.test.ts

@@ -1,4 +1,4 @@
-import { mkdirSync, readFileSync, rmSync, writeFileSync } from "node:fs"
+import { chmodSync, mkdirSync, readFileSync, rmSync, statSync, writeFileSync } from "node:fs"
 import { join } from "node:path"
 import { expect, it } from "vitest"
 import { doctorReport } from "./commands/doctor.js"
@@ -58,10 +58,43 @@ it("doctor reports an unknown setting with the command that removes it", async (
 })
 
 it("doctor reports settings an environment variable breaks, and still answers", () => {
-  const report = doctorReport({ ...process.env, CLIDEV_EMBEDDING_DIMS: "many" })
+  const { report } = doctorReport({ ...process.env, CLIDEV_EMBEDDING_DIMS: "many" })
   expect(report).toMatchObject({ config: { issues: [] }, settings: { error: expect.any(String) }, profiles: null })
 })
 
+it.skipIf(process.platform === "win32")(
+  "doctor names a config file and folder others can read, with the chmod that fixes each",
+  async () => {
+    const dir = process.env.CLIDEV_CONFIG_DIR ?? ""
+    const file = join(dir, "config.json")
+    mkdirSync(dir, { recursive: true })
+    writeFileSync(file, "{}")
+    chmodSync(file, 0o644)
+    chmodSync(dir, 0o775)
+    try {
+      const { code, out, err } = await ran(["doctor", "--json"])
+      expect(code).toBe(0)
+      expect(json(out).item.files).toEqual({
+        checked: true,
+        ok: false,
+        problems: [
+          { path: file, mode: "0644", want: "0600" },
+          { path: dir, mode: "0775", want: "0700" },
+        ],
+      })
+      expect(err).toContain(`to make it private: chmod 600 '${file}'`)
+      expect(err).toContain(`to make it private: chmod 700 '${dir}'`)
+
+      chmodSync(dir, 0o700)
+      expect((await ran(["personal", "config", "set", "--defaults", "limit", "5"])).code).toBe(0)
+      expect(statSync(file).mode & 0o777).toBe(0o600)
+      expect(json((await ran(["doctor", "--json"])).out).item.files).toEqual({ checked: true, ok: true, problems: [] })
+    } finally {
+      rmSync(file)
+    }
+  },
+)
+
 it("config show names the profile it was given", async () => {
   const { code, out } = await ran(["personal", "config", "show", "--json"])
   expect(code).toBe(0)

package/src/roots.test.ts

@@ -73,6 +73,22 @@ it("skill install writes the skill for Claude Code", async () => {
   expect(existsSync(join(homedir(), ".claude", "skills", "clidev-cli", "SKILL.md"))).toBe(true)
 })
 
+it("skill status and skill show --for read what skill install wrote", async () => {
+  expect((await ran(["skill", "install", "--for", "claude", "--json"])).code).toBe(0)
+  const status = await ran(["skill", "status", "--json"])
+  expect(status.code).toBe(0)
+  expect(JSON.parse(status.out).targets).toEqual([
+    expect.objectContaining({ target: "claude", state: "current" }),
+    expect.objectContaining({ target: "agents", installed: null, state: "missing" }),
+  ])
+  const shown = await ran(["skill", "show", "--for", "claude", "--json"])
+  expect(shown.code).toBe(0)
+  expect(JSON.parse(shown.out)).toMatchObject({
+    target: "claude",
+    content: expect.stringContaining("name: clidev-cli"),
+  })
+})
+
 it("help prints the first screen", async () => {
   const { code, out } = await ran(["help"])
   expect(code).toBe(0)

package/src/testing/sandbox.ts

@@ -5,13 +5,10 @@ import { ARGV_LOG } from "./argv-log.js"
 
 const saved = { ...process.env }
 
-// cli-base's setup already gave this file a fresh TMPDIR and HOME, a keyring that throws and no network.
-const sandbox = tmpdir()
-process.env.CLIDEV_CONFIG_DIR = join(sandbox, "config")
-process.env.CLIDEV_STATE_DIR = join(sandbox, "state")
-process.env.CLIDEV_CACHE_DIR = join(sandbox, "cache")
-process.env.MESSAGING_STORE = join(sandbox, "messages.db")
-process.env.WIRECAT_STATE_DIR = join(sandbox, "wirecat")
+// cli-base's setup already gave this file a fresh TMPDIR and HOME, a keyring that throws and no network, and points
+// every folder and store variable at them (WC-TEST-1, WC-STORE-14). The config folder is set only because tests
+// write into it by name.
+process.env.CLIDEV_CONFIG_DIR = join(tmpdir(), "config")
 process.env.CLIDEV_NO_UPDATE_CHECK = "1"
 process.env.CLIDEV_TEST_ARGV_LOG = ARGV_LOG
 delete process.env.CLIDEV_PROFILE

package/src/version.ts

@@ -1 +1 @@
-export const VERSION = "0.1.5"
+export const VERSION = "0.1.7"

package/vitest.config.ts

@@ -10,13 +10,15 @@ export default defineConfig({
     globalSetup: ["src/testing/argv-log.ts"],
     coverage: {
       provider: "v8",
-      include: ["src/**/*.ts"],
+      include: ["src/**/*.ts", "kit/src/**/*.ts"],
       exclude: [
         "src/**/*.test.ts",
         // Test setup: the sandbox and the argv log run only under vitest.
         "src/testing/**",
         // The entry point: one line handing argv to run(), which the suite drives directly.
         "src/bin/**",
+        // The kit's entry point runs only as a separate process, in kit/test/package.test.ts.
+        "kit/src/main.ts",
       ],
       reporter: ["text-summary", "json-summary"],
       thresholds: { lines: 90, statements: 90, functions: 80, branches: 70, perFile: { lines: 50 } },