refactor(v3): retire override audit trail + capture loop, keep escalation#6
Merged
Merged
Conversation
…tion Step #1 validation found 0 overrides across all 12 projects in ~7 weeks (production included; only 4/12 adopted behaviors). The auditable override trail — the v3-vs-hookify differential — has no empirical use, and the v4 capture loop processed an always-empty log. Retired as dead weight. The graduated escalation engine stays: it is exercised and the happy path is "verify", not "override". Removed: - scripts/process-override-log.sh, tests/test-process-override-log.sh - template/hooks + .claude/hooks session-start-process-overrides.sh - .forge/audit/overrides.log (git-tracked) - lib.sh: forge_override_append, FORGE_AUDIT_LOG, audit-log append in try_override - SessionStart wiring in settings.json + template/settings.json.tmpl - audit Dimension B item B3 (override loop); B now 4 items (forge_adoption 0-4) Kept: - forge_pending_block_try_override (override-via-reinvocation, state.json only) - full soft_block escalation Updated both scoring engines (score.sh + audit_all.py), checklist/scoring/SKILL, registry, CLAUDE.md, README, usage-guide, guia-uso. 19 tests green. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
🟢 dotforge Native Health: 9/10 (Excelente)dotforge Adoption: 3/5 (Most) — informational, does not affect Native Health
Dimension A — Native Health
Dimension B — dotforge Adoption (informational)
Score computed by |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Primera cesión concreta del principio native-first (paso #1 del plan → ejecución). -562 líneas netas.
Por qué
Scan de los 12 proyectos: 0 overrides en ~7 semanas (production incluido; solo 4/12 adoptaron behaviors). El override auditable —el diferencial v3-vs-hookify— no tiene uso empírico, y el capture loop de v4 procesaba un log siempre vacío. Es peso muerto.
Qué se RETIRA
scripts/process-override-log.sh+tests/test-process-override-log.shsession-start-process-overrides.sh(template + .claude).forge/audit/overrides.log(git-tracked)lib.sh:forge_override_append,FORGE_AUDIT_LOG, append al log entry_overridesettings.json+template/settings.json.tmplforge_adoption0-4Qué se MANTIENE (el delta real vs hookify)
forge_pending_block_try_override— override-via-reinvocación (registro enstate.jsonefímero)Coordinación
Ambos motores de scoring actualizados en lockstep (
score.sh+audit_all.py), más checklist/scoring/SKILL/registry/CLAUDE.md/README/usage-guide/guia-uso. 19 tests verdes. Histórico (changelog, docs/v3, docs/v4/SPEC) intacto.🤖 Generated with Claude Code