opencode-froggy 0.12.0 → 1.0.1

package/README.md

@@ -56,16 +56,18 @@ Add the plugin to your OpenCode configuration file (`opencode.json`):
 ```json
 {
   "$schema": "https://opencode.ai/config.json",
-  "plugin": ["opencode-froggy"]
+  "plugins": ["opencode-froggy"]
 }
 ```
 
+Requires OpenCode V2.
+
 ### From local files
 
 Alternatively, clone or copy the plugin files to one of these directories:
 
-- **Project-local**: `.opencode/plugin/opencode-froggy/`
-- **Global**: `~/.config/opencode/plugin/opencode-froggy/`
+- **Project-local**: `.opencode/plugins/opencode-froggy/`
+- **Global**: `~/.config/opencode/plugins/opencode-froggy/`
 
 ---
 
@@ -140,7 +142,7 @@ It does not modify Linear issues, add comments, or update project files.
 
 ## Skills
 
-Skills are contextual instructions loaded on demand via the `skill` tool. The agent invokes `skill({ name: "skill-name" })` to load the instructions when needed.
+Skills are contextual instructions loaded on demand via the `skill` tool. The agent invokes `skill({ id: "skill-name" })` to load the instructions when needed.
 
 ### Overview
 
@@ -270,7 +272,7 @@ A skill without `name` or `description` will be ignored.
 If a skill defines `use_when`, a directive is injected into the system prompt:
 
 ```
-MANDATORY: Call skill({ name: "my-skill" }) <use_when content>
+MANDATORY: Call skill({ id: "my-skill" }) <use_when content>
 ```
 
 This instructs the agent to load the skill when the specified condition is met.
@@ -361,8 +363,8 @@ promptSession({
 
 #### Behavior
 
-- If `sessionId` is not provided, the tool automatically targets the most recently created child session
-- Returns the text response from the child session
+- If `sessionId` is not provided, the tool targets the most recently created child session tracked since the plugin loaded (pass `sessionId` explicitly for older sessions)
+- Returns a confirmation once the message is admitted to the child session
 - Returns an error message if no child session exists for the current session
 
 ---
@@ -468,8 +470,7 @@ Promote an agent to primary (default) or specify a grade.
 #### Notes
 
 - Only agents from this plugin can be promoted (see [Agents](#agents) table)
-- Changes persist in memory until OpenCode restarts
-- After promotion, use `Tab` or `<leader>a` to select the agent
+- Changes apply immediately and persist across restarts via plugin storage
 
 ---
 

package/command/diff-summary.md

@@ -4,30 +4,49 @@ description: Show working tree changes or diff between branches ($1=source, $2=t
 
 # Diff Summary
 
+Branch arguments (if provided): source=$1 target=$2
+
 !`bash -c '
 SOURCE="$1"
 TARGET="$2"
 TARGET="${TARGET:-HEAD}"
 
+valid_ref() {
+  case "$REF" in
+    ""|-*) return 1 ;;
+  esac
+  git rev-parse --verify --quiet "$REF^{commit}" >/dev/null
+}
+
 if [ -n "$SOURCE" ]; then
+  REF="$SOURCE"
+  if ! valid_ref; then
+    echo "Invalid source ref: $SOURCE"
+    exit 0
+  fi
+  REF="$TARGET"
+  if ! valid_ref; then
+    echo "Invalid target ref: $TARGET"
+    exit 0
+  fi
   echo "## Branch Comparison: $SOURCE → $TARGET"
   echo ""
   git fetch --all --prune 2>/dev/null || true
-  
+
   echo "### Stats Overview"
-  git diff --stat "$TARGET"..."$SOURCE"
-  
+  git diff --stat "$TARGET"..."$SOURCE" --
+
   echo ""
   echo "### Commits"
-  git log --oneline --no-merges "$TARGET".."$SOURCE"
-  
+  git log --oneline --no-merges "$TARGET".."$SOURCE" --
+
   echo ""
   echo "### Files Changed"
-  git diff --name-only "$TARGET"..."$SOURCE"
-  
+  git diff --name-only "$TARGET"..."$SOURCE" --
+
   echo ""
   echo "### Full Diff"
-  git diff "$TARGET"..."$SOURCE"
+  git diff "$TARGET"..."$SOURCE" --
 else
   echo "## Status"
   git status --porcelain

package/command/doc-changes.md

@@ -1,6 +1,7 @@
 ---
 description: Update documentation based on uncommitted changes (new features only)
 agent: doc-writer
+subagent: true
 ---
 
 ## Analysis Phase

package/command/review-changes.md

@@ -1,6 +1,7 @@
 ---
 description: Review uncommitted changes (staged + unstaged, incl. untracked diffs)
 agent: code-reviewer
+subagent: true
 ---
 
 # Review: Working Tree → HEAD

package/command/review-pr.md

@@ -1,6 +1,7 @@
 ---
 description: Review changes from source branch into target branch
 agent: code-reviewer
+subagent: true
 ---
 
 # Review: $1 → $2

package/command/simplify-changes.md

@@ -1,6 +1,7 @@
 ---
 description: Simplify uncommitted changes (staged + unstaged, incl. untracked diffs)
 agent: code-simplifier
+subagent: true
 ---
 
 # Simplify: Working Tree → HEAD

package/dist/command-installer.d.ts

@@ -0,0 +1,6 @@
+export interface InstallFilesResult {
+    installed: string[];
+    skipped: string[];
+    updated: string[];
+}
+export declare function installBundledFiles(sourceDir: string, targetDir: string): InstallFilesResult;

package/dist/command-installer.js

@@ -0,0 +1,49 @@
+import { constants, copyFileSync, existsSync, mkdirSync, readdirSync, readFileSync, } from "node:fs";
+import { join } from "node:path";
+import { log } from "./logger";
+export function installBundledFiles(sourceDir, targetDir) {
+    const installed = [];
+    const skipped = [];
+    const updated = [];
+    if (!existsSync(sourceDir))
+        return { installed, skipped, updated };
+    try {
+        mkdirSync(targetDir, { recursive: true });
+    }
+    catch (error) {
+        log("[commands] failed to prepare target dir", { targetDir, error: String(error) });
+        return { installed, skipped, updated };
+    }
+    let files;
+    try {
+        files = readdirSync(sourceDir);
+    }
+    catch (error) {
+        log("[commands] failed to read bundled files", { sourceDir, error: String(error) });
+        return { installed, skipped, updated };
+    }
+    for (const file of files) {
+        if (!file.endsWith(".md"))
+            continue;
+        const source = join(sourceDir, file);
+        const target = join(targetDir, file);
+        try {
+            const content = readFileSync(source, "utf-8");
+            if (!existsSync(target)) {
+                copyFileSync(source, target, constants.COPYFILE_EXCL);
+                installed.push(file);
+                continue;
+            }
+            if (readFileSync(target, "utf-8") !== content) {
+                copyFileSync(source, target);
+                updated.push(file);
+                continue;
+            }
+            skipped.push(file);
+        }
+        catch (error) {
+            log("[commands] failed to install bundled file", { file, error: String(error) });
+        }
+    }
+    return { installed, skipped, updated };
+}

package/dist/command-installer.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/command-installer.test.js

@@ -0,0 +1,58 @@
+import { existsSync, mkdirSync, readFileSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { afterEach, beforeEach, describe, expect, it } from "vitest";
+import { installBundledFiles } from "./command-installer";
+describe("installBundledFiles", () => {
+    let sourceDir;
+    let targetDir;
+    beforeEach(() => {
+        const base = join(tmpdir(), `froggy-files-${Date.now()}-${Math.random().toString(16).slice(2)}`);
+        sourceDir = join(base, "source");
+        targetDir = join(base, "target");
+        mkdirSync(sourceDir, { recursive: true });
+    });
+    afterEach(() => {
+        rmSync(join(sourceDir, ".."), { recursive: true, force: true });
+    });
+    it("installs missing files", () => {
+        writeFileSync(join(sourceDir, "review-pr.md"), "# review");
+        const result = installBundledFiles(sourceDir, targetDir);
+        expect(result.installed).toEqual(["review-pr.md"]);
+        expect(readFileSync(join(targetDir, "review-pr.md"), "utf-8")).toBe("# review");
+    });
+    it("skips existing files with identical content", () => {
+        writeFileSync(join(sourceDir, "review-pr.md"), "# bundled");
+        mkdirSync(targetDir, { recursive: true });
+        writeFileSync(join(targetDir, "review-pr.md"), "# bundled");
+        const result = installBundledFiles(sourceDir, targetDir);
+        expect(result.skipped).toEqual(["review-pr.md"]);
+        expect(readFileSync(join(targetDir, "review-pr.md"), "utf-8")).toBe("# bundled");
+    });
+    it("updates an existing file when the bundled content differs", () => {
+        writeFileSync(join(sourceDir, "review-pr.md"), "# v2");
+        mkdirSync(targetDir, { recursive: true });
+        writeFileSync(join(targetDir, "review-pr.md"), "# v1");
+        const result = installBundledFiles(sourceDir, targetDir);
+        expect(result.updated).toEqual(["review-pr.md"]);
+        expect(readFileSync(join(targetDir, "review-pr.md"), "utf-8")).toBe("# v2");
+    });
+    it("never deletes a user-customized file", () => {
+        writeFileSync(join(sourceDir, "review-pr.md"), "# bundled");
+        mkdirSync(targetDir, { recursive: true });
+        writeFileSync(join(targetDir, "review-pr.md"), "# user customization");
+        const result = installBundledFiles(sourceDir, targetDir);
+        expect(result.updated).toEqual(["review-pr.md"]);
+        expect(existsSync(join(targetDir, "review-pr.md"))).toBe(true);
+    });
+    it("ignores non-markdown files", () => {
+        writeFileSync(join(sourceDir, "notes.txt"), "notes");
+        const result = installBundledFiles(sourceDir, targetDir);
+        expect(result.installed).toEqual([]);
+        expect(existsSync(join(targetDir, "notes.txt"))).toBe(false);
+    });
+    it("returns empty result for missing source dir", () => {
+        const result = installBundledFiles(join(sourceDir, "missing"), targetDir);
+        expect(result).toEqual({ installed: [], skipped: [], updated: [] });
+    });
+});

package/dist/config-paths.d.ts

@@ -9,3 +9,5 @@
 export declare function getUserConfigDir(): string;
 export declare function getGlobalHookDir(): string;
 export declare function getProjectHookDir(directory: string): string;
+export declare function getGlobalCommandDir(): string;
+export declare function getGlobalAgentDir(): string;

package/dist/config-paths.js

@@ -30,3 +30,9 @@ export function getGlobalHookDir() {
 export function getProjectHookDir(directory) {
     return join(directory, ".opencode", "hook");
 }
+export function getGlobalCommandDir() {
+    return join(getUserConfigDir(), "opencode", "commands");
+}
+export function getGlobalAgentDir() {
+    return join(getUserConfigDir(), "opencode", "agents");
+}

package/dist/config-paths.test.js

@@ -86,6 +86,22 @@ describe("config-paths", () => {
             expect(getGlobalHookDir()).toBe("/custom/config/opencode/hook");
         });
     });
+    describe("getGlobalCommandDir", () => {
+        it("should return correct path on Linux", async () => {
+            Object.defineProperty(process, "platform", { value: "linux" });
+            delete process.env.XDG_CONFIG_HOME;
+            const { getGlobalCommandDir } = await import("./config-paths");
+            expect(getGlobalCommandDir()).toBe(join(homedir(), ".config", "opencode", "commands"));
+        });
+    });
+    describe("getGlobalAgentDir", () => {
+        it("should return correct path on Linux", async () => {
+            Object.defineProperty(process, "platform", { value: "linux" });
+            delete process.env.XDG_CONFIG_HOME;
+            const { getGlobalAgentDir } = await import("./config-paths");
+            expect(getGlobalAgentDir()).toBe(join(homedir(), ".config", "opencode", "agents"));
+        });
+    });
     describe("getProjectHookDir", () => {
         it("should return correct path for project directory", async () => {
             const { getProjectHookDir } = await import("./config-paths");

package/dist/index.d.ts

@@ -1,5 +1,5 @@
-import { type Plugin } from "@opencode-ai/plugin";
+import { Plugin } from "@opencode/plugin";
 export { parseFrontmatter, loadAgents, loadCommands, type LoadedSkill } from "./loaders";
 export { buildSkillActivationBlock } from "./skill-activation";
-declare const SmartfrogPlugin: Plugin;
-export default SmartfrogPlugin;
+declare const _default: Plugin.Plugin;
+export default _default;