Keep dependency updates compatible with the OpenAI continuation patch
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- github-actions, typescript
- Lĩnh vực
- ci-cd, devops, testing-qa
Hướng nghiên cứu
Bắt đầu bằng cách tái hiện bước đã được lên lịch pnpm up "@tanstack/*" --latest và chạy tests/openai-responses-tool-continuation.test.ts với dependency patch hiện tại. Kiểm tra updater workflow, cấu hình pnpm và cách xử lý patch được mô tả trong issue. Được xem là hoàn tất khi họ TanStack AI có thể được cập nhật mà không làm hồi quy hành vi tiếp tục, và workflow đi đến lint và build.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Evidence
The scheduled Update TanStack Dependencies run failed before lint or build on August 30. The exact pnpm up "@tanstack/*" --latest step exits with ERR_PNPM_UNUSED_PATCH because it moves @tanstack/openai-base away from the patched 0.10.5 version.
I reproduced the same failure from current main at 6584b153. In an isolated worktree, allowing unused patches let the update finish and changed @tanstack/ai-openai from 0.22.0 to 0.22.3, with @tanstack/openai-base moving from patched 0.10.5 to unpatched 0.10.8. The focused tests/openai-responses-tool-continuation.test.ts test then failed because the second request had no previous_response_id. The current patch is still required for sequential server-tool turns with provider-side reasoning.
Exact title, body, workflow, patch-package, and keyword searches found no open issue or PR that owns this failure.
Expected impact
The weekly dependency updater is currently guaranteed to fail whenever it tries the latest TanStack AI family. It never reaches the repository lint or build steps, so unrelated TanStack dependency updates also stop. Simply setting allowUnusedPatches or deleting the patch would make the workflow advance while regressing the Builder tool-continuation contract.
Proposed direction
Keep the focused continuation test as the release gate, then choose one owner for the patched AI family:
- Preferred long-term direction: ship the continuation behavior upstream, update the AI packages to the first release containing it, prove the focused test passes without the local patch, and delete the patch.
- Until that release exists: upgrade the pinned pnpm from
11.1.0to a version that supportsupdate.ignoreDepsand exclude the cohesive TanStack AI package set from the scheduled bulk updater. Upgrade that set separately, carrying the patch forward only after its focused test and the full suite pass.
Alternatives and tradeoffs
- Manually port the 321-line patch to every new
openai-baserelease. This keeps AI packages moving, but each port needs careful review against upstream adapter changes and should not be committed straight tomainby a broad scheduled update. - Let
pnpm upignore unused patches. This unblocks the job but silently removes required runtime behavior, and the isolated focused test already proves the regression. - Exclude only
@tanstack/ai-openai. This may still let related direct or transitive AI packages move out of a tested version set, so excluding and reviewing the cohesive AI family is safer.
Decision needed
@tannerlinsley, should we first upgrade pnpm and hold the TanStack AI family out of the weekly bulk updater, or prioritize landing the continuation behavior upstream and keep the weekly job blocked until a release contains it?
- Ngôn ngữ chính
- TypeScript
- Star
- 1.1k
- Fork
- 401
- Merge trung bình
- 18 giờ 44 phút
- Pull request đã merge (30 ngày)
- 55
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của TanStack/tanstack.com
-
source-audit
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 84/100
TanStack/tanstack.com#1195 ·
-
source-audit
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
TanStack/tanstack.com#1194 · 2 bình luận ·
-
source-audit
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
TanStack/tanstack.com#1177 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 64/100
TanStack/tanstack.com#1059 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
TanStack/tanstack.com#1047 ·
Tất cả issue của TanStack/tanstack.com
Issue tương tự
-
VerificationGate: ATTRIBUTION quote guard never matches a normal quotation (\b around the quote) Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
danielmiessler/LifeOS#2234 ·
-
T: Bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
-
Mend: dependency security vulnerability untriaged
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100