Skip to content

chore(providers): align state dependency ranges - #777

Merged
ytallo merged 1 commit into
mainfrom
chore/all-provider-state-022-ranges
Aug 12, 2026
Merged

chore(providers): align state dependency ranges#777
ytallo merged 1 commit into
mainfrom
chore/all-provider-state-022-ranges

Conversation

@ytallo

@ytallo ytallo commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • update every remaining LLM provider to depend on state ^0.22.0
  • extend the dependency compatibility regression to discover all provider-* manifests automatically

Why

Provider manifests still required state ^0.21.x, which cannot be resolved in the same worker graph as Harness's state ^0.22.0 requirement. This caused the same version conflict whenever an additional provider was included in a Harness deployment.

Impact

After publishing new provider versions, every provider can participate in the Harness worker graph with the shared State 0.22 dependency range.

Validation

  • dependency compatibility tests passed
  • git diff --check passed

Summary by CodeRabbit

  • Compatibility

    • Updated AI provider integrations to support the latest shared state dependency version.
    • Improved compatibility coverage across all available provider integrations and the routing service.
  • Tests

    • Compatibility checks now automatically discover provider integrations, helping ensure newly added providers are included in validation.

@vercel

vercel Bot commented Aug 12, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
workers Ready Ready Preview Aug 12, 2026 12:48pm
workers-tech-spec Ready Ready Preview Aug 12, 2026 12:48pm

Request Review

@ytallo ytallo added the no-ticket PR deliberately has no Linear ticket (bump/typo/CI-only) label Aug 12, 2026
@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8966b06a-ead9-4cb5-abb4-026ed6da4022

📥 Commits

Reviewing files that changed from the base of the PR and between 8403582 and 1f4d942.

📒 Files selected for processing (10)
  • .github/scripts/tests/test_worker_dependency_compatibility.py
  • provider-claude-code/iii.worker.yaml
  • provider-deepseek/iii.worker.yaml
  • provider-github-copilot/iii.worker.yaml
  • provider-kimi/iii.worker.yaml
  • provider-llamacpp/iii.worker.yaml
  • provider-openai-codex/iii.worker.yaml
  • provider-openrouter/iii.worker.yaml
  • provider-xai/iii.worker.yaml
  • provider-zai/iii.worker.yaml

📝 Walkthrough

Walkthrough

The PR updates nine worker manifests to require state ^0.22.0. The compatibility test now discovers all provider-* manifests dynamically and validates them with llm-router.

Changes

State dependency compatibility

Layer / File(s) Summary
Worker manifest dependency alignment
provider-*/iii.worker.yaml
Nine provider manifests now require state ^0.22.0.
Dynamic compatibility test coverage
.github/scripts/tests/test_worker_dependency_compatibility.py
The test discovers all provider-* manifests and checks them with llm-router against the shared harness state dependency range.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • iii-hq/workers#774: Updates worker manifests and compatibility checks for state ^0.22.0.
  • iii-hq/workers#548: Updates worker manifests and compatibility tests for the standalone state dependency.
  • iii-hq/workers#483: Updates worker dependency ranges and the shared compatibility test.

Suggested reviewers: rohitg00, guibeira

Poem

A rabbit checks each worker’s state,
Now all manifests align their fate.
Provider paths are found with care,
llm-router joins the testing hare.
Version 0.22.0 leads the way.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: aligning provider State dependency ranges.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/all-provider-state-022-ranges

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.

@github-actions

Copy link
Copy Markdown
Contributor

skill-check — worker

0 verified, 57 skipped (no docs/).

Layer Result
structure
vale
ai
render

Four for four. Nicely done.

@ytallo
ytallo marked this pull request as ready for review August 12, 2026 12:50
@ytallo
ytallo merged commit 36b57fb into main Aug 12, 2026
41 of 43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-ticket PR deliberately has no Linear ticket (bump/typo/CI-only)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant