-
Notifications
You must be signed in to change notification settings - Fork 1
refactor(ddd): restore contextual-orchestrator ownership boundary #899
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
seonghobae
wants to merge
30
commits into
main
Choose a base branch
from
fix/contextual-orchestrator-owner-boundary
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
88ad120
test(ddd): forbid LineageWeave-owned provider boundary
seonghobae 31e0b8c
refactor(ddd): remove LineageWeave-owned orchestrator bootstrap
seonghobae 49da27d
refactor(ddd): remove LineageWeave-owned model agent config
seonghobae e3a954b
refactor(ddd): remove embedded orchestrator image
seonghobae 78f0945
refactor(ddd): keep provider config outside LineageWeave
seonghobae 5e17f36
refactor(ddd): consume external contextual-orchestrator
seonghobae 5b99ba0
fix(llm): require contextual-orchestrator consumer contract
seonghobae 4a0a2c4
fix(llm): remove provider gateway fallback from estimation
seonghobae 3530fa1
test(ddd): retire embedded orchestrator bootstrap tests
seonghobae 308e8e3
docs(adr): assign provider runtime to contextual-orchestrator
seonghobae 8779632
docs(changelog): record orchestrator ownership repair
seonghobae 7ff72bd
test(ddd): reject direct provider SDK imports
seonghobae 736ff3e
refactor(ddd): stop loading provider env into Compose
seonghobae dc9128b
test(ddd): keep provider env out of Compose launcher
seonghobae b2b8812
docs(ddd): align ecosystem ownership with canonical services
seonghobae eac6674
docs(ddd): define LineageWeave bounded-context map
seonghobae 9318c4a
docs(ddd): define measurement ubiquitous language
seonghobae ccd626f
test(ddd): lock measurement ownership vocabulary
seonghobae 2368b90
fix(ddd): align repository guidance with owner boundary
379c44b
docs(ddd): correct historical vision boundary note
seonghobae 3295d8f
chore(ddd): restack orchestrator boundary on live main
seonghobae 668e053
chore(ddd): restack orchestrator boundary on protected main
seonghobae 47ebcee
chore(ddd): converge orchestrator boundary with protected main
seonghobae 1f70eb5
test(ci): require current draft admission contract
seonghobae b008a4f
fix(ci): adopt protected Tests draft admission
seonghobae 3780bfa
fix(ci): adopt protected PROV-O draft admission
seonghobae 6dbf089
fix(ci): adopt protected Ontology Pages draft admission
seonghobae e571128
chore(stack): converge contextual-orchestrator boundary onto protecte…
seonghobae d331d1f
docs: remove trailing whitespace from domain glossary
seonghobae a2da587
docs(adr): keep owner-boundary decision proposed until merge
seonghobae File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| ## Changed | ||
|
|
||
| - LineageWeave now consumes contextual-orchestrator only through `ORCHESTRATOR_BASE_URL` and `ORCHESTRATOR_API_KEY`; provider endpoints, provider credentials, model-agent bootstrap, discovery and fallback remain owned by contextual-orchestrator. | ||
| - The default Compose stack no longer embeds or starts a LineageWeave-owned contextual-orchestrator runtime. Missing orchestration configuration leaves model-backed channels unavailable instead of falling back to a provider gateway. | ||
| - Added architectural fitness coverage that rejects provider-boundary configuration and an embedded orchestrator runtime in LineageWeave. |
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
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
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
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
🔎 Supported by static analysis
🏁 Script executed:
Repository: ContextualWisdomLab/LineageWeave
Length of output: 12376
make up서비스 목록을 정확히 기록하세요.make up은postgres,database_migration,valkey,searxng,keycloak,backend,frontend를 시작합니다. 현재 주석은database_migration과searxng을 생략합니다. 두 서비스를 목록에 추가하거나, 현재 목록을 “핵심 장기 실행 서비스”라고 명시하세요.🤖 Prompt for AI Agents