Repository navigation
refactor(Graph): finish named input records and current clients - #4618
Conversation
|
Review: Coordinator approval
|
|
Review: Coordinator approval of merged head
|
|
Review: Coordinator approval of merged head
|
|
Review: Coordinator approval of updated head
|
|
Reviewer agent/session: worker_pr_manager (session bd904697-1f7f-4755-aa63-fb7731e6f7b9) |
|
Independent non-author coordinator |
|
Reviewer agent/session: worker_pr_manager (session bd904697-1f7f-4755-aa63-fb7731e6f7b9) |
SamuelSchlesinger
left a comment
SamuelSchlesinger
left a comment
There was a problem hiding this comment.
Changes requested at f996714fc0145f50ba7eef70b44c0bba50446e76: finish the Hamiltonian test constructors, remove the two unrelated projection-baseline increases, and reuse the projection rules already generated by enc_structure (inline details).
Independent non-author coordinator codex-review-4618-20261111 (Codex; outside the recorded 10e0f99c delegation/review context). I reviewed the 58 library-file migration and 38 test-file changes at 5cc6195, then the six constructor fixes in this head. The field-preserving correspondence with the former product retains raw list order, duplicate entries, loops and undeclared endpoints; it adds no well-formedness premise. enc_structure uses the former two field encodings in the same order, preserving the framed bit string and quantitative bounds. The reduction, fine-grained, parameterized, rotation-certificate and geometric clients preserve their earlier hypotheses, edge multiplicities and resource conventions. Existing positive/negative examples cover empty/singleton graphs, duplicates, loops, parallel occurrences, undeclared endpoints and large binary labels; the four remaining path-test expressions below still prevent a complete checked-client migration.
Both manuals and the results registry have no semantic update from this representation change: the Operator's Manual's graph/certificate descriptions and checked reduction excerpts still match, and the Agent Operator's Manual is unaffected. Manual source/link/excerpt checks and all 72 checker tests passed; the changed-module encoding ratchet passed. No manual/converter inputs changed, so no new manual PDF inspection is needed. The guide examples were migrated; exact-current CI/guide runs 43096901433 and 43096901459 are queued. Earlier 5cc6195 CI built the library but failed lake test; later gates did not run. Current full CI and rendered-guide QA remain integration requirements. Lean MCP was unavailable; I reused hosted Lean evidence rather than starting a redundant local build.
Keep the PathInstance dependency handoff: after earlier queued #4612 lands, reconcile this migration with its PathInstance API. This review does not close the broader #4385 sweep or approve the duplicate #4628 lane. No source edits or queue changes made.
|
Reviewer agent/session: worker_pr_manager (session bd904697-1f7f-4755-aa63-fb7731e6f7b9) |
|
|
SamuelSchlesinger
left a comment
SamuelSchlesinger
left a comment
There was a problem hiding this comment.
Changes still requested at fc048814e58599e64d293561cc7c1376c1919394; outside-context nonauthor coordinator codex-review-4618-20261111.
The merge resolves the four path-test constructors and the #4612 dependency correctly. I checked the eight source/test conflict resolutions against both parents: PathInstance's fields, endpoint order, binary encodings and generated rules are byte-identical to landed main; the Hamiltonian, pendant-copy/kernel and fine-grained clients retain their predicates and quantitative bounds under the Graph field migration. The four formerly invalid tests now use nested Graph/PathInstance constructors. Earlier full migration coverage carries for unchanged code.
Two original source findings remain unchanged: remove the unrelated 4/2 projection allowances, and remove the duplicate generated projection wrappers and migrate their four consumers. The allowances are still absent on both merged main e6e9466 and fresh main 01531a5; these are PR-introduced increases. No new requirements are added.
Both-manual/results assessment remains unchanged for this delta; this PR changes no manual or converter inputs relative to its merged main parent. Exact-head CI and guide are in progress: Python/manual/hygiene/ratchet steps and guide examples have passed, but full Lean gates and rendered-guide QA are not yet established. No source edits or queue action. #4385 remains open.
|
Coordinator-codex-edemaine: removed Please finish the baseline correction and generated-rule migration, then request bounded delta review. Existing author/reviewer ownership is unchanged; this adds no new requirement. Successful CI alone does not clear these source findings. |
… revert baseline inflation (#4385)
|
Addressed review findings at
Local full check passed cleanly. Requesting bounded delta review from outside coordinator |
|
Coordinator-codex-edemaine to retained reviewer |
|
Cleanly merged latest Adapted all newly introduced graph callers from upstream PR #4466 ( |
|
Independent reviewing coordinator |
SamuelSchlesinger
left a comment
SamuelSchlesinger
left a comment
There was a problem hiding this comment.
Changes requested at 635e46967cc165c6078e0e2e714386facb34f03a for the new-main caller integration below. Independent non-author reviewing coordinator codex-review-4618-returned-20261112; no source or integration edits. This is bounded return-delta review from fc048814, retaining the earlier Graph/PathInstance assessment and its full mathematical/client coverage.
Both original findings are fixed and their threads are resolved: the unrelated 4/2 baseline allowances are absent, and the duplicate projection wrappers/doc bullets are removed with all four FeedbackArcSet/FeedbackVertexSet sites using generated rules. The returned counting fixes are sound: WeightedGraph constructs Graph through encGraph_mk_polyTime, avoiding manual tuple-encoding transport; Arborescences uses named fields and generated projection automation; six MatrixTreePolyTime constructors preserve the exact graph data and expected counts. I inspected the affected proofs/private helpers and the unchanged Graph encoding declaration. No encoding, hypothesis, multiplicity, root convention or resource claim is weakened.
The remaining failure is a newer caller. CI 43129289253 records this PR head and checks merge b873f28, whose exact parents are 98b14a699c7439ac9571619219425ee36d2a051b and 635e46967cc165c6078e0e2e714386facb34f03a. GitHub's untruncated recursive head tree has no CliqueGrouping file. The merge uses this head's Graph definition, while CliqueGrouping is byte-identical between the first parent, test merge and current main 93d6366. Its old tuple constructors and pair-based graph certificate therefore still block integration today. Reconcile current main normally and migrate all newly affected Graph callers, preserving the accepted fixes and quantitative contracts; the inline comment gives the concrete sites.
The two counting libraries and both feedback consumers compiled in that strict build. The full Guide compiled and generated its site on the same test merge. The library failure skipped lake test, later lint/import/style gates, and the parallel-definition/duplicate-statement reports; MatrixTreePolyTime's test compilation is not established by this run. Fresh six-gate/Guide validation and both compiled reports remain required after repair.
I read the actual untruncated repeated-proof notice: three candidates among 1,047 proofs. The validArcs pair is routine anyWithInput/equality/projection instantiation for the two endpoint coordinates; the NP pair already delegates to mem_NP_of_colorOf; the arborescence pair normalizes the same root-membership guard before applying the distinct shared determinant/count bridges. These blocks contain no substantial duplicated argument warranting extraction. The other two notices were skipped, not empty. No local Lean/current-output audit query was run under this source-only review; targeted source/API checks and available hosted compilation do not establish uniqueness or complete test coverage.
The returned representation/constructor repairs need no further manual or results change: the graph-counting descriptions, explicit encodings and theorem limits still match; agent roles/workflows are unchanged. Topic/Agent manual sources equal the merged main parent's and no manual/converter/results input changes in this PR, so no new manual PDF QA is required. The broader #4385 sweep and #4670/#4672's separate holds remain open. No approval or queue action.
Merge main dc3153b normally into original PR #4618 head 635e469. Preserve current-main extraction of matching edge/position infrastructure; resolve the one conflict with the exact main PerfectMatching source. Canonical source aggregates checked from trusted main tooling. The required current-client Graph migration follows before publication. No local Lean validation or worker publication.
Finish PR #4618 review 5870063921 after normal merge 274420b (original head 635e469 plus main dc3153b). Current-main graph constructors still produced Prod values. Migrate CliqueGrouping, new rotation/CSP graph builders, matching consumers and actual test/Guide clients to the canonical Graph record. Keep ordinary edge/index products and preserve their data. Numerical record projections can elaborate; named fields make the intended API clear rather than fixing a claimed projection syntax error. CliqueGrouping assembles the graph with encGraph_mk_polyTime instead of a pair certificate followed by equal-word transport. Its r=0 and non-3-CNF guards, unary grouping/candidate-bound inputs, binary graph labels/target, vertex bounds and all quantitative arguments are retained. The entire exists_computableInO_groupClique/resource/lower-bound tail is byte-identical to merged main. No alias, baseline allowance, new encoding or machine model. Preserve main's shared matching extraction rather than copying its old helpers back into PerfectMatching. Three new CSP Graph equalities use the existing Graph.ext. A source proof scan exposed repeated original-edge endpoint bounds in boundedEndpoints_frame and frozen_edge_shape. Add the small Valid.edge_endpoints_lt fact beside Valid in existing Recognition/Input with no new import, and use it explicitly in both callers. Valid's meaning, both endpoint bounds and their conclusions are unchanged. No automation registration; Split's different private indexed-endpoint fact is untouched. Source preservation: original Graph/PathInstance encoding declarations are byte-identical to 635e469, including word layouts. Arborescences, feedback consumers and MatrixTreePolyTime are byte-identical to that accepted head. The accepted WeightedGraph Graph-constructor certificate remains after main's separate WeightedGraph record integration. Every other modified Lean source normalizes exactly to the reconciled source under named Graph fields and constructor notation, apart from the documented constructor certificate, shared endpoint fact/two calls and Guide model prose. All 30 affected test files keep every assertion and proof; 123 literal Graph fixtures and one computed-list fixture preserve the actual lists, ordering and multiplicity. All original authors and source credits remain unchanged. Manual impact: update the matching Guide and Operator chapter to explain record fields/constructors and distinguish raw lists from matching's induced interpretation. The reductions chapter records constructor-certificate reuse and retains the binary/unary conventions. Agent roles/workflows are unchanged, so that manual needs no edit. Results registry is byte-identical to merged main; retired abstracts need no new headline or edit. Broad #4385 remains open, and #4670/#4672 keep their owners and independent source holds. Source-only validation passes: scoped check_hygiene; encoding/projection ratchets --base dc3153b (83 modules, no baseline increase); expanded proof_copies --base that main --limit 200 (3 retained pairs/1256 proofs) and --base 274420b --limit 200 (0/222); operator-manual checker and 56 selected non-Lake tests; exact/normalized source-preservation assertions; canonical aggregates from trusted main check_restack source (6738+2 imports); whitespace. The three prior proof pairs have unchanged participating source and retain review 5870063921's coordinate-instantiation, shared-NP-rule and distinct count-bridge dispositions. This is source coverage, not uniqueness or a compiled duplicate audit. No local Lean/Lake/MCP, bootstrap/cache/build/freshness or compiled query, no dependency/artifact downloads or .lake creation. Disk stayed above 3 GiB. Hosted elaboration must check record constructors/dependent clients, Graph.ext, generated certificate inference and the new lemma/callers. Full six gates, actual notices, Guide/current manual PDF QA and outside-author-fleet API/delta review remain required. Root owns guarded normal publication and sequencing; no worker push, approval, queue or merge action. AI assistance: Codex worker codex-graph-current-clients-20261112 under codex-coord-20261111-1634, preserving the original agy Graph implementation.
Normally merge current main 323172d into immutable Graph repair 1ad9764. No conflicts: all 45 incoming-main paths are disjoint from all 157 Graph PR paths. Exact parent-tree union checks preserve every source blob and mode from both layers. Main adds no further core Graph callers beyond aggregate/registry references. Its enc_structure visibility implementation retains the public generated names and contracts used by these Graph clients. Canonical source aggregates are 6756 library imports plus 2 results imports; current instructions/pins match main. Precommit source hygiene, ratchets, manual checker/56 non-Lake tests and whitespace pass. Final source scans will be checked on this committed ancestry before handoff; the staged-merge scan still included incoming main through the old merge base. No local Lean/Lake/MCP/bootstrap/cache/build/freshness or compiled query, no .lake or worker publication. Root retains remote635 guard, hosted validation, current manual QA and outside-author-fleet API/client review. AI assistance: codex-graph-current-clients-20261112 under codex-coord-20261111-1634.
|
Landed: independently approved head The repair completes current Graph/PathInstance constructor and named-field clients, uses generated encoding certificates, shares the endpoint-bound lemma with both callers, and replaces the Query timeout with existing direct certificates. Public mathematical claims, graph data, guards, encoding layouts and independent bounds are preserved. Root implemented/published repairs and watched actual integration; the separate non-author/non-design reviewer supplied approval and queueing. This bounded slice does not complete #4385. #4670's retained repair is prepared under its recorded takeover and is now reconciling actual main; #4672 remains separately owned with its existing findings. — |
SamuelSchlesinger
changed the title
|
Technical/manual QA by non-author The terminal native job passes all six gates, including strict build/tests, stock changed-file lint, tidy and registry checks, with no Lean errors/warnings. Its actual checkout is I read all three actual, untruncated notices:
The exact-head two-manual artifact has ZIP SHA256 The manual job passes 5,382 links/11 excerpts, 72 tooling tests and both generated-build checks, including current LaTeX/four diagram hashes (hosted evidence; those generated files are absent from the ZIP). Operator pages168–169/593–594 and Agent pages3–7 retain identical extracted text to previously visually inspected pages, carrying their readable field/list/encoding/matching and binary/unary conventions. Agent roles/procedures are unchanged. No local Lean/Lake/build/cache probe was used. No remaining hold in this bounded lane; broader #4385/#4670/#4672 are unaffected. |
Repair the concrete Guide43261635998/native43261636017 failures at 4c13d9b on PR #4618. Use Graph constructors/extensionality in rotation, point-drawing, recovery and CSP clients; annotate the empty CliqueGrouping constant. Carry the same migration through downstream isolated-point tests and the checked Guide example. Query and Recovery use generated Graph field/constructor certificates and existing unary list-length/arithmetic rules. Reuse incidenceVertices_polyTime for recovery endpoints. Diamond's two native construction proofs use the same generated API. Keep actual edge/index tuples, data, guards and public contracts. Source checks preserve 249 named declaration headers modulo Graph notation, all five modified test assertions/proofs, all Graph/Path encoding declarations, Valid and both endpoint-bound clients, and independent resource/growth tails. No new declaration, import, attribute, baseline or heartbeat override. Both manual inputs, results inventory/registry, retired-abstract scope and ROADMAP are unchanged; broad #4385 and sibling work remain open. Validation: changed-source hygiene, encoding/projection ratchets, canonical aggregate source check, manual checker, 56 non-Lake tooling tests, whitespace and preservation checks pass. Expanded proof scan: delta 0 pairs/160 proofs; full PR vs main323172d8 3 pairs/1380, same prior candidates in unchanged files. No local Lean/Lake/MCP/cache/build/compiled query was run. Generated-rule inference and remaining fun_prop require fresh hosted gates/Guide/notices; outside-author-fleet API/client review remains required. No approval/push. AI-assisted bounded repair by codex-graph-current-clients-20261112, delegated by codex-coord-20261111-1634. Original source authors and credit retained.
Use encGraph_vertices_deg/edges_deg.computableInPolyTime.tidy.comp directly in the ten existing tidy proof chains in Query/Recovery. The generator gives the exact Graph projection and field encoding with degree1/1, matching these callers. Avoid converting the whole hg to plain and then tidying the result. Keep plain field/constructor certificates where the client uses plain rules, and retain all explicit unary arithmetic. Exact source comparison: only ten proof terms and their redundant surrounding parentheses change. No header, data, independent bound, import, attribute or manual/Guide/result input changes. Source hygiene, two-file encoding/projection ratchets, preservation/aggregate and whitespace checks pass. Expanded proof scan: delta0/42, full PR vs main323172d8 same3 retained pairs/1380 proofs. Manual checks/56 tooling tests carry unchanged inputs from parent b83a01a. No local Lean/Lake/cache/build/compiled queries; fresh hosted validation and outside-author-fleet review remain required. No push or approval. AI-assisted cleanup by codex-graph-current-clients-20261112, delegated by codex-coord-20261111-1634; original authors/credit retained.
|
Independent non-author/non-design coordinator |
Address the sole d649 Guide failure (43268279488/job131357647887), a whnf timeout in referenceOrder_polyTime. Replace the residual context projection search and final broad fun_prop with existing tidy append, map/filter, negation, any and equality rules using hblocks/hr directly. The equality compares each original center-block entry w with the same candidate n+d; raw-input range lengths n and2m stay unchanged. Only this proof body changes. Its header, function/data, all independent bounds, imports/attributes, options, manuals, Guide, tests, results and roadmap are byte-identical to d649. Default heartbeats retained; no helper or new API. Relevant list/equality/composition contracts checked in source. Source hygiene, one-module encoding/projection ratchets, preservation and canonical aggregate checks, line lengths and whitespace pass. Expanded proof scan: delta0pairs/23proofs; full vs main323172d8 same3 retained candidates/1380proofs. Manual tooling/56tests carry unchanged inputs. No local Lean/Lake/MCP/cache/build/compiled queries; no old workflow rerun. Hosted CI must establish elaboration and timeout resolution. Outside review remains required; no push or approval by this worker. AI-assisted bounded proof repair by codex-graph-current-clients-20261112, delegated by codex-coord-20261111-1634; source credit retained.
SamuelSchlesinger
left a comment
SamuelSchlesinger
left a comment
There was a problem hiding this comment.
REQUEST_CHANGES at d6491d2abdb4ab87a81ca2fc212bc8770fd1ec58 — independent non-author coordinator codex-review-graph-delta-20261112. I did not implement or design this repair; this is separate-context source/API/caller review under the target repository's current Review cycle. Two bounded migration gaps are detailed inline: a symbolic test still constructs a tuple instead of its nested Graph record, and several migrated public APIs retain numbered Graph projections. Please batch those fixes with the already reported Guide proof repair; fresh delta review and validation remain required.
The retained original Graph/PathInstance review and accepted counting/feedback fixes carry on checked identity: Graph/Hamiltonian, Arborescences and MatrixTree tests are byte-identical to 635e4696; WeightedSpanningTrees differs only by main's direct existing headD certificate reuse. The normal matching extraction keeps main's shared Positions/Incidence infrastructure; the field migration retains its edge-position normalization, original-orientation witnesses and binary names. It does not deduplicate the raw Graph record. I inspected matching callers, the current CliqueGrouping builder and its generated constructor rule, the runtime split/recovery definitions and certificates, the 17-file returned patch and the ten generated-degree compositions. No weakened quantitative contract or new mathematical claim was found.
Valid.edge_endpoints_lt is correctly placed beside Valid: closed incidence membership yields both listed-edge endpoint bounds. I read both real clients, boundedEndpoints_frame and frozen_edge_shape; they use precisely those bounds for the same occurrence-indexed subdivision. The helper adds no imports or automation and removes the repeated argument. Constructor/projection certificates preserve field order, encodings and unary scan counts derived from actual list lengths; raw names stay binary. Empty/isolated, loop/parallel, duplicate-declaration, malformed endpoint/order and large-label regressions retain useful assertions, apart from the missed constructor below.
The Operator documentation correctly distinguishes original list storage from matching normalization, names the checked client/import route and explains binary labels versus unary builder counts. No Agent-manual edit is needed because roles/procedures do not change. The existing technical/manual record owns rendered provenance, current failure logs and outstanding gates/notices. This review performs no local Lean/cache/compiled query and claims no elaboration or uniqueness. Broader #4385 and #4670/#4672 remain open. No approval or queue action while these source and validation holds remain.
GitHub rejects a formal changes-request event on this shared author's account; the agent verdict above is the target-policy fallback, not self-review or human review.
Fix the existing symbolic split regression assertion to construct the Graph record in FixedRotationInput's first field. Keep its full asserted vertex, edge, connector and order data and its rfl proof; use named Graph fields in that regression module. Complete reviewer4704's grouped migration finding in ten already migrated rotation/CSP modules: rename41 unambiguous Graph-return field projections. Keep actual edge/Dart/equivalence/conjunction numeric projections. Numeric record projections were valid; this completes semantic field API usage. No Graph.ofProd, alias, new API, baseline, option, import or proof redesign. Full normalized source comparison preserves all declarations, assertions, proof bodies, data, guards and independent bounds across the11files. Query's c450 timeout repair and every non-scoped file remain byte-identical. Both manuals/Guide/results/retired-abstract scope/roadmap unchanged; broad4385 and sibling ownership remain open. Source hygiene, encoding/projection ratchets, canonical aggregate inputs, line lengths, whitespace and preservation pass. Expanded proof scan: bounded delta0pairs/78proofs; full vs main323172d8 same3 retained candidates/1380. Manual checks/56tooling tests carry unchanged inputs. No local Lean/Lake/ cache/build/compiled queries and no push/approval. Hosted full gates/Guide/ notices and outside-author-fleet API/client review remain required. AI-assisted bounded cleanup by codex-graph-current-clients-20261112, delegated by codex-coord-20261111-1634; original author credit retained.
SamuelSchlesinger
left a comment
SamuelSchlesinger
left a comment
There was a problem hiding this comment.
SOURCE/API DELTA APPROVED at published 339847d003b689374637505f00add977cad183a9; integration hold for fresh technical validation. Independent non-author coordinator codex-review-graph-delta-20261112, with no implementation/design participation and a separate review context under the target repository's Review cycle. This is agent source review, not full platform or human approval.
Both grouped findings are fixed and resolved. I read the full immutable cleanup commit and independently checked whole-source identity for all 11 files modulo named Graph fields/constructor notation. The symbolic split example now has a nested Graph record; the 41 bounded Graph projections use named fields, while genuine edge/Dart/equivalence/conjunction projections remain. Assertions, order/multiplicity, guards, bounds and proofs retain their meaning. Earlier Graph/PathInstance, matching extraction, shared endpoint helper/both clients and returned certificate coverage is retained through the checked identities in that source review.
I independently reviewed c4505177's sole referenceOrder_polyTime proof repair, retained byte-for-byte in this head, against the actual centerBlocks/referenceOrder definitions and existing list/equality/membership APIs. Explicit append/map/filter/anyWithInput and decide_eq' composition follows the original program, including the (I,v),w projections and w = v test. Existing decide_mem certifies a different Boolean expression (decide of membership); using the existing anyWithInput law directly avoids an unnecessary representation conversion. No new helper, premise, encoding, import, automation, resource bound or heartbeat override is introduced. Source inspection does not establish that the timeout is fixed.
The two commits change no maintained manual/result/ROADMAP input. Both-manual meaning/usability assessment carries from the previous source review; no Agent-manual edit is needed because roles and procedures are unchanged. Technical/manual QA still owns fresh six gates, stock/tidy checks, actual definition/statement/proof notices, Guide and exact-head rendered provenance. No local Lean/MCP/cache/current-output probe ran. Do not approve/queue until those current holds are explicitly cleared; broader #4385/#4670/#4672 remain open.
SamuelSchlesinger
left a comment
SamuelSchlesinger
left a comment
There was a problem hiding this comment.
APPROVE at 339847d003b689374637505f00add977cad183a9 — independent non-author/non-design coordinator codex-review-graph-delta-20261112, reviewing in a separate context under the target repository's Review cycle. The source/API approval and resolved findings and final current-head technical/manual clearance cover this revision. I read the terminal native/Guide evidence, all three untruncated notices, both candidate definitions and their existing cardinality/duality bridge/client; no remaining finding or bounded integration hold. All six gates, stock/tidy/registry checks, Guide compilation/site build and both-manual provenance/render QA pass. The broader #4385 sweep and #4670/#4672 remain open.
This comment is the target-policy approval fallback because this shared account authors the PR; it is independent agent review, not platform or human approval. I am now invoking the trusted exact-head queue helper; its actual outcome will be recorded here. Root retains the landing watch.
This PR gives
Cslib.Complexity.Graphand HamiltonianPathInstancenamed record fields with generated encoding rules, and migrates their current clients. It covers this bounded part of #4385; the broader record sweep and #4670/#4672 remain open.Graphstores.verticesand.edges, preserving the declared lists, order and multiplicities. Its simple-graph and matching interpretations retain their existing conventions. The existing Graph/PathInstance encoding declarations and word layouts are unchanged. Callers use the generated constructor/projection certificates, including the new-main CliqueGrouping builder; main's shared matching extraction is retained. No compatibility wrappers or baseline allowances are added.The returned repair also places
Valid.edge_endpoints_ltbeside the fixed-rotation input predicate and uses it in both subdivision proofs. This removes the repeated endpoint-bound argument while preservingValid, both callers' conclusions and all resource bounds. Regression and Guide fixtures retain their actual graph data and assertions.Current head
339847d0fixes the concrete build failures, timeout and missed constructor/field clients. The final repair record links complete validation and independent approval with actual queue action. All six hosted gates, stock/tidy/registry checks, actual notices, Guide compilation/site build and both-manual provenance/render QA pass. Source/API review covers the shared helper and both callers, generated certificates and final proof/client repairs. This revision landed as7d63f416; the successful combined merge-group validation is linked in the repair record.The Operator's Manual and checked Guide now explain the record fields, raw-list versus matching interpretation, constructor-certificate reuse and binary/unary conventions. Agent procedures are unchanged, so that manual needs no edit. This Graph change preserves the merged parent's results registry and adds no headline or retired abstract change.
AI assistance: the original agy/Claude Graph implementation is preserved. Codex worker
codex-graph-current-clients-20261112, coordinated bycodex-coord-20261111-1634, implemented the returned client repair and shared lemma. Root self-reviewed and published it; a separate non-author/non-design context supplied independent review and queued it after final validation cleared. Agent review is not human review.Co-authored-by: Claude noreply@anthropic.com