Skip to content

Correct assay boundaries, strain direction and modeled mechanisms (batch 34) - #1416

Closed
realmarcin wants to merge 1 commit into
causal-graph-review-20261005-batch33from
causal-graph-review-20261005-batch34
Closed

realmarcin wants to merge 1 commit into
causal-graph-review-20261005-batch33from
causal-graph-review-20261005-batch34

Conversation

@realmarcin

Copy link
Copy Markdown
Contributor

Scope

Three completed existing-graph reviews: drought-iron rhizosphere, dual Bacillus/Pseudomonas LA fermentation, and ENIGMA denitrification. Corpus: 144 reviewed / 309 pending / 7 needs_research, across 460 records in both roots.

Closes #1410. Closes #1411. Closes #1412. Non-graph #1413/#1414/#1415 and existing cultivation-boundary #647 remain open.

Changes And Review

Nine nodes retained, four arrows qualified and two unsupported cross-assay arrows removed. No new causal structure or Edison job/charge. Field genomic associations, maize TOM1 controls and single-isolate sorghum amendments remain distinct.

Correct the proposed Bacillus exclusion direction to CC17B-1 toward DSM1-25280, preserving strain identity despite a shared species ID. CFU decline is not demonstrated extinction; competition, product inhibition and culturability alternatives remain explicit. No direct PAD-vanillin catalysis or donor-neutrality claim. Preserve aerobic 30 C ZL pretreatment and the subsequent anaerobic 50 C Bacillus relay.

ENIGMA joint growth and measured chemistry are distinct from modeled reverse NO transfer and high-condition nitrite exposure. Community scope resolves complementary exchange without a false single direction, obligate dependence or categorical competition. Negative causal link refers to nitrite, not N2O toxicity.

Verification

376 passed in 175.08s (0:02:55); eight new tests. Schema, strict, terms, independent snippets and history checks passed. Three reference-validator commands exit 0 but perform zero checks: a no-op, not evidence validation. Text matching is not scientific support.

Zero structural errors and 31 exactly unchanged baseline warnings. Participant registry 178 to 180 adds only Dual Bacillus and ENIGMA. ENIGMA changes mixed to community-level-only, accounting for two additional sole-rule taxa; no taxon, algorithm or numeric test-bound change.

Three guarded writes, append-only histories, llm-assisted events and regenerated pages. All non-graph raw blocks, 457 other records and 920 caches preserved. Repeated renders deterministic; exact descriptions checked across 456 pages. Actual post-commit docs-current passed with a clean tree.

Committed decision ledger, validation output and adversarial-review records: reports/causal_graph_review, dated 20261006, batch34. Primary cached main bodies, captions and tables read; no visual figures, supplements or raw data reviewed. Stale abstract_only cache headers disclosed, not changed. Source-based self-adversarial review, not independent approval.

Dependency

Draft depends on #1409; do not merge into the topic parent. Require protected exact-head checks and main merge queue after parents merge. Move dependents before branch deletion. Full-corpus completion is not claimed.

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Source-based self-adversarial review at exact head fcc5445; not independent approval.

Addressed #1410/#1411/#1412 after complete recorded primary-text and actual diff review. Nine nodes retained, four arrows qualified and two cross-assay arrows removed. Distinguish drought field genomic capacity, maize watered-control responses and single-isolate iron amendment. Correct CC17B-1-to-DSM1-25280 proposed exclusion, retaining shared-ID strain distinctions, CFU limits and staged temperature/oxygen conditions. ENIGMA measured physiology remains separate from modeled NO exchange and high-condition nitrite exposure; no obligate dependence, categorical competition or N2O-toxicity claim.

376 passed in 175.08s (0:02:55); eight new tests. Focused schema/strict/terms/snippet/history checks, deterministic renders, exact descriptions across 456 pages and actual post-commit docs-current passed. Reference-validator performed zero checks despite exit 0: no-op is not evidence validation. Zero structural errors, 31 unchanged warnings. Participant registry gains two records; ENIGMA scope shift accounts for measured census change, with no algorithm/numeric bound changes. All non-graph raw blocks, 457 other records and 920 caches preserved.

#647/#1413/#1414/#1415 remain open. No new causal structure or provider job/charge. Cached main-body text reviewed; no visual figures, supplements, raw data or browser screenshots claimed. Committed decision, validation and adversarial output records disclose limitations. Protected main merge and full-corpus completion are not asserted.

@realmarcin

Copy link
Copy Markdown
Contributor Author

CI scheduling note: 73 cancellation requests were accepted for current heads of task-owned dependent draft PRs; 7 of the 80 identified runs had already completed. Those drafts are not ready to merge and their ancestry changes as parents land. No cancelled check is counted as successful validation.

The seven normal required checks on merge-ready #1315 at 8eed5884638ba74c5493d0b2fc9765b3aa6fba2a were preserved. Fresh exact-head required checks and the protected main merge queue remain mandatory when each dependent PR is restacked and made ready. No workflow, required-check rule, queue rule or test gate was weakened. Other work was not cancelled.

Local audited plan/receipt: /private/tmp/communitymech-batch34/dependent-draft-ci-plan.json and dependent-draft-ci-deferrals.json. Batch 34 is published/reviewed in #1416 and registered in the guarded lifecycle through batch 34. Main merge and branch deletion remain pending actual protected completion.

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-adversarial review refreshed at exact head 320ccc53dbb11013b5ce6b545fc78e2c015fbd14
after protected main merge of #1315. This is not independent approval.

The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.

The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.

@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from 320ccc5 to a4d2690 Compare October 6, 2026 10:37
@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch33 branch from 3507e2c to 7441e72 Compare October 6, 2026 10:37

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-adversarial review refreshed at exact head a4d269002b4ad68a71bb062252d4d38fe01e6ebf
after protected main merge of #1318. This is not independent approval.

The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.

The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.

@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown

Network integrity findings

Warnings only — a member with no interaction yet, or a participant matched by ontology id rather than by name, or one on a community-level interaction that resolves to no member. Reported, but does not fail the build.

Network Integrity Audit Report
================================================================================

0 error, 31 warning across 12 records with findings
Only error-severity findings fail the build.

Algal_Methanotroph_Biogas_Valorization_Coculture
--------------------------------------------------------------------------------
  [warning] DISCONNECTED: Taxon 'Leptolyngbya' has no interactions

Total: 1 issues (0 error, 1 warning)

At_RSPHERE_SynCom
--------------------------------------------------------------------------------
  [warning] DISCONNECTED: Taxon 'Flavobacterium sp.' has no interactions
  [warning] DISCONNECTED: Taxon 'Pseudomonas sp.' has no interactions
  [warning] DISCONNECTED: Taxon 'Rhodococcus sp.' has no interactions
  [warning] DISCONNECTED: Taxon 'Streptomyces sp.' has no interactions
  [warning] DISCONNECTED: Taxon 'Variovorax sp.' has no interactions

Total: 5 issues (0 error, 5 warning)

Australian_Lead_Zinc_Polymetallic
--------------------------------------------------------------------------------
  [warning] DISCONNECTED: Taxon 'Acidithiobacillus ferrooxidans' has no interactions
  [warning] DISCONNECTED: Taxon 'Leptospirillum ferriphilum' has no interactions

Total: 2 issues (0 error, 2 warning)

Bacillus_G12_Y4_X25_Tobacco_Biocontrol_SynCom
--------------------------------------------------------------------------------
  [warning] UNKNOWN_TARGET: Target taxon 'tobacco' not found in taxonomy section (community-level scope)
  [warning] UNKNOWN_TARGET: Target taxon 'tobacco' not found in taxonomy section (community-level scope)

Total: 2 issues (0 error, 2 warning)

Bacillus_siamensis_vallismortis_HT_Masson_Pine_SynCom
--------------------------------------------------------------------------------
  [warning] UNKNOWN_TARGET: Target taxon 'Fusarium oxysporum' not found in taxonomy section (community-level scope)
  [warning] UNKNOWN_TARGET: Target taxon 'Pinus massoniana' not found in taxonomy section (community-level scope)
  [warning] UNKNOWN_TARGET: Target taxon 'Pinus massoniana' not found in taxonomy section (community-level scope)

Total: 3 issues (0 error, 3 warning)

Brocadia_Low_Nitrogen_Anammox_Consortium
--------------------------------------------------------------------------------
  [warning] DISCONNECTED: Taxon 'Ca. Kuenenia early-acclimation lineage' has no interactions

Total: 1 issues (0 error, 1 warning)

Dangl_SynComm_35
--------------------------------------------------------------------------------
  [warning] DISCONNECTED: Taxon 'Escherichia coli DH5α' has no interactions
  [warning] DISCONNECTED: Taxon 'Flavobacterium species' has no interactions
  [warning] DISCONNECTED: Taxon 'Microbacterium species' has no interactions
  [warning] DISCONNECTED: Taxon 'Stenotrophomonas species' has no interactions
  [warning] DISCONNECTED: Taxon 'Xanthomonadales members' has no interactions

Total: 5 issues (0 error, 5 warning)

Deepwater_Horizon_Deep_Sea_Oil_Plume_Succession
--------------------------------------------------------------------------------
  [warning] DISCONNECTED: Taxon 'Methylophaga' has no interactions

Total: 1 issues (0 error, 1 warning)

Mangrove_Benzene_MFC_Bioanode_Consortium
--------------------------------------------------------------------------------
  [warning] DISCONNECTED: Taxon 'Pelobacter' has no interactions
  [warning] DISCONNECTED: Taxon 'Pseudomonas' has no interactions

Total: 2 issues (0 error, 2 warning)

Nitrifying_Wastewater_Ammonia_Oxidizing_Consortium
--------------------------------------------------------------------------------
  [warning] DISCONNECTED: Taxon 'Legionella MAGs' has no interactions

Total: 1 issues (0 error, 1 warning)

Sclerotinia_Sclerotia_12Strain_Biocontrol_SynCom
--------------------------------------------------------------------------------
  [warning] UNKNOWN_TARGET: Target taxon 'Sclerotinia sclerotiorum' not found in taxonomy section (community-level scope)
  [warning] UNKNOWN_TARGET: Target taxon 'Sclerotinia sclerotiorum' not found in taxonomy section (community-level scope)

Total: 2 issues (0 error, 2 warning)

Space_Habitat_SevenMember_Stress_Tolerance_SynCom
--------------------------------------------------------------------------------
  [warning] DISCONNECTED: Taxon 'Acinetobacter radioresistens 50V1' has no interactions
  [warning] DISCONNECTED: Taxon 'Micrococcus luteus W4-Al-K1' has no interactions
  [warning] DISCONNECTED: Taxon 'Propionibacterium cyclohexanicum TA-12T' has no interactions
  [warning] DISCONNECTED: Taxon 'Pseudomonas antarctica CMS 35T' has no interactions
  [warning] DISCONNECTED: Taxon 'Staphylococcus capitis K1-2-2-23' has no interactions
  [warning] DISCONNECTED: Taxon 'Streptococcus halotolerans HTS9T' has no interactions

Total: 6 issues (0 error, 6 warning)

The full report is attached to the workflow run as an artifact.

@realmarcin

Copy link
Copy Markdown
Contributor Author

Reference-counter correction (#1469)

The earlier statement that linkml-reference-validator 0.1.7's Total checks: 0 means zero executed checks or a no-op is incorrect and withdrawn. The installed CLI prints the number of validation issues, not checks performed. Existing positive/negative and field-discovery canaries pass; EvidenceItem snippets are genuinely checked. SupportingReference coverage remains incomplete under #476. Text matching is not scientific support or independent source provenance.

The source-backed correction and historical candidate inventory are committed in #1470 at 22b5b12c2386a949d1ec4ddb1f79ee288085bd1c, under reports/causal_graph_review/README.md and reference-validator-correction-20261006-batch42.json. Earlier raw command output, source review, biological decisions and tested bytes are preserved; this notice does not claim a rerun of this PR's validation or change its biological findings. The older CLI counter is not a coverage metric.

The new Fucoidan batch separately retains 14 baseline reference-CLI failures and 15 baseline snippet-audit mismatches for supplement-sidecar excerpts. Those are not failures attributed to this older PR. This correction is self-review, not independent approval or merge authorization beyond the existing protected workflow.

@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch33 branch from 7441e72 to 4d900a7 Compare October 6, 2026 11:53
@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from a4d2690 to 4413e26 Compare October 6, 2026 11:53

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-adversarial review refreshed at exact head 4413e2662d9e737d0578017913c704a9085e32f1
after protected main merge of #1323. This is not independent approval.

The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.

The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.

@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch33 branch from 4d900a7 to d572bc7 Compare October 6, 2026 13:02
@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from 4413e26 to 7a775c8 Compare October 6, 2026 13:02

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-adversarial review refreshed at exact head 7a775c81789d54275d11d46c9f46e4d822a5e03a
after protected main merge of #1328. This is not independent approval.

The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.

The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.

@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch33 branch from d572bc7 to 94c2336 Compare October 6, 2026 14:15
@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from 7a775c8 to 1a05dfe Compare October 6, 2026 14:15

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-adversarial review refreshed at exact head 1a05dfe6a682b416397d2bbc4ef8d27cdf168acb
after protected main merge of #1334. This is not independent approval.

The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.

The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.

@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from 1a05dfe to d49713b Compare October 6, 2026 15:19
@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch33 branch from 94c2336 to d3db0fe Compare October 6, 2026 15:19

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-adversarial review refreshed at exact head d49713bd482730168aa95a6e8f0ab9723a91a3db
after protected main merge of #1339. This is not independent approval.

The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.

The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.

@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from d49713b to f8afe3f Compare October 6, 2026 16:37
@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch33 branch from d3db0fe to f6543e9 Compare October 6, 2026 16:37

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-adversarial review refreshed at exact head f8afe3fa4f0d21cff683ba51910b498e2dfdcb60
after protected main merge of #1346. This is not independent approval.

The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.

The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.

@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from f8afe3f to cd6f44c Compare October 6, 2026 17:57

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-adversarial review refreshed at exact head cd6f44c3f117db2e1a01fc237c28b45fbd34c4fa
after protected main merge of #1350. This is not independent approval.

The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.

The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.

@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from cd6f44c to af73094 Compare October 6, 2026 19:09

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-adversarial review refreshed at exact head af7309465652b6150e7239613bf1c2e2eee9e4f4
after protected main merge of #1355. This is not independent approval.

The entire tree remains identical to the preceding reviewed tree b1133b3c622afdf3a22e1514fe2ff2b10948d9dc.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.

The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.

@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch33 branch from 195a0a3 to e44c5c0 Compare October 7, 2026 09:31
@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from af73094 to 6b2d3ac Compare October 7, 2026 09:31

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-adversarial integration review at exact head 6b2d3ac97c21972fc9a3fe25b016714057bc4bf8. Not independent approval.

Integrated already-merged schema prerequisite #1692 (67603e192525899fae75420e2c19e8212b16afc9). Compared with the prior reviewed head af7309465652b6150e7239613bf1c2e2eee9e4f4, the only changes are the canonical schema pin, shared schema and generated datamodel from that main commit. Every other tracked file is identical; the batch-owned binary patch remains byte-identical (SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18). All 60 published batch patches were checked this way. No scientific assertion, reference cache, history, decision ledger or generated page was edited by this integration.

Adversarial checks: no inherited schema-file edits lost; generated CrossCorpusLink remains standalone and does not add a causal claim; normal strict validation passed all 460 records; required CrossCorpusLink fields enforced. Fresh broad regression runs on both ends of the stack: 302 passed in 172.77s (0:02:52); 756 passed in 255.55s (0:04:15). Docs-current passed clean at batches25/84, and vendored-sync passed. The 58 intermediate combinations were not independently retested. Existing primary-source review limitations and unresolved research issues remain applicable; historic validation receipts are not rewritten as new scientific reviews.

An additional generated-datamodel loading check FAILED and is retained in the integration receipt. Differential testing found exactly the same failures on all 460 paths before and after this integration:417 unhashable TaxonDescriptor and43 unhashable CultureCollectionEnum exceptions, with linkml-runtime1.11.1. Existing tests explicitly use plain YAML instead of this loader. Pre-existing defect #1703 remains OPEN; no datamodel-loader success is claimed. Normal strict schema validation is a separate passing path, not a relabeling of that failure.

Required CI must pass on this exact head and the protected merge-queue candidate. No manual workflow dispatch, artificial status, administrative merge or protection bypass. Dependents stay draft until their parent is confirmed merged; move dependents before branch deletion.

@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch33 branch from e44c5c0 to bd27199 Compare October 7, 2026 14:18
@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from 6b2d3ac to abf2a11 Compare October 7, 2026 14:18

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-adversarial review refreshed at exact head abf2a113461638161dc5a826c1b742d45ef06a87
after protected main merge of #1361. This is not independent approval.

The entire tree remains identical to the preceding reviewed tree 7d91d383dc06c5701e62a2f688c4e7abf87e088b.
The batch-owned binary patch is byte-identical
(SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18); only ancestry changed.
Scientific records, caches, source-review output, histories, regression tests, and generated pages
are unchanged. Historical validation results and recorded source/access limitations therefore
remain applicable, but required CI must pass at this new head. No new source review or local test
rerun is claimed by this ancestry-only update.

The protected main queue remains mandatory. Keep dependents on their reviewed parent until that
parent is confirmed merged; move dependents before deleting merged branches.

@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from abf2a11 to 6fc6a28 Compare October 7, 2026 21:32

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-adversarial governance integration review at exact head 6fc6a2830d8a2a9aa68f667d96ede15fe4f067bf. This is a COMMENT, not independent approval.

Integrated already-merged main PR #1730 (fe1752572a688051e8105e5b89d3099fe2202c63). Compared with prior reviewed head abf2a113461638161dc5a826c1b742d45ef06a87, exactly two files changed: scripts/.vendored_canon_ref and tests/test_id_label_empty_adapter.py. Each resulting tree differs from its old tree by the exact binary patch from main, and each batch-owned binary patch remains byte-identical (SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18). These invariants passed on all 74 published branches. Original heads are retained in local archive refs/archive/communitymech-pre-governance-20261007. No scientific record, source cache, history, decision ledger, schema, runtime code or generated page changed in this integration.

Adversarial inspection: the adapter regression still checks the empty sentinel and case-insensitive caching, now with an isolated fake oaklib module and a one-call selector assertion. No validation threshold or runtime behavior was relaxed. Fresh regressions at both stack ends: 337 passed in 214.19s (0:03:34); 917 passed in 402.58s (0:06:42). Docs-current and pinned vendored-sync passed at batches26/99. The 72 intermediate combinations were checked by tree/patch hashes, not independently retested. Historical source-review limitations and unresolved issues, including loader #1703 and cache #1091, remain applicable. This is not new paper review, independent approval, or a claim that those outstanding issues are fixed.

Digest-bound integration receipt/logs: /private/tmp/communitymech-governance-integration-20261007/receipt.json. Prior output/review records in reports/causal_graph_review/ remain unchanged. The prior lifecycle receipt is archived and will be updated only after all exact-head comments are verified.

Required CI and independent review must pass for the exact head and protected merge-queue candidate. No CI rerun, manual dispatch, fabricated status, admin merge or protection bypass. Dependents remain draft; move them before deleting a merged parent branch.

@realmarcin
realmarcin added this pull request to stack #1973 October 8, 2026 18:56
@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch33 branch from 2b8f2b0 to 3c9c9ad Compare October 8, 2026 19:04
@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from 6fc6a28 to 17d5884 Compare October 8, 2026 19:04

@realmarcin realmarcin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exact-head self-adversarial restack review at 17d5884900312be64475b81b952c75aa5e3c7435.
COMMENT, not independent approval.

The batch-owned binary patch remains byte-identical:
SHA-256 6488e9c21e74f30855b7167ca69d84821d2cf46ba4d0d3d6294ae4d833011c18.
Tree 0eb69c383ec4be43714eea3420d845f18831ae09 matches the independently computed clean merge.
Compared with historical tree 674aff3457be13c1a2a2a47da66bbafd1139e5b4, only scripts/.vendored_canon_ref
changes, incorporating main PR #1834's pin 849f336e025510316a5f235eb0af8547b8bd50cc.
All curated records, source caches, review outputs, histories, regression tests, and rendered
pages remain unchanged. No new primary-source review or local scientific-test rerun is claimed.

This PR is now part of native stack #1973, rooted at #1363 on main. No protected merge
has occurred. Fresh eligible PR/merge-group CI remains mandatory. No rule bypass, fabricated
status, merge into a topic parent, or premature branch deletion.

@realmarcin
realmarcin marked this pull request as ready for review October 8, 2026 19:09
@realmarcin
realmarcin force-pushed the causal-graph-review-20261005-batch34 branch from 17d5884 to 2805b96 Compare October 9, 2026 08:02
@realmarcin
realmarcin removed this pull request from stack #1973 October 10, 2026 02:32
@realmarcin

Copy link
Copy Markdown
Contributor Author

Superseded by the protected cumulative merge #1975 at 42959bef2549f9a42c3e3303a0e10fd5be99f30e. This PR's reviewed source head 2805b964449340ac12a9e578e6b3d45134539bcf was included in the cumulative branch; the final squash tree matches the reviewed integration tree exactly, and all seven required merge-group checks passed.

Patch-inclusion receipt. Existing issue-closing links were transferred to #1975; unresolved research and non-graph gaps are not closed by this action. This PR is being closed as superseded, not represented as individually merged. Archival provenance tags retain source commits; cleanup removes branch refs only and preserves worktree files.

@realmarcin realmarcin closed this Oct 10, 2026
@realmarcin
realmarcin deleted the causal-graph-review-20261005-batch34 branch October 10, 2026 02:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant