Skip to content

fix(core): ReslideServerEmbed の use client 境界を正しく分離 - #80

Merged
nisshi-dev merged 2 commits into
mainfrom
fix/server-embed-use-client
Apr 5, 2026
Merged

fix(core): ReslideServerEmbed の use client 境界を正しく分離#80
nisshi-dev merged 2 commits into
mainfrom
fix/server-embed-use-client

Conversation

@nisshi-dev

Copy link
Copy Markdown
Owner

Summary

  • tsdown が ReslideEmbedClientserver.mjs にインライン化し、"use client" ディレクティブが消失していたバグを修正
  • ReslideEmbedClient.tsx を独立した tsdown entry に追加し、banner"use client" を付与
  • embed.mjs にも "use client" banner を付与(軽量 Client Component エントリポイント)
  • ビルド出力の RSC 境界を検証するテスト(5件)を追加

ビルド出力の変更

Before:

dist/server.mjs ← ReslideEmbedClient がインライン化、"use client" 消失
  └── import ReslideEmbed from "./ReslideEmbed-xxx.mjs"  ← createContext 💥

After:

dist/server.mjs ← "use client" なし(Server Component)
  └── import { ReslideEmbedClient } from "./ReslideEmbedClient.mjs"

dist/ReslideEmbedClient.mjs ← "use client" が先頭(Client Component 境界)
  └── import ReslideEmbed from "./ReslideEmbed-xxx.mjs"

dist/embed.mjs ← "use client" が先頭(Client Component 境界)

Test plan

  • vp check — lint, format, 型チェック通過
  • vp test — 全99件パス(新規5件含む)
  • ポートフォリオサイトで ReslideServerEmbed が createContext エラーなく動作することを確認

🤖 Generated with Claude Code

nisshi-dev and others added 2 commits April 5, 2026 17:04
tsdown が ReslideEmbedClient を server.mjs にインライン化し "use client"
ディレクティブが消失していた。ReslideEmbedClient を独立 entry にし、
banner で "use client" を付与することで、Next.js bundler が正しく
Server/Client 境界を認識するようにした。

- ReslideEmbedClient.tsx を tsdown entry に追加
- banner 関数で ReslideEmbedClient.mjs と embed.mjs に "use client" 付与
- ビルド出力の RSC 境界を検証するテスト追加

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
ビルド出力テストで node:fs を使用するため明示的に追加。

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Apr 5, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
reslide-website Building Building Preview Apr 5, 2026 8:18am

@nisshi-dev
nisshi-dev merged commit f04b13e into main Apr 5, 2026
3 checks passed
@nisshi-dev
nisshi-dev deleted the fix/server-embed-use-client branch April 5, 2026 08:20
@github-actions github-actions Bot mentioned this pull request Apr 5, 2026
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