Skip to content

Conversation

@LGLabGreg
Copy link
Owner

@LGLabGreg LGLabGreg commented Dec 21, 2025

Summary by CodeRabbit

  • New Features

    • Added AI tools integration documentation (llms.txt standard support)
    • Redesigned homepage with hero section and feature cards for improved visual presentation
    • Added new navigation entry for AI documentation
  • Documentation

    • Introduced comprehensive technical reference documentation for QR code library
  • Chores

    • Updated runtime dependencies (lucide-react, react-hook-form)
    • Added code formatter tooling

✏️ Tip: You can customize this high-level summary in your review settings.

dependabot bot and others added 7 commits December 20, 2025 12:15
Bumps [lucide-react](https://github.com/lucide-icons/lucide/tree/HEAD/packages/lucide-react) from 0.561.0 to 0.562.0.
- [Release notes](https://github.com/lucide-icons/lucide/releases)
- [Commits](https://github.com/lucide-icons/lucide/commits/0.562.0/packages/lucide-react)

---
updated-dependencies:
- dependency-name: lucide-react
  dependency-version: 0.562.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
…ucide-react-0.562.0

chore(deps): bump lucide-react from 0.561.0 to 0.562.0
Bumps the react group with 1 update in the / directory: [react-hook-form](https://github.com/react-hook-form/react-hook-form).


Updates `react-hook-form` from 7.68.0 to 7.69.0
- [Release notes](https://github.com/react-hook-form/react-hook-form/releases)
- [Changelog](https://github.com/react-hook-form/react-hook-form/blob/master/CHANGELOG.md)
- [Commits](react-hook-form/react-hook-form@v7.68.0...v7.69.0)

---
updated-dependencies:
- dependency-name: react-hook-form
  dependency-version: 7.69.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: react
...

Signed-off-by: dependabot[bot] <support@github.com>
…eact-4cbd9dffb5

chore(deps): bump react-hook-form from 7.68.0 to 7.69.0 in the react group across 1 directory
@vercel
Copy link

vercel bot commented Dec 21, 2025

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

Project Deployment Review Updated (UTC)
reactqrcode-com Ready Ready Preview, Comment Dec 21, 2025 10:35am

@coderabbitai
Copy link

coderabbitai bot commented Dec 21, 2025

Walkthrough

This pull request adds llms.txt documentation support to the docs site, creates a new Hero component, refactors the homepage with a feature grid layout, updates dependencies (lucide-react and react-hook-form), adds prettier as a dev dependency, and introduces new navigation entries for the llms.txt page.

Changes

Cohort / File(s) Summary
Dependency updates
apps/docs/package.json, package.json
Updated lucide-react to ^0.562.0 and react-hook-form to ^7.69.0; added prettier ^3.7.4 as a devDependency
Documentation for llms.txt standard
apps/docs/public/llms.txt, apps/docs/public/llms-full.txt
Added llms.txt documentation files with library overview, links to documentation pages, and technical reference for @lglab/react-qr-code component props and API
New llms.txt page
apps/docs/src/app/llms-txt/page.tsx
Created new page component documenting the llms.txt standard with links to available documentation files and Cursor integration guidance
UI component additions
apps/docs/src/components/hero.tsx
Added Hero component with title, lead text, and call-to-action buttons linking to installation and demo pages
Homepage refactor
apps/docs/src/app/page.tsx
Replaced static heading/typography layout with Hero component and card-based feature grid; added AI-focused section with link to /llms-txt
Component styling tweaks
apps/docs/src/components/ui/typography.tsx
Adjusted TypographyP bottom margin from mb-6 to mb-4 and updated TypographyTable alignment selector syntax
Navigation configuration
apps/docs/src/config/navigation.ts
Added 'llms.txt' entry to Getting Started section in mainNav

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Most changes are presentational components, documentation files, and configuration updates
  • Homepage refactor is straightforward layout/component swapping without complex logic
  • Dependency version bumps are minor patch/minor updates

Possibly related PRs

Poem

🐰 A new hero rises, cards in array bright,
llms.txt guides the curious through the night,
Dependencies bumped, docs complete,
The homepage now dances with features so neat! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
Title check ❓ Inconclusive The title 'Feat/llm' is extremely vague and does not clearly communicate the purpose of the changes, which include adding llms.txt support, updating dependencies, and redesigning the homepage. Revise the title to be more descriptive, such as 'Add llms.txt support and update documentation homepage' or 'Add LLM documentation support with homepage redesign'.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/llm

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🧹 Nitpick comments (2)
apps/docs/src/app/page.tsx (2)

15-39: Consider moving the features array outside the component.

The features array is recreated on every render. While not critical for a static homepage, moving it outside the component body would prevent unnecessary array recreations.

🔎 Proposed refactor
+const features = [
+  {
+    title: 'Highly Customizable',
+    description:
+      'Style finder patterns, modules, and colors exactly how you want with advanced configuration.',
+    icon: <Sparkles className='h-5 w-5 text-primary' />,
+  },
+  {
+    title: 'Performance Optimized',
+    description:
+      'Generates QR codes efficiently without sacrificing quality or bundle size.',
+    icon: <Zap className='h-5 w-5 text-primary' />,
+  },
+  {
+    title: 'SVG-Based Rendering',
+    description: 'Crisp and scalable output for web and print, powered by SVG.',
+    icon: <Layers className='h-5 w-5 text-primary' />,
+  },
+  {
+    title: 'Developer-Friendly',
+    description:
+      'Built with TypeScript, easy to use, and focused on a great developer experience.',
+    icon: <Code className='h-5 w-5 text-primary' />,
+  },
+]
+
 export default function Page() {
-  const features = [
-    {
-      title: 'Highly Customizable',
-      description:
-        'Style finder patterns, modules, and colors exactly how you want with advanced configuration.',
-      icon: <Sparkles className='h-5 w-5 text-primary' />,
-    },
-    {
-      title: 'Performance Optimized',
-      description:
-        'Generates QR codes efficiently without sacrificing quality or bundle size.',
-      icon: <Zap className='h-5 w-5 text-primary' />,
-    },
-    {
-      title: 'SVG-Based Rendering',
-      description: 'Crisp and scalable output for web and print, powered by SVG.',
-      icon: <Layers className='h-5 w-5 text-primary' />,
-    },
-    {
-      title: 'Developer-Friendly',
-      description:
-        'Built with TypeScript, easy to use, and focused on a great developer experience.',
-      icon: <Code className='h-5 w-5 text-primary' />,
-    },
-  ]
-
   return (

44-60: Add aria-labels to icon containers for better accessibility.

The decorative icons should either have aria-hidden="true" (if purely decorative) or aria-label attributes (if they convey meaning). This helps screen reader users understand the feature's focus.

🔎 Proposed refactor
 {features.map((feature) => (
   <Card key={feature.title}>
     <CardHeader className='flex-row items-start gap-4 space-y-0'>
-      <div className='mb-2 inline-flex h-9 w-9 items-center justify-center rounded-lg bg-background shadow-sm'>
+      <div 
+        className='mb-2 inline-flex h-9 w-9 items-center justify-center rounded-lg bg-background shadow-sm'
+        aria-hidden='true'
+      >
         {feature.icon}
       </div>
       <div>
         <CardTitle className='text-lg'>{feature.title}</CardTitle>
         <CardDescription className='text-sm leading-relaxed'>
           {feature.description}
         </CardDescription>
       </div>
     </CardHeader>
   </Card>
 ))}

Similarly, apply to the AI card icon container:

-<div className='mt-1 shrink-0 rounded-lg bg-background p-2 shadow-sm'>
+<div 
+  className='mt-1 shrink-0 rounded-lg bg-background p-2 shadow-sm'
+  aria-hidden='true'
+>
   <Sparkles className='h-6 w-6 text-primary' />
 </div>
📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between eb02e5b and 2f0e6cc.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (9)
  • apps/docs/package.json (1 hunks)
  • apps/docs/public/llms-full.txt (1 hunks)
  • apps/docs/public/llms.txt (1 hunks)
  • apps/docs/src/app/llms-txt/page.tsx (1 hunks)
  • apps/docs/src/app/page.tsx (1 hunks)
  • apps/docs/src/components/hero.tsx (1 hunks)
  • apps/docs/src/components/ui/typography.tsx (3 hunks)
  • apps/docs/src/config/navigation.ts (1 hunks)
  • package.json (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-02-15T10:34:32.155Z
Learnt from: LGLabGreg
Repo: LGLabGreg/react-qr-code PR: 68
File: packages/react-qr-code/src/components/data-modules.tsx:34-34
Timestamp: 2025-02-15T10:34:32.155Z
Learning: DataModulesProps extends GeneratePathFnProps in the react-qr-code package, inheriting properties like gradientId.

Applied to files:

  • apps/docs/public/llms-full.txt
  • apps/docs/src/app/page.tsx
🧬 Code graph analysis (2)
apps/docs/src/components/hero.tsx (1)
apps/docs/src/components/ui/typography.tsx (1)
  • TypographyLead (94-96)
apps/docs/src/app/page.tsx (2)
apps/docs/src/components/hero.tsx (1)
  • Hero (8-35)
apps/docs/src/components/ui/typography.tsx (1)
  • TypographyBold (110-112)
🔇 Additional comments (8)
apps/docs/src/config/navigation.ts (1)

24-27: LGTM!

The navigation entry for llms.txt is correctly structured and appropriately placed under the Getting Started section. The URL follows the existing naming conventions.

apps/docs/src/components/hero.tsx (1)

8-35: LGTM!

The Hero component is well-structured with proper use of Next.js Link via the Button's asChild prop. The component provides clear calls-to-action and follows the established UI patterns.

apps/docs/public/llms-full.txt (1)

1-138: LGTM!

The technical reference documentation is comprehensive and well-structured. It provides detailed information about component props, settings interfaces, the imperative API, and practical usage examples in a format that's ideal for LLM consumption.

apps/docs/public/llms.txt (1)

1-26: LGTM!

The llms.txt file provides a clear overview and well-organized documentation hub with links to all relevant pages and external resources. The structure follows the llms.txt standard appropriately.

apps/docs/src/components/ui/typography.tsx (1)

60-60: Reduced paragraph spacing.

The bottom margin for paragraphs has been reduced from mb-6 to mb-4. This is a minor visual adjustment that tightens the vertical rhythm of text content.

apps/docs/src/app/llms-txt/page.tsx (1)

10-111: LGTM!

The llms.txt documentation page is well-structured and provides clear information about the llms.txt standard, available routes, and usage instructions. The component correctly uses typography components and follows security best practices for external links (using rel='noreferrer' with target='_blank').

package.json (1)

53-53: No changes needed—prettier version 3.7.4 is valid and secure.

No direct vulnerabilities have been found for the prettier package in Snyk's vulnerability database, and the package is currently considered safe to use.

apps/docs/src/app/page.tsx (1)

1-12: Nice refactoring! The Hero + Card-based layout improves the homepage structure.

The migration from static typography components to a Hero component and feature card grid creates a more modern, maintainable layout. The integration with the new llms.txt documentation is well-executed.

Also applies to: 41-87

@LGLabGreg LGLabGreg merged commit 539a703 into main Dec 21, 2025
5 checks passed
@LGLabGreg LGLabGreg deleted the feat/llm branch December 21, 2025 10:42
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.

2 participants