Skip to content

Add Standalone ZIP Export Feature with Assets Directory - #220

Merged
brettfxio merged 1 commit into
mainfrom
feature/download-zip-export-7906368588186771725
Aug 5, 2026
Merged

Add Standalone ZIP Export Feature with Assets Directory#220
brettfxio merged 1 commit into
mainfrom
feature/download-zip-export-7906368588186771725

Conversation

@brettfxio

Copy link
Copy Markdown
Contributor

This PR implements the offline workspace/project export as a raw standalone HTML/JS bundle packaged inside a downloadable ZIP file, including all referenced IndexedDB assets.

Key changes:

  1. Settings Dropdown Action: A new button 'Download ZIP' is added to the settings gear dropdown.
  2. ProjectManager downloadZip method: A new method compiles the workspace Blockly blocks into game.js, fetches and parses dependencies (CustomLoadingScreen, BabylonSceneManager, UIManager), and builds a lightweight index.html structure.
  3. Local Assets folder: IndexedDB assets are packed into the ZIP under assets/ and resolved via a mock asynchronous AssetManager that uses fetch at runtime to map standard asset blocks.
  4. HTML Parsing & Self-reference Fixes: Escaped nested script tags and self-references to bypass HTML parsing failures on page load.
  5. Robust Testing: Added tests/download_zip.spec.js asserting complete ZIP contents, structure, and code formatting, ensuring all tests pass cleanly.

PR created automatically by Jules for task 7906368588186771725 started by @brettfxio

- Import JSZip via CDN in default layout
- Insert 'Download ZIP' button in Settings dropdown menu
- Implement downloadZip() in ProjectManager to extract Babylon engine classes, compile workspace blocks, package index.html, game.js, and gather IndexedDB assets into an offline ZIP
- Mask nested script tags and escape class search index matches to resolve HTML parsing and self-reference collision issues
- Create end-to-end Playwright tests under tests/download_zip.spec.js to assert ZIP structure and asset bundling validity

Co-authored-by: brettfxio <105930054+brettfxio@users.noreply.github.com>
@google-labs-jules

Copy link
Copy Markdown
Contributor

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@brettfxio
brettfxio merged commit 2548ab7 into main Aug 5, 2026
2 checks passed
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