docs(spec): architecture corrections cycle 3 — sandbox state persistence and LangGraph TypedDict requirement #5966

Closed
HAL9000 wants to merge 1 commits from spec/architecture-corrections-cycle3 into master

1 Commits

Author SHA1 Message Date
HAL9000 67dc076861 docs(spec): architecture corrections cycle 3 — sandbox state persistence and LangGraph TypedDict requirement
CI / push-validation (pull_request) Successful in 21s
CI / helm (pull_request) Successful in 25s
CI / lint (pull_request) Successful in 27s
CI / typecheck (pull_request) Successful in 52s
CI / build (pull_request) Successful in 3m19s
CI / quality (pull_request) Successful in 3m59s
CI / integration_tests (pull_request) Successful in 4m1s
CI / security (pull_request) Successful in 4m6s
CI / e2e_tests (pull_request) Successful in 7m59s
CI / unit_tests (pull_request) Successful in 8m42s
CI / docker (pull_request) Successful in 1m29s
CI / coverage (pull_request) Successful in 13m44s
CI / status-check (pull_request) Successful in 1s
CI / benchmark-publish (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Successful in 57m21s
- Add explicit sandbox state persistence requirement: SandboxManager must
  persist sandbox records to database (v3_sandboxes table) so that plan
  execute, diff, and apply can work across separate CLI invocations.
  Fixes spec gap identified by UAT issue #5721.

- Clarify LangGraph state schema requirement: GraphState passed to
  StateGraph() must be TypedDict, not Pydantic BaseModel. LangGraph
  requires TypedDict for state merging semantics and reducer annotations.
  Addresses architectural deviation identified in UAT issue #5587.

- Clarify RxPY role: reactive stream processing is for event routing
  BETWEEN actors, not for graph node execution. Graph node execution
  uses native LangGraph StateGraph exclusively. Addresses critical
  architectural deviation identified in UAT issue #5598.
2026-04-09 12:11:35 +00:00