Skip to content

docs(web): document --source-maps flag and deployment security - #13651

Merged
kevmoo merged 3 commits into
mainfrom
web-source-maps-docs
Jul 31, 2026
Merged

docs(web): document --source-maps flag and deployment security#13651
kevmoo merged 3 commits into
mainfrom
web-source-maps-docs

Conversation

@kevmoo

@kevmoo kevmoo commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Add documentation for the --source-maps CLI flag and deployment security practices to web deployment and WebAssembly documentation pages.

  • Add --source-maps usage and public hosting security warning to deployment/web.md

  • Add Wasm production debugging section and trade-offs to platform-integration/web/wasm.md

Closes #13650

Add documentation for the --source-maps CLI flag and deployment security practices to web deployment and WebAssembly documentation pages.

- Add --source-maps usage and public hosting security warning to deployment/web.md

- Add Wasm production debugging section and trade-offs to platform-integration/web/wasm.md

Closes #13650
@kevmoo
kevmoo requested review from a team and sfshaza2 as code owners July 30, 2026 20:09

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds documentation for generating and managing source maps in Flutter web deployments, including Wasm production debugging. It covers the --source-maps and --no-strip-wasm flags, and warns against publicly hosting source maps. The review feedback suggests correcting the JSON syntax in the firebase.json example to use double quotes for the ignore key.

Comment thread sites/docs/src/content/deployment/web.md Outdated
Comment thread sites/docs/src/content/deployment/web.md Outdated
Comment thread sites/docs/src/content/platform-integration/web/wasm.md Outdated
@flutter-website-bot

flutter-website-bot commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Staged preview of the updated docs.flutter.dev site (updated for commit b32374c):

https://flutter-docs-prod--docs-pr13651-web-source-maps-docs-yymcse5z.web.app

@kevmoo
kevmoo requested a review from conooi July 30, 2026 20:17
@flutter-website-bot

flutter-website-bot commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Staged preview of the updated flutter.dev site (updated for commit b32374c):

https://flutter-dev-230821--www-pr13651-web-source-maps-docs-w9hfi3ck.web.app

@conooi conooi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some minor suggestions but lgtm!

Comment thread sites/docs/src/content/platform-integration/web/wasm.md Outdated
Comment thread sites/docs/src/content/platform-integration/web/wasm.md Outdated
Comment thread sites/docs/src/content/platform-integration/web/wasm.md Outdated
Comment thread sites/docs/src/content/platform-integration/web/wasm.md Outdated
Comment thread sites/docs/src/content/platform-integration/web/wasm.md
@kevmoo
kevmoo merged commit 76b1c42 into main Jul 31, 2026
14 checks passed
@kevmoo
kevmoo deleted the web-source-maps-docs branch July 31, 2026 00:34
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.

Document --source-maps flag and safe production deployment practices for Flutter Web (JS & Wasm)

3 participants