From 4df51974e39c3f799c1f5fe9aaef8008f35af6b4 Mon Sep 17 00:00:00 2001
From: sean <seano@wix.com>
Date: Sun, 26 Jan 2025 17:11:07 +0200
Subject: [PATCH 1/2] call errorMessage

---
 packages/ata/src/index.ts | 32 ++++++++++++++++++++++++++------
 1 file changed, 26 insertions(+), 6 deletions(-)

diff --git a/packages/ata/src/index.ts b/packages/ata/src/index.ts
index f52d3dcb04f8..92fc0f917a8f 100644
--- a/packages/ata/src/index.ts
+++ b/packages/ata/src/index.ts
@@ -68,7 +68,15 @@ export const setupTypeAcquisition = (config: ATABootstrapConfig) => {
 
     // Grab the module trees which gives us a list of files to download
     const trees = await Promise.all(depsToGet.map(f => getFileTreeForModuleWithTag(config, f.module, f.version)))
-    const treesOnly = trees.filter(t => !("error" in t)) as NPMTreeMeta[]
+    const treesOnly: NPMTreeMeta[] = []
+
+    trees.forEach(t => {
+      if ("error" in t) {
+        config.delegate.errorMessage?.(t.userFacingMessage, t.error)
+      } else {
+        treesOnly.push(t)
+      }
+    })
 
     // These are the modules which we can grab directly
     const hasDTS = treesOnly.filter(t => t.files.find(f => isDtsFile(f.name)))
@@ -81,7 +89,16 @@ export const setupTypeAcquisition = (config: ATABootstrapConfig) => {
       mightBeOnDT.map(f => getFileTreeForModuleWithTag(config, `@types/${getDTName(f.moduleName)}`, "latest"))
     )
 
-    const dtTreesOnly = dtTrees.filter(t => !("error" in t)) as NPMTreeMeta[]
+    const dtTreesOnly: NPMTreeMeta[] = []
+
+    dtTrees.forEach(t => {
+      if ("error" in t) {
+        config.delegate.errorMessage?.(t.userFacingMessage, t.error)
+      } else {
+        dtTreesOnly.push(t)
+      }
+    })
+
     const dtsFilesFromDT = dtTreesOnly.map(t => treeToDTSFiles(t, `/node_modules/@types/${getDTName(t.moduleName).replace("types__", "")}`))
 
     // Collect all the npm and DT DTS requests and flatten their arrays
@@ -102,7 +119,9 @@ export const setupTypeAcquisition = (config: ATABootstrapConfig) => {
         fsMap.set(path, pkgJSON)
         config.delegate.receivedFile?.(pkgJSON, path)
       } else {
-        config.logger?.error(`Could not download package.json for ${tree.moduleName}`)
+        const userFacingMessage = `Could not download package.json for ${tree.moduleName}`
+        config.logger?.error(userFacingMessage)
+        config.delegate.errorMessage?.(userFacingMessage, pkgJSON)
       }
     }
 
@@ -112,8 +131,9 @@ export const setupTypeAcquisition = (config: ATABootstrapConfig) => {
         const dtsCode = await getDTSFileForModuleWithVersion(config, dts.moduleName, dts.moduleVersion, dts.path)
         estimatedDownloaded++
         if (dtsCode instanceof Error) {
-          // TODO?
-          config.logger?.error(`Had an issue getting ${dts.path} for ${dts.moduleName}`)
+          const userFacingMessage = `Had an issue getting ${dts.path} for ${dts.moduleName}`
+          config.logger?.error(userFacingMessage)
+          config.delegate.errorMessage?.(userFacingMessage, dtsCode)
         } else {
           fsMap.set(dts.vfsPath, dtsCode)
           config.delegate.receivedFile?.(dtsCode, dts.vfsPath)
@@ -226,7 +246,7 @@ export const getFileTreeForModuleWithTag = async (
       const versions = await getNPMVersionsForModule(config, moduleName)
       if (versions instanceof Error) {
         return {
-          error: response,
+          error: versions,
           userFacingMessage: `Could not get versions on npm for ${moduleName} - possible typo?`,
         }
       }

From fb127cea6f7e7bc4728da6840cd823e8bff8469d Mon Sep 17 00:00:00 2001
From: sean <seano@wix.com>
Date: Sun, 26 Jan 2025 23:23:56 +0200
Subject: [PATCH 2/2] added changeset

---
 .changeset/green-bulldogs-vanish.md | 5 +++++
 1 file changed, 5 insertions(+)
 create mode 100644 .changeset/green-bulldogs-vanish.md

diff --git a/.changeset/green-bulldogs-vanish.md b/.changeset/green-bulldogs-vanish.md
new file mode 100644
index 000000000000..54fcbcae0814
--- /dev/null
+++ b/.changeset/green-bulldogs-vanish.md
@@ -0,0 +1,5 @@
+---
+"@typescript/ata": patch
+---
+
+Added calls to delegate.errorMessage() on errors