Skip to content

fix(memory-wiki): avoid implicit error coercion#99307

Merged
RomneyDa merged 1 commit into
mainfrom
fix/memory-wiki-no-implicit-coercion
Jul 3, 2026
Merged

fix(memory-wiki): avoid implicit error coercion#99307
RomneyDa merged 1 commit into
mainfrom
fix/memory-wiki-no-implicit-coercion

Conversation

@RomneyDa

@RomneyDa RomneyDa commented Jul 3, 2026

Copy link
Copy Markdown
Member

fixes broken ci lint on main

What Problem This Solves

The extension lint gate fails on extensions/memory-wiki/src/ingest.ts because the existing-source error predicate uses implicit boolean coercion. Since extension lint runs across the full bundled plugin surface, the failure blocks otherwise unrelated pull requests.

Why This Change Was Made

Replaces !!error with an explicit non-null object guard. This satisfies no-implicit-coercion while preserving the predicate's existing ENOENT and EISDIR behavior.

User Impact

No user-visible behavior change. Pull requests can pass the bundled extension lint gates again.

Evidence

  • Exact oxlint check passed for extensions/memory-wiki/src/ingest.ts.
  • Formatting check passed for the changed file.
  • node scripts/run-vitest.mjs extensions/memory-wiki/src/wiki-notes-read-retry.test.ts --reporter=verbose — 1 file, 9 tests passed.
  • Fresh Codex autoreview completed with no accepted/actionable findings and 0.99 confidence.
  • Scope: 1 file changed, 1 insertion, 1 deletion.

@RomneyDa
RomneyDa merged commit b47fe7a into main Jul 3, 2026
108 of 112 checks passed
@RomneyDa
RomneyDa deleted the fix/memory-wiki-no-implicit-coercion branch July 3, 2026 01:56
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jul 3, 2026
sheyanmin pushed a commit to sheyanmin/openclaw that referenced this pull request Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant