惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

V
V2EX
aimingoo的专栏
aimingoo的专栏
S
SegmentFault 最新的问题
博客园_首页
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
IT之家
IT之家
博客园 - 【当耐特】
月光博客
月光博客
C
Check Point Blog
T
The Blog of Author Tim Ferriss
罗磊的独立博客
博客园 - Franky
MongoDB | Blog
MongoDB | Blog
H
Help Net Security
Microsoft Security Blog
Microsoft Security Blog
B
Blog
阮一峰的网络日志
阮一峰的网络日志
腾讯CDC
美团技术团队
N
Netflix TechBlog - Medium
Stack Overflow Blog
Stack Overflow Blog
Y
Y Combinator Blog
L
LangChain Blog
The Cloudflare Blog

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
refactor(github): share guard comment helpers · openclaw/...
vincentkoc · 2026-06-20 · via Recent Commits to openclaw:main

@@ -9,7 +9,10 @@ import {

99

GITHUB_RESPONSE_BODY_MAX_BYTES,

1010

createGitHubApi,

1111

createGuardApproverChecks,

12+

createIssueMutationHelpers,

13+

guardCommentHeadSha,

1214

guardTrustedActorCandidates,

15+

isCommentNewerThan,

1316

readBoundedGitHubErrorText,

1417

readBoundedGitHubJson,

1518

} from "./guard-shared.mjs";

@@ -195,29 +198,8 @@ export async function findDependencyOverrideCommandAsync(input) {

195198

return null;

196199

}

197200198-

function isCommentNewerThan(comment, newerThan) {

199-

if (!newerThan) {

200-

return false;

201-

}

202-

const commentTime = Date.parse(comment.created_at ?? "");

203-

const barrierTime = Date.parse(newerThan);

204-

return Number.isFinite(commentTime) && Number.isFinite(barrierTime) && commentTime > barrierTime;

205-

}

206-207201

export function dependencyGuardCommentHeadSha(comment) {

208-

const body = comment?.body ?? "";

209-

const patterns = [

210-

/Approved SHA:\s+`([a-f0-9]{40})`/iu,

211-

/current head SHA\s+\(`([a-f0-9]{40})`\)/iu,

212-

/Current SHA:\s+`([a-f0-9]{40})`/iu,

213-

];

214-

for (const pattern of patterns) {

215-

const match = body.match(pattern);

216-

if (match?.[1]) {

217-

return match[1];

218-

}

219-

}

220-

return null;

202+

return guardCommentHeadSha(comment);

221203

}

222204223205

export function dependencyOverrideExpectedSha(existingGuardComment, currentHeadSha) {

@@ -497,17 +479,6 @@ export function githubApi(token, options = {}) {

497479

}

498480

return result.data;

499481

},

500-

paginate: async (path) => {

501-

const items = [];

502-

for (let page = 1; ; page += 1) {

503-

const separator = path.includes("?") ? "&" : "?";

504-

const pageItems = await api.request(`${path}${separator}per_page=100&page=${page}`);

505-

items.push(...pageItems);

506-

if (pageItems.length < 100) {

507-

return items;

508-

}

509-

}

510-

},

511482

};

512483

}

513484

@@ -716,66 +687,8 @@ async function main() {

716687

const existingGuardComment = findDependencyGuardComment(dependencyGraphGuardMarker);

717688

const labelNames = new Set(labels.map((label) => label.name));

718689719-

const ignoreUnavailableWritePermission = (action) => (error) => {

720-

if (error?.status === 403) {

721-

console.warn(`Skipping ${action}; token does not have write permission.`);

722-

return;

723-

}

724-

if (error?.status === 404 || error?.status === 422) {

725-

console.warn(`${action} is unavailable.`);

726-

return;

727-

}

728-

throw error;

729-

};

730-

const removeLabelIfPresent = async (label) => {

731-

if (!labelNames.has(label)) {

732-

return;

733-

}

734-

await api

735-

.request(`${issuePath}/labels/${encodeURIComponent(label)}`, {

736-

method: "DELETE",

737-

})

738-

.catch(ignoreUnavailableWritePermission(`label "${label}" removal`));

739-

labelNames.delete(label);

740-

};

741-

const addLabelIfMissing = async (label) => {

742-

if (labelNames.has(label)) {

743-

return;

744-

}

745-

await api

746-

.request(`${issuePath}/labels`, {

747-

method: "POST",

748-

body: JSON.stringify({ labels: [label] }),

749-

})

750-

.catch(ignoreUnavailableWritePermission(`label "${label}" update`));

751-

labelNames.add(label);

752-

};

753-

const deleteCommentIfPresent = async (comment) => {

754-

if (!comment) {

755-

return;

756-

}

757-

await api

758-

.request(`/repos/${owner}/${repo}/issues/comments/${comment.id}`, {

759-

method: "DELETE",

760-

})

761-

.catch(ignoreUnavailableWritePermission("comment deletion"));

762-

};

763-

const upsertComment = async (comment, body) => {

764-

if (comment) {

765-

return await api

766-

.request(`/repos/${owner}/${repo}/issues/comments/${comment.id}`, {

767-

method: "PATCH",

768-

body: JSON.stringify({ body }),

769-

})

770-

.catch(ignoreUnavailableWritePermission("comment update"));

771-

}

772-

return await api

773-

.request(`${issuePath}/comments`, {

774-

method: "POST",

775-

body: JSON.stringify({ body }),

776-

})

777-

.catch(ignoreUnavailableWritePermission("comment creation"));

778-

};

690+

const { removeLabelIfPresent, addLabelIfMissing, deleteCommentIfPresent, upsertComment } =

691+

createIssueMutationHelpers({ api, issuePath, owner, repo, labelNames });

779692780693

if (dependencyGraphFiles.length === 0) {

781694

await removeLabelIfPresent(dependencyChangedLabel);