Skip to content

refactor(ddd): restore contextual-orchestrator ownership boundary - #899

Draft
seonghobae wants to merge 23 commits into
mainfrom
fix/contextual-orchestrator-owner-boundary
Draft

refactor(ddd): restore contextual-orchestrator ownership boundary#899
seonghobae wants to merge 23 commits into
mainfrom
fix/contextual-orchestrator-owner-boundary

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Scope

Restores the DDD/source-of-truth boundary between LineageWeave and ContextualWisdomLab/contextual-orchestrator.

LineageWeave owns product measurement/evidence policy and consumes the published orchestrator contract. Provider credentials, provider endpoints, model-agent bootstrap, provider/model discovery, routing and fallback remain in contextual-orchestrator.

Changes

  • architectural fitness regression rejects LineageWeave-owned provider configuration or an embedded orchestrator runtime;
  • removes docker/contextual-orchestrator/{Dockerfile,agents.json,start.py} and its obsolete bootstrap test;
  • removes the local orchestrator Compose service; backend/MCP consume only externally supplied ORCHESTRATOR_BASE_URL / ORCHESTRATOR_API_KEY;
  • removes provider/gateway settings from .env.example;
  • Keyman and LLM channel-weight operator paths refuse provider-gateway credential fallbacks;
  • ADR 0300, Context Map, ubiquitous-language and changelog evidence describe the consumer boundary;
  • historical Vision doctoring no longer presents provider-gateway variables as current LineageWeave configuration.

Fail-closed behavior

With no contextual-orchestrator consumer endpoint/credential configured, model-backed channels remain unavailable rather than calling a provider directly. No provider key or provider URL is accepted as an orchestrator credential.

TDD / review evidence

The architecture-fitness RED began at 88ad120a22fd286306e0ba49e66eb2b670c22ebc; subsequent commits remove the causes. A documentation audit then found docs/doctoring/ACTUAL_RUNTIME_EVIDENCE_2026-08-19_VISION.md still described LLM_GATEWAY_API_URL / LLM_GATEWAY_API_KEY as current LineageWeave configuration; 379c44b0eb09bfb8b3aa087acd7b7729482a4360 corrected that statement while preserving the historical 2026-08-19 experiment.

Protected main later advanced through #923 from 3f61c8242b9c02dec307a7396e83e28f7cdd9f3d to 0f1d44726cc8fd9fd05a4f9b91ec40ff6be8802d. The sole intervening delta is .github/workflows/tests.yml docs-only paths-ignore; this branch does not modify that workflow. The branch adopted the protected-main delta by ordinary two-parent merge commit 668e053c8e4cdc1b9092dba914e1cf796a683a45, preserving the ownership-boundary tree and avoiding force rewrite.

Exact live identity / governance

  • protected base: main@0f1d44726cc8fd9fd05a4f9b91ec40ff6be8802d;
  • exact current head: 668e053c8e4cdc1b9092dba914e1cf796a683a45;
  • GitHub reports Draft/open/mechanically mergeable;
  • exact-head OSV 33712534269, Tests 33712533872, SAST 33712533843, Scorecard 33712533851, and Security 33712533880 are queued and therefore non-passing;
  • predecessor checks, including the earlier dependency-review HTTP 403 evidence, do not transfer to this new head;
  • no qualifying formal approval is claimed for the changed head.

Keep Draft until unchanged-current-head security/governance evidence is terminal-clean and the required independent review is present. Do not self-approve, bypass, weaken gates, or inherit predecessor evidence.

@coderabbitai

coderabbitai Bot commented Sep 1, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Current-head local evidence for 2368b9052f33c77c33b72ac87264efd4b7b9cbff:

  • docker compose config --quiet passed; services no longer include an embedded orchestrator.
  • Owner-boundary, DDD fitness, client/default, and documentation checks: 39 passed.
  • Full collection: 1,889 passed, 17 skipped; 10 live API failures all reduce to the known migration-fixture drift already repaired on PR fix(voice): distinguish carrying and evidence exports #780.

This PR remains draft until #780 lands (or its exact fix is otherwise present) and current-head hosted checks/review are terminal. No provider credentials or production records were inspected.

seonghobae added a commit that referenced this pull request Sep 1, 2026
Merge the current #899 ownership-boundary head into the dichotomous measurement-policy stack without rewriting child history. The parent delta is the code-current Vision runtime note; the measurement policy delta remains unchanged.
Merge protected main@3f61c824 into the ownership-boundary root without rewriting history. The intervening main delta only adds the repository Pages landing source at docs/index.md and does not overlap the DDD boundary repair.
seonghobae added a commit that referenced this pull request Sep 1, 2026
Merge the current #899 root after it incorporated protected main@3f61c824. The only new parent delta is docs/index.md; the dichotomous measurement-policy semantic delta is unchanged.
@seonghobae seonghobae added the bug Something isn't working label Sep 2, 2026 — with ChatGPT Codex Connector
seonghobae added a commit that referenced this pull request Sep 2, 2026
Merge the current #899 contextual-orchestrator ownership foundation into #919 without rewriting either history. Preserve #919's operator naming/admission delta while adopting #899's DDD/docs/config removals; the overlapping Keyman script is reconciled in the immediately following causal commit.
Adopt protected main@0f1d4472 without rewriting the ownership-boundary branch. The intervening protected-main delta is limited to the Tests workflow docs-only paths-ignore change and does not overlap this PR's product/DDD files.
seonghobae added a commit that referenced this pull request Sep 3, 2026
Adopt #899668e053 after its non-force convergence to protected main@0f1d4472. Preserve the measurement-policy delta and current Tests workflow without rewriting branch history.
seonghobae added a commit that referenced this pull request Sep 3, 2026
Adopt #899668e053 after its protected-main convergence. Preserve the bounded operator policy delta and current Tests workflow without rewriting branch history.
Adopt #924's restored docs-contract test triggering while preserving the contextual-orchestrator ownership boundary without force rewrite.
seonghobae added a commit that referenced this pull request Sep 3, 2026
Adopt #89947ebcee after protected main #924 while preserving the dichotomous measurement-policy delta without force rewrite.
seonghobae added a commit that referenced this pull request Sep 3, 2026
Adopt #89947ebcee after protected main #924 while preserving bounded operator policy without force rewrite.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant