Skip to content

chore(workers): finish stale configuration/iii-state dependency pin refresh - #459

Merged
ytallo merged 1 commit into
mainfrom
chore/bump-configuration-dep-pins
Jul 9, 2026
Merged

chore(workers): finish stale configuration/iii-state dependency pin refresh#459
ytallo merged 1 commit into
mainfrom
chore/bump-configuration-dep-pins

Conversation

@ytallo

@ytallo ytallo commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • chore(harness): refresh stale worker dependency pins #456 bumped harness's own builtin-engine pins (configuration, iii-state, iii-queue, iii-cron, iii-observability, iii-stream) from ^0.19.0 to ^0.21.3, but every worker harness depends on still declared ^0.19.0 for configuration/iii-state.
  • ^0.19.0 and ^0.21.3 don't overlap, so resolving harness's dependency graph now fails: iii worker reinstall harness → HTTP 422 version_conflict, dependency_path: harness -> iii-directory -> configuration.
  • Bumps the same pins to ^0.21.3 in the seven affected manifests: iii-directory, llm-router (iii-state + configuration), session-manager, context-manager, web, provider-anthropic (iii-state), provider-openai (iii-state).

Test plan

  • git diff scoped to exactly these 7 iii.worker.yaml files, no stray changes
  • After merge, cut a patch release for each of the 7 workers (Create Tag workflow) so the registry has manifests declaring the new range
  • Confirm iii worker reinstall harness resolves cleanly once all 7 are republished

Summary by CodeRabbit

  • Chores
    • Updated dependency version constraints across several workers to align with newer releases.
    • Brought core configuration and state packages up to a newer compatible version range.
    • No user-facing behavior or public interfaces were changed.

…efresh

#456 bumped harness's own builtin-engine pins (configuration, iii-state,
iii-queue, iii-cron, iii-observability, iii-stream) from ^0.19.0 to
^0.21.3, but every worker harness also depends on still declared the old
^0.19.0 range for configuration/iii-state. That diamond is unsatisfiable
(^0.19.0 and ^0.21.3 don't overlap), so resolving harness's dependency
graph now fails with a 422 version_conflict against iii-directory (and
would hit the same wall against llm-router, session-manager,
context-manager, web, provider-anthropic, and provider-openai next).

Bump the same pins in all seven to ^0.21.3 so the graph is resolvable
again once each is republished.
@ytallo ytallo added the no-ticket PR deliberately has no Linear ticket (bump/typo/CI-only) label Jul 9, 2026
@vercel

vercel Bot commented Jul 9, 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, Comment Jul 9, 2026 12:13pm
workers-tech-spec Ready Ready Preview, Comment Jul 9, 2026 12:13pm

Request Review

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

skill-check — worker

0 verified, 41 skipped (no docs/).

Layer Result
structure
vale
ai
render

Four for four. Nicely done.

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This change updates version constraints for the configuration and iii-state dependencies in seven iii.worker.yaml manifest files, bumping them from ^0.19.0 to ^0.21.3 across context-manager, iii-directory, llm-router, provider-anthropic, provider-openai, session-manager, and web workers.

Changes

Worker Manifest Dependency Bumps

Layer / File(s) Summary
Configuration dependency bump
context-manager/iii.worker.yaml, iii-directory/iii.worker.yaml, llm-router/iii.worker.yaml, session-manager/iii.worker.yaml, web/iii.worker.yaml
The configuration dependency version constraint is updated from ^0.19.0 to ^0.21.3.
iii-state dependency bump
llm-router/iii.worker.yaml, provider-anthropic/iii.worker.yaml, provider-openai/iii.worker.yaml
The iii-state dependency version constraint is updated from ^0.19.0 to ^0.21.3.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Poem

Seven little manifests, all in a row,
Hopping up their versions, watch them grow!
From point-nineteen to twenty-one-point-three,
A rabbit's bump commit, quick as can be. 🐇
Carrots and configs, aligned just right—
Thump thump, ship it, goodnight!

🚥 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 summarizes the main change: refreshing stale configuration/iii-state dependency pins in worker manifests.
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
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/bump-configuration-dep-pins

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@provider-openai/iii.worker.yaml`:
- Around line 10-11: Several remaining iii.worker.yaml manifests still pin
configuration and/or iii-state to ^0.19.0, which will keep causing
version_conflict during iii worker reinstall harness. Update the worker manifest
entries in each affected package (including provider-openai and the listed
pubsub, worktree, cron, queue, http, approval-gate, slack, state, telegram-bot,
rbac-proxy, bridge, provider-llamacpp, provider-zai, provider-xai, and
provider-openai-codex manifests) so both configuration and iii-state are aligned
to ^0.21.3, matching the other workers.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ca6cdc9b-ab75-4634-a497-6c2c4a05a749

📥 Commits

Reviewing files that changed from the base of the PR and between a543214 and f86d365.

📒 Files selected for processing (7)
  • context-manager/iii.worker.yaml
  • iii-directory/iii.worker.yaml
  • llm-router/iii.worker.yaml
  • provider-anthropic/iii.worker.yaml
  • provider-openai/iii.worker.yaml
  • session-manager/iii.worker.yaml
  • web/iii.worker.yaml

Comment on lines +10 to 11
iii-state: "^0.21.3"
llm-router: "^1.0.0"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Description: Search all iii.worker.yaml files for stale ^0.19.0 constraints on configuration or iii-state.

# Find all worker manifests and check for old version pins
fd -t f 'iii.worker.yaml' --exec rg -n 'configuration:\s*"?\^0\.19' {} +
fd -t f 'iii.worker.yaml' --exec rg -n 'iii-state:\s*"?\^0\.19' {} +

Repository: iii-hq/workers

Length of output: 5177


Update the remaining worker manifests to ^0.21.3. Several iii.worker.yaml files still pin configuration and/or iii-state to ^0.19.0, so iii worker reinstall harness will still hit version_conflict for those workers: pubsub/iii.worker.yaml, worktree/iii.worker.yaml, cron/iii.worker.yaml, queue/iii.worker.yaml, http/iii.worker.yaml, approval-gate/iii.worker.yaml, slack/iii.worker.yaml, state/iii.worker.yaml, telegram-bot/iii.worker.yaml, rbac-proxy/iii.worker.yaml, bridge/iii.worker.yaml, provider-llamacpp/iii.worker.yaml, provider-zai/iii.worker.yaml, provider-xai/iii.worker.yaml, provider-openai-codex/iii.worker.yaml.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@provider-openai/iii.worker.yaml` around lines 10 - 11, Several remaining
iii.worker.yaml manifests still pin configuration and/or iii-state to ^0.19.0,
which will keep causing version_conflict during iii worker reinstall harness.
Update the worker manifest entries in each affected package (including
provider-openai and the listed pubsub, worktree, cron, queue, http,
approval-gate, slack, state, telegram-bot, rbac-proxy, bridge,
provider-llamacpp, provider-zai, provider-xai, and provider-openai-codex
manifests) so both configuration and iii-state are aligned to ^0.21.3, matching
the other workers.

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