Skip to content

fix: resolve missing interactive components from islands migration - #245

Merged
avivkeller merged 1 commit into
webpack:mainfrom
moshams272:fix/islands-migration
Aug 15, 2026
Merged

fix: resolve missing interactive components from islands migration#245
avivkeller merged 1 commit into
webpack:mainfrom
moshams272:fix/islands-migration

Conversation

@moshams272

@moshams272 moshams272 commented Aug 15, 2026

Copy link
Copy Markdown
Member

Summary

This PR fixes issues that happened during the recent migration to the Islands architecture, where interactive components failed to load because their client-side JavaScript wasn't loaded.

Related Issues:
Fixes: #244 , #242

Note: fix typo in my GitHub handle ;)

Summary by CodeRabbit

  • Performance

    • Improved loading efficiency by enabling idle loading for blog layouts, author bylines, and metadata bars.
  • Documentation

    • Corrected the author attribution on the guides page.
    • Added blog layout and metadata components to the documentation toolkit for improved rendering support.

@vercel

vercel Bot commented Aug 15, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
webpack-doc-kit Ready Ready Preview Aug 15, 2026 10:48am

Request Review

@coderabbitai

coderabbitai Bot commented Aug 15, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 9b3b201f-1ecd-437c-b641-7d596d707c4f

📥 Commits

Reviewing files that changed from the base of the PR and between c252791 and 37af35d.

📒 Files selected for processing (5)
  • components/Blog/Byline/index.jsx
  • components/MetaBar/index.jsx
  • layouts/Blog/index.jsx
  • pages/guides/index.md
  • scripts/html/doc-kit.config.mjs

Walkthrough

The PR wraps Byline, MetaBar, and BlogLayout with withIsland using idle loading. It adds Doc Kit theme imports and HTML component registrations for these components. It also corrects the guides page author identifier from moshams272s to moshams272.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning The guides page author correction is unrelated to the Islands migration and issue #244. Remove the unrelated author metadata correction or move it to a separate pull request.
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The PR hydrates Byline and related components as islands, addressing the missing author initial described in issue #244.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: restoring missing interactive components after the Islands migration.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@avivkeller
avivkeller merged commit df0d506 into webpack:main Aug 15, 2026
14 checks passed
@moshams272
moshams272 deleted the fix/islands-migration branch August 15, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blog documentation Improvements or additions to documentation theme tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(authors list): is broken

3 participants