chore(plugin): update OpenClaw SDK to 2026.5.9#1
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughThis PR adds TypeScript tooling and CI type-checking, bumps package and plugin versions, updates dependency overrides, prefers OpenClaw's ChangesOpenClaw Plugin Update & Tooling
Sequence Diagram(s)sequenceDiagram
participant CI
participant Typecheck
participant Test
participant Codecov
CI->>Typecheck: run npm run typecheck
Typecheck-->>CI: pass/fail
CI->>Test: run vitest / coverage
Test->>Codecov: upload coverage (informational)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
There was a problem hiding this comment.
Pull request overview
Updates this plugin to align with the latest OpenClaw SDK (v2026.5.9-beta.1) and its task-flow runtime namespace changes, while tightening CI/typechecking and keeping doctor output stable across environments.
Changes:
- Add a strict TypeScript
tsconfig.json, atypecheckscript, and run it in CI. - Update task-flow mutation binding to prefer
api.runtime.tasks.managedFlows, retaining 2026.4-compatible fallbacks. - Keep
doctordeep checks stable by reporting API-key status regardless of CLI binary presence, and explicitly reporting live-test skipped status when binaries are missing.
Reviewed changes
Copilot reviewed 9 out of 11 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
tsconfig.json |
Adds strict TS config used by the new typecheck CI step. |
src/pipeline/taskflow-bridge.ts |
Switches runtime probing to prefer runtime.tasks.managedFlows with compatibility fallbacks. |
src/pipeline/taskflow-bridge.test.ts |
Updates fixtures and adds coverage for managedFlows + fallback selection behavior. |
src/infra/doctor.ts |
Always reports API-key check; adds explicit “live test skipped” when binaries are missing. |
src/infra/doctor.test.ts |
Adjusts mocking cleanup and updates checkFilesAndDirs test setup for dispatch-state path. |
README.md |
Updates OpenClaw badge/link and documents npm run typecheck. |
package.json |
Bumps version, sets Node engine floor, updates OpenClaw devDependency, adds typecheck, refreshes overrides. |
openclaw.plugin.json |
Updates plugin manifest version to 0.9.24. |
.github/workflows/ci.yml |
Runs npm run typecheck in CI prior to tests. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
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 `@src/pipeline/taskflow-bridge.ts`:
- Around line 118-120: The fallback selection currently uses nullish coalescing
so a present but invalid tasks.managedFlows can short-circuit the chain; change
the logic to probe each candidate in order (tasks.managedFlows, then tasks.flow,
then runtime.taskFlow) and pick the first one where typeof candidate.bindSession
=== "function" before returning; reference the variables tasks, managedFlows,
flow, runtime.taskFlow and the validation of bindSession to implement an
explicit sequential check and only return null if none are valid.
🪄 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: ASSERTIVE
Plan: Pro Plus
Run ID: 03c4a53b-6c29-4859-bc0f-a9e0076cf872
📒 Files selected for processing (2)
src/infra/doctor.tssrc/pipeline/taskflow-bridge.ts
Summary
Release Notes Checked
Testing