Skip to content

chore(iii-directory): bump iii-sdk to 0.20.0 - #343

Merged
guibeira merged 1 commit into
mainfrom
chore/sdk-0.20-iii-directory
Jun 26, 2026
Merged

chore(iii-directory): bump iii-sdk to 0.20.0#343
guibeira merged 1 commit into
mainfrom
chore/sdk-0.20-iii-directory

Conversation

@guibeira

@guibeira guibeira commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Bump iii-sdk 0.16.0-next.2 → 0.20.0 per https://iii.dev/docs/upgrading/from-0-19-x.md. Standard symbol renames (IIIError→errors::Error, III→IIIClient, protocol/runtime/trigger submodules). Build, fmt, clippy green; 249 unit + 44 BDD tests pass; surface parity (13 functions + 2 triggers) 1:1. No import aliases. No iii-helpers needed.

Summary by CodeRabbit

  • Chores

    • Upgraded the SDK to a newer stable version for broader compatibility and long-term support.
    • Refreshed internal integration points across directory features to work with the updated client and error handling flow.
  • Tests

    • Updated test helpers and step definitions to match the latest SDK paths and client types.

@vercel

vercel Bot commented Jun 26, 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 26, 2026 11:28am

Request Review

@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The PR pins iii-sdk to 0.20.0 and switches runtime, configuration, function, skills, and test wiring from III/IIIError to IIIClient/Error. Several test step imports also move TriggerRequest to iii_sdk::protocol.

Changes

iii-sdk 0.20 migration

Layer / File(s) Summary
Runtime surface and SDK bump
iii-directory/Cargo.toml, iii-directory/src/main.rs, iii-directory/src/trigger_types.rs
iii-sdk is pinned to 0.20.0, and the boot, trigger registration, and handler wiring use IIIClient, Error, and the updated trigger request imports.
Configuration worker wiring
iii-directory/src/configuration.rs
Configuration registration, fetch helpers, and config-change trigger handlers switch from III/IIIError to IIIClient/Error.
Function registrations
iii-directory/src/functions/mod.rs, iii-directory/src/functions/engine_fn.rs, iii-directory/src/functions/download.rs, iii-directory/src/functions/prompts.rs, iii-directory/src/functions/registry.rs
Directory function entrypoints and async registration handlers switch to IIIClient/Error, including the download pipeline and worker registry lookups.
Skills wiring
iii-directory/src/functions/skills.rs
Skills cache fetches, visibility resolution, and list/get/index registrations switch to IIIClient/Error.
Test helpers and step imports
iii-directory/tests/common/*, iii-directory/tests/steps/*
Test engine helpers, shared worker setup, world state, and TriggerRequest imports switch to the new client type and protocol request path.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • iii-hq/workers#330: Also upgrades iii-sdk to 0.20.0 and migrates worker wiring from III/IIIError to IIIClient/Error.
  • iii-hq/workers#341: Shares the same iii-sdk version bump and client/error type migration across related worker modules.
  • iii-hq/workers#267: Updates iii-directory/src/configuration.rs with the same III/IIIError to IIIClient/Error transition.

Suggested reviewers

  • sergiofilhowz
  • ytallo

Poem

🐇 I nibbled an SDK leaf so green,
From III to IIIClient, the path was clean.
Error hopped softly through trigger and song,
While tests in the burrow all marched along. 🥕
Thump-thump, little directory, carry on!

🚥 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 accurately summarizes the main change: upgrading iii-sdk in iii-directory to version 0.20.0.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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 docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/sdk-0.20-iii-directory

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, 27 skipped (no docs/).

Layer Result
structure
vale
ai
render

Four for four. Nicely done.

@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 `@iii-directory/Cargo.toml`:
- Line 18: The iii-directory Cargo.toml dependency on iii-sdk is pointing to a
non-public crate version, so cargo resolution will fail in environments without
external private registry access. Update the iii-sdk entry to a resolvable
source by either switching to the correct public package name, or replacing the
version pin with a git/path dependency or a [patch.crates-io] override that
targets the intended crate source; use the iii-sdk dependency declaration in
Cargo.toml as the location to fix.
🪄 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: 8a7ad146-038b-4cb1-b9d8-a1f78cce2b1a

📥 Commits

Reviewing files that changed from the base of the PR and between 6a9212a and eb3e5f5.

⛔ Files ignored due to path filters (1)
  • iii-directory/Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (18)
  • iii-directory/Cargo.toml
  • iii-directory/src/configuration.rs
  • iii-directory/src/functions/download.rs
  • iii-directory/src/functions/engine_fn.rs
  • iii-directory/src/functions/mod.rs
  • iii-directory/src/functions/prompts.rs
  • iii-directory/src/functions/registry.rs
  • iii-directory/src/functions/skills.rs
  • iii-directory/src/main.rs
  • iii-directory/src/trigger_types.rs
  • iii-directory/tests/common/engine.rs
  • iii-directory/tests/common/workers.rs
  • iii-directory/tests/common/world.rs
  • iii-directory/tests/steps/download_registry.rs
  • iii-directory/tests/steps/download_repo.rs
  • iii-directory/tests/steps/prompts.rs
  • iii-directory/tests/steps/read.rs
  • iii-directory/tests/steps/registry.rs

Comment thread iii-directory/Cargo.toml
@guibeira
guibeira merged commit 2599395 into main Jun 26, 2026
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.

1 participant