Wrap createPackage in try/catch

microsoft · Feb 12, 2024 · e0b48ed · e0b48ed
1 parent 4b738ac
commit e0b48ed
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/packages/dtslint/src/checks.ts b/packages/dtslint/src/checks.ts
@@ -275,7 +275,14 @@ export async function checkNpmVersionAndGetMatchingImplementationPackage(
             `that does not conflict with an existing npm package.`,
         );
       } else {
-        implementationPackage = await attw.createPackageFromTarballUrl(tarballUrl);
+        try {
+          implementationPackage = await attw.createPackageFromTarballUrl(tarballUrl);
+        } catch (err: any) {
+          (warnings ??= []).push(
+            `Failed to extract implementation package from ${tarballUrl}. This is likely a problem with @arethetypeswrong/core ` +
+            `or the tarball data itself. @arethetypeswrong/cli will not run. Error:\n${err.stack ?? err.message}`
+          );
+        }
       }
     }
   } else if (packageJson.nonNpm === "conflict") {

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml