Skip to content

docs: align runtime architecture and document 2119 workflows - #32

Merged
tylerwillis merged 1 commit into
mainfrom
codex/fix-workflow-documentation
Oct 29, 2026
Merged

tylerwillis merged 1 commit into
mainfrom
codex/fix-workflow-documentation

Conversation

@tylerwillis

@tylerwillis tylerwillis commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Problem and fix

The architecture and container docs describe an older runtime: a read-only checkout before slug assignment, three image tiers, and a Claude-only launcher. The three shipped 2119 workflows also lack dedicated documentation.

Align the docs with the current source: writable per-task clones before slug assignment, in-place branch provisioning, base/harness/workflow/repository image composition, layer ownership and freshness, harness-specific transport and credentials, and the complete container-status precedence rules. Add linked pages for 2119-human-spec, 2119-auto-spec, and 2119-auto-sol, covering their state flow, review gates, artifacts, and current default reviewer models.

This is a documentation-only change against the current implementation.

Validation

  • Checked 33 relative documentation links and anchors.
  • Source-backed independent documentation review passes, including a fresh review after correcting status precedence and adding paused/gated/healing.
  • Ruff lint/format, mypy, wheel build, and clean-wheel smoke pass.
  • The change-scoped 2119 gate passes with no affected requirements. The full gate retains the 203 pre-existing verdict failures/staleness on main; no requirements or verdicts are changed here.
  • Local broad-suite run on baseline source (documentation branch): 3819 passed, 5 skipped, 4 failed. Three directory-name-sensitive assertions pass in a clean baseline checkout; the remaining native-Claude MCP test also fails on clean main because the CLI sends two valid direct requests while the assertion expects exactly one. These unrelated baseline/test-environment issues are not changed here.
  • GitHub CI and both Linux/macOS startup smoke jobs pass on this PR head.

Replace stale read-only checkout and three-tier image descriptions with the current writable clone, in-place branch provisioning, and base/harness/workflow/repository composition. Correct layer ownership, source freshness, harness transport and credentials, and the full container-status selection rules.

Add linked pages for all three 2119 workflows covering lifecycle, human gates, reviewer defaults and evidence, repository overrides, artifacts, and the distinction between recorded responsibilities and actual test execution.

Validation: all 33 local links and anchors pass. Source-backed independent review passes, including a fresh review of corrected status precedence. Ruff, format, mypy, wheel build, and clean-wheel smoke pass. The change-scoped 2119 check has zero affected requirements; the full gate retains 203 pre-existing verdict violations.
@tylerwillis
tylerwillis merged commit 4f5b13d into main Oct 29, 2026
3 checks passed
@tylerwillis
tylerwillis deleted the codex/fix-workflow-documentation branch October 29, 2026 07:55
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