test(crypto): fix 1/256 flaky tampered-ciphertext test

[CallMeTechie]

Flaky-Test-Fix: crypto.test.js „throws on tampered ciphertext"

Symptom: Build & Release #197 (Merge feature/pihole-followups) schlug an genau diesem Test fehl (Missing expected exception, 1 von 1913) → release-Job übersprungen.

Ursache (NICHT die PR-Logik): Der Test „tampert" den Ciphertext, indem er das erste Byte mit 'ff' überschreibt:

parts[2] = 'ff' + parts[2].slice(2);

Das Ciphertext-Format ist iv:tag:encrypted (Hex, AES-256-GCM, zufälliger IV pro Aufruf). Beginnt die zufällige Nutzlast bereits mit 0xff (≈1/256 pro Lauf), ist das „Tampern" ein No-Op → der GCM-Auth-Tag bleibt gültig → decrypt() wirft NICHT → assert.throws schlägt fehl. Reiner ~0,39 %-Flake.

Fix: Deterministisch tampern via XOR-Flip (matcht die bestehende Konvention in guac_token.test.js, buf[...] ^= 0xff):

const buf = Buffer.from(parts[2], 'hex');
buf[0] ^= 0xff;            // garantiert geändertes Byte
parts[2] = buf.toString('hex');

Evidenz:

• Fix: crypto.test.js 200/200 grün (deterministisch).
• Alt: empirisch 26/5120 (~0,51 %) No-Op-Tamper — bestätigt den 1/256-Mechanismus.

Nur die Test-Datei geändert, keine Produktionslogik. Geschwister-Tamper-Tests (guac_token.test.js, e2ee.test.js) nutzen bereits sichere Muster — kein weiterer Handlungsbedarf.

🤖 Generated with Claude Code