fix: memoize apt update/init via micromemoize

Author: aminya

dist/actions/setup-cpp.js

@@ -26,7 +26,8 @@
     "which": "4.0.0",
     "execa": "^7.2.0",
     "escape-string-regexp": "^5.0.0",
-    "node-downloader-helper": "2.1.9"
+    "node-downloader-helper": "2.1.9",
+    "micro-memoize": "^4.1.2"
   },
   "engines": {
     "node": ">=12"

dist/actions/setup-cpp.js.map

@@ -2,13 +2,19 @@ import { defaultExecOptions, execRootSync } from "admina"
 import { getAptEnv } from "./apt-env.js"
 import { aptTimeout } from "./apt-timeout.js"
 import { getApt } from "./get-apt.js"
+import { initAptMemoized } from "./init-apt.js"
 import { isAptPackInstalled } from "./is-installed.js"
-import { updateAptRepos } from "./update.js"
+import { updateAptReposMemoized } from "./update.js"
 
 export async function addAptRepository(repo: string, apt = getApt()) {
+  await initAptMemoized(apt)
   await installAddAptRepo(apt)
   execRootSync("add-apt-repository", ["-y", "--no-update", repo], { ...defaultExecOptions, env: getAptEnv(apt) })
-  updateAptRepos(apt)
+
+  // clear the cache
+  updateAptReposMemoized.cache.keys = []
+  updateAptReposMemoized.cache.values = []
+  updateAptReposMemoized(apt)
 }
 
 export async function installAddAptRepo(apt: string) {

dist/legacy/setup-cpp.js

@@ -1,13 +1,14 @@
 import { defaultExecOptions, execRootSync } from "admina"
+import memoize from "micro-memoize"
 import { getAptEnv } from "./apt-env.js"
 import { aptTimeout } from "./apt-timeout.js"
 import { filterAndQualifyAptPackages } from "./qualify-install.js"
-import { updateAptRepos } from "./update.js"
+import { updateAptReposMemoized } from "./update.js"
 
 /** Install gnupg and certificates (usually missing from docker containers) */
 export async function initApt(apt: string) {
   // Update the repos
-  updateAptRepos(apt)
+  updateAptReposMemoized(apt)
 
   const toInstall = await filterAndQualifyAptPackages(apt, [
     { name: "ca-certificates" },
@@ -22,3 +23,6 @@ export async function initApt(apt: string) {
     })
   }
 }
+
+/** Install gnupg and certificates (usually missing from docker containers) (memoized) */
+export const initAptMemoized = memoize(initApt, { isPromise: true })

dist/legacy/setup-cpp.js.map

@@ -6,9 +6,9 @@ import { type AddAptKeyOptions, addAptKey } from "./apt-key.js"
 import { addAptRepository } from "./apt-repository.js"
 import { aptTimeout } from "./apt-timeout.js"
 import { getApt } from "./get-apt.js"
-import { initApt } from "./init-apt.js"
+import { initAptMemoized } from "./init-apt.js"
 import { filterAndQualifyAptPackages } from "./qualify-install.js"
-import { updateAptRepos } from "./update.js"
+import { updateAptReposMemoized } from "./update.js"
 
 /**
  * The information about an installation result
@@ -22,10 +22,6 @@ export type InstallationInfo = {
   bin?: string
 }
 
-/* eslint-disable require-atomic-updates */
-export let didUpdate: boolean = false
-let didInit: boolean = false
-
 /**
  * The information about an apt package
  */
@@ -80,8 +76,7 @@ export async function installAptPack(packages: AptPackage[], update = false): Pr
 
   // Update the repos if needed
   if (update) {
-    updateAptRepos(apt)
-    didUpdate = true
+    updateAptReposMemoized(apt)
   }
 
   // Add the repos if needed
@@ -95,10 +90,7 @@ export async function installAptPack(packages: AptPackage[], update = false): Pr
   }
 
   // Initialize apt if needed
-  if (!didInit) {
-    await initApt(apt)
-    didInit = true
-  }
+  await initAptMemoized(apt)
 
   try {
     // Add the keys if needed