Skip to content

Chore (scanner) defensive hardening#16

Merged
St0rmz1 merged 3 commits intomainfrom
chore/scanner-defensive-hardening
Apr 23, 2026
Merged

Chore (scanner) defensive hardening#16
St0rmz1 merged 3 commits intomainfrom
chore/scanner-defensive-hardening

Conversation

@St0rmz1
Copy link
Copy Markdown
Collaborator

@St0rmz1 St0rmz1 commented Apr 23, 2026

  • Openclaw's plugin checking system has a false positive on how it declared something "bad"
  • Defensive mechanisms to help avoid it and also detect it for future releases.
  • This is basic regex check right now, pending something better

St0rmz1 added 3 commits April 23, 2026 09:53
The log-guard comment introduced in 0.2.2 mentioned 'web-fetch' as an example OpenClaw runtime. On gateways running OpenClaw v2026.4.9 (and other pre-2026-04-16 builds), the install-time scanner's env-harvesting rule used the broader regex /\\bfetch\\b|\\bpost\\b|http\\.request/i, which matched that bare word and — combined with process.env reads elsewhere in index.ts — blocked the install with 'possible credential harvesting'. Upstream tightened the regex to require an open paren in commit 678b019467 (2026-04-16), but older KiloClaw instances are still on the broader rule. Rephrased to 'web-retrieval' so the comment is inert under either variant.
@St0rmz1 St0rmz1 changed the title Chore/scanner defensive hardening Chore (scanner) defensive hardening Apr 23, 2026
@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot Bot commented Apr 23, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (6 files)
  • .github/workflows/scan.yml
  • CHANGELOG.md
  • index.ts
  • package.json
  • script/scan.ts
  • src/env.ts

Reviewed by gpt-5.4-2026-03-05 · 213,414 tokens

@St0rmz1 St0rmz1 merged commit da71f03 into main Apr 23, 2026
8 checks passed
@St0rmz1 St0rmz1 deleted the chore/scanner-defensive-hardening branch April 23, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants