$ the-wire · showcase
OpenClaw prunes stale agent APIs, Clownfish tightens security review routing
By RepoJournal · Filed · About OpenClaw
OpenClaw dropped five stale agent APIs while Clownfish hardened its security review pipeline against false positives and Clownfish docs replaced a vulnerable parser.
OpenClaw removed a set of stale agent-facing APIs in one cleanup pass, deleting the live model switch API [1], exec helpers [2], the subagent depth policy [3], the ACP error alias [4], and runner aliases [5]. Anything still calling these names after the next upgrade will break, so check agent integrations against current docs.
Clownfish review and routing logic changed in three places that affect how security findings are handled. Security guard check names are now ignored [6], false security routing is rejected [7], and negated security item evidence is ignored [8]. The earlier behavior could route or report on signals that masqueraded as security evidence; these commits stop that, so review output should more accurately reflect real risks.
Docs took a security fix by removing the vulnerable parser [9][10]. The PR replaces gray-matter with yaml 2.9.0 for frontmatter parsing, removing js-yaml 3.x from the dependency graph, and refreshes the Mermaid DOMPurify lockfile entry to 3.4.10. It resolves open Dependabot alerts for js-yaml and dompurify, so update docs dependencies before the next deploy if you vendor them.
Elsewhere, Clownfish preserved executed apply actions so a later post-flight idempotency replay cannot overwrite an earlier executed closure in run summaries [11], and the executor captures a `check:changed` baseline before Codex edits to keep unchanged diagnostics from blocking repair [12].
Action items
- → Replace the docs parser and bump yaml to 2.9.0 and DOMPurify lockfile to 3.4.10 before next deploy openclaw/docs [immediate]
- → Audit agent code calling any of the removed stale agent APIs and migrate before upgrade openclaw/openclaw [immediate]
- → Re-run security review after Clownfish changes to confirm routing decisions reflect the new false-positive rules openclaw/clownfish [plan]
References
- [1] refactor(agents): remove stale live model switch api ↗ openclaw/openclaw
- [2] refactor(agents): remove stale exec helpers ↗ openclaw/openclaw
- [3] refactor(agents): remove stale subagent depth policy ↗ openclaw/openclaw
- [4] refactor(agents): remove stale acp error alias ↗ openclaw/openclaw
- [5] refactor(agents): remove stale runner aliases ↗ openclaw/openclaw
- [6] fix(review): ignore security guard check names (#97) ↗ openclaw/clownfish
- [7] fix: reject false security routing (#86) ↗ openclaw/clownfish
- [8] fix(review): ignore negated security item evidence (#91) ↗ openclaw/clownfish
- [9] fix(security): replace vulnerable docs parser (#53) ↗ openclaw/docs
- [10] fix(security): remove vulnerable docs parser ↗ openclaw/docs
- [11] fix(results): preserve executed apply actions ↗ openclaw/clownfish
- [12] fix(executor): preserve unchanged changed-gate diagnostics ↗ openclaw/clownfish