@c-rex/components 0.1.31 → 0.1.33

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.
+