mirror of
https://github.com/openclaw/openclaw.git
synced 2026-04-18 20:27:27 +00:00
diffs: move guidance from prompt hook to plugin skill
This commit is contained in:
committed by
Gustavo Madeira Santana
parent
997197c6c9
commit
f5150f03b9
@@ -12,6 +12,8 @@ read_when:
|
||||
|
||||
`diffs` is an optional plugin tool that turns change content into a read-only diff artifact for agents.
|
||||
|
||||
When the plugin is enabled, it can also provide a companion skill with usage guidance. It does not rely on unconditional `before_prompt_build` prompt injection.
|
||||
|
||||
It accepts either:
|
||||
|
||||
- `before` and `after` text
|
||||
|
||||
@@ -4,7 +4,7 @@ import { createMockServerResponse } from "../../src/test-utils/mock-http-respons
|
||||
import plugin from "./index.js";
|
||||
|
||||
describe("diffs plugin registration", () => {
|
||||
it("registers the tool, http route, and prompt guidance hook", () => {
|
||||
it("registers the tool and http route", () => {
|
||||
const registerTool = vi.fn();
|
||||
const registerHttpRoute = vi.fn();
|
||||
const on = vi.fn();
|
||||
@@ -43,8 +43,7 @@ describe("diffs plugin registration", () => {
|
||||
auth: "plugin",
|
||||
match: "prefix",
|
||||
});
|
||||
expect(on).toHaveBeenCalledTimes(1);
|
||||
expect(on.mock.calls[0]?.[0]).toBe("before_prompt_build");
|
||||
expect(on).not.toHaveBeenCalled();
|
||||
});
|
||||
|
||||
it("applies plugin-config defaults through registered tool and viewer handler", async () => {
|
||||
|
||||
@@ -7,7 +7,6 @@ import {
|
||||
resolveDiffsPluginSecurity,
|
||||
} from "./src/config.js";
|
||||
import { createDiffsHttpHandler } from "./src/http.js";
|
||||
import { DIFFS_AGENT_GUIDANCE } from "./src/prompt-guidance.js";
|
||||
import { DiffArtifactStore } from "./src/store.js";
|
||||
import { createDiffsTool } from "./src/tool.js";
|
||||
|
||||
@@ -35,9 +34,6 @@ const plugin = {
|
||||
allowRemoteViewer: security.allowRemoteViewer,
|
||||
}),
|
||||
});
|
||||
api.on("before_prompt_build", async () => ({
|
||||
prependContext: DIFFS_AGENT_GUIDANCE,
|
||||
}));
|
||||
},
|
||||
};
|
||||
|
||||
|
||||
@@ -2,6 +2,7 @@
|
||||
"id": "diffs",
|
||||
"name": "Diffs",
|
||||
"description": "Read-only diff viewer and file renderer for agents.",
|
||||
"skills": ["skills"],
|
||||
"uiHints": {
|
||||
"defaults.fontFamily": {
|
||||
"label": "Default Font",
|
||||
|
||||
22
extensions/diffs/skills/diffs/SKILL.md
Normal file
22
extensions/diffs/skills/diffs/SKILL.md
Normal file
@@ -0,0 +1,22 @@
|
||||
---
|
||||
name: diffs
|
||||
description: Use the diffs tool to produce real, shareable diffs (viewer URL, file artifact, or both) instead of manual edit summaries.
|
||||
---
|
||||
|
||||
When you need to show edits as a real diff, prefer the `diffs` tool instead of writing a manual summary.
|
||||
|
||||
The `diffs` tool accepts either `before` + `after` text, or a unified `patch` string.
|
||||
|
||||
Use `mode=view` when you want an interactive gateway-hosted viewer. After the tool returns, use `details.viewerUrl` with the canvas tool via `canvas present` or `canvas navigate`.
|
||||
|
||||
Use `mode=file` when you need a rendered file artifact. Set `fileFormat=png` (default) or `fileFormat=pdf`. The tool result includes `details.filePath`.
|
||||
|
||||
For large or high-fidelity files, use `fileQuality` (`standard`|`hq`|`print`) and optionally override `fileScale`/`fileMaxWidth`.
|
||||
|
||||
When you need to deliver the rendered file to a user or channel, do not rely on the raw tool-result renderer. Instead, call the `message` tool and pass `details.filePath` through `path` or `filePath`.
|
||||
|
||||
Use `mode=both` when you want both the gateway viewer URL and the rendered artifact.
|
||||
|
||||
If the user has configured diffs plugin defaults, prefer omitting `mode`, `theme`, `layout`, and related presentation options unless you need to override them for this specific diff.
|
||||
|
||||
Include `path` for before/after text when you know the file name.
|
||||
@@ -1,11 +0,0 @@
|
||||
export const DIFFS_AGENT_GUIDANCE = [
|
||||
"When you need to show edits as a real diff, prefer the `diffs` tool instead of writing a manual summary.",
|
||||
"The `diffs` tool accepts either `before` + `after` text, or a unified `patch` string.",
|
||||
"Use `mode=view` when you want an interactive gateway-hosted viewer. After the tool returns, use `details.viewerUrl` with the canvas tool via `canvas present` or `canvas navigate`.",
|
||||
"Use `mode=file` when you need a rendered file artifact. Set `fileFormat=png` (default) or `fileFormat=pdf`. The tool result includes `details.filePath`.",
|
||||
"For large or high-fidelity files, use `fileQuality` (`standard`|`hq`|`print`) and optionally override `fileScale`/`fileMaxWidth`.",
|
||||
"When you need to deliver the rendered file to a user or channel, do not rely on the raw tool-result renderer. Instead, call the `message` tool and pass `details.filePath` through `path` or `filePath`.",
|
||||
"Use `mode=both` when you want both the gateway viewer URL and the rendered artifact.",
|
||||
"If the user has configured diffs plugin defaults, prefer omitting `mode`, `theme`, `layout`, and related presentation options unless you need to override them for this specific diff.",
|
||||
"Include `path` for before/after text when you know the file name.",
|
||||
].join("\n");
|
||||
Reference in New Issue
Block a user