Sanity

[KomrakovaAnna]

• Оптимізовано Recommended Posts: GROQ лімітує 3 пости, без fetch‑all + shuffle. PostDetails.tsx
• Додано click‑outside‑to‑close для search dropdown. BlogHeaderSearch.tsx
• Додано description у metadata для /blog/[slug]. page.tsx
• Додано Schema.org JSON‑LD (Article + BreadcrumbList). PostDetails.tsx
• Додано для дат у блозі.
  BlogCard.tsx, BlogFilters.tsx, PostDetails.tsx, page.tsx

Summary by CodeRabbit

• New Features

  • Added SEO schema markup (JSON-LD) for blog posts with breadcrumbs and article metadata.
  • Enhanced search dropdown with click-outside-to-close behavior.

• Improvements

  • Updated date rendering across blog with semantic HTML time elements.
  • Extended blog post metadata to include descriptions.

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

[netlify]

✅ Deploy Preview for develop-devlovers ready!

Name	Link
🔨 Latest commit	69a8190
🔍 Latest deploy log	https://app.netlify.com/projects/develop-devlovers/deploys/69760deff9cacd0008b96bb8
😎 Deploy Preview	https://deploy-preview-202--develop-devlovers.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR enhances blog components with SEO improvements and semantic HTML. PostDetails.tsx adds JSON-LD structured data for breadcrumbs and blog posts while introducing <time> elements across multiple blog files. AuthorProfile state management is refactored in BlogFilters, and BlogHeaderSearch improves UX with click-outside dropdown closure.

Changes

Cohort / File(s)	Summary
SEO & Metadata frontend/app/[locale]/blog/[slug]/PostDetails.tsx, frontend/app/[locale]/blog/[slug]/page.tsx	Added JSON-LD structured data (BreadcrumbList and BlogPosting schemas) to PostDetails; extended generateMetadata query to fetch and include truncated description field in metadata object.
Semantic Date Rendering frontend/app/[locale]/blog/category/[category]/page.tsx, frontend/components/blog/BlogCard.tsx, frontend/components/blog/BlogFilters.tsx	Replaced plain date spans with semantic <time> elements using formatBlogDate and publishedAt dateTime attributes across featured posts, blog cards, and filter sections.
Author Profile State Refactoring frontend/components/blog/BlogFilters.tsx	Changed authorProfile state shape from Author | null to { name: string; data: Author } | null; introduced selectedAuthorData memoization; adjusted effect dependencies on locale and author name.
Search UX Enhancement frontend/components/blog/BlogHeaderSearch.tsx	Introduced click-outside-to-close behavior via containerRef; improved loading gate to fetch only when open with no items and loading active; added startLoading helper.
Minor Cleanup frontend/components/blog/BlogCategoryLinks.tsx	Removed single comment line with no functional changes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• Sanity #145 — Overlaps with PostDetails.tsx modifications for metadata, date rendering, and recommended-posts section.
• Sanity #196 — Shares direct code-level changes across blog files (PostDetails.tsx, BlogCard.tsx, BlogFilters.tsx) with semantic <time> element rendering and date formatting updates.
• Sanity #119 — Modifies the same PostDetails.tsx and page.tsx files with related locale-aware data fetching and component signature changes.

Suggested labels

enhancement

Suggested reviewers

• AM1007
• ViktorSvertoka

Poem

🐰 Blog posts bloom with schemas bright,
JSON-LD bringing SEO light,
Time elements nest so clean and true,
Semantic HTML through and through!
Click outside, the dropdowns close—
A cleaner web from whisker to nose!

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 16.67% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The title "Sanity" is vague and generic, providing no meaningful information about the changeset's primary purpose or content.	Replace with a descriptive title that summarizes the main changes, such as 'Add SEO metadata, semantic time elements, and search improvements to blog' or similar.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

frontend/components/blog/BlogHeaderSearch.tsx (1)

69-85: Reset isLoading when the dropdown closes to avoid a stuck state.
If the menu is closed before the fetch resolves, active becomes false and the finally block never clears isLoading. That prevents future loads because startLoading won’t run while isLoading stays true.

🧯 Suggested fix

  useEffect(() => {
-    if (!open || items.length || !isLoading) return;
+    if (!open) {
+      if (isLoading) setIsLoading(false);
+      return;
+    }
+    if (items.length || !isLoading) return;
     let active = true;
     fetch(SEARCH_ENDPOINT, { cache: 'no-store' })
       .then(response => (response.ok ? response.json() : []))
       .then((result: PostSearchItem[]) => {
         if (!active) return;
         setItems(Array.isArray(result) ? result : []);
       })
       .catch(() => {
         if (!active) return;
         setItems([]);
       })
       .finally(() => {
         if (!active) return;
         setIsLoading(false);
       });
     return () => {
       active = false;
     };
   }, [open, items.length, isLoading]);

🧹 Nitpick comments (2)

frontend/app/[locale]/blog/[slug]/PostDetails.tsx (1)

215-219: Consider handling missing NEXT_PUBLIC_SITE_URL more explicitly.

If NEXT_PUBLIC_SITE_URL is not set, baseUrl will be undefined, and no JSON-LD will be generated. This is safe but silent. Consider logging a warning during development or documenting this requirement.

frontend/components/blog/BlogHeaderSearch.tsx (1)

175-181: Optional cleanup: remove the unreachable if (!open) branch in onChange.
The input only renders when open is true, so this branch never executes. Consider dropping it or moving the open/startLoading logic to the open trigger only.