docs(adr): ADR-0078 — first-class knowledge layer (design for #1175)#1264
Open
ohdearquant wants to merge 1 commit into
Open
docs(adr): ADR-0078 — first-class knowledge layer (design for #1175)#1264ohdearquant wants to merge 1 commit into
ohdearquant wants to merge 1 commit into
Conversation
ohdearquant
added a commit
that referenced
this pull request
Jun 3, 2026
…ollision) The play numbered its 4 ADRs 0077-0080, but on sibling open PRs ADR-0077 is taken by orch-emissions (lifecycle-signal-contract) and ADR-0078 by #1264 (first-class-knowledge-layer). Shift the whole cluster +2 so it merges without colliding: - ADR-0079 substrate-executor-provider-interface (#1196) - ADR-0080 remote-sandbox-substrate-execution (#1195) - ADR-0081 configurable-flow-planning (#1197) - ADR-0082 role-substrate-routing-policy (#1210) All inter-ADR cross-references updated; ADR-0074/0072/0057 (on main) unchanged. The #1210 routing slice (flow/fanout --pack routing display + tests) is carried from the prior commit; tests/cli/orchestrate/test_flow_planning.py → 16 passed. Refs #1195 #1196 #1197 #1210 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
ohdearquant
added a commit
that referenced
this pull request
Jun 11, 2026
… slice (#1267) * docs(adr): ADR-0079 + ADR-0080 — substrate executor + remote sandbox seam Two design ADRs (Status: Proposed) for the orchestration substrate: - ADR-0079 (#1196): typed executor-provider contract — an ExecutorProvider protocol + a shared ExecutionTarget data type, retrofitting the existing claude_code / codex / API endpoints and adding an arbitrary-process provider. No parallel executor registry; reuses EndpointRegistry/iModel/run(). - ADR-0080 (#1195): remote sandbox substrate execution — extend the existing lionagi/tools/sandbox.py + daytona.py behind the same ExecutionTarget so a flow/play op can run remotely (Daytona) and stream typed events back. Local default path stays byte-for-byte unchanged. Grounded in real primitives (iModel, EndpointRegistry, AgenticEndpoint, SandboxSession, DaytonaSandbox) with cited file:line; shared lionagi/substrate/types.py contract; alternatives + failure modes + migration. Refs #1195 #1196. Related #1197 (configurable planning) and #1210 (casts role→substrate routing) — inventoried by this play but no ADR written yet. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * docs(adr): substrate ADR cluster 0077-0080 + role→substrate routing slice Design ADRs for the heterogeneous orchestration substrate, with the executor/sandbox ADRs renumbered 0079/0080 → 0077/0078 to resolve a numbering collision and validate cross-references: - ADR-0077 substrate executor provider interface (#1196) - ADR-0078 remote sandbox substrate execution (#1195) - ADR-0079 configurable flow planning (#1197) - ADR-0080 role→substrate routing policy (#1210) Also ships the #1210 thin reference slice: roles carry an optional --pack routing target consumed by flow/fanout DAG planning, with a focused test (tests/cli/orchestrate/test_flow_planning.py). Refs #1195 #1196 #1197 #1210 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * docs(adr): renumber substrate cluster 0079-0082 (avoid cross-PR ADR collision) The play numbered its 4 ADRs 0077-0080, but on sibling open PRs ADR-0077 is taken by orch-emissions (lifecycle-signal-contract) and ADR-0078 by #1264 (first-class-knowledge-layer). Shift the whole cluster +2 so it merges without colliding: - ADR-0079 substrate-executor-provider-interface (#1196) - ADR-0080 remote-sandbox-substrate-execution (#1195) - ADR-0081 configurable-flow-planning (#1197) - ADR-0082 role-substrate-routing-policy (#1210) All inter-ADR cross-references updated; ADR-0074/0072/0057 (on main) unchanged. The #1210 routing slice (flow/fanout --pack routing display + tests) is carried from the prior commit; tests/cli/orchestrate/test_flow_planning.py → 16 passed. Refs #1195 #1196 #1197 #1210 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * chore: add sweep summary Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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
1 participant
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.
Design play from the
lionagi-sweepshow — an ADR proposing a first-class knowledge substrate for review, NOT an implementation. Critic APPROVE (CRIT:0 MAJ:0 MIN:2): every cited primitive verified against live source; OSS/commercial boundary confirmed clean (zero khive/commercial references — the claim/evidence vocabulary is OSS prior art from the existing ADR-0039, referenced and deferred).ADR-0078 covers
Graph.get_predecessors/get_successors/find_node_edge/find_path,ID.Ref).operate/ReActread/write knowledge; relationship toGraph/Node; serialization.This is the seam, not the implementation — for your design sign-off before any build. Leaving #1175 open pending that approval.
Refs #1175 (design ADR for review)
🤖 Generated with Claude Code