wehub-resource-sync
b4fbd6fe9f
Deploy Site / deploy-vercel (push) Has been skipped
Deploy Site / deploy-docs (push) Has been skipped
Build Skills Index / build-index (push) Has been skipped
Build Skills Index / trigger-deploy (push) Waiting to run
CI / Deny unrelated histories (push) Has been skipped
CI / CI timing report (push) Blocked by required conditions
CI / Detect affected areas (push) Successful in 27m35s
CI / OSV scan (push) Failing after 4s
CI / Build&Test Docker image (push) Successful in 9s
CI / Supply-chain scan (push) Has been skipped
CI / Lint Docker scripts (push) Failing after 5m13s
CI / Check contributors (push) Failing after 12m8s
CI / Docs Site (push) Failing after 12m8s
CI / TypeScript (push) Failing after 12m8s
CI / Python lints (push) Failing after 12m9s
CI / Python tests (push) Failing after 12m9s
CI / Check uv.lock (push) Failing after 23m22s
CI / All required checks pass (push) Waiting to run
70 lines
2.6 KiB
Python
70 lines
2.6 KiB
Python
"""Verification-loop helpers for the ``pre_verify`` round-end gate.
|
|
|
|
When the agent has edited code and is about to verify/finish, the loop fires the
|
|
``pre_verify`` hook (user directives resolved by
|
|
:func:`hermes_cli.plugins.get_pre_verify_continue_message`). A directive keeps
|
|
the agent going one more turn — run a check, defer it, tidy the diff — instead of
|
|
stopping immediately.
|
|
|
|
The shipped coding guidance lives on the evidence-based verification-stop nudge
|
|
(``agent/verification_stop.py``), not as a second default stop gate. That keeps
|
|
the default token cost tied to the existing "missing verification evidence"
|
|
decision while preserving ``pre_verify`` for user/plugin policy.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
from typing import Any, Optional
|
|
|
|
from utils import is_truthy_value
|
|
|
|
DEFAULT_MAX_VERIFY_NUDGES = 3
|
|
|
|
# Shipped guidance appended to the verification-stop nudge when code lacks fresh
|
|
# verification evidence. Wording mirrors the user-facing "clean your work"
|
|
# workflow, but does not create its own extra model turn.
|
|
CODING_VERIFY_GUIDANCE = (
|
|
"[Coding] Before you run tests/linters or call this done: if this is "
|
|
"creative UI/visual work, hold off on tests and linters until the user says "
|
|
"they like the result or you're about to commit. And before every commit, "
|
|
"clean your work: keep it KISS/DRY, match the surrounding code style, and be "
|
|
"elitist, shorthand, clever, concise, efficient, and elegant."
|
|
)
|
|
|
|
|
|
def max_verify_nudges(config: Optional[dict[str, Any]] = None) -> int:
|
|
"""Bound on consecutive ``pre_verify`` continue directives per turn (>= 0)."""
|
|
agent_cfg = _agent_cfg(config)
|
|
raw = agent_cfg.get("max_verify_nudges")
|
|
try:
|
|
return max(0, int(raw))
|
|
except (TypeError, ValueError):
|
|
return DEFAULT_MAX_VERIFY_NUDGES
|
|
|
|
|
|
def coding_verify_guidance(config: Optional[dict[str, Any]] = None) -> Optional[str]:
|
|
"""Return the optional guidance appended to verification-stop nudges."""
|
|
if not is_truthy_value(_agent_cfg(config).get("verify_guidance", True), default=True):
|
|
return None
|
|
return CODING_VERIFY_GUIDANCE
|
|
|
|
|
|
def _agent_cfg(config: Optional[dict[str, Any]]) -> dict[str, Any]:
|
|
if config is None:
|
|
try:
|
|
from hermes_cli.config import load_config
|
|
|
|
config = load_config()
|
|
except Exception:
|
|
config = {}
|
|
agent_cfg = (config or {}).get("agent") if isinstance(config, dict) else None
|
|
return agent_cfg if isinstance(agent_cfg, dict) else {}
|
|
|
|
|
|
__all__ = [
|
|
"CODING_VERIFY_GUIDANCE",
|
|
"DEFAULT_MAX_VERIFY_NUDGES",
|
|
"coding_verify_guidance",
|
|
"max_verify_nudges",
|
|
]
|