fix: 4 находки из глубокого ревью конвертера

Глубокий adversarial-проход по алгоритму IR-кодирования (compressor greedy/optimal, decoder, encoder, base64, round-trip) нашёл 4 подтверждённых недочёта. Три критичные подсистемы (LZ77-компрессор, DP-парсер, decoder-рефакторинг) прошли ревью без замечаний.

- tuya-encoder.ts: фильтр `t < MAX_SIGNAL_VALUE` → `t <= MAX_SIGNAL_VALUE`.
  Тайминг ровно 65535 (max uint16 LE) — валидное значение, но
  силентно отбрасывался. Golden-тесты по-прежнему совпадают —
  фикстура не содержит таких значений.

- smartir.ts (processCommands): non-string листья (число, boolean, null)
  и массивы больше не проходят молчаливым passthrough. Раньше вход
  `{"commands":{"heat":{"17":42}}}` давал внешне валидный JSON с
  `{"17":42}` — MOES UFO-R11 такой payload не принял бы, но
  пользователь узнавал об этом только на устройстве. Теперь бросается
  IRCodeError с указанием ключа и типа.

- golden.test.ts: тест "should decode correct number of timings"
  раньше сверял константы фикстуры сами с собой (BroadlinkDecoder не
  вызывался). Теперь реально декодирует testInput и сравнивает
  с фикстурой — off-by-one в parseTimings был бы пойман.

- +5 новых тестов: boundary encoder (65535 сохраняется, >65535
  отсеивается), 4 случая с non-string/массивом в SmartIR commands.

Всего: 36 → 42 tests, все зелёные.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: dzerik

frontend/package.json

@@ -1,6 +1,6 @@
 {
   "name": "btu-frontend",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "private": true,
   "scripts": {
     "dev": "next dev --turbopack",

frontend/src/__tests__/converter/encoder-errors.test.ts

@@ -1,5 +1,6 @@
 import { describe, it, expect } from 'vitest';
 import { IRConverter, IRCodeError } from '../../lib/converter';
+import { TuyaEncoder } from '../../lib/converter/tuya-encoder';
 
 /**
  * Строит валидный Broadlink Base64 из массива таймингов (в единицах ~unit).
@@ -43,4 +44,22 @@ describe('TuyaEncoder — фильтрация и граничные случа
     expect(typeof result).toBe('string');
     expect(result.length).toBeGreaterThan(0);
   });
+
+  it('тайминг ровно 65535 (max uint16) НЕ отсеивается', () => {
+    // Тестирую границу напрямую в encoder'е: [500, 65535, 500] должен
+    // упаковать все три сэмпла, а не два.
+    const encoder = new TuyaEncoder();
+    const result = encoder.encode([500, 65535, 500]);
+    expect(result.length).toBeGreaterThan(0);
+    // Обратный маркер: если бы 65535 отбрасывался, вход [500, 500] дал
+    // бы КОРОЧЕ результат. Сравним.
+    const shorter = encoder.encode([500, 500]);
+    expect(result.length).toBeGreaterThan(shorter.length);
+  });
+
+  it('тайминги > 65535 отсеиваются', () => {
+    const encoder = new TuyaEncoder();
+    expect(() => encoder.encode([70000, 80000])).toThrow(IRCodeError);
+    expect(() => encoder.encode([70000, 80000])).toThrow(/All timings filtered/);
+  });
 });

frontend/src/__tests__/converter/golden.test.ts

@@ -1,18 +1,19 @@
 import { describe, it, expect } from 'vitest';
 import { IRConverter, CompressionLevel } from '../../lib/converter';
+import { BroadlinkDecoder } from '../../lib/converter/broadlink-decoder';
 import goldenData from '../fixtures/golden-data.json';
 
 describe('Golden tests — output must match Python converter', () => {
   const testInput = goldenData.test_input;
 
   it('should decode correct number of timings', () => {
-    const converter = new IRConverter(CompressionLevel.BALANCED);
-    // Access decoder indirectly through a known output
-    // The decoded_timings_count is 200 and first10 are known
-    expect(goldenData.decoded_timings_count).toBe(200);
-    expect(goldenData.decoded_timings_first10).toEqual([
-      4447, 4386, 579, 1584, 579, 518, 549, 1584, 579, 1584,
-    ]);
+    // Реально вызываем decoder на testInput и сравниваем с фикстурой —
+    // раньше тест сверял константы фикстуры сами с собой и пропустил бы
+    // любую регрессию в parseTimings.
+    const decoder = new BroadlinkDecoder();
+    const timings = decoder.decode(testInput);
+    expect(timings).toHaveLength(goldenData.decoded_timings_count);
+    expect(timings.slice(0, 10)).toEqual(goldenData.decoded_timings_first10);
   });
 
   it('should match Python output at compression level NONE (0)', () => {

frontend/src/__tests__/converter/smartir.test.ts

@@ -1,6 +1,7 @@
 import { describe, it, expect } from 'vitest';
 import {
   IRConverter,
+  IRCodeError,
   CompressionLevel,
   isSmartIRData,
   countSmartIRCommands,
@@ -108,14 +109,17 @@ describe('IRConverter.processSmartIRData', () => {
   it('prototype pollution через __proto__ не подменяет [[Prototype]]', () => {
     // Payload с ключом "__proto__" на верхнем уровне commands: без защиты
     // processed["__proto__"] = innerObject через нативный setter меняет
-    // [[Prototype]] аккумулятора → getters с prototype-цепочки начинают
-    // возвращать "evil" данные. С Object.create(null) __proto__ становится
-    // обычным own-property, и .evil остаётся undefined.
-    const payload = JSON.parse('{"commands":{"__proto__":{"evil":true}}}');
+    // [[Prototype]] аккумулятора → доступ через prototype-цепочку возвращает
+    // "внешние" данные. С Object.create(null) __proto__ становится обычным
+    // own-property, и посторонний ключ остаётся невидимым.
+    const payload = JSON.parse(
+      `{"commands":{"__proto__":{"pwned":${JSON.stringify(validCode)}}}}`
+    );
     const result = converter.processSmartIRData(payload);
-    expect((result.commands as Record<string, unknown>).evil).toBeUndefined();
-    // Object.prototype глобально не загрязнён:
-    expect(({} as Record<string, unknown>).evil).toBeUndefined();
+    // pwned не доступен как свойство результата через prototype-цепочку:
+    expect((result.commands as Record<string, unknown>).pwned).toBeUndefined();
+    // И Object.prototype глобально не загрязнён:
+    expect(({} as Record<string, unknown>).pwned).toBeUndefined();
   });
 
   it('commands=null не бросает TypeError (валидатор отсеивает раньше)', () => {
@@ -124,4 +128,18 @@ describe('IRConverter.processSmartIRData', () => {
     const result = converter.processSmartIRData({ commands: undefined });
     expect(result.commands).toEqual({});
   });
+
+  it.each([
+    ['число как лист', { commands: { heat: { '17': 42 } } }, /number/],
+    ['boolean как лист', { commands: { power: true as unknown } }, /boolean/],
+    ['null как лист', { commands: { off: null } }, /null/],
+    ['массив в команде', { commands: { turn_on: ['some-code'] } }, /array/],
+  ])('бросает IRCodeError на %s вместо тихого passthrough', (_desc, input, expected) => {
+    expect(() =>
+      converter.processSmartIRData(input as SmartIRData)
+    ).toThrow(IRCodeError);
+    expect(() =>
+      converter.processSmartIRData(input as SmartIRData)
+    ).toThrow(expected as RegExp);
+  });
 });

frontend/src/lib/converter/smartir.ts

@@ -1,4 +1,5 @@
 import { IRConverter } from './index';
+import { IRCodeError } from './errors';
 
 export interface SmartIRData {
   [key: string]: unknown;
@@ -79,16 +80,28 @@ function processCommands(
       processed[key] = wrapWithIrCode
         ? `{"ir_code_to_send": "${irCode}"}`
         : irCode;
-    } else if (Array.isArray(value)) {
-      processed[key] = value;
-    } else if (value !== null && typeof value === 'object') {
+    } else if (
+      typeof value === 'object' &&
+      value !== null &&
+      !Array.isArray(value)
+    ) {
       processed[key] = processCommands(
         value as Record<string, unknown>,
         converter,
         wrapWithIrCode
       );
     } else {
-      processed[key] = value;
+      // Массивы, числа, boolean, null — не поддерживаются как значения
+      // команд. Молчаливый passthrough давал внешне валидный JSON, но
+      // MOES UFO-R11 не смог бы его обработать. Явная ошибка лучше.
+      const kind = value === null
+        ? 'null'
+        : Array.isArray(value)
+          ? 'array'
+          : typeof value;
+      throw new IRCodeError(
+        `Unsupported command value type at "${key}": ${kind} (expected string or nested object)`
+      );
     }
   }
 

frontend/src/lib/converter/tuya-encoder.ts

@@ -15,7 +15,9 @@ export class TuyaEncoder {
       throw new IRCodeError('Empty timings list');
     }
 
-    const filtered = timings.filter(t => t < MAX_SIGNAL_VALUE);
+    // <= вместо <: MAX_SIGNAL_VALUE (0xFFFF) — сам максимум uint16 LE,
+    // а не эксклюзивная верхняя граница; setUint16 принимает и 65535.
+    const filtered = timings.filter(t => t <= MAX_SIGNAL_VALUE);
     if (!filtered.length) {
       throw new IRCodeError('All timings filtered out');
     }