Skip to content

Create 'serve-html' skill for serving HTML files - #59

Merged
pmillspaugh merged 3 commits into
mainfrom
serve-html
Sep 24, 2026
Merged

pmillspaugh merged 3 commits into
mainfrom
serve-html

Conversation

@pmillspaugh

Copy link
Copy Markdown
Member

As mentioned here, instead of deploying a claude artifact as an html file to val town via mcp, claude created a new .tsx file. This skill should help encourage the pattern of uploading the html file directly and serving it from a simple server (even simpler now with deno text imports)

@changeset-bot

changeset-bot Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: d80142a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@valtown/skills Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pmillspaugh
pmillspaugh requested a review from potluck September 17, 2026 22:16
jxnblk
jxnblk previously approved these changes Sep 24, 2026

@jxnblk jxnblk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Generally, I like it. Not sure about the dynamic import piece I mention, but feel free to ship as-is

Comment thread plugin/skills/serve-html Outdated
1. A static Deno text import
2. `serveFile`, which fetches the HTML file from esm.town

**Default for a fixed, known file: a static import (`with { type: "text" }`).**

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm pretty sure Deno supports with { type: "text" } with dynamic imports, which also seems to work on Val Town. Not sure if that's limited in what it can do, but this part sounds slightly dishonest

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes the dynamic text imports work (example)

@pmillspaugh
pmillspaugh merged commit 555ff74 into main Sep 24, 2026
1 check 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.

2 participants