Skip to content

fix: preserve structured exhaustion on provider embedding stack - #969

Merged
seonghobae merged 2 commits into
fix/provider-backed-embedding-batchfrom
fix/17a7-structured-synthesis-integration
Aug 31, 2026
Merged

fix: preserve structured exhaustion on provider embedding stack#969
seonghobae merged 2 commits into
fix/provider-backed-embedding-batchfrom
fix/17a7-structured-synthesis-integration

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • stack the structured-synthesis provider-failure fix on PR fix: use provider-backed embeddings for remote agents #857 provider-backed embeddings
  • preserve the stack's existing 413 cross-provider handling
  • map virtual structured synthesis exhaustion to the existing redacted 503 no_viable_agent retry contract

Runtime diagnosis

The canonical LineageWeave runtime at orchestrator pin 17a7b13f5317e930bae496e6d65268fb15ac4e34 reproduced synthetic orchestrator/auto + response_format=json_object calls returning 500 internal_error before readiness quarantine changed later requests to typed unavailability. The failure is in final provider-facing synthesis, not embeddings or the Lineage parser. No real source identifiers or content were inspected or recorded.

Validation

  • exact fix: use provider-backed embeddings for remote agents #857 base: 63a6e65e8008d275bb765b4e948d505dc9e3cc13 (contains deployed 17a7b13f)
  • make test built the pinned Rust token packer and pinned Rust-backed fast-mlsirm successfully
  • full stack run: 2748 passed, 2 skipped, 1 failed only because an existing assertion expected the raw provider RuntimeError now intentionally redacted
  • after updating that expectation, Docker-native affected rerun: 70 passed
  • focused provider-backed embedding + structured path run: 30 passed
  • git diff --check: passed

This PR is intentionally stacked on #857 and must not merge independently before its base.


Devin Review

Signed-off-by: Seongho Bae <me@seonghobae.me>
Signed-off-by: Seongho Bae <me@seonghobae.me>
@coderabbitai

coderabbitai Bot commented Aug 31, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 3ca4fd10-2908-4991-8d11-667ee6bd78bd

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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 merged commit 0e04a04 into fix/provider-backed-embedding-batch Aug 31, 2026
1 of 2 checks passed
@seonghobae
seonghobae deleted the fix/17a7-structured-synthesis-integration branch August 31, 2026 08:53

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Devin Review found 2 potential issues.

Devin Review

Comment thread contextual_orchestrator/orchestrator.py
Comment thread contextual_orchestrator/orchestrator.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant