docs(spec-064): refresh stale references after monolith splits#4959
Merged
Conversation
Correct code anchors that drifted after the agent-loop monolith splits and update planning metadata now that the implementation epic and child issues exist. - GitHub/Branch headers point to epic #4707 and the per-issue feat/m28/* series - P1 agent-loop adapter: tier_loop.rs / execute_tier_join is in zeph-core (src/agent/tool_execution/tier_loop.rs), not zeph-agent-tools - P2: plan.rs / handle_plan_resume_as_string is in zeph-core; resume_from at zeph-orchestration/src/scheduler/mod.rs:380 - JobStore::init precedent: store.rs:137 - NFR-DE-01 harness: per-crate zeph-durable/benches/, not zeph-bench - spec-039 reconciliation: reference zeph-common::TaskSupervisor over bare JoinSet - Persistence Schema: note the pre-existing sqlite/postgres migration parity divergence (#4957) blocking the durable migration numbering Refs #4707, #4957
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Refreshes
specs/064-durable-execution/spec.mdafter an SDD currency + traceability review against HEADb15adeb9. The spec was authored before the agent-loop monolith splits (#4933, #4938) and before the implementation epic was filed, so several code anchors and planning-metadata fields had drifted. No semantics change — only references, paths, and one new caveat.The review verdict was that the spec is substantively sound and fully traceable (all 15 FR + 14 INV + 7 NFR are covered by the child issues #4944–#4954, no orphaned IDs). These edits fix the stale references that would otherwise mislead implementors.
Changes
**GitHub:**→ epic implement native durable execution layer (zeph-durable, spec 064) #4707 + child issues feat(durable): scaffold zeph-durable crate, core newtypes, journal schema #4944–feat(sub-agents): durable promise for subagent spawn/await resume (P4) #4954 (M28);**Branch:**→ per-issuefeat/m28/*series.tier_loop.rs/execute_tier_joinis inzeph-core/src/agent/tool_execution/tier_loop.rs(L1705), notzeph-agent-tools— the loop never moved tozeph-agent-tools; the monolith splits relocated it withinzeph-core. Corrected in the §P1 header/body, the §Scope adapter list, the layer-placement rationale, the LLM-gate note, and §Three Hard Problems.plan.rs/handle_plan_resume_as_stringis inzeph-core(def L1006, dispatch L1161) →resume_loaded_graph(L935) →DagScheduler::resume_fromatzeph-orchestration/src/scheduler/mod.rs:380.JobStore::initline ref136→137.crates/zeph-durable/benches/(workspace convention), notzeph-bench(the agent-eval harness).zeph-common::TaskSupervisor(the unifiedJoinSetwrapper) instead of a bareJoinSet.075), tracked in zeph-db sqlite/postgres migration sequences diverge (file count + content parity) #4957 — it blocks the durable migration numbering in C1 (feat(durable): scaffold zeph-durable crate, core newtypes, journal schema #4944).Verification
Docs-only change (single markdown file, no Rust code or manifests touched) —
cargofmt/clippy/test gates are not applicable. Diff:1 file changed, +26 / −15.Related
.local/handoff/sdd-review-spec-064.md