Repository navigation
Fail closed on bad hook input and settle the Morvali Creek calls - #11
Merged
Merged
Conversation
Carries Vigil's 11 November guardrail review over to Aranya and answers the calls the Morvali Creek run had to make on its own. - The write-guard blocks an event it cannot read, a file tool with no path, and NotebookEdit outside the layout. The post-write validator reports an unreadable event instead of skipping it. - A finding that names a numbered section, para, rule or schedule on an instrument-level catalog_id fails validation (Morvali run bug 3). - AGENTS.md gains the fallbacks and settles the run's open calls. - matter.json gains other_forums, model and aranya_version; flags accept kind "case"; matters/*/inputs/*.json is schema-checked. Guard, schema and settings edits were applied through the shell under maintainer direction, since the write-guard blocks agent edits there. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The instrument-level check rejected any numbered part cited on an instrument entry. cpcb-ec-2019 is catalogued as an instrument, but its summary quotes clauses 1.3.2, 1.5.1 and the rest, so a correct finding citing "clause 1.5.1" failed validation. The check now passes a part the entry's own summary names as a whole reference, and still rejects "s.999" on water-1974, which has no summary. Known limit, in the code and SECURITY.md: a number the summary uses for something else (the scale factor 1.5) passes as a clause. Applied through the shell under maintainer direction, since the write-guard blocks agent edits to .claude/scripts. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Nov 11, 2026
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.
Carries Vigil's 11 November guardrail review (prathikcannotcode/Vigil#10) over to Aranya, and answers the calls the Morvali Creek run had to make on its own (
examples/morvali-creek.run-1/SCORE.md, "Where a skill was confusing").What was wrong
Each of these was reproduced against
mainat3240241before the fix.not json,{}and a Write with no path. NotebookEdit wasn't in the matcher at all, so it could write tocatalog/provisions.json.validate-on-writeexited 0 on an event it couldn't read.catalog_id: water-1974validated clean. This is bug 3 in the Morvali score.AGENTS.mdhad no fallbacks. The Morvali run logged 15 calls it had to make without guidance.What changes
instrumentsentry fails if itsprovisionnames a numbered section, para, rule, schedule or appendix. "The Act as a whole", "Hazardous Waste Rules 2016" and "the Act's 2010 amendment" still pass. All 18 Morvali findings still validate.AGENTS.mdgains Vigil's five fallbacks and two new sections that settle the run's open calls:matters/{id}/inputs/AGENTS.mdwins over conflicting instructionsmatter.jsongainsother_forums,modelandaranya_version.target.kind: "case".matters/*/inputs/*.jsonis checked againstinjury-params.schema.json.SECURITY.md, the changelog (it also covers Close four write-guard bypasses #8), three skill lines, and the README test count.Read by eye
Aranya's write-guard blocked agent edits to
.claude/scripts,.claude/schemasand.claude/settings.json, as it should. Those edits were applied through the shell under maintainer direction, so no hook checked them.Known limits
SECURITY.mdstates both.Checks
npm run validate -- --strict: 43 data files, 5 skills, 0 errorsnpm test: 30 tests, 28 pass, 2 skipped on Windows (symlink and bash hook), 0 failmatter.jsonandflags.jsonlvalidate under the new rules🤖 Generated with Claude Code