Skip to content

T097, release 1's bookkeeping (plan 034) - #1233

Merged
brettheap merged 17 commits into
mainfrom
bookkeeping/034-t097-release-1-ticks
Oct 5, 2026
Merged

brettheap merged 17 commits into
mainfrom
bookkeeping/034-t097-release-1-ticks

Conversation

@brettheap

@brettheap brettheap commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

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

Plan 034 T097, release 1's bookkeeping. This PR ticks all 63 release-1 boxes of #1144's tasks.md (add-neutral-product-standalone-operability) that a realization task closes, each with its evidence note, and records research R16's non-normative corrections. Its After set has landed: batch P (#1230), T096 (#1241 → 8a37a087) and T099 (the publish at the cut, and openDox#19 → d77f8cbf). On the holder's rulings it also ticks plan 034's T095, T099, T002, T007 and T097, each with its bullet, so plan 034 closes at 92 of its 96 tasks. T090 to T093 tick at the arc's close.

  • Bookkeeping, so it carries no Arc: trailer (R1Q20 (a), 5817152735; T091) and no closing keyword. It touches openspec/changes/, so lane openxfactory-4 lands it under a Rule 6 LANDING/LANDED window.

  • Based on main 8a37a087 (T096: AT-R1 at openDox-code 389e5a4a (RELEASE1_TIP), both halves, with T095's acceptance job; tick T096 (plan 034) #1241, T096), merged in by 0cde3961. The PR was first based on c8d80020 (Q-GRC-4 bookkeeping: tick 6.9-6.14 against T2 (#1006 → 765d8c6f); 6.15 and 6.18 left open, one named clause each #1232). The merges since are listed below. Its commits:

    • 76d8ff37: the ticks, the notes and R16;
    • b7f49673 and e2efb3a1: from Copilot's review (below), which edit note text only;
    • bd32ea9c: the merge of main after batch P landed;
    • cf2c6bdc: 16.3a's tick, on the T100 follow-on's landing (openDox-code#86 → 651c35fe);
    • 5870cd00: 4.3's and 9.2's ticks, on T086's landing (openXdox-code#37 → 56e1c238);
    • 23665543: 5.5's note reworded (Copilot at 5870cd00), note text only;
    • 96c74eab: the merge of main after T094 and T098 landed;
    • 935164af: 3.3's tick, on T098's landing (openxFactory#1237 → 20ce593e);
    • e93ab9fb: 5.4a's note reworded (Copilot at 935164af), note text only;
    • f6c6dc3c: the merge of main after T089 landed;
    • e873b25f: F4.1's, F5.2's and F16.1's ticks, on T089's landing (openxFactory#1238 → fab575ad);
    • 9759313b: the merge of main f2acc88c;
    • 10b6ba05: the phase-1 and phase-2 checkpoint citations by path (below), note text only;
    • 0cde3961: the merge of main after T096 landed;
    • f94be2fd: 10.3's tick, on T099's publish, and plan 034's ticks (below);
    • 9713f991: plan 034's opening status, which still said batch P was to land (Copilot at f94be2fd, below), plan text only.

    If main moves before the landing, I merge it in. No rebase and no force-push.

Measured, not copied

Lane 4's table (2026-10-03T20:05Z, at de2ab703: 52 CLOSED-LANDED, 11 PENDING, 5 LEFT-OPEN, 2 ALREADY-TICKED) was regenerated by its own generator, patched, at c8d80020, at ba6bb870, and again after each closing landing:

Result: 63 CLOSED-LANDED, 0 PENDING, 5 LEFT-OPEN and 2 ALREADY-TICKED, of 70. Since lane 4's table:

  • T082 (openDox-code#76 → ca9e1bd5), T100 (openDox-code#82 → 38d3350e), T102 (openDox-code#81 → 0116293a) and T083 (openxFactory#1231 → 0e01ca85) have landed. That closes 16.4, 16.5 and 16.6.
  • The T100 follow-on (openDox-code#86 → 651c35fe) has landed. That closes 16.3a.
  • T086 (openXdox-code#37 → 56e1c238) has landed. That closes 4.3 and 9.2.
  • T098 (openxFactory#1237 → 20ce593e), phase 3's interim F11.1, has landed. That closes 3.3 (below).
  • T089 (openxFactory#1238 → fab575ad), phase 3's checkpoint, has landed. That closes F4.1, F5.2 and F16.1 (below).
  • T076 (openDox#17 → 504324de) and T101 (openDox-code#78 → d59f3f26, openDox-code#79 → dede32b4) have landed.
  • T095 (openDox-code#75 → 389e5a4a) and T096 (T096: AT-R1 at openDox-code 389e5a4a (RELEASE1_TIP), both halves, with T095's acceptance job; tick T096 (plan 034) #1241 → 8a37a087), AT-R1's two halves, have landed. They close no box of File add-neutral-product-standalone-operability: open the BUILD arc, recording ten rulings #1144.
  • T099 has published opendox 0.1.0, and its root README PR has landed (openDox#19 → d77f8cbf). That closes 10.3 (below), the last box.

Batch P closes and rewords no box. In #1144's tasks.md it only inserts addenda at the ends of 12.4a, 16.3a and F16.1: no box line moves and no box changes state. Since ba6bb870, no landing has touched #1144's tasks.md. In plan 034's tasks.md, T097's own text is unchanged.

The census

python3 box_census.py openspec/changes/add-neutral-product-standalone-operability/tasks.md, with the tool written from research.md § Appendix (byte-identical to the Appendix's heredoc).

Before, at main 8a37a087 (identical to c8d80020's, ba6bb870's, 20ce593e's and a43845e6's):

2 8 2.1[ ] 2.1a[ ] 2.2[ ] 2.3[ ] 2.4[ ] 2.5[ ] 2.6[ ] F2.1[ ]
3 5 3.0[x] 3.1[ ] 3.2[ ] 3.3[ ] F3.1[ ]
4 5 4.1[ ] 4.1a[ ] 4.2[ ] 4.3[ ] F4.1[ ]
5 12 5.0[ ] 5.1[ ] 5.2[ ] 5.3[ ] 5.3a[ ] F5.1[ ] 5.4[ ] 5.4a[ ] F5.2[ ] 5.5[ ] 5.6[x] F5.3[ ]
7 8 7.0[ ] 7.1[ ] 7.1b[ ] 7.1a[ ] 7.2[ ] 7.3[ ] F7.1[ ] F7.2[ ]
9 8 9.1[ ] 9.2[ ] 9.2a[ ] 9.3[ ] 9.4[ ] 9.5[ ] F9.1[ ] F9.2[ ]
10 5 10.1[ ] 10.2[ ] 10.2a[ ] 10.3[ ] F10.1[ ]
11 3 11.0[ ] 11.1[ ] F11.1[ ]
13 8 13.1[ ] 13.2[ ] 13.3[ ] 13.4[ ] 13.4a[ ] 13.5[ ] 13.6[ ] F13.1[ ]
16 8 16.1[ ] 16.2[ ] 16.3[ ] 16.3a[ ] 16.4[ ] 16.5[ ] 16.6[ ] F16.1[ ]
total 125 Counter({' ': 105, 'x': 11, '~': 9})

After, at this head 9713f991 (9713f991 changes only plan 034, so the census is f94be2fd's):

2 8 2.1[x] 2.1a[x] 2.2[x] 2.3[x] 2.4[x] 2.5[x] 2.6[x] F2.1[x]
3 5 3.0[x] 3.1[x] 3.2[x] 3.3[x] F3.1[x]
4 5 4.1[x] 4.1a[x] 4.2[x] 4.3[x] F4.1[x]
5 12 5.0[x] 5.1[x] 5.2[x] 5.3[x] 5.3a[x] F5.1[x] 5.4[x] 5.4a[x] F5.2[x] 5.5[x] 5.6[x] F5.3[x]
7 8 7.0[x] 7.1[x] 7.1b[x] 7.1a[x] 7.2[x] 7.3[x] F7.1[x] F7.2[x]
9 8 9.1[x] 9.2[x] 9.2a[x] 9.3[x] 9.4[x] 9.5[ ] F9.1[x] F9.2[ ]
10 5 10.1[x] 10.2[x] 10.2a[x] 10.3[x] F10.1[x]
11 3 11.0[ ] 11.1[ ] F11.1[ ]
13 8 13.1[x] 13.2[x] 13.3[x] 13.4[x] 13.4a[x] 13.5[x] 13.6[x] F13.1[x]
16 8 16.1[x] 16.2[x] 16.3[x] 16.3a[x] 16.4[x] 16.5[x] 16.6[x] F16.1[x]
total 125 Counter({'x': 74, ' ': 42, '~': 9})

The release-1 groups are shown. Groups 1, 6, 8, 12, 14, 15 and the follow-ons read the same before and after. Earlier heads read 'x': 66, ' ': 50 (76d8ff37 to bd32ea9c), 'x': 67, ' ': 49 (cf2c6bdc), 'x': 69, ' ': 47 (5870cd00 and 23665543), 'x': 70, ' ': 46 (935164af to f6c6dc3c) and 'x': 73, ' ': 43 (e873b25f to 0cde3961).

The change: #1144's tasks.md, and plan 034's ticks

  • 63 boxes ticked, each with its evidence note as the item's last paragraph, after any addenda. The note is in 1.8's and 3.0's form: a bold dated lead, **Landed <date>**, where the date is the last landing of the box's realizing tasks; the tasks; the landings as repo#n → sha8; and the falsifier's result, with where it was quoted. For example, 2.4:

    Landed 2026-09-27 (T030, in T011's PR): openDox-code#46 → 0e88454a. tests/test_imports_standalone.py::test_every_module_imports_with_no_sibling first asserts the four siblings absent, then imports every module, failing with the first that still needs one (evidence/checkpoint-phase1.md § 1).

  • No ratified line moves. Against main 8a37a087, File add-neutral-product-standalone-operability: open the BUILD arc, recording ten rulings #1144's diff is the 63 checkbox lines, [ ] → [x], and 64 inserted paragraphs (482 lines): 63 notes, and 1 R16 note on an open box (9.5). This was proven at f94be2fd by setting every checkbox to [ ] in both files and diffing: the result is 64 insertions and nothing else, and the lines the two files share differ only by the 63 flips. The only other file the PR changes is plan 034's tasks.md (below). The file is byte-identical to a fresh re-apply of the regenerated decisions onto main's file.

  • Research R16 (research.md § R16), each figure re-checked against research R2, R4, R5, R10 and R13 and plan 034's records:

R16 item box where it is recorded
1 2.5 in 2.5's note: 22 modules, not 26, were reached by no CI step
2 9.4 in 9.4's note: openXdox-code's floors read 564/558/6 at 626f2c8d, not 539/533/6
3 4.3 in 4.3's note, since 4.3 closed on T086 (an open-box note until then): run_scoped_doc_health makes three of the four reaches, and the fourth is session_documents
4 3.2 in 3.2's note: the proxy's refusal was written for NOTHING REGISTERED
5 10.1 in 10.1's note: Q-R4 is recorded at runtime/cli.py:39-42, not :40-46
6 9.5 a note on the open box (open for the arc's close, T090): the openDox pin was 11 commits behind, not 9, and T086 has since moved it to dede32b4
7 Group 2's heading in 2.1's note, the box the heading introduces: 1,305 and 1,239 at 1e4a57fb
8 3.0 no new line: 3.0's own note already records it ("Read at ratification, 2026-09-24")

10.3, closed on T099

10.3's box says the openDox root documents the entry point and does not host it, and batch H's addendum makes the documented command pip install "opendox[local]", then opendox generate-and-open --local …. Plan 034 maps it to T076, T101 and T099:

  • T076 (openDox#17 → 504324de) documents the install and the start, and adds no make target.
  • T101 adds the release workflow and the version bump (openDox-code#78 → d59f3f26, openDox-code#79 → dede32b4).
  • T099 publishes that commit at the cut, on Brett Heap's word (5997633384). Read from the forge and from PyPI:
    • the annotated tag v0.1.0 (9153c254) names dede32b4, which the openDox root's contracts/code-pin.yaml names;
    • openDox-code's release run 37339111713, dispatched on that tag, completed success in all four jobs: build and verify, publish to TestPyPI (the dry run), install opendox[local] from TestPyPI and publish to PyPI, whose steps include PyPI serves the files the build job verified;
    • https://pypi.org/pypi/opendox/0.1.0/json lists exactly two files, neither yanked: opendox-0.1.0-py3-none-any.whl (sha256 8ecea00db6f9…) and opendox-0.1.0.tar.gz (sha256 56869b6208a8…);
    • openDox#19 → d77f8cbf replaces the README's stand-in paragraph. At that commit, the README's first line resolves from PyPI, and no paragraph says that no release is published.

Plan 034: T095, T099, T002, T007 and T097

Lane openxfactory-4's delegation of 2026-10-05T16:37:31Z asks #1233 to tick these, each with a Landed or Done bullet in #1226's form, so that release 1 closes at 92 of 96. Its ruling 5999175092 settles three points: (a), (a) and (a). Every sha in the bullets was read from GitHub.

The checkpoint citations, by path

Lane openxfactory-4 ruled at 2026-10-05T16:25:25Z that the notes' CP1 § n and CP2 § n citations name their records, as the phase-3 notes name evidence/checkpoint-phase3.md. CP1 and CP2 are the shorthand of lane 4's table, and #1144 never defines them. The change is non-normative and inside T097's R16 scope. 10b6ba05 makes it. The 26 citations, in 25 notes, now read evidence/checkpoint-phase1.md (T049, #1204) and evidence/checkpoint-phase2.md (T063, #1218). A word diff against e873b25f finds those 26 tokens and nothing else; only the 25 paragraphs' line breaks move.

F4.1, F5.2 and F16.1, closed on T089

T089, phase 3's checkpoint, landed on 2026-10-05T15:26:43Z as #1238 → fab575ad. Its record, specs/034-opendox-standalone-operation/evidence/checkpoint-phase3.md, extracts each block from #1144 by anchor at 36908480 and runs it from a fresh worktree at the commits phase 3 pinned: openDox-code dede32b4, the code of the openDox root e1e3a3c3 (T087), and openXdox-code 56e1c238 (T086). It says of F4.1 and F5.2 "T097 ticks both in #1144; this record ticks nothing there", and plan 034's T089 Run bullet says the same. Each box closes on that run, by plan 034's Box accounting:

  • F4.1 (F4.1 → T089): as extracted (sha256 8900b985e72f) and run unchanged, it exits 0 with 1 passed, 1 passed, then no deferred reach names the consumer or the publisher (§ 1).
  • F5.2 (F5.2 → T086, T089; RULED 5962785556, item 1, moves its close to phase 3): whole, as batches C, F, G and K amend it (OPENXFACTORY at 36908480, last step --chains), it exits 0. The seven suites pass whole (45, 23, 18, 6, 40, 9 and 12 passed), test -s holds, and the last step prints five admitted: lines, then ok: 5 protected edit(s), each entered and holding (§ 5). One of the five admits T086's edits to tests/test_session_snapshot.py, by entries 16, 17 and 19 of tests/protected_suite_respellings.yaml at 56e1c238: the three pre-arc reds that T063 quoted, each entry naming the test it repairs.
  • F16.1 (F16.1 → T083 (at 38d3350e), T089 (batch P's cases, at T087's pin)): with batch M's line, it exits 0 with 24 passed, the dialect line, no model configured: the catalog offers nothing, 83 passed and 532 passed (§ 4). dede32b4 carries the T100 follow-on (openDox-code#86 → 651c35fe is its ancestor, and the only commit between T100's landing and dede32b4 that touches tests/test_model_binding_trust.py). That file is the same at both: 160 tests, among them batch P's six test_F16_1_batch_p_* cases and A8's test_A8_a_state_directory_that_is_a_link_trusts_nothing, none of which exists at 38d3350e. T083's run there is quoted beside it, as before.

3.3, closed on T098

3.3 asks that the carve manifest's deleted_at_carve row for scripts/ideation_dashboard/profile_openxfactory.py stay BYTE-IDENTICAL. T017, which realizes it, reads the row through the three interim F11.1 runs, "and T097 ticks 3.3 on all three". Each was re-checked on main 20ce593e:

4.3 and 9.2, closed on T086

T086 landed on 2026-10-05T12:23:03Z as openXdox-code#37 → 56e1c238. It is a squash whose tree (78ef1ed1) is that of its head 57bebd8b. Its pin is openDox-code dede32b4, the commit T087 pins (openDox#18 → e1e3a3c3).

  • 4.3 closes "with the last of" the nineteen reaches into openXdox, and "openXdox-code's ratchet (OPENDOX_BACK_IMPORTS) is tightened to (0, 0) in the same landing". At 56e1c238:
    • tests/test_dependency_direction.py holds OPENDOX_BACK_IMPORTS = {}, and test_the_pinned_opendox_does_not_import_openxdox_back asserts (0, 0).
    • openXdox-code#37's body reads that file 17 passed at the pin, with the census empty.
    • consumer_reach.py, absent at dede32b4, was retired by T084 (openDox-code#77 → e49b17c3).
  • 9.2 closes in phase 3 (batch F's addendum), with batch B's note. At 56e1c238:
    • tests/declared_exclusion.yaml holds 66 entries carrying 68 reasons: 60 doc_health, 3 status-exemption-rail and 5 openxfactory-contracts.
    • CI's validate at the PR's head (openXdox-code run 37307785866) printed open extraction: the declared exclusion and triple: selected=1099 passed=1095 skipped=4 failures=0 errors=0.
    • Requirement 9 stays open until T008.

16.3a, closed on the T100 follow-on

16.3a is "Carried out by T100", and batch P's addendum adds "Carried out by a T100 follow-on openDox-code PR (claim 5982447319), which is no task of plan 034 and lands before T087":

  • T100 landed as openDox-code#82 → 38d3350e.
  • The follow-on landed as openDox-code#86 → 651c35fe (09:49:55Z). It is a squash whose tree is that of its head aecac805, and it landed before T087 (openDox#18, 11:58:27Z) and T101.
  • It carries out Brett Heap's 5982436447, item 2, and 5983805990, with the holder's 5984069416, 5985046107 (C1 to C5) and 5985553609, and the holder's A8.
  • Its body records the falsifier: F16.1's block as batch P amends it, run by T089's runner at aecac805, exits 0 with 532 passed in tests/test_model_binding_trust.py. The six test_F16_1_batch_p_* tests are present at 651c35fe.
  • The open trust-time existence check, and the follow-ons of openDox-code#86's eighth Copilot review, move no box of release 1. 16.3a's own text says so for the first, and the holder's 5992038800 for the second.

Left open, by T097's own text

  • 9.5, 11.0, 11.1 and F11.1, for the arc's close (T090 to T093), and F9.2, until T008. Nothing else is held.

T097 is ticked

T097's task is "Tick #1144's release-1 boxes, each with its evidence note: the 63 in the table below." All 63 are ticked, so f94be2fd ticks T097 in the same commit as 10.3, the last box, as lane 4's brief requires.

The merges of main

Gates, at main 8a37a087 and at this head 9713f991, in one full clone named openxFactory

gate main 8a37a087 this head 9713f991
python3 scripts/validate-openspec-cli-pin.py --all --no-cache (CI's form; pinned 1.12.0) rc 0, 110 passed, 1 failed (111 items), 0 UNDISPOSITIONED failures, 1 accepted exception (add-chain-attestation) identical output, but for the pinned CLI's temporary directory
… --change add-neutral-product-standalone-operability — rc 0, 1 passed, 0 failed, validated --strict clean
scripts/doc-health.py --single-repo . --as-of 2026-10-05 31 critical, 22 error, 69 warning, 20 info, 0 new regressions byte-identical report
proposal-support.py . verify add-neutral-product-standalone-operability ok identical
validate-sequenced-after.py . --ledger-diff 229 rows, consistent identical
validate-code-surface.py, validate-target-release.py, validate-pin-registrations.py, validate-document-catalog.py rc 0 each (the catalog: 0 errors, 0 warnings) identical
pytest -m "not postgres" over tests/doc-health, sequenced_after, proposal-support, former_id_arrival, citation_remainder, signed_execution_chain, openspec_cli_pin, target_release, code_surface, packet_reference, scope_globs the 8 below fail identically 8 failed, 3427 passed, 1 skipped, the same 8 nodes as at 10b6ba05

The 8 failures are environmental, the same 8 at every head. Each needs the openDox and openXdox legs initialized, as CI's pytest-suite does, or a reachable pinned validator: test_ideation_readiness.py (3), test_readiness_dispatch.py (1), test_sentinel_vocabulary.py (2), test_status_reader_real_lines.py (1) and test_chain_reader.py (1). Re-run alone at main 8a37a087, they read 8 failed with the same messages. The venv was built by CI's own pip install --require-hashes -r requirements/hermes-runtime-contracts.lock, with LANG=C.UTF-8 and every GIT_*/XF_* variable unset. The earlier rounds read the same: main c8d80020 against 76d8ff37, b7f49673 and e2efb3a1, main ba6bb870 against bd32ea9c, cf2c6bdc, 5870cd00 and 23665543, main 20ce593e against 935164af, main a43845e6 against e873b25f, and main f2acc88c against 10b6ba05. At f94be2fd the local run was stopped at 31% on a session swap, and CI's pytest-suite passed there. No test, script or workflow reads plan 034's files (git grep over tests, scripts and .github finds none).

Also checked, on the inserted text at 9713f991:

CI and review

  • CI at this head 9713f991: every check-run but Sourcery's completed success, the 8 required among them (pytest-suite 17:54:16Z to 18:15:30Z). Sourcery's reads skipped. lane-line re-ran on each edit of this description and passed.
  • CI at f94be2fd: every check-run completed success, the 8 required among them (pytest-suite 17:13:56Z to 17:31:21Z).
  • CI at 10b6ba05: every check-run completed success (pytest-suite until 17:04:09Z).
  • CI at e873b25f: every check-run completed success, the 8 required among them (pytest-suite 15:57:01Z to 16:22:01Z). lane-line re-ran on each edit of this description and passed.
  • CI at 935164af, 23665543, cf2c6bdc, bd32ea9c and e2efb3a1: every check-run completed success, the 8 required among them (signed-execution-chain-gate, lane-line, former-id-arrival-gate, openspec-cli-pin, wallet-validation, pytest-suite, release-tag-gate, openxdox-consumer-gate). At 5870cd00, 23665543's run superseded its pytest-suite.
  • Copilot at 76d8ff37: one finding, r4179674026: F7.2's note named no landing commit for T058 or T063. b7f49673 names them (openDox-code#68 → 047bb4fa, openxFactory#1218 → a883bbf6), and 7.1's changelog landing too (openDox#15 → 66758438). The thread is answered and resolved.
  • Copilot at b7f49673: no new finding, and one "previously missed" note: that 5.4a's note wrongly said T059's run deselected the three pre-arc cases. The note was right, and comment 5985467371 gives the records. The cited lines are T059's Falsifier line, written before it landed; T059's Landed record and openXdox-code#35's body (rulings 4 and 6, and its F5.2 run) record the deselections. e2efb3a1 makes the note exact and cites them.
  • Copilot at 5870cd00: "Findings: None", and one "previously missed" low note: 5.5's note read as though consumer_reach "names the task lists". 23665543 says what it meant, that the seven consumer_reach names T055 lists are retired.
  • Copilot at e2efb3a1, bd32ea9c, cf2c6bdc and 23665543: "Findings: None". Where its status line read "Changes recommended", it named only what this DRAFT waits for.
  • Copilot at 935164af: one finding, r4185667778: 5.4a's note read "the seven red at both pins". e93ab9fb reads "the seven reds", and the thread is answered and resolved.
  • Copilot at e873b25f: one finding, r4186043242: this description was stale after T089's ticks. It is refreshed here, and the thread is answered and resolved. A second request at that head registered no new review.
  • Copilot at 10b6ba05: "Findings: None".
  • Copilot at f94be2fd: two findings, review 5418196219. r4186761333: plan 034's opening status still said batch P was to land, against this PR's T007 tick. 9713f991 says that all sixteen batches have landed, A (T007 batch A: record #1144's ruled amendments (F3.1, 2.2, 3.2, 4.3, 10.1, 11.0, 11.1, F11.1) #1171) to P (T007 batch P: the rulings of 5982436447 in #1144 (12.4a's hint line and history limit, 16.3a's in-repo refusal and served-repo limit, F16.1's note) and the T100 follow-on (plan 034) #1230), and that T007 is closed. r4186761413: this description was stale at e873b25f. It was refreshed at 17:16:28Z, while that review ran, and is refreshed again here for 9713f991. Both threads are answered and resolved.
  • Copilot at 9713f991: review 5418607294, on the push. Its one open item was r4186761413, this description, refreshed here for 9713f991. Its two "previously missed" low notes, at plan 034's lines 129 ("whose message lacks") and 4496 ("the drive"), need no change, because each phrase is the wording of the record it cites, T091's and oracle-verdict.md's. Comment 6000157580 gives the records. A request through the reviewer API, at 18:01:14Z, registered no new review at this head.
  • Sourcery posted a reviewer's guide, and no comment on the diff. At 17:16:25Z it reported its review budget used, and its check-run reads skipped at f94be2fd and at 9713f991.

🤖 Generated with Claude Code

…s, and record research R16 (plan 034)

Bookkeeping for plan 034's T097, its "Now" part, delegated by lane
openxfactory-4 (#656 comment 5984809190). Measured at openxFactory main
c8d8002, with every landing re-read from the forge:

- 55 boxes of #1144's tasks.md whose realizing tasks have all landed are
  ticked. Each carries its evidence note as the item's last paragraph, in
  the form of 1.8 and 3.0: a bold dated lead, the tasks, the landings
  (repo#n -> sha8) and the falsifier's quoted result, from the phase
  checkpoints and the F10.1, F13.1 and F16.1 run records.
- Research R16's non-normative corrections are recorded: items 1, 2, 4
  and 5 in the notes of 2.5, 9.4, 3.2 and 10.1, and item 7 in 2.1's note.
  Items 3 and 6 are notes on boxes that stay open: 4.3 until T086, and
  9.5 until the arc's close. Item 8 is already 3.0's own note.
- Left open by T097's own text: 9.5, 11.0, 11.1 and F11.1 (the arc's
  close) and F9.2 (after T008). Still pending: 3.3, 4.3, F4.1, F5.2, 9.2,
  10.3, 16.3a and F16.1. 16.3a and F16.1 are held to Brett Heap's rulings
  5982436447 and 5983805990, which batch P records.

No other line moves: the diff is 55 checkbox flips and insertions only.
box_census.py: 105 [ ], 11 [x] and 9 [~] before; 50, 66 and 9 after.

Lane: openxfactory-3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 4, 2026 22:36
@sourcery-ai

sourcery-ai Bot commented Oct 4, 2026

Copy link
Copy Markdown

Reviewer's Guide

This documentation-only PR records release-1 bookkeeping for plan 034 by ticking 55 realized task boxes and appending dated evidence and R16 corrections, while deliberately leaving eight dependency- and arc-close-dependent boxes—and T097 itself—unticked until the stated follow-up landings arrive. The diff is intended to be insertion-only aside from checkbox changes, with validation showing no behavioral or ratified-text changes.

Sequence diagram for bookkeeping merge and final T097 tick

sequenceDiagram
    participant Lane4
    participant Main
    participant BatchP
    participant FollowUps
    participant Tasks

    Lane4->>Main: merge latest main
    Main-->>Lane4: updated tasks.md
    Lane4->>BatchP: resolve 16.4 conflict
    BatchP-->>Lane4: retain batch P text
    Lane4->>FollowUps: wait for pending landings
    FollowUps-->>Lane4: final pending box lands
    Lane4->>Tasks: tick final pending box and T097
    Tasks-->>Lane4: release-1 bookkeeping complete
Loading

Flow diagram for release-1 bookkeeping and pending landings

flowchart LR
    A["Plan 034 tasks.md at main"] --> B["T097 bookkeeping PR"]
    B --> C["55 boxes ticked"]
    B --> D["55 dated evidence notes"]
    B --> E["2 R16 correction notes"]
    B --> F["8 boxes remain pending"]
    F --> G["T086"]
    F --> H["T089"]
    F --> I["T098"]
    F --> J["T099"]
    F --> K["T100 follow-on"]
    F --> L["T083 and batch P"]
    G --> M["T097 becomes tickable"]
    H --> M
    I --> M
    J --> M
    K --> M
    L --> M
Loading

File-Level Changes

Change Details Files
Bookkeeping updates the release-1 task ledger with completion evidence and researched corrections.
  • Ticks 55 completed release-1 task and falsifier boxes.
  • Adds dated landing notes with repository PRs, merge SHAs, test results, and cited evidence.
  • Records eight non-normative R16 corrections, including two notes on boxes that remain open.
  • Leaves ratified task text unchanged and intentionally holds pending boxes awaiting T086, T089, T098, T099, T100 follow-on, and batch P.
openspec/changes/add-neutral-product-standalone-operability/tasks.md
The bookkeeping preserves the PR's staged landing and merge constraints.
  • Does not tick T097 until all eight pending release-1 boxes land.
  • Documents the expected batch-P merge conflict at 16.4 and its prescribed resolution.
  • Keeps plan-034 content outside the diff while recording the remaining release-1 dependencies.
openspec/changes/add-neutral-product-standalone-operability/tasks.md
The updated ledger is validated as an insertion-only documentation change.
  • Regenerates the task census from 11 to 66 checked boxes while preserving blocked and open states.
  • Verifies evidence references, landing dates, merge commits, line width, and absence of ratified-line changes.
  • Reports unchanged validation, documentation-health, support, sequencing, catalog, and test-gate results relative to main, with eight known environmental failures.
openspec/changes/add-neutral-product-standalone-operability/tasks.md

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

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Eight pending boxes and T097 remain open until the declared prerequisite landings are incorporated.

Review effort: Balanced
Findings: None

What changed in this PR

Records release-1 bookkeeping for the standalone openDox operability change.

Changes:

  • Marks 55 realized release-1 boxes complete.
  • Adds landing evidence and R16 corrections.
  • Preserves eight pending boxes for prerequisite landings.
File Description
openspec/​changes/​add-neutral-product-standalone-operability/​tasks.md Updates release-1 completion and evidence records.

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

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The F7.2 evidence note omits the required landing commit references.

Review effort: Balanced
Findings: 1 Low severity

Open (1)

…r4179674026) (plan 034)

F7.2's note named T058 and T063 but neither landing commit, and 7.1's note
left out T053's changelog landing (openDox#15). Both now name every landing
as repo#n -> sha8, as the other notes do: F7.2 gains openDox-code#68 ->
047bb4fa and openxFactory#1218 -> a883bbf, and 7.1 gains openDox#15 ->
66758438. A check that every ticked note names each of its tasks' landings
now finds none missing. The census is unchanged: 50 [ ], 66 [x], 9 [~].

Lane: openxfactory-3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 4, 2026 22:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The 5.4a evidence note contradicts the recorded T059 and T063 test executions.

Review effort: Balanced
Findings: None

Resolved since last review (1)
Previously missed (1)

In code that hasn't changed since last review

Low severity Correct inaccurate claim that three pre-arc cases were deselected

openspec/​changes/​add-neutral-product-standalone-operability/​tasks.md:840

This reverses the recorded T059 run. Plan 034 says only the two consumer-schema cases in test_snapshot.py were deselected and every other suite ran whole (specs/034-opendox-standalone-operation/tasks.md:1620-1630); T063 then reports the three pre-arc cases as executed and red (:1965-1978). Saying those three were deselected makes this evidence note inaccurate.

… note (Copilot at b7f4967) (plan 034)

Copilot's review at b7f4967 read 5.4a's note against T059's Falsifier
line, the plan written before the landing (two schema cases deselected,
every other suite whole). The note follows the landing instead: T059's
record in plan 034 and openXdox-code#35's body (its rulings 4 and 6)
deselect the seven consumer-schema reds red at both pins, the two schema
cases of tests/test_snapshot.py, and the three pre-arc reds of
tests/test_session_snapshot.py, which T063 then quotes red (CP2 section 2).
The note now gives those counts and cites its source, so it no longer
reads as a contradiction. No other line moves, and the census is
unchanged: 50 [ ], 66 [x], 9 [~].

Lane: openxfactory-3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 4, 2026 23:11
@brettheap

brettheap commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor Author

Lane: openxfactory-3 (openXfactory-3)

Copilot's review at b7f49673, "previously missed": 5.4a's note and T059's deselections. The review had no thread, so this comment answers it.

The note described T059's run as it landed, and that is correct. The cited lines, plan 034 tasks.md:1620-1630, are T059's Falsifier line. That line is the plan written before the landing, and it deselects only tests/test_snapshot.py's two schema cases. The landed run deselected more, and three records say so:

  • T059's Landed record in plan 034: "Seven consumer-schema reds that fail at both pins are deselected in T059's F5.2 run … Three pre-arc reds in tests/test_session_snapshot.py are deselected with the reason "red at both pins; pre-arc carve residue; not the arc's", and stay open for F5.2 whole (T061, T063)."
  • openXdox-code#35's body, rulings 4 and 6, and its F5.2 run: test_session_snapshot 20 passed, 3 deselected, test_snapshot 15 passed, 2 deselected, test_generator 41 passed, 4 deselected, test_snapshot_determinism 5 passed, 1 deselected, test_snapshot_registry 38 passed, 2 deselected.
  • :1965-1978 is T063's later run of F5.2 whole. There the three pre-arc cases do run, and are red, as RULED 5962785556 (CP2 § 2). That run is T063's, not T059's.

The terse wording invited the misreading, so e2efb3a1 makes the note exact: the seven consumer-schema reds, tests/test_snapshot.py's two, and the three pre-arc reds of tests/test_session_snapshot.py, citing T059's record and openXdox-code#35's rulings 4 and 6, with T063's red quote. No other line moves.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

It awaits batch P, T096, and T099, after which the reviewed ledger will materially change.

Review effort: Balanced
Findings: None

…the ticks and notes re-applied on main's text (plan 034)

Batch P landed on main as ba6bb87, the squash of its head 051f994. In
#1144's tasks.md it inserts addenda at the ends of 12.4a, 16.3a and F16.1
and changes no other line: no box line moves and no box changes state.

The merge conflicted in one place, 16.4's checkbox line, which follows
batch P's insertion at the end of 16.3a. Main's text is taken for every
hunk batch P changed, and this PR's 55 ticks and 57 notes are re-applied
with the same tool (apply_t097.py). The result is byte-identical to that
tool's fresh re-apply onto main's file, and to the conflict resolved by
the rule (batch P's text, then 16.4's [x]). 16.3a and F16.1 keep main's
text and stay open.

Every box was re-classified against ba6bb87, with every landing PR
re-read: 55 CLOSED-LANDED, 8 PENDING, 5 LEFT-OPEN and 2 ALREADY-TICKED,
as before, and every note is unchanged. box_census.py reads 105 [ ],
11 [x] and 9 [~] at main, and 50, 66 and 9 after.

Lane: openxfactory-3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 5, 2026 00:52

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The draft remains incomplete pending eight boxes and final T097 bookkeeping.

Review effort: Balanced
Findings: None

…651c35fe), with its evidence note (plan 034)

16.3a is carried out by T100 (openDox-code#82 -> 38d3350e) and, as batch
P's addendum on main says, by a T100 follow-on openDox-code PR (claim
5982447319) that lands before T087. The follow-on landed on 2026-10-05 as
openDox-code#86 -> 651c35fe, a squash whose tree is that of its head
aecac805, and T087 has not landed. It carries out Brett Heap's rulings
5982436447 (item 2) and 5983805990, with the holder's 5984069416,
5985046107 (C1 to C5) and 5985553609, and the holder's A8. Item 3 (A14)
changes no code.

The note quotes the falsifier as the follow-on's body records it: F16.1's
block as batch P amends it, run by T089's runner against aecac805, exits 0
with 532 passed in tests/test_model_binding_trust.py, batch P's cases among
them. F16.1 itself stays open, since its Box accounting row is T083 and
T089, and T089 runs it whole at T087's pin.

The file is a fresh re-apply onto main's (ba6bb87) with the regenerated
decisions. Against the previous head it moves 16.3a's checkbox and inserts
its note, and nothing else. box_census.py reads 105 [ ], 11 [x] and 9 [~]
at main, and 49, 67 and 9 after. Seven boxes stay pending: 3.3, 4.3,
F4.1, F5.2, 9.2, 10.3 and F16.1.

Lane: openxfactory-3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 5, 2026 10:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

It remains intentionally incomplete pending T096, T099, the remaining seven boxes, and T097’s final tick.

Review effort: Balanced
Findings: None

…8), with their evidence notes (plan 034)

T086 landed on 2026-10-05 as openXdox-code#37 -> 56e1c238, a squash
whose tree is that of its head 57bebd8b. Its pin is openDox-code
dede32b4, the commit T087 pins (openDox root e1e3a3c3).

4.3 closes with the last of the nineteen reaches into openXdox. T084
(openDox-code#77) routed openDox's half and retired consumer_reach.py.
T086 contributes openXdox's governed columns at those seams and empties
OPENDOX_BACK_IMPORTS to (0, 0) in the same landing that moves the pin, as
4.3 asks. At that pin tests/test_dependency_direction.py reads 17 passed,
with an empty census (openXdox-code#37's body).

9.2 closes in phase 3 (batch F's addendum), with batch B's note. The
whole suite runs less the declared exclusion, which at 56e1c238 holds 66
files carrying 68 reasons: 60 doc_health, 3 status-exemption-rail and 5
openxfactory-contracts. CI's validate at that PR's head prints it as an
open extraction and reads selected=1099 passed=1095 skipped=4.
Requirement 9 stays open for openXdox-code until T008.

4.3's tick note now carries research R16 item 3, which was its open-box
note. 9.5's R16 item 6 note now says that T086 moved the pin to
dede32b4, which it did. F5.2 still waits on T089. 10.3 still waits on
T099, now that T076 (openDox#17 -> 504324de) and T101 (openDox-code#78
-> d59f3f26, openDox-code#79 -> dede32b4) have landed.

The file is a fresh re-apply onto main's (ba6bb87, unchanged).
box_census.py reads 105 [ ], 11 [x] and 9 [~] at main, and 47, 69 and 9
after. Five boxes stay pending: 3.3, F4.1, F5.2, 10.3 and F16.1.

Lane: openxfactory-3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 5, 2026 12:47
fab575a), with their evidence notes (plan 034)

T089, phase 3's checkpoint, landed on 2026-10-05 as openxFactory#1238
-> fab575a. Its record, evidence/checkpoint-phase3.md, runs each block
as extracted from #1144 at the commits phase 3 pinned, and each passes:

- F4.1 (plan 034's box map: F4.1 -> T089), run unchanged at openDox-code
  dede32b4, T087's pin, exits 0: "1 passed", "1 passed", then "no
  deferred reach names the consumer or the publisher" (section 1).
- F5.2 (F5.2 -> T086, T089; RULED 5962785556, item 1: the box closes at
  phase 3's checkpoint), whole as batches C, F, G and K amend it, at
  openXdox-code 56e1c238 with openDox-code dede32b4 installed over it
  and OPENXFACTORY at 3690848, exits 0: the seven suites pass whole,
  "test -s" holds, and the --chains step prints five admitted: lines,
  then "ok: 5 protected edit(s), each entered and holding" (section 5).
  T086 (openXdox-code#37 -> 56e1c238) had repaired the three pre-arc
  reds that T063 quoted.
- F16.1 (F16.1 -> T083 at 38d3350e, T089 with batch P's cases at T087's
  pin), with batch M's line, exits 0 at dede32b4, which carries the T100
  follow-on (openDox-code#86 -> 651c35fe) and with it batch P's cases:
  "24 passed", the dialect line, "no model configured: the catalog
  offers nothing", "83 passed" and "532 passed" (section 4). T083's run
  at 38d3350e is quoted beside it.

The file is a fresh re-apply onto main's (a43845e, the same as
20ce593's for this file), in which only these three boxes changed.
box_census.py reads 105 [ ], 11 [x] and 9 [~] at main, and 43, 73 and 9
after. One box stays pending, 10.3, for T099.

Lane: openxfactory-3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 5, 2026 15:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The PR description and census are stale, and final T096/T099 bookkeeping remains pending.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
Resolved since last review (1)

brettheap and others added 2 commits October 5, 2026 16:30
main moved from a43845e by one landing: openxFactory#1240 (f2acc88,
Q-GRC-4's second bookkeeping). It does not touch #1144's tasks.md, so
the merge is clean, and this PR's file is unchanged by it: it is still
byte-identical to a fresh re-apply of the decisions onto main's file.
The other files take main's text.

Lane: openxfactory-3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…notes (plan 034)

The notes cited the phase 1 and phase 2 checkpoint records as "CP1 § n"
and "CP2 § n", the shorthand of lane openxfactory-4's evidence table,
which #1144 never defines. Lane openxfactory-4 ruled on 2026-10-05 at
16:25:25Z that each citation names its record, as the phase-3 notes
name evidence/checkpoint-phase3.md: non-normative, inside T097's R16
scope. The 26 citations, in 25 notes, now read
evidence/checkpoint-phase1.md (T049, openxFactory#1204) and
evidence/checkpoint-phase2.md (T063, openxFactory#1218), the records on
main. No other word changes, and no box changes state; the paragraphs
are rewrapped at 80 characters.

The file is a fresh re-apply onto main's (f2acc88, the same as
a43845e's for this file). box_census.py reads 43 [ ], 73 [x] and 9 [~],
as at e873b25.

Lane: openxfactory-3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 5, 2026 16:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

T099, box 10.3, and T097 remain intentionally pending before this draft can be approved.

Review effort: Balanced
Findings: None

Resolved since last review (1)

brettheap and others added 2 commits October 5, 2026 16:47
main moved from f2acc88 by one landing: openxFactory#1241 (8a37a08,
T096, AT-R1 at openDox-code 389e5a4a, with T095's acceptance job; it
ticks T096). It does not touch #1144's tasks.md, so the merge is clean,
and this PR's file is unchanged by it: it is still byte-identical to a
fresh re-apply of the decisions onto main's file. Plan 034's tasks.md,
plan.md, README.md and evidence/at-r1/ take main's text.

Lane: openxfactory-3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…007 and T097 (plan 034)

The last landings of release 1: T095 (openDox-code#75 -> 389e5a4a, X),
T096 (#1241 -> 8a37a08) and T099, the publish at the cut and the root
README PR (openDox#19 -> d77f8cbf).

#1144's tasks.md: 10.3 is ticked with its note. T076 (openDox#17)
documents the install and the start, T101 (openDox-code#78 and
openDox-code#79) adds the release workflow and the version bump, and
T099 publishes that commit. The annotated tag v0.1.0 names dede32b4.
openDox-code's release run 37339111713 succeeds in all four jobs, among
them "publish to PyPI" with its step "PyPI serves the files the build
job verified". PyPI serves the wheel and the sdist of opendox 0.1.0. The
file is a fresh re-apply onto main's (8a37a08, the same as f2acc88's
for this file). box_census.py reads 105 [ ], 11 [x] and 9 [~] at main,
and 42, 74 and 9 after: all 63 release-1 boxes that a realization task
closes.

Plan 034's tasks.md, on the holder's rulings (the delegation of
2026-10-05T16:37:31Z, and 5999175092, (a), (a) and (a)), each with a
Landed or Done bullet in #1226's form:
- T095: the harness, its dispatched acceptance run 37333323932 at X;
- T099: P against X, the tag, the release run, PyPI's two files and
  openDox#19;
- T002: the 45 claims cited, and the five slices without one named, in
  T091's form; nothing is back-filled;
- T007: batches A to P landed, and its status sentence says so;
- T097: the census, R16's corrections, and two non-normative notes, the
  T096 refusals ("refused by name, with no browser signal") and the
  reachability of #1241's first commit, f632d70.
92 of plan 034's 96 tasks are ticked; T090 to T093 tick at the arc's
close.

Lane: openxfactory-3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 5, 2026 17:13
@brettheap brettheap changed the title DRAFT (after T096, T099 and batch P): T097, release 1's bookkeeping (plan 034) T097, release 1's bookkeeping (plan 034) Oct 5, 2026
@brettheap
brettheap marked this pull request as ready for review October 5, 2026 17:16
@sourcery-ai

sourcery-ai Bot commented Oct 5, 2026

Copy link
Copy Markdown

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 4 days and 3 hours by commenting @sourcery-ai review. Upgrade to get a review now.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The PR description and an internal status summary contradict the newly completed bookkeeping.

Review effort: Balanced
Findings: 2 Low severity

Open (2)

…nded (Copilot at f94be2f) (plan 034)

Copilot's review at f94be2f (r4186761333) found that the opening
status of plan 034's tasks.md still said fifteen of T007's sixteen
batches had landed, and that batch P lands before T087, while this PR
ticks T007. On the holder's ruling (fix now), that summary now reads
that all sixteen batches have landed, from A (#1171) to P (#1230, on
Brett Heap's words 5982436447 and 5983805990 and the holder's rulings),
that batch P landed before T087, and that T007 is closed. No other line
moves.

Lane: openxfactory-3

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 5, 2026 17:54

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The extensive cross-repository provenance requires final human verification, and two CI checks remain in progress.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
Resolved since last review (1)
Previously missed (2)

In code that hasn't changed since last review

Low severity Fix plural agreement in relative clause

specs/​034-opendox-standalone-operation/​tasks.md:129

The antecedent is plural (“two landings”), so the relative clause should also be plural: “whose messages lack.”

Low severity Clarify artifact reference as the browser driver

specs/​034-opendox-standalone-operation/​tasks.md:4496

“Drive” refers to no artifact here; the evidence names the browser driver (at_r1_browser.py). Change this to “driver” so the provenance note identifies the file whose bytes ran.

@sonarqubecloud

sonarqubecloud Bot commented Oct 5, 2026

Copy link
Copy Markdown

@brettheap

Copy link
Copy Markdown
Contributor Author

Lane: openxfactory-3 (openXfactory-3)

Copilot's review at 9713f991 (5418607294), "previously missed": two low notes, on text f94be2fd added to plan 034's tasks.md. Neither needs a change, because each phrase is the wording of the record it cites.

  • Line 129, "the two landings whose message lacks 11.0's line". T002's Done bullet repeats T091's own words. T091 (line 4309 at 9713f991) names openDox-code#37 and openDox-code#44 as "the two landings whose message lacks the line". The singular is distributive, one message per landing, and a change here would make T002 cite T091 in words T091 does not use.
  • Line 4496, "the runbook and the drive that ran". "The drive" is the name evidence/at-r1/oracle-verdict.md gives at_r1_browser.py: its § "How it ran" has the bullets The runbook. (run-at-r1.sh, line 124) and The drive. (at_r1_browser.py, line 167). The note names the two files by the verdict's own headings. The note's next sentences then name each by its sha256: the bytes that ran, 702e2a4b… for run-at-r1.sh and 763b5919… for at_r1_browser.py, as the verdict records them, and the reviewed files that SHA256SUMS lists at 8a37a087.

Plan 034's opening status, Copilot's one finding at f94be2fd (r4186761333), is fixed in 9713f991. The description is refreshed for 9713f991 (r4186761413). Both threads are answered and resolved.

@brettheap

Copy link
Copy Markdown
Contributor Author

READY at 9713f99 — Lane: openxfactory-4 (openXfactory-4-openDox_extraction)

Holder: lane openXfactory-3's D1, T097, release 1's bookkeeping. It is the last landing of release 1. It touches openspec/changes/, so it lands under a Rule 6 LANDING/LANDED window. It carries no Arc: trailer (R1Q20 (a)) and no closing keyword.

@brettheap

Copy link
Copy Markdown
Contributor Author

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

LANDING — lane openxfactory-4, session faabad8b-8c31-4958-abd5-b35bdc9cd282@Eagle, 2026-10-05T18:17:07Z, PR #1233 into opensoft/openxFactory main

@brettheap
brettheap merged commit 12753f2 into main Oct 5, 2026
17 checks passed
@brettheap

Copy link
Copy Markdown
Contributor Author

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

LANDED — lane openxfactory-4, 2026-10-05T18:17:36Z, PR #1233 → 12753f2 (opensoft/openxFactory main; squash)

brettheap added a commit that referenced this pull request Oct 5, 2026
Takes #1233 (plan 034 bookkeeping) so the branch and the main baseline
of the full-suite comparison share one base.

Lane: openxfactory-5 (openXfactory-5)
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
brettheap added a commit that referenced this pull request Oct 5, 2026
Brings in origin/main 12753f2 (#1233, plan 034 docs only). It overlaps nothing
in the dashboard lane files or the product pins.

Lane: openxfactory-5 (openXfactory-5)
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.

2 participants