Skip to content

Fix/harness dependencies - #299

Merged
ytallo merged 2 commits into
mainfrom
fix/harness-dependencies
Jun 19, 2026
Merged

Fix/harness dependencies#299
ytallo merged 2 commits into
mainfrom
fix/harness-dependencies

Conversation

@ytallo

@ytallo ytallo commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated and added service dependencies including new modules for cron scheduling, observability, and streaming capabilities.
    • Removed unused dependency while maintaining existing service configurations.

@vercel

vercel Bot commented Jun 19, 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 Jun 19, 2026 7:18pm

Request Review

@coderabbitai

coderabbitai Bot commented Jun 19, 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

Run ID: c94fbe98-83a4-4db3-9a72-1e874d58f657

📥 Commits

Reviewing files that changed from the base of the PR and between c87a865 and dbe6bbe.

⛔ Files ignored due to path filters (1)
  • harness/Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • harness/iii.worker.yaml

📝 Walkthrough

Walkthrough

The harness/iii.worker.yaml dependencies block is updated to add five new entries (iii-cron, iii-observability, iii-stream, shell, web) pinned to new semver ranges, and to remove approval-gate. All previously existing dependencies are retained.

Changes

Worker Harness Dependency Set Update

Layer / File(s) Summary
Worker harness dependency changes
harness/iii.worker.yaml
Adds iii-cron, iii-observability, iii-stream, shell, and web to the dependencies block and removes approval-gate; existing entries for configuration, iii-directory, session-manager, context-manager, provider-anthropic, and provider-openai are unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • iii-hq/workers#134: Also modifies the harness/iii.worker.yaml dependencies block, including changes to approval-gate and the same dependency set structure.
  • iii-hq/workers#288: Modifies the same harness/iii.worker.yaml worker dependency list with version bumps and dependency set changes, including approval-gate.

Poem

🐇 Hippity-hop, the deps have changed today,
approval-gate has hopped along its way.
iii-cron ticks, iii-stream flows free,
shell and web join the dependency spree!
Observability watches with a watchful ear — 🌟
The harness grows stronger, hop hop, never fear!

🚥 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 'Fix/harness dependencies' directly relates to the main change—updating dependency versions in harness/iii.worker.yaml and managing the dependency list.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/harness-dependencies

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 and usage tips.

@github-actions

Copy link
Copy Markdown
Contributor

skill-check — worker

0 verified, 24 skipped (no docs/).

Layer Result
structure
vale
ai
render

Four for four. Nicely done.

@ytallo
ytallo merged commit 4501599 into main Jun 19, 2026
11 of 12 checks passed
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.

2 participants