novox/hq
18
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge pull request 'Developer definition: the manifest validator gets one predefined key list for both cases, always (issue 1585)' (#518) from agents/validator-key-list into main

Browse Source
This commit was merged in pull request #518.
This commit is contained in:
mesh-tools
2026-11-05 01:28:03 +00:00
parent 331e570792 304d17b561
commit d1e2059dab
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.claude/agents/mesh-developer.md
+3 -2
View File
@@ -36,8 +36,9 @@ When the change touches any `module.json`, in any repository, run the manifest v
program `cmd/mesh-judge` in the indexer's module, built at mesh-delivery's trunk, or at the change's mesh-delivery program `cmd/mesh-judge` in the indexer's module, built at mesh-delivery's trunk, or at the change's mesh-delivery
head when that head changes the validator itself (then the indexer's own manifest is also validated at the trunk). head when that head changes the validator itself (then the indexer's own manifest is also validated at the trunk).
Give it raw manifests, not digests: the trunk's, with the change's laid over them, as the two cases `change` and Give it raw manifests, not digests: the trunk's, with the change's laid over them, as the two cases `change` and
`trunk` the checker's `judgeRequest` names, with the predefined key list when the change alters it. The merge check `trunk` the checker's `judgeRequest` names, with one predefined key list for both cases, always: the change's when it alters that
fails a changed manifest the validator refuses or holds, and also an unchanged one in which the change causes a list, else the trunk's. The merge check fails a changed manifest the validator refuses or holds,
and also an unchanged one in which the change causes a
problem the trunk case lacks; it says so only after a full build, and no module's unit tests run the validator. problem the trunk case lacks; it says so only after a full build, and no module's unit tests run the validator.
Report every manifest whose verdict differs between the two cases, with your heads. The reviewer asks for those Report every manifest whose verdict differs between the two cases, with your heads. The reviewer asks for those
verdicts, and a head reported without them is not reviewed. verdicts, and a head reported without them is not reviewed.