Skip to content

chore: upgrade Pi SDK to 0.85.1 - #27

Merged
benedict2310 merged 1 commit into
mainfrom
chore/pi-0.85.1
Sep 7, 2026
Merged

benedict2310 merged 1 commit into
mainfrom
chore/pi-0.85.1

Conversation

@benedict2310

Copy link
Copy Markdown
Owner

Summary

  • upgrade the Pi SDK family from 0.84.3 to 0.85.1
  • retain the direct Undici 8.9.0 pin aligned with Pi
  • update the Pi SDK compatibility contract and external-extension loader check

Compatibility

Pi 0.85.0 adds session/provider fixes without a TelePi-relevant breaking API change. Pi 0.85.1 specifically restores/fixes supported SDK imports after the short-lived 0.85.0 packaging regression. TelePi's extension-loader compatibility test exercises that import path.

Validation

  • Characterization: npx vitest run test/pi-sdk-compatibility.test.ts test/pi-session.test.ts (84 passed)
  • TDD RED: updating the manifest caused the explicit 0.84.3 compatibility assertion to fail
  • GREEN: focused suite 84 passed
  • npm test (442 passed)
  • npm run test:coverage (passed; statements/lines 86.63%, branches 80.99%, functions 90.09%)
  • npm run build

Local runtime is Node 26.7.0; the Node 26 gzip/SSE regression ran as part of the full suite.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 7, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-07T06:35:53.170841Z 1bd9cf5 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@benedict2310
benedict2310 merged commit aecce4c into main Sep 7, 2026
4 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