scriptoriumadmin/OpenSpace
1
0
Fork 0
mirror of https://github.com/HKUDS/OpenSpace.git synced 2026-09-07 08:25:54 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(evolver): use word-boundary matching in _parse_confirmation to prevent false positives

Browse source
This commit is contained in:
warren618 2026-03-26 12:06:11 +08:00
parent 38b51c8605
commit 9333eaed42
1 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
openspace/skill_engine/evolver.py
View file

@ -652,10 +652,15 @@ class SkillEvolver:
return bool(data.get("proceed", False))
except (json.JSONDecodeError, ValueError):
pass
# Fallback: look for keywords
if any(w in response for w in ("\"proceed\": true", "proceed: true", "yes", "confirm")):
# Fallback: look for keywords (use word boundaries to avoid
# false positives from substrings like "know" matching "no")
_wb = re.search # shorthand
if any(w in response for w in ("\"proceed\": true", "proceed: true")) \
or _wb(r"\byes\b", response) or _wb(r"\bconfirm\b", response):
return True
if any(w in response for w in ("\"proceed\": false", "proceed: false", "no", "reject", "skip")):
if any(w in response for w in ("\"proceed\": false", "proceed: false")) \
or _wb(r"\bno\b", response) or _wb(r"\breject\b", response) \
or _wb(r"\bskip\b", response):
return False
# Default: skip — ambiguous response should not trigger costly evolution
logger.debug("LLM confirmation response was ambiguous, defaulting to skip")