@appthrust/kest 0.12.0 → 0.13.1

package/README.md

@@ -1180,6 +1180,26 @@ KEST_PRESERVE_ON_FAILURE=1 bun test
 
 When active, Kest skips cleanup for failed scenarios so you can inspect the cluster with `kubectl`. The test report will show a "Cleanup (skipped)" notice instead of the usual cleanup table. Remember to delete the leftover resources manually once you're done investigating.
 
+### Biome configuration
+
+If you use [Biome](https://biomejs.dev/) for linting, the `noDoneCallback` rule may flag kest's `test` callback parameter as a false positive. The parameter is a `Scenario` object, not a done callback.
+
+To suppress this, extend kest's shareable Biome config in your `biome.json`:
+
+```json
+{
+  "extends": ["@appthrust/kest/biome/recommended.json"]
+}
+```
+
+This disables `lint/style/noDoneCallback` while leaving all other rules untouched. If you already extend another config, add kest's config after it:
+
+```json
+{
+  "extends": ["@suin/biome.json", "@appthrust/kest/biome/recommended.json"]
+}
+```
+
 ## Environment Variables
 
 | Variable                    | Description                                                             |

package/biome/recommended.json

@@ -0,0 +1,10 @@
+{
+  "$schema": "https://biomejs.dev/schemas/2.3.13/schema.json",
+  "linter": {
+    "rules": {
+      "style": {
+        "noDoneCallback": "off"
+      }
+    }
+  }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@appthrust/kest",
-  "version": "0.12.0",
+  "version": "0.13.1",
   "description": "Kubernetes E2E testing framework designed for humans and AI alike",
   "type": "module",
   "module": "ts/index.ts",
@@ -8,11 +8,13 @@
     ".": {
       "types": "./ts/index.ts",
       "default": "./ts/index.ts"
-    }
+    },
+    "./biome/recommended.json": "./biome/recommended.json"
   },
   "files": [
     "ts/**/*.ts",
     "!ts/**/*.test.ts",
+    "biome/",
     "example/",
     "LICENSE",
     "README.md"

package/ts/test.ts

@@ -17,8 +17,7 @@ interface TestOptions {
   timeout?: string;
 }
 
-const defaultTimeout = 60_000;
-setDefaultTimeout(defaultTimeout);
+setDefaultTimeout(60_000);
 
 type Callback = (scenario: Scenario) => Promise<unknown>;
 
@@ -98,12 +97,12 @@ function makeScenarioTest(runner: BunTestRunner): TestFunction {
 function convertTestOptions(
   options?: undefined | TestOptions
 ): undefined | BunTestOptions {
-  const timeout = options?.timeout
-    ? parseDuration(options.timeout).toMilliseconds()
-    : defaultTimeout;
+  if (!options?.timeout) {
+    return undefined;
+  }
   return {
     ...options,
-    timeout,
+    timeout: parseDuration(options.timeout).toMilliseconds(),
   };
 }