Skip to content

fix(svelte): avoid runtime snippet casts for slots#209

Merged
atinux merged 3 commits into
mainfrom
fix/svelte-named-slot-snippets
May 22, 2026
Merged

fix(svelte): avoid runtime snippet casts for slots#209
atinux merged 3 commits into
mainfrom
fix/svelte-named-slot-snippets

Conversation

@atinux
Copy link
Copy Markdown
Contributor

@atinux atinux commented May 21, 2026

Summary

  • replace hand-cast runtime snippets for named slots with compiler-created Svelte snippets
  • add an internal ComarkComponent helper to accumulate runtime-named snippet props
  • cover multiple named slots in SSR and browser tests

Waiting for @elliott-with-the-longest-name-on-github review

Tests

  • pnpm --filter @comark/svelte run check
  • pnpm --filter @comark/svelte run test
  • pnpm --filter @comark/svelte run build

@vercel
Copy link
Copy Markdown

vercel Bot commented May 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
comark Ready Ready Preview May 22, 2026 7:51am
comark-json-render Ready Ready Preview May 22, 2026 7:51am
comark-nextjs Ready Ready Preview May 22, 2026 7:51am
comark-nuxt Ready Ready Preview May 22, 2026 7:51am
comark-svelte Ready Ready Preview May 22, 2026 7:51am
comark-sveltekit Ready Ready Preview May 22, 2026 7:51am
comark-twoslash Ready Ready Preview May 22, 2026 7:51am
comark-vue Ready Ready Preview May 22, 2026 7:51am

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 21, 2026

Open in StackBlitz

comark

npm i https://pkg.pr.new/comark@209

@comark/ansi

npm i https://pkg.pr.new/@comark/ansi@209

@comark/html

npm i https://pkg.pr.new/@comark/html@209

@comark/nuxt

npm i https://pkg.pr.new/@comark/nuxt@209

@comark/react

npm i https://pkg.pr.new/@comark/react@209

@comark/svelte

npm i https://pkg.pr.new/@comark/svelte@209

@comark/vue

npm i https://pkg.pr.new/@comark/vue@209

commit: 1ca52d5

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