Skip to content

docs: align README exec guidance with durable-by-default persistence#167

Closed
andrei-hasna wants to merge 2 commits into
mainfrom
openloops/open-codewith/9b038ca7-a443-4750-8cf0-c6c7b4bf9107-609f1806
Closed

docs: align README exec guidance with durable-by-default persistence#167
andrei-hasna wants to merge 2 commits into
mainfrom
openloops/open-codewith/9b038ca7-a443-4750-8cf0-c6c7b4bf9107-609f1806

Conversation

@andrei-hasna

Copy link
Copy Markdown
Contributor

Summary

Aligns the codewith exec onboarding text in codex-rs/README.md with the durable persistence contract already documented in docs/exec.md.

Previously codex-rs/README.md:54 read:

Use codewith exec --ephemeral ... to run without persisting session rollout files to disk.

This framed --ephemeral as the generic programmatic exec mode, nudging automation authors toward lifecycle bypass instead of durable headless execution.

The README now states the same contract as docs/exec.md:

  • Headless codewith exec persists session rollout files by default so long-running automation can resume/inspect the thread later.
  • --durable/--persist state that contract explicitly.
  • --ephemeral is only for intentional one-off runs that should not be materialized on disk.

Scope

Docs-only. No product code changes. Single file: codex-rs/README.md.

Validation

  • rg -n -- "--ephemeral|--durable|--persist|persist session|persisting session" docs/exec.md codex-rs/README.md — both docs now describe the same contract.
  • git diff --check — clean.
  • git diff --name-status HEAD^..HEAD — only codex-rs/README.md.

Fingerprint: open-codewith/architecture-drift/exec-docs/headless-exec-persistence-guidance/a6ff8387f3ad5afb

codex-rs/README.md still framed `codewith exec --ephemeral` as the generic
way to run without persisting session rollout files, nudging automation
authors toward lifecycle bypass. Align the exec section with docs/exec.md:
headless exec persists rollout files by default, --durable/--persist state
that contract explicitly, and --ephemeral is only for intentional one-off
runs that should not be materialized on disk.
@andrei-hasna

Copy link
Copy Markdown
Contributor Author

[codewith] Worker merge check update: I refreshed this PR branch to current main via gh pr update-branch. New head is 1d7523c8802038b2afe6d68a67cd5f35a1cfa66d; base is 3c77b665654ddb7ee706423859c114e8a51dc6d2. The diff remains README-only (codex-rs/README.md, 1 insertion/1 deletion), aligns with docs/exec.md, and git diff --check is clean.\n\nNot merged: fresh required CI is red. cargo-deny fails on RUSTSEC-2026-0204, which is unrelated to this docs line. Existing task 34ae34a8 / PR #174 is already tracking the crossbeam-epoch remediation. Please recheck/merge #167 after that advisory/check blocker lands, unless an explicit docs-only required-check override is approved.

@andrei-hasna

Copy link
Copy Markdown
Contributor Author

Closing as superseded by #207. Replacement #207 carries the current provider/docs cleanup slice and explicitly lists this PR in its carried scope. Please continue review on #207.

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