pluidr 0.7.5 → 0.8.0

package/src/core/configBuilder.test.js

@@ -1,93 +1,93 @@
-import { describe, it } from "node:test"
-import assert from "node:assert"
-import { mkdtempSync, writeFileSync, rmSync } from "node:fs"
-import { join } from "node:path"
-import { tmpdir } from "node:os"
-import { buildConfig } from "./configBuilder.js"
-
-describe("configBuilder", () => {
-  it("injects models into agent config per tier", () => {
-    const tmpDir = mkdtempSync(join(tmpdir(), "configbuilder-test-"))
-
-    const configJson = {
-      agent: {
-        composer: {},
-        reviewer: {},
-        coder: {},
-      },
-    }
-    const defaultsJson = {
-      reasoning: {
-        provider: "test",
-        model: "big-model",
-        agents: ["composer"],
-      },
-      precision: {
-        provider: "test",
-        model: "mid-model",
-        agents: ["reviewer"],
-      },
-      fast: {
-        provider: "test",
-        model: "small-model",
-        agents: ["coder"],
-      },
-    }
-
-    writeFileSync(join(tmpDir, "opencode.config.json"), JSON.stringify(configJson), "utf-8")
-    writeFileSync(join(tmpDir, "model-defaults.json"), JSON.stringify(defaultsJson), "utf-8")
-
-    const result = buildConfig(
-      {
-        reasoning: "test/big-model",
-        precision: "test/mid-model",
-        fast: "test/small-model",
-      },
-      tmpDir,
-    )
-
-    assert.strictEqual(result.agent.composer.model, "test/big-model")
-    assert.strictEqual(result.agent.reviewer.model, "test/mid-model")
-    assert.strictEqual(result.agent.coder.model, "test/small-model")
-
-    rmSync(tmpDir, { recursive: true })
-  })
-
-  it("throws an error if a defaulted agent is missing from config template", () => {
-    const tmpDir = mkdtempSync(join(tmpdir(), "configbuilder-test-err-"))
-
-    const configJson = {
-      agent: {
-        // missing 'composer'
-        coder: {},
-      },
-    }
-    const defaultsJson = {
-      reasoning: {
-        provider: "test",
-        model: "big-model",
-        agents: ["composer"],
-      },
-      fast: {
-        provider: "test",
-        model: "small-model",
-        agents: ["coder"],
-      },
-    }
-
-    writeFileSync(join(tmpDir, "opencode.config.json"), JSON.stringify(configJson), "utf-8")
-    writeFileSync(join(tmpDir, "model-defaults.json"), JSON.stringify(defaultsJson), "utf-8")
-
-    assert.throws(() => {
-      buildConfig(
-        {
-          reasoning: "test/big-model",
-          fast: "test/small-model",
-        },
-        tmpDir,
-      )
-    }, /Template validation failed: agent\(s\) not found/)
-
-    rmSync(tmpDir, { recursive: true })
-  })
-})
+import { describe, it } from "node:test"
+import assert from "node:assert"
+import { mkdtempSync, writeFileSync, rmSync } from "node:fs"
+import { join } from "node:path"
+import { tmpdir } from "node:os"
+import { buildConfig } from "./configBuilder.js"
+
+describe("configBuilder", () => {
+  it("injects models into agent config per tier", () => {
+    const tmpDir = mkdtempSync(join(tmpdir(), "configbuilder-test-"))
+
+    const configJson = {
+      agent: {
+        compose: {},
+        verifier: {},
+        builder: {},
+      },
+    }
+    const defaultsJson = {
+      reasoning: {
+        provider: "test",
+        model: "big-model",
+        agents: ["compose"],
+      },
+      precision: {
+        provider: "test",
+        model: "mid-model",
+        agents: ["verifier"],
+      },
+      fast: {
+        provider: "test",
+        model: "small-model",
+        agents: ["builder"],
+      },
+    }
+
+    writeFileSync(join(tmpDir, "opencode.config.json"), JSON.stringify(configJson), "utf-8")
+    writeFileSync(join(tmpDir, "model-defaults.json"), JSON.stringify(defaultsJson), "utf-8")
+
+    const result = buildConfig(
+      {
+        reasoning: "test/big-model",
+        precision: "test/mid-model",
+        fast: "test/small-model",
+      },
+      tmpDir,
+    )
+
+    assert.strictEqual(result.agent.compose.model, "test/big-model")
+    assert.strictEqual(result.agent.verifier.model, "test/mid-model")
+    assert.strictEqual(result.agent.builder.model, "test/small-model")
+
+    rmSync(tmpDir, { recursive: true })
+  })
+
+  it("throws an error if a defaulted agent is missing from config template", () => {
+    const tmpDir = mkdtempSync(join(tmpdir(), "configbuilder-test-err-"))
+
+    const configJson = {
+      agent: {
+        // missing 'compose'
+        builder: {},
+      },
+    }
+    const defaultsJson = {
+      reasoning: {
+        provider: "test",
+        model: "big-model",
+        agents: ["compose"],
+      },
+      fast: {
+        provider: "test",
+        model: "small-model",
+        agents: ["builder"],
+      },
+    }
+
+    writeFileSync(join(tmpDir, "opencode.config.json"), JSON.stringify(configJson), "utf-8")
+    writeFileSync(join(tmpDir, "model-defaults.json"), JSON.stringify(defaultsJson), "utf-8")
+
+    assert.throws(() => {
+      buildConfig(
+        {
+          reasoning: "test/big-model",
+          fast: "test/small-model",
+        },
+        tmpDir,
+      )
+    }, /Template validation failed: agent\(s\) not found/)
+
+    rmSync(tmpDir, { recursive: true })
+  })
+})

package/src/core/configWriter.js

@@ -1,10 +1,10 @@
-import { mkdirSync, writeFileSync } from "node:fs"
-import { join } from "node:path"
-import { getConfigDir } from "./paths.js"
-
-export function writeConfig(configObject, targetDir) {
-  const baseDir = targetDir || getConfigDir()
-  const configPath = join(baseDir, "opencode.jsonc")
-  mkdirSync(baseDir, { recursive: true })
-  writeFileSync(configPath, JSON.stringify(configObject, null, 2), "utf-8")
-}
+import { mkdirSync, writeFileSync } from "node:fs"
+import { join } from "node:path"
+import { getConfigDir } from "./paths.js"
+
+export function writeConfig(configObject, targetDir) {
+  const baseDir = targetDir || getConfigDir()
+  const configPath = join(baseDir, "opencode.jsonc")
+  mkdirSync(baseDir, { recursive: true })
+  writeFileSync(configPath, JSON.stringify(configObject, null, 2), "utf-8")
+}

package/src/core/configWriter.test.js

@@ -13,7 +13,7 @@ describe("configWriter", () => {
   })
 
   it("writes config as formatted JSON to the correct path", () => {
-    const configObj = { agent: { composer: { model: "test/model" } } }
+    const configObj = { agent: { compose: { model: "test/model" } } }
     const expected = JSON.stringify(configObj, null, 2)
 
     writeConfig(configObj, tempDir)

package/src/core/identityHeader.js

@@ -1,8 +1,8 @@
-// Single source of truth for the agent identity prefix prepended to every
-// generated prompt file. Mirrors the system identity concept from the
-// dropped hook-based AgentSelfIdentityPlugin, but as a static prepend
-// (KISS: agent names are known at install time).
-
-export default function IDENTITY_HEADER(agentName) {
-  return `You are the "${agentName}" agent.\n\n`
-}
+// Single source of truth for the agent identity prefix prepended to every
+// generated prompt file. Mirrors the system identity concept from the
+// dropped hook-based AgentSelfIdentityPlugin, but as a static prepend
+// (KISS: agent names are known at install time).
+
+export default function IDENTITY_HEADER(agentName) {
+  return `You are the "${agentName}" agent.\n\n`
+}

package/src/core/paths.js

@@ -1,11 +1,13 @@
-import { homedir } from "node:os"
-import { join } from "node:path"
-
-const BASE = join(homedir(), ".config", "opencode")
-
-export const getConfigDir = () => BASE
-export const getConfigPath = () => join(BASE, "opencode.jsonc")
-export const getTuiConfigPath = () => join(BASE, "tui.json")
-export const getPromptsDir = () => join(BASE, "prompts")
-export const getThemesDir = () => join(BASE, "themes")
-// ponytail: getBackupPath removed (YAGNI)
+import { homedir } from "node:os"
+import { join } from "node:path"
+
+const BASE = join(homedir(), ".config", "opencode")
+
+export const getConfigDir = () => BASE
+export const getConfigPath = () => join(BASE, "opencode.jsonc")
+export const getTuiConfigPath = () => join(BASE, "tui.json")
+export const getPromptsDir = () => join(BASE, "prompts")
+export const getThemesDir = () => join(BASE, "themes")
+export const getSkillsDir = () => join(BASE, "skills")
+export const getCommandsDir = () => join(BASE, "commands")
+// ponytail: getBackupPath removed (YAGNI)

package/src/core/paths.test.js

@@ -1,29 +1,33 @@
-import { describe, it } from "node:test"
-import assert from "node:assert"
-import { homedir } from "node:os"
-import { join } from "node:path"
-import { getConfigDir, getConfigPath, getTuiConfigPath, getPromptsDir, getThemesDir } from "./paths.js"
-
-const BASE = join(homedir(), ".config", "opencode")
-
-describe("paths", () => {
-  it("getConfigDir returns ~/.config/opencode", () => {
-    assert.strictEqual(getConfigDir(), BASE)
-  })
-
-  it("getConfigPath returns opencode.jsonc path", () => {
-    assert.strictEqual(getConfigPath(), join(BASE, "opencode.jsonc"))
-  })
-
-  it("getTuiConfigPath returns tui.json path", () => {
-    assert.strictEqual(getTuiConfigPath(), join(BASE, "tui.json"))
-  })
-
-  it("getPromptsDir returns prompts path", () => {
-    assert.strictEqual(getPromptsDir(), join(BASE, "prompts"))
-  })
-
-  it("getThemesDir returns themes path", () => {
-    assert.strictEqual(getThemesDir(), join(BASE, "themes"))
-  })
-})
+import { describe, it } from "node:test"
+import assert from "node:assert"
+import { homedir } from "node:os"
+import { join } from "node:path"
+import { getConfigDir, getConfigPath, getTuiConfigPath, getPromptsDir, getThemesDir, getSkillsDir } from "./paths.js"
+
+const BASE = join(homedir(), ".config", "opencode")
+
+describe("paths", () => {
+  it("getConfigDir returns ~/.config/opencode", () => {
+    assert.strictEqual(getConfigDir(), BASE)
+  })
+
+  it("getConfigPath returns opencode.jsonc path", () => {
+    assert.strictEqual(getConfigPath(), join(BASE, "opencode.jsonc"))
+  })
+
+  it("getTuiConfigPath returns tui.json path", () => {
+    assert.strictEqual(getTuiConfigPath(), join(BASE, "tui.json"))
+  })
+
+  it("getPromptsDir returns prompts path", () => {
+    assert.strictEqual(getPromptsDir(), join(BASE, "prompts"))
+  })
+
+  it("getThemesDir returns themes path", () => {
+    assert.strictEqual(getThemesDir(), join(BASE, "themes"))
+  })
+
+  it("getSkillsDir returns skills path", () => {
+    assert.strictEqual(getSkillsDir(), join(BASE, "skills"))
+  })
+})

package/src/core/pluginWriter.js

@@ -1,29 +1,43 @@
-import { copyFileSync, existsSync, mkdirSync, writeFileSync } from "node:fs"
-import { dirname, join, resolve } from "node:path"
-import { fileURLToPath } from "node:url"
-import { getConfigDir } from "./paths.js"
-
-const __dirname = dirname(fileURLToPath(import.meta.url))
-const PLUGINS = ["pluidr-flow.js", "pluidr-squeeze.js"]
-const PACKAGE_JSON = {
-  type: "module",
-  dependencies: { "@opencode-ai/plugin": "^1.17.9" },
-}
-
-export function writePluginBundle(targetDir) {
-  const baseDir = targetDir || getConfigDir()
-  const pluginsDir = join(baseDir, "plugins")
-  mkdirSync(pluginsDir, { recursive: true })
-
-  for (const name of PLUGINS) {
-    const sourcePath = resolve(__dirname, "../plugins", name)
-    copyFileSync(sourcePath, join(pluginsDir, name))
-  }
-}
-
-export function writePluginPackageJson(targetDir) {
-  const baseDir = targetDir || getConfigDir()
-  const packageJsonPath = join(baseDir, "package.json")
-
-  writeFileSync(packageJsonPath, JSON.stringify(PACKAGE_JSON, null, 2), "utf-8")
-}
+import { copyFileSync, existsSync, mkdirSync, writeFileSync, readdirSync, unlinkSync } from "node:fs"
+import { dirname, join, resolve } from "node:path"
+import { fileURLToPath } from "node:url"
+import { getConfigDir } from "./paths.js"
+
+const __dirname = dirname(fileURLToPath(import.meta.url))
+const PLUGINS = ["pluidr-flow.js", "pluidr-squeeze.js"]
+const PACKAGE_JSON = {
+  type: "module",
+  dependencies: { "@opencode-ai/plugin": "^1.17.9" },
+}
+
+export function writePluginBundle(targetDir) {
+  const baseDir = targetDir || getConfigDir()
+  const pluginsDir = join(baseDir, "plugins")
+
+  // Clear any existing .js files in pluginsDir to prevent stale plugins (like wevr-*)
+  if (existsSync(pluginsDir)) {
+    try {
+      for (const file of readdirSync(pluginsDir)) {
+        if (file.endsWith(".js")) {
+          unlinkSync(join(pluginsDir, file))
+        }
+      }
+    } catch {
+      // Ignore
+    }
+  }
+
+  mkdirSync(pluginsDir, { recursive: true })
+
+  for (const name of PLUGINS) {
+    const sourcePath = resolve(__dirname, "../plugins", name)
+    copyFileSync(sourcePath, join(pluginsDir, name))
+  }
+}
+
+export function writePluginPackageJson(targetDir) {
+  const baseDir = targetDir || getConfigDir()
+  const packageJsonPath = join(baseDir, "package.json")
+
+  writeFileSync(packageJsonPath, JSON.stringify(PACKAGE_JSON, null, 2), "utf-8")
+}

package/src/core/skillsWriter.js

@@ -0,0 +1,21 @@
+import { mkdirSync, cpSync, existsSync, rmSync } from "node:fs"
+import { resolve } from "node:path"
+import { getSkillsDir } from "./paths.js"
+
+export function writeSkills(templatesDir, destDir) {
+  const sourcePath = resolve(templatesDir, "skills")
+  const targetDir = destDir || getSkillsDir()
+
+  if (existsSync(targetDir)) {
+    try {
+      rmSync(targetDir, { recursive: true, force: true })
+    } catch {
+      // Ignore
+    }
+  }
+
+  mkdirSync(targetDir, { recursive: true })
+  if (existsSync(sourcePath)) {
+    cpSync(sourcePath, targetDir, { recursive: true })
+  }
+}

package/src/core/skillsWriter.test.js

@@ -0,0 +1,30 @@
+import { describe, it, after } from "node:test"
+import assert from "node:assert"
+import { existsSync, mkdtempSync, readFileSync, rmSync, mkdirSync, writeFileSync } from "node:fs"
+import { join } from "node:path"
+import { tmpdir } from "node:os"
+import { writeSkills } from "./skillsWriter.js"
+
+describe("skillsWriter", () => {
+  const destDir = mkdtempSync(join(tmpdir(), "pluidr-skills-dest-"))
+
+  after(() => {
+    if (existsSync(destDir)) rmSync(destDir, { recursive: true })
+  })
+
+  it("copies skills directories recursively to destination directory", () => {
+    const tmpSrcDir = mkdtempSync(join(tmpdir(), "pluidr-skills-src-"))
+    const srcSkillsDir = join(tmpSrcDir, "skills")
+    const testSkillDir = join(srcSkillsDir, "test-skill")
+    mkdirSync(testSkillDir, { recursive: true })
+    writeFileSync(join(testSkillDir, "SKILL.md"), "mock-skill-content", "utf-8")
+
+    writeSkills(tmpSrcDir, destDir)
+
+    const destPath = join(destDir, "test-skill", "SKILL.md")
+    assert.ok(existsSync(destPath), "SKILL.md should be copied recursively")
+    assert.strictEqual(readFileSync(destPath, "utf-8"), "mock-skill-content")
+
+    rmSync(tmpSrcDir, { recursive: true })
+  })
+})