Carry the gesture no-effect fact on the snapshot tree like unsettledGesture
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 55/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- typescript
Hướng nghiên cứu
The issue references specific files: src/daemon/gesture-no-effect.ts and src/daemon/selector-capture-runtime.ts. Start by understanding how SnapshotState.unsettledGesture works from #2839. Then, trace where gestureNoEffect is currently annotated and how to move it to the snapshot tree stamp. The goal is to delete withGestureNoEffectWarning and ensure selector reads (is, get, find) show the warning. Run existing tests related to gesture warnings to verify the change.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Problem
A proven no-effect gesture (gestureNoEffect, #1600) reaches the agent only through capture annotations (withGestureNoEffectWarning in src/daemon/gesture-no-effect.ts). src/daemon/selector-capture-runtime.ts keeps only the SnapshotState and drops those annotations. So is, get, and find never show the no-effect warning; only snapshot does.
#2839 made SnapshotState.unsettledGesture the single carrier for the unsettled post-gesture fact. Every route derives its disclosure from that stamp: selector reads through withCaptureDisclosures, and snapshot through buildSnapshotWarnings.
Proposal
Move gestureNoEffect to the same stamp on the tree and derive its disclosure the same way. This deletes withGestureNoEffectWarning and the annotation channel for it, and gives selector reads the warning they miss today.
Follow-up to #2839.
- Ngôn ngữ chính
- TypeScript
- Star
- 4.7k
- Fork
- 304
- Merge trung bình
- 11 giờ 29 phút
- Pull request đã merge (30 ngày)
- 514
Hướng dẫn đóng góp
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 callstack/agent-device
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
callstack/agent-device#1869 ·
-
needs-triage
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 32/100
callstack/agent-device#2948 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 55/100
callstack/agent-device#2946 ·
-
needs-triage
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
callstack/agent-device#2940 ·
-
bug ready-for-agent
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
callstack/agent-device#2935 ·
Tất cả issue của callstack/agent-device
Issue tương tự
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Crush Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
ElementsProject/cln-application#167 · 1 bình luận · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Quantco/pnpm-licenses#17 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100