Skip to content

docs(track-a): define canonical live runtime lease model - #311

Merged
blakinio merged 8 commits into
mainfrom
docs/OTC-20260815-track-a-canonical-live-runtime
Aug 16, 2026
Merged

docs(track-a): define canonical live runtime lease model#311
blakinio merged 8 commits into
mainfrom
docs/OTC-20260815-track-a-canonical-live-runtime

Conversation

@blakinio

@blakinio blakinio commented Aug 15, 2026

Copy link
Copy Markdown
Owner

Track A canonical-live governance finalization, clean-restacked on final current main@b0fd474e34c0252220b773b2304d889821080727 after the cancellation-safe manager and fresh manager closeout reached terminal main state.

Final boundaries:

  • Gate A: current authoritative lease plus the final PR fix(track-a): retain canonical lease through process-group cancellation #321 cancellation-safe out-of-band Linux child-subreaper supervisor. It acquires coordination.lock, validates the current lease under that flock, keeps the flock out-of-band for the complete mutation/process-tree lifetime, launches mutation descendants with no flock FD, and does not drop serialization merely because the foreground process group is cancelled while a guarded descendant survives.
  • Registration generation rebind: an authoritative registration from an older controller generation is not ordinary Gate B-ready. A dedicated reviewed transition must run under current Gate A + the canonical flock, freshly prove the same boot/PID/start/exact-fence/display/window/state and unique target, then atomically increment registration_generation and bind lease_generation to the current controller. It cannot create a missing registration, bless a new/reused PID, repair a changed fence or ambiguous client, or mutate the client. This governance PR defines but does not implement that primitive.
  • Gate B: after any required rebind, the one authoritative /home/runner/_work/_otclient_tibia_re_state/canonical-live-runtime/runtime-registration.json must match the current lease generation and pass fresh boot/PID/start/exact-fence/display/window/state plus target-uniqueness preflight before ordinary reuse/mutation.
  • Initial creation: remains the separate fail-closed TRACK_A_CANONICAL_LIVE_BOOTSTRAP_V1.md transition originally promoted by PR docs(track-a): define canonical live bootstrap transition #318/docs(track-a): close out canonical live bootstrap contract #320 and reconciled here to PR fix(track-a): retain canonical lease through process-group cancellation #321/docs(track-a): archive final cancellation-safe lease manager #322 plus the post-bootstrap rebind boundary. Ordinary Gate B/rebind is not weakened to create the first runtime.

A deterministic GitHub-hosted Track A governance acceptance audit now falsifies these required invariants on every relevant PR head. Repository required CI, the fresh audit and zero unresolved material review findings are required before protected merge.

Exact fence remains 15.32.df7b29 / 51965216 / e6c244bd39fe2e0632f6f000efd3147164696efa8e901718668e0442325ff7fe. Current :98, 6082, PID and session status remains UNKNOWN / NOT_REGISTERED until direct evidence.

No live client launch/login/mutation, no registration/rebind runtime execution, no credentials, no PR #303 runtime-owned process/path access, no Track B mutation, no protection/security weakening, and no owner-funded Codex/OpenAI API quota use are part of this governance PR.

@blakinio
blakinio marked this pull request as ready for review August 15, 2026 19:58
@blakinio
blakinio enabled auto-merge (squash) August 15, 2026 19:58
@blakinio blakinio added the programme:client Oteryn client programme label Aug 15, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cf361a44dd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/agents/TIBIA_RESEARCH_TRACKS.md Outdated
@blakinio
blakinio marked this pull request as draft August 15, 2026 20:10
auto-merge was automatically disabled August 15, 2026 20:10

Pull request was converted to draft

@blakinio
blakinio force-pushed the docs/OTC-20260815-track-a-canonical-live-runtime branch from 0b7683b to fca15ac Compare August 15, 2026 21:10
@blakinio
blakinio marked this pull request as ready for review August 15, 2026 21:12

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: af81291066

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/agents/TIBIA_RESEARCH_TRACKS.md Outdated
Comment thread docs/agents/TIBIA_RESEARCH_TRACKS.md
@blakinio
blakinio marked this pull request as draft August 15, 2026 21:20
@blakinio
blakinio marked this pull request as ready for review August 15, 2026 21:20
@blakinio
blakinio enabled auto-merge (squash) August 15, 2026 21:22

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: af81291066

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/agents/TIBIA_RESEARCH_TRACKS.md Outdated
@blakinio
blakinio marked this pull request as draft August 15, 2026 21:25
auto-merge was automatically disabled August 15, 2026 21:25

Pull request was converted to draft

@blakinio
blakinio force-pushed the docs/OTC-20260815-track-a-canonical-live-runtime branch from 58fe771 to 91ec453 Compare August 15, 2026 21:29

Copy link
Copy Markdown
Owner Author

Post-merge P1 findings are now satisfied by merged PR #313 (f6fa2264904c6ffb3734d4a63e1edbb29260fcc1). #313 specifically (1) samples time-sensitive lease validity only after acquiring the coordination flock and (2) preserves the live flock FD in surviving guard-run children so controller serialization remains held if the wrapper parent exits. Final policy v4 explicitly binds Gate A to the manager from #312 plus #313 remediation, while Gate B separately requires current runtime-identity registration before reuse/mutation. These fixes do not register :98 or any runtime as canonical.

@blakinio
blakinio marked this pull request as ready for review August 15, 2026 21:32

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5a491373f0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/agents/TIBIA_RESEARCH_TRACKS.md Outdated
Comment thread docs/agents/TIBIA_RESEARCH_TRACKS.md Outdated
@blakinio
blakinio marked this pull request as draft August 15, 2026 21:42
@blakinio
blakinio marked this pull request as ready for review August 15, 2026 21:43

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6f8774e522

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/agents/TIBIA_RESEARCH_TRACKS.md Outdated

Copy link
Copy Markdown
Owner Author

New material governance finding from read-only PR #315: current persistent :98 exists, but final run 31910131938 / job 95073832354 found zero visible/hidden Tibia windows and zero exact-fenced client processes globally. Therefore there is no live runtime to register/reuse now.

Policy v4 currently says Gate B must prove a selected live process before the first mutation/reuse, while Gate A says every canonical mutation must stay in guard-run for the full mutation-descendant lifetime. With no live process this creates a bootstrap deadlock: identity cannot exist before launch, and a naive guard-run launch of a long-lived client would retain the controller flock for the client's whole lifetime rather than transition to an idle reusable runtime.

Fail-closed resolution should distinguish reuse from initial creation. Initial canonical-session creation must remain disabled until a reviewed bootstrap primitive can, under current lease, create the exact-fenced process in the declared canonical namespace, prove/register its PID+start identity+display/window/fence, and then end the mutation lock without leaving an untracked mutation child. Do not weaken this into standalone validate + detached launch. PR #316 supervisor remediation remains a prerequisite for ordinary guarded mutations; this bootstrap transition is a separate follow-up boundary. :98 remains candidate only.

@blakinio blakinio closed this Aug 16, 2026
@blakinio
blakinio force-pushed the docs/OTC-20260815-track-a-canonical-live-runtime branch from 6f8774e to 150460e Compare August 16, 2026 03:26
@blakinio blakinio reopened this Aug 16, 2026
@blakinio
blakinio marked this pull request as ready for review August 16, 2026 03:34

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 71045e0283

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/agents/TIBIA_RESEARCH_TRACKS.md Outdated
Comment thread docs/agents/decisions/ADR-0001-track-a-canonical-live-runtime.md Outdated
@blakinio
blakinio force-pushed the docs/OTC-20260815-track-a-canonical-live-runtime branch from 71045e0 to 0ad4759 Compare August 16, 2026 05:36
@blakinio
blakinio enabled auto-merge (squash) August 16, 2026 05:38
@blakinio
blakinio merged commit f180378 into main Aug 16, 2026
11 checks passed
@blakinio
blakinio deleted the docs/OTC-20260815-track-a-canonical-live-runtime branch August 16, 2026 05:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

programme:client Oteryn client programme

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant