@m5kdev/web-ui 0.1.0 → 0.1.2

package/.turbo/turbo-lint$colon$fix.log

@@ -0,0 +1,381 @@
+
+
+> @repo/web-ui@0.0.1 lint:fix /Users/michalkow/Projects/m5/packages/web-ui
+> biome check . --write
+
+src/components/ColorPicker.tsx:17:12 ]8;;https://biomejs.dev/linter/rules/no-unused-variables\lint/correctness/noUnusedVariables]8;;\  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ This variable err is unused.
+  
+    15 │   try {
+    16 │     return hexToHsva(hex);
+  > 17 │   } catch (err) {
+       │            ^^^
+    18 │     return { h: 0, s: 0, v: 0, a: 1 };
+    19 │   }
+  
+  ℹ Unused variables are often the result of an incomplete refactoring, typos, or other sources of bugs.
+  
+  ℹ Unsafe fix: If this is intentional, prepend err with an underscore.
+  
+    15 15 │     try {␍
+    16 16 │       return hexToHsva(hex);␍
+    17    │ - ··}·catch·(err)·{␍
+       17 │ + ··}·catch·(_err)·{␍
+    18 18 │       return { h: 0, s: 0, v: 0, a: 1 };␍
+    19 19 │     }␍
+  
+
+src/components/MultiSelectDropdown.tsx:103:7 ]8;;https://biomejs.dev/linter/rules/no-for-each\lint/complexity/noForEach]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Prefer for...of instead of forEach.
+  
+    101 │       const ungrouped: MultiSelectOption[] = [];
+    102 │ 
+  > 103 │       options.forEach((option) => {
+        │       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+  > 104 │         if (option.group) {
+         ...
+  > 111 │         }
+  > 112 │       });
+        │       ^^
+    113 │ 
+    114 │       return { groups, ungrouped };
+  
+  ℹ forEach may lead to performance issues when working with large arrays. When combined with functions like filter or map, this causes multiple iterations over the same type.
+  
+
+src/components/app-header.tsx:22:42 ]8;;https://biomejs.dev/linter/rules/no-array-index-key\lint/suspicious/noArrayIndexKey]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Avoid using the index of an array as key property in an element.
+  
+    20 │           <BreadcrumbList>
+    21 │             {breadcrumbs.map(({ href, label }, index) => (
+  > 22 │               <Fragment key={`${label}-${index}`}>
+       │                                          ^^^^^
+    23 │                 <BreadcrumbItem>
+    24 │                   {href ? (
+  
+  ℹ This is the source of the key value.
+  
+    19 │         <Breadcrumb>
+    20 │           <BreadcrumbList>
+  > 21 │             {breadcrumbs.map(({ href, label }, index) => (
+       │                                                ^^^^^
+    22 │               <Fragment key={`${label}-${index}`}>
+    23 │                 <BreadcrumbItem>
+  
+  ℹ The order of the items may change, and this also affects performances and component state.
+  
+  ℹ Check the ]8;;https://reactjs.org/docs/lists-and-keys.html#keys\React documentation]8;;\. 
+  
+
+src/components/features-section-demo-1.tsx:93:73 ]8;;https://biomejs.dev/linter/rules/no-explicit-any\lint/suspicious/noExplicitAny]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Unexpected any. Specify a different type.
+  
+    91 │ };
+    92 │ 
+  > 93 │ export function GridPattern({ width, height, x, y, squares, ...props }: any) {
+       │                                                                         ^^^
+    94 │   const patternId = useId();
+    95 │ 
+  
+  ℹ any disables many type checking rules. Its use should be avoided.
+  
+
+src/components/features-section-demo-1.tsx:112:9 ]8;;https://biomejs.dev/linter/rules/no-svg-without-title\lint/a11y/noSvgWithoutTitle]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Alternative text title element cannot be empty
+  
+    110 │       <rect width="100%" height="100%" strokeWidth={0} fill={`url(#${patternId})`} />
+    111 │       {squares && (
+  > 112 │         <svg x={x} y={y} className="overflow-visible">
+        │         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+    113 │           {squares.map(([x, y]: any) => (
+    114 │             <rect
+  
+  ℹ For accessibility purposes, SVGs should have an alternative text, provided via title element. If the svg element has role="img", you should add the aria-label or aria-labelledby attribute.
+  
+
+src/components/features-section-demo-1.tsx:113:33 ]8;;https://biomejs.dev/linter/rules/no-explicit-any\lint/suspicious/noExplicitAny]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Unexpected any. Specify a different type.
+  
+    111 │       {squares && (
+    112 │         <svg x={x} y={y} className="overflow-visible">
+  > 113 │           {squares.map(([x, y]: any) => (
+        │                                 ^^^
+    114 │             <rect
+    115 │               strokeWidth="0"
+  
+  ℹ any disables many type checking rules. Its use should be avoided.
+  
+
+src/components/features-section-demo-3.tsx:171:18 ]8;;https://biomejs.dev/linter/rules/use-template\lint/style/useTemplate]8;;\  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Template literals are preferred over string concatenation.
+  
+    169 │           <motion.div
+    170 │             variants={imageVariants}
+  > 171 │             key={"images-first" + idx}
+        │                  ^^^^^^^^^^^^^^^^^^^^
+    172 │             style={{
+    173 │               rotate: Math.random() * 20 - 10,
+  
+  ℹ Unsafe fix: Use a template literal.
+  
+    169 169 │             <motion.div␍
+    170 170 │               variants={imageVariants}␍
+    171     │ - ············key={"images-first"·+·idx}␍
+        171 │ + ············key={`images-first${idx}`}␍
+    172 172 │               style={{␍
+    173 173 │                 rotate: Math.random() * 20 - 10,␍
+  
+
+src/components/features-section-demo-3.tsx:192:18 ]8;;https://biomejs.dev/linter/rules/use-template\lint/style/useTemplate]8;;\  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Template literals are preferred over string concatenation.
+  
+    190 │         {images.map((image, idx) => (
+    191 │           <motion.div
+  > 192 │             key={"images-second" + idx}
+        │                  ^^^^^^^^^^^^^^^^^^^^^
+    193 │             style={{
+    194 │               rotate: Math.random() * 20 - 10,
+  
+  ℹ Unsafe fix: Use a template literal.
+  
+    190 190 │           {images.map((image, idx) => (␍
+    191 191 │             <motion.div␍
+    192     │ - ············key={"images-second"·+·idx}␍
+        192 │ + ············key={`images-second${idx}`}␍
+    193 193 │               style={{␍
+    194 194 │                 rotate: Math.random() * 20 - 10,␍
+  
+
+src/components/features-section-demo-3.tsx:171:35 ]8;;https://biomejs.dev/linter/rules/no-array-index-key\lint/suspicious/noArrayIndexKey]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Avoid using the index of an array as key property in an element.
+  
+    169 │           <motion.div
+    170 │             variants={imageVariants}
+  > 171 │             key={"images-first" + idx}
+        │                                   ^^^
+    172 │             style={{
+    173 │               rotate: Math.random() * 20 - 10,
+  
+  ℹ This is the source of the key value.
+  
+    166 │       {/* TODO */}
+    167 │       <div className="flex flex-row -ml-20">
+  > 168 │         {images.map((image, idx) => (
+        │                             ^^^
+    169 │           <motion.div
+    170 │             variants={imageVariants}
+  
+  ℹ The order of the items may change, and this also affects performances and component state.
+  
+  ℹ Check the ]8;;https://reactjs.org/docs/lists-and-keys.html#keys\React documentation]8;;\. 
+  
+
+src/components/features-section-demo-3.tsx:192:36 ]8;;https://biomejs.dev/linter/rules/no-array-index-key\lint/suspicious/noArrayIndexKey]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Avoid using the index of an array as key property in an element.
+  
+    190 │         {images.map((image, idx) => (
+    191 │           <motion.div
+  > 192 │             key={"images-second" + idx}
+        │                                    ^^^
+    193 │             style={{
+    194 │               rotate: Math.random() * 20 - 10,
+  
+  ℹ This is the source of the key value.
+  
+    188 │       </div>
+    189 │       <div className="flex flex-row">
+  > 190 │         {images.map((image, idx) => (
+        │                             ^^^
+    191 │           <motion.div
+    192 │             key={"images-second" + idx}
+  
+  ℹ The order of the items may change, and this also affects performances and component state.
+  
+  ℹ Check the ]8;;https://reactjs.org/docs/lists-and-keys.html#keys\React documentation]8;;\. 
+  
+
+src/components/pricing-cards.tsx:49:25 ]8;;https://biomejs.dev/linter/rules/no-array-index-key\lint/suspicious/noArrayIndexKey]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Avoid using the index of an array as key property in an element.
+  
+    47 │           <div className="flex flex-col gap-4 justify-start">
+    48 │             {features.map((feature, index) => (
+  > 49 │               <div key={index} className="flex flex-row gap-4">
+       │                         ^^^^^
+    50 │                 <Check className="w-4 h-4 mt-2 text-primary" />
+    51 │                 <div className="flex flex-col">
+  
+  ℹ This is the source of the key value.
+  
+    46 │           </p>
+    47 │           <div className="flex flex-col gap-4 justify-start">
+  > 48 │             {features.map((feature, index) => (
+       │                                     ^^^^^
+    49 │               <div key={index} className="flex flex-row gap-4">
+    50 │                 <Check className="w-4 h-4 mt-2 text-primary" />
+  
+  ℹ The order of the items may change, and this also affects performances and component state.
+  
+  ℹ Check the ]8;;https://reactjs.org/docs/lists-and-keys.html#keys\React documentation]8;;\. 
+  
+
+src/components/theme-provider.tsx:31:3 ]8;;https://biomejs.dev/linter/rules/use-exhaustive-dependencies\lint/correctness/useExhaustiveDependencies]8;;\  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ This hook does not specify its dependency on storageKey.
+  
+    29 │   const [theme, setTheme] = useState<Theme>(defaultTheme);
+    30 │ 
+  > 31 │   useEffect(() => {
+       │   ^^^^^^^^^
+    32 │     const savedTheme = localStorage.getItem(storageKey);
+    33 │     if (savedTheme) setTheme(savedTheme as Theme);
+  
+  ℹ This dependency is being used here, but is not specified in the hook dependency list.
+  
+    31 │   useEffect(() => {
+  > 32 │     const savedTheme = localStorage.getItem(storageKey);
+       │                                             ^^^^^^^^^^
+    33 │     if (savedTheme) setTheme(savedTheme as Theme);
+    34 │   }, []);
+  
+  ℹ Either include it or remove the dependency array.
+  
+  ℹ Unsafe fix: Add the missing dependency to the list.
+  
+    34 │ ··},·[storageKey]);␍
+       │       ++++++++++   
+
+src/components/ui/bento-grid2.tsx:27:9 ]8;;https://biomejs.dev/linter/rules/no-explicit-any\lint/suspicious/noExplicitAny]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Unexpected any. Specify a different type.
+  
+    25 │   className: string;
+    26 │   background: ReactNode;
+  > 27 │   Icon: any;
+       │         ^^^
+    28 │   description: string;
+    29 │   href: string;
+  
+  ℹ any disables many type checking rules. Its use should be avoided.
+  
+
+src/components/ui/floating-navbar.tsx:25:25 ]8;;https://biomejs.dev/linter/rules/no-non-null-assertion\lint/style/noNonNullAssertion]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Forbidden non-null assertion.
+  
+    23 │     // Check if current is not undefined and is a number
+    24 │     if (typeof current === "number") {
+  > 25 │       const direction = current! - scrollYProgress.getPrevious()!;
+       │                         ^^^^^^^^
+    26 │ 
+    27 │       if (scrollYProgress.get() < 0.05) {
+  
+
+src/components/ui/floating-navbar.tsx:25:36 ]8;;https://biomejs.dev/linter/rules/no-non-null-assertion\lint/style/noNonNullAssertion]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Forbidden non-null assertion.
+  
+    23 │     // Check if current is not undefined and is a number
+    24 │     if (typeof current === "number") {
+  > 25 │       const direction = current! - scrollYProgress.getPrevious()!;
+       │                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+    26 │ 
+    27 │       if (scrollYProgress.get() < 0.05) {
+  
+
+src/components/ui/floating-navbar.tsx:60:33 ]8;;https://biomejs.dev/linter/rules/no-explicit-any\lint/suspicious/noExplicitAny]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Unexpected any. Specify a different type.
+  
+    58 │         )}
+    59 │       >
+  > 60 │         {navItems.map((navItem: any, idx: number) => (
+       │                                 ^^^
+    61 │           <a
+    62 │             key={`link=${idx}`}
+  
+  ℹ any disables many type checking rules. Its use should be avoided.
+  
+
+src/components/ui/floating-navbar.tsx:62:26 ]8;;https://biomejs.dev/linter/rules/no-array-index-key\lint/suspicious/noArrayIndexKey]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Avoid using the index of an array as key property in an element.
+  
+    60 │         {navItems.map((navItem: any, idx: number) => (
+    61 │           <a
+  > 62 │             key={`link=${idx}`}
+       │                          ^^^
+    63 │             href={navItem.link}
+    64 │             className={cn(
+  
+  ℹ This is the source of the key value.
+  
+    58 │         )}
+    59 │       >
+  > 60 │         {navItems.map((navItem: any, idx: number) => (
+       │                                      ^^^
+    61 │           <a
+    62 │             key={`link=${idx}`}
+  
+  ℹ The order of the items may change, and this also affects performances and component state.
+  
+  ℹ Check the ]8;;https://reactjs.org/docs/lists-and-keys.html#keys\React documentation]8;;\. 
+  
+
+src/modules/auth/LoginForm.tsx:53:13 ]8;;https://biomejs.dev/linter/rules/no-svg-without-title\lint/a11y/noSvgWithoutTitle]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Alternative text title element cannot be empty
+  
+    51 │         <div className="flex flex-col gap-4">
+    52 │           <Button variant="outline" className="w-full">
+  > 53 │             <svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24">
+       │             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+    54 │               <path
+    55 │                 d="M12.152 6.896c-.948 0-2.415-1.078-3.96-1.04-2.04.027-3.91 1.183-4.961 3.014-2.117 3.675-.546 9.103 1.519 12.09 1.013 1.454 2.208 3.09 3.792 3.039 1.52-.065 2.09-.987 3.935-.987 1.831 0 2.35.987 3.96.948 1.637-.026 2.676-1.48 3.676-2.948 1.156-1.688 1.636-3.325 1.662-3.415-.039-.013-3.182-1.221-3.22-4.857-.026-3.04 2.48-4.494 2.597-4.559-1.429-2.09-3.623-2.324-4.39-2.376-2-.156-3.675 1.09-4.61 1.09zM15.53 3.83c.843-1.012 1.4-2.427 1.245-3.83-1.207.052-2.662.805-3.532 1.818-.78.896-1.454 2.338-1.273 3.714 1.338.104 2.715-.688 3.559-1.701"
+  
+  ℹ For accessibility purposes, SVGs should have an alternative text, provided via title element. If the svg element has role="img", you should add the aria-label or aria-labelledby attribute.
+  
+
+src/modules/auth/LoginForm.tsx:62:13 ]8;;https://biomejs.dev/linter/rules/no-svg-without-title\lint/a11y/noSvgWithoutTitle]8;;\ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Alternative text title element cannot be empty
+  
+    60 │           </Button>
+    61 │           <Button variant="outline" className="w-full">
+  > 62 │             <svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24">
+       │             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+    63 │               <path
+    64 │                 d="M12.48 10.92v3.28h7.84c-.24 1.84-.853 3.187-1.787 4.133-1.147 1.147-2.933 2.4-6.053 2.4-4.827 0-8.6-3.893-8.6-8.72s3.773-8.72 8.6-8.72c2.6 0 4.507 1.027 5.907 2.347l2.307-2.307C18.747 1.44 16.133 0 12.48 0 5.867 0 .307 5.387.307 12s5.56 12 12.173 12c3.573 0 6.267-1.173 8.373-3.36 2.16-2.16 2.84-5.213 2.84-7.667 0-.76-.053-1.467-.173-2.053H12.48z"
+  
+  ℹ For accessibility purposes, SVGs should have an alternative text, provided via title element. If the svg element has role="img", you should add the aria-label or aria-labelledby attribute.
+  
+
+src/modules/auth/LogoutRoute.tsx:9:3 suppressions/unused ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  ⚠ Suppression comment has no effect. Remove the suppression or make sure you are suppressing the correct rule.
+  
+     7 │   const navigate = useNavigate();
+     8 │ 
+   > 9 │   // biome-ignore lint/correctness/useExhaustiveDependencies(authClient): authClient is a singleton
+       │   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+    10 │   // biome-ignore lint/correctness/useExhaustiveDependencies(navigate): navigate is a global hook
+    11 │   useEffect(() => {
+  
+
+Skipped 10 suggested fixes.
+If you wish to apply the suggested (unsafe) fixes, use the command biome check --write --unsafe
+
+The number of diagnostics exceeds the limit allowed. Use --max-diagnostics to increase it.
+Diagnostics not shown: 15.
+Checked 80 files in 227ms. No fixes applied.
+Found 35 warnings.