docs(gap): close delivered video ownership gap by seonghobae · Pull Request #973 · ContextualWisdomLab/contextual-orchestrator

seonghobae · 2026-08-31T11:36:49Z

Summary

reconcile two stale video-job gap statements with protected-main PR feat(video): normalize async job resource state #912
document the accepted Valkey/process-local durability boundary from ADR 0037
preserve the provider-status non-inference boundary

Verification

uv run pytest -q tests/test_commercial_gap_register.py tests/test_video_jobs.py tests/test_video_job_registry_atomicity.py tests/test_multimodal_model_group_http.py (54 passed)
git diff --check

coderabbitai · 2026-08-31T11:36:58Z

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

Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.

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.}

devin-ai-integration

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

cwl-noema-review

Noema LLM review

PR #973 appears to be a documentation update, closing a gap in the product-technical baseline. However, the diff does not contain any exact changed-side lines, making it difficult to provide a formal verdict. Further investigation is required to determine the impact of this change on the codebase.

Reviewed changed lines

docs/product-technical-gap-baseline.md:1221 (RIGHT): diff does not contain exact changed-side lines

Adversarial validation

Residual risk: low

Findings

No blocking findings.
Result: COMMENT
Head SHA: 48a9bf321bd8ec8c6ff8f2563abcb6f6e341edfc
Reviewer credential: noema-review-github-app
Actor: cwl-noema-review[bot]

seonghobae · 2026-09-01T00:05:17Z

Cross-PR integration contract: routing identity is provider-neutral model_group only; do not add or preserve a provider-family abstraction. OpenRouter discovery must retain concrete free model IDs, while the aggregate openrouter/free router is not a serving candidate. OpenCode, Noema, and Strix must call contextual-orchestrator. Do not impose fixed wall-clock deadlines on inference, initial ping, readiness/health, provider discovery, or OpenRouter ZDR-list retrieval; use explicit cancellation and evidence-backed transport failure instead. Reconcile this PR with #971 and central .github #1508 before merge.

opencode-agent · 2026-09-02T02:19:16Z