feat: Time Sync (Type=10) パケットの実装

[9c5s]

概要

Time Sync (Type=10) パケットの送受信を実装する。

仕様 (TCNET V3.5.1B)

• サイズ: 32バイト / ユニキャスト / レスポンス必須
• ノード間のタイムシンクに使用
• Step 0 (Initialize) → Step 1 (Response) のハンドシェイク
• RemoteTimestamp (マイクロ秒精度)

パケット構造

Byte	Size	Type	説明
24	1	UInt8	Step (0=Initialize, 1=Response)
25	1	-	RESERVED
26	2	UInt16LE	Node Listener Port
28	4	UInt32LE	Remote Timestamp (マイクロ秒)

実装タスク

• TCNetTimeSyncPacket クラスの実装 (read/write)
• TCNetPackets[10] へのマッピング登録
• テスト

[coderabbitai]

Coding Plan

Summary

• 既存の TCNetOptInPacket パターン（フィールド宣言、Buffer 読み書き、length/type メソッド）を踏襲
• 双方向通信（Initialize 送信 → Response 受信）のため read() と write() の両方を実装
• TCNetPackets マッピングの null エントリを新クラスに置き換えてパケット解析を有効化
• テストは writeValidHeader() ヘルパーとラウンドトリップパターンを使用

Implementation Steps

Time Sync パケット実装

Time Sync パケットクラスを既存のパケット実装パターン（TCNetOptInPacket を参考）に従って作成し、パケットレジストリに登録し、テストカバレッジを追加する。

Task 1: TCNetTimeSyncPacket クラスの実装

src/network.ts に新しいパケットクラスを追加し、仕様通りのフィールド読み書きを実装する。

• TCNetPacket を継承した TCNetTimeSyncPacket クラスを src/network.ts の他のパケットクラス付近に作成
• フィールド宣言: step!: number, nodeListenerPort!: number, remoteTimestamp!: number
• read() メソッド:
  • offset 24: step (UInt8)
  • offset 25: RESERVED（スキップ）
  • offset 26: nodeListenerPort (UInt16LE)
  • offset 28: remoteTimestamp (UInt32LE)
• write() メソッド: 同じオフセットで各フィールドを書き込み（双方向通信のため read/write 両方必要）
• length(): 32 を返す（24バイトヘッダー + 8バイトペイロード）
• type(): TCNetMessageType.TimeSync (10) を返す

Task 2: TCNetPackets マッピングへの登録

パケットレジストリを更新して Time Sync パケットを有効化する。

• src/network.ts の TCNetPackets オブジェクト内で [TCNetMessageType.TimeSync]: null を [TCNetMessageType.TimeSync]: TCNetTimeSyncPacket に変更
• コメント // 未実装 を削除

Task 3: ユニットテストの追加

既存のテストパターンに従って包括的なテストを作成する。

• tests/packet-parsers.test.ts に TCNetTimeSyncPacket 用の describe ブロックを追加
• read() テスト: writeValidHeader(buffer, TCNetMessageType.TimeSync) でバッファを作成し、offset 24/26/28 にテストデータを書き込み、read() 後のフィールド値を検証
• write() テスト: フィールドを設定して write() を呼び、別インスタンスで read() してラウンドトリップを検証
• length() テスト: 戻り値が 32 であることを検証
• type() テスト: 戻り値が TCNetMessageType.TimeSync (10) であることを検証
• Step 値のテストケース: Initialize (0) と Response (1) の両方をカバー

🤖 Prompt for AI agents

既存の `TCNetOptInPacket` パターンを参考に、`src/network.ts` へ `TCNetTimeSyncPacket`
クラスを追加し、パケットレジストリへの登録とテストの追加を行うフェーズです。

**Task 1.1: `TCNetTimeSyncPacket` クラスの実装（`src/network.ts`）**
- 既存の他のパケットクラス付近に `TCNetPacket` を継承した `TCNetTimeSyncPacket` クラスを作成する
- 以下のフィールドを宣言する:
  - `step!: number`
  - `nodeListenerPort!: number`
  - `remoteTimestamp!: number`
- `read()` メソッドを実装し、以下のオフセットでバッファから読み込む:
  - offset 24: `step` (UInt8)
  - offset 25: RESERVED（スキップ）
  - offset 26: `nodeListenerPort` (UInt16LE)
  - offset 28: `remoteTimestamp` (UInt32LE)
- `write()` メソッドを実装し、`read()` と同じオフセットで各フィールドをバッファに書き込む（双方向通信のため両方必要）
- `length()` メソッドは `32` を返す（24バイトヘッダー + 8バイトペイロード）
- `type()` メソッドは `TCNetMessageType.TimeSync`（値: 10）を返す

**Task 1.2: `TCNetPackets` マッピングへの登録（`src/network.ts`）**
- `TCNetPackets` オブジェクト内の `[TCNetMessageType.TimeSync]: null` を
`[TCNetMessageType.TimeSync]: TCNetTimeSyncPacket` に変更する
- 対応する `// 未実装` コメントを削除する

**Task 1.3: ユニットテストの追加（`tests/packet-parsers.test.ts`）**
- `TCNetTimeSyncPacket` 用の `describe` ブロックを追加する
- **read() テスト**: `writeValidHeader(buffer, TCNetMessageType.TimeSync)`
でバッファを準備し、offset 24/26/28 にテストデータを書き込み、`read()` 後のフィールド値を検証する
- **write() テスト**: フィールドを設定して `write()` を呼び出し、別インスタンスで `read()` してラウンドトリップを検証する
- **length() テスト**: 戻り値が `32` であることを検証する
- **type() テスト**: 戻り値が `TCNetMessageType.TimeSync`（10）であることを検証する
- Step 値のテストケースとして Initialize (0) と Response (1) の両方をカバーする

Research

TCNet パケットは src/network.ts で定義され、抽象クラス TCNetPacket を継承する。各パケットは read(), write(), length(), type() メソッドを実装し、フィールドは Buffer の絶対オフセットで読み書きする。TCNetPackets マッピングが TCNetMessageType から対応するパケットクラスへのディスパッチを管理している。TCNetMessageType.TimeSync = 10 は既に定義されているが、現在は null にマッピングされている。テストは tests/packet-parsers.test.ts で writeValidHeader() ヘルパーを使用したバッファ構築パターンに従う。

---

🚀 Next Steps

🤖 All AI agent prompts combined

既存の `TCNetOptInPacket` パターンを参考に、`src/network.ts` へ `TCNetTimeSyncPacket`
クラスを追加し、パケットレジストリへの登録とテストの追加を行うフェーズです。

**Task 1.1: `TCNetTimeSyncPacket` クラスの実装（`src/network.ts`）**
- 既存の他のパケットクラス付近に `TCNetPacket` を継承した `TCNetTimeSyncPacket` クラスを作成する
- 以下のフィールドを宣言する:
  - `step!: number`
  - `nodeListenerPort!: number`
  - `remoteTimestamp!: number`
- `read()` メソッドを実装し、以下のオフセットでバッファから読み込む:
  - offset 24: `step` (UInt8)
  - offset 25: RESERVED（スキップ）
  - offset 26: `nodeListenerPort` (UInt16LE)
  - offset 28: `remoteTimestamp` (UInt32LE)
- `write()` メソッドを実装し、`read()` と同じオフセットで各フィールドをバッファに書き込む（双方向通信のため両方必要）
- `length()` メソッドは `32` を返す（24バイトヘッダー + 8バイトペイロード）
- `type()` メソッドは `TCNetMessageType.TimeSync`（値: 10）を返す

**Task 1.2: `TCNetPackets` マッピングへの登録（`src/network.ts`）**
- `TCNetPackets` オブジェクト内の `[TCNetMessageType.TimeSync]: null` を
`[TCNetMessageType.TimeSync]: TCNetTimeSyncPacket` に変更する
- 対応する `// 未実装` コメントを削除する

**Task 1.3: ユニットテストの追加（`tests/packet-parsers.test.ts`）**
- `TCNetTimeSyncPacket` 用の `describe` ブロックを追加する
- **read() テスト**: `writeValidHeader(buffer, TCNetMessageType.TimeSync)`
でバッファを準備し、offset 24/26/28 にテストデータを書き込み、`read()` 後のフィールド値を検証する
- **write() テスト**: フィールドを設定して `write()` を呼び出し、別インスタンスで `read()` してラウンドトリップを検証する
- **length() テスト**: 戻り値が `32` であることを検証する
- **type() テスト**: 戻り値が `TCNetMessageType.TimeSync`（10）であることを検証する
- Step 値のテストケースとして Initialize (0) と Response (1) の両方をカバーする

💡 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!

[9c5s]

調査メモ

Bridgeバイナリ解析で cmdOfferTCNetSync / cmdOfferTCNetSyncResponse の存在を確認。Bridge側にTimeSync処理コードは実装されている。

ただしBridge-ShowKontrol間の60秒・41万パケットキャプチャではType=10は0件。通常運用では発生していない可能性がある (初回接続時や特定条件でのみ発火?)。

なお、Control (#52), Text (#53), Keyboard (#54) はBridgeに処理コードが存在せず死仕様として閉じた。TimeSyncはコードが存在するため残す。