7 Commits

Author	SHA1	Message	Date
Sylpheed port agent	e647368ddd	port: FORMAT.md declared the port's own export invalid; assert audio.json's stems ... Audits the open format spec against the validator by breaking each documented requirement. Four of five caught. The fifth is the doc's error: FORMAT.md said check refuses any peak >= 0 dBFS, where the implementation is kind-dependent -- a bgm is a sum we produced and is refused at full scale, an se/voice is a disc wave whose lossy decode overshoots and is allowed to +1.0. The doc was wrong about our own export: confirm ships at +0.18 and the ADV voice at +0.31. A consumer implementing a validator from FORMAT.md would have rejected a valid tree -- the file that exists so someone can check our work without trusting us. Corrected, with the +1.0 marked as a judgement. Also closes the last unread authored value: audio.json's was carried as stems_why only, so serde ignored the value. Now deserialised and asserted in the exporter (only sum is implemented); the assertion is proved to fire. Files, not fixes: a failed export leaves a tree with no manifest, and every tool then says 'is that an export tree?' -- which nearly made me conclude the validator was checking nothing. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01N7FiFFFwbvG2uxdcEh8HyF	2026-08-30 03:46:13 +00:00
Sylpheed port agent	5dbc9aeac0	port: delete exit_ramp_units, invert the format's own rule, and guard a scale-0 leaf ... FOUR THINGS, and the first is what MISSION section 3 calls the measure of progress. DELETED `exit_ramp_units` AND `exit_ramp_seconds`. They were authored because the disc had no time slot on a group's final keyframe, so the ramp into it was the one unknown duration per screen. Under the corrected record layout that keyframe does not exist -- a group is an 8-byte header then frames x {u32 time; 36-byte pose} and every pose is timed. VERIFIED DEAD BEFORE DELETING: setting it to 9999 (166 s) moved the boot's transitions by 0.04 s, which is wall-clock jitter, and both uses in ScreenView are gated on a condition that no longer fires on any of the export's 866 keyframes. INVERTED THE FORMAT'S OWN RULE. `check.rs` enforced "the final keyframe has no `t`; the disc has no time slot there" and FORMAT.md stated it. Both are now backwards, and the validator fired 150 times on a re-export. I had not run `check` between pinning the tag and measuring against the oracle -- the pixel harness was green while the format validator was failing on every screen with a multi-keyframe group. A correctness harness does not replace a format one; they fail at different layers. GUARDED A SCALE-0 LEAF, which the Decoder hit in its own renderer: its leaf branch marked the element drawn unconditionally while the blit returned early on zero scale, so a scale-0 leaf suppressed its parent and blanked the element -- live on all four loading screens. This port did not have the bug only because authored/rendering.json happens not to list pgloading_loop5. That is an accident of a gate written for another reason, not a defence, so `_draw_leaf` now reports whether it drew and `_draw` falls back to the parent. ISOLATED THE PACING QUESTION rather than leaving it as a suspected regression. Legacy association: publisher 4.70 agrees, developer 3.92 DIFFERS. Corrected: publisher 4.26 DIFFERS, developer 3.62 agrees. Both misses are ~0.03 s outside a composite bound. The association traded which screen is marginally out; it did not regress the pacing. Bumped the pin c -> d for the parser and audio changes. Its headline renderer change does not reach this port: sylpheed-cli builds from the workspace crate. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01N7FiFFFwbvG2uxdcEh8HyF	2026-08-29 19:19:46 +00:00
Sylpheed port agent	53a93e2e9e	port: the intro had no dialogue because the voice is a separate asset, and I concatenated it wrongly first ... A human play-test heard music under the boot intro and no voices. The obvious reading -- the 5.1 fold dropped the centre channel -- is wrong. `ADV.wmv` carries music and effects only; a cutscene's voice is a separate continuous XMA stream in `sound.pak`, bound to the movie by the manifest in `tables.pak`. Nothing was dropped. The exporter had never been asked for it, so every fidelity measurement in AUDIO-VERIFICATION.md would have come back clean. `audio::export_voice` resolves it with `media::resolve_movie_voice_region` and never by filename: `RT01A`'s voice lives inside `VOICE_ADV.slb`, so a name match is correct on exactly the two movies this port would have spot-checked. Decoded, not authored -- so it runs outside the `authored/audio.json` block. THE FIRST VERSION CONCATENATED THE REGION'S CHUNKS AND WAS WRONG. It produced 359 s of dialogue for a 137 s movie. Decoding and timing each chunk shows two of them equal to six decimals and each spanning the whole movie -- HANDOFF Q10's decoded two-stem shape on a second asset kind -- so they are summed at 1/n. The error was visible only because the first version recorded the decoded length against the movie's instead of clamping to it; the clamp `media`'s own doc comment invites, and which `sylpheed-viewer` applies, would have produced a file of exactly the right duration containing the wrong audio. The dropped leading chunk matches no duration in its region and is NOT closed here. It is the same signature as `BGM_103`'s third sub-wave, already open in BLOCKED.md, now corroborated on an independent asset kind. Raised with the Decoder; the manifest names every chunk dropped and its length. Also in this commit, and separable: * `--skip-at=SECONDS` -- `--script` structurally cannot press during a movie, because `_script_settled` waits while `_player != null`. That is why "does (A) skip the intro" had been read out of the source rather than measured. * MISSION section 6 pins a 5.1->stereo matrix and this exporter has shipped a different one since P4 -- the same weighting, 7.65 dB quieter -- and said so nowhere. Re-measured with the right instrument (float decode, whole file, count the samples that would clamp, not a peak reading): the pinned matrix puts ADV at +4.26 dBFS on 4406 samples, while S00A never clips. So the pin overloads one movie and the constant is over-broad for the other. NOT changed -- the level of a mix is what section 6 reserves to a human. The export now carries a warning with the numbers. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01N7FiFFFwbvG2uxdcEh8HyF	2026-08-29 14:51:38 +00:00
Sylpheed port agent	2ad839460c	port: P6 -- the menu has sound, and the BGM I "chose" was decoded all along ... The three Static.slb cues and the menu bed now export to Ogg Vorbis and play. `sylpheed_formats::media` does the assembly; nothing in port/ has heard of XMA. Three things this milestone got wrong before it got right, all recorded in docs/port/DECISIONS.md because the corrections are the useful part: 1. The cue offsets were a Rust `const` in the exporter. They are MEASURED, not decoded -- a measured value compiled into the exporter is a measurement wearing the costume of a decoded field, and nobody deletes it because nobody can see it. They are authored/audio.json now. 2. I picked BGM_001 and wrote a careful `why` calling the choice arbitrary. The menu's music is BGM_103, and it is in HANDOFF at 9ca1eb5 -- the exact commit BLOCKED.md says that row was reconciled against. Not stale: wrong when written. I had summarised a negative without its reach, so "the TABLES cannot say which BGM a screen plays" became "it is not on the disc". One word of scope was the whole answer, and the export failed only because BGM_001 without its .slb extension hashes to nothing. That is luck, not design. 3. The comment above the BGM sum argued for unity gain "because halving is a mix decision nobody made". It clipped at +1.8 dBFS. 1/n is the smallest constant that provably cannot clip -- the same reasoning video.rs already carried for its 5.1 downmix, in this repository, unread. Unsettled and shipped as such: media::sound_bank_riffs returns THREE sub-waves for BGM_103.slb where HANDOFF Q10's census says exactly two (the third is the leading headerless region slb.rs emits for the voice path). The exporter sums all three and writes a manifest warning, because which bytes belong together is the decoders' question, not this exporter's -- and dropping one would destroy the evidence, since a corrected export looks exactly like a correct one. Raised with the Decoder; row in BLOCKED.md. The gate is a null control, not a peak reading. A master-bus WAV that is non-silent proves nothing -- the bed alone would look identical. So the same scripted walk was run with <- in place of <v>, which fires no cue (Q5, measured), and the difference is one 0.55 s burst at t=1.10 s and silence everywhere else. The first attempt at that control returned bit-identical zero and I nearly filed it as "cues never reach the bus": both runs ended at 1.115 s and the first press lands at 1.17 s. A null result from an instrument that was not running is not a null result. Refutation attempt: HANDOFF Q8's three cue durations. They looked attackable -- 0.133/0.172/0.169 s per packet, no shared rate -- but an XMA1 packet carries a variable number of 512-sample frames, and the three come to 50.0/32.3/95.3 frames. Measured off the decoded Ogg: 0.533, 0.344, 1.016 s, every published digit. SURVIVES, with its reach stated -- it confirms the assembly path and my transcription, not the event bindings, which only an oracle can retake. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WM5XL4HfrHuxz8RiMWdCMC	2026-08-29 12:29:13 +00:00
MechaCat02	65cefa74c3	monorepo: one repository for the decoders, the port and the corpus ... Merges the Godot port into the reverse-engineering repository, preserving both histories -- 1019 commits of corpus plus the port's 31, brought in by subtree merge and then moved into place so git can follow each file across the rename. The reason is not tidiness. The two-repo split forced the exporter to depend on the decoders by pinned revision, and that created a whole class of failure that now disappears: a sha reachable only from a topic branch, orphaned by a squash-merge, breaking a fresh checkout silently at build time. It also forced a live read-only mount of one agent's working tree into another's container, which is why a contract file could move mid-iteration. With a path dependency, a decoder change and the exporter change it requires land in the same commit or not at all. Canary stays separate: it is a fork tracking upstream. New structure for the long term: docs/game/ how the game is NAVIGATED -- menus, modals, prompts, alerts, and in-game flight. Written so nobody rediscovers it. Mostly open questions on purpose; the in-game tutorials are the resource for the flight half. docs/port/MODDING.md modding as a constraint on the exporter TODAY, not a later feature: one logical asset in one file (the disc splits nearly everything, and resolving that is the exporter's job), names a person recognises, PNG/OGG/OGV/JSON only, base-and-overrides so re-exporting is always safe, provenance in every file. data/base + data/mods generated tree and drop-in overrides, both gitignored exchange/ transient inter-agent files, deliberately outside history docs/agents/ the team protocol Both the README and the navigation doc lead with the correction that cost the most: the oracle is the real game under Xenia Canary. Reborn's renderer is a hypothesis under test, it has been wrong, and treating it as ground truth propagated into three documents and both agents before a human caught it. Scripted modding stays possible without being built: no screen name is hardcoded in GDScript and there is no native code in port/, which is what Godot Mod Loader needs to be able to substitute behaviour later.	2026-08-29 11:34:46 +02:00
Sylpheed RE agent	ecbd70f69a	port: the container agent answers the questions, it does not build the port ... Corrects the split I got wrong in b5a1938. The container is for reverse engineering, now focused on the menu port; a SEPARATE agent builds the port from its investigation results. My previous version had the container agent writing the exporter and the Godot project, which is not the intent. This lands on the research/engineering line that was already in the estimate: the container agent takes the research half, the port agent the engineering half. docs/port/MISSION.md is now a list of open QUESTIONS (Q1-Q9) rather than build milestones, ordered by what blocks the port earliest -- the keyframe time unit, which build is which screen state, paint order for the six screens, button -> GamePart, navigation semantics, the boot sequence driver, transitions, menu audio bindings, and video binding. Each is done when a written result with evidence exists, not when something compiles. S1, the Ready Room probe, stays gated at one iteration and a go/no-go. Most of these are BEHAVIOUR questions -- timing, transitions, what a d-pad press does at the end of a list -- so the mission and the loop prompt both push hard on measuring the oracle rather than reasoning from the file. docs/port/FORMAT.md is deleted. The export schema is the port agent's design and was not mine to specify. It is replaced by docs/port/HANDOFF.md, the single page the port agent reads: a status table, what is already settled and can be relied on today, and the facts that will trip the port up (the WMV3/WMA Pro intro, the Static.slb size over-declaration, the voice-vs-music downmix, JNGL_001). The derived/authored idea survives as the thing it always was -- a finding, not a design. Every answer must be classified DECODED, MEASURED or UNDECODABLE-with- reach, and never a fourth thing, because measured and undecodable both mean the port agent is authoring that value and has to know it. Labelling a guess as a decode would put it into the port wearing the badge of a measurement. Reverts the Godot install from the RE container, its AGENT.md section, and the export/ gitignore entry -- none of that belongs on this side of the wall. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-28 17:16:47 +02:00
Sylpheed RE agent	b5a193839c	port: make the Godot menu shell the agent's primary objective ... Sets a new mission: boot the real disc through developer splash -> intro video -> title -> main menu -> submenus in Godot 4, interactively, with no gameplay, no 3D and no emulator. docs/port/MISSION.md defines it -- eight gated milestones, each finished by an ARTIFACT rather than by compiling, plus the Ready Room as an explicitly gated stretch goal with a one-iteration probe that decides go/no-go. GP_READY_ROOM is 1106 entries with 6 recoverable names and is ISL-scripted, so it is either a week or a quarter, and the agent must not start it on its own authority. Architecture, per the user's decision: the Godot project is INDEPENDENT of the Rust viewer and never reads a disc format. An offline Rust exporter converts the disc into open formats; Godot reads only those. No GDExtension, no Rust in the Godot project, and sylpheed-viewer is off limits -- it stays the human's verification tool with its static-data rule intact. docs/port/FORMAT.md specifies the open format, versioned, because modding is the port's second goal and that makes the layout a deliverable rather than a temp directory: JSON over XML (Godot parses JSON natively; its XMLParser is SAX), names never hashes, provenance in every generated file, and unknowns listed rather than guessed. The discipline the whole thing rests on is the derived/authored split. `export/` is regenerated wholesale and never hand-edited; `authored/` is hand-written and survives a re-export. Three things this milestone needs are NOT on the disc in any decoded form -- which button does what, paint order, and menu sound cues -- so they live in `authored/` with a stated `why`. Deleting an authored entry because the exporter can now emit it IS the measure of progress. `export/` is gitignored: it is generated from the user's own disc and this stays a clean-room repo. Container: adds a pinned Godot 4 (windowed under Xvfb for screenshots, plus a headless wrapper). ffmpeg already carries libtheora, which is the video target -- Godot 4 plays only Ogg Theora natively and the disc's ADV.wmv is WMV3/WMA Pro. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-28 17:11:35 +02:00