Skip to content

fix(site): GLB model for GitHub Pages + gallery, journal section, Mermaid diagrams#103

Merged
AndrewAltimit merged 4 commits intomainfrom
fix/glb-no-lfs
Mar 31, 2026
Merged

fix(site): GLB model for GitHub Pages + gallery, journal section, Mermaid diagrams#103
AndrewAltimit merged 4 commits intomainfrom
fix/glb-no-lfs

Conversation

@AndrewAltimit
Copy link
Copy Markdown
Owner

Summary

  • Fix PSP 3D model on GitHub Pages -- GitHub Pages doesn't serve Git LFS files. Removed LFS tracking for .glb and stored as regular binary so the real PSP model renders instead of the procedural fallback.
  • Blueprint gallery index -- blueprint.html without ?diagram= now shows a card gallery with thumbnail images of both diagrams instead of auto-loading the first one.
  • README Hardware Blueprints section -- Thumbnail table with links to both interactive 3D diagrams on GitHub Pages.
  • Developer's Journal section on site index -- 4 featured article cards (Media Engine RE, TLS 1.3, USB Research, Zero-Touch Dev) with links to full journal and blueprints. Added Blueprints to nav bar.
  • 5 Mermaid diagram conversions across 4 journal entries -- deployment topology, CDN redirect flow, PSP streaming pipeline, frame rendering flow, dependency direction. Dark theme styling.

Test plan

Generated with Claude Code

AndrewAltimit and others added 4 commits March 31, 2026 06:34
  GitHub Pages doesn't serve Git LFS files — it returns the pointer
  file instead of the actual binary, causing the GLTFLoader to fail
  and fall back to the procedural PSP. Remove LFS tracking for .glb
  and store as a regular 2.7MB binary in the repo.

  Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…DME cards

- Blueprint gallery page when no ?diagram= param is given, showing
  both hardware diagrams as clickable cards with thumbnail images
- Engine shows gallery index instead of loading first diagram by default
- README: Hardware Blueprints section with thumbnail table linking to
  interactive 3D diagrams on GitHub Pages
- Site index: Developer's Journal section with 4 featured article cards
  (Media Engine RE, TLS 1.3, USB Research, Zero-Touch Dev) plus links
  to full journal and blueprints. Added Blueprints to nav bar.
- Convert 5 ASCII art diagrams to Mermaid across 4 journal entries:
  deployment topology, CDN flow, PSP streaming pipeline, frame
  rendering flow, dependency direction. Dark theme styling.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add screenshot thumbnails of the interactive 3D blueprint viewer to
all locations that link to diagrams:
- README Hardware Blueprints section: 3D screenshots replace 2D diagrams
- Journal entry 05: 3D adapter thumbnail below the 2D schematic
- Journal entry 08: 3D relay thumbnail below the 2D wiring diagram
- Blueprint gallery: 3D screenshots as card thumbnails

Each thumbnail links to the interactive viewer. The 2D schematics are
kept as the primary technical reference; the 3D thumbnails serve as a
visual preview of the interactive experience.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

Gemini AI Code Review

Issues (if any)

  • [CRITICAL] site/journal/01-unified-embedded-os.html:319 - Invalid Mermaid JS import URL
    • The CDN URL contains an erroneous space (mermaid @11), causing a 404/syntax error that breaks diagram rendering.
    • Remove the space: https://cdn.jsdelivr.net/npm/mermaid@11/dist/mermaid.esm.min.mjs. Same error in:
      • site/journal/02-tls-on-psp.html:333
      • site/journal/03-streaming-video.html:359
      • site/journal/06-backend-trait-system.html:437

Previous Issues (for incremental reviews)

(none)

Suggestions (if any)

(none)

Notes

  • The LFS tracking removal for *.glb looks correct and should allow GitHub Pages to serve the 3D model properly.
  • The gallery layout in site/blueprint.html handles the missing ?diagram= parameter correctly.
  • site/journal/07-psp-media-engine.html was not updated with Mermaid diagrams, but this matches the PR description which specified 5 conversions across 4 entries.

Generated by Gemini AI (gemini-3.1-pro-preview). Supplementary to human reviews.

@AndrewAltimit
Copy link
Copy Markdown
Owner Author

Review Response Agent (Iteration 1)

Status: No changes needed

Fixed Issues

  • (none)

Ignored Issues

  • [CRITICAL] Mermaid JS import URL "erroneous space" — Gemini reported that mermaid @11 contains a space, but verification shows the actual URLs in all four files are mermaid@11 (no space). The @ is the standard npm version specifier. This is a false positive.

Deferred to Human

  • (none)

Notes

  • All four Mermaid import URLs (01-unified-embedded-os.html:319, 02-tls-on-psp.html:333, 03-streaming-video.html:359, 06-backend-trait-system.html:437) are correctly formatted as https://cdn.jsdelivr.net/npm/mermaid@11/dist/mermaid.esm.min.mjs.
  • The Gemini review's other observations (LFS removal correctness, gallery parameter handling) were informational and required no action.

The agent reviewed feedback but determined no code changes were required.

@AndrewAltimit AndrewAltimit merged commit 211099f into main Mar 31, 2026
7 checks passed
@AndrewAltimit AndrewAltimit deleted the fix/glb-no-lfs branch March 31, 2026 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant