Repository navigation
fix(werkstoff): Länge vor Magic, Regelbetrieb-Pin, Bundle-Wächter vor den Tests (Nachzieh #48) - #49
Merged
Merged
Conversation
… den Tests (Nachzieh craft#48 + cody-cad#71 S2) - glbKnoten: eine Datei unter 20 Bytes ist »zu kurz für einen Header (N Bytes, mindestens 20)« — auch mit gültigem Magic, und VOR der Magic-Prüfung. Bisher hieß beides »Magic fehlt«; bei einem abgebrochenen Download sah der Handwerker die falsche Ursache. Parität zu cody-cad#71 (Fälle 0/3/12/19 → zu kurz, 20 → leerer JSON-Chunk). - panel.test.tsx: Regelbetrieb-Pin — das ausgelieferte Bundle unverändert durch die Seite: Szene mit URL und Lücke, kein Fehlersatz (R48b-5). Veralteter Kommentar (»Modell nicht in git«) korrigiert. - build-demo-bundle.sh: der Auftrag↔Modell-Wächter steht jetzt VOR pnpm test — eigenes Schloss, nicht das zweite an derselben Tür (R48b-4). Beweis: Red-First — vier »zu kurz«-Fälle rot vor dem Fix (Meldung »Magic«); Saboteur »Magic vor Länge« (Byte-Vergleich) → genau 0-Bytes + 3-Bytes rot; Saboteur »ein Zeichen im ausgelieferten glb_sha256« → Regelbetrieb-Pin rot (+ bundle.test.ts, + Knoten-Zweig-Test, der dasselbe Paar braucht), der Fixture-Test bleibt grün — der Pin hält den Weg des Menschen, nicht das Fixture. Werkstoff 88/88 (vorher 82), Suite 452 + 24 Altbestand, tsc 0, eslint 0. Nicht enthalten: R48b-6 (»Demo-Plan ohne Bohrungen« als getragene Lücke im Auftrag) — Vertragsänderung in drei Repos, Vorschlag im PR. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Owner
Author
Tiefen-Review (Cody #2, Head 6b7d7e4) — Grün ohne BedingungGemessen im Prüfstand, CI-Form: 452 passed / 24 skipped (alle Altbestand), Werkstoff 88/88 (+6 = genau die neuen Tests), core 149/149, tsc 0, lint 0,
Notizen für den nächsten Satz (mit R48b-6), kein Blocker:
Merge als Reviewer. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Folge-PR zu #48 (gemergt
0d22916) — die Nachziehposten R48b-4/5 von Cody #2 plus die craft-Parität S2 aus dem Review von cody-cad#71.Was
glbKnoten— Länge vor Magic (S2 aus cody-cad#71). Eine Datei unter 20 Bytes ist „glTF-Binary zu kurz für einen Header (N Bytes, mindestens 20)" — auch mit gültigem Magic, und VOR der Magic-Prüfung. Bisher hieß beides „Kein glTF-Binary (Magic »glTF« fehlt)": bei einem abgebrochenen Download sah der Handwerker die falsche Ursache. Trennlinie wie in cody-cad: 0/3/12/19 Bytes → „zu kurz"; 20 Bytes (Header mit leerem JSON-Chunk) → „JSON-Chunk unlesbar", nicht „zu kurz".panel.test.tsx: das ausgelieferte Bundle unverändert durch die Seite — Szene mitdata-url, Lücketeil:Rw, vier Schaltflächen, kein ⚠, kein „Kein 3D-Modell". Der Weg des Menschen, nicht der des Fixtures. Veralteter Kommentar („Modell nicht in git") korrigiert.build-demo-bundle.sh: Auftrag↔Modell-Wächter jetzt VORpnpm test— eigenes Schloss, nicht das zweite an derselben Tür.Beweis (Red-First)
0 Bytes+3 Bytesrot, 33 grün. Ein erster Saboteur mitbyteLength >= 4 &&vor dem Magic hatte NICHTS rot gemacht: er hatte die Längenprüfung mit hineingeschmuggelt (Regel 17 — der Saboteur muss an der Stelle sitzen, die geprüft wird).glb_sha256" → Regelbetrieb-Pin rot (+bundle.test.ts2, + Knoten-Zweig-Test, der dasselbe Paar braucht); der Fixture-Test (eigener Hash) bleibt grün — der Pin hält das ausgelieferte Paar, nicht das Fixture.bash -ngrün ·cm:gate-verifygrün.Nicht enthalten — R48b-6 („Demo-Plan ohne Bohrungen" als getragene Lücke)
Das ist eine Vertragsänderung in drei Repos (cody-cad
projektion.pyschreibt den Auftrag; voaiwerkstoff.schema.tsiststrictObjectund lehnt ein unbekanntes Feld ab; craft zeigt es). Vorschlag:auftrag.hinweise?: string[](optional, Klartext, max. 5 × 200 Zeichen), gesetzt vom Bundle-Erzeuger (--hinweis "Demo-Plan ohne Bohrungen — Bohrbild kommt mit dem großen Modell"), in voai als optionales Feld, in craft als Zeile unter dem Plan-Hash. Reihenfolge: voai (Schema, tolerant) → cody-cad (schreibt) → craft (zeigt). Eigener PR-Satz nach Absprache mit Cody #2.Doppel-Grün: CI + Tiefen-Review des anderen Cody; Reviewer mergt.
🤖 Generated with Claude Code