Skip to content

Squint source maps - #54

Merged
rschmukler merged 1 commit into
dynamic-alpha:mainfrom
borkdude:squint-source-maps
Oct 14, 2026
Merged

rschmukler merged 1 commit into
dynamic-alpha:mainfrom
borkdude:squint-source-maps

Conversation

@borkdude

Copy link
Copy Markdown
Contributor

@rschmukler This PR adds source maps for defc components. They are off by default because including server side code might not be what you want in a production app. Pass :source-map? true to create-handler to enable them. The example app does.

To try it, throw an error in a defc body in example/app.clj and look at the console: the stack trace points at the .clj line. With DevTools open, press Cmd-P and open example/app.clj. You can also just set breakpoints in there.

@borkdude

Copy link
Copy Markdown
Contributor Author

@rschmukler Once you're done reviewing, let me squash this PR to just one commit to not pollute history too much.

@rschmukler rschmukler 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.

Looks great! Ready for squash and merge. Thanks!

@borkdude

Copy link
Copy Markdown
Contributor Author

Squashed!

@rschmukler
rschmukler merged commit 91a33ff into dynamic-alpha:main Oct 14, 2026
1 check passed
@borkdude
borkdude deleted the squint-source-maps branch October 14, 2026 20:55
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.

2 participants