Compare commits
1 Commits
fix/953
...
claude/iss
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
11bdef4e3d |
@@ -1,4 +1,5 @@
|
||||
from agent.telemetry_logger import log_token_usage\n"""Shared auxiliary client router for side tasks.
|
||||
from agent.telemetry_logger import log_token_usage
|
||||
"""Shared auxiliary client router for side tasks.
|
||||
|
||||
Provides a single resolution chain so every consumer (context compression,
|
||||
session search, web extraction, vision analysis, browser vision) picks up
|
||||
@@ -396,7 +397,8 @@ class _CodexCompletionsAdapter:
|
||||
prompt_tokens=getattr(resp_usage, "input_tokens", 0),
|
||||
completion_tokens=getattr(resp_usage, "output_tokens", 0),
|
||||
total_tokens=getattr(resp_usage, "total_tokens", 0),
|
||||
)\n log_token_usage(usage.prompt_tokens, usage.completion_tokens, model)
|
||||
)
|
||||
log_token_usage(usage.prompt_tokens, usage.completion_tokens, model)
|
||||
except Exception as exc:
|
||||
logger.debug("Codex auxiliary Responses API call failed: %s", exc)
|
||||
raise
|
||||
@@ -529,7 +531,8 @@ class _AnthropicCompletionsAdapter:
|
||||
prompt_tokens=prompt_tokens,
|
||||
completion_tokens=completion_tokens,
|
||||
total_tokens=total_tokens,
|
||||
)\n log_token_usage(usage.prompt_tokens, usage.completion_tokens, model)
|
||||
)
|
||||
log_token_usage(usage.prompt_tokens, usage.completion_tokens, model)
|
||||
|
||||
choice = SimpleNamespace(
|
||||
index=0,
|
||||
|
||||
0
skills/creative/excalidraw/scripts/upload.py
Executable file → Normal file
0
skills/creative/excalidraw/scripts/upload.py
Executable file → Normal file
0
skills/leisure/find-nearby/scripts/find_nearby.py
Executable file → Normal file
0
skills/leisure/find-nearby/scripts/find_nearby.py
Executable file → Normal file
0
skills/media/youtube-content/scripts/fetch_transcript.py
Executable file → Normal file
0
skills/media/youtube-content/scripts/fetch_transcript.py
Executable file → Normal file
0
skills/productivity/google-workspace/scripts/google_api.py
Executable file → Normal file
0
skills/productivity/google-workspace/scripts/google_api.py
Executable file → Normal file
0
skills/productivity/google-workspace/scripts/setup.py
Executable file → Normal file
0
skills/productivity/google-workspace/scripts/setup.py
Executable file → Normal file
0
skills/productivity/ocr-and-documents/scripts/extract_marker.py
Executable file → Normal file
0
skills/productivity/ocr-and-documents/scripts/extract_marker.py
Executable file → Normal file
0
skills/productivity/ocr-and-documents/scripts/extract_pymupdf.py
Executable file → Normal file
0
skills/productivity/ocr-and-documents/scripts/extract_pymupdf.py
Executable file → Normal file
0
skills/red-teaming/godmode/scripts/auto_jailbreak.py
Executable file → Normal file
0
skills/red-teaming/godmode/scripts/auto_jailbreak.py
Executable file → Normal file
0
skills/red-teaming/godmode/scripts/godmode_race.py
Executable file → Normal file
0
skills/red-teaming/godmode/scripts/godmode_race.py
Executable file → Normal file
0
skills/red-teaming/godmode/scripts/parseltongue.py
Executable file → Normal file
0
skills/red-teaming/godmode/scripts/parseltongue.py
Executable file → Normal file
0
skills/research/arxiv/scripts/search_arxiv.py
Executable file → Normal file
0
skills/research/arxiv/scripts/search_arxiv.py
Executable file → Normal file
0
skills/research/polymarket/scripts/polymarket.py
Executable file → Normal file
0
skills/research/polymarket/scripts/polymarket.py
Executable file → Normal file
@@ -1,63 +0,0 @@
|
||||
"""Regression tests for bundled skill scripts and local shell execution.
|
||||
|
||||
Issue #953 verifies that bundled skill scripts run out of the box from the
|
||||
installed ~/.hermes/skills tree without manual chmod or PATH surgery.
|
||||
"""
|
||||
|
||||
import shlex
|
||||
import shutil
|
||||
import stat
|
||||
from pathlib import Path
|
||||
|
||||
from tools.environments.local import LocalEnvironment
|
||||
|
||||
|
||||
REPO_ROOT = Path(__file__).resolve().parents[2]
|
||||
SKILLS_ROOT = REPO_ROOT / "skills"
|
||||
|
||||
|
||||
def _bundled_shebang_scripts() -> list[Path]:
|
||||
scripts: list[Path] = []
|
||||
for path in SKILLS_ROOT.rglob("*"):
|
||||
if not path.is_file() or path.is_symlink() or "scripts" not in path.parts:
|
||||
continue
|
||||
first_line = path.read_bytes().splitlines()[:1]
|
||||
if first_line and first_line[0].startswith(b"#!"):
|
||||
scripts.append(path)
|
||||
return sorted(scripts)
|
||||
|
||||
|
||||
def test_bundled_skill_shebang_scripts_are_executable():
|
||||
missing = []
|
||||
for path in _bundled_shebang_scripts():
|
||||
mode = stat.S_IMODE(path.stat().st_mode)
|
||||
if mode & 0o111 == 0:
|
||||
missing.append(f"{path.relative_to(REPO_ROOT)} ({oct(mode)})")
|
||||
|
||||
assert not missing, (
|
||||
"Bundled shebang scripts must ship executable so synced skill copies run "
|
||||
"without manual chmod:\n" + "\n".join(missing)
|
||||
)
|
||||
|
||||
|
||||
def test_local_environment_executes_installed_skill_script_without_manual_prep(tmp_path):
|
||||
hermes_home = tmp_path / ".hermes"
|
||||
installed_skill = hermes_home / "skills" / "research" / "arxiv"
|
||||
installed_skill.parent.mkdir(parents=True, exist_ok=True)
|
||||
shutil.copytree(SKILLS_ROOT / "research" / "arxiv", installed_skill)
|
||||
|
||||
script_path = installed_skill / "scripts" / "search_arxiv.py"
|
||||
env = LocalEnvironment(
|
||||
cwd=str(tmp_path),
|
||||
timeout=15,
|
||||
env={
|
||||
"HERMES_HOME": str(hermes_home),
|
||||
"PATH": "/custom/bin",
|
||||
},
|
||||
)
|
||||
|
||||
result = env.execute(f"{shlex.quote(str(script_path))} --help")
|
||||
|
||||
assert result["returncode"] == 0, result["output"]
|
||||
assert "Search arXiv and display results in a clean format." in result["output"]
|
||||
assert "python search_arxiv.py" in result["output"]
|
||||
Reference in New Issue
Block a user