vouchington-tooling 0.13.3 → 0.13.4

package/ast-grep/rules/no-three-sequential-awaits-tsx.yml

@@ -12,6 +12,10 @@ ignores:
   - 'web/test-helpers/**'
 rule:
   all:
+    - not:
+        any:
+          - matches: async-disposable-acquisition
+          - matches: async-disposable-commit
     - not:
         follows:
           stopBy: neighbor
@@ -72,6 +76,10 @@ rule:
                     pattern: $W = await Promise.all($$$)
                 - not:
                     pattern: $W = await Promise.allSettled($$$)
+          not:
+            any:
+              - matches: async-disposable-acquisition
+              - matches: async-disposable-commit
     - any:
         - all:
             - pattern:
@@ -189,8 +197,16 @@ rule:
                       pattern: $Y = await Promise.all($$$)
                   - not:
                       pattern: $Y = await Promise.allSettled($$$)
+          - not:
+              any:
+                - matches: async-disposable-acquisition
+                - matches: async-disposable-commit
           - precedes:
               stopBy: neighbor
+              not:
+                any:
+                  - matches: async-disposable-acquisition
+                  - matches: async-disposable-commit
               any:
                 - all:
                     - pattern:
@@ -261,6 +277,69 @@ constraints:
   P:
     not:
       regex: '^Promise\.all(Settled)?\b'
+utils:
+  async-disposable-acquisition:
+    all:
+      - pattern:
+          context: |
+            async function run() {
+              await using $RESOURCE = await $ACQUIRE
+            }
+          selector: expression_statement
+      - precedes:
+          # Pair only with the nearest lifecycle endpoint to the right.
+          # Overlapping-resource policy belongs to a separate lifecycle rule.
+          stopBy:
+            any:
+              - pattern:
+                  context: |
+                    async function run() {
+                      await using $_ = await $_
+                    }
+                  selector: expression_statement
+              - pattern:
+                  context: |
+                    async function run() {
+                      await $_.commit($$$)
+                    }
+                  selector: expression_statement
+          pattern:
+            context: |
+              async function run() {
+                await $RESOURCE.commit($$$)
+              }
+            selector: expression_statement
+  async-disposable-commit:
+    all:
+      - pattern:
+          context: |
+            async function run() {
+              await $RESOURCE.commit($$$)
+            }
+          selector: expression_statement
+      - follows:
+          # Pair only with the nearest lifecycle endpoint to the left.
+          # Overlapping-resource policy belongs to a separate lifecycle rule.
+          stopBy:
+            any:
+              - pattern:
+                  context: |
+                    async function run() {
+                      await using $_ = await $_
+                    }
+                  selector: expression_statement
+              - pattern:
+                  context: |
+                    async function run() {
+                      await $_.commit($$$)
+                    }
+                  selector: expression_statement
+          pattern:
+            context: |
+              async function run() {
+                await using $RESOURCE = await $ACQUIRE
+              }
+            selector: expression_statement
 examples:
   - code: |
       async function run() {
@@ -286,3 +365,102 @@ examples:
       }
     isValid: true
     file: 'web/app/example.tsx'
+  - code: |
+      async function run() {
+        await using transaction = await beginTransaction()
+        await applyChange(transaction)
+        await transaction.commit()
+        return <div />
+      }
+    isValid: true
+    file: 'web/app/example.tsx'
+  - code: |
+      async function run() {
+        await using transaction = await beginBoundedTransaction({ timeoutMs: 1000 })
+        await applyFirstChange(transaction)
+        await applySecondChange(transaction)
+        await transaction.commit()
+        return <div />
+      }
+    isValid: true
+    file: 'web/app/example.tsx'
+  - code: |
+      async function run() {
+        await using transaction = await beginTransaction()
+        await applyFirstChange(transaction)
+        await applySecondChange(transaction)
+        await applyThirdChange(transaction)
+        await transaction.commit()
+        return <div />
+      }
+    isValid: false
+    file: 'web/app/example.tsx'
+  - code: |
+      async function run() {
+        await unrelatedFirst()
+        await unrelatedSecond()
+        await unrelatedThird()
+        await using transaction = await beginTransaction()
+        await applyChange(transaction)
+        await transaction.commit()
+        return <div />
+      }
+    isValid: false
+    file: 'web/app/example.tsx'
+  - code: |
+      async function run() {
+        await using transaction = await beginTransaction()
+        await applyFirstChange(transaction)
+        await applySecondChange(transaction)
+        await other.commit()
+        return <div />
+      }
+    isValid: false
+    file: 'web/app/example.tsx'
+  - code: |
+      async function run() {
+        await using transaction = await beginTransaction()
+        await applyChange(transaction)
+        await transaction.commit()
+        await afterFirst()
+        await afterSecond()
+        await afterThird()
+        return <div />
+      }
+    isValid: false
+    file: 'web/app/example.tsx'
+  - code: |
+      async function run() {
+        await using outer = await acquire()
+        {
+          await using inner = await acquire()
+          await innerFirst(inner)
+          await innerSecond(inner)
+          await innerThird(inner)
+          await inner.commit()
+        }
+        await outer.commit()
+        return <div />
+      }
+    isValid: false
+    file: 'web/app/example.tsx'
+  - code: |
+      async function run() {
+        await using transaction = await beginTransaction()
+        await applyFirstChange(transaction)
+        await applySecondChange(transaction)
+        await transaction.rollback()
+        return <div />
+      }
+    isValid: false
+    file: 'web/app/example.tsx'
+  - code: |
+      async function run() {
+        await using resource = await acquire()
+        await useFirst(resource)
+        await useSecond(resource)
+        await resource.dispose()
+        return <div />
+      }
+    isValid: false
+    file: 'web/app/example.tsx'

package/ast-grep/rules/no-three-sequential-awaits.yml

@@ -33,6 +33,10 @@ ignores:
   - 'backend/scripts/explain-analyze/**'
 rule:
   all:
+    - not:
+        any:
+          - matches: async-disposable-acquisition
+          - matches: async-disposable-commit
     - not:
         follows:
           stopBy: neighbor
@@ -73,6 +77,10 @@ rule:
                     pattern: $W = await Promise.all($$$)
                 - not:
                     pattern: $W = await Promise.allSettled($$$)
+          not:
+            any:
+              - matches: async-disposable-acquisition
+              - matches: async-disposable-commit
     - any:
         - all:
             - pattern:
@@ -150,8 +158,16 @@ rule:
                       pattern: $Y = await Promise.all($$$)
                   - not:
                       pattern: $Y = await Promise.allSettled($$$)
+          - not:
+              any:
+                - matches: async-disposable-acquisition
+                - matches: async-disposable-commit
           - precedes:
               stopBy: neighbor
+              not:
+                any:
+                  - matches: async-disposable-acquisition
+                  - matches: async-disposable-commit
               any:
                 - all:
                     - pattern:
@@ -202,6 +218,69 @@ constraints:
   P:
     not:
       regex: '^Promise\.all(Settled)?\b'
+utils:
+  async-disposable-acquisition:
+    all:
+      - pattern:
+          context: |
+            async function run() {
+              await using $RESOURCE = await $ACQUIRE
+            }
+          selector: expression_statement
+      - precedes:
+          # Pair only with the nearest lifecycle endpoint to the right.
+          # Overlapping-resource policy belongs to a separate lifecycle rule.
+          stopBy:
+            any:
+              - pattern:
+                  context: |
+                    async function run() {
+                      await using $_ = await $_
+                    }
+                  selector: expression_statement
+              - pattern:
+                  context: |
+                    async function run() {
+                      await $_.commit($$$)
+                    }
+                  selector: expression_statement
+          pattern:
+            context: |
+              async function run() {
+                await $RESOURCE.commit($$$)
+              }
+            selector: expression_statement
+  async-disposable-commit:
+    all:
+      - pattern:
+          context: |
+            async function run() {
+              await $RESOURCE.commit($$$)
+            }
+          selector: expression_statement
+      - follows:
+          # Pair only with the nearest lifecycle endpoint to the left.
+          # Overlapping-resource policy belongs to a separate lifecycle rule.
+          stopBy:
+            any:
+              - pattern:
+                  context: |
+                    async function run() {
+                      await using $_ = await $_
+                    }
+                  selector: expression_statement
+              - pattern:
+                  context: |
+                    async function run() {
+                      await $_.commit($$$)
+                    }
+                  selector: expression_statement
+          pattern:
+            context: |
+              async function run() {
+                await using $RESOURCE = await $ACQUIRE
+              }
+            selector: expression_statement
 examples:
   - code: |
       async function run() {
@@ -240,3 +319,93 @@ examples:
       }
     isValid: true
     file: 'backend/services/example.mts'
+  - code: |
+      async function run() {
+        await using transaction = await beginTransaction()
+        await applyChange(transaction)
+        await transaction.commit()
+      }
+    isValid: true
+    file: 'backend/services/example.mts'
+  - code: |
+      async function run() {
+        await using transaction = await beginBoundedTransaction({ timeoutMs: 1000 })
+        await applyFirstChange(transaction)
+        await applySecondChange(transaction)
+        await transaction.commit()
+      }
+    isValid: true
+    file: 'backend/services/example.mts'
+  - code: |
+      async function run() {
+        await using transaction = await beginTransaction()
+        await applyFirstChange(transaction)
+        await applySecondChange(transaction)
+        await applyThirdChange(transaction)
+        await transaction.commit()
+      }
+    isValid: false
+    file: 'backend/services/example.mts'
+  - code: |
+      async function run() {
+        await unrelatedFirst()
+        await unrelatedSecond()
+        await unrelatedThird()
+        await using transaction = await beginTransaction()
+        await applyChange(transaction)
+        await transaction.commit()
+      }
+    isValid: false
+    file: 'backend/services/example.mts'
+  - code: |
+      async function run() {
+        await using transaction = await beginTransaction()
+        await applyFirstChange(transaction)
+        await applySecondChange(transaction)
+        await other.commit()
+      }
+    isValid: false
+    file: 'backend/services/example.mts'
+  - code: |
+      async function run() {
+        await using transaction = await beginTransaction()
+        await applyChange(transaction)
+        await transaction.commit()
+        await afterFirst()
+        await afterSecond()
+        await afterThird()
+      }
+    isValid: false
+    file: 'backend/services/example.mts'
+  - code: |
+      async function run() {
+        await using outer = await acquire()
+        {
+          await using inner = await acquire()
+          await innerFirst(inner)
+          await innerSecond(inner)
+          await innerThird(inner)
+          await inner.commit()
+        }
+        await outer.commit()
+      }
+    isValid: false
+    file: 'backend/services/example.mts'
+  - code: |
+      async function run() {
+        await using transaction = await beginTransaction()
+        await applyFirstChange(transaction)
+        await applySecondChange(transaction)
+        await transaction.rollback()
+      }
+    isValid: false
+    file: 'backend/services/example.mts'
+  - code: |
+      async function run() {
+        await using resource = await acquire()
+        await useFirst(resource)
+        await useSecond(resource)
+        await resource.dispose()
+      }
+    isValid: false
+    file: 'backend/services/example.mts'

package/dist/coverage-transport/outcome.d.mts

@@ -14,15 +14,16 @@ export type BlobPrimaryState = 'true' | 'false' | 'skipped';
 export declare function isBlobPrimaryState(value: string | undefined): value is BlobPrimaryState;
 /**
  * Sibling to `assertCoverageTransportOutcome` for the Vitest blob. `true`/`false` are the S3
- * upload step's `blob` output; `skipped` means that step never ran. GitHub fallback is always
- * attempted when enabled — workflows must not gate it on `blob != 'true'`.
+ * upload step's `blob` output; `skipped` means that step never ran. Workflows gate the GitHub
+ * fallback on `blob != 'true'`, so a persisted S3 primary with no GitHub artifact attempt is the
+ * expected steady state, not a degraded one.
  */
 export declare function assertCoverageTransportBlobOutcome(suite: string, primaryPersisted: BlobPrimaryState, artifactAttempt1: StepOutcome, artifactAttempt2: StepOutcome, emit?: (line: string) => void): boolean;
 export type AppendOutput = (path: string, data: string) => void;
 /**
- * Writes `blob=true|false` to `$GITHUB_OUTPUT` for outcome reporting. GitHub-fallback blob upload
- * must always be attempted when enabled; do not gate it on this signal. The upload subcommand's
- * exit code tracks the coverage pair, not the blob. A no-op outside CI (`githubOutputPath` unset).
+ * Writes `blob=true|false` to `$GITHUB_OUTPUT` for outcome reporting. Workflows gate the
+ * GitHub-fallback blob upload on this signal (`blob != 'true'`). The upload subcommand's exit
+ * code tracks the coverage pair, not the blob. A no-op outside CI (`githubOutputPath` unset).
  */
 export declare function writeUploadOutcomeOutput(outcome: {
     readonly blob: boolean;

package/dist/coverage-transport/outcome.mjs

@@ -41,15 +41,13 @@ export function isBlobPrimaryState(value) {
 }
 /**
  * Sibling to `assertCoverageTransportOutcome` for the Vitest blob. `true`/`false` are the S3
- * upload step's `blob` output; `skipped` means that step never ran. GitHub fallback is always
- * attempted when enabled — workflows must not gate it on `blob != 'true'`.
+ * upload step's `blob` output; `skipped` means that step never ran. Workflows gate the GitHub
+ * fallback on `blob != 'true'`, so a persisted S3 primary with no GitHub artifact attempt is the
+ * expected steady state, not a degraded one.
  */
 export function assertCoverageTransportBlobOutcome(suite, primaryPersisted, artifactAttempt1, artifactAttempt2, emit = (line) => process.stderr.write(`${line}\n`)) {
     const artifactSucceeded = artifactAttempt1 === 'success' || artifactAttempt2 === 'success';
     if (primaryPersisted === 'true') {
-        if (!artifactSucceeded) {
-            emit(`::warning::Vitest blob persisted only to S3 for suite=${suite}; GitHub artifact fallback is degraded.`);
-        }
         return true;
     }
     if (artifactSucceeded) {
@@ -62,9 +60,9 @@ export function assertCoverageTransportBlobOutcome(suite, primaryPersisted, arti
     return false;
 }
 /**
- * Writes `blob=true|false` to `$GITHUB_OUTPUT` for outcome reporting. GitHub-fallback blob upload
- * must always be attempted when enabled; do not gate it on this signal. The upload subcommand's
- * exit code tracks the coverage pair, not the blob. A no-op outside CI (`githubOutputPath` unset).
+ * Writes `blob=true|false` to `$GITHUB_OUTPUT` for outcome reporting. Workflows gate the
+ * GitHub-fallback blob upload on this signal (`blob != 'true'`). The upload subcommand's exit
+ * code tracks the coverage pair, not the blob. A no-op outside CI (`githubOutputPath` unset).
  */
 export function writeUploadOutcomeOutput(outcome, githubOutputPath, appendOutput = appendFileSync) {
     if (githubOutputPath)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vouchington-tooling",
-  "version": "0.13.3",
+  "version": "0.13.4",
   "description": "Vouchington CLI and extractable tooling libraries.",
   "homepage": "https://github.com/vouchington/vouchington-tooling/tree/main/packages/vouchington-tooling#readme",
   "bugs": {