Conversation
Add a blog section to the documentation site using the starlight-blog
plugin. Includes the first post ("Introducing Erode") and a Blog link
in the navigation bar.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdds the starlight-blog dependency and config to Astro, extends the docs content collection with a loader and integrated blog schema, updates the site nav and styles, and introduces two new content pages (a blog post and an announcement). Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~40 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@packages/web/src/components/nav-bar.astro`:
- Line 5: The new top-level <a class="nav-link">Blog</a> breaks the .nav-inner
spacing because .nav-inner uses justify-content: space-between; fix by grouping
primary links into a left-side wrapper so only the GitHub icon is pushed to the
edge: wrap the Docs and Blog links in a container (e.g., class="primary-links"
or "nav-left") inside nav-bar.astro, leave the GitHub/icon link as the separate
right-side child, and ensure CSS sets .primary-links (or .nav-left) to
display:flex with desired gap so .nav-inner can keep justify-content:
space-between without separating Docs and Blog.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 69be3673-6ec2-4cc9-937b-818ebee37456
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (5)
packages/web/astro.config.mjspackages/web/package.jsonpackages/web/src/components/nav-bar.astropackages/web/src/content.config.tspackages/web/src/content/docs/blog/introducing-erode.md
| <div class="container nav-inner"> | ||
| <a href="/" class="wordmark">erode</a> | ||
| <a href="/docs/getting-started/" class="nav-link">Docs</a> | ||
| <a href="/blog/" class="nav-link">Blog</a> |
There was a problem hiding this comment.
Adding another top-level nav item breaks the current spacing model.
.nav-inner still uses justify-content: space-between, so Line 5 turns Docs and Blog into separate flex stops across the full navbar. Keep the primary links grouped and push only the GitHub icon to the edge.
🧭 Minimal layout fix
.nav-inner {
display: flex;
- justify-content: space-between;
align-items: center;
+ gap: var(--space-md);
padding-block: var(--space-md);
}
.github-link {
+ margin-inline-start: auto;
color: var(--text-tertiary);
transition: color 0.2s;
display: flex;
}🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@packages/web/src/components/nav-bar.astro` at line 5, The new top-level <a
class="nav-link">Blog</a> breaks the .nav-inner spacing because .nav-inner uses
justify-content: space-between; fix by grouping primary links into a left-side
wrapper so only the GitHub icon is pushed to the edge: wrap the Docs and Blog
links in a container (e.g., class="primary-links" or "nav-left") inside
nav-bar.astro, leave the GitHub/icon link as the separate right-side child, and
ensure CSS sets .primary-links (or .nav-left) to display:flex with desired gap
so .nav-inner can keep justify-content: space-between without separating Docs
and Blog.
Rewrite intro to be more direct. Update title and excerpt to match.
Replace justify-content: space-between with gap on .nav-inner and add margin-inline-start: auto on .github-link so the wordmark and nav links group on the left while the GitHub icon stays right-aligned.
Summary
starlight-blogplugin with blog and announcements sectionsSummary by CodeRabbit