@sablier/devkit 1.6.0 → 1.7.0

package/biome/base.jsonc

@@ -6,7 +6,7 @@
 // which files to lint and format. For example:
 //
 //   {
-//     "extends": ["@sablier/devkit/biome"],
+//     "extends": ["@sablier/devkit/biome/base"],
 //     "files": {
 //       "includes": ["**/*.{css,js,jsx,json,ts,tsx}", "!node_modules"]
 //     }
@@ -38,23 +38,41 @@
   "linter": {
     "enabled": true,
     "rules": {
+      "recommended": true,
+      "complexity": {
+        "noImplicitCoercions": "error", // forbid `!!foo`, allow `Boolean(foo)`
+        "noVoid": "off" // void is useful in some cases e.g. `useEffect` callbacks
+      },
       "correctness": {
-        "noUnusedImports": "off",
-        "noUnusedVariables": "error",
-        "useExhaustiveDependencies": "error"
+        "noUnusedImports": "off", // allow unused imports during development
+        "noUnusedVariables": "error"
       },
       "nursery": {
-        "noFloatingPromises": "error"
+        "noFloatingPromises": "error", // floating promises can lead to bugs
+        "noUselessUndefined": "off" // we want explicit undefined return values
+      },
+      "performance": {
+        "noBarrelFile": "off", // barrel exports lead to cleaner imports
+        "noDelete": "off", // disabled due to https://github.com/biomejs/biome/issues/4093
+        "noNamespaceImport": "off" // namespaces are cool
       },
-      "recommended": true,
       "style": {
+        "noNamespace": "off", // namespaces are cool
+        "useDefaultSwitchClause": "off", // already handled by TypeScript
+        "useFilenamingConvention": {
+          "level": "error",
+          "options": {
+            "filenameCases": ["kebab-case", "camelCase", "PascalCase", "export"]
+          }
+        },
+        // disabled due to https://github.com/biomejs/biome/issues/8450
         "useImportType": {
           "level": "warn",
           "options": {
             "style": "separatedType"
           }
         },
-        "useTemplate": "off"
+        "useTemplate": "off" // allow string concatenation using `+`
       }
     }
   },
@@ -70,6 +88,11 @@
       "includes": ["**/*.{json,json5,jsonc}"]
     }
   ],
+  "json": {
+    "formatter": {
+      "lineWidth": 100
+    }
+  },
   "vcs": {
     "clientKind": "git",
     "enabled": true,

package/biome/ui.jsonc

@@ -1,13 +1,13 @@
 // Sablier Devkit UI configuration for Biome v2
 //
-// Standalone configuration with base rules plus UI-specific rules for
-// accessibility, Tailwind CSS class sorting, and CSS modules support.
+// Extends base configuration with UI-specific rules for accessibility,
+// Tailwind CSS class sorting, and CSS modules support.
 //
-// IMPORTANT: Consumers must add their own `files.includes` patterns to specify
-// which files to lint and format. For example:
+// IMPORTANT: Consumers must extend BOTH configs and add their own
+// `files.includes` patterns. For example:
 //
 //   {
-//     "extends": ["@sablier/devkit/biome/ui"],
+//     "extends": ["@sablier/devkit/biome/base", "@sablier/devkit/biome/ui"],
 //     "files": {
 //       "includes": ["**/*.{css,js,jsx,json,ts,tsx}", "!node_modules"]
 //     }
@@ -18,40 +18,28 @@
     "enabled": true,
     "actions": {
       "source": {
-        "organizeImports": "on",
-        "useSortedKeys": "on"
+        "useSortedAttributes": "on"
       }
     }
   },
   "css": {
+    "formatter": {
+      "lineWidth": 100
+    },
     "parser": {
       "cssModules": true,
       "tailwindDirectives": true
     }
   },
-  "files": {
-    "maxSize": 5242880 // 5MB
-  },
-  "formatter": {
-    "enabled": true,
-    "formatWithErrors": true,
-    "indentStyle": "space",
-    "lineWidth": 100
-  },
   "linter": {
-    "enabled": true,
     "rules": {
       "a11y": {
         "noSvgWithoutTitle": "off",
         "useKeyWithClickEvents": "off"
       },
-      "correctness": {
-        "noUnusedImports": "off",
-        "noUnusedVariables": "error",
-        "useExhaustiveDependencies": "error"
-      },
       "nursery": {
-        "noFloatingPromises": "error",
+        // Sort Tailwind CSS classes alphabetically
+        // See https://github.com/biomejs/biome/issues/1274#issuecomment-3490774696
         "useSortedClasses": {
           "fix": "safe",
           "level": "warn",
@@ -60,34 +48,7 @@
             "functions": ["clsx", "cva", "cn", "tv", "tw", "tw.*"]
           }
         }
-      },
-      "recommended": true,
-      "style": {
-        "useImportType": {
-          "level": "warn",
-          "options": {
-            "style": "separatedType"
-          }
-        },
-        "useTemplate": "off"
       }
     }
-  },
-  "overrides": [
-    {
-      "assist": {
-        "actions": {
-          "source": {
-            "useSortedKeys": "off"
-          }
-        }
-      },
-      "includes": ["**/*.{json,json5,jsonc}"]
-    }
-  ],
-  "vcs": {
-    "clientKind": "git",
-    "enabled": true,
-    "useIgnoreFile": true
   }
 }

package/just/csv.just

@@ -0,0 +1,76 @@
+import "./settings.just"
+
+# ---------------------------------------------------------------------------- #
+#                                    SCRIPTS                                   #
+# ---------------------------------------------------------------------------- #
+
+# Check CSV/TSV files using qsv: https://github.com/dathere/qsv
+[group("checks"), script("bash")]
+_csv-check globs="data/*/*.csv" schema="" ignore="*.csv.invalid|*.csv.valid|*validation-errors.csv" ext="CSV":
+    if ! command -v qsv >/dev/null 2>&1; then
+        echo "✗ qsv CLI not found"
+        echo "Install it: https://github.com/dathere/qsv"
+        exit 1
+    fi
+
+    echo "Validating {{ ext }} files..."
+    found=0
+    for file in {{ globs }}; do
+        # Skip validation artifact files (only if ignore pattern is set)
+        {{ if ignore != "" { "case \"$file\" in\n            " + ignore + ")\n                continue\n                ;;\n        esac" } else { "" } }}
+
+        # Skip if no files match the pattern
+        [ -e "$file" ] || continue
+
+        found=$((found + 1))
+        schema_arg="{{ schema }}"
+        if ! qsv validate "$file" ${schema_arg:+"$schema_arg"} > /dev/null 2>&1; then
+            echo "❌ Validation failed for: $file"
+            just _csv-show-errors "$file" "{{ ext }}"
+            exit 1
+        fi
+    done
+
+    if [ "$found" -eq 0 ]; then
+        echo "ℹ️  No {{ ext }} files found to validate"
+    else
+        echo "✅ All {{ ext }} files are valid"
+    fi
+
+# Show validation errors for a CSV/TSV file
+[group("checks"), script("bash")]
+_csv-show-errors file ext="CSV":
+    error_file="{{ file }}.validation-errors.tsv"
+
+    if [ ! -f "$error_file" ]; then
+        echo "Error file not found: $error_file"
+        exit 0
+    fi
+
+    # Try to count total errors (subtract 1 for header row)
+    total_errors=$(qsv count "$error_file" 2>/dev/null || echo "0")
+    if [ "$total_errors" != "0" ]; then
+        total_errors=$((total_errors - 1))
+    fi
+
+    echo ""
+    if [ "$total_errors" != "0" ] && [ "$total_errors" -gt 0 ]; then
+        echo "First 20 validation errors (${total_errors} total):"
+    else
+        echo "Validation errors:"
+    fi
+    echo ""
+
+    # Try to use qsv table for nice formatting, fall back to basic display
+    if qsv slice --start 0 --len 21 "$error_file" 2>/dev/null | qsv table 2>/dev/null; then
+        :  # Success, output already shown
+    else
+        head -n 21 "$error_file"
+    fi
+
+    if [ "$total_errors" != "0" ] && [ "$total_errors" -gt 20 ]; then
+        echo ""
+        echo "... and $((total_errors - 20)) more errors"
+    fi
+    echo ""
+    echo "Full details: $error_file"

package/just/tsv.just

@@ -1,80 +1,15 @@
-import "./settings.just"
+import "./csv.just"
 
 # ---------------------------------------------------------------------------- #
-#                                    SCRIPTS                                   #
+#                                 TSV WRAPPERS                                 #
 # ---------------------------------------------------------------------------- #
 
 # Check TSV files using qsv: https://github.com/dathere/qsv
-[group("checks"), script("bash")]
+[group("checks")]
 _tsv-check globs="data/*/*.tsv" schema="" ignore="*.tsv.invalid|*.tsv.valid|*validation-errors.tsv":
-    if ! command -v qsv >/dev/null 2>&1; then
-        echo "✗ qsv CLI not found"
-        echo "Install it: https://github.com/dathere/qsv"
-        exit 1
-    fi
-
-    echo "Validating TSV files..."
-    found=0
-    for file in {{ globs }}; do
-        # Skip validation artifact files
-        case "$file" in
-            {{ ignore }})
-                continue
-                ;;
-        esac
-
-        # Skip if no files match the pattern
-        [ -e "$file" ] || continue
-
-        found=$((found + 1))
-        schema_arg="{{ schema }}"
-        if ! qsv validate "$file" ${schema_arg:+"$schema_arg"} > /dev/null 2>&1; then
-            echo "❌ Validation failed for: $file"
-            just _tsv-show-errors "$file"
-            exit 1
-        fi
-    done
-
-    if [ "$found" -eq 0 ]; then
-        echo "ℹ️  No TSV files found to validate"
-    else
-        echo "✅ All TSV files are valid"
-    fi
+    @just _csv-check "{{ globs }}" "{{ schema }}" "{{ ignore }}" "TSV"
 
 # Show validation errors for a TSV file
-[group("checks"), script("bash")]
+[group("checks")]
 _tsv-show-errors file:
-    error_file="{{ file }}.validation-errors.tsv"
-
-    if [ ! -f "$error_file" ]; then
-        echo "Error file not found: $error_file"
-        exit 0
-    fi
-
-    # Try to count total errors (subtract 1 for header row)
-    total_errors=$(qsv count "$error_file" 2>/dev/null || echo "0")
-    if [ "$total_errors" != "0" ]; then
-        total_errors=$((total_errors - 1))
-    fi
-
-    echo ""
-    if [ "$total_errors" != "0" ] && [ "$total_errors" -gt 0 ]; then
-        echo "First 20 validation errors (${total_errors} total):"
-    else
-        echo "Validation errors:"
-    fi
-    echo ""
-
-    # Try to use qsv table for nice formatting, fall back to basic display
-    if qsv slice --start 0 --len 21 "$error_file" 2>/dev/null | qsv table 2>/dev/null; then
-        :  # Success, output already shown
-    else
-        head -n 21 "$error_file"
-    fi
-
-    if [ "$total_errors" != "0" ] && [ "$total_errors" -gt 20 ]; then
-        echo ""
-        echo "... and $((total_errors - 20)) more errors"
-    fi
-    echo ""
-    echo "Full details: $error_file"
+    @just _csv-show-errors "{{ file }}" "TSV"

package/package.json

@@ -3,7 +3,7 @@
   "description": "Configuration files and reusable scripts for Sablier repositories",
   "license": "MIT",
   "type": "module",
-  "version": "1.6.0",
+  "version": "1.7.0",
   "author": {
     "name": "Sablier Labs Ltd",
     "url": "https://sablier.com"
@@ -26,15 +26,13 @@
   },
   "exports": {
     "./biome": "./biome/base.jsonc",
+    "./biome/base": "./biome/base.jsonc",
     "./biome/ui": "./biome/ui.jsonc",
     "./prettier": "./.prettierrc.json",
     "./tsconfig/base": "./tsconfig/base.json",
     "./tsconfig/build": "./tsconfig/build.json",
     "./tsconfig/next": "./tsconfig/next.json",
-    "./vitest": {
-      "import": "./vitest/index.js",
-      "types": "./vitest/index.d.ts"
-    }
+    "./vitest": "./vitest/base.js"
   },
   "engines": {
     "node": ">=20"
@@ -42,9 +40,9 @@
   "files": [
     "biome/",
     "just/",
-    ".prettierrc.json",
     "tsconfig/",
-    "vitest/"
+    "vitest/",
+    ".prettierrc.json"
   ],
   "keywords": [
     "biome",

package/vitest/base.js

@@ -0,0 +1,32 @@
+import { defineConfig, mergeConfig } from "vitest/config";
+
+/**
+ * @typedef {Object} DevkitVitestOptions
+ * @property {"node" | "jsdom" | "happy-dom"} [environment]
+ * @property {string[]} [setupFiles]
+ * @property {boolean} [coverage]
+ */
+
+/**
+ * @param {DevkitVitestOptions} [options]
+ */
+export function defineDevkitConfig(options = {}) {
+  const isCI = Boolean(process.env.CI);
+
+  const baseConfig = {
+    test: {
+      coverage: options.coverage ? { provider: "v8" } : undefined,
+      environment: options.environment ?? "node",
+      globals: true,
+      reporters: isCI ? ["basic"] : ["verbose"],
+      retry: isCI ? 2 : 0,
+      setupFiles: options.setupFiles,
+      testTimeout: isCI ? 30_000 : 10_000,
+    },
+  };
+
+  return defineConfig(baseConfig);
+}
+
+// Re-export for merging with existing vite configs
+export { mergeConfig };

package/vitest/index.ts

@@ -1,29 +0,0 @@
-import type { UserConfig } from "vitest/config";
-import { defineConfig } from "vitest/config";
-
-export interface DevkitVitestOptions {
-  environment?: "node" | "jsdom" | "happy-dom";
-  setupFiles?: string[];
-  coverage?: boolean;
-}
-
-export function defineDevkitConfig(options: DevkitVitestOptions = {}) {
-  const isCI = !!process.env.CI;
-
-  const baseConfig: UserConfig = {
-    test: {
-      coverage: options.coverage ? { provider: "v8" } : undefined,
-      environment: options.environment ?? "node",
-      globals: true,
-      reporters: isCI ? ["basic"] : ["verbose"],
-      retry: isCI ? 2 : 0,
-      setupFiles: options.setupFiles,
-      testTimeout: isCI ? 30_000 : 10_000,
-    },
-  };
-
-  return defineConfig(baseConfig);
-}
-
-// Re-export for merging with existing vite configs
-export { mergeConfig } from "vitest/config";