Skip to content

fix: retain sequential pool roll history and image bindings - #272

Merged
drewstone merged 1 commit into
mainfrom
fix/runtime-repeat-roll
Oct 23, 2026
Merged

drewstone merged 1 commit into
mainfrom
fix/runtime-repeat-roll

Conversation

@drewstone

Copy link
Copy Markdown
Owner

A completed pool roll currently rejects the next immutable artifact. The existing authenticated owner now starts a new transaction only from the completed candidate's exact ID after its slot is drained and idle, appends the previous receipt to history, and retains original adoption identities and stopped predecessors. Missing local images leave the completed journal unchanged. Status reports each current slot's immutable image binding, including mixed-image pools, for the existing security-report consumer.

Exact API-only head7b3509c2bdc68e608b309ac8187ada95f3193f9d has a maintenance module and real Docker/HTTP fixture byte-identical to qualified936748602e3ca28535b39441cc5cb11776cd95d9 (empty Git diff for both files). That Beelink2 frozen-install/typecheck/integration gate passed209s (beelink2-a2bb99ec5e50c9cddefb6c8842500c8b): real active work survives drain, unrelated slots serve, two successive artifacts preserve credential volumes/environment/resources, missing images leave the journal unchanged, stale intents refuse, and a real owner restart reconciles the latest candidate. Independent source review passed; this isolated change contains only the owner, its integration fixture and API documentation. No image dependency changes or claims of reduced live vulnerability counts.

Run ID: run_4b1d54cfdf2c05412b39044bf98511e1

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review 🔄 Running since 2026-10-23T01:37:15.998563Z 7b3509c PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@tangletools tangletools 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.

✅ Auto-approved PR — 7b3509c2

Blanket team auto-approval is intentional. This is not a code review.
No automated review runs on this PR. This approval rests on the rule above alone.

tangletools · auto-approval · reason: blanket_auto_approve · 2026-10-23T01:37:32Z

@drewstone
drewstone merged commit 407ba02 into main Oct 23, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants