mirror of
https://github.com/boshu2/agentops.git
synced 2026-09-14 15:08:13 +08:00
Boden Fuller
1374cfb357
Adds scripts/validate-context-map-drift.sh — runs the context-map generator into a temp location, diffs against the committed map, fails on drift with a helpful "To fix:" message and a trap-based restore so a failed drift check leaves the working tree clean (prevents accidental "git add" of a regenerated map). Wires the check into two places per pre-mortem Fix 3: 1. scripts/pre-push-gate.sh — new section 29b, conditionally runs the check when skills/*/SKILL.md, the generator, the drift script itself, or the context-map are in the diff. 2. .github/workflows/validate.yml — new validate-context-map-drift job, registered in summary.needs and the result-echo block. AGENTS.md CI-jobs table updated to keep parity invariant green. Issue #5 of the DDD+Hexagonal v1 plan, applying finding-2026-05-07-ci-parity-as-wave-acceptance and pre-mortem Fix 3. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
61 lines
1.9 KiB
Bash
Executable File
61 lines
1.9 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
# validate-context-map-drift.sh — detect drift between SKILL.md frontmatter and
|
|
# the generated docs/contracts/context-map.md.
|
|
#
|
|
# Per DDD+Hexagonal v1 plan Issue #5 (Fix 3): the context map is a generated
|
|
# artifact whose source of truth is `skills/*/SKILL.md` frontmatter. Drift
|
|
# happens when a SKILL.md is edited (hexagonal_role/consumes/produces/context_rel
|
|
# changes) without regenerating the map. This gate forces the regeneration to
|
|
# be committed alongside the SKILL.md edit.
|
|
#
|
|
# Behaviour:
|
|
# - Exit 0 if regenerating yields no diff against the committed context-map.
|
|
# - Exit 1 if drift is detected, with a helpful "how to fix" message on stderr.
|
|
#
|
|
# Safety: the script never leaves the working tree dirty. The committed
|
|
# context-map.md is backed up to a temp file and restored on exit (trap),
|
|
# whether the script exits cleanly or via error.
|
|
set -euo pipefail
|
|
|
|
CONTEXT_MAP="docs/contracts/context-map.md"
|
|
GENERATOR="scripts/generate-context-map.sh"
|
|
|
|
if [[ ! -f "$CONTEXT_MAP" ]]; then
|
|
echo "validate-context-map-drift: missing $CONTEXT_MAP" >&2
|
|
exit 1
|
|
fi
|
|
|
|
if [[ ! -x "$GENERATOR" && ! -f "$GENERATOR" ]]; then
|
|
echo "validate-context-map-drift: missing $GENERATOR" >&2
|
|
exit 1
|
|
fi
|
|
|
|
TMP_BACKUP="$(mktemp -t context-map-backup.XXXXXX.md)"
|
|
cp -f "$CONTEXT_MAP" "$TMP_BACKUP"
|
|
|
|
cleanup() {
|
|
# Always restore the original committed context-map so partial failures
|
|
# (or a real drift detection) never leave the working tree dirty.
|
|
if [[ -f "$TMP_BACKUP" ]]; then
|
|
cp -f "$TMP_BACKUP" "$CONTEXT_MAP"
|
|
rm -f "$TMP_BACKUP"
|
|
fi
|
|
}
|
|
trap cleanup EXIT
|
|
|
|
bash "$GENERATOR" >/dev/null
|
|
|
|
if git diff --exit-code -- "$CONTEXT_MAP" >/dev/null 2>&1; then
|
|
# No drift. Cleanup runs via trap.
|
|
exit 0
|
|
fi
|
|
|
|
cat >&2 <<'EOF'
|
|
Context map drift detected. To fix:
|
|
bash scripts/generate-context-map.sh
|
|
git add docs/contracts/context-map.md
|
|
Then commit.
|
|
EOF
|
|
|
|
exit 1
|