Skip to content

refactor: extract embedded static assets#1084

Open
lapismyt wants to merge 1 commit into
ton-blockchain:masterfrom
lapismyt:refactor/extract-static-assets
Open

refactor: extract embedded static assets#1084
lapismyt wants to merge 1 commit into
ton-blockchain:masterfrom
lapismyt:refactor/extract-static-assets

Conversation

@lapismyt

Copy link
Copy Markdown

This refactors large embedded production string literals into static files loaded with include_str!.

@lapismyt
lapismyt requested a review from i582 as a code owner May 19, 2026 17:26
@netlify

netlify Bot commented May 19, 2026

Copy link
Copy Markdown

Deploy Preview for acton-staging ready!

Name Link
🔨 Latest commit 4c5abf2
🔍 Latest deploy log https://app.netlify.com/projects/acton-staging/deploys/6a0c9d631b21e7000876f5bd
😎 Deploy Preview https://deploy-preview-1084--acton-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the ton-emulator Tasks related to TON emulator and executor label May 19, 2026
@Danil42Russia

Copy link
Copy Markdown
Member

Why is this necessary, and what problem does it solve?

@lapismyt

Copy link
Copy Markdown
Author

Why is this necessary, and what problem does it solve?

This is not required for runtime behavior; it is a maintainability refactor.

These strings are production assets, but they were embedded as very large Rust literals. That makes the Rust modules harder to read, makes diffs noisy, and makes editing/reviewing the actual asset content awkward. Moving them into files keeps the code focused on behavior while letting the assets live in their native formats: HTML as HTML, licenses as text files, base templates as template files, and the default config as a BOC64 asset.

The binary behavior stays the same because the files are still embedded at compile time with include_str!. There is no runtime file lookup or packaging change. I also verified the extracted DEFAULT_CONFIG byte-for-byte against the previous inline string, so this should be behavior-preserving.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ton-emulator Tasks related to TON emulator and executor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants