-
-
Notifications
You must be signed in to change notification settings - Fork 79.2k
memory-wiki: wiki_lint tool can fail with 'unable to resolve opened file path' while CLI lint succeeds #83420
Copy link
Copy link
Closed
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.clawsweeper:needs-live-reproClawSweeper needs live local, crabbox, or manual validation to confirm this issue.ClawSweeper needs live local, crabbox, or manual validation to confirm this issue.clawsweeper:queueable-fixClawSweeper marked this issue as an existing queue_fix_pr work candidate.ClawSweeper marked this issue as an existing queue_fix_pr work candidate.issue-rating: 🐚 platinum hermitGood issue quality with a plausible reproduction path needing some confirmation.Good issue quality with a plausible reproduction path needing some confirmation.
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.clawsweeper:needs-live-reproClawSweeper needs live local, crabbox, or manual validation to confirm this issue.ClawSweeper needs live local, crabbox, or manual validation to confirm this issue.clawsweeper:queueable-fixClawSweeper marked this issue as an existing queue_fix_pr work candidate.ClawSweeper marked this issue as an existing queue_fix_pr work candidate.issue-rating: 🐚 platinum hermitGood issue quality with a plausible reproduction path needing some confirmation.Good issue quality with a plausible reproduction path needing some confirmation.
Type
Fields
Give feedbackNo fields configured for issues without a type.
Summary
wiki_lintcan fail at the tool layer with:unable to resolve opened file pathwhen the bundled memory-wiki tool returns the raw
lintMemoryWikiVault()result object directly. In practice, the CLI lint path can succeed while thewiki_linttool invocation fails.Environment
wiki_lintopenclaw wiki lintReproduction
wiki_lint.At the same time, the CLI path can still succeed:
Actual behavior
The tool wrapper returns the raw lint result object, including absolute
reportPathand the full details payload:This appears to interact badly with the tool/result handling path and can produce the
unable to resolve opened file pathfailure even though lint itself succeeded.Expected behavior
wiki_lintshould return successfully, the same way the CLI does, and should not require callers to consume raw absolute-path-heavy internals.Validated fix
A minimal fix in
extensions/memory-wiki/src/tool.tsis to:reportPathto be relative to the vault root for tool outputdetailsobject instead of the rawresultExample shape:
Validation
I added a focused test in
extensions/memory-wiki/src/tool.test.tsfor:reportPath(reports/lint.md)detailsand ran:
corepack pnpm exec vitest run --project extension-memory \ extensions/memory-wiki/src/markdown.test.ts \ extensions/memory-wiki/src/tool.test.tsResult:
2 passed9 passed0On the live install, after applying the tool-side change,
wiki_lintreturned normally instead of failing.Why open separately
There is already existing discussion around lint false positives and wikilink resolution, but this is a different bug surface:
wiki_lintAcceptance criteria
wiki_linttool invocation does not fail withunable to resolve opened file pathopenclaw wiki lintandwiki_lintboth succeed against the same vaultreportPathand a bounded details payload