test: split schema coverage and peer dependency updates from #34532#34568
test: split schema coverage and peer dependency updates from #34532#34568Kansodata wants to merge 7 commits intoopenclaw:mainfrom
Conversation
Greptile SummaryThis PR cleanly splits low-risk, non-documentation changes out of #34532. All changes are minimal and safe:
No runtime behavior changes, all updates are low-risk and code-quality focused. Confidence Score: 5/5
Last reviewed commit: 4846b77 |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4846b77c57
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
CI note: the previous Bun lane failure was in setup ( |
|
Applied fix for the Bun lane instability:
Result: the Bun check now passes in this PR run. |
|
Quick follow-up: the peer optionality concern has been applied.
This keeps direct extension installs from hard-failing on peer resolution. |
|
Status update: If maintainers are aligned with the split scope, this should be ready for final review. |
Summary
Splits low-risk code, test, and peer dependency updates out of #34532 so that the Teams + Canvas host-side guide PR remains docs-only.
Changes
src/agents/pi-tools.schema.tssrc/plugins/commands.test.tsNotes
This PR contains only the non-documentation changes that were previously mixed into #34532. It does not introduce intended runtime behavior changes.