@c-rex/components 0.1.31 → 0.1.32

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@c-rex/components",
-  "version": "0.1.31",
+  "version": "0.1.32",
   "files": [
     "src"
   ],

package/src/article/article-action-bar.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: article-action-bar
+
+- Component: packages/components/src/article/article-action-bar.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=yes, window/document=no, effects=yes, fetch/call=no
+
+## Possible re-render causes
+- Store subscriptions can re-render this component whenever the selected slice changes.
+- Effects that update state can add extra renders on mount and dependency changes.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/article/article-content.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: article-content
+
+- Component: packages/components/src/article/article-content.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=yes
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- Async flows can hit race conditions during fast param/route changes.
+
+## Recommended improvements
+- Consider cancellation guards (isMounted or AbortController) and standardized error handling.
+

package/src/article/article-content.tsx

@@ -43,8 +43,8 @@ export const ArticleContent: FC<Props> = async ({ renditions }) => {
 
     return (
         <>
-            {metaTags.map((tag, index) => (
-                <meta key={index} name={tag.name} content={tag.content} />
+            {metaTags.map((tag) => (
+                <meta key={`${tag.name}-${tag.content}`} name={tag.name} content={tag.content} />
             ))}
             <div className="pr-4 relative">
                 <RenderArticle htmlContent={articleHtml} contentLang="" />
@@ -52,4 +52,4 @@ export const ArticleContent: FC<Props> = async ({ renditions }) => {
             <ArticleActionBar />
         </>
     )
-}
+}

package/src/autocomplete.analysis.md

@@ -0,0 +1,17 @@
+# Analysis: autocomplete
+
+- Component: packages/components/src/autocomplete.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=yes, effects=yes, fetch/call=no
+
+## Possible re-render causes
+- Effects that update state can add extra renders on mount and dependency changes.
+
+## Possible bugs/risks
+- Access to window/document needs care to avoid hydration mismatch and non-browser runtime issues.
+- The component renders list-heavy UI; key stability and memoization strongly affect perceived performance.
+
+## Recommended improvements
+- Keep browser-only access inside event handlers or useEffect, not in initial render paths.
+- Review key strategy and extract memoized subcomponents for large-list rendering.
+

package/src/autocomplete.tsx

@@ -135,9 +135,9 @@ export const AutoComplete = ({
                         <>
                             {suggestions.length > 0 ? (
                                 <>
-                                    {suggestions.map((option, index) => (
+                                    {suggestions.map((option) => (
                                         <li
-                                            key={index}
+                                            key={option}
                                             className="px-4 py-2 hover:bg-accent cursor-pointer text-sm"
                                             onClick={() => handleSelect(`"${option}"`)}
                                         >

package/src/breadcrumb.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: breadcrumb
+
+- Component: packages/components/src/breadcrumb.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- The component renders list-heavy UI; key stability and memoization strongly affect perceived performance.
+
+## Recommended improvements
+- Review key strategy and extract memoized subcomponents for large-list rendering.
+

package/src/breadcrumb.tsx

@@ -53,9 +53,9 @@ export const Breadcrumb: FC<BreadcrumbProps> = ({ tree, lang }) => {
                             </DrawerHeader>
 
                             <div className="grid gap-1 px-4">
-                                {items.slice(0, -1).map((item, index) => (
+                                {items.slice(0, -1).map((item) => (
                                     <Link
-                                        key={index}
+                                        key={item.linkId}
                                         href={`/topics/${item.linkId}/content`}
                                         className="py-1 text-sm"
                                     >
@@ -103,7 +103,7 @@ export const Breadcrumb: FC<BreadcrumbProps> = ({ tree, lang }) => {
 
                             return (
                                 <Fragment key={`${item.label}-fragment`}>
-                                    <BreadcrumbItem key={`${item.label}-item`} className={index.toString()}>
+                                    <BreadcrumbItem key={`${item.label}-item`}>
                                         {renderLink(!isLast, item)}
                                     </BreadcrumbItem>
                                     {!isLast && (
@@ -121,4 +121,4 @@ export const Breadcrumb: FC<BreadcrumbProps> = ({ tree, lang }) => {
             </BreadcrumbList>
         </BreadcrumbComponent>
     );
-};
+};

package/src/carousel/carousel.analysis.md

@@ -0,0 +1,17 @@
+# Analysis: carousel
+
+- Component: packages/components/src/carousel/carousel.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=yes, effects=yes, fetch/call=no
+
+## Possible re-render causes
+- Effects that update state can add extra renders on mount and dependency changes.
+
+## Possible bugs/risks
+- Access to window/document needs care to avoid hydration mismatch and non-browser runtime issues.
+- The component renders list-heavy UI; key stability and memoization strongly affect perceived performance.
+
+## Recommended improvements
+- Keep browser-only access inside event handlers or useEffect, not in initial render paths.
+- Review key strategy and extract memoized subcomponents for large-list rendering.
+

package/src/carousel/carousel.tsx

@@ -283,7 +283,7 @@ const CarouselIndicators: FC<{ className?: string }> = ({ className }) => {
         return (
             <ol className={cn("flex gap-2 z-20 list-none p-0 m-0", className)}>
                 {Array.from({ length: totalOfIndicators }).map((_, index) => (
-                    <li key={index}>
+                    <li key={`indicator-${index}`}>
                         <Button
                             className={`w-3 h-3 rounded-full border-0 cursor-pointer transition-colors ${index === current ? "bg-gray-800" : "bg-gray-300"}`}
                             size="xs"
@@ -349,4 +349,4 @@ const DefaultRenderCarouselItem: FC<{ item: CommonItemsModel; showImages: boolea
             </Card>
         </div>
     );
-};
+};

package/src/check-article-lang.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: check-article-lang
+
+- Component: packages/components/src/check-article-lang.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=yes, effects=yes, fetch/call=no
+
+## Possible re-render causes
+- Effects that update state can add extra renders on mount and dependency changes.
+
+## Possible bugs/risks
+- Access to window/document needs care to avoid hydration mismatch and non-browser runtime issues.
+
+## Recommended improvements
+- Keep browser-only access inside event handlers or useEffect, not in initial render paths.
+

package/src/directoryNodes/tree-of-content.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: tree-of-content
+
+- Component: packages/components/src/directoryNodes/tree-of-content.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/documents/result-list.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: result-list
+
+- Component: packages/components/src/documents/result-list.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/documents/result-list.tsx

@@ -56,7 +56,7 @@ export const DocumentsResultList: FC<Props> = ({
 
                 return (
                     <div
-                        key={index}
+                        key={item.shortId}
                         className={cn(
                             "min-h-12 flex flex-wrap items-center border px-4 py-2 gap-2 rounded",
                             index == items.length - 1 ? "" : "mb-4",
@@ -124,4 +124,4 @@ export const DocumentsResultList: FC<Props> = ({
             })}
         </div>
     );
-};
+};

package/src/favorites/bookmark-button.analysis.md

@@ -0,0 +1,17 @@
+# Analysis: bookmark-button
+
+- Component: packages/components/src/favorites/bookmark-button.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=yes, effects=yes, fetch/call=no
+
+## Possible re-render causes
+- Effects that update state can add extra renders on mount and dependency changes.
+
+## Possible bugs/risks
+- Access to window/document needs care to avoid hydration mismatch and non-browser runtime issues.
+- markersList is loaded only on mount/shortId change; later store updates may not refresh the badge.
+
+## Recommended improvements
+- Keep browser-only access inside event handlers or useEffect, not in initial render paths.
+- Read documents[shortId]?.topics directly through a Zustand selector to keep UI reactive.
+

package/src/favorites/favorite-button.analysis.md

@@ -0,0 +1,18 @@
+# Analysis: favorite-button
+
+- Component: packages/components/src/favorites/favorite-button.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=yes, window/document=no, effects=yes, fetch/call=yes
+
+## Possible re-render causes
+- Store subscriptions can re-render this component whenever the selected slice changes.
+- Effects that update state can add extra renders on mount and dependency changes.
+
+## Possible bugs/risks
+- Async flows can hit race conditions during fast param/route changes.
+- useEffect with empty deps calls getTopicDocumentData(id) and does not react if id/type changes.
+
+## Recommended improvements
+- Consider cancellation guards (isMounted or AbortController) and standardized error handling.
+- Include id and type in effect dependencies and handle fetch cancellation on unmount.
+

package/src/generated/create-client-request.tsx

@@ -7,7 +7,7 @@
 
 "use client";
 
-import { useState, useEffect, useCallback } from 'react';
+import { useState, useEffect, useCallback, useMemo } from 'react';
 import { replacePathParams, call } from "@c-rex/utils"
 
 type SerializedError = {
@@ -40,11 +40,21 @@ export function createClientDataRequest<T, PathParams, QueryParams>(
             isLoading: true,
         });
 
+        const pathParamsKey = useMemo(
+            () => JSON.stringify(pathParams ?? {}),
+            [pathParams]
+        );
+        const queryParamsKey = useMemo(
+            () => JSON.stringify(queryParams ?? {}),
+            [queryParams]
+        );
+
         const fetchData = useCallback(async () => {
-            setState({
-                ...state,
+            setState((prev) => ({
+                ...prev,
                 error: null,
-            });
+                isLoading: true,
+            }));
 
             try {
                 const url = replacePathParams(endpointTemplate, pathParams as Record<string, string>);
@@ -66,8 +76,7 @@ export function createClientDataRequest<T, PathParams, QueryParams>(
                     isLoading: false,
                 });
             }
-            // eslint-disable-next-line react-hooks/exhaustive-deps
-        }, [endpointTemplate, pathParams]);
+        }, [endpointTemplate, pathParamsKey, queryParamsKey]);
 
 
         // Initial fetch

package/src/icons/file-icon.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: file-icon
+
+- Component: packages/components/src/icons/file-icon.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/icons/flag-icon.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: flag-icon
+
+- Component: packages/components/src/icons/flag-icon.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/icons/loading.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: loading
+
+- Component: packages/components/src/icons/loading.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/info/info-table.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: info-table
+
+- Component: packages/components/src/info/info-table.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- The component renders list-heavy UI; key stability and memoization strongly affect perceived performance.
+
+## Recommended improvements
+- Review key strategy and extract memoized subcomponents for large-list rendering.
+

package/src/info/info-table.tsx

@@ -56,8 +56,8 @@ export const InfoTable: FC<Props> = async ({
             <CardContent className="space-y-3 !p-0">
                 <Table>
                     <TableBody>
-                        {processDataToLabelValuePairs(data).map((item, index) => (
-                            <TableRow key={index} className="min-h-12">
+                        {processDataToLabelValuePairs(data).map((item) => (
+                            <TableRow key={item.label} className="min-h-12">
                                 <TableCell className="font-medium w-28 pl-4">
                                     <h4 className="text-sm font-medium capitalize">{t(item.label)}</h4>
                                 </TableCell>
@@ -108,24 +108,24 @@ export const InfoTable: FC<Props> = async ({
                                 </TableCell>
                                 <TableCell className="text-xs text-muted-foreground flex items-center gap-2 min-h-12">
 
-                                    {Object.keys(files).map((item, index) => {
-                                        if (!files[item]) return null
+                                    {Object.keys(files).map((fileKey) => {
+                                        if (!files[fileKey]) return null
 
                                         return (
-                                            <DropdownMenu key={index}>
+                                            <DropdownMenu key={fileKey}>
                                                 <DropdownMenuTrigger className="mr-2" asChild >
                                                     <Button variant="outline" size="icon" >
-                                                        <FileIcon extension={item} />
+                                                        <FileIcon extension={fileKey} />
                                                     </Button>
                                                 </DropdownMenuTrigger>
                                                 <DropdownMenuContent>
                                                     <DropdownMenuItem>
-                                                        <a href={files[item].view} target="_blank" rel="noreferrer" className="flex items-center">
+                                                        <a href={files[fileKey].view} target="_blank" rel="noreferrer" className="flex items-center">
                                                             <Eye className="mr-2" /> Open
                                                         </a>
                                                     </DropdownMenuItem>
                                                     <DropdownMenuItem>
-                                                        <a href={files[item].download} target="_blank" rel="noreferrer" className="flex items-center">
+                                                        <a href={files[fileKey].download} target="_blank" rel="noreferrer" className="flex items-center">
                                                             <CloudDownload className="mr-2" /> Download
                                                         </a>
                                                     </DropdownMenuItem>

package/src/info/shared.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: shared
+
+- Component: packages/components/src/info/shared.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/navbar/language-switcher/content-language-switch.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: content-language-switch
+
+- Component: packages/components/src/navbar/language-switcher/content-language-switch.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=yes, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- Access to window/document needs care to avoid hydration mismatch and non-browser runtime issues.
+
+## Recommended improvements
+- Keep browser-only access inside event handlers or useEffect, not in initial render paths.
+

package/src/navbar/language-switcher/shared.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: shared
+
+- Component: packages/components/src/navbar/language-switcher/shared.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/navbar/language-switcher/ui-language-switch.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: ui-language-switch
+
+- Component: packages/components/src/navbar/language-switcher/ui-language-switch.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=yes, window/document=yes, effects=no, fetch/call=no
+
+## Possible re-render causes
+- Store subscriptions can re-render this component whenever the selected slice changes.
+
+## Possible bugs/risks
+- Access to window/document needs care to avoid hydration mismatch and non-browser runtime issues.
+
+## Recommended improvements
+- Keep browser-only access inside event handlers or useEffect, not in initial render paths.
+

package/src/navbar/navbar.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: navbar
+
+- Component: packages/components/src/navbar/navbar.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/navbar/settings.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: settings
+
+- Component: packages/components/src/navbar/settings.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/navbar/sign-in-out-btns.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: sign-in-out-btns
+
+- Component: packages/components/src/navbar/sign-in-out-btns.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/navbar/user-menu.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: user-menu
+
+- Component: packages/components/src/navbar/user-menu.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/page-wrapper.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: page-wrapper
+
+- Component: packages/components/src/page-wrapper.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/render-article.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: render-article
+
+- Component: packages/components/src/render-article.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=yes, window/document=no, effects=yes, fetch/call=no
+
+## Possible re-render causes
+- Store subscriptions can re-render this component whenever the selected slice changes.
+- Effects that update state can add extra renders on mount and dependency changes.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/renditions/file-download.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: file-download
+
+- Component: packages/components/src/renditions/file-download.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/renditions/html.analysis.md

@@ -0,0 +1,17 @@
+# Analysis: html
+
+- Component: packages/components/src/renditions/html.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=yes
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- Async flows can hit race conditions during fast param/route changes.
+- fetch(url) runs inside async server rendering path and can increase list/page latency.
+
+## Recommended improvements
+- Consider cancellation guards (isMounted or AbortController) and standardized error handling.
+- Consider caching/revalidation and lighter fallback strategy to reduce N+1 impact.
+

package/src/renditions/image/container.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: container
+
+- Component: packages/components/src/renditions/image/container.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- imageRestrictions prop exists but is not applied to Restrict composition.
+
+## Recommended improvements
+- Apply imageRestrictions to Restrict or remove the prop to avoid misleading API behavior.
+

package/src/renditions/image/rendition.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: rendition
+
+- Component: packages/components/src/renditions/image/rendition.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/restriction-menu/restriction-menu-container.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: restriction-menu-container
+
+- Component: packages/components/src/restriction-menu/restriction-menu-container.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/restriction-menu/restriction-menu-container.tsx

@@ -38,7 +38,7 @@ export const RestrictionMenuContainer: FC<Props> = ({
                     <div className="flex justify-between">
                         <Skeleton className="w-12 h-9 rounded-full" />
                         {Array(itemsToRender).fill(0).map((_, index) => (
-                            <Skeleton key={index} className="w-28 h-9 rounded-full" />
+                            <Skeleton key={`skeleton-${index}`} className="w-28 h-9 rounded-full" />
                         ))}
                         <Skeleton className="w-20 h-9 rounded-full" />
                     </div>
@@ -50,4 +50,4 @@ export const RestrictionMenuContainer: FC<Props> = ({
             }}
         </RequestComponent>
     );
-};
+};

package/src/restriction-menu/restriction-menu-item.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: restriction-menu-item
+
+- Component: packages/components/src/restriction-menu/restriction-menu-item.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/restriction-menu/restriction-menu.analysis.md

@@ -0,0 +1,17 @@
+# Analysis: restriction-menu
+
+- Component: packages/components/src/restriction-menu/restriction-menu.tsx
+- Type: Client component
+- Detected signals: useQueryStates=yes, stores=yes, window/document=no, effects=yes, fetch/call=no
+
+## Possible re-render causes
+- useQueryStates triggers re-renders whenever relevant query-string values change.
+- Store subscriptions can re-render this component whenever the selected slice changes.
+- Effects that update state can add extra renders on mount and dependency changes.
+
+## Possible bugs/risks
+- Breakpoint-driven sorting/partitioning can trigger chained updates (visibleCount, visibleItems, hiddenItems).
+
+## Recommended improvements
+- Derive visibleItems/hiddenItems via useMemo from sortedItems + visibleCount to reduce duplicated state.
+

package/src/restriction-menu/restriction-menu.tsx

@@ -1,15 +1,13 @@
 "use client";
 
-import { FC, use, useEffect, useLayoutEffect, useMemo, useRef, useState } from "react"
+import { FC, useEffect, useMemo, useState } from "react"
 import {
     NavigationMenu,
     NavigationMenuList,
     NavigationMenuItem,
-    NavigationMenuLink,
 } from "@c-rex/ui/navigation-menu";
-import Link from "next/link";
 import { Button } from "@c-rex/ui/button";
-import { ChevronDown, Menu } from "lucide-react";
+import { ChevronDown } from "lucide-react";
 import { DropdownHoverItem } from "@c-rex/ui/dropdown-hover-item";
 import { RestrictionDropdownItem, RestrictionNavigationItem } from "./restriction-menu-item";
 import { parseAsString, useQueryStates } from "nuqs";
@@ -58,7 +56,7 @@ export const RestrictionMenu: FC<Props> = ({
     const uiLang = useLanguageStore.getState().uiLang;
     const lang = uiLang?.split("-")[0] ?? "";
 
-    useMemo(() => {
+    useEffect(() => {
         const map = new Map<string, string>();
         items.forEach((item) => {
             const label = getLabelByLang(item.labels, lang);
@@ -154,4 +152,4 @@ export const RestrictionMenu: FC<Props> = ({
             </NavigationMenuList>
         </NavigationMenu>
     );
-};
+};

package/src/results/cards.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: cards
+
+- Component: packages/components/src/results/cards.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/results/dialog-filter.analysis.md

@@ -0,0 +1,17 @@
+# Analysis: dialog-filter
+
+- Component: packages/components/src/results/dialog-filter.tsx
+- Type: Client component
+- Detected signals: useQueryStates=yes, stores=yes, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- useQueryStates triggers re-renders whenever relevant query-string values change.
+- Store subscriptions can re-render this component whenever the selected slice changes.
+- Form changes and URL/store synchronization can re-render the dialog multiple times in dev mode.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- If needed, memoize trigger in parent and consider React.memo when props are stable.
+

package/src/results/dialog-filter.tsx

@@ -26,18 +26,18 @@ interface Props {
     trigger: React.ReactNode;
 }
 export const DialogFilter: FC<Props> = ({ trigger }) => {
-    console.log("Render DialogFilter")
     const t = useTranslations("filter");
-    const searchLanguage = useSearchSettingsStore(state => state.language);
     const availableLanguagesAndCountries = useLanguageStore(state => state.availableLanguages);
+    const initialSettings = useSearchSettingsStore.getState();
 
-    const [like, setLike] = useState<boolean>(useSearchSettingsStore.getState().like);
-    const [operator, setOperator] = useState<string>(useSearchSettingsStore.getState().operator);
-    const [wildcard, setWildcard] = useState<string>(useSearchSettingsStore.getState().wildcard);
-    const [language, setLanguage] = useState<string>(searchLanguage || '');
+    const [form, setForm] = useState({
+        like: initialSettings.like,
+        operator: initialSettings.operator as string,
+        wildcard: initialSettings.wildcard as string,
+        language: initialSettings.language || '',
+    });
 
-    const [params, setParams] = useQueryStates({
-        search: parseAsString,
+    const [, setParams] = useQueryStates({
         language: parseAsString,
         page: parseAsInteger,
         wildcard: parseAsString,
@@ -50,27 +50,29 @@ export const DialogFilter: FC<Props> = ({ trigger }) => {
 
     const apply = () => {
         useSearchSettingsStore.getState().updatePreferences({
-            like,
-            operator: operator as OperatorType,
-            wildcard: wildcard as WildCardType,
-            language: language,
+            like: form.like,
+            operator: form.operator as OperatorType,
+            wildcard: form.wildcard as WildCardType,
+            language: form.language,
         });
 
         setParams({
             page: 1,
-            language: language,
-            operator: operator,
-            wildcard: wildcard,
-            like: like,
+            language: form.language,
+            operator: form.operator,
+            wildcard: form.wildcard,
+            like: form.like,
         });
     }
 
     const reload = () => {
-        const language = useSearchSettingsStore.getState().language || '';
-        if (language) setLanguage(language);
-        setWildcard(useSearchSettingsStore.getState().wildcard);
-        setOperator(useSearchSettingsStore.getState().operator);
-        setLike(useSearchSettingsStore.getState().like);
+        const settings = useSearchSettingsStore.getState();
+        setForm({
+            like: settings.like,
+            operator: settings.operator,
+            wildcard: settings.wildcard,
+            language: settings.language || '',
+        });
     };
 
     return (
@@ -88,7 +90,7 @@ export const DialogFilter: FC<Props> = ({ trigger }) => {
                         {t("languages")}:
                     </Label>
                     <div className="flex items-center">
-                        <Select onValueChange={setLanguage} value={language}>
+                        <Select onValueChange={(language) => setForm((prev) => ({ ...prev, language }))} value={form.language}>
                             <SelectTrigger className="w-[180px]">
                                 <SelectValue placeholder="" />
                             </SelectTrigger>
@@ -110,7 +112,10 @@ export const DialogFilter: FC<Props> = ({ trigger }) => {
                         {t("wildcard")}:
                     </Label>
                     <div className="flex items-center">
-                        <Select onValueChange={setWildcard} defaultValue={wildcard}>
+                        <Select
+                            onValueChange={(wildcard) => setForm((prev) => ({ ...prev, wildcard }))}
+                            value={form.wildcard}
+                        >
                             <SelectTrigger className="w-[180px]">
                                 <SelectValue placeholder="" />
                             </SelectTrigger>
@@ -136,8 +141,8 @@ export const DialogFilter: FC<Props> = ({ trigger }) => {
 
                     <Switch
                         id="operator"
-                        onCheckedChange={(value) => setOperator(value ? OPERATOR_OPTIONS.AND : OPERATOR_OPTIONS.OR)}
-                        checked={operator == OPERATOR_OPTIONS.AND}
+                        onCheckedChange={(value) => setForm((prev) => ({ ...prev, operator: value ? OPERATOR_OPTIONS.AND : OPERATOR_OPTIONS.OR }))}
+                        checked={form.operator == OPERATOR_OPTIONS.AND}
                     />
                 </div>
 
@@ -145,13 +150,13 @@ export const DialogFilter: FC<Props> = ({ trigger }) => {
                     <Label htmlFor="like" className="text-right">
                         {t("like")}:
                     </Label>
-                    <Switch id="like" onCheckedChange={(value) => setLike(value)} checked={like} />
+                    <Switch id="like" onCheckedChange={(like) => setForm((prev) => ({ ...prev, like }))} checked={form.like} />
                 </div>
 
                 <DialogFooter className="pt-2">
                     <DialogClose asChild>
                         <Button onClick={apply}>
-                            {params.search && params.search.length > 0 ? t("applyAndSearch") : t("apply")}
+                            {t("apply")}
                         </Button>
                     </DialogClose>
                 </DialogFooter>

package/src/results/empty.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: empty
+
+- Component: packages/components/src/results/empty.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/results/filter-navbar.analysis.md

@@ -0,0 +1,16 @@
+# Analysis: filter-navbar
+
+- Component: packages/components/src/results/filter-navbar.tsx
+- Type: Client component
+- Detected signals: useQueryStates=yes, stores=yes, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- useQueryStates triggers re-renders whenever relevant query-string values change.
+- Store subscriptions can re-render this component whenever the selected slice changes.
+
+## Possible bugs/risks
+- Query defaults based on getState() may diverge between SSR and CSR when cookie/store state changes between renders.
+
+## Recommended improvements
+- Centralize defaults from server-provided props or a stable context to reduce mismatch risk.
+

package/src/results/filter-sidebar/index.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: index
+
+- Component: packages/components/src/results/filter-sidebar/index.tsx
+- Type: Client component
+- Detected signals: useQueryStates=yes, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- useQueryStates triggers re-renders whenever relevant query-string values change.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/results/generic/table-result-list.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: table-result-list
+
+- Component: packages/components/src/results/generic/table-result-list.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- The component renders list-heavy UI; key stability and memoization strongly affect perceived performance.
+
+## Recommended improvements
+- Review key strategy and extract memoized subcomponents for large-list rendering.
+

package/src/results/generic/table-result-list.tsx

@@ -36,7 +36,7 @@ export const GenericTableResultList: FC<Props> = async ({
                 }
             </div>
 
-            {items.map((item, index) => {
+            {items.map((item) => {
                 const title = getTitle(item.titles, item.labels);
                 const itemType = getType(item.class);
                 const language = getLanguage(item.languages);
@@ -66,7 +66,7 @@ export const GenericTableResultList: FC<Props> = async ({
                             "min-h-12 c-rex-result-item flex flex-wrap items-center border-b",
                             `c-rex-result-${itemType.toLowerCase()}`,
                         )}
-                        key={index}
+                        key={item.shortId}
                     >
                         <div className="flex-1 p-2">
                             <Link href={link} className="hover:underline">{title}</Link>
@@ -96,4 +96,4 @@ export const GenericTableResultList: FC<Props> = async ({
             })}
         </div>
     );
-};
+};

package/src/results/pagination.analysis.md

@@ -0,0 +1,14 @@
+# Analysis: pagination
+
+- Component: packages/components/src/results/pagination.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- No critical bug is evident in this quick static pass; keep tests for error and loading paths.
+
+## Recommended improvements
+- Standardize loading/error handling and add regression tests to prevent divergent behavior across consuming apps.

package/src/results/table-with-images.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: table-with-images
+
+- Component: packages/components/src/results/table-with-images.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- The component renders list-heavy UI; key stability and memoization strongly affect perceived performance.
+
+## Recommended improvements
+- Review key strategy and extract memoized subcomponents for large-list rendering.
+

package/src/results/table-with-images.tsx

@@ -51,7 +51,7 @@ export const TableWithImage: FC<TableWithImageProps> = ({
 
                 return (
                     <div
-                        key={index}
+                        key={item.shortId}
                         className={cn(
                             "min-h-12 flex flex-wrap items-center border px-4 py-2 gap-2 rounded",
                             index == items.length - 1 ? "" : "mb-4",
@@ -124,4 +124,4 @@ export const TableWithImage: FC<TableWithImageProps> = ({
             })}
         </div>
     );
-};
+};

package/src/results/table.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: table
+
+- Component: packages/components/src/results/table.tsx
+- Type: Server component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=no, fetch/call=no
+
+## Possible re-render causes
+- No obvious excessive re-render trigger beyond normal prop/context updates.
+
+## Possible bugs/risks
+- The component renders list-heavy UI; key stability and memoization strongly affect perceived performance.
+
+## Recommended improvements
+- Review key strategy and extract memoized subcomponents for large-list rendering.
+

package/src/results/table.tsx

@@ -31,7 +31,7 @@ const TableView: FC<TableViewProps> = ({ items }) => {
                 <div className="w-1/2 md:w-1/5 p-2 flex justify-center">{t("files")}</div>
             </div>
 
-            {items.map((item, index) => (
+            {items.map((item) => (
                 <div
                     className={cn(
                         "min-h-12 c-rex-result-item flex flex-wrap items-center",
@@ -39,7 +39,7 @@ const TableView: FC<TableViewProps> = ({ items }) => {
                         item.disabled && "c-rex-result-item-disabled",
                         "border-b"
                     )}
-                    key={index}
+                    key={item.shortId}
                 >
                     <div className="w-4/5 md:w-2/5 p-2">
                         {item.disabled ? (item.title) : (<a href={`${item.link}?q=${query}`}>{item.title}</a>)}
@@ -64,14 +64,14 @@ const TableView: FC<TableViewProps> = ({ items }) => {
                             null // remove null rendering
                         ) : (
                             <>
-                                {Object.keys(item.files).map((fileKey, index) => {
+                                {Object.keys(item.files).map((fileKey) => {
 
                                     if (!item.files[fileKey]) {
                                         return null
                                     }
 
                                     return (
-                                        <DropdownMenu key={index}>
+                                        <DropdownMenu key={fileKey}>
                                             <DropdownMenuTrigger className="mx-2">
                                                 {IconsToFileExtension[fileKey]}
                                             </DropdownMenuTrigger>

package/src/search-input.analysis.md

@@ -0,0 +1,15 @@
+# Analysis: search-input
+
+- Component: packages/components/src/search-input.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=no, effects=yes, fetch/call=no
+
+## Possible re-render causes
+- Effects that update state can add extra renders on mount and dependency changes.
+
+## Possible bugs/risks
+- Dynamic autocomplete component selection based on pathname/local state can add redundant mount renders.
+
+## Recommended improvements
+- Avoid derived state when possible; compute autocomplete type with useMemo.
+

package/src/share-button.analysis.md

@@ -0,0 +1,19 @@
+# Analysis: share-button
+
+- Component: packages/components/src/share-button.tsx
+- Type: Client component
+- Detected signals: useQueryStates=no, stores=no, window/document=yes, effects=yes, fetch/call=no
+
+## Possible re-render causes
+- Effects that update state can add extra renders on mount and dependency changes.
+
+## Possible bugs/risks
+- Access to window/document needs care to avoid hydration mismatch and non-browser runtime issues.
+- Console logs in production can add noise and leak internal details.
+- Timeout used to reset copied state is not cleaned up on unmount.
+
+## Recommended improvements
+- Keep browser-only access inside event handlers or useEffect, not in initial render paths.
+- Remove debug logs or gate them by environment flag.
+- Store timer in ref and clear it in effect cleanup.
+