[codex] Update OpenClaw plugin for SDK 2026.6.5 - #10
Conversation
|
| Filename | Overview |
|---|---|
| openclaw/openclaw.plugin.json | Removed SDK 2026.3-era top-level platforms, requires.binaries, and requires.osPermissions fields; migrated to package.json's openclaw.environment and hostTargets per SDK 2026.6.5 schema. |
| openclaw/package.json | Updated openclaw dependency to 2026.6.5, bumped compat and build metadata, tightened engines to >=22.19.0, added prepack hook, and removed --dts from build; tsup build target (node20) is now inconsistent with the engines floor (22.19.0). |
| openclaw/tsconfig.json | Disabled TypeScript declaration emit (declaration: false) to align with removal of --dts from the tsup build script; intentional for a runtime-only plugin package. |
| openclaw/package-lock.json | Lock file regenerated for openclaw 2026.6.5; large reduction in transitive entries as SDK now no longer provides runtime packages transitively. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[openclaw.plugin.json] -->|SDK 2026.6.5 schema| B[Removed: platforms / requires]
B --> C[Metadata now lives in package.json]
C --> D[openclaw.environment.binaries\nfindmy, findmy-helper]
C --> E[openclaw.environment.osPermissions\nmacos:screen-recording]
C --> F[openclaw.hostTargets\ndarwin-arm64, darwin-x64]
G[package.json] -->|engines bumped| H[node >=22.19.0\nwas >=18]
G -->|dependency bumped| I[openclaw ^2026.6.5\nwas ^2026.3.23]
G -->|scripts updated| J[prepack: npm run build\nbuild: removed --dts]
K[tsconfig.json] -->|declaration| L[false\nwas true]
L --> M[No .d.ts output — intentional for runtime-only plugin]
Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 1
openclaw/package.json:81
The `tsup` build target is `node20`, but the `engines` floor was bumped to `>=22.19.0`. Targeting Node 20 tells tsup/esbuild to down-level syntax and avoid APIs introduced after Node 20, while the actual runtime is guaranteed to be Node 22.19.0+. Consider aligning the target to `node22` so the emitted code can take advantage of the guaranteed runtime and the transpilation contract is consistent with what users actually run.
```suggestion
"build": "tsup src/index.ts --format esm --out-dir dist --clean --target node22",
```
Reviews (1): Last reviewed commit: "Update OpenClaw plugin for SDK 2026.6.5" | Re-trigger Greptile
| }, | ||
| "scripts": { | ||
| "build": "tsup src/index.ts --format esm --out-dir dist --dts --clean --target node20", | ||
| "build": "tsup src/index.ts --format esm --out-dir dist --clean --target node20", |
There was a problem hiding this comment.
The
tsup build target is node20, but the engines floor was bumped to >=22.19.0. Targeting Node 20 tells tsup/esbuild to down-level syntax and avoid APIs introduced after Node 20, while the actual runtime is guaranteed to be Node 22.19.0+. Consider aligning the target to node22 so the emitted code can take advantage of the guaranteed runtime and the transpilation contract is consistent with what users actually run.
| "build": "tsup src/index.ts --format esm --out-dir dist --clean --target node20", | |
| "build": "tsup src/index.ts --format esm --out-dir dist --clean --target node22", |
Prompt To Fix With AI
This is a comment left during a code review.
Path: openclaw/package.json
Line: 81
Comment:
The `tsup` build target is `node20`, but the `engines` floor was bumped to `>=22.19.0`. Targeting Node 20 tells tsup/esbuild to down-level syntax and avoid APIs introduced after Node 20, while the actual runtime is guaranteed to be Node 22.19.0+. Consider aligning the target to `node22` so the emitted code can take advantage of the guaranteed runtime and the transpilation contract is consistent with what users actually run.
```suggestion
"build": "tsup src/index.ts --format esm --out-dir dist --clean --target node22",
```
How can I resolve this? If you propose a fix, please make it concise.Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Summary
Validation
@openclaw/plugin-inspector@latest inspect --openclaw /Users/omarshahine/GitHub/openclaw/openclaw --no-runtimeacross the 16-plugin inventory with no live P0/P1 issuesnpm pack --dry-runchecks for published packages to confirm compiled runtime files are included