Animation keys follow-ups from the #364 review: Redo docs, sourced-scene error codes, test temp path

Đang mở
  • Có gì không ổn?
#397 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

@calmasacow đang làm issue này rồi.

Từ ngày 8/11/2026.

  • #408 của @calmasacow — đang mở

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
52/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ệ
rust
Lĩnh vực
desktop, documentation, testing

Hướng nghiên cứu

Read .project-docs/mcp.md and the Undo/Redo arm in src/app/dispatch.rs to understand the documented refusal behavior. Compare the error codes for key commands and session-material commands on opened .mb scenes, then inspect the SDK tests and src/app/features/timeline/tests.rs. Done means the Redo row documents the refusal, both command slices use one tested error convention, and the temp-path test uses a unique directory.

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

Mô tả

priority/p3 type/chore urgency/this-quarter

Follow-ups from claude-fleet's full review of #364 (session-only animation keys) at b06cf39. All are low or info.

  1. Redo row in mcp.md. .project-docs/mcp.md explains, in the Undo row only, that a pose re-evaluation refusal after the step still answers changed, with the refusal in inspect_scene status_message. Redo takes the same path (src/app/dispatch.rs, Undo/Redo arm), so add the same sentence to the Redo row.
  2. Error codes for opened .mb scenes disagree between slices. Key commands on an opened Maya Binary return scene_error, while session-material commands on the same scene return unsupported_edit. Both are documented, but a client sees two codes for the same "sourced scene" refusal. Pick one convention and align both slices, with SDK tests.
  3. Fixed temp path in a test. src/app/features/timeline/tests.rs saves to a fixed temp_dir() path and asserts that it doesn't exist. Use a unique tempdir (tempfile), so parallel runs or leftovers can't make it flaky.

Refs #361, #364

Ngôn ngữ chính
Rust
Star
4
Fork
2
Merge trung bình
1 ngày 5 phút
Pull request đã merge (30 ngày)
213

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 cristim/AztecCraft

Tất cả issue của cristim/AztecCraft

Issue tương tự

Thêm issue về Rust

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.