Skip to content

fix(memory-wiki): make wiki_lint report path tool-safe#83714

Closed
clawsweeper[bot] wants to merge 1 commit into
mainfrom
clawsweeper/automerge-openclaw-openclaw-83464
Closed

fix(memory-wiki): make wiki_lint report path tool-safe#83714
clawsweeper[bot] wants to merge 1 commit into
mainfrom
clawsweeper/automerge-openclaw-openclaw-83464

Conversation

@clawsweeper

@clawsweeper clawsweeper Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor

Makes #83464 merge-ready for the ClawSweeper automerge loop.
The edit pass should inspect the live PR diff, review comments, and failing checks; rebase if needed; keep the contributor branch credited; and stop only when validation is green or an external blocker is proven.

ClawSweeper 🐠 replacement reef notes:

Co-author credit kept:

fish notes: model gpt-5.5, reasoning high; reviewed against da27925.

Co-authored-by: Thiago Costa <71539514+ThiagoCAltoe@users.noreply.github.com>

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
@clawsweeper clawsweeper Bot added size: S extensions: memory-wiki clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge proof: supplied External PR includes structured after-fix real behavior proof. proof: sufficient ClawSweeper judged the real behavior proof convincing. labels May 18, 2026
@clawsweeper clawsweeper Bot added the P2 Normal backlog priority with limited blast radius. label May 18, 2026
@clawsweeper clawsweeper Bot added rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. merge-risk: 🚨 compatibility 🚨 May break existing users, config, migrations, defaults, or upgrade paths. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. clawsweeper Tracked by ClawSweeper automation labels May 18, 2026
@openclaw-barnacle openclaw-barnacle Bot removed proof: supplied External PR includes structured after-fix real behavior proof. proof: sufficient ClawSweeper judged the real behavior proof convincing. labels May 18, 2026
@clawsweeper

clawsweeper Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the context here. I swept through the related work, and this is now duplicate or superseded.

Close: this replacement PR is superseded by the landed memory-wiki fix in #83687, and the remaining branch diff only adds an already-closed source PR reference and contributor credit to the changelog.

Canonical path: Close this replacement PR as superseded by #83687; any optional changelog credit adjustment can be handled during maintainer release cleanup.

So I’m closing this here and keeping the remaining discussion on #83687.

Review details

Best possible solution:

Close this replacement PR as superseded by #83687; any optional changelog credit adjustment can be handled during maintainer release cleanup.

Do we have a high-confidence way to reproduce the issue?

Not applicable for this PR’s remaining diff: it changes only CHANGELOG.md. The underlying wiki_lint behavior is already covered on current main by source and regression-test evidence.

Is this the best way to solve the issue?

No. Keeping this replacement branch open is not the best path because the code/test fix already landed through #83687 and the remaining diff is non-essential changelog credit churn.

Security review:

Security review cleared: The diff only edits CHANGELOG.md and introduces no code execution, dependency, workflow, permission, credential, or supply-chain surface.

What I checked:

Likely related people:

  • vincentkoc: The memory-wiki stack and agent lint tool path were introduced/restored through commits 5716d83 and 9ce4abf. (role: feature introducer; confidence: high; commits: 5716d83336fd, 9ce4abfe558e; files: extensions/memory-wiki/src/tool.ts, extensions/memory-wiki/src/lint.ts)
  • LLagoon3: The landed superseding fix commit credits LLagoon3 as co-author for the current-main wiki_lint path-safe implementation. (role: source fix contributor; confidence: medium; commits: 9dc7bd4d056f; files: extensions/memory-wiki/src/tool.ts, extensions/memory-wiki/src/tool.test.ts, CHANGELOG.md)
  • Takhoffman: The landed superseding fix records approval by Takhoffman, and the replacement PR body records the automerge request by the same reviewer. (role: recent reviewer and automerge requester; confidence: medium; commits: 9dc7bd4d056f; files: extensions/memory-wiki/src/tool.ts, CHANGELOG.md)

Codex review notes: model gpt-5.5, reasoning high; reviewed against 06a39015f21c; fix evidence: commit 9dc7bd4d056f, main fix timestamp 2026-05-18T16:34:11Z.

@clawsweeper clawsweeper Bot added rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. status: 🚀 automerge armed This PR is in ClawSweeper's automerge lane. P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. and removed rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. P2 Normal backlog priority with limited blast radius. merge-risk: 🚨 compatibility 🚨 May break existing users, config, migrations, defaults, or upgrade paths. labels May 18, 2026
@clawsweeper

clawsweeper Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor Author

ClawSweeper applied the proposed close for this PR.

@clawsweeper clawsweeper Bot closed this May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge clawsweeper Tracked by ClawSweeper automation P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. size: XS status: 🚀 automerge armed This PR is in ClawSweeper's automerge lane.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants