feat: migrate from ESLint/Prettier to Biome + Ultracite - #6
Merged
Conversation
- Replace ESLint and Prettier with Biome 2.3.13 and Ultracite 7.1.1 - Add draft field to content schema for publish/unpublish functionality - Update all source files with new formatting and lint rules - Remove deprecated ESLint/Prettier config files - Add biome.json with Ultracite Astro preset
✅ Deploy Preview for soumyo-dey ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Preview Deployment Ready!Preview URL: https://d4692b1c.soumyo-com.pages.dev This preview will be automatically updated when you push new commits. |
- Fix draft content exposure in llms.txt (P1) - Add draft filtering to getBlogPosts() in generate-llms-txt.js - Add getProjects() function with draft filtering for agent parity - Fix Biome lint scanning generated directories (P1) - Add files.includes with force-ignore patterns for .astro and dist - Update CLAUDE.md documentation (P2) - Replace ESLint/Prettier references with Biome - Extract content utilities (P3) - Create src/utils/content.ts with isPublished and sortByDateDesc - Update all 6 pages to use shared utilities - Split Layout.astro into components (P3) - Extract Navigation.astro (150 lines) - Extract Footer.astro (95 lines) - Reduce Layout.astro from 623 to 402 lines Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add justify-center and text-center classes to the hero eyebrow flex container so text remains centered when it wraps to two lines on narrow screens. Verified: text center offset reduced from 36px to 0px at 375px viewport. Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Code Review Fixes
After comprehensive multi-agent code review, the following improvements were implemented:
🔴 P1 - Critical (Fixed)
generate-llms-txt.jsso unpublished posts are not visible to AI agentsfiles.includeswith force-ignore patterns for.astro/anddist/directories🟡 P2 - Important (Fixed)
getProjects()function with draft filtering for agent-native parity🔵 P3 - Nice-to-Have (Fixed)
isPublishedandsortByDateDescutilities tosrc/utils/content.ts, replacing 7 duplicate lambdas across 6 pagesNavigation.astro(150 lines) - Nav bar, theme toggle, mobile menuFooter.astro(95 lines) - Footer with nav links and social iconsLayout.astroreduced to 402 linesMobile Responsiveness Fix
justify-centerandtext-centerclasses to flex containerFiles Changed
biome.json,CLAUDE.mdscripts/generate-llms-txt.js,public/llms.txtNavigation.astro(new),Footer.astro(new)src/utils/content.ts(new)src/layouts/Layout.astroblog.astro,projects.astro,index.astro,tags/[tag].astro,blog/[...slug].astro,projects/[...slug].astroReview Agents Used
Test Plan
bun run buildpassesbun run lintruns without errors on source files🤖 Generated with Claude Code