T103: the T074 files' composed reds, respelled to their carved homes and the composed tree (plan 038, T094's R2-INV-R9)

[brettheap]

Lane: openxfactory-4 (openXfactory-4-openDox_extraction)
Authored by lane openXfactory-3, delegated by lane openxfactory-4 (#656 6021862323)

Arc: neutral-product-standalone-operability

T103: the T074 files (plan 038; T094's map R2-INV-R9, slice outline row T103)

There is no tasks.md entry yet: lane 4 records T094's slices after T002 (openxFactory#1251). Until then, the contract is the slice outline row plus the holder's rulings, openxFactory#656 6021830531.

• Realizes: the composed half of requirement 9 for these four declared files (T073's F9.1: each declared file runs green in the composed workflow). The means are PP, plus MN, HR and CS for test_gate_console.py.
• Falsifier (the outline row): run the four files composed, each alone. 28 reds go to 0: completeness 2, packet 1, gate_console 1, round_trip 24. test_gate_console.py then reads 127 passed and 5 skipped. Its 5 skips are OQ-R9-3, which this slice respells so they run composed: it reads 132 passed, 0 skipped.
• Ruled: openxFactory#656 6021830531:
  • OQ-R9-1 to OQ-R9-4 at their defaults;
  • W7: each of these four files' chain is T103 → T074, so this lands before T074.
  • The ruling it sits under is 6016982816 (ARC-Q2 (a)).
• After: T095 (T095: the composed harness for the declared integration tests (plan 038, T094's R2-INV-R9) #46, landed as e2340bb6), which this PR codes against: openxfactory_root(), CONTRACTS_DIR and DECLARED_HOST_SUITES. T021 (T021, U-2: the gate console's schemas, each read where its owner keeps it, nothing vendored (plan 038, P4F-4) #40) landed as f70bc9c3. Both are met.

What changed (4 test files, nothing else)

file	nodes	means	change
tests/test_completeness.py	2	MN, PP	The purity check reads completeness.py at its carved home (carved_module_path), and doc_health.lines where this process imports it from. The gating bound scans this leg's import root src/ and expects openxdox/generator.py and openxdox/kickoff.py. Its doc_health half scans the doc_health package this process imports.
tests/test_round_trip.py	24	PP	_MOVER reads openxFactory's scripts/proposal-support.py through openxfactory_root() (23 nodes). The shared-predicate pin reads round_trip.py at its carved home and families.py from families.__file__ (1 node).
tests/test_doxbench_packet.py	1, the rail node	PP	The status-read drift check reads openxFactory's corpus, openxfactory_root() / "docs", instead of this checkout's docs/.
tests/test_gate_console.py	1 collection error, and the 3 cases behind it	PP, MN, HR, CS	The module-level forward-gate load goes through openxfactory_root(). The two in-test from ideation_dashboard import cli imports become from opendox import cli. HR (the two cases that parse gate) is T095's DECLARED_HOST_SUITES, and CS (the receipt case) is T095's CONTRACTS_DIR; neither needs an edit in this file.
tests/test_gate_console.py, OQ-R9-3	5 skips	CS	_contract_carries_the_new_verbs reads each schema where the pinned validator reads it. gate-action-record comes from the installed distribution's packaged copy (openxdox.contracts.verified_path). gate-intent comes from the directory CONTRACTS_DIR names. A lone checkout has no CONTRACTS_DIR, so the group still skips there. Once CONTRACTS_DIR is supplied, a schema that cannot be read fails collection instead of reading as "not released" (Copilot r4199436303).

Edits are kept local to the failing reads, so T074's later respelling of these files' doc_health imports stays clean. The two doc_health reads in completeness and round_trip resolve through the import system, so they follow whatever T074 makes those imports.

Flagged, a widening beyond the failing assertion: test_the_scoring_modules_only_importers_are_the_generator_and_the_guard's second loop scanned REPO_ROOT / "scripts" / "doc_health". This leg has no such directory, so since the carve the loop has scanned nothing and passed vacuously. It now scans the composed doc_health package, 38 modules, and passes. Mutant M2b below shows the check is live.

The falsifier, measured

Measured in R1Q23 (a)'s composed form:

• openxFactory 92010d3e (DRAFT: T029, U-9: the composed run, permanent: composed.yml and its openxFactory pin (plan 038) #44's tests/composed_host_pin.yaml), with openDox, openXdox and openXwallet initialized recursively, and its scripts/ on PYTHONPATH;
• openDox installed from openDox-code main 84f8ed83;
• colour variables unset, TERM=dumb;
• each file ALONE: python -m pytest -q -p no:cacheprovider -rfE --junitxml=… <file>.

The integration branches are LOCAL and never pushed:

• base = main e2340bb6, which carries T021 (T021, U-2: the gate console's schemas, each read where its owner keeps it, nothing vendored (plan 038, P4F-4) #40) and T095 (T095: the composed harness for the declared integration tests (plan 038, T094's R2-INV-R9) #46), both landed;
• head = this PR at 683b9562, which is main merged in and differs from it by exactly these four files.

Earlier runs over local integrations read the same, file for file: main d45a0939 + #40 + #46 at 6aeb6797, and main f70bc9c3 + #46 c2192e3e at 57a970fc.

The rail node needs T073's status-exemption rail. Without it, the node fails on pk.STATUS_SCAN_LINES, at the rail, before it reaches its corpus read. So the four files are also measured with #45 26fe63e7 merged in, with T022's placement made first and removed after.

Base + #45, 28 red:

tests/test_completeness.py: exit 1; 29 passed, 2 failed, 0 errors, 0 skipped
tests/test_doxbench_packet.py: exit 1; 85 passed, 1 failed, 0 errors, 0 skipped
tests/test_gate_console.py: exit 2; 0 passed, 0 failed, 1 errors, 0 skipped
tests/test_round_trip.py: exit 1; 50 passed, 24 failed, 0 errors, 0 skipped
TOTAL over 4 files: 164 passed, 27 failed, 1 errors, 0 skipped; 28 red

Head + #45, 0 red:

tests/test_completeness.py: exit 0; 31 passed, 0 failed, 0 errors, 0 skipped
tests/test_doxbench_packet.py: exit 0; 86 passed, 0 failed, 0 errors, 0 skipped
tests/test_gate_console.py: exit 0; 132 passed, 0 failed, 0 errors, 0 skipped
tests/test_round_trip.py: exit 0; 74 passed, 0 failed, 0 errors, 0 skipped
TOTAL over 4 files: 323 passed, 0 failed, 0 errors, 0 skipped; 0 red

The 28 red at base, each with its first failure:

• test_completeness.py: test_the_scoring_module_touches_nothing_outside_its_arguments (FileNotFoundError, scripts/ideation_dashboard/completeness.py); test_the_scoring_modules_only_importers_are_the_generator_and_the_guard (AssertionError: []).
• test_doxbench_packet.py::test_the_status_read_agrees_with_the_repositorys_own_corpus_reader: "the drift check needs real documents to be a check".
• test_gate_console.py: a collection error (FileNotFoundError, scripts/proposal-support.py).
• test_round_trip.py:
  • test_both_gates_agree_where_a_documents_status_header_is ×12, test_the_mover_agrees_with_the_other_fence_implementations ×10 and test_the_movers_line_split_is_the_three_real_endings_and_nothing_else: FileNotFoundError, scripts/proposal-support.py;
  • test_the_shared_predicate_is_spelled_the_same_in_all_three: FileNotFoundError, scripts/ideation_dashboard/round_trip.py.

Without #45, base vs head reads completeness 2→0, round_trip 24→0 and gate_console 1→0 (132 passed). Packet reads 57 red at both: its 56 rail nodes and the rail node all fail at the rail, which is T073's.

The two "rejected" conformance cases are rejected by the schema (validator exit 1), not by a harness failure (exit 2). Run by hand with the farm:

• a valid promote-to-staging record: exit 0;
• one with no workflow job: exit 1, "2 error(s)";
• a derive-possibles record with no cluster_id: exit 1, "1 error(s)".

Mutants of the respellings

This slice adds no guard and no fixture. Each mutant reverts or misdirects one respelled read; every one was run with that file alone, composed, over head + #45.

id	mutation	result
M1	purity check reads generator.py, not completeness.py	killed: 1 failed (AssertionError on the import set)
M2b	the doc_health loop scans src/ instead	killed: 1 failed (the loop's assertion names completeness.py)
M3	importer scan back to REPO_ROOT / "scripts"	killed: 1 failed (AssertionError: [])
M4	shared-predicate pin reads gate_console.py	killed: 1 failed
M5	rail node back to REPO_ROOT / "docs"	killed: 1 failed ("the drift check needs real documents to be a check")
M6	conformance group: CONTRACTS_DIR test inverted	127 passed, 5 skipped: the group reverts to never running
M7	conformance group reads a schema without the verbs	127 passed, 5 skipped: the skip condition reads the schema's content
M8	a broken farm: a pytest plugin removes gate-intent.schema.yaml from the farm after the conftest builds it	at 6aeb6797, before the fix: 127 passed, 5 skipped, so it survived (the masking Copilot r4199436303 named). At 57a970fc: a collection error (FileNotFoundError), so it is killed

Unchanged

Base (main e2340bb6) vs head (683b9562), each file alone and composed: 974 nodes, 0 differ, and no node goes from green to red anywhere.

• 12.5's 16 governed suites: identical at both, node for node. Their remaining reds are T073's and the other slices', not this PR's.
• F5.2's 5 protected files: generator 45, session_snapshot 23, snapshot_determinism 6, snapshot_registry 40 and snapshot_validation_launch 9, all passed at both.
• test_column_contributions_governed.py: 9 passed at both.
• tests/test_declared_exclusion.py, lone (no PYTHONPATH): 207 passed at base and at head.
• None of the four files is in 12.5's computed set (16) or F5.2's (7), so no allow-list entry is needed.

CI

T095 has landed, so this head carries openxfactory_root and CONTRACTS_DIR from main. Locally, in validate's lone form at 683b9562 (pinned openDox dede32b4, --deselect of LEFT_OUT): selected=1165 passed=1161 skipped=4 failures=0 errors=0, against the floors 1099 / 1095 / 4. tests/test_declared_exclusion.py run alone reads 207 passed at base and at head.

CI at 683b9562:

• validate passed (run 37548600589): selected=1165 passed=1161 skipped=4 failures=0 errors=0, against the floors 1099 / 1095 / 4 (margin 66);
• SonarCloud Code Analysis passed;
• Copilot's review 5435805891 at 683b9562: Findings: None. It rates the PR "Needs a closer look" only because the PR was a draft and the composed packet result depends on T073's rail, which lands separately.

Threads: 1 in all (r4199436303, fixed at 57a970fc, answered and resolved), 0 unresolved.

Not done / what remains

• Nothing is left on this PR's own After set: T021 (T021, U-2: the gate console's schemas, each read where its owner keeps it, nothing vendored (plan 038, P4F-4) #40) and T095 (T095: the composed harness for the declared integration tests (plan 038, T094's R2-INV-R9) #46) have landed, and this branch merges main e2340bb6. If main moves again, the holder's loop updates the branch.
• The rail node's composed green also needs T073 (DRAFT: T073: the composed declarations (ARC-Q2 (a)): every declared entry runs composed, the rail files get the host's rail (plan 038) #45), which lands after this slice. Until then it is measured over a local integration with DRAFT: T073: the composed declarations (ARC-Q2 (a)): every declared entry runs composed, the rail files get the host's rail (plan 038) #45, as above.
• T074 later respells these files' doc_health imports; this PR leaves them alone.

🤖 Generated with Claude Code

Summary by Sourcery

Update composed-workflow test paths and imports so the carved repositories validate their own and shared sources correctly.

Bug Fixes:

• Update four test suites to resolve source files, support scripts, documentation, and contract schemas from their correct locations in the composed repository workflow.
• Restore the gate-console CLI and contract conformance checks after the repository carve, including failure behavior for missing composed schemas.

Enhancements:

• Align completeness, round-trip, packet, and gate-console tests with the split openDox/openXfactory repository layout while preserving their existing coverage.

Tests:

• Make the four affected test files pass in composed isolation, eliminating 28 baseline failures and previously skipped contract cases.

[sourcery-ai]

Reviewer's Guide

This draft adapts four test suites to the post-carve composed workflow by resolving source modules, support scripts, corpus documents, and contract schemas from the locations actually used by openDox and openxFactory. The changes are limited to test read paths and imports, including activation of previously vacuous contract checks in composed runs while retaining skips for incomplete lone-checkout environments.

File-Level Changes

Change	Details	Files
Repoint completeness and importer-integrity checks to the carved openDox modules and the composed doc_health package.	Read completeness.py from its carved module path and doc_health.lines from the active import location. Scan src/openxdox for scoring importers and the composed doc_health package for forbidden dependencies.	tests/test_completeness.py
Align corpus and support-script reads with the composed openxFactory tree.	Use openxfactory_root() for the corpus used by the packet status-drift check. Load proposal-support.py from openxFactory rather than the checkout-local scripts directory.	tests/test_doxbench_packet.py tests/test_gate_console.py tests/test_round_trip.py
Update gate-console tests to exercise the carved openDox CLI and activate contract conformance checks against the correct schema locations.	Import the CLI from opendox. Read gate-intent from CONTRACTS_DIR and gate-action-record from packaged verified contracts. Preserve lone-checkout skipping when composed contract inputs are unavailable.	tests/test_gate_console.py
Update round-trip source-pinning checks to resolve each implementation from its actual composed-tree location.	Read round-trip.py from its carved home and families.py from the imported module path. Continue loading the mover from openxFactory's composed scripts directory.	tests/test_round_trip.py

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it. You can also reply to a
  review comment with @sourcery-ai issue to create an issue from it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time. You can also comment
  @sourcery-ai title on the pull request to (re-)generate the title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time exactly where you
  want it. You can also comment @sourcery-ai summary on the pull request to
  (re-)generate the summary at any time.
• Generate reviewer's guide: Comment @sourcery-ai guide on the pull
  request to (re-)generate the reviewer's guide at any time.
• Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
  pull request to resolve all Sourcery comments. Useful if you've already
  addressed all the comments and don't want to see them anymore.
• Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
  request to dismiss all existing Sourcery reviews. Especially useful if you
  want to start fresh with a new review - don't forget to comment
  @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[Copilot]

Copilot review overview

🟡 Changes recommended

A missing composed schema is incorrectly converted into passing test skips.

Review effort: Balanced
Findings: 1

Open (1)

• Do not mask gate-intent schema I/O errors as version deferrals · New

What changed in this PR

Updates four test suites for carved module locations and composed openxFactory resources.

Changes:

• Resolve source modules and schemas from their owning packages.
• Read composed tooling and documentation through openxfactory_root().
• Switch CLI tests to opendox.cli.

File	Description
tests/​test_completeness.py	Updates purity and importer scans.
tests/​test_round_trip.py	Resolves mover and predicate sources.
tests/​test_doxbench_packet.py	Scans the composed documentation corpus.
tests/​test_gate_console.py	Updates CLI, support script, and schema resolution.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Copilot review overview

🔵 Needs a closer look

Required dependency PRs #45 and #46 remain open, leaving this draft’s standalone CI intentionally red.

Review effort: Balanced
Findings: None

Resolved since last review (1)

• Do not mask gate-intent schema I/O errors as version deferrals

[Copilot]

Copilot review overview

🔵 Needs a closer look

It remains draft and its composed packet result depends on the separately landing T073 rail changes.

Review effort: Balanced
Findings: None

[sourcery-ai]

Sorry @brettheap, you've used your own review budget of 250,000 diff characters for the last 7 days.

You can request another review in 16 hours and 54 minutes by commenting @sourcery-ai review. Upgrade to get a review now.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[sourcery-ai]

Sourcery assessment

Approved.

[Copilot]

Copilot review overview

🟢 Approval recommended

The changes consistently implement the ruled composed-path behavior with no unresolved correctness issues found.

Review effort: Balanced
Findings: None

[brettheap]

READY at 7383719

Lane: openxfactory-4 (openXfactory-4-openDox_extraction)

Holder, automatic READY on Brett's word "land lane 3's PRs when green" (2026-10-06). At the head: CLEAN, rollup SUCCESS, nothing pending, 0 unresolved threads, and Copilot at 73837199 reads "🟢 Approval recommended". The per-repo lander checks closing keywords and lands it.

[brettheap]

Lane: openxfactory-4 (openXfactory-4-openDox_extraction)

LANDED — lane openxfactory-4, 2026-10-07T00:31:40Z, PR #49 → 6f25915 (opensoft/openXdox-code main; plain gate)

Brett: plan 038 RULED incl. lane openxfactory-4 lands every PR (#656 6013547504); Start release 2 (#656 6001702967)