Gracefully handle inaccessible private package (#1528)

Co-authored-by: Björn Ahrens <[email protected]>

Author: bjoernahrens

src/lib/queryVersions.ts

@@ -86,6 +86,9 @@ async function queryVersions(packageMap: Index<VersionSpec>, options: Options =
         // allow downgrading when explicit tag is used
         pre: options.pre != null ? options.pre : targetString.startsWith('@') || isPre(version),
         retry: options.retry ?? 2,
+      }).catch(reason => {
+        // This might happen if a (private) package cannot be accessed due to a missing or invalid token.
+        return { error: reason?.body?.error || reason.toString() }
       })
 
       versionResult.version =

test/queryVersions.test.ts

@@ -25,6 +25,22 @@ describe('queryVersions', function () {
     result.should.deep.equal({})
   })
 
+  it('error while querying version should be handled', async () => {
+    const stub = stubVersions(() => {
+      throw new Error(`Package inaccessible`)
+    })
+
+    const result = await queryVersions({ async: '1.5.1' }, { loglevel: 'silent' })
+    result.should.deep.equal({
+      async: {
+        error: 'Error: Package inaccessible',
+        version: null,
+      },
+    })
+
+    stub.restore()
+  })
+
   it('local file urls should be ignored', async () => {
     const result = await queryVersions(
       { 'eslint-plugin-internal': 'file:devtools/eslint-rules' },