CUEデータパーサーのcueStart offsetが3バイトずれている

[9c5s]

概要

TCNetDataPacketCUE.read() の cueStart が 50 に設定されているが、仕様書 (V3.5.1B) では CUE 1 の開始は byte 47。この3バイトのずれにより、CUE IN TIME / OUT TIME が常に0として読み取られる。

再現手順

1. CDJで曲をロードし、CUEポイントが設定された楽曲を再生
2. requestData(CUEData, layer) でCUEデータを取得
3. 全CUEエントリの inTime / outTime が 0 になる

原因

// network.ts - TCNetDataPacketCUE.read()
const cueStart = 50; // 仕様書では byte 47

仕様書のCUEエントリレイアウト (22バイト/エントリ):

Offset	Size	説明
+0	1	CUE Type
+1	1	RESERVED
+2	4	CUE IN Time (UInt32LE)
+6	4	CUE OUT Time (UInt32LE)
+10	1	RESERVED
+11	3	CUE Color (R, G, B)
+14	8	RESERVED

start=50 の場合、IN TIME 読み取り位置 (byte 52) は実データの3バイト後方のゼロ領域にあたる。

実機ダンプによる検証

BRIDGE64 + CDJ-3000 で取得したパケットダンプ:

header (24-50): 0c01...7203000000000033
entry[0] @50: 020000...00d5
entry[1] @72: 040000...00f8

start=47 基準で IN TIME (offset +2 = byte 49) を UInt32LE で読むと:

• byte 49-52: 33 02 00 00 → 563ms
• byte 71-74: D5 04 00 00 → 1237ms
• byte 93-96: F8 2E 00 00 → 12024ms
• byte 115-118: 1A 59 00 00 → 22810ms
• byte 137-140: 73 FE 01 00 → 130675ms (≈2:10)

いずれも楽曲のCUEポイント位置として妥当な値。

補足: TYPE フィールド

仕様上 byte 47 に CUE Type が存在するが、BRIDGE64 では常に 0 が送信される。
空エントリ判定を type === 0 から inTime === 0 && outTime === 0 に変更する必要がある。

修正方針

1. cueStart を 50 → 47 に変更
2. 空エントリ判定を type === 0 → inTime === 0 && outTime === 0 に変更

[coderabbitai]

Coding Plan

Summary

• 仕様書V3.5.1Bに準拠し、cueStart を47に修正してCUE時間データの正しい読み取りを実現
• BRIDGE64の実機動作（TYPEフィールドが常に0）に対応するため、空エントリ判定を時間ベース（inTime === 0 && outTime === 0）に変更
• 既存のテストをオフセット変更に合わせて更新し、新しい判定ロジックの検証テストを追加

Implementation Steps

Phase 1: CUEパーサーのオフセット修正と空エントリ判定ロジックの変更

CUEデータパーサーの2つの問題を修正する。仕様書V3.5.1Bに準拠した正しいオフセットへの変更と、BRIDGE64実機の動作に合わせた空エントリ判定条件の変更を行う。

Task 1: cueStart オフセットの修正

cueStart を仕様書に準拠した値に変更し、CUE IN TIME / OUT TIME が正しく読み取れるようにする。

• src/network.ts の TCNetDataPacketCUE.read() 内で cueStart 定数を 50 から 47 に変更
• 既存のコメント（仕様書の誤記に関する記載）を削除または更新し、仕様書V3.5.1Bに準拠している旨に書き換え

Task 2: 空エントリ判定ロジックの変更

BRIDGE64では TYPE フィールドが常に0を返すため、空エントリ判定を時間ベースに変更する。

• src/network.ts の TCNetDataPacketCUE.read() 内で、inTime と outTime の読み取りを空エントリ判定の前に移動
• 判定条件を type === 0 から inTime === 0 && outTime === 0 に変更
• type フィールドは引き続き読み取り、CuePoint オブジェクトに含める（既存の型定義との互換性維持）

🤖 Prompt for AI agents

このフェーズでは `src/network.ts` の `TCNetDataPacketCUE.read()`
メソッドに対して2つの修正を行う。目標は、仕様書V3.5.1Bへの準拠とBRIDGE64実機との互換性確保である。

**Task 1.1: `cueStart` オフセットの修正**
- `src/network.ts` の `TCNetDataPacketCUE.read()` 内にある `cueStart` 定数の値を `50` から
`47` に変更する
- 仕様書の誤記に関する既存コメントを削除または書き換え、仕様書V3.5.1Bに準拠している旨を記載する

**Task 1.2: 空エントリ判定ロジックの変更**
- `src/network.ts` の `TCNetDataPacketCUE.read()` 内で、`inTime` と `outTime`
の読み取り処理を空エントリ判定の条件チェックより前に移動する
- 空エントリの判定条件を `type === 0` から `inTime === 0 && outTime === 0` に変更する
- `type` フィールドは引き続き読み取りを行い、`CuePoint` オブジェクトのプロパティとして含め、既存の型定義（`src/types.ts` の
`CuePoint` / `CueData`）との互換性を維持する

Phase 2: テストケースの更新と追加

オフセット変更に伴う既存テストの修正と、新しい空エントリ判定ロジックを検証するテストケースの追加。

Task 1: 既存テストのオフセット更新

新しい cueStart = 47 に合わせてテストのバッファ書き込み位置を調整する。

• tests/types-and-cue.test.ts の各テストケースで、CUEエントリの書き込みオフセットを更新
  • Entry 0: offset 50 → 47
  • Entry 1: offset 72 → 69
• 各フィールドのオフセット（type: +0, inTime: +2, outTime: +6, color: +11〜13）は変更なし

Task 2: 新しい空エントリ判定のテストケース追加

inTime === 0 && outTime === 0 による空エントリ判定が正しく動作することを検証する。

• type !== 0 かつ inTime === 0 && outTime === 0 のエントリがスキップされることを確認するテストを追加
• type === 0 かつ inTime !== 0 のエントリが正しくパースされることを確認するテストを追加（BRIDGE64互換性の検証）
• 既存の「全CUEスロットのtypeが0の場合」テストの意図を確認し、必要に応じて更新

🤖 Prompt for AI agents

このフェーズでは `tests/types-and-cue.test.ts`
の既存テストを修正し、新しいテストケースを追加する。目標は、フェーズ1の変更内容（オフセット修正・空エントリ判定ロジック変更）が正しく動作することを検証することである。

**Task 2.1: 既存テストのオフセット更新**
- `tests/types-and-cue.test.ts` 内の各テストケースで `Buffer`
へのCUEエントリ書き込みオフセットを以下のとおり更新する
  - Entry 0: offset `50` → `47`
  - Entry 1: offset `72` → `69`
- 各フィールドの相対オフセット（type: +0, inTime: +2, outTime: +6, color: +11〜13）は変更しない

**Task 2.2: 新しい空エントリ判定のテストケース追加**
- `type !== 0` かつ `inTime === 0 && outTime === 0`
のエントリが空エントリとしてスキップされることを確認するテストを追加する
- `type === 0` かつ `inTime !== 0` のエントリが正しくパースされることを確認するテストを追加し、BRIDGE64互換性を検証する
- 既存の「全CUEスロットのtypeが0の場合」テストの意図を確認し、新しい判定ロジックに合わせて必要であれば内容を更新する

Research

TypeScript製のTCNetプロトコルライブラリ。src/network.ts にパケットパーサー群が実装されており、TCNetDataPacketCUE クラスがCUEデータのパース処理を担当。型定義は src/types.ts に CuePoint / CueData として定義済み。テストは tests/types-and-cue.test.ts にあり、バイナリフィクスチャファイルは使用せず、Buffer.alloc() でプログラム的にパケットデータを構築するパターンを採用している。

---

🚀 Next Steps

🤖 All AI agent prompts combined

Task: 1

このフェーズでは `src/network.ts` の `TCNetDataPacketCUE.read()`
メソッドに対して2つの修正を行う。目標は、仕様書V3.5.1Bへの準拠とBRIDGE64実機との互換性確保である。

**Task 1.1: `cueStart` オフセットの修正**
- `src/network.ts` の `TCNetDataPacketCUE.read()` 内にある `cueStart` 定数の値を `50` から
`47` に変更する
- 仕様書の誤記に関する既存コメントを削除または書き換え、仕様書V3.5.1Bに準拠している旨を記載する

**Task 1.2: 空エントリ判定ロジックの変更**
- `src/network.ts` の `TCNetDataPacketCUE.read()` 内で、`inTime` と `outTime`
の読み取り処理を空エントリ判定の条件チェックより前に移動する
- 空エントリの判定条件を `type === 0` から `inTime === 0 && outTime === 0` に変更する
- `type` フィールドは引き続き読み取りを行い、`CuePoint` オブジェクトのプロパティとして含め、既存の型定義（`src/types.ts` の
`CuePoint` / `CueData`）との互換性を維持する
===============================================================================

Task: 2

このフェーズでは `tests/types-and-cue.test.ts`
の既存テストを修正し、新しいテストケースを追加する。目標は、フェーズ1の変更内容（オフセット修正・空エントリ判定ロジック変更）が正しく動作することを検証することである。

**Task 2.1: 既存テストのオフセット更新**
- `tests/types-and-cue.test.ts` 内の各テストケースで `Buffer`
へのCUEエントリ書き込みオフセットを以下のとおり更新する
  - Entry 0: offset `50` → `47`
  - Entry 1: offset `72` → `69`
- 各フィールドの相対オフセット（type: +0, inTime: +2, outTime: +6, color: +11〜13）は変更しない

**Task 2.2: 新しい空エントリ判定のテストケース追加**
- `type !== 0` かつ `inTime === 0 && outTime === 0`
のエントリが空エントリとしてスキップされることを確認するテストを追加する
- `type === 0` かつ `inTime !== 0` のエントリが正しくパースされることを確認するテストを追加し、BRIDGE64互換性を検証する
- 既存の「全CUEスロットのtypeが0の場合」テストの意図を確認し、新しい判定ロジックに合わせて必要であれば内容を更新する

💡 Iterate on the plan with: @coderabbitai <feedback>

Example Feedback
- `@coderabbitai` You can skip phase 3. Add a simple unit test case for phase 2.
- `@coderabbitai` For design choice 1 go ahead with option 3 and replan.

---

💬 Have feedback or questions? Drop into our discord!

[coderabbitai]

🔗 Related PRs

#2 - fix: TCNet仕様準拠のプロトコル実装バグ修正 [merged]
#19 - feat: 全DataPacketタイプの専用パケットクラスを実装 [merged]
#22 - docs: Wiki更新 - 実装済みDataPacketクラスのAPI追加と実装状況修正 [merged]

---

📝 Issue Planner

_{Check the box below or use the @coderabbitai plan command to generate an implementation plan and prompts that you can use with your favorite coding assistant.}

• Create Plan

---

🧪 Issue enrichment is currently in open beta.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false

💬 Have feedback or questions? Drop into our discord!

[9c5s]

cueStart=47 + 空エントリ判定(type===0 && inTime===0 && outTime===0)で修正済み (network.ts L636-646)