chore: record ClawSweeper event comment routing
[skip ci]
This commit is contained in:
clawsweeper
parent
8858c46bb8
commit
32a44b8121
@ -1,20 +1,20 @@
|
||||
{
|
||||
"status": "executed",
|
||||
"generated_at": "2026-05-08T23:29:10.694Z",
|
||||
"generated_at": "2026-05-08T23:30:10.988Z",
|
||||
"repo": "openclaw/openclaw",
|
||||
"repair_repo": "openclaw/clawsweeper",
|
||||
"review_repo": "openclaw/clawsweeper",
|
||||
"since": "2026-05-08T20:29:08.196Z",
|
||||
"since": "2026-05-08T20:30:09.014Z",
|
||||
"execute": true,
|
||||
"force_reprocess": false,
|
||||
"max_comments": 1000,
|
||||
"item_numbers": [
|
||||
78595
|
||||
79549
|
||||
],
|
||||
"comment_ids": [],
|
||||
"max_autoclose_targets": 8,
|
||||
"scanned_comments": 164,
|
||||
"commands_seen": 77,
|
||||
"scanned_comments": 2,
|
||||
"commands_seen": 1,
|
||||
"actionable": 0,
|
||||
"trusted_bots": [
|
||||
"clawsweeper[bot]",
|
||||
@ -30,16 +30,16 @@
|
||||
"lookup_concurrency": 8,
|
||||
"commands": [
|
||||
{
|
||||
"idempotency_key": "clawsweeper-repair:openclaw/openclaw:78595:4391143349:2026-05-08T23:29:05Z:clawsweeper_needs_human",
|
||||
"comment_id": "4391143349",
|
||||
"comment_version_key": "4391143349:2026-05-08T23:29:05Z",
|
||||
"comment_url": "https://github.com/openclaw/openclaw/pull/78595#issuecomment-4391143349",
|
||||
"idempotency_key": "clawsweeper-repair:openclaw/openclaw:79549:4410396270:2026-05-08T23:30:06Z:clawsweeper_needs_human",
|
||||
"comment_id": "4410396270",
|
||||
"comment_version_key": "4410396270:2026-05-08T23:30:06Z",
|
||||
"comment_url": "https://github.com/openclaw/openclaw/pull/79549#issuecomment-4410396270",
|
||||
"repo": "openclaw/openclaw",
|
||||
"issue_number": 78595,
|
||||
"issue_number": 79549,
|
||||
"author": "clawsweeper[bot]",
|
||||
"author_association": "CONTRIBUTOR",
|
||||
"comment_created_at": "2026-05-06T19:15:41Z",
|
||||
"comment_updated_at": "2026-05-08T23:29:05Z",
|
||||
"comment_created_at": "2026-05-08T22:42:32Z",
|
||||
"comment_updated_at": "2026-05-08T23:30:06Z",
|
||||
"trigger": "trusted_bot",
|
||||
"command": "clawsweeper needs human",
|
||||
"intent": "clawsweeper_needs_human",
|
||||
@ -48,143 +48,50 @@
|
||||
"trusted_bot": true,
|
||||
"trusted_bot_author": "clawsweeper[bot]",
|
||||
"automation_source": "clawsweeper",
|
||||
"repair_reason": "Protected security/maintainer labels, XL storage architecture scope, missing real behavior proof, and P1 correctness/security findings require human maintainer and security review rather than an autonomous repair lane.; Needs attention: several legacy import paths can roll back newer SQLite auth, pairing, and access-control state during delayed doctor migrations.; Review finding: [P1] Re-allow configuredAgentsOnly i... (sha=e0c37e64ecf62bc1202fb7cc8f80d6e50fdc34a6)",
|
||||
"review_summary": "The PR replaces scattered file-backed runtime, session, transcript, plugin, cron, device, and pairing state with global and per-agent SQLite stores, plus migrations, backup, docs, client, and test updates.\n\nReproducibility: yes. for the review findings by source inspection against PR head `e0c37e64ecf62bc1202fb7cc8f80d6e50fdc34a6`: the schema/client mismatch, stale overwrite imports, and unbounded readers are visible in the changed sources. I did not run live scenarios because this is a read-only review and the PR lacks real behavior proof.",
|
||||
"review_followup": "Protected `security`/`maintainer` labels, XL storage architecture scope, missing real behavior proof, and P1 correctness/security findings require human maintainer and security review rather than an autonomous repair lane.",
|
||||
"repair_reason": "Contributor action is required to supply real behavior proof, and maintainer review should confirm the auth cleanup regression is fixed before merge.; Needs attention: The diff introduces a concrete auth-config persistence regression: inactive gateway auth secrets can remain on disk after the CLI reports that they were removed.; Review finding: [P1] Pass auth cleanup deletions through unsetPaths — src/config/io.writ... (sha=7035cd5b098a06e02f13ff3fee4420bdf074235a)",
|
||||
"review_summary": "The PR changes config merge-patch generation to preserve absent keys by default, applies explicit `unsetPaths` deletions in config write paths, and adds config persistence regression tests.\n\nReproducibility: yes. from source inspection: current main generates `null` delete markers for keys omitted from a partial target and then applies those markers to the persisted source config. I did not run tests because this review was constrained to read-only inspection.",
|
||||
"review_followup": "Contributor action is required to supply real behavior proof, and maintainer review should confirm the auth cleanup regression is fixed before merge.",
|
||||
"freeform_prompt": null,
|
||||
"expected_head_sha": "e0c37e64ecf62bc1202fb7cc8f80d6e50fdc34a6",
|
||||
"finding_id": null,
|
||||
"status": "skipped",
|
||||
"actions": [],
|
||||
"reason": "comment version already processed in ledger"
|
||||
},
|
||||
{
|
||||
"idempotency_key": "clawsweeper-repair:openclaw/openclaw:79541:4410250697:2026-05-08T23:25:07Z:clawsweeper_auto_repair",
|
||||
"comment_id": "4410250697",
|
||||
"comment_version_key": "4410250697:2026-05-08T23:25:07Z",
|
||||
"comment_url": "https://github.com/openclaw/openclaw/pull/79541#issuecomment-4410250697",
|
||||
"repo": "openclaw/openclaw",
|
||||
"issue_number": 79541,
|
||||
"author": "clawsweeper[bot]",
|
||||
"author_association": "CONTRIBUTOR",
|
||||
"comment_created_at": "2026-05-08T22:18:10Z",
|
||||
"comment_updated_at": "2026-05-08T23:25:07Z",
|
||||
"trigger": "trusted_bot",
|
||||
"command": "clawsweeper auto repair",
|
||||
"intent": "clawsweeper_auto_repair",
|
||||
"autoclose_message": null,
|
||||
"implementation_prompt": null,
|
||||
"trusted_bot": true,
|
||||
"trusted_bot_author": "clawsweeper[bot]",
|
||||
"automation_source": "clawsweeper",
|
||||
"repair_reason": "structured ClawSweeper needs-human verdict with repairable P-severity findings (sha=282fbcf1aef62d6a16791a717c244c267716eced)",
|
||||
"review_summary": "The PR widens ACP startup identity reconciliation to warm-restore resolved persistent sessions, adds an ACPX persisted-PID liveness check before stable reuse, exports the process liveness helper, and adds ACP/ACPX regression tests.\n\nReproducibility: yes. Current main source shows the startup scan skips resolved identities and ACPX stable reuse lacks a PID-liveness gate; I did not run the live docker restart path in this read-only review.",
|
||||
"review_followup": "Contributor-provided after-fix real behavior proof is still insufficient, and automation cannot supply proof from the contributor's deployed setup.",
|
||||
"freeform_prompt": null,
|
||||
"expected_head_sha": "282fbcf1aef62d6a16791a717c244c267716eced",
|
||||
"expected_head_sha": "7035cd5b098a06e02f13ff3fee4420bdf074235a",
|
||||
"finding_id": null,
|
||||
"status": "skipped",
|
||||
"actions": [],
|
||||
"target": {
|
||||
"kind": "pull_request",
|
||||
"title": "Refactor runtime state into SQLite",
|
||||
"branch": "exp-vfs",
|
||||
"head_sha": "e0c37e64ecf62bc1202fb7cc8f80d6e50fdc34a6",
|
||||
"author": "steipete",
|
||||
"title": "fix(config): preserve absent keys on partial writes; require unsetPaths for deletion",
|
||||
"branch": "edpiva/config-merge-patch-preserve",
|
||||
"head_sha": "7035cd5b098a06e02f13ff3fee4420bdf074235a",
|
||||
"author": "efpiva",
|
||||
"labels": [
|
||||
"docs",
|
||||
"channel: discord",
|
||||
"channel: googlechat",
|
||||
"channel: imessage",
|
||||
"channel: line",
|
||||
"channel: matrix",
|
||||
"channel: mattermost",
|
||||
"channel: msteams",
|
||||
"channel: nextcloud-talk",
|
||||
"channel: nostr",
|
||||
"channel: signal",
|
||||
"channel: slack",
|
||||
"channel: telegram",
|
||||
"channel: tlon",
|
||||
"channel: voice-call",
|
||||
"channel: whatsapp-web",
|
||||
"channel: zalo",
|
||||
"channel: zalouser",
|
||||
"app: macos",
|
||||
"app: web-ui",
|
||||
"gateway",
|
||||
"extensions: diagnostics-otel",
|
||||
"extensions: llm-task",
|
||||
"extensions: memory-core",
|
||||
"cli",
|
||||
"security",
|
||||
"scripts",
|
||||
"commands",
|
||||
"docker",
|
||||
"agents",
|
||||
"maintainer",
|
||||
"channel: feishu",
|
||||
"channel: twitch",
|
||||
"extensions: device-pair",
|
||||
"extensions: phone-control",
|
||||
"channel: irc",
|
||||
"size: XL",
|
||||
"extensions: acpx",
|
||||
"extensions: anthropic",
|
||||
"extensions: openai",
|
||||
"extensions: minimax",
|
||||
"extensions: cloudflare-ai-gateway",
|
||||
"extensions: kimi-coding",
|
||||
"extensions: kilocode",
|
||||
"channel: qqbot",
|
||||
"channel: qa-channel",
|
||||
"extensions: qa-lab",
|
||||
"extensions: memory-wiki",
|
||||
"extensions: codex",
|
||||
"extensions: lmstudio",
|
||||
"plugin: google-meet",
|
||||
"plugin: migrate-hermes",
|
||||
"channel: synology-chat",
|
||||
"plugin: migrate-claude",
|
||||
"plugin: file-transfer"
|
||||
"size: M",
|
||||
"triage: needs-real-behavior-proof"
|
||||
],
|
||||
"is_clawsweeper_pr": false,
|
||||
"cluster_id": null,
|
||||
"job_path": null,
|
||||
"automerge_cluster_id": "automerge-openclaw-openclaw-78595",
|
||||
"automerge_job_path": "jobs/openclaw/inbox/automerge-openclaw-openclaw-78595.md",
|
||||
"automerge_cluster_id": "automerge-openclaw-openclaw-79549",
|
||||
"automerge_job_path": "jobs/openclaw/inbox/automerge-openclaw-openclaw-79549.md",
|
||||
"mode": "autonomous",
|
||||
"mergeable": "CONFLICTING",
|
||||
"merge_state_status": "DIRTY",
|
||||
"mergeable": "MERGEABLE",
|
||||
"merge_state_status": "UNSTABLE",
|
||||
"review_decision": "",
|
||||
"checks": {
|
||||
"total": 106,
|
||||
"total": 116,
|
||||
"counts": {
|
||||
"FAILURE": 8,
|
||||
"SUCCESS": 91,
|
||||
"SKIPPED": 6,
|
||||
"FAILURE": 2,
|
||||
"CANCELLED": 1,
|
||||
"SUCCESS": 90,
|
||||
"SKIPPED": 22,
|
||||
"NEUTRAL": 1
|
||||
},
|
||||
"blockers": [
|
||||
"Real behavior proof:FAILURE",
|
||||
"checks-node-core-fast:FAILURE",
|
||||
"checks-node-core-src-security:FAILURE",
|
||||
"check-lint:FAILURE",
|
||||
"checks-node-channels:FAILURE",
|
||||
"checks-node-core-support-boundary:FAILURE",
|
||||
"check:FAILURE",
|
||||
"checks-node-core:FAILURE"
|
||||
"Real behavior proof:FAILURE"
|
||||
],
|
||||
"pending": [],
|
||||
"terminalBlockers": [
|
||||
"Real behavior proof:FAILURE",
|
||||
"checks-node-core-fast:FAILURE",
|
||||
"checks-node-core-src-security:FAILURE",
|
||||
"check-lint:FAILURE",
|
||||
"checks-node-channels:FAILURE",
|
||||
"checks-node-core-support-boundary:FAILURE",
|
||||
"check:FAILURE",
|
||||
"checks-node-core:FAILURE"
|
||||
"Real behavior proof:FAILURE"
|
||||
]
|
||||
}
|
||||
},
|
||||
@ -193,27 +100,27 @@
|
||||
],
|
||||
"ledger_changed": true,
|
||||
"timings": {
|
||||
"total_ms": 2765,
|
||||
"total_ms": 2192,
|
||||
"phases": [
|
||||
{
|
||||
"name": "list_candidate_comments",
|
||||
"ms": 308
|
||||
"ms": 233
|
||||
},
|
||||
{
|
||||
"name": "prehydrate_command_lookups",
|
||||
"ms": 2175
|
||||
"ms": 1726
|
||||
},
|
||||
{
|
||||
"name": "classify_commands",
|
||||
"ms": 8
|
||||
"ms": 2
|
||||
},
|
||||
{
|
||||
"name": "execute_commands",
|
||||
"ms": 256
|
||||
"ms": 208
|
||||
},
|
||||
{
|
||||
"name": "append_ledger",
|
||||
"ms": 0
|
||||
"ms": 1
|
||||
}
|
||||
]
|
||||
}
|
||||
|
||||
@ -1,36 +1,6 @@
|
||||
{
|
||||
"updated_at": "2026-05-08T23:29:10.950Z",
|
||||
"updated_at": "2026-05-08T23:30:11.197Z",
|
||||
"commands": [
|
||||
{
|
||||
"idempotency_key": "clawsweeper-repair:openclaw/openclaw:64512:4323924559:2026-05-08T05:03:19Z:clawsweeper_auto_repair",
|
||||
"comment_id": "4323924559",
|
||||
"comment_version_key": "4323924559:2026-05-08T05:03:19Z",
|
||||
"comment_url": "https://github.com/openclaw/openclaw/pull/64512#issuecomment-4323924559",
|
||||
"comment_created_at": "2026-04-27T03:22:13Z",
|
||||
"comment_updated_at": "2026-05-08T05:03:19Z",
|
||||
"repo": "openclaw/openclaw",
|
||||
"issue_number": 64512,
|
||||
"author": "clawsweeper[bot]",
|
||||
"author_association": "CONTRIBUTOR",
|
||||
"trigger": "trusted_bot",
|
||||
"command": "clawsweeper auto repair",
|
||||
"intent": "clawsweeper_auto_repair",
|
||||
"trusted_bot": true,
|
||||
"trusted_bot_author": "clawsweeper[bot]",
|
||||
"automation_source": "clawsweeper",
|
||||
"repair_reason": "structured ClawSweeper marker: fix-required (finding=review-feedback sha=98b6ea945dcbeaf84f35a7566515005215052e0b)",
|
||||
"expected_head_sha": "98b6ea945dcbeaf84f35a7566515005215052e0b",
|
||||
"finding_id": "review-feedback",
|
||||
"status": "skipped",
|
||||
"processed_at": "2026-05-08T05:03:23.820Z",
|
||||
"target": {
|
||||
"kind": "pull_request",
|
||||
"branch": "service-tier-bedrock",
|
||||
"head_sha": "98b6ea945dcbeaf84f35a7566515005215052e0b",
|
||||
"cluster_id": null,
|
||||
"job_path": null
|
||||
}
|
||||
},
|
||||
{
|
||||
"idempotency_key": "clawsweeper-repair:openclaw/openclaw:79204:4402996164:2026-05-08T05:05:16Z:clawsweeper_needs_human",
|
||||
"comment_id": "4402996164",
|
||||
@ -30244,6 +30214,36 @@
|
||||
"cluster_id": null,
|
||||
"job_path": null
|
||||
}
|
||||
},
|
||||
{
|
||||
"idempotency_key": "clawsweeper-repair:openclaw/openclaw:79549:4410396270:2026-05-08T23:30:06Z:clawsweeper_needs_human",
|
||||
"comment_id": "4410396270",
|
||||
"comment_version_key": "4410396270:2026-05-08T23:30:06Z",
|
||||
"comment_url": "https://github.com/openclaw/openclaw/pull/79549#issuecomment-4410396270",
|
||||
"comment_created_at": "2026-05-08T22:42:32Z",
|
||||
"comment_updated_at": "2026-05-08T23:30:06Z",
|
||||
"repo": "openclaw/openclaw",
|
||||
"issue_number": 79549,
|
||||
"author": "clawsweeper[bot]",
|
||||
"author_association": "CONTRIBUTOR",
|
||||
"trigger": "trusted_bot",
|
||||
"command": "clawsweeper needs human",
|
||||
"intent": "clawsweeper_needs_human",
|
||||
"trusted_bot": true,
|
||||
"trusted_bot_author": "clawsweeper[bot]",
|
||||
"automation_source": "clawsweeper",
|
||||
"repair_reason": "Contributor action is required to supply real behavior proof, and maintainer review should confirm the auth cleanup regression is fixed before merge.; Needs attention: The diff introduces a concrete auth-config persistence regression: inactive gateway auth secrets can remain on disk after the CLI reports that they were removed.; Review finding: [P1] Pass auth cleanup deletions through unsetPaths — src/config/io.writ... (sha=7035cd5b098a06e02f13ff3fee4420bdf074235a)",
|
||||
"expected_head_sha": "7035cd5b098a06e02f13ff3fee4420bdf074235a",
|
||||
"finding_id": null,
|
||||
"status": "skipped",
|
||||
"processed_at": "2026-05-08T23:30:11.196Z",
|
||||
"target": {
|
||||
"kind": "pull_request",
|
||||
"branch": "edpiva/config-merge-patch-preserve",
|
||||
"head_sha": "7035cd5b098a06e02f13ff3fee4420bdf074235a",
|
||||
"cluster_id": null,
|
||||
"job_path": null
|
||||
}
|
||||
}
|
||||
]
|
||||
}
|
||||
|
||||
Loading…
Reference in New Issue
Block a user