@jterrazz/typescript 10.1.3 → 10.1.5

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@jterrazz/typescript",
-    "version": "10.1.3",
+    "version": "10.1.5",
     "license": "MIT",
     "author": "Jean-Baptiste Terrazzoni <contact@jterrazz.com>",
     "repository": {

package/rules/core/typescript.js

@@ -126,7 +126,6 @@ export default fragment({
                 'prefer-regexp-exec',
                 'prefer-return-this-type',
                 'prefer-string-starts-ends-with',
-                'promise-function-async',
                 'related-getter-setter-pairs',
                 'require-array-sort-compare',
                 'require-await',
@@ -163,6 +162,10 @@ export default fragment({
             typeAware(),
             'wraps a returned promise, after which nothing awaits it',
         ),
+        'typescript/promise-function-async': unsafeFix(
+            typeAware(),
+            'wraps a function that hands out an existing promise in `async … await`, so the caller receives a fresh promise nobody has claimed',
+        ),
         'typescript/no-unnecessary-type-assertion': unsafeFix(
             typeAware(),
             'drops a cast a widened platform type needs',

package/rules/core/unicorn.js

@@ -69,7 +69,6 @@ export default fragment({
                 'no-useless-error-capture-stack-trace',
                 'no-useless-fallback-in-spread',
                 'no-useless-length-check',
-                'no-useless-promise-resolve-reject',
                 'no-useless-spread',
                 'no-useless-switch-case',
                 'no-zero-fractions',
@@ -168,6 +167,10 @@ export default fragment({
             on(),
             "rewrites `fileURLToPath(new URL('.', import.meta.url))` into `import.meta.dirname`, which carries no trailing slash",
         ),
+        'unicorn/no-useless-promise-resolve-reject': unsafeFix(
+            on(),
+            'unwraps `Promise.resolve({ … })` in an arrow body into a bare object literal, which is a block',
+        ),
 
         // -- Off, each with its one reason -------------------------------------
         'unicorn/explicit-length-check': off({

package/rules/vitest.js

@@ -49,7 +49,6 @@ const ON_IN_TESTS = [
     'padding-around-test-blocks',
     'prefer-called-exactly-once-with',
     'prefer-called-once',
-    'prefer-called-with',
     'prefer-comparison-matcher',
     'prefer-each',
     'prefer-equality-matcher',
@@ -63,7 +62,6 @@ const ON_IN_TESTS = [
     'prefer-mock-return-shorthand',
     'prefer-snapshot-hint',
     'prefer-spy-on',
-    'prefer-strict-boolean-matchers',
     'prefer-strict-equal',
     'prefer-to-be',
     'prefer-to-be-object',
@@ -104,6 +102,14 @@ const REQUIRE_MOCK_TYPE_PARAMETERS = unsafeFix(
     on(),
     "rewrites `vi.mock('x', f)` into `vi.mock(import('x'), f)`, after which the factory owes the module's full type",
 );
+const PREFER_STRICT_BOOLEAN_MATCHERS = unsafeFix(
+    on(),
+    'rewrites `toBeTruthy()` into `toBe(true)` whatever the subject is — an element, a string — which is never `true`',
+);
+const PREFER_CALLED_WITH = unsafeFix(
+    on(),
+    'rewrites `toHaveBeenCalled()` into `toHaveBeenCalledWith()`, an assertion of NO arguments',
+);
 
 export default fragment({
     id: 'vitest',
@@ -117,6 +123,8 @@ export default fragment({
                 'vitest/consistent-test-it': CONSISTENT_TEST_IT,
                 'vitest/prefer-lowercase-title': PREFER_LOWERCASE_TITLE,
                 'vitest/require-mock-type-parameters': REQUIRE_MOCK_TYPE_PARAMETERS,
+                'vitest/prefer-strict-boolean-matchers': PREFER_STRICT_BOOLEAN_MATCHERS,
+                'vitest/prefer-called-with': PREFER_CALLED_WITH,
 
                 'vitest/no-conditional-in-test': off({
                     by: "vitest/no-conditional-expect — the defect is an assertion that may not run, and that rule names it; this one also refuses a golden suite's TEST_UPDATE branch and every comparator",