Design Ohio government data coverage #53

Merged
lost-rob0t merged 73 commits from design/ohio-government-data-catalog into main 2026-08-02 20:13:20 +00:00
lost-rob0t commented 2026-07-28 21:38:44 +00:00 (Migrated from github.com)

Consolidated scope

This PR now targets main directly and contains the complete former #48 → #52 → #53 stack. PRs #48 and #52 are superseded once this aggregate PR is verified.

Summary

  • add the U.S. government-data acquisition research and supervised actor design
  • add the generated 52-state/territory and 3,222-county-equivalent source catalog
  • make Ohio the first state-level government-data coverage design
  • define Franklin County (39049) as the first complete pilot
  • preserve governments and components as revisioned durable records and bounded job keys, not permanent per-county actors
  • define immutable unit, component, deployment, profile, search-plan, subfamily-decision, gap, route-assessment, review-decision, and coverage-gate revisions
  • enumerate all sixteen Ohio source routes with a machine-readable applicability fixture
  • define operational metadata as a derived gate assertion that emits no standalone jobs
  • bind completion to exact independently reviewed subject revisions under deterministic atomic gates
  • separate durable jobs from lease-bound attempts and require fencing tokens
  • require durable evidence-delivery intents between artifact commit and retrieval/normalization delivery
  • add deterministic Franklin identity, review, delivery, hash, expiry, invalidation, and supersession fixtures
  • add a Franklin pilot runbook
  • add offline CI that validates the contracts and fixtures and reproduces every generated jurisdiction file without network access

Normative protocol set

The government-data runtime protocol is reviewed as one set:

  1. STAR-RESEARCH-PIPELINE-003 Government Data Acquisition Actors
  2. STAR-RESEARCH-PIPELINE-004 Government Data Revision Protocol Amendment
  3. STAR-RESEARCH-PIPELINE-004A Government Data Delivery and Identity Clarification

The envelope remains star.government-data.v1.1. PIPELINE-004A normatively resolves the remaining cross-document mismatches without creating another envelope version.

Identity and review model

  • government units and components are canonical immutable revisions
  • applicability contracts have stable logical IDs and immutable revision IDs
  • each subject revision has one CAS-linear independent review stream
  • subject owners cannot self-review
  • later revisions never mutate historical replay
  • gates pin exact unit, component, applicability, profile, deployment, search, gap, assessment, review, policy, manifest, and runtime revisions

Evidence-delivery model

The normative retained-artifact sequence is:

  1. validate contracts, job, attempt, lease, and fence
  2. stage, bound, and hash bytes
  3. commit artifact plus durable evidence-delivery intent atomically when possible
  4. otherwise use a durable outbox and orphan-artifact recovery
  5. deliver the retrieval event from the intent
  6. release normalization work from the intent
  7. acknowledge each delivery step idempotently
  8. commit a checkpoint with acknowledged and pending step state
  9. permit terminal success only after every success-required step is acknowledged

Results and checkpoints expose delivery-intent IDs and acknowledged/pending delivery-step keys.

Completion gates

A jurisdiction reaches DONE only through a current, independently reviewed, passed jurisdiction-completion gate. The gate includes a canonical basis hash, freshness deadline, and valid-through.

Current completion becomes false immediately after:

  • valid-through passes
  • a bound review is invalidated
  • a bound identity or applicability revision is superseded
  • a required delivery intent becomes missing, orphaned, invalid, or incomplete
  • the canonical basis hash no longer validates

Historical gates remain immutable and replayable.

Validation

The Government Data Design workflow runs entirely offline and verifies:

  • exact sixteen-route ordering and unique machine-readable subfamily rules
  • route 16 is derived, emits no jobs, and uses derived-gate-assertion
  • required Franklin identity, CAS, independent-review, delivery, fencing, recovery, hash, expiry, and supersession fixtures
  • required normative clarification tokens
  • exactly 52 state/territory groups and 3,222 unique county-equivalent GEOIDs
  • byte-for-byte regeneration of all 17 checked-in jurisdiction catalog files from a locally reconstructed roster

No source, county, design, or implementation state is marked complete or approved by this PR.

Merge gate

Merge only when:

  • the aggregate branch is conflict-free against current main
  • all applicable workflows complete successfully
  • the current aggregate head receives a fresh review
## Consolidated scope This PR now targets `main` directly and contains the complete former #48 → #52 → #53 stack. PRs #48 and #52 are superseded once this aggregate PR is verified. ## Summary - add the U.S. government-data acquisition research and supervised actor design - add the generated 52-state/territory and 3,222-county-equivalent source catalog - make Ohio the first state-level government-data coverage design - define Franklin County (`39049`) as the first complete pilot - preserve governments and components as revisioned durable records and bounded job keys, not permanent per-county actors - define immutable unit, component, deployment, profile, search-plan, subfamily-decision, gap, route-assessment, review-decision, and coverage-gate revisions - enumerate all sixteen Ohio source routes with a machine-readable applicability fixture - define operational metadata as a derived gate assertion that emits no standalone jobs - bind completion to exact independently reviewed subject revisions under deterministic atomic gates - separate durable jobs from lease-bound attempts and require fencing tokens - require durable evidence-delivery intents between artifact commit and retrieval/normalization delivery - add deterministic Franklin identity, review, delivery, hash, expiry, invalidation, and supersession fixtures - add a Franklin pilot runbook - add offline CI that validates the contracts and fixtures and reproduces every generated jurisdiction file without network access ## Normative protocol set The government-data runtime protocol is reviewed as one set: 1. `STAR-RESEARCH-PIPELINE-003 Government Data Acquisition Actors` 2. `STAR-RESEARCH-PIPELINE-004 Government Data Revision Protocol Amendment` 3. `STAR-RESEARCH-PIPELINE-004A Government Data Delivery and Identity Clarification` The envelope remains `star.government-data.v1.1`. `PIPELINE-004A` normatively resolves the remaining cross-document mismatches without creating another envelope version. ## Identity and review model - government units and components are canonical immutable revisions - applicability contracts have stable logical IDs and immutable revision IDs - each subject revision has one CAS-linear independent review stream - subject owners cannot self-review - later revisions never mutate historical replay - gates pin exact unit, component, applicability, profile, deployment, search, gap, assessment, review, policy, manifest, and runtime revisions ## Evidence-delivery model The normative retained-artifact sequence is: 1. validate contracts, job, attempt, lease, and fence 2. stage, bound, and hash bytes 3. commit artifact plus durable evidence-delivery intent atomically when possible 4. otherwise use a durable outbox and orphan-artifact recovery 5. deliver the retrieval event from the intent 6. release normalization work from the intent 7. acknowledge each delivery step idempotently 8. commit a checkpoint with acknowledged and pending step state 9. permit terminal success only after every success-required step is acknowledged Results and checkpoints expose delivery-intent IDs and acknowledged/pending delivery-step keys. ## Completion gates A jurisdiction reaches `DONE` only through a current, independently reviewed, passed `jurisdiction-completion` gate. The gate includes a canonical basis hash, freshness deadline, and `valid-through`. Current completion becomes false immediately after: - `valid-through` passes - a bound review is invalidated - a bound identity or applicability revision is superseded - a required delivery intent becomes missing, orphaned, invalid, or incomplete - the canonical basis hash no longer validates Historical gates remain immutable and replayable. ## Validation The `Government Data Design` workflow runs entirely offline and verifies: - exact sixteen-route ordering and unique machine-readable subfamily rules - route 16 is derived, emits no jobs, and uses `derived-gate-assertion` - required Franklin identity, CAS, independent-review, delivery, fencing, recovery, hash, expiry, and supersession fixtures - required normative clarification tokens - exactly 52 state/territory groups and 3,222 unique county-equivalent GEOIDs - byte-for-byte regeneration of all 17 checked-in jurisdiction catalog files from a locally reconstructed roster No source, county, design, or implementation state is marked complete or approved by this PR. ## Merge gate Merge only when: - the aggregate branch is conflict-free against current `main` - all applicable workflows complete successfully - the current aggregate head receives a fresh review
lost-rob0t (Migrated from github.com) reviewed 2026-07-28 22:48:04 +00:00
lost-rob0t (Migrated from github.com) left a comment

Full review — changes required before merge

The Ohio design pack has a strong bounded-actor and evidence-first foundation, but it is not merge-ready.

Blocking issues

  1. The dependency stack is unresolved. This draft targets draft PR #52, which depends on open, currently non-mergeable draft PR #48. Land or repair the stack bottom-up, then rebase this PR.
  2. No CI/check evidence exists for this head. There are no workflow or status checks attached to 4449825. Run the repository validation or document why these design paths do not trigger CI.
  3. The execution dependency order is inverted. Wave 0/1 and county pilots require source-profile storage, immutable artifacts, provenance validation, the coverage ledger, and review authority before they can satisfy their own replay/evidence gates.
  4. Route completion is non-deterministic. Required subfamilies and applicability rules are not defined by route and government/component type.
  5. Assessments do not pin immutable profile revisions. Route assessments reference logical profile IDs, so old decisions can appear to change after later revisions.
  6. The lifecycle mixes source and review state. This contradicts the stated invariant that review state is separate from acquisition state.
  7. The Franklin exit gate is too weak. Merely requiring routes to be “assessed” permits candidate-only or unresolved states to pass.
  8. The diverse-county set is not sufficiently platform-driven. It overweights large metros and demographic labels instead of demonstrable vendor, authority, shared-service, court/health, and publication variation.
  9. Historical identity starts too late. Predecessor, successor, archive, and migration evidence cannot all be deferred until after current-profile review.
  10. Core semantic decisions remain open. Negative-finding expiry, local court topology, and Ohio Checkbook’s evidentiary role affect completion correctness and must be resolved before approval.

Required before merge

  • reorder the implementation dependency list;
  • add a versioned route/subfamily/applicability contract;
  • bind route assessments to profile revision IDs and make assessments append-only/superseding;
  • define independent source-state and review-state enums/transitions;
  • strengthen Franklin and Wave 3→4 exit gates with measurable criteria;
  • resolve negative-finding freshness, court scope, and Checkbook authority;
  • make the Wave 3 county set provisional and fingerprint-driven;
  • capture minimal historical identity during discovery;
  • run validation after the PR stack is rebased.

Add a shared source-deployment/tenant record, structured jurisdiction and record-type coverage scope, a bounded OCR/manual-review outcome, and reproducible browser evidence beyond screenshot metadata.

Disposition: changes required before merge. GitHub does not allow the PR author to submit REQUEST_CHANGES on their own PR, so this is posted as a comment review with blocking inline threads.

## Full review — changes required before merge The Ohio design pack has a strong bounded-actor and evidence-first foundation, but it is not merge-ready. ### Blocking issues 1. **The dependency stack is unresolved.** This draft targets draft PR #52, which depends on open, currently non-mergeable draft PR #48. Land or repair the stack bottom-up, then rebase this PR. 2. **No CI/check evidence exists for this head.** There are no workflow or status checks attached to `4449825`. Run the repository validation or document why these design paths do not trigger CI. 3. **The execution dependency order is inverted.** Wave 0/1 and county pilots require source-profile storage, immutable artifacts, provenance validation, the coverage ledger, and review authority before they can satisfy their own replay/evidence gates. 4. **Route completion is non-deterministic.** Required subfamilies and applicability rules are not defined by route and government/component type. 5. **Assessments do not pin immutable profile revisions.** Route assessments reference logical profile IDs, so old decisions can appear to change after later revisions. 6. **The lifecycle mixes source and review state.** This contradicts the stated invariant that review state is separate from acquisition state. 7. **The Franklin exit gate is too weak.** Merely requiring routes to be “assessed” permits candidate-only or unresolved states to pass. 8. **The diverse-county set is not sufficiently platform-driven.** It overweights large metros and demographic labels instead of demonstrable vendor, authority, shared-service, court/health, and publication variation. 9. **Historical identity starts too late.** Predecessor, successor, archive, and migration evidence cannot all be deferred until after current-profile review. 10. **Core semantic decisions remain open.** Negative-finding expiry, local court topology, and Ohio Checkbook’s evidentiary role affect completion correctness and must be resolved before approval. ### Required before merge - reorder the implementation dependency list; - add a versioned route/subfamily/applicability contract; - bind route assessments to profile revision IDs and make assessments append-only/superseding; - define independent source-state and review-state enums/transitions; - strengthen Franklin and Wave 3→4 exit gates with measurable criteria; - resolve negative-finding freshness, court scope, and Checkbook authority; - make the Wave 3 county set provisional and fingerprint-driven; - capture minimal historical identity during discovery; - run validation after the PR stack is rebased. ### Recommended follow-ups Add a shared source-deployment/tenant record, structured jurisdiction and record-type coverage scope, a bounded OCR/manual-review outcome, and reproducible browser evidence beyond screenshot metadata. **Disposition: changes required before merge.** GitHub does not allow the PR author to submit `REQUEST_CHANGES` on their own PR, so this is posted as a comment review with blocking inline threads.
@ -0,0 +1,178 @@
:PROPERTIES:
lost-rob0t (Migrated from github.com) commented 2026-07-28 22:48:04 +00:00

Assessed is too weak for an end-to-end pilot exit gate. This currently allows all sixteen routes to remain candidate, ambiguous, blocked, or unreviewed and still pass. Define the minimum allowed reviewed terminal state per route, require applicability decisions for missing county functions, and require exact profile revisions/search plans/gaps to be bound into the pilot decision.

`Assessed` is too weak for an end-to-end pilot exit gate. This currently allows all sixteen routes to remain candidate, ambiguous, blocked, or unreviewed and still pass. Define the minimum allowed reviewed terminal state per route, require applicability decisions for missing county functions, and require exact profile revisions/search plans/gaps to be bound into the pilot decision.
lost-rob0t (Migrated from github.com) commented 2026-07-28 22:48:04 +00:00

The set is useful demographically but overweights metro counties and does not prove platform or organizational diversity. Make this list provisional until Wave 0 fingerprinting, then select counties to maximize vendor/platform family, official-versus-delegated hosting, publication maturity, independent-office fragmentation, shared/regional services, court/health topology, geography, and size. Include an explicit mid-sized non-metro or western agricultural case unless fingerprint evidence shows the current set covers it.

The set is useful demographically but overweights metro counties and does not prove platform or organizational diversity. Make this list provisional until Wave 0 fingerprinting, then select counties to maximize vendor/platform family, official-versus-delegated hosting, publication maturity, independent-office fragmentation, shared/regional services, court/health topology, geography, and size. Include an explicit mid-sized non-metro or western agricultural case unless fingerprint evidence shows the current set covers it.
lost-rob0t (Migrated from github.com) commented 2026-07-28 22:48:04 +00:00

Full historical acquisition can remain Wave 5, but historical identity cannot be deferred this late. Every earlier discovery/profile wave should record predecessor locators, successor relationships, archive candidates, first/last observed dates, and immediately visible coverage boundaries. Otherwise ownership verification and source identity may need to be redone during reconciliation.

Full historical acquisition can remain Wave 5, but historical **identity** cannot be deferred this late. Every earlier discovery/profile wave should record predecessor locators, successor relationships, archive candidates, first/last observed dates, and immediately visible coverage boundaries. Otherwise ownership verification and source identity may need to be redone during reconciliation.
lost-rob0t (Migrated from github.com) commented 2026-07-28 22:48:04 +00:00

These are not all safely deferable. County ordering can remain a policy choice, but negative-finding freshness, county-court scope, and Ohio Checkbook's evidentiary role directly affect route completion. Resolve them before approval: require a default reassess-after policy by route volatility; define whether local courts are part of county coverage or a linked subprogram; and state that Checkbook participation is discovery/reconciliation evidence, never proof of complete county finance coverage.

These are not all safely deferable. County ordering can remain a policy choice, but negative-finding freshness, county-court scope, and Ohio Checkbook's evidentiary role directly affect route completion. Resolve them before approval: require a default `reassess-after` policy by route volatility; define whether local courts are part of county coverage or a linked subprogram; and state that Checkbook participation is discovery/reconciliation evidence, never proof of complete county finance coverage.
@ -0,0 +1,321 @@
:PROPERTIES:
lost-rob0t (Migrated from github.com) commented 2026-07-28 22:48:04 +00:00

Route assessments need to bind the exact immutable revisions that were reviewed, not only logical profile IDs. Otherwise a later profile revision changes the apparent basis of an older assessment. Replace or supplement this with source-profile-revision-ids, add an assessment revision/prior-assessment ID, and make route decisions append-only with explicit supersession.

Route assessments need to bind the exact immutable revisions that were reviewed, not only logical profile IDs. Otherwise a later profile revision changes the apparent basis of an older assessment. Replace or supplement this with `source-profile-revision-ids`, add an assessment revision/prior-assessment ID, and make route decisions append-only with explicit supersession.
lost-rob0t (Migrated from github.com) commented 2026-07-28 22:48:04 +00:00

Blocking — required subfamilies is undefined. Add a versioned route-subfamily/applicability contract keyed by route and government/component kind. It must declare required versus optional subfamilies, evidence needed for :not-applicable, allowed completion states, and minimum negative-search coverage. Without this, identical evidence can produce different :verified decisions.

**Blocking — `required subfamilies` is undefined.** Add a versioned route-subfamily/applicability contract keyed by route and government/component kind. It must declare required versus optional subfamilies, evidence needed for `:not-applicable`, allowed completion states, and minimum negative-search coverage. Without this, identical evidence can produce different `:verified` decisions.
lost-rob0t (Migrated from github.com) commented 2026-07-28 22:48:04 +00:00

This mixes acquisition/source lifecycle (candidate, active, changed, retired) with human review lifecycle (verification-pending, reviewed) even though invariant 10 says they are separate. Define two enums and two transition tables, plus which authority may perform each transition and how a source change invalidates or reopens review.

This mixes acquisition/source lifecycle (`candidate`, `active`, `changed`, `retired`) with human review lifecycle (`verification-pending`, `reviewed`) even though invariant 10 says they are separate. Define two enums and two transition tables, plus which authority may perform each transition and how a source change invalidates or reopens review.
@ -47,0 +88,4 @@
Actor groups:
1. control and policy
lost-rob0t (Migrated from github.com) commented 2026-07-28 22:48:04 +00:00

Blocking — dependency order is inverted. Waves 0–3 cannot satisfy their own raw-evidence, replay, review, and coverage-ledger gates before this infrastructure exists. Move source-profile persistence, immutable raw artifacts, provenance validation, the Coverage Ledger, and the review authority ahead of Wave 0/1 execution. Generic bounded probe adapters can follow the fake actors but must exist before live source probes.

**Blocking — dependency order is inverted.** Waves 0–3 cannot satisfy their own raw-evidence, replay, review, and coverage-ledger gates before this infrastructure exists. Move source-profile persistence, immutable raw artifacts, provenance validation, the Coverage Ledger, and the review authority ahead of Wave 0/1 execution. Generic bounded probe adapters can follow the fake actors but must exist before live source probes.
lost-rob0t (Migrated from github.com) reviewed 2026-07-28 22:59:44 +00:00
@ -47,0 +88,4 @@
Actor groups:
1. control and policy
lost-rob0t (Migrated from github.com) commented 2026-07-28 22:59:44 +00:00

Addressed in 807b4ec. The government-data dependency order now implements immutable profile/assessment records, raw artifacts, provenance validation, the Coverage Ledger, and independent review authority before Wave 0/1. The bounded generic probe adapters also precede live source probes, with an explicit no-live-wave prerequisite.

Addressed in `807b4ec`. The government-data dependency order now implements immutable profile/assessment records, raw artifacts, provenance validation, the Coverage Ledger, and independent review authority before Wave 0/1. The bounded generic probe adapters also precede live source probes, with an explicit no-live-wave prerequisite.
lost-rob0t (Migrated from github.com) reviewed 2026-07-28 22:59:54 +00:00
@ -0,0 +1,321 @@
:PROPERTIES:
lost-rob0t (Migrated from github.com) commented 2026-07-28 22:59:54 +00:00

Addressed in 5a2db17. route-assessment-revision now has stable and immutable assessment IDs, prior/supersedes links, exact source-profile-revision-ids, exact search-plan revisions, evidence/gap bindings, and applicability-contract versioning. Reassessment creates a superseding revision instead of mutating the old decision.

Addressed in `5a2db17`. `route-assessment-revision` now has stable and immutable assessment IDs, prior/supersedes links, exact `source-profile-revision-ids`, exact search-plan revisions, evidence/gap bindings, and applicability-contract versioning. Reassessment creates a superseding revision instead of mutating the old decision.
lost-rob0t (Migrated from github.com) reviewed 2026-07-28 23:00:05 +00:00
@ -0,0 +1,321 @@
:PROPERTIES:
lost-rob0t (Migrated from github.com) commented 2026-07-28 23:00:05 +00:00

Addressed in 5a2db17. Added versioned route-applicability-contract and route-subfamily-rule records covering unit/component kinds, required/optional/conditional subfamilies, activation predicates, allowed terminal states, minimum search plans, state-source substitution, service-provider resolution, negative freshness, and positive evidence required for :not-applicable.

Addressed in `5a2db17`. Added versioned `route-applicability-contract` and `route-subfamily-rule` records covering unit/component kinds, required/optional/conditional subfamilies, activation predicates, allowed terminal states, minimum search plans, state-source substitution, service-provider resolution, negative freshness, and positive evidence required for `:not-applicable`.
lost-rob0t (Migrated from github.com) reviewed 2026-07-28 23:00:12 +00:00
@ -0,0 +1,321 @@
:PROPERTIES:
lost-rob0t (Migrated from github.com) commented 2026-07-28 23:00:12 +00:00

Addressed in 5a2db17. Source state and human review state now have separate enums, transition tables, and authorities. A detected change creates a new profile revision with reopened/invalidated review; it does not alter the previously reviewed revision. Retirement is explicitly a source-state transition that still requires reviewed evidence.

Addressed in `5a2db17`. Source state and human review state now have separate enums, transition tables, and authorities. A detected change creates a new profile revision with reopened/invalidated review; it does not alter the previously reviewed revision. Retirement is explicitly a source-state transition that still requires reviewed evidence.
lost-rob0t (Migrated from github.com) reviewed 2026-07-28 23:00:25 +00:00
@ -0,0 +1,178 @@
:PROPERTIES:
lost-rob0t (Migrated from github.com) commented 2026-07-28 23:00:25 +00:00

Addressed in 140a7e8. Franklin now requires reviewed revisions for all sixteen routes under one pinned applicability contract. Candidate, ambiguous, and unassessed states cannot pass; negative terminal states require completed search-plan revisions, gaps, and reassessment dates; :not-applicable requires authority/service evidence; and the gate pins exact profile revisions, evidence, and gaps with deterministic replay.

Addressed in `140a7e8`. Franklin now requires reviewed revisions for all sixteen routes under one pinned applicability contract. Candidate, ambiguous, and unassessed states cannot pass; negative terminal states require completed search-plan revisions, gaps, and reassessment dates; `:not-applicable` requires authority/service evidence; and the gate pins exact profile revisions, evidence, and gaps with deterministic replay.
lost-rob0t (Migrated from github.com) reviewed 2026-07-28 23:00:31 +00:00
@ -0,0 +1,178 @@
:PROPERTIES:
lost-rob0t (Migrated from github.com) commented 2026-07-28 23:00:31 +00:00

Addressed in 140a7e8. Wave 3 is now explicitly provisional until the Wave 0 fingerprint matrix is reviewed. Selection maximizes platform/vendor, hosting authority, publication maturity, office fragmentation, shared services, court/health topology, geography, size, and migration patterns. Hancock County (39063) was added as the initial mid-sized northwest agricultural/industrial candidate, with a recorded replacement rule.

Addressed in `140a7e8`. Wave 3 is now explicitly provisional until the Wave 0 fingerprint matrix is reviewed. Selection maximizes platform/vendor, hosting authority, publication maturity, office fragmentation, shared services, court/health topology, geography, size, and migration patterns. Hancock County (`39063`) was added as the initial mid-sized northwest agricultural/industrial candidate, with a recorded replacement rule.
lost-rob0t (Migrated from github.com) reviewed 2026-07-28 23:00:40 +00:00
@ -0,0 +1,178 @@
:PROPERTIES:
lost-rob0t (Migrated from github.com) commented 2026-07-28 23:00:39 +00:00

Addressed in 140a7e8 and 5a2db17. Every wave now records minimal historical identity: first/last seen, predecessor/successor locators, redirects, migrations, archive candidates, and visible coverage bounds. Locator records carry those relationships. Wave 5 is now historical expansion and reconciliation, not the first point where source identity is captured.

Addressed in `140a7e8` and `5a2db17`. Every wave now records minimal historical identity: first/last seen, predecessor/successor locators, redirects, migrations, archive candidates, and visible coverage bounds. Locator records carry those relationships. Wave 5 is now historical expansion and reconciliation, not the first point where source identity is captured.
lost-rob0t (Migrated from github.com) reviewed 2026-07-28 23:00:49 +00:00
@ -0,0 +1,178 @@
:PROPERTIES:
lost-rob0t (Migrated from github.com) commented 2026-07-28 23:00:49 +00:00

Addressed in 140a7e8 and 5a2db17. Checkbook participation is explicitly discovery/reconciliation evidence only; county courts and linked local-court assessments must be bound into the county route; and negative terminal results now use route-volatility defaults of 30, 90, or 180 days, with legal-versioned sources reassessed by the next effective date or 180 days. Batch ordering is the only remaining open policy choice.

Addressed in `140a7e8` and `5a2db17`. Checkbook participation is explicitly discovery/reconciliation evidence only; county courts and linked local-court assessments must be bound into the county route; and negative terminal results now use route-volatility defaults of 30, 90, or 180 days, with legal-versioned sources reassessed by the next effective date or 180 days. Batch ordering is the only remaining open policy choice.
lost-rob0t commented 2026-07-28 23:02:02 +00:00 (Migrated from github.com)

Review follow-up

Addressed and resolved all eight inline review threads.

Commits

  • 807b4ec — reorder government-data implementation dependencies so durable records, evidence, provenance, ledger, review authority, and generic probes precede live waves
  • 140a7e8 — strengthen Franklin and Wave 3 gates; make county selection fingerprint-driven; add cross-wave historical identity; resolve Checkbook, court, and negative-finding semantics
  • 5a2db17 — add immutable profile/search-plan/assessment revisions, versioned route applicability, shared deployments, structured scope, independent source/review lifecycles, and replayable browser/OCR boundaries

The PR description now reflects the revised architecture and pilot order.

Remaining merge blockers

  • PR #53 is still a draft stacked on #52; the stack must land or be rebased cleanly.
  • The current head 5a2db17 has no attached GitHub Actions workflow runs or commit statuses, so repository validation is still outstanding.
  • The full initial sixteen-route subfamily contract must be enumerated before Franklin execution; this is now an explicit design and run gate rather than an undefined reviewer convention.

No source, county, design, or implementation state was marked complete or approved.

## Review follow-up Addressed and resolved all eight inline review threads. ### Commits - `807b4ec` — reorder government-data implementation dependencies so durable records, evidence, provenance, ledger, review authority, and generic probes precede live waves - `140a7e8` — strengthen Franklin and Wave 3 gates; make county selection fingerprint-driven; add cross-wave historical identity; resolve Checkbook, court, and negative-finding semantics - `5a2db17` — add immutable profile/search-plan/assessment revisions, versioned route applicability, shared deployments, structured scope, independent source/review lifecycles, and replayable browser/OCR boundaries The PR description now reflects the revised architecture and pilot order. ### Remaining merge blockers - PR #53 is still a draft stacked on #52; the stack must land or be rebased cleanly. - The current head `5a2db17` has no attached GitHub Actions workflow runs or commit statuses, so repository validation is still outstanding. - The full initial sixteen-route subfamily contract must be enumerated before Franklin execution; this is now an explicit design and run gate rather than an undefined reviewer convention. No source, county, design, or implementation state was marked complete or approved.
lost-rob0t commented 2026-07-28 23:23:37 +00:00 (Migrated from github.com)

Deeper architecture pass

I kept reviewing after resolving the original eight threads and found additional contract-level problems. These are now addressed on head 136bded.

Additional fixes

  • Enumerated the minimum requirements for all sixteen routes instead of leaving the applicability contract as examples only.
  • Made operational metadata a derived cross-cutting route rather than a fake independent source.
  • Revisioned subfamily decisions and gaps, including predecessor/supersession links.
  • Bound gap revisions directly to applicability contract, policy snapshot, exact profile revisions, and completed search-plan revision.
  • Required a new immutable revision for every source-state or review-state transition.
  • Updated the jurisdiction DONE contract so a logical source-profile ID cannot complete a county.
  • Pinned adapter jobs to contract, policy, adapter version, manifest hash, runtime hash, profile/proposal, deployment, and search-plan revisions.
  • Split durable adapter jobs from lease-bound execution attempts. Retries get a new attempt and lease fence rather than mutating the job.
  • Required raw-artifact/retrieval commits before evidence or checkpoint advancement.
  • Added stale-lease, out-of-scope redirect, artifact-commit, checkpoint, OCR, browser, and truncation fixtures.

New blocker exposed

The Ohio design is now stricter than its parent actor protocol. STAR-RESEARCH-PIPELINE-003 still models logical source-profile IDs and does not define the Route Assessment Authority, independent review authority, revision-aware message fields, or job/attempt lease semantics required by this PR.

That parent protocol must be amended before implementation. I recorded this explicitly in the Ohio documents, catalog, project index, and PR description rather than silently pretending the contracts align.

Current state

  • all original inline review threads remain resolved
  • PR remains draft and mergeable relative to its current base
  • parent PR #48 remains non-mergeable
  • head 136bded has no Actions runs or commit statuses
  • machine-readable route-rule and Franklin gate fixtures still need to be generated
## Deeper architecture pass I kept reviewing after resolving the original eight threads and found additional contract-level problems. These are now addressed on head `136bded`. ### Additional fixes - Enumerated the minimum requirements for all sixteen routes instead of leaving the applicability contract as examples only. - Made operational metadata a derived cross-cutting route rather than a fake independent source. - Revisioned subfamily decisions and gaps, including predecessor/supersession links. - Bound gap revisions directly to applicability contract, policy snapshot, exact profile revisions, and completed search-plan revision. - Required a new immutable revision for every source-state or review-state transition. - Updated the jurisdiction `DONE` contract so a logical source-profile ID cannot complete a county. - Pinned adapter jobs to contract, policy, adapter version, manifest hash, runtime hash, profile/proposal, deployment, and search-plan revisions. - Split durable adapter jobs from lease-bound execution attempts. Retries get a new attempt and lease fence rather than mutating the job. - Required raw-artifact/retrieval commits before evidence or checkpoint advancement. - Added stale-lease, out-of-scope redirect, artifact-commit, checkpoint, OCR, browser, and truncation fixtures. ### New blocker exposed The Ohio design is now stricter than its parent actor protocol. `STAR-RESEARCH-PIPELINE-003` still models logical source-profile IDs and does not define the Route Assessment Authority, independent review authority, revision-aware message fields, or job/attempt lease semantics required by this PR. That parent protocol must be amended before implementation. I recorded this explicitly in the Ohio documents, catalog, project index, and PR description rather than silently pretending the contracts align. ### Current state - all original inline review threads remain resolved - PR remains draft and mergeable relative to its current base - parent PR #48 remains non-mergeable - head `136bded` has no Actions runs or commit statuses - machine-readable route-rule and Franklin gate fixtures still need to be generated
lost-rob0t commented 2026-07-28 23:56:06 +00:00 (Migrated from github.com)

Final deeper-review pass — head cc2fea6

I continued past the original eight review threads and found several additional correctness problems. They are now addressed in the branch.

Additional defects found and fixed

  1. Parent protocol mismatch was only documented, not designed.

    • Added STAR-RESEARCH-PIPELINE-004 Government Data Revision Protocol Amendment.
    • It is the normative v1.1 amendment to PIPELINE-003 and defines revision-aware authorities, messages, routing, scheduling, checkpoints, leases, and migration.
  2. Independent review created a one-writer violation.

    • Review state is no longer written into profiles, gaps, assessments, or gates.
    • The Independent Review Authority owns separate immutable review-decision revisions.
    • Review streams are compare-and-swap linearized; concurrent forks are rejected rather than timestamp-resolved.
  3. Sixteen moving route records could not form an atomic county decision.

    • Added jurisdiction, wave, and state coverage-gate revisions.
    • A gate binds all exact route/subject/review revisions under one canonical basis hash.
    • Catalog DONE now requires a currently valid, independently reviewed, passed jurisdiction gate.
  4. Passed gates had no deterministic expiry or hash semantics.

    • Added canonical ordering/serialization, declared hash algorithm, basis-as-of, freshness-deadline, and valid-through.
    • Current completion becomes false immediately after validity expires or a bound review is invalidated; it does not wait for a later mutation.
  5. Record interpretation depended on an implied document version.

    • Defined runtime source-record contract star.govdata.source-profile@1.0.0 separately from design-document versions, payload schemas, envelope protocol, applicability contracts, and policy snapshots.
    • Commands, jobs, results, checkpoints, canonical records, reviews, and gates now pin it explicitly.
  6. Adapter jobs incorrectly mixed logical work with leases issued later.

    • Durable jobs contain immutable logical scope and contracts, but no future lease.
    • Each execution/retry creates a separate immutable attempt with lease generation and fencing token.
  7. Artifacts and checkpoints could race.

    • The protocol now commits artifact bytes/hash and retrieval events before evidence/proposals and checkpoint advancement.
  8. The catalog completion contract was generated and would be overwritten.

    • Updated tools/generate-government-jurisdiction-todos.py to generate the revised 0.4.0 master contract.
    • Added generator validation for the atomic gate, reviewed source contract, and valid-through requirements.
    • The 16 jurisdiction shards remain structurally unchanged.

Current validation state

  • all eight original inline threads remain resolved
  • full-diff scan found no stale mutable review-state, logical source-profile-id, or old 1.3 protocol/source-contract fields
  • PR #53 is open, draft, and mergeable against its current stacked base
  • head cc2fea6 has no Actions runs and no commit statuses

Remaining blockers

  1. PR #48 is still draft and non-mergeable; #52 and #53 remain stacked drafts.
  2. PIPELINE-003 and the normative PIPELINE-004 amendment require joint review/approval.
  3. The machine-readable sixteen-route applicability fixture still needs generation.
  4. Franklin needs subject/review/gate, canonical-hash, expiry, and supersession fixtures plus its runbook.
  5. The generator reproduction check and repository CI must run after the stack is repaired and rebased.

Disposition remains: keep draft; do not merge yet.

## Final deeper-review pass — head `cc2fea6` I continued past the original eight review threads and found several additional correctness problems. They are now addressed in the branch. ### Additional defects found and fixed 1. **Parent protocol mismatch was only documented, not designed.** - Added `STAR-RESEARCH-PIPELINE-004 Government Data Revision Protocol Amendment`. - It is the normative v1.1 amendment to `PIPELINE-003` and defines revision-aware authorities, messages, routing, scheduling, checkpoints, leases, and migration. 2. **Independent review created a one-writer violation.** - Review state is no longer written into profiles, gaps, assessments, or gates. - The Independent Review Authority owns separate immutable review-decision revisions. - Review streams are compare-and-swap linearized; concurrent forks are rejected rather than timestamp-resolved. 3. **Sixteen moving route records could not form an atomic county decision.** - Added jurisdiction, wave, and state coverage-gate revisions. - A gate binds all exact route/subject/review revisions under one canonical basis hash. - Catalog `DONE` now requires a currently valid, independently reviewed, passed jurisdiction gate. 4. **Passed gates had no deterministic expiry or hash semantics.** - Added canonical ordering/serialization, declared hash algorithm, `basis-as-of`, `freshness-deadline`, and `valid-through`. - Current completion becomes false immediately after validity expires or a bound review is invalidated; it does not wait for a later mutation. 5. **Record interpretation depended on an implied document version.** - Defined runtime source-record contract `star.govdata.source-profile@1.0.0` separately from design-document versions, payload schemas, envelope protocol, applicability contracts, and policy snapshots. - Commands, jobs, results, checkpoints, canonical records, reviews, and gates now pin it explicitly. 6. **Adapter jobs incorrectly mixed logical work with leases issued later.** - Durable jobs contain immutable logical scope and contracts, but no future lease. - Each execution/retry creates a separate immutable attempt with lease generation and fencing token. 7. **Artifacts and checkpoints could race.** - The protocol now commits artifact bytes/hash and retrieval events before evidence/proposals and checkpoint advancement. 8. **The catalog completion contract was generated and would be overwritten.** - Updated `tools/generate-government-jurisdiction-todos.py` to generate the revised `0.4.0` master contract. - Added generator validation for the atomic gate, reviewed source contract, and `valid-through` requirements. - The 16 jurisdiction shards remain structurally unchanged. ### Current validation state - all eight original inline threads remain resolved - full-diff scan found no stale mutable `review-state`, logical `source-profile-id`, or old `1.3` protocol/source-contract fields - PR #53 is open, draft, and mergeable against its current stacked base - head `cc2fea6` has no Actions runs and no commit statuses ### Remaining blockers 1. PR #48 is still draft and non-mergeable; #52 and #53 remain stacked drafts. 2. `PIPELINE-003` and the normative `PIPELINE-004` amendment require joint review/approval. 3. The machine-readable sixteen-route applicability fixture still needs generation. 4. Franklin needs subject/review/gate, canonical-hash, expiry, and supersession fixtures plus its runbook. 5. The generator reproduction check and repository CI must run after the stack is repaired and rebased. **Disposition remains: keep draft; do not merge yet.**
lost-rob0t (Migrated from github.com) reviewed 2026-08-02 20:01:32 +00:00
lost-rob0t (Migrated from github.com) left a comment

Current-head review — aggregate branch

Reviewed the consolidated main-based head b327476 after merging the former #48 → #52 → #53 stack.

Resolved in this head

  • stale stacked-base conflict chain removed by merging current main and overlaying only the aggregate PR files
  • shared adaptive-research and StarLang indexes reconciled with current main
  • canonical unit/component and applicability-contract revision identities defined
  • durable evidence-delivery intents added to the normative protocol
  • result/checkpoint delivery state and terminal-success rules defined
  • all sixteen Ohio routes represented in a machine-readable applicability fixture
  • operational metadata explicitly derived and prohibited from emitting standalone jobs
  • Franklin identity, review, delivery, fencing, recovery, hash, expiry, invalidation, and supersession fixtures added
  • Franklin execution runbook added
  • offline generator reproduction and contract validation added to GitHub Actions

Review result

No remaining design inconsistency was found in the addressed blocker set. Merge remains gated on all applicable workflow runs completing successfully. This is a comment review because the connected account is the PR author and cannot provide an independent GitHub approval.

## Current-head review — aggregate branch Reviewed the consolidated `main`-based head `b327476` after merging the former #48 → #52 → #53 stack. ### Resolved in this head - stale stacked-base conflict chain removed by merging current `main` and overlaying only the aggregate PR files - shared adaptive-research and StarLang indexes reconciled with current `main` - canonical unit/component and applicability-contract revision identities defined - durable evidence-delivery intents added to the normative protocol - result/checkpoint delivery state and terminal-success rules defined - all sixteen Ohio routes represented in a machine-readable applicability fixture - operational metadata explicitly derived and prohibited from emitting standalone jobs - Franklin identity, review, delivery, fencing, recovery, hash, expiry, invalidation, and supersession fixtures added - Franklin execution runbook added - offline generator reproduction and contract validation added to GitHub Actions ### Review result No remaining design inconsistency was found in the addressed blocker set. Merge remains gated on all applicable workflow runs completing successfully. This is a comment review because the connected account is the PR author and cannot provide an independent GitHub approval.
lost-rob0t (Migrated from github.com) reviewed 2026-08-02 20:02:17 +00:00
lost-rob0t (Migrated from github.com) left a comment

Final-head review refresh

Reviewed final head 022648c after the StarLang CI boundary correction.

The change removes execution of prototype files that were already deleted from this repository and replaces it with validation that:

  • no alternate StarLang implementation exists here;
  • the canonical Common Lisp runtime repository is explicitly recorded;
  • no live Lisp implementation remains under the historical prototypes/star-lang path.

The government-data validation suite passed on the immediately preceding merged head, including contract/fixture checks and byte-for-byte offline regeneration of all 17 jurisdiction files. Final merge remains gated on workflows attached to this exact head.

## Final-head review refresh Reviewed final head `022648c` after the StarLang CI boundary correction. The change removes execution of prototype files that were already deleted from this repository and replaces it with validation that: - no alternate StarLang implementation exists here; - the canonical Common Lisp runtime repository is explicitly recorded; - no live Lisp implementation remains under the historical `prototypes/star-lang` path. The government-data validation suite passed on the immediately preceding merged head, including contract/fixture checks and byte-for-byte offline regeneration of all 17 jurisdiction files. Final merge remains gated on workflows attached to this exact head.
lost-rob0t (Migrated from github.com) reviewed 2026-08-02 20:13:01 +00:00
lost-rob0t (Migrated from github.com) left a comment

Final current-head review

Reviewed final head 5655d53 after resolving both Pages failures.

Final CI fixes

  • added an org-id fallback when a valid file-level ID is absent from the Org-roam node table
  • normalized legacy implicit PlantUML component targets before rendering
  • retained strict failure for genuinely unresolved IDs and genuinely invalid diagrams

Validation result

All workflows on this exact head completed successfully:

  • Government Data Design
  • Star-Lang Common Lisp Research
  • Child Identity Firewall
  • Kindle EPUB
  • Org-roam Pages, including the site build and internal-link check

No unresolved review threads remain. The aggregate PR is ready to merge subject to GitHub's final mergeability check. This is a comment review because the connected account is the PR author and cannot provide an independent approval.

## Final current-head review Reviewed final head `5655d53` after resolving both Pages failures. ### Final CI fixes - added an `org-id` fallback when a valid file-level ID is absent from the Org-roam node table - normalized legacy implicit PlantUML component targets before rendering - retained strict failure for genuinely unresolved IDs and genuinely invalid diagrams ### Validation result All workflows on this exact head completed successfully: - Government Data Design - Star-Lang Common Lisp Research - Child Identity Firewall - Kindle EPUB - Org-roam Pages, including the site build and internal-link check No unresolved review threads remain. The aggregate PR is ready to merge subject to GitHub's final mergeability check. This is a comment review because the connected account is the PR author and cannot provide an independent approval.
Sign in to join this conversation.
No description provided.