Some checks failed
ClawSweeper Dispatch / dispatch (push) Has been cancelled
CodeQL / Security High (actions) (push) Has been cancelled
CodeQL / Security High (channel-runtime-boundary) (push) Has been cancelled
CodeQL / Security High (core-auth-secrets) (push) Has been cancelled
CodeQL / Security High (mcp-process-tool-boundary) (push) Has been cancelled
CodeQL / Security High (network-ssrf-boundary) (push) Has been cancelled
CodeQL / Security High (plugin-trust-boundary) (push) Has been cancelled
CodeQL / Security High (process-exec-boundary) (push) Has been cancelled
Docs Sync Publish Repo / sync-publish-repo (push) Has been cancelled
Docs / docs (push) Has been cancelled
OpenClaw Stable Main Closeout / Resolve stable release closeout inputs (push) Has been cancelled
OpenClaw Stable Main Closeout / Verify stable main closeout (push) Has been cancelled
Workflow Sanity / no-tabs (push) Has been cancelled
Workflow Sanity / actionlint (push) Has been cancelled
Workflow Sanity / generated-doc-baselines (push) Has been cancelled
CI / runner-admission (push) Has been cancelled
CI / preflight (push) Has been cancelled
CI / security-fast (push) Has been cancelled
CI / pnpm-store-warmup (push) Has been cancelled
CI / build-artifacts (push) Has been cancelled
CI / native-i18n (push) Has been cancelled
CI / ${{ matrix.check_name }} (push) Has been cancelled
CI / ${{ matrix.checkName }} (push) Has been cancelled
CI / checks-node-compat-node22 (push) Has been cancelled
CI / check-bundled-channel-config-metadata (push) Has been cancelled
CI / check-dependencies (push) Has been cancelled
CI / check-guards (push) Has been cancelled
CI / check-lint (push) Has been cancelled
CI / check-prod-types (push) Has been cancelled
CI / check-shrinkwrap (push) Has been cancelled
CI / check-test-types (push) Has been cancelled
CI / check-additional-boundaries-a (push) Has been cancelled
CI / check-additional-boundaries-bcd (push) Has been cancelled
CI / check-additional-extension-bundled (push) Has been cancelled
CI / check-additional-extension-channels (push) Has been cancelled
CI / check-additional-extension-package-boundary (push) Has been cancelled
CI / check-additional-runtime-topology-architecture (push) Has been cancelled
CI / check-session-accessor-boundary (push) Has been cancelled
CI / check-session-transcript-reader-boundary (push) Has been cancelled
CI / check-docs (push) Has been cancelled
CI / skills-python (push) Has been cancelled
CI / macos-swift (push) Has been cancelled
CI / ios-build (push) Has been cancelled
CI / ci-timings-summary (push) Has been cancelled
Native App Locale Refresh / Refresh native fa (push) Has been cancelled
Native App Locale Refresh / Refresh native fr (push) Has been cancelled
Native App Locale Refresh / Refresh native hi (push) Has been cancelled
Native App Locale Refresh / Refresh native id (push) Has been cancelled
Native App Locale Refresh / Refresh native it (push) Has been cancelled
Native App Locale Refresh / Refresh native ja-JP (push) Has been cancelled
Control UI Locale Refresh / plan (push) Has been cancelled
Control UI Locale Refresh / Refresh ${{ matrix.locale }} (push) Has been cancelled
Control UI Locale Refresh / Commit control UI locale refresh (push) Has been cancelled
Live Media Runner Image / Build live media runner image (push) Has been cancelled
Native App Locale Refresh / Refresh native ar (push) Has been cancelled
Native App Locale Refresh / Refresh native de (push) Has been cancelled
Native App Locale Refresh / Refresh native es (push) Has been cancelled
Native App Locale Refresh / Refresh native ko (push) Has been cancelled
Native App Locale Refresh / Refresh native nl (push) Has been cancelled
Native App Locale Refresh / Refresh native pl (push) Has been cancelled
Native App Locale Refresh / Refresh native pt-BR (push) Has been cancelled
Native App Locale Refresh / Refresh native ru (push) Has been cancelled
Native App Locale Refresh / Refresh native sv (push) Has been cancelled
Native App Locale Refresh / Refresh native th (push) Has been cancelled
Native App Locale Refresh / Refresh native tr (push) Has been cancelled
Native App Locale Refresh / Refresh native uk (push) Has been cancelled
Native App Locale Refresh / Refresh native vi (push) Has been cancelled
Native App Locale Refresh / Refresh native zh-CN (push) Has been cancelled
Native App Locale Refresh / Refresh native zh-TW (push) Has been cancelled
Native App Locale Refresh / Commit native locale refresh (push) Has been cancelled
Plugin Init Scaffold Validation / Validate provider scaffold (push) Has been cancelled
Plugin NPM Release / preview_plugins_npm (push) Has been cancelled
Plugin NPM Release / Validate release publish approval (push) Has been cancelled
Plugin NPM Release / preview_plugin_pack (push) Has been cancelled
Plugin NPM Release / publish_plugins_npm (push) Has been cancelled
Sandbox Common Smoke / sandbox-common-smoke (push) Has been cancelled
Website Installer Sync / static (push) Has been cancelled
Website Installer Sync / linux-docker (push) Has been cancelled
Website Installer Sync / macos-installer (push) Has been cancelled
Website Installer Sync / windows-installer (push) Has been cancelled
Website Installer Sync / sync-website (push) Has been cancelled
Adolf is a fork/vendored clone of github.com/openclaw/openclaw (v2026.6.11), free to diverge. Tree copied sans upstream .git; upstream remote added for future syncs. Node pinned to 24 (.nvmrc); engines already require >=22.19. Preserves docs/ARCHITECTURE.md. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LeqyaxJF2nbRXJtae2kNB2
182 lines
5.2 KiB
JavaScript
182 lines
5.2 KiB
JavaScript
#!/usr/bin/env node
|
|
|
|
// Prevents direct pairing-store group auth reads outside resolver helpers.
|
|
import ts from "typescript";
|
|
import { createPairingGuardContext } from "./lib/pairing-guard-context.mjs";
|
|
import {
|
|
collectFileViolations,
|
|
getPropertyNameText,
|
|
runAsScript,
|
|
toLine,
|
|
} from "./lib/ts-guard-utils.mjs";
|
|
|
|
const { repoRoot, sourceRoots, resolveFromRepo } = createPairingGuardContext(import.meta.url);
|
|
|
|
const allowedFiles = new Set([
|
|
resolveFromRepo("src/channels/message-access/legacy-policy.ts"),
|
|
resolveFromRepo("src/channels/allow-from.ts"),
|
|
// Config migration/audit logic may intentionally reference store + group fields.
|
|
resolveFromRepo("src/security/fix.ts"),
|
|
resolveFromRepo("src/security/audit-channel.ts"),
|
|
]);
|
|
|
|
const storeIdentifierRe = /^(?:storeAllowFrom|storedAllowFrom|storeAllowList)$/i;
|
|
const groupNameRe =
|
|
/(?:groupAllowFrom|effectiveGroupAllowFrom|groupAllowed|groupAllow|groupAuth|groupSender)/i;
|
|
const storeSourceCallNames = new Set([
|
|
"readChannelAllowFromStore",
|
|
"readChannelAllowFromStoreSync",
|
|
"readStoreAllowFromForDmPolicy",
|
|
]);
|
|
const allowedResolverCallNames = new Set([
|
|
"resolveEffectiveAllowFromLists",
|
|
"resolveDmGroupAccessWithLists",
|
|
"resolveMattermostEffectiveAllowFromLists",
|
|
"resolveIrcEffectiveAllowlists",
|
|
]);
|
|
|
|
function getDeclarationNameText(name) {
|
|
if (ts.isIdentifier(name)) {
|
|
return name.text;
|
|
}
|
|
if (ts.isObjectBindingPattern(name) || ts.isArrayBindingPattern(name)) {
|
|
return name.getText();
|
|
}
|
|
return null;
|
|
}
|
|
|
|
function containsPairingStoreSource(node) {
|
|
let found = false;
|
|
const visit = (current) => {
|
|
if (found) {
|
|
return;
|
|
}
|
|
if (ts.isIdentifier(current) && storeIdentifierRe.test(current.text)) {
|
|
found = true;
|
|
return;
|
|
}
|
|
if (ts.isCallExpression(current)) {
|
|
const callName = getCallName(current);
|
|
if (callName && storeSourceCallNames.has(callName)) {
|
|
found = true;
|
|
return;
|
|
}
|
|
}
|
|
ts.forEachChild(current, visit);
|
|
};
|
|
visit(node);
|
|
return found;
|
|
}
|
|
|
|
function getCallName(node) {
|
|
if (!ts.isCallExpression(node)) {
|
|
return null;
|
|
}
|
|
if (ts.isIdentifier(node.expression)) {
|
|
return node.expression.text;
|
|
}
|
|
if (ts.isPropertyAccessExpression(node.expression)) {
|
|
return node.expression.name.text;
|
|
}
|
|
return null;
|
|
}
|
|
|
|
function isSuspiciousNormalizeWithStoreCall(node) {
|
|
if (!ts.isCallExpression(node)) {
|
|
return false;
|
|
}
|
|
if (!ts.isIdentifier(node.expression) || node.expression.text !== "normalizeAllowFromWithStore") {
|
|
return false;
|
|
}
|
|
const firstArg = node.arguments[0];
|
|
if (!firstArg || !ts.isObjectLiteralExpression(firstArg)) {
|
|
return false;
|
|
}
|
|
let hasStoreProp = false;
|
|
let hasGroupAllowProp = false;
|
|
for (const property of firstArg.properties) {
|
|
if (!ts.isPropertyAssignment(property)) {
|
|
continue;
|
|
}
|
|
const name = getPropertyNameText(property.name);
|
|
if (!name) {
|
|
continue;
|
|
}
|
|
if (name === "storeAllowFrom" && containsPairingStoreSource(property.initializer)) {
|
|
hasStoreProp = true;
|
|
}
|
|
if (name === "allowFrom" && groupNameRe.test(property.initializer.getText())) {
|
|
hasGroupAllowProp = true;
|
|
}
|
|
}
|
|
return hasStoreProp && hasGroupAllowProp;
|
|
}
|
|
|
|
function findViolations(content, filePath) {
|
|
const sourceFile = ts.createSourceFile(filePath, content, ts.ScriptTarget.Latest, true);
|
|
const violations = [];
|
|
|
|
const visit = (node) => {
|
|
if (ts.isVariableDeclaration(node) && node.initializer) {
|
|
const name = getDeclarationNameText(node.name);
|
|
if (name && groupNameRe.test(name) && containsPairingStoreSource(node.initializer)) {
|
|
const callName = getCallName(node.initializer);
|
|
if (callName && allowedResolverCallNames.has(callName)) {
|
|
ts.forEachChild(node, visit);
|
|
return;
|
|
}
|
|
violations.push({
|
|
line: toLine(sourceFile, node),
|
|
reason: `group-scoped variable "${name}" references pairing-store identifiers`,
|
|
});
|
|
}
|
|
}
|
|
|
|
if (ts.isPropertyAssignment(node)) {
|
|
const propName = getPropertyNameText(node.name);
|
|
if (propName && groupNameRe.test(propName) && containsPairingStoreSource(node.initializer)) {
|
|
violations.push({
|
|
line: toLine(sourceFile, node),
|
|
reason: `group-scoped property "${propName}" references pairing-store identifiers`,
|
|
});
|
|
}
|
|
}
|
|
|
|
if (isSuspiciousNormalizeWithStoreCall(node)) {
|
|
violations.push({
|
|
line: toLine(sourceFile, node),
|
|
reason: "group allowlist uses normalizeAllowFromWithStore(...) with pairing-store entries",
|
|
});
|
|
}
|
|
|
|
ts.forEachChild(node, visit);
|
|
};
|
|
|
|
visit(sourceFile);
|
|
return violations;
|
|
}
|
|
|
|
async function main() {
|
|
const violations = await collectFileViolations({
|
|
sourceRoots,
|
|
repoRoot,
|
|
findViolations,
|
|
skipFile: (filePath) => allowedFiles.has(filePath),
|
|
});
|
|
|
|
if (violations.length === 0) {
|
|
return;
|
|
}
|
|
|
|
console.error("Found pairing-store identifiers referenced in group auth composition:");
|
|
for (const violation of violations) {
|
|
console.error(`- ${violation.path}:${violation.line} (${violation.reason})`);
|
|
}
|
|
console.error(
|
|
"Group auth must be composed via shared resolvers (resolveDmGroupAccessWithLists / resolveEffectiveAllowFromLists).",
|
|
);
|
|
process.exit(1);
|
|
}
|
|
|
|
runAsScript(import.meta.url, main);
|