Contribution plan: strzhao's open PRs (small fixes, HarmonyOS, new features)

Đang mở
#4,318 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

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
20/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
github, node.js, typescript

Hướng nghiên cứu

Start with the linked pull requests #4282–#4297 and the HarmonyOS extraction tracked in #4317; this issue coordinates their review rather than specifying a standalone code change. Read the direction for section C and the device evidence requirements in section B. There is no single completion condition here: the tracked PRs must be reviewed and landed or their design direction resolved.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

enhancement

@strzhao thanks for the large batch of contributions. There are 16 open PRs, and a lot of this is useful. To keep review moving and the codebase coherent, we'll take them in a fixed order. This issue tracks it.

Please hold new PRs until the open set is down to about five.

A. Small fixes: landing now

We'll help finish these, and may push small fixes to your branches.

  • #4282 docs: route the management command family in the skill and install help
  • #4284 render scoped snapshot refs in their pinned form
  • #4289 guide screenshot callers to the session prerequisite

B. HarmonyOS: one at a time, with committed evidence

CI has no HarmonyOS device or HDC lane, so green CI only proves the mocked unit tests. For each device-facing change, please commit the live run as an evidence file under docs/evidence/ (device, build, commands, output), and propose a landing order. We'll review them one at a time.

  • #4239 inject gesture pan as a densified touch-panel trajectory
  • #4285 report the keyboard band in snapshot responses
  • #4287 lower open --launch-args to aa start parameters
  • #4290 keep-awake on and off across Android and HarmonyOS
  • #4292 support app uninstall beside deploy
  • #4293 verify foreground after open instead of trusting the launch line
  • Parked: #4240, the experimental test-HAP carrier for locked-down builds. It's about 2k lines of experimental surface we'd have to maintain, so it waits until there's a supported need.

HarmonyOS is moving out of core: #4317 extracts it behind a platform plugin interface and ships it as @agent-device/harmonyos in 0.22, like our other providers and platforms. Changes that land before then move with it.

C. New commands and flags: design first

These add new public CLI and daemon surface without an agreed design. We're reviewing them together for coherence (overlaps such as press sequences versus --expect readback) and for performance impact on paths that don't use them. We'll post a direction per PR here. Where we accept the direction, we'll ask for a short issue (use case, smallest version, CLI example, completion conditions) before review continues. Some may change shape or be declined.

  • #4296 atomic multi-ref press sequences
  • #4297 guaranteed press baselines and --expect readback
  • #4288 scripted takeover hold and release subcommands
  • #4291 burst screenshots with --count and --interval-ms
  • #4286 advise sessions stuck in the snapshot-per-action loop
  • #4294 advise session list readers about cross-workspace claims

Direction for C: design review in progress; posted here when done.

Ngôn ngữ chính
TypeScript
Star
5.3k
Fork
377
Merge trung bình
13 giờ 58 phút
Pull request đã merge (30 ngày)
654

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của callstack/agent-device

Tất cả issue của callstack/agent-device

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.