bvandeusenandClaude Opus 5.5
c5eeec471f
CI & Build / Python lint (push) Successful in 3s
CI & Build / Plugin hooks (push) Successful in 14s
CI & Build / TypeScript typecheck (push) Successful in 53s
CI & Build / integration (push) Successful in 1m2s
CI & Build / Python tests (push) Failing after 1m26s
CI & Build / Build & push image (push) Skipped
Step 3 of the Scribe -> Grimoire rename. - Plugin `grimoire` in marketplace `grimoire-plugin`, MCP server key `grimoire`, so tools reach a session as mcp__plugin_grimoire_grimoire__*. - hooks/scribe_* -> hooks/grimoire_*, their shell functions and the TMPDIR ledgers with them; SCRIBE_URL / SCRIBE_TOKEN / SCRIBE_HOOK_DIR / SCRIBE_LEDGER_DIRS -> GRIMOIRE_*; the project marker is `.grimoire`. - skills/using-scribe -> using-grimoire, /grimoire:sync, stored processes arrive as grimoire-proc-<slug>; moment_actions names them the same way. - Migration 0124 rewrites stored skill names (rule_moments, rule_moment_judgments, moment_mappings) from scribe:/scribe-proc- to grimoire:/grimoire-proc-, so a rule mounted on a skill keeps firing. - Settings' install commands and MCP-only defaults, the docs and the tests follow. - scripts/check_plugin.py drops the smoke event for a report-check hook that has no file in plugin/hooks. Kept on purpose: the secret-sealing label, and the names that let backups written as Scribe restore and be counted. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
60 lines
2.6 KiB
Python
60 lines
2.6 KiB
Python
"""Under `pipefail`, an early-exiting `head` must not void the output it kept.
|
||
|
||
The hooks run with `set -uo pipefail`. `hits=$(git grep -l … | head -4) ||
|
||
hits=""` loses its four hits whenever git grep is still writing when head
|
||
exits: SIGPIPE, a failed substitution, and the fallback wipes the result. It
|
||
only shows on big outputs, so the by-name duplicate arm goes silent for
|
||
exactly the most-duplicated names.
|
||
|
||
Two halves: the real function on a repo big enough to overflow the pipe, and
|
||
the check_plugin pattern that keeps the shape out of every hook (shown able to
|
||
fail).
|
||
"""
|
||
from __future__ import annotations
|
||
|
||
import os
|
||
import shutil
|
||
import subprocess
|
||
from pathlib import Path
|
||
|
||
import pytest
|
||
|
||
ROOT = Path(__file__).resolve().parents[1]
|
||
DEFS = ROOT / "plugin" / "hooks" / "grimoire_defs.sh"
|
||
|
||
|
||
def test_the_duplicate_arm_still_names_a_name_defined_in_thousands_of_files(tmp_path):
|
||
for tool in ("git", "bash"):
|
||
if shutil.which(tool) is None:
|
||
pytest.skip(f"{tool!r} not installed")
|
||
env = {"PATH": os.environ["PATH"], "HOME": str(tmp_path),
|
||
"GIT_AUTHOR_NAME": "t", "GIT_AUTHOR_EMAIL": "t@x",
|
||
"GIT_COMMITTER_NAME": "t", "GIT_COMMITTER_EMAIL": "t@x"}
|
||
repo = tmp_path / "repo"
|
||
repo.mkdir()
|
||
subprocess.run(["git", "init", "-q"], cwd=repo, check=True, env=env)
|
||
# ~3000 × ~60 bytes of `git grep -l` output: well past a 64 KiB pipe buffer,
|
||
# so git grep is still writing when head has its four lines.
|
||
for i in range(3000):
|
||
(repo / f"module_with_a_fairly_long_descriptive_name_{i}.py").write_text("def slug(t):\n return t\n")
|
||
subprocess.run(["git", "add", "."], cwd=repo, check=True, env=env)
|
||
subprocess.run(["git", "commit", "-q", "-m", "base"], cwd=repo, check=True, env=env)
|
||
|
||
script = f'set -uo pipefail\n. "{DEFS}"\nprintf "sym\\tslug\\n" | grimoire_local_dups "{repo}" new.py\n'
|
||
out = subprocess.run(["bash", "-c", script], capture_output=True, text=True, env=env, timeout=60)
|
||
assert out.returncode == 0, out.stderr
|
||
assert "`slug` is already defined in 4 other file(s)" in out.stdout
|
||
|
||
|
||
def test_the_lint_pattern_catches_the_shape_and_passes_the_fix():
|
||
from scripts.check_plugin import PATTERNS
|
||
|
||
pattern = next(p for p, label, _why in PATTERNS if label.startswith("early-exit consumer"))
|
||
for bad in ('hits=$(git grep -l x | head -4) || hits=""',
|
||
" | grep -m1 -v '^[[:space:]]*$') || old_first=\"\"",
|
||
'x=$(a | grep -q b) || x=""'):
|
||
assert pattern.search(bad), bad
|
||
for good in ('hits=$(git grep -l x | head -4 || true)',
|
||
'ln=$(grep -nF -m1 -- "$a" "$f" | cut -d: -f1) || ln=""'):
|
||
assert not pattern.search(good), good
|