Skip to content

refactor(operator): name bounded entry points and validate Keyman admissions - #919

Draft
seonghobae wants to merge 48 commits into
fix/contextual-orchestrator-owner-boundaryfrom
refactor/post-keymen-backfill-semantic-identifiers
Draft

refactor(operator): name bounded entry points and validate Keyman admissions#919
seonghobae wants to merge 48 commits into
fix/contextual-orchestrator-owner-boundaryfrom
refactor/post-keymen-backfill-semantic-identifiers

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Purpose

Repair verified LineageWeave-owned generic operator naming defects, make the post-Keyman backfill genuinely bounded, and keep model-backed work on the published contextual-orchestrator consumer boundary. Provider credentials/endpoints, model selection, routing and fallback remain outside LineageWeave.

RED → GREEN lineage

  • Post Keyman naming 7b4b5046aa136943124df965ba0cb630f8238b9c76335b36e68c5f42b46538c81d57e39b0b54599a;
  • channel-weight naming b8897a282e884b2e3420141db4c71c247153df8f6fa817f9253b5f23acf00b7967bfc8a19cb161d4;
  • thread-group naming 0247bf38dcfc4ab7131694d864d4180d458c2916b83abd7ad3fb12392e7782106a8fbeaaa68cb55d;
  • timeout admission e15c43f31ce21e42fb95757a773e577ef0336d3436d514b59e8b45dc84bcae0e1b2bfdb9ee3bf08a, plus overflow-totality d047a869b471363b435410a8f8919bd51bebcb2b5b49b923751cb1cae18a0e70ad94afbedff93e12;
  • selector admission/type/canonical UUID c3bf8505a8de0a2dca80ecf7f510abe7035e2602ab7807b890c349cbe5550e3637a97bfc50751d0a, f2ddd8372b4f0d6026cb6a19186a491ac87f86e7fc757f1a37cf13a3b361b40b75ea16515fdbff30, 21223c10dfd39f3833c2ba7d736c246d8231c156d0c0a05beff0a89493ce9057b5b4c1d53012bab1;
  • batch-limit/mode admission 25ca75981659d1a8ff8c3a462e45978646769c3a23387d7531de4d2cc5dcce46dd3d06b2a6f26e54, aace26ff5d870e98b51defed678384d32f014006e9934ba08398d222098dfc14ccd768343c0e6902, upper bound cf97ac5ac5e8b5b5ee1534e7d9c50e594dcf2364ab769da595b907aa61ba4b068fc35ca02e8b3097;
  • Keyman timeout coherence 4a22ed3bfa4fa5edefa8146a22f3f4f95b1714a575aefd7b9f3f987d4d2dd75eaa1c142e65b80ecb;
  • Vision timeout coherence 78f1633e96dbafc6f60ded80a80951aac3f3391e0f1f7f003700d43c8595117701e7b45b507b40e6, with 756f321d3493179776bfaa3da022c05fcb7c14ff and 63b292337f50c5b5127b7f778d4cb2404cae75ac carrying the admitted timeout to HTTP work;
  • cross-field selector admission 61d94c3afdeadc3634785181d6b40a1f55d50b23fa109e6fb117c985624310d2230dd779287d9b3d; non-default limits require explicit --all, and explicit post selection cannot be combined with batch mode. ADR 0082 is current at 933d28b99122daf8297108be4db2b65c474f7f3e, changelog at 39a4803e2e282a34e5f284a4815933c749196bb9.

Boundary / migration safety

The behavior remains LineageWeave-owned operator policy. Keyman and Vision consume contextual-orchestrator only through the released consumer boundary; schema, report keys, persistence semantics, transaction behavior and idempotent ingestion stay unchanged. Inputs fail before client creation/database access rather than being silently normalized or collapsed.

Current stack / merge boundary

Parent #899 moved non-force to e5711282c48cc20d0a88fb56a9e382d500989c72 after adopting protected main@83eba56149eb802cd63642c507c324c9976ec78e. Review 5118124470 verified this PR's 14 bounded-operator files do not overlap the four inherited workflow/contract repairs. Two-parent convergence caf8ee0f576eb23dc80d907fccfdcd6a4cecb232 preserves the bounded-operator tree, adopts the exact protected workflow contract, and joins the moved #899 parent without force rewrite.

  • exact parent/base: #899@e5711282c48cc20d0a88fb56a9e382d500989c72;
  • exact current head: caf8ee0f576eb23dc80d907fccfdcd6a4cecb232;
  • Draft/open/mechanically mergeable;
  • exact-head Tests 33923125661 completed skipped under Draft admission and is not product GREEN;
  • predecessor checks/reviews do not transfer and no qualifying independent approval is claimed.

Keep Draft until the unchanged current head is eligible for full exact-head required evidence and a qualifying independent review. No self-approval, force-push, destructive rebase, bypass, or gate weakening.

@coderabbitai

coderabbitai Bot commented Sep 2, 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 seonghobae changed the title refactor(naming): name bounded post Keyman backfill operation refactor(naming): name bounded operator entry points Sep 2, 2026
@seonghobae seonghobae changed the title refactor(naming): name bounded operator entry points refactor(operator): name bounded entry points and validate Keyman timeout Sep 2, 2026
@seonghobae
seonghobae changed the base branch from main to fix/contextual-orchestrator-owner-boundary September 2, 2026 20:50
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.
@seonghobae seonghobae changed the title refactor(operator): name bounded entry points and validate Keyman timeout refactor(operator): name bounded entry points and validate Keyman admissions Sep 2, 2026
Adopt #899668e053 after its protected-main convergence. Preserve the bounded operator policy delta and current Tests workflow without rewriting branch history.
Adopt #89947ebcee after protected main #924 while preserving bounded operator policy without force rewrite.
@opencode-agent

opencode-agent Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Scheduled review-feedback autofix for this PR head.

  • Head SHA: 996a3ca66d5c52e0421c0a14b60b9394516eb9b2

@seonghobae seonghobae left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Parent #899 moved non-force to e5711282…; this branch is now mechanically conflicting only because it still embeds the pre-#931 workflow contract. Its 14 bounded-operator files are disjoint from the parent’s four workflow/contract repairs. Preserve the operator delta exactly, adopt those protected blobs, and advance with a two-parent non-force convergence commit. This is a wrong-base repair; predecessor checks/reviews do not transfer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant