mirror of
https://github.com/openclaw/openclaw.git
synced 2026-05-06 09:30:43 +00:00
Summary: - The PR enables Slack draft preview streaming for flat DMs in all non-off modes, updates Slack streaming tests/docs/config metadata/changelog, and refreshes small guard baselines. - Reproducibility: yes. source-reproducible on current main: the helper returns false for `mode: "partial"` in ... current test asserts that disabled path. I did not run tests because this review was explicitly read-only. Automerge notes: - Ran the ClawSweeper repair loop before final review. - Included post-review commit in the final squash: fix(slack): enable preview streaming in flat DMs (replyToMode: off) - Included post-review commit in the final squash: fix(clawsweeper): address review for automerge-openclaw-openclaw-5654… Validation: - ClawSweeper review passed for head52e5d74ef9. - Required merge gates passed before the squash merge. Prepared head SHA:52e5d74ef9Review: https://github.com/openclaw/openclaw/pull/76330#issuecomment-4365017023 Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com> Co-authored-by: bob <637186+HangGlidersRule@users.noreply.github.com>
105 lines
4.9 KiB
JavaScript
105 lines
4.9 KiB
JavaScript
#!/usr/bin/env node
|
|
|
|
import ts from "typescript";
|
|
import { bundledPluginCallsite } from "./lib/bundled-plugin-paths.mjs";
|
|
import { runCallsiteGuard } from "./lib/callsite-guard.mjs";
|
|
import {
|
|
collectCallExpressionLines,
|
|
runAsScript,
|
|
unwrapExpression,
|
|
} from "./lib/ts-guard-utils.mjs";
|
|
|
|
const sourceRoots = ["src/channels", "src/routing", "src/line", "extensions"];
|
|
|
|
// Temporary allowlist for legacy callsites. New raw fetch callsites in channel/plugin runtime
|
|
// code should be rejected and migrated to fetchWithSsrFGuard/shared channel helpers.
|
|
const allowedRawFetchCallsites = new Set([
|
|
bundledPluginCallsite("bluebubbles", "src/test-harness.ts", 132),
|
|
bundledPluginCallsite("bluebubbles", "src/types.ts", 204),
|
|
bundledPluginCallsite("browser", "src/browser/cdp.helpers.ts", 268),
|
|
bundledPluginCallsite("browser", "src/browser/client-fetch.ts", 192),
|
|
bundledPluginCallsite("chutes", "models.ts", 536),
|
|
bundledPluginCallsite("chutes", "models.ts", 543),
|
|
bundledPluginCallsite("discord", "src/monitor/gateway-plugin.ts", 417),
|
|
bundledPluginCallsite("discord", "src/monitor/gateway-plugin.ts", 483),
|
|
bundledPluginCallsite("discord", "src/voice-message.ts", 298),
|
|
bundledPluginCallsite("discord", "src/voice-message.ts", 333),
|
|
bundledPluginCallsite("elevenlabs", "speech-provider.ts", 295),
|
|
bundledPluginCallsite("elevenlabs", "tts.ts", 74),
|
|
bundledPluginCallsite("feishu", "src/monitor.webhook.test-helpers.ts", 25),
|
|
bundledPluginCallsite("github-copilot", "login.ts", 69),
|
|
bundledPluginCallsite("github-copilot", "login.ts", 101),
|
|
bundledPluginCallsite("googlechat", "src/auth.ts", 83),
|
|
bundledPluginCallsite("huggingface", "models.ts", 143),
|
|
bundledPluginCallsite("kilocode", "provider-models.ts", 130),
|
|
bundledPluginCallsite("matrix", "src/matrix/sdk/transport.ts", 112),
|
|
bundledPluginCallsite("microsoft-foundry", "onboard.ts", 479),
|
|
bundledPluginCallsite("microsoft", "speech-provider.ts", 140),
|
|
bundledPluginCallsite("minimax", "oauth.ts", 66),
|
|
bundledPluginCallsite("minimax", "oauth.ts", 107),
|
|
bundledPluginCallsite("minimax", "tts.ts", 52),
|
|
bundledPluginCallsite("msteams", "src/graph.ts", 47),
|
|
bundledPluginCallsite("msteams", "src/sdk.ts", 400),
|
|
bundledPluginCallsite("msteams", "src/sdk.ts", 441),
|
|
bundledPluginCallsite("ollama", "src/stream.ts", 649),
|
|
bundledPluginCallsite("openai", "tts.ts", 149),
|
|
bundledPluginCallsite("qa-channel", "src/bus-client.ts", 41),
|
|
bundledPluginCallsite("qa-channel", "src/bus-client.ts", 221),
|
|
bundledPluginCallsite("qa-lab", "src/docker-up.runtime.ts", 274),
|
|
bundledPluginCallsite("qa-lab", "src/gateway-child.ts", 489),
|
|
bundledPluginCallsite("qa-lab", "src/suite.ts", 330),
|
|
bundledPluginCallsite("qa-lab", "src/suite.ts", 341),
|
|
bundledPluginCallsite("qa-lab", "web/src/app.ts", 21),
|
|
bundledPluginCallsite("qa-lab", "web/src/app.ts", 29),
|
|
bundledPluginCallsite("qa-lab", "web/src/app.ts", 37),
|
|
bundledPluginCallsite("qqbot", "src/engine/api/api-client.ts", 124),
|
|
bundledPluginCallsite("qqbot", "src/engine/api/media-chunked.ts", 554),
|
|
bundledPluginCallsite("qqbot", "src/engine/api/token.ts", 211),
|
|
bundledPluginCallsite("qqbot", "src/engine/tools/channel-api.ts", 178),
|
|
bundledPluginCallsite("qqbot", "src/engine/utils/stt.ts", 87),
|
|
bundledPluginCallsite("signal", "src/install-signal-cli.ts", 224),
|
|
bundledPluginCallsite("slack", "src/monitor/media.ts", 106),
|
|
bundledPluginCallsite("slack", "src/monitor/media.ts", 125),
|
|
bundledPluginCallsite("slack", "src/monitor/media.ts", 130),
|
|
bundledPluginCallsite("venice", "models.ts", 552),
|
|
bundledPluginCallsite("vercel-ai-gateway", "models.ts", 181),
|
|
bundledPluginCallsite("voice-call", "src/providers/twilio/api.ts", 23),
|
|
]);
|
|
|
|
function isRawFetchCall(expression) {
|
|
const callee = unwrapExpression(expression);
|
|
if (ts.isIdentifier(callee)) {
|
|
return callee.text === "fetch";
|
|
}
|
|
if (ts.isPropertyAccessExpression(callee)) {
|
|
return (
|
|
ts.isIdentifier(callee.expression) &&
|
|
callee.expression.text === "globalThis" &&
|
|
callee.name.text === "fetch"
|
|
);
|
|
}
|
|
return false;
|
|
}
|
|
|
|
export function findRawFetchCallLines(content, fileName = "source.ts") {
|
|
const sourceFile = ts.createSourceFile(fileName, content, ts.ScriptTarget.Latest, true);
|
|
return collectCallExpressionLines(ts, sourceFile, (node) =>
|
|
isRawFetchCall(node.expression) ? node.expression : null,
|
|
);
|
|
}
|
|
|
|
export async function main() {
|
|
await runCallsiteGuard({
|
|
importMetaUrl: import.meta.url,
|
|
sourceRoots,
|
|
extraTestSuffixes: [".browser.test.ts", ".node.test.ts"],
|
|
findCallLines: findRawFetchCallLines,
|
|
skipRelativePath: (relPath) => relPath.includes("/test-support/"),
|
|
allowCallsite: (callsite) => allowedRawFetchCallsites.has(callsite),
|
|
header: "Found raw fetch() usage in channel/plugin runtime sources outside allowlist:",
|
|
footer: "Use fetchWithSsrFGuard() or existing channel/plugin SDK wrappers for network calls.",
|
|
});
|
|
}
|
|
|
|
runAsScript(import.meta.url, main);
|