opencode-goal-mode 0.1.0 → 0.2.1

package/plugins/package.json

@@ -0,0 +1,3 @@
+{
+  "type": "module"
+}

package/scripts/install.mjs

@@ -1,7 +1,18 @@
 #!/usr/bin/env node
 
-import { mkdirSync, copyFileSync, readdirSync, statSync, existsSync, readFileSync } from "node:fs";
-import { join, resolve } from "node:path";
+import {
+  mkdirSync,
+  copyFileSync,
+  readdirSync,
+  statSync,
+  lstatSync,
+  rmdirSync,
+  existsSync,
+  readFileSync,
+  writeFileSync,
+  rmSync,
+} from "node:fs";
+import { join, resolve, relative, dirname } from "node:path";
 import { fileURLToPath } from "node:url";
 import { createHash } from "node:crypto";
 import { parseArgs } from "node:util";
@@ -11,6 +22,7 @@ const { values } = parseArgs({
     global: { type: "boolean", default: false },
     force: { type: "boolean", default: false },
     "dry-run": { type: "boolean", default: false },
+    uninstall: { type: "boolean", default: false },
     target: { type: "string" },
     help: { type: "boolean", short: "h", default: false },
   },
@@ -18,19 +30,30 @@ const { values } = parseArgs({
 });
 
 const root = resolve(fileURLToPath(new URL("..", import.meta.url)));
+const pkg = JSON.parse(readFileSync(join(root, "package.json"), "utf8"));
+
+/** Component directories installed into an OpenCode config dir. */
+const COMPONENT_DIRS = ["agents", "commands", "plugins"];
+const MANIFEST_NAME = ".goal-mode-manifest.json";
 
 if (values.help) {
-  console.log(`Install OpenCode Goal Mode components.
+  console.log(`Install or remove OpenCode Goal Mode components.
 
 Usage:
   node scripts/install.mjs [--global | --target <dir>] [--force] [--dry-run]
+  node scripts/install.mjs --uninstall [--global | --target <dir>] [--dry-run]
 
 Options:
   --global       Install into ~/.config/opencode.
   --target DIR   Install into a specific OpenCode config directory.
-  --force        Replace changed destination files.
-  --dry-run      Show planned copies without writing files.
-  -h, --help     Show this help text.`);
+  --force        Replace destination files even if locally modified.
+  --uninstall    Remove files this installer previously wrote (per manifest).
+  --dry-run      Show planned changes without writing.
+  -h, --help     Show this help text.
+
+The installer records a manifest of the files it writes so that a later
+upgrade can distinguish files it owns (safe to replace) from files you have
+locally customized (left untouched unless --force).`);
   process.exit(0);
 }
 
@@ -49,48 +72,152 @@ function resolveTarget() {
 }
 
 function fileHash(path) {
-  const data = readFileSync(path);
-  return createHash("sha256").update(data).digest("hex").slice(0, 16);
+  return createHash("sha256").update(readFileSync(path)).digest("hex").slice(0, 16);
+}
+
+/** Recursively list regular files under a directory, returning paths relative to
+ * `base`. Uses lstat and skips symlinks so the installer only copies files it can
+ * reason about (no following links outside the package tree). */
+function listFiles(dir, base = dir, out = []) {
+  if (!existsSync(dir)) return out;
+  for (const entry of readdirSync(dir)) {
+    const abs = join(dir, entry);
+    const st = lstatSync(abs);
+    if (st.isSymbolicLink()) continue;
+    if (st.isDirectory()) listFiles(abs, base, out);
+    else if (st.isFile()) out.push(relative(base, abs));
+  }
+  return out;
+}
+
+/** Remove directories left empty by file removal, bottom-up, within the target. */
+function pruneEmptyDirs(targetRoot, relFiles) {
+  const dirs = new Set();
+  for (const rel of relFiles) {
+    let d = dirname(rel);
+    while (d && d !== "." && d !== "/") {
+      dirs.add(d);
+      d = dirname(d);
+    }
+  }
+  // Deepest first so parents become empty after their children are removed.
+  for (const rel of [...dirs].sort((a, b) => b.length - a.length)) {
+    const abs = join(targetRoot, rel);
+    // Containment guard: never touch anything resolving outside the target.
+    if (relative(targetRoot, abs).startsWith("..")) continue;
+    try {
+      if (existsSync(abs) && statSync(abs).isDirectory() && readdirSync(abs).length === 0) rmdirSync(abs);
+    } catch {
+      /* ignore */
+    }
+  }
+}
+
+const target = resolveTarget();
+const manifestPath = join(target, MANIFEST_NAME);
+
+function loadManifest() {
+  try {
+    const data = JSON.parse(readFileSync(manifestPath, "utf8"));
+    return data && typeof data === "object" && data.files ? data : { version: null, files: {} };
+  } catch {
+    return { version: null, files: {} };
+  }
+}
+
+// ---------------------------------------------------------------------------
+// Uninstall
+// ---------------------------------------------------------------------------
+
+if (values.uninstall) {
+  const manifest = loadManifest();
+  const removed = [];
+  const kept = [];
+  for (const [rel, hash] of Object.entries(manifest.files)) {
+    const dest = join(target, rel);
+    if (!existsSync(dest)) continue;
+    if (fileHash(dest) === hash) {
+      if (!values["dry-run"]) rmSync(dest, { force: true });
+      removed.push(rel);
+    } else {
+      kept.push(rel);
+    }
+  }
+  if (!values["dry-run"] && existsSync(manifestPath)) rmSync(manifestPath, { force: true });
+  if (!values["dry-run"]) pruneEmptyDirs(target, Object.keys(manifest.files));
+  const verb = values["dry-run"] ? "Would remove" : "Removed";
+  console.log(`${verb} ${removed.length} Goal Mode files from ${target}.`);
+  if (kept.length) {
+    console.log(`Left ${kept.length} locally-modified file(s) in place:`);
+    for (const rel of kept) console.log(`- ${rel}`);
+  }
+  console.log("Restart OpenCode to unload the components.");
+  process.exit(0);
 }
 
-function copyDirFiles(from, to, summary) {
-  if (!values["dry-run"]) mkdirSync(to, { recursive: true });
-  for (const entry of readdirSync(from)) {
-    const source = join(from, entry);
-    const dest = join(to, entry);
-    if (!statSync(source).isFile()) continue;
+// ---------------------------------------------------------------------------
+// Install
+// ---------------------------------------------------------------------------
+
+const manifest = loadManifest();
+const summary = { copied: [], unchanged: [], conflicts: [], pruned: [] };
+const newManifestFiles = {};
+
+for (const dir of COMPONENT_DIRS) {
+  const from = join(root, dir);
+  for (const rel of listFiles(from)) {
+    const relKey = join(dir, rel);
+    const source = join(from, rel);
+    const dest = join(target, relKey);
+    const srcHash = fileHash(source);
+    newManifestFiles[relKey] = srcHash;
+
     if (existsSync(dest) && !statSync(dest).isFile()) {
       summary.conflicts.push(`${dest} exists but is not a file`);
       continue;
     }
-    if (values.force) {
-      if (!values["dry-run"]) copyFileSync(source, dest);
+
+    if (!existsSync(dest)) {
+      if (!values["dry-run"]) {
+        mkdirSync(dirname(dest), { recursive: true });
+        copyFileSync(source, dest);
+      }
       summary.copied.push(dest);
       continue;
     }
-    if (existsSync(dest)) {
-      const srcHash = fileHash(source);
-      const dstHash = fileHash(dest);
-      if (srcHash === dstHash) {
-        summary.unchanged.push(dest);
-        continue;
-      }
-      summary.conflicts.push(`${dest} differs from packaged ${entry}`);
+
+    const dstHash = fileHash(dest);
+    if (dstHash === srcHash) {
+      summary.unchanged.push(dest);
       continue;
     }
-    if (!values["dry-run"]) {
-      copyFileSync(source, dest);
+
+    const ownedHash = manifest.files[relKey];
+    const weOwnIt = ownedHash !== undefined && ownedHash === dstHash;
+    if (values.force || weOwnIt) {
+      if (!values["dry-run"]) {
+        mkdirSync(dirname(dest), { recursive: true });
+        copyFileSync(source, dest);
+      }
+      summary.copied.push(dest);
+      continue;
     }
-    summary.copied.push(dest);
+    summary.conflicts.push(`${dest} differs from packaged ${relKey} and was locally modified`);
   }
 }
 
-const target = resolveTarget();
-const summary = { copied: [], unchanged: [], conflicts: [] };
-
-copyDirFiles(join(root, "agents"), join(target, "agents"), summary);
-copyDirFiles(join(root, "commands"), join(target, "commands"), summary);
-copyDirFiles(join(root, "plugins"), join(target, "plugins"), summary);
+// Prune files we installed in a previous version that no longer ship (e.g. a
+// plugin split into modules), but only if the user hasn't modified them.
+for (const [relKey, oldHash] of Object.entries(manifest.files)) {
+  if (newManifestFiles[relKey] !== undefined) continue;
+  const dest = join(target, relKey);
+  if (!existsSync(dest)) continue;
+  if (fileHash(dest) === oldHash) {
+    if (!values["dry-run"]) rmSync(dest, { force: true });
+    summary.pruned.push(relKey);
+  }
+}
+if (!values["dry-run"] && summary.pruned.length) pruneEmptyDirs(target, summary.pruned);
 
 if (summary.conflicts.length) {
   throw new Error(
@@ -98,11 +225,18 @@ if (summary.conflicts.length) {
       "Refusing to overwrite changed OpenCode component files.",
       ...summary.conflicts.map((conflict) => `- ${conflict}`),
       "Use --force to replace them or remove the conflicting files manually.",
-    ].join("\n")
+    ].join("\n"),
   );
 }
 
+if (!values["dry-run"]) {
+  mkdirSync(target, { recursive: true });
+  writeFileSync(manifestPath, JSON.stringify({ version: pkg.version, files: newManifestFiles }, null, 2), "utf8");
+}
+
 const verb = values["dry-run"] ? "Would install" : "Installed";
-console.log(`${verb} OpenCode Goal Mode into ${target}`);
-console.log(`Files copied: ${summary.copied.length}; unchanged: ${summary.unchanged.length}`);
+console.log(`${verb} OpenCode Goal Mode ${pkg.version} into ${target}`);
+console.log(
+  `Files copied: ${summary.copied.length}; unchanged: ${summary.unchanged.length}; pruned: ${summary.pruned.length}`,
+);
 console.log("Restart OpenCode for agents, commands, and plugins to load.");

package/docs/research-report.md

@@ -1,37 +0,0 @@
-# Research Report: Goal Mode Hardening
-
-## Sources Checked
-
-- OpenCode agents, commands, permissions, plugins, and skills documentation.
-- Claude Code subagents and hooks documentation.
-- Codex CLI, Codex Web, subagents/review/sandbox concepts, and repository overview.
-
-## Findings
-
-Prompt-only Goal Mode is not enough. A strong mode needs separate context windows for exploration and reviews, explicit review handoffs, stale-review invalidation after edits, state preservation through compaction, safer permissions, and repeatable commands.
-
-## Claude-Inspired Patterns
-
-- Use subagents to protect main context from search logs and broad file reads.
-- Give subagents narrow descriptions and narrow permissions.
-- Keep orchestration in the main agent; subagents return summaries only.
-- Use hooks/lifecycle events to enforce safety and preserve state.
-
-## Codex-Inspired Patterns
-
-- Run local code review through a separate agent before claiming completion.
-- Use explicit approval/safety modes and sandbox thinking for risky operations.
-- Treat review and verification as a repair loop, not a final paragraph.
-- Track outcomes and repeated failures as part of an improvement loop.
-
-## OpenCode Implementation Choices
-
-- Agents are Markdown files in `agents/`.
-- Commands are Markdown files in `commands/`.
-- Guardrails live in a local plugin with OpenCode hooks.
-- Tests validate the installable package without copying secrets.
-
-## Remaining Platform Limits
-
-- OpenCode plugins cannot fully prevent every possible final answer, but `experimental.text.complete` can inject blocking warnings when `Goal Completed` appears while the session is dirty.
-- Review cycle state is runtime in-memory per OpenCode process; compaction preserves it in prompt context, but restart resets plugin memory.

package/scripts/check-npm-publish-ready.mjs

@@ -1,54 +0,0 @@
-#!/usr/bin/env node
-
-import { readFileSync } from "node:fs";
-import { join } from "node:path";
-import { fileURLToPath } from "node:url";
-import { parseArgs } from "node:util";
-
-const { values } = parseArgs({
-  options: {
-    "skip-registry": { type: "boolean", default: false },
-    "skip-tag": { type: "boolean", default: false },
-  },
-  allowPositionals: false,
-});
-
-const root = fileURLToPath(new URL("..", import.meta.url));
-const pkg = JSON.parse(readFileSync(join(root, "package.json"), "utf8"));
-
-if (!pkg.name) throw new Error("package.json missing package name");
-if (!pkg.version) throw new Error("package.json missing package version");
-if (pkg.private) throw new Error("Refusing to publish a private package");
-if (pkg.publishConfig?.access !== "public") throw new Error("publishConfig.access must be public");
-if (pkg.publishConfig?.registry !== "https://registry.npmjs.org/") {
-  throw new Error("publishConfig.registry must be https://registry.npmjs.org/");
-}
-
-const releaseTag = process.env.GITHUB_REF_TYPE === "tag" ? process.env.GITHUB_REF_NAME : "";
-if (!values["skip-tag"] && releaseTag) {
-  const normalizedTag = releaseTag.startsWith("v") ? releaseTag.slice(1) : releaseTag;
-  if (normalizedTag !== pkg.version) {
-    throw new Error(`Release tag ${releaseTag} does not match package version ${pkg.version}`);
-  }
-}
-
-function packageMetadataUrl(name) {
-  const registry = pkg.publishConfig.registry.replace(/\/$/, "");
-  return `${registry}/${encodeURIComponent(name)}`;
-}
-
-if (!values["skip-registry"]) {
-  const response = await fetch(packageMetadataUrl(pkg.name), {
-    headers: { accept: "application/vnd.npm.install-v1+json" },
-  });
-
-  if (response.status !== 404) {
-    if (!response.ok) throw new Error(`npm registry check failed with HTTP ${response.status}`);
-    const metadata = await response.json();
-    if (metadata?.versions?.[pkg.version]) {
-      throw new Error(`${pkg.name}@${pkg.version} already exists on npm`);
-    }
-  }
-}
-
-console.log(`${pkg.name}@${pkg.version} is ready for npm publishing`);

package/scripts/validate-opencode-config.mjs

@@ -1,82 +0,0 @@
-import { existsSync, readFileSync, readdirSync, statSync } from "node:fs";
-import { join } from "node:path";
-import { fileURLToPath } from "node:url";
-
-const root = fileURLToPath(new URL("..", import.meta.url));
-const pkg = JSON.parse(readFileSync(join(root, "package.json"), "utf8"));
-
-if (!pkg.type || pkg.type !== "module") throw new Error("package.json must use ESM type module");
-if (pkg.private) throw new Error("package.json must not be private when npm publishing is enabled");
-if (!pkg.engines?.node) throw new Error("package.json missing Node engine requirement");
-if (!pkg.bin?.["opencode-goal-mode-install"]) throw new Error("package.json missing installer bin");
-if (pkg.publishConfig?.access !== "public") throw new Error("package.json publishConfig.access must be public");
-if (pkg.publishConfig?.registry !== "https://registry.npmjs.org/") {
-  throw new Error("package.json publishConfig.registry must target npmjs.org");
-}
-if (!pkg.files?.includes("agents/") || !pkg.files?.includes("commands/") || !pkg.files?.includes("plugins/")) {
-  throw new Error("package.json files must include installable OpenCode component directories");
-}
-for (const script of [
-  "test",
-  "validate",
-  "ci",
-  "prepublishOnly",
-  "install:local",
-  "install:global",
-  "pack:check",
-  "publish:check",
-  "audit",
-]) {
-  if (!pkg.scripts?.[script]) throw new Error(`package.json missing ${script} script`);
-}
-for (const file of ["README.md", "LICENSE", ".npmignore", ".nvmrc"]) {
-  if (!existsSync(join(root, file))) throw new Error(`${file} missing`);
-}
-
-const agentFiles = readdirSync(join(root, "agents")).filter((file) => file.endsWith(".md"));
-const commandFiles = readdirSync(join(root, "commands")).filter((file) => file.endsWith(".md"));
-const pluginFiles = readdirSync(join(root, "plugins")).filter((file) => file.endsWith(".js"));
-
-if (!agentFiles.includes("goal.md")) throw new Error("primary goal agent missing");
-if (!commandFiles.includes("goal.md")) throw new Error("primary goal command missing");
-if (!pluginFiles.includes("goal-guard.js")) throw new Error("goal guard plugin missing");
-
-const forbiddenComponentName = /(auth|session|token|secret|preauth|failures|hosts\.ya?ml)/i;
-for (const dir of ["agents", "commands", "plugins"]) {
-  for (const file of readdirSync(join(root, dir))) {
-    const path = join(root, dir, file);
-    if (!statSync(path).isFile()) continue;
-    if (forbiddenComponentName.test(file)) throw new Error(`forbidden component filename: ${dir}/${file}`);
-  }
-}
-
-for (const file of agentFiles) {
-  const text = readFileSync(join(root, "agents", file), "utf8");
-  if (!text.startsWith("---\n")) throw new Error(`${file} missing frontmatter`);
-  if (!/^description:/m.test(text)) throw new Error(`${file} missing description`);
-  if (!/^mode:\s+(primary|subagent|all)$/m.test(text)) throw new Error(`${file} has invalid mode`);
-  if (!/^permission:/m.test(text)) throw new Error(`${file} missing permission`);
-}
-
-for (const file of commandFiles) {
-  const text = readFileSync(join(root, "commands", file), "utf8");
-  if (!text.startsWith("---\n")) throw new Error(`${file} missing frontmatter`);
-  if (!/^description:/m.test(text)) throw new Error(`${file} missing description`);
-  if (!/^agent:/m.test(text)) throw new Error(`${file} missing agent`);
-}
-
-const plugin = await import(join(root, "plugins", "goal-guard.js"));
-if (typeof plugin.default !== "function") throw new Error("goal-guard plugin must default-export a function");
-const hooks = await plugin.default({ client: { app: { log: async () => undefined } } });
-for (const hook of [
-  "chat.params",
-  "tool.execute.before",
-  "tool.execute.after",
-  "experimental.session.compacting",
-  "experimental.text.complete",
-  "event",
-]) {
-  if (typeof hooks?.[hook] !== "function") throw new Error(`goal-guard plugin missing ${hook} hook`);
-}
-
-console.log("OpenCode Goal Mode package validation passed");

package/tests/agents.test.mjs

@@ -1,70 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { filesIn, readRepo, frontmatter, hasLine } from "./helpers.mjs";
-
-const requiredAgents = [
-  "goal.md",
-  "goal-explorer.md",
-  "goal-researcher.md",
-  "goal-implementer.md",
-  "goal-reviewer.md",
-  "goal-prompt-auditor.md",
-  "goal-diff-reviewer.md",
-  "goal-verifier.md",
-  "goal-test-reviewer.md",
-  "goal-security-reviewer.md",
-  "goal-ux-reviewer.md",
-  "goal-ops-reviewer.md",
-  "goal-doc-reviewer.md",
-  "goal-final-auditor.md",
-  "goal-deep-researcher.md",
-  "goal-web-researcher.md",
-  "goal-architect.md",
-  "goal-mapper.md",
-  "goal-planner.md",
-  "goal-coordinator.md",
-  "goal-doc-writer.md",
-  "goal-commentator.md",
-  "goal-api-reviewer.md",
-  "goal-data-reviewer.md",
-  "goal-perf-reviewer.md",
-  "goal-quality-gate.md",
-  "goal-completion-guard.md",
-];
-
-test("all required agents exist", () => {
-  const files = filesIn("agents");
-  for (const agent of requiredAgents) assert.ok(files.includes(agent), `${agent} missing`);
-});
-
-test("agents have required frontmatter", () => {
-  for (const file of requiredAgents) {
-    const fm = frontmatter(readRepo(`agents/${file}`));
-    assert.ok(hasLine(fm, "description"), `${file} missing description`);
-    assert.ok(hasLine(fm, "mode"), `${file} missing mode`);
-    assert.ok(hasLine(fm, "permission"), `${file} missing permission`);
-  }
-});
-
-test("primary goal enforces review artifacts and final contract", () => {
-  const text = readRepo("agents/goal.md");
-  for (const phrase of [
-    "Goal Contract",
-    "Verification Ledger",
-    "Review Ledger",
-    "Required review matrix",
-    "Review handoff template",
-    "Goal Completed",
-    "Review cycles: N",
-  ]) {
-    assert.ok(text.includes(phrase), `goal.md missing ${phrase}`);
-  }
-});
-
-test("reviewers are read-only", () => {
-  for (const file of requiredAgents.filter((name) => name.includes("reviewer") || name.includes("auditor") || name.includes("verifier"))) {
-    const fm = frontmatter(readRepo(`agents/${file}`));
-    assert.match(fm, /edit:\s+deny/, `${file} must deny edit`);
-    assert.match(fm, /task:\s+deny/, `${file} must deny task nesting`);
-  }
-});

package/tests/commands.test.mjs

@@ -1,23 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { filesIn, readRepo, frontmatter, hasLine } from "./helpers.mjs";
-
-const requiredCommands = ["goal.md", "goal-contract.md", "goal-review.md", "goal-status.md", "goal-repair.md", "goal-final.md"];
-
-test("all required commands exist", () => {
-  const files = filesIn("commands");
-  for (const command of requiredCommands) assert.ok(files.includes(command), `${command} missing`);
-});
-
-test("commands have descriptions and agent bindings", () => {
-  for (const file of requiredCommands) {
-    const fm = frontmatter(readRepo(`commands/${file}`));
-    assert.ok(hasLine(fm, "description"), `${file} missing description`);
-    assert.ok(hasLine(fm, "agent"), `${file} missing agent`);
-  }
-});
-
-test("review and final commands run as subtasks", () => {
-  assert.match(frontmatter(readRepo("commands/goal-review.md")), /subtask:\s+true/);
-  assert.match(frontmatter(readRepo("commands/goal-final.md")), /subtask:\s+true/);
-});

package/tests/helpers.mjs

@@ -1,23 +0,0 @@
-import { readFileSync, readdirSync } from "node:fs";
-import { fileURLToPath } from "node:url";
-import { join } from "node:path";
-
-export const root = fileURLToPath(new URL("..", import.meta.url));
-
-export function filesIn(dir) {
-  return readdirSync(join(root, dir)).filter((file) => file.endsWith(".md") || file.endsWith(".js"));
-}
-
-export function readRepo(path) {
-  return readFileSync(join(root, path), "utf8");
-}
-
-export function frontmatter(text) {
-  const match = text.match(/^---\n([\s\S]*?)\n---\n/);
-  if (!match) throw new Error("Missing YAML frontmatter");
-  return match[1];
-}
-
-export function hasLine(fm, key) {
-  return new RegExp(`^${key}:`, "m").test(fm);
-}

package/tests/install.test.mjs

@@ -1,64 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { mkdtempSync, existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-import { fileURLToPath } from "node:url";
-import { tmpdir } from "node:os";
-import { execFileSync } from "node:child_process";
-import { readRepo } from "./helpers.mjs";
-
-const repoRoot = fileURLToPath(new URL("..", import.meta.url));
-
-test("installer copies only safe OpenCode component directories", () => {
-  const text = readRepo("scripts/install.mjs");
-  assert.match(text, /copyDirFiles\(join\(root, "agents"\)/);
-  assert.match(text, /copyDirFiles\(join\(root, "commands"\)/);
-  assert.match(text, /copyDirFiles\(join\(root, "plugins"\)/);
-  assert.doesNotMatch(text, /auth|sessions|preauth|failures|hosts\.yml/);
-});
-
-test("gitignore excludes secrets and dependencies", () => {
-  const text = readRepo(".gitignore");
-  for (const pattern of ["node_modules/", ".env", ".env.*"]) assert.ok(text.includes(pattern));
-});
-
-test("installer functionally copies components to project .opencode", () => {
-  const temp = mkdtempSync(join(tmpdir(), "goal-install-"));
-  execFileSync("node", [join(repoRoot, "scripts", "install.mjs")], { cwd: temp, stdio: "pipe" });
-  assert.equal(existsSync(join(temp, ".opencode", "agents", "goal.md")), true);
-  assert.equal(existsSync(join(temp, ".opencode", "commands", "goal.md")), true);
-  assert.equal(existsSync(join(temp, ".opencode", "plugins", "goal-guard.js")), true);
-});
-
-test("installer dry run does not write files", () => {
-  const temp = mkdtempSync(join(tmpdir(), "goal-install-dry-"));
-  execFileSync("node", [join(repoRoot, "scripts", "install.mjs"), "--dry-run"], { cwd: temp, stdio: "pipe" });
-  assert.equal(existsSync(join(temp, ".opencode")), false);
-});
-
-test("installer refuses to overwrite changed destination files", () => {
-  const temp = mkdtempSync(join(tmpdir(), "goal-install-conflict-"));
-  mkdirSync(join(temp, ".opencode", "agents"), { recursive: true });
-  writeFileSync(join(temp, ".opencode", "agents", "goal.md"), "local change\n");
-  assert.throws(
-    () => execFileSync("node", [join(repoRoot, "scripts", "install.mjs")], { cwd: temp, stdio: "pipe" }),
-    /Refusing to overwrite changed OpenCode component files/
-  );
-});
-
-test("installer force replaces changed destination files", () => {
-  const temp = mkdtempSync(join(tmpdir(), "goal-install-force-"));
-  mkdirSync(join(temp, ".opencode", "agents"), { recursive: true });
-  const dest = join(temp, ".opencode", "agents", "goal.md");
-  writeFileSync(dest, "local change\n");
-  execFileSync("node", [join(repoRoot, "scripts", "install.mjs"), "--force"], { cwd: temp, stdio: "pipe" });
-  assert.equal(readFileSync(dest, "utf8"), readFileSync(join(repoRoot, "agents", "goal.md"), "utf8"));
-});
-
-test("installer supports explicit target directory", () => {
-  const temp = mkdtempSync(join(tmpdir(), "goal-install-target-"));
-  const target = join(temp, "custom-opencode");
-  execFileSync("node", [join(repoRoot, "scripts", "install.mjs"), "--target", target], { cwd: temp, stdio: "pipe" });
-  assert.equal(existsSync(join(target, "agents", "goal.md")), true);
-  assert.equal(existsSync(join(target, "plugins", "goal-guard.js")), true);
-});