vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-06 15:50:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

Gateway/skills: dedupe skills prefix-match + drop dead fallback on log

Browse Source
This commit is contained in:
Xan Torres
2026-04-15 23:43:23 +02:00
committed by Ayaan Zaidi
parent b555214c96
commit d7f489f85e
1 changed files with 12 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
src/gateway/config-reload.ts
View File

@@ -35,20 +35,18 @@ const MISSING_CONFIG_MAX_RETRIES = 2;
*/
const SKILLS_INVALIDATION_PREFIXES = ["skills"] as const;
export function shouldInvalidateSkillsSnapshotForPaths(changedPaths: string[]): boolean {
return changedPaths.some((path) =>
SKILLS_INVALIDATION_PREFIXES.some(
(prefix) => path === prefix || path.startsWith(`${prefix}.`),
),
function matchesSkillsInvalidationPrefix(path: string): boolean {
return SKILLS_INVALIDATION_PREFIXES.some(
(prefix) => path === prefix || path.startsWith(`${prefix}.`),
);
}
function firstSkillsChangedPath(changedPaths: string[]): string | undefined {
return changedPaths.find((path) =>
SKILLS_INVALIDATION_PREFIXES.some(
(prefix) => path === prefix || path.startsWith(`${prefix}.`),
),
);
return changedPaths.find(matchesSkillsInvalidationPrefix);
}
export function shouldInvalidateSkillsSnapshotForPaths(changedPaths: string[]): boolean {
return firstSkillsChangedPath(changedPaths) !== undefined;
}
export function diffConfigPaths(prev: unknown, next: unknown, prefix = ""): string[] {
@@ -194,12 +192,10 @@ export function startGatewayConfigReloader(opts: {
// the user touches skills.* config. Without this, sessions keep advertising
// tools that no longer exist in the allowlist, which causes infinite
// tool-not-found loops against the model.
if (shouldInvalidateSkillsSnapshotForPaths(changedPaths)) {
const changedPath = firstSkillsChangedPath(changedPaths);
bumpSkillsSnapshotVersion({ reason: "config-change", changedPath });
opts.log.info(
`skills snapshot invalidated by config change (${changedPath ?? "skills.*"})`,
);
const skillsChangedPath = firstSkillsChangedPath(changedPaths);
if (skillsChangedPath !== undefined) {
bumpSkillsSnapshotVersion({ reason: "config-change", changedPath: skillsChangedPath });
opts.log.info(`skills snapshot invalidated by config change (${skillsChangedPath})`);
}
opts.log.info(`config change detected; evaluating reload (${changedPaths.join(", ")})`);