Skip to content

fix: don't copy theme layout.html to public output#5

Merged
lucasnevespereira merged 1 commit intomainfrom
fix/dont-copy-layout
Apr 26, 2026
Merged

fix: don't copy theme layout.html to public output#5
lucasnevespereira merged 1 commit intomainfrom
fix/dont-copy-layout

Conversation

@lucasnevespereira
Copy link
Copy Markdown
Member

Summary

copyTheme was writing layout.html to public/themes/<name>/layout.html alongside the actual assets. The layout is a server-side template — it's never referenced by the browser and shouldn't ship.

Now only styles.css and script.js are copied. Same fix applied to both the embedded-theme path and the local-theme path (skips layout.html explicitly).

Test plan

  • go vet ./... clean
  • Build examples/minimal — output public/themes/minimal/ contains only script.js and styles.css
  • No regression: pages still render correctly (layout still resolved server-side)

@lucasnevespereira lucasnevespereira merged commit 8988170 into main Apr 26, 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