chore: make lint-staged target only source and scripts (no advanced ignore)

Author: jremy42

biome.json

@@ -112,6 +112,28 @@
     }
   },
   "overrides": [
+    {
+      "includes": ["**/dist/**"],
+      "linter": {
+        "enabled": false
+      }
+    },
+    {
+      "includes": ["coverage/**"],
+      "linter": {
+        "enabled": false
+      }
+    },
+    {
+      "includes": ["scripts/**/*.ts"],
+      "linter": {
+        "rules": {
+          "style": {
+            "useNamingConvention": "off"
+          }
+        }
+      }
+    },
     {
       "includes": [
         "packages/client/src/internal/interceptors/types.ts"

examples/nodejs-minimal/src/index.ts

@@ -3,7 +3,7 @@
 import { loadProfileFromConfigurationFile } from '@scaleway/configuration-loader'
 import { Registry } from '@scaleway/sdk'
 import { createClient } from '@scaleway/sdk-client'
-import * as http from 'http'
+import * as http from 'node:http'
 
 const hostname = '127.0.0.1'
 const port = 3000

examples/serverless-function-minimal/src/index.ts

@@ -18,7 +18,11 @@ const buildErrorRes = (error?: unknown) => ({
   statusCode: error instanceof Errors.ScalewayError ? error.status : 500,
 })
 
-export const list = async (event: any, context: any, callback: any) => {
+export const list = async (
+  _event: unknown,
+  _context: unknown,
+  _callback: unknown,
+) => {
   try {
     return buildJsonRes(await api.listNamespaces().all())
   } catch (error) {

package.json

@@ -26,7 +26,7 @@
     "format": "biome check --linter-enabled=false --write .",
     "format:check": "biome check --linter-enabled=false .",
     "prettier": "prettier --write '**/*.{md,mdx,yml,yaml}'",
-    "lint": "biome lint packages/*/src scripts/**/*.ts",
+    "lint": "biome lint .",
     "lint:fix": "biome check --write .",
     "prepare": "husky",
     "test": "vitest run",
@@ -35,17 +35,10 @@
     "test:ui": "vitest --ui"
   },
   "lint-staged": {
-    "globOptions": {
-      "ignore": [
-        "**/*.gen.ts",
-        "packages_generated/**",
-        "**/dist/**",
-        "**/build/**",
-        "**/coverage/**",
-        "**/examples/**"
-      ]
-    },
-    "*.{ts,tsx,js,jsx}": [
+    "packages/*/src/**/*.{ts,tsx,js,jsx}": [
+      "biome check --write --no-errors-on-unmatched --files-ignore-unknown=true"
+    ],
+    "scripts/**/*.ts": [
       "biome check --write --no-errors-on-unmatched --files-ignore-unknown=true"
     ],
     "*.json": [

packages/client/src/helpers/__tests__/json.test.ts

@@ -9,7 +9,7 @@ describe('isJSON', () => {
     },
   )
 
-  test.each([undefined, () => {}, Symbol(42)])(
+  test.each([undefined, () => { /* noop */ }, Symbol(42)])(
     `rejects %s as a valid JSON value`,
     obj => {
       expect(isJSON(obj)).toBeFalsy()
@@ -32,7 +32,7 @@ describe('isJSONObject', () => {
     null,
     [true, 'two', 3],
     undefined,
-    () => {},
+    () => { /* noop */ },
     Symbol(42),
   ])(`rejects %s as a valid JSONObject value`, obj => {
     expect(isJSONObject(obj)).toBeFalsy()

packages/client/src/helpers/marshalling.ts

@@ -67,6 +67,7 @@ type URLParameterValue = string | number | boolean | Date | null
  *
  * @internal
  */
+/* biome-ignore lint/complexity/noExcessiveCognitiveComplexity: flattening would be intrusive; keep current structure */
 export const urlParams = (
   ...paramTuples: Readonly<
     [string, URLParameterValue | URLParameterValue[] | undefined]