FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add source maps documentation by mununki · Pull Request #1333 · rescript-lang/rescript-lang.org · GitHub

/ rescript-lang.org Public

Add source maps documentation - #1333

Open
mununki wants to merge 2 commits into
v13from
doc/sourcemap
ConversationCommits (2)ChecksFiles changed
Open

Add source maps documentation#1333
mununki wants to merge 2 commits into
v13from
doc/sourcemap

Conversation

mununki commented Aug 24, 2026

Copy link
Copy Markdown
Member

Summary

  • document the new ReScript v13 source map configuration
  • explain linked, inline, and hidden output modes, etc.
  • add usage examples for Node.js and @rescript/vite-plugin

chatgpt-codex-connector Bot reviewed Aug 24, 2026
View reviewed changes

chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9ba5a9881a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread apps/docs/markdown-pages/docs/manual/source-maps.mdx Outdated
Node.js can use linked or inline maps to report `.res` locations in stack traces:

```sh
node --enable-source-maps ./src/Main.mjs

chatgpt-codex-connector Bot Aug 24, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Use the generated .res.mjs filename in the Node command

For projects created from the documented basic template, a Main.res module is emitted as Main.res.mjs (the installation guide likewise runs src/Demo.res.mjs), so this command targets a nonexistent file and exits before demonstrating source-map support. Use ./src/Main.res.mjs, or explicitly state that Main.mjs assumes a custom "suffix": ".mjs" configuration.

Useful? React with 👍 / 👎.

cloudflare-workers-and-pages Bot commented Aug 24, 2026 •
edited
Loading

Copy link
Copy Markdown

Deploying with    Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs		 guide a635ed6 Commit Preview URL

Branch Preview URL		 Aug 25 2026, 04:19 PM

fhammerschmidt commented Aug 25, 2026

Copy link
Copy Markdown
Member

Super cool. I am inclined to merge this already (please fix just the small note that codex found).
However there is also no vite plugin live yet, so I will wait for that.

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Guide Cloudflare deployment

⛅️ wrangler 4.114.0
────────────────────
🌀 Building list of assets...
✨ Read 10 files from the assets directory /home/runner/work/rescript-lang.org/rescript-lang.org/apps/guide/out
🌀 Starting asset upload...
No updated asset files to upload. Proceeding with deployment...
Total Upload: 0.31 KiB / gzip: 0.22 KiB
Worker Startup Time: 0 ms
Uploaded guide (2.42 sec)
Worker Version ID: 7d06ea92-c3b7-450b-b529-8d65d4f75c50
Version Preview URL: https://7d06ea92-guide.rescript.workers.dev
Version Preview Alias URL: https://doc-sourcemap-guide.rescript.workers.dev

To deploy this version to production traffic use the command wrangler versions deploy

Changes to non-versioned settings (config properties 'logpush' or 'tail_consumers') take effect after your next deployment using the command wrangler versions deploy

Changes to triggers (routes, custom domains, cron schedules, etc) must be applied with the command wrangler triggers deploy

mununki commented Aug 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

Super cool. I am inclined to merge this already (please fix just the small note that codex found). However there is also no vite plugin live yet, so I will wait for that.

fixed a635ed6

I agree. I’ll work on @rescript/vite-plugin first, and I think it would be better to merge this PR once the plugin is available. I’ll follow up here when it’s published.

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Cloudflare deployment

Deployement ID: 709e325e-c189-44b8-bea0-19fc9708f99d
Deployment Environment: preview

⛅️ wrangler 4.114.0
────────────────────
✨ Compiled Worker successfully
Uploading... (8765/8772)
Uploading... (8767/8772)
Uploading... (8769/8772)
Uploading... (8772/8772)
✨ Success! Uploaded 7 files (8765 already uploaded) (1.69 sec)

✨ Uploading _redirects
✨ Uploading Functions bundle
🌎 Deploying...
✨ Deployment complete! Take a peek over at https://709e325e.rescript-lang.pages.dev
✨ Deployment alias URL: https://doc-sourcemap.rescript-lang.pages.dev

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL