Repository navigation
fix(vscode-lm): complete Copilot model port with safe input ceilings - #2052
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📜 Recent review details
📝 Summary
Merge Risk: ⚪ Minimal · up to
|
Review statusThanks for contributing. This comment tracks the review sequence and the next action. Current step: Awaiting fresh human maintainer or CODEOWNER approval. Automated review is complete for the latest commit but does not replace human approval. Review-state labels are managed by this workflow; do not edit them manually. |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Summary
Validation
Run in the isolated publication worktree based on Zoo-Code-Org/Zoo-Code main at
58341d882e57e9dd7eb50c5187a52917541789eb:pnpm --dir packages/types exec vitest run src/__tests__/vscode-llm.spec.ts: 15 passed.pnpm --dir webview-ui exec vitest run src/components/ui/hooks/__tests__/useSelectedModel.spec.ts: 110 passed.pnpm --dir packages/types check-types: passed.--prune-suppressions --max-warnings=0: passed for all three files, with no suppressions added.git diff --check: passed.A fresh frozen-lockfile install failed because the configured package feed returned HTTP 404 for a locked package. Validation reused installed dependencies with workspace package links directed to the publication worktree. Vitest reported 4.1.9 rather than the declared 4.1.11. The installed Node 24 also differs from the requested Node 22.23.1. The full suite and live Copilot probes were not run.
Related issue and scope
No approved Zoo Code issue was supplied or identified for this exact port. Both source PRs and the prior merged port are linked above. This PR does not claim to close an issue.
This supports provider reliability. No separate documentation or visual snapshots are needed for this catalog correction. Implementation, publication, and verification were AI-assisted.