Skip to content

Prepare controlled-session cleanup manifest - #33

Draft
omry wants to merge 1 commit into
pr32from
pr33
Draft

Prepare controlled-session cleanup manifest#33
omry wants to merge 1 commit into
pr32from
pr33

Conversation

@omry

@omry omry commented Aug 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • derive an immutable cleanup manifest from durably recorded session ownership
  • bind it to the exact prepared containers and private channel before startup
  • keep absent lease networks and volumes explicit and reject later resource selection
  • use strict canonical encoding without a separate schema-version marker

Why

The upcoming short-lived watchdog must receive a frozen cleanup target instead of accepting resource choices after startup.

Validation

  • full Go test suite
  • focused race-enabled ownership and manifest tests
  • live Docker controlled-session integration test

@omry
omry changed the base branch from main to pr32 August 9, 2026 10:05
@omry

omry commented Aug 9, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit: ed5f4a187b

ℹ️ 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".

@omry

omry commented Aug 9, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Delightful!

Reviewed commit: da288dd3a3

ℹ️ 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".

@omry omry added the approved PR reviewed and approved label Aug 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved PR reviewed and approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant