From f429f6b1d31f3aeb903ffc2fa965ff3f99b2bbbd Mon Sep 17 00:00:00 2001 From: 3dtours Date: Mon, 27 Jul 2026 20:32:30 +0700 Subject: [PATCH] docs: verify instrument assignment rules for MAIN SESSION All 4 rules confirmed working: 1. MIDI item inherits track instrument 2. Section item does NOT inherit 3. Track isolation preserved 4. Piano roll instrument change propagates back to track --- wiki.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/wiki.md b/wiki.md index 7c85e0d..f7c6bf9 100644 --- a/wiki.md +++ b/wiki.md @@ -467,3 +467,9 @@ - **Tóm tắt thay đổi:** Thêm ghost notes cho Piano Roll tab: tất cả MIDI items không được chọn thành ghost notes (25% opacity, dashed border). Dropdown thay thế tab title để chuyển nhanh MIDI item đang edit. Hai chế độ xem: Session Sync (ghost visible, bar labels aligned với session) và Isolated (bar 0, no ghost). - **Các file ảnh hưởng:** `app/static/js/services/ghostNoteExtractor.js`, `app/static/js/app.jsx`, `app/static/js/app.precompiled.js`, `app/templates/index.html` - **Ghi chú/Test (nếu có):** Kiểm tra dropdown list đúng tất cả MIDI items. Switch item → ghost notes của item cũ hiện ra. Nút 🌐 Session/📋 Isolated chuyển chế độ. 👻 Ghost toggle chỉ hoạt động ở Session mode. + +### [2026-07-27 20:28] Task: Verify instrument assignment rules for MAIN SESSION +- **Tóm tắt thay đổi:** Verify 4 rules: (1) MIDI item nhận instrument từ track cha qua `handleSwitchMidiItem` copy `instrumentProgram` từ track → sub-tab; (2) Section item KHÔNG nhận instrument (reset về null khi clone tracks trong `handleEditSectionInTab`); (3) Track khác không bị ảnh hưởng do `setTrackInstrumentWithProgram` filter by `trackId`; (4) Đổi instrument trong PianoRoll tab → áp dụng ngược lại cho track qua `openInstrumentSelector` → `setTrackInstrumentWithProgram`. Tất cả đều OK, không bug. +- **Các file ảnh hưởng:** `app/static/js/app.jsx` (verify lines 4642-4667, 5905, 6763-6807, 8422-8424), `app/core/render_engine.py` (verify lines 95-115) +- **Ghi chú/Test (nếu có):** No code changes needed. All 4 rules confirmed working. +---