Skip to content

docs: define next architecture roadmap - #50

Open
Alajmah wants to merge 3 commits into
masterfrom
docs/next-architecture-roadmap
Open

Alajmah wants to merge 3 commits into
masterfrom
docs/next-architecture-roadmap

Conversation

@Alajmah

@Alajmah Alajmah commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Replaces the mostly-historical active roadmap with the next architecture execution plan agreed after the current master deep-dive.

What changes

  • makes docs/ROADMAP.md the authoritative forward plan starting at Phase 8
  • archives completed Phases 0–7 and their carried-forward principles in docs/ROADMAP-HISTORY.md
  • aligns docs/INDEX.md with the active-vs-historical split

Active sequencing

  1. stabilize current reality
  2. canonical runtime model
  3. ChatGPT Web compatibility contract
  4. transport-neutral session runtime built from the existing MCP pool
  5. centralized recovery + bounded backpressure
  6. attachment subsystem with explicit host-file security boundaries
  7. compatibility laboratory / record-replay / minimal live certification
  8. hardened operation
  9. feature expansion

Important gates captured in the roadmap

  • re-certify the August fresh-chat regression before assuming it recovered
  • restore green master and truthful release/docs state first
  • review feat/sse-pool-status-endpoint before designing another MCP health path
  • do not merge PR feat: add ChatGPT file uploads #49 with unrestricted server-local paths, post-spool-only limits, driver-global upload state, or a non-content-dependent E2E
  • preserve the no-auto-resend invariant and zero-tolerance for silent wrong-turn responses
  • generalize the existing MCP session pool rather than building a second pool

This PR is documentation/planning only; it intentionally makes no runtime changes.

@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: 1eedbfaa69

ℹ️ 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/ROADMAP.md
capacity accounting, and account-level throttling;
- reactive drift diagnostics and redacted evidence capture.

See [ROADMAP-HISTORY.md](ROADMAP-HISTORY.md) for the historical phase record.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Retarget references to archived roadmap phases

Archiving Phases 0–7 here leaves existing references pointing at the wrong document: docs/os-supervision.md still links to ROADMAP.md for Phases 3 and 4, while docs/architecture.md and CHANGELOG.md direct readers to its Phase 5 audit. Those sections no longer exist in the active roadmap, so operators following the supervision policy and maintainers looking for the extraction rationale land on unrelated Phase 8+ planning. Update these inbound references to ROADMAP-HISTORY.md or to the immutable archived commit as part of the move.

Useful? React with 👍 / 👎.

@Alajmah Alajmah left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified follow-up for Phase 8.2: current master has 9 Ruff errors across three files (src/chatgpt_web2api/cdp_driver.py, tests/test_composer_verification.py, and tests/test_send_acknowledgment.py). PR #48 covers only the last file and also damages the explanatory module docstring, so it should not be treated as the complete green-master fix. Execute 8.2 as one human-authored lint cleanup across all three files, preserving test rationale/docstrings, then close #48 as superseded once the replacement PR exists. This does not change the roadmap sequencing or block review of this docs-only PR.

@gitwire-hq gitwire-hq Bot added the size/XL label Sep 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant