Skip to content

Archive correct-naming-families-count: doctrine #1270 → e83259b4, packet closed - #1275

Merged
brettheap merged 6 commits into
mainfrom
archive/correct-naming-families-count
Oct 8, 2026
Merged

brettheap merged 6 commits into
mainfrom
archive/correct-naming-families-count

Conversation

@brettheap

@brettheap brettheap commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Lane: codeXfactory-5

Authority

What moved

  1. The archive act. 60ea054a closes owner boxes 1.1 and 1.2 as DISCHARGED BY WORD, each citing its own word and time, and says each tick records Brett Heap's act, not the agent's; the section 1 heading is unchanged. It ticks 4.4 on its evidence: at Propose correct-naming-families-count: the naming families are those the pinned policy declares, restated in canon by neither number nor list #1266's final head d296570b, every required context passed, no review was requested, and there were 0 threads. 5c28ede6 ticks 6.1 and 6.2 ahead of the run, because the wrapper refuses an open box. 960e0d5b is the governed wrapper's act: TZ=UTC python3 scripts/proposal-support.py . archive correct-naming-families-count --yes, exit 0, 2026-10-08T20:07:29Z, over the pinned @fission-ai/openspec@1.12.0.
    • ORIGIN RETAINED correct-naming-families-count (declaration unchanged since the ratifying commit a38585e97962)
    • Applying changes to openspec/specs/project-repo-schema/spec.md: ~ 2 modified
    • Totals: + 0, ~ 2, - 0, → 0
    • archived as '2026-10-08-correct-naming-families-count'
  2. The archive directory. openspec/changes/archive/2026-10-08-correct-naming-families-count/. The six packet files move as R100 renames in 960e0d5b, and tasks.md is ticked inside the archived copy. 7995a20e records the run under 6.1.
  3. Promoted requirements, onto openspec/specs/project-repo-schema/spec.md. Both blocks are byte-identical between the archived delta and canon, each extracted by heading and hashed:
    • A project's repositories are named <Project>, <Project>-spec and <Project>-code: 1,821 → 2,871 bytes.
    • The naming families are governed by the pinned standard, and a descendant form is a claim that needs a declared pin: 2,593 → 6,148 bytes.
    • The other nine requirement blocks are byte-identical, and the eleven titles keep their order. Canon numstat is +35 −14, and no other file under openspec/specs/ changes.
    • No block was brought forward. Canon was last touched by 19d008723 (Cut contract-v3.1: add-project-repo-schema realized; archive it #616), and git diff 16779816 80f47483 of the file is empty.
  4. README. The OpenSpec Records entry moves from Active to Archived, and its two links (the proposal and review/ratification-2026-10-07.md) are repointed to the archive path.
  5. Links left as written.
    • docs/project-repo-schema.md:24 is left as written. It names openspec/changes/correct-naming-families-count/tasks.md in code type, not as a link, inside the Amended by: header of a Status: ratified document that records how the amendment was realized. The same document's links to add-project-repo-schema (:36, :508) were likewise left when that change archived in Cut contract-v3.1: add-project-repo-schema realized; archive it #616.
    • Nothing in docs/, ideation/ or specs/ links to the active path.
    • The historical records that say "four" (design.md D7) are not edited.
  6. Ledger. tests/sequenced_after/corpus-ledger.yaml, one line: a727715a seeds the row by machine (--seed-ledger --moved-by '#1275': 235 rows, 1 moved, correct-naming-families-count active → archived, moved_on 2026-10-08) and replaces the README entry's ARCHIVE-PR-NUMBER-PENDING marker with a link to this pull request. a6e2a841 records 6.2's runs under the archived tasks.md 6.2.

6.2, before and after

Both sides ran in one scratch clone named openxFactory, with the submodules CI's pytest-suite initializes (openXwallet, then openXdox and openDox recursively). Before is main 80f47483. After is this pull request's head a6e2a841; a727715a gave the same results.

Gate Before, main 80f47483 After
validate-openspec-cli-pin.py --all --strict exit 0; 114 passed, 1 failed (115 items) exit 0; 113 passed, 1 failed (114 items)
validate-sequenced-after.py . (incl. archive-date-vs-commit) exit 0; 48 active changes exit 0; 47 active changes
validate-sequenced-after.py . --ledger-diff exit 0; 235 rows consistent exit 0; 235 rows consistent
doc-health.py --single-repo . --as-of 2026-10-08 31 critical, 26 error, 69 warning, 19 info 31 critical, 26 error, 69 warning, 19 info; 0 new regressions
  • CLI. With the clone path normalized, the two outputs differ only in the totals line. The item that left is this change, and every finding line is identical. The one failure on both sides is the accepted add-chain-attestation exception (Brett Heap, 2026-09-05, "take exit 2").
  • Ledger. On the archived tree before the row was seeded, --ledger-diff exits 1 on exactly this row (state: ledger 'active', live 'archived'); the seed repairs it.
  • doc-health. The reports differ in two lines, both this archive's own effect: canon words rise by 715 (382,979 → 383,694), which is the promoted text, and the promoted-specs row moves with them. No finding names a file this archive touches.

Lands by MERGE commit under Rule 6 on Brett Heap's word "archive the naming families change".

Copilot review not requested (Brett Heap, 2026-10-06).

🤖 Generated with Claude Code

brettheap and others added 4 commits October 8, 2026 20:06
…, tick 4.4

Owner boxes 1.1 and 1.2 are closed as DISCHARGED BY WORD, each citing its
own word and time: 1.1 "ratify as recommended when the PR opens" (RULED
2026-10-07T09:47:04Z), 1.2 "merge the naming families PR when green"
(RULED 2026-10-07T09:59:10Z, #1266 -> a38585e). Each tick records Brett
Heap's act, not the agent's, under his selection of 2026-10-08, verbatim
"Yes, here and prefer-triad (Recommended)" (RULED 2026-10-08T20:05:35Z),
as archive #953 did. The section 1 heading is unchanged.

4.4 is ticked on its verified evidence: every required context green at
#1266's final head d296570, no review requested, 0 review threads.

Lane: codeXfactory-5
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…rapper's run

The wrapper refuses an archive while any open box remains, so 6.1 and 6.2
are ticked before the run, as #1252 did. 6.1 records that 5.1 is merged
(#1270 -> e83259b) and that canon has not moved under either block since
19d0087 (#616), so no block is brought forward. A dated paragraph below
the section 4 open-box note records that every box is now ticked; the note
itself is kept as written.

Lane: codeXfactory-5
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…families onto project-repo-schema

The governed wrapper's act, on Brett Heap's word "archive the naming
families change" (RULED on the lane register at 2026-10-08T19:49:19Z):
TZ=UTC python3 scripts/proposal-support.py . archive
correct-naming-families-count --yes, exit 0, over the content-addressed
@fission-ai/openspec@1.12.0 pin. "ORIGIN RETAINED ... (declaration
unchanged since the ratifying commit a38585e)", "~ 2 modified",
"Totals: + 0, ~ 2, - 0, -> 0", archived as
'2026-10-08-correct-naming-families-count'.

Both MODIFIED requirements are byte-identical between the archived delta
and canon; the other nine blocks and the title order are unchanged. The
six packet files move as pure renames. The README Records entry moves from
Active to Archived, its two links repointed to the archive path, with the
ARCHIVE-PR-NUMBER-PENDING marker until the pull request exists.

Lane: codeXfactory-5
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The run's decisive lines and timing, the promoted blocks verified by
content (both byte-identical between delta and canon; nine others and the
title order unchanged; numstat +35 -14), provenance (six R100 renames,
origin unchanged since a38585e), and the links: the README entry's two
links repointed, docs/project-repo-schema.md:24 left as written.

Lane: codeXfactory-5
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sourcery-ai

sourcery-ai Bot commented Oct 8, 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 23 hours and 53 minutes by commenting @sourcery-ai review. Upgrade to get a review now.

…l request

python3 scripts/validate-sequenced-after.py . --seed-ledger --moved-by
'#1275': 235 rows, 1 moved (correct-naming-families-count, active ->
archived, moved_on 2026-10-08). The README Records entry's
ARCHIVE-PR-NUMBER-PENDING marker becomes a link to PR #1275.

Lane: codeXfactory-5
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sourcery-ai

sourcery-ai Bot commented Oct 8, 2026

Copy link
Copy Markdown

Reviewer's Guide

The PR archives the completed correct-naming-families-count change via the pinned governed wrapper, closes and documents its task/provenance state, promotes its two count-free naming-family requirements into canonical schema, and updates repository indexes and ledger metadata.

Sequence diagram for governed change archival

sequenceDiagram
    actor BrettHeap
    participant Wrapper as proposal-support.py
    participant Packet as Active change packet
    participant Canon as project-repo-schema spec
    participant README
    participant Archive as Archive directory

    BrettHeap->>Wrapper: archive correct-naming-families-count --yes
    Wrapper->>Packet: Verify tasks complete and origin retained
    Wrapper->>Canon: Promote 2 modified requirements
    Wrapper->>Archive: Move six packet files
    Wrapper->>README: Move record from Active to Archived
    Wrapper-->>BrettHeap: Archive completed with exit 0
Loading

File-Level Changes

Change Details Files
Archived the completed OpenSpec change through the governed wrapper and finalized its task/provenance records.
  • Moved the six proposal packet files into the dated archive directory as renames.
  • Closed owner, verification, and archive task boxes with recorded authority and execution evidence.
  • Recorded the pinned CLI archive run and preserved the ratification/origin metadata.
openspec/changes/archive/2026-10-08-correct-naming-families-count/proposal.md
openspec/changes/archive/2026-10-08-correct-naming-families-count/design.md
openspec/changes/archive/2026-10-08-correct-naming-families-count/specs/project-repo-schema/spec.md
openspec/changes/archive/2026-10-08-correct-naming-families-count/tasks.md
openspec/changes/archive/2026-10-08-correct-naming-families-count/.openspec.yaml
openspec/changes/archive/2026-10-08-correct-naming-families-count/review/ratification-2026-10-07.md
Promoted the count-free naming-family doctrine into the canonical project repository schema.
  • Removed the hard-coded family count and closed-list assumptions from two requirements.
  • Added language deferring family membership and count to the pinned naming policy.
  • Added replacement provenance markers and a scenario covering families introduced by later pins.
openspec/specs/project-repo-schema/spec.md
Updated repository records to reflect the change's archived status and canonical links.
  • Moved the OpenSpec Records entry from Active to Archived.
  • Repointed proposal and ratification links to the dated archive path.
  • Seeded the sequenced-after corpus ledger with the archive row.
README.md
tests/sequenced_after/corpus-ledger.yaml

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

Pinned CLI --all --strict 114/1 (115) -> 113/1 (114), finding lines
identical; validate-sequenced-after.py 48 -> 47 active, exit 0 both;
--ledger-diff 235 rows consistent both; doc-health 31/26/69/19 both, the
reports differing only by the promoted text's 715 canon words. Before on
main 80f4748, after at a727715, in one openxFactory-named scratch
checkout with CI's submodules.

Lane: codeXfactory-5
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sonarqubecloud

sonarqubecloud Bot commented Oct 8, 2026

Copy link
Copy Markdown

@brettheap

Copy link
Copy Markdown
Contributor Author

LANDING — lane codeXfactory-5, session ed23f049-7e99-4601-8a6d-760b6aeb5f26@Eagle, 2026-10-08T21:10:15Z, PR #1275 into opensoft/openxFactory main

Landing on Brett Heap's word "merge the archive PR when green" (2026-10-08), following his "archive the naming families change". Every check is green at head a6e2a84, including merge-master-approval; Sourcery skipped. Main has moved one commit (ideation/brainstorm files only), which doesn't overlap this PR. Merging by merge commit, as #1252 was.

Lane: codeXfactory-5

@brettheap
brettheap merged commit c8dde13 into main Oct 8, 2026
16 checks passed
@brettheap

Copy link
Copy Markdown
Contributor Author

LANDED — lane codeXfactory-5, session ed23f049-7e99-4601-8a6d-760b6aeb5f26@Eagle, 2026-10-08T21:10:44Z, PR #1275 into opensoft/openxFactory main → c8dde13

Merged at 2026-10-08T21:10:26Z by merge commit, on Brett Heap's word "merge the archive PR when green". This closes the window. correct-naming-families-count is archived.

Lane: codeXfactory-5

@brettheap
brettheap deleted the archive/correct-naming-families-count branch October 8, 2026 21:11
brettheap added a commit that referenced this pull request Oct 8, 2026
…#1277

Lane: openxfactory-4

main moved through #1272, #1275 (the archive of correct-naming-families-count)
and #1277 (a bot review-lane repin), which touch README.md, openspec/, the sweep
ledger and the review-lane contracts and workflows, and not this change's
directory. Section 4 names the new base and the commits since fc4fa0f; section
6 carries the re-measured numbers (pinned gate 113 passed of 114 items, ledger
235 rows). The doc-health report is again byte-identical to main alone.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
brettheap added a commit that referenced this pull request Oct 9, 2026
…23375303) (#1261)

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

Refs #656 (stays OPEN).

**THE RATIFICATION RECORD for `realize-doc-health-direction-arc`** (plan 038's T071), the doc_health direction arc's own change. It landed as a draft through #1247 → `51456835` and #1253 → `fc4fa0ff`. No requirement, decision or task text moves: the change has no spec delta (`skip_specs: true`). It lands through the holder's lander in a Rule 6 window; this actor merges nothing.

## The word

RULED by Brett Heap, 2026-10-06 at about 18:59Z, by interactive multi-choice, verbatim *"Ratify it (Recommended)"*, recorded at #656 comment [`6023375303`](#656 (comment)) (created 2026-10-06T19:00:45Z).

The option text he saw read *"I post your word on #656 and land the ratification record (Status: ratified, Ratified by) under a Rule 6 window. T072 can start right away; T074 and T075 then follow in order."*

## The form followed

`cd494e4c` / #1151, "Ratify add-neutral-product-standalone-operability on Brett Heap's word (#656 5815412869)", file for file and field for field. The holder's ruling at #656 [`6023619783`](#656 (comment)) confirms the one choice that differs from the change's as-filed text (below).

## What is ratified, and what the word authorizes

- **Ratified:** the change as landed on `main` at `fc4fa0ff` (#1253, merged 2026-10-06T18:56:42Z). Its change-directory tree, `02d362f6ebcaf49e18aa107f7ce2b3a409ca2275`, is identical at #1253's final head `e4f3f1db` and on current `main`, and the last commit on `main` that touches it is `fc4fa0ff`. Copilot's review at `e4f3f1db` read "Approval recommended", Findings: None.
- **D1–D7 stand as recommended.** `tasks.md` 1.2 provides that the word rules `design.md` § 11's D1–D7 as recommended unless it says otherwise, and the word does not. The record says that this applies 1.2's own rule and does not claim Brett Heap was shown the seven one by one.
- **Authorized:** per the ruling record, the realization in plan 038's order: T072 (opportunistic, ARC-6), then T074 and T075. None of it happens by the word itself.
- **Nothing archives or promotes.** The change stays ACTIVE until its realization is merged and green, and it gates neither release 2's close nor #1144's archive (ARC-Q3 (a)).

## What this PR changes

Six files. None of the changes is requirement or scenario text.

**`openspec/changes/realize-doc-health-direction-arc/proposal.md`**
- `Status: ratified`, with a `Ratified:` citation.
- A qualifier above the filing's "DRAFT. FILING IS NOT RATIFYING" paragraph, which is kept verbatim as the filing's record.
- A `## Ratification record` section.
- A read-at-ratification note after the D1–D7 list.

**`design.md` and `tasks.md`**
- `Status: ratified`, with a `Ratified by:` citation. This is the citation split #1144's record carried.
- `design.md` § 11: the same read-at-ratification note.
- `tasks.md` 1.1 (the filing, citing #1247 → `51456835` and #1253 → `fc4fa0ff`) and 1.2 (this act) are ticked. Boxes go from 0 `[x]` / 23 `[ ]` to **2 `[x]` / 21 `[ ]`**.
- 1.2 keeps its as-filed wording and gains a marked italic note on the one divergence (below), as #1151 did for its own 1.8.

**`.openspec.yaml`**
- `approved_by` / `approved_on` are ADDED after `proposed_on`. `kind`, `id`, `path`, `reason`, `proposed_by` and `proposed_on` do not move.

**New record:** `review/ratification-2026-10-06.md` (`Status: ratified`, `Kind: report`, one `Ratified:` citation).

**`README.md`:** the *Active changes* entry's status, now linking the record.

The sequenced-after ledger row does not move.

**The one divergence from the text as filed.** `tasks.md` 1.2 says the record carries `Status: record`. It carries `Status: ratified` with a `Ratified:` citation, as #1144's record did. `document-lifecycle` sanctions both spellings, the option put to Brett read "(Status: ratified, Ratified by)", and a `Status: record` file trips `record-immutability` (CRITICAL) on any later edit, such as a Copilot fix commit on this PR. Confirmed by the holder at #656 `6023619783`.

## Gate state at `f9f71063`

The tree is `main` at `40d6e5c1` plus this PR's twelve commits (the change itself, five merges of `main` and six re-measurements of the record), with the submodules initialised. After the first head, `main` moved five times: the archive of `add-worker-input-budget` (#1258), then #1266 and #1264, then the archive of `amend-worker-input-budget-dispatch-wording` (#1269), then #1263, #1270, #1271 and #1267 (renew-resolved-council-protocol), then #1272, #1275 (the archive of `correct-naming-families-count`) and #1277 (a bot review-lane repin). They touch `README.md`, `docs/`, the sweep ledger, other changes, the review-lane contracts and workflows and the doc-health and project-repo-schema specs, and not the change directory (its tree is still `02d362f6ebcaf49e18aa107f7ce2b3a409ca2275` on `main`, last commit `fc4fa0ff`). So `main` was merged in, never rebased, and everything below was re-measured. `main` has since moved again, to `b4b9d903` (a new change, `refresh-install-repository-enumerations-opsxfactory-gateway`: its own directory, a README entry and a ledger row). It is not merged here, per the holder's one-merge instruction, and a trial merge with it is clean.

```
OPENSPEC_TELEMETRY=0 openspec validate realize-doc-health-direction-arc --strict → valid (the pinned 1.12.0)
validate-openspec-cli-pin.py --all --strict   → exit 0, 0 UNDISPOSITIONED failures
                                                113 passed, 1 failed (114 items); the 1 = the accepted exception, as on main alone
validate-code-surface.py . / validate-target-release.py .  → passed
validate-sequenced-after.py . / --ledger-diff → pass; ledger consistent (235 rows), this row unmoved
proposal-support.py . verify realize-doc-health-direction-arc → ok
doc-health.py --single-repo .                 → 31 critical, 26 error, 69 warning, 19 info
                                                report byte-identical to main alone (same clone kind); 0 findings name this change
doc-health families (1.2's falsifier)         → proposal-origin 0 findings, status-validity 0 findings
                                                (negative control: dropping proposal.md's Ratified: line takes ratified-provenance 27 → 28 critical, naming proposal.md)
pytest -m "not postgres" -k "ledger or records or sequenced or openspec"  → 615 passed, 3 skipped
pytest sequenced_after code_surface target_release proposal-support scope_globs packet_reference → 866 passed, 311 subtests passed
pytest tests/doc-health                       → 2157 passed
```

An earlier `tests/doc-health` run showed 8 failures in `test_neutrality_dispatch.py`. They were this actor's own environment, not the tree: an exported `REPO` variable collided with the fake scout worker's `REPO`. With it unset the file gives 25 passed, and the whole directory is green above.

## Copilot

- **First head `0039a47e`:** review `5436306140`, "Approval recommended", Findings: None, 0 threads.
- **Head `739da6e0`** (after the first merge of `main`): review `5440756736`, "Approval recommended", Findings: None, 0 threads.
- **Head `69479c77`** (after the second merge of `main`): review `5459920482`, "Approval recommended", 0 threads.
- **Head `83e3511e`** (after the third merge of `main`): review `5460937374`, "Approval recommended", 0 open findings, 0 threads.
- **Head `b186eb62`** (after the fourth merge of `main`): review `5462168108`, "Needs a closer look", **0 open findings**, 0 threads. Its stated reason is that final approval depends on human validation of the governance authority and the historical provenance assertions. The holder ruled at #656 [`6069024568`](#656 (comment)), item 5, that this is NOT a block, because that authority is Brett Heap's ratify word `6023375303`.
- **Checks at `b186eb62`:** the rollup was SUCCESS with 15 status checks, all SUCCESS, including `merge-master-approval`, `pytest-suite` and SonarCloud Code Analysis. 0 threads unresolved.
- **Head `f89fc853`** (after the fifth merge of `main`): review `5464071466`, "Needs a closer look", **0 open findings**, 0 threads. Its stated reason is that the exact-head checks and the Codex review were then pending, and that final approval also depends on governance provenance, which the holder's ruling at #656 `6069024568`, item 5, covers.
- **Final head `f9f71063`** (a record-only correction of the corpus subtest figure, 312 to 311, which the completed run measured): Copilot requested through the reviewer API after the push. Its result, the checks and the thread count are recorded here once the exact final head has been reviewed.

## Codex

- At `b186eb62`, `@codex review` was posted twice (comments `6067925769` and `6069046343`), and the `chatgpt-codex-connector[bot]` answered both with the usage-limit message (comments `6067927988` and `6069048581`). No Codex review existed at that head.
- At `f89fc853`, `@codex review` (comment `6071059884`) was answered with a review: **Reviewed commit `f89fc853cb`**, "Didn't find any major issues" (comment `6071083487`; summary comment `6071062721`, Code Review completed at 2026-10-08T23:29:24Z).
- At the final head `f9f71063`, `@codex review` was posted once after the push (comment `6071252263`). The bot's reply is recorded here once it arrives. READY needs a Codex "Reviewed commit" at the exact head, or Brett Heap's explicit waiver (#656 `6069024568`, item 4).

## What is NOT done

- No code, pin, gitlink, contract bundle or release tag moves. T072, T074 and T075 are the realization, and none of them starts by this PR.
- Plan 038's `specs/038-…/tasks.md` is not edited here. T071's tick there goes in the holder's own bookkeeping batch.
- Nothing archives or promotes.

**Rule 6:** this PR touches `openspec/changes/` and the README OpenSpec Records block, so it lands in a declared window by the holder, and not by this actor.

🤖 Generated with [Claude Code](https://claude.com/claude-code)


Lane: openxfactory-4
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
brettheap added a commit that referenced this pull request Oct 9, 2026
…two early-wave words, and the holder's wave-1 and wave-2 rulings (plan 038) (#1279)

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

## T002d: plan 038's fourth bookkeeping batch (plan 038)

Bookkeeping: no `Arc:` line, and no `openspec/changes/` path. It encodes rulings already posted on #656 and a landing already merged, each cited by comment id or merge commit, with no new decision. It follows #1263 in form and redoes nothing #1263 did. Claim: #656 `6069520093`. `origin/main` is merged into the branch twice, `40d6e5c1` and then `564f565a`, never rebased.

**Realizes / Falsifier:** none (bookkeeping). The gates are listed under "Gates" below.

### What changed (specs/038-opendox-document-tool-self-maintenance/)

**A. Ticks.** Each merge commit was read through GraphQL (`pullRequest{mergeCommit{oid}}`) before it was written. The sweep covered every PR merged since #1263's landing (2026-10-08T18:12Z) in openDox-code, openXdox-code, openDox, openXdox, openDox-spec and openxFactory, plus xFactory and codexFactory, and was re-run at 2026-10-08T23:19Z:
- **openDox-code:** T012 and T013, #90 → `6827cafc` (2026-10-08T21:08:43Z), in one PR as T012's "Lands with" requires. T012's Landed line records the accepted limits that #90's body states:
  - the holder's `6067853207`: Copilot 4223441552 with Codex P2 4223462534, Copilot 4223441643, and Codex P1 4223462529. Item 3 widens the fast-forward limit to both paths.
  - `6069024568` item 1: lane 3's R2-DELTA MINOR at `landing.py:836`.
  - beside those, `6026622117` item 1 and `6035797819`.
- **Nothing else** merged in those repositories realizes a plan-038 task. The openxFactory merges since (#1267, #1270, #1271, #1272, #1275 and #1277) belong to other changes.
- **The previous batch,** #1263 → `3a2ab303`, is this batch's "merged since" anchor in the revision paragraph, as #1263 treated #1257. It is not added to T002's Landed line.
- **The task count** reads T012 and T013 done.

**B. Brett's words on phase 5's early waves,** recorded under tasks.md "What can start":
- wave 1 (T041, T042 and T043) is written now and lands after T027 (`6035546708`, *"Write now, land after T027 (Recommended)"*);
- wave 2 (T044 and T046) is written early, stacked (`6067802993`, *"Write now, stacked (Recommended)"*), and lands in plan order. Every After line keeps its landing meaning.

**C. The holder's wave-1 rulings (`6069024023`).**
- **Item 1.** `contracts/health-finding.md` § The id rule names the engine categories, T041's nine plus `dispositions-refused` and `manifest-refused`. It states that a category is also the finding's `kind`, and gains the four mappings:
  - stdout over the cap is `pack-bound-hit`;
  - non-JSON or contract-breaking output is `pack-output-refused`;
  - one finding per (category, entry) per run, with the reasons in `evidence`;
  - each category's `pack_id`, as `6072086385` item 1 rules it (see "The review questions, ruled" below). It replaces the phrase "one `pack_id` per category", which named no value.

  T045's text records its refusal extension, extending `6065680005` item 3: it refuses a pack family that declares an engine kind, and a pack finding with an empty path. T041's note records the two added categories and points to the contract's list.
- **Item 2, option (D).** `contracts/health-finding.md` (the re-raise bullet) and `data-model.md` § Baseline classes carry the re-raise's two forms, with no schema change:
  - `{disappeared_id}`, with the original's path;
  - the pathless form: `path ""`, `{category: <the original's kind>, entry: <its entry, or "">}`, and `disappeared_id` and the baseline run in `evidence`.

  T046's falsifier gains the pathless-original node (run 2 re-raises once, run 3 raises nothing). T041's note says that `disappearance_identity()` returns either form.
- **Item 3.** `data-model.md` § Health run gains the `run_seq` row, and a note that `baseline_class` is NOT NULL, with no fourth value. The note is also on § Finding's own row. T046's text gains the per-corpus advisory transaction lock, and its falsifier gains the node that two overlapping runs serialize. T042's column list and Ruled line name `run_seq` (item 3 accepts it as T042's).

**D. The holder's admissions (`6069024568`).**
- **Item 2.** T015's Files gain `src/opendox/web/views/account-menu.js` (one word, `"submit"` in `WRITE_ACTIONS`) and `pyproject.toml` (one line, `:312`'s 42 → 43), each with the chain T015 → T016 → any later verb. T016's Files record that it follows T015 in both. plan.md gains a row for `account-menu.js`, and its `pyproject.toml` row gains that line's chain.
- **Item 3.** T042's Files gain `tests_runtime/test_api_endpoints.py`, for one edit.

**E. The holder's wave-2 rulings (`6069507373`).**
- **T044 item 3.** `tasks.md:1163` now reads "the top-level directories named by the six role keys or their declared words", as FR-010 words it at `spec.md:819-821`.
- **T046 item 8.** T046's After line gains T016: the `cli.py` and `default_profile.py` chains at `plan.md:456-457`, cited by row name because this PR moves plan.md's lines. Its text says that its `default_profile.py` and `cli.py` contributions, and the admitted `tests/test_default_profile.py` line, join its PR only after T016 lands, by a merge of `main`. That line joins T046's Files.
- **T046 item 2.** T041's note gains the module-level, patchable `finding_id()`, the single hash path `make_finding` calls. Its interface is unchanged (lane 3, `6069516931`).
- **T046 item 4.** T056's Files gain ONE `engine.py` line binding T046's pack hook. It is written after T046 lands, with nothing else in `engine.py` (lane 3's ACK, `6069516931`). T046's Files mark `engine.py` as "first: T046 → T056", and plan.md gains its row.
- **`contracts/health-finding.md:43`.** `baseline_class` is NOT NULL in the store (`0003_`). The row also says that the finding schema's `finding-keys` rule lists it as optional, since no schema change is ruled.
- **N-14 and OQ-H-15, one owner each.**
  - N-14 is T046's: suppression on an injected source of accepted ids, with T054 supplying the parser (T046 item 3).
  - OQ-H-15 is T044's: in the families, through `excluded=`, with no `doxbench_intake.py` edit (T044 item 8).
  - T054's text and its Decisions line drop both.
- **`plan.md:437-438`.** "rebases" becomes "merges `main`".

tasks.md's header and plan.md each carry a revision paragraph.

### The review questions, ruled (`6072086385`), encoded in `8b76a05d`
- **Item 1 (Q1, Codex 4225201227): each engine category's `pack_id`, option (b),** as T041's `ENTRY_CATEGORIES` implements it and as dox-v1.2's schema text for `pack_id` reads.
  - The entry's id, for a finding against a VALID manifest entry's pack: `fetch-failed`, `digest-mismatch`, `declaration-refused`, `pack-crashed`, `pack-timed-out`, `pack-bound-hit` and `pack-output-refused`.
  - `opendox` for every other category: `no-sandbox`, `manifest-refused`, `dispositions-refused` and `entry-refused`. A refused entry rides only in `identity.entry`.
  - The rule is stated in `contracts/health-finding.md` § The id rule (replacing "one `pack_id` per category") and in its Rules section's install-level bullet. T041's note and Ruled line cite it.
- **Item 2 (Q2, Copilot 4225214488), option (b): no plan change.** #99 already pins NOT NULL, the run order and the uniqueness. T042 adds a catalog assertion (GENERATED ALWAYS, CACHE 1) in its own tests.
- **Item 3 (Q3, Copilot 4225214548), option (a).** T045's Falsifier gains one node for each refusal that `6069024023` item 1 extends from `6065680005` item 3: a family that declares an engine kind is refused, and a pack finding with an empty path is refused. T045's Ruled line cites item 3.
- **Item 4 (the NUL seam)** is not in this batch, as the holder directs. It is left to the next batch.

### Ruled and Decisions
- **Ruled (all on #656):** `6035546708`, `6067802993`, `6067853207`, `6069024023`, `6069024568`, `6069507373`, `6069516931` and `6072086385` (items 1 to 3).
- **Cited beside them:** `5988818366`, `6064244857`, `6065680005`, `6026622117` and `6035797819`.
- **Decisions:** none new.

### Readings, disclosed
- **6035546708's placement.** It was recorded nowhere in the tree. Its own record line asks for "beside the release map (T005 item 12)", which is #1144's `openspec/changes/` path, outside this batch. It is recorded with `6067802993` under "What can start", where tasks.md words N-6 (a) as "start". The holder accepted this placement when the writer resumed.
- **`plan.md:437-438`.** A literal swap would read "merges `main` onto the previous slice's landing", so "onto" also became "after".
- **The single-writer rows.** `6069024568` item 2 and `6069507373` T046 item 4 set chains, and plan.md's table records them as #1263 did for its chains:
  - rows for `account-menu.js` and `health/engine.py`;
  - `pyproject.toml`'s `:312` chain, beside its T048 → T055 → T061 chain.
- **The engine categories in the contract.** The ruling adds two categories to T041's nine, so the contract names all eleven. Item 1 accepts the nine as written, and its mappings name `pack-bound-hit` and `pack-output-refused`.
- **The `baseline_class` NOT NULL note** is in `data-model.md` § Health run, as the ruling words it, and on § Finding's own row.
- **Line drift.** Every cited line matched by content at the base `c8dde131`:
  - `tasks.md:1163` and `:1238-1240`;
  - `plan.md:437-438` and `:456-457`;
  - `data-model.md:260-262`;
  - `contracts/health-finding.md:43`, `:59-62` and `:64-69`.

  One exception: `6069024023` item 1 says T041's nine copy `data-model.md:330-334`. In this file, those lines are § Pack declaration, which lists no categories, so nothing is encoded from that cite.

### NOT encoded (the writers' own)
- **`6069507373` T046 item 2's edge,** where two pathless originals of one (category, entry) yield ONE re-raise, and T044's other items: these are implementation rulings for those writers, not bookkeeping.
- **plan.md:664's R2Q11 summary row,** which the ruling does not name.
- **#90's readings 1 to 5 and 7** (inferred, not ruled).
- **`6072086385` item 4,** the NUL seam: left to the next batch.
- **T071's tick** (#1261 → `a020b34a`, landed at 01:13:36Z, after this PR's last push): left to the next batch, as the holder directs.

### Gates
- **Validator.** `python3 scripts/validate-openspec-cli-pin.py --all` and `--all --strict` at `8b76a05d` both exit 0, with `Totals: 113 passed, 1 failed (114 items)`. The one failure is the accepted exception, `add-chain-attestation`'s disposition in the pin, as on main.
- **CI at the head `8b76a05d`:** 14 of 14 checks are SUCCESS (rollup SUCCESS, read through GraphQL), including `pytest-suite`, `doc-health-py314`, `openspec-cli-pin`, `lane-line` and SonarCloud.
  - `pytest-suite` reports `selected=9838 passed=9832 skipped=6 failures=0 errors=0` (9208 tests passed, 6 skipped).
  - Its freshness verifier and its vector replay both passed.
- **Local pytest at `8b76a05d`** (`-m "not postgres" -n 8`, with the three gitlinks initialized as CI does): `2 failed, 9205 passed, 7 skipped`.
  - Both failures are 30 s subprocess timeouts of `validate-hermes-runtime-contracts.py` in `tests/hermes_runtime_contracts/test_validator_cli.py`, on a shared host at a load average of about 85 on 32 cores.
  - Serially, the release-mode node also fails on main, as #1263 recorded.
  - The candidate-mode node passed serially at `897dce40` and on main `b4b9d903`, and timed out under that load at `8b76a05d`.
  - CI runs both green.
- **doc-health** (`scripts/doc-health.py --single-repo .`), at `897dce40` against main `b4b9d903` in the same clone: identical reports, 31 critical, 26 error, 69 warning and 19 info, with 0 differences after SHA normalization. `doc-health-py314` is green at the head.
- **Copilot.**
  - At `897dce40`: "Changes recommended", with two threads (4225214488 and 4225214548), ruled by the holder in `6072086385` items 2 and 3.
  - At `8b76a05d`: "Approval recommended", with 0 open findings.
- **Codex.**
  - At `897dce40` it reviewed the head ("Reviewed commit: `897dce405c`") and opened one P2 thread (4225201227). The holder ruled it in `6072086385` item 1, and `8b76a05d` encodes the ruling.
  - At `8b76a05d`, the trigger posted at 01:02:53Z was answered at 01:02:56Z with "You have reached your Codex usage limits". There is no Codex review at the head yet.
- **Threads:** all 3 are answered, citing `6072086385`, and resolved.

### NOT done
- spec.md, every `openspec/changes/` path and #1144 are untouched.
- No box is ticked in #1144, and no task is added.

refs #656

🤖 Generated with [Claude Code](https://claude.com/claude-code)


Lane: openxfactory-4
Co-Authored-By: Claude Opus 5.5 (1M context) <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.

1 participant