Fix CJS typecheck regression in loadModelsJson cache path (#42491)

Author: Copilot

actions/setup/js/apply_safe_outputs_replay.cjs

@@ -108,11 +108,7 @@ function buildHandlerConfigFromOutput(agentOutputFile) {
   }
 
   // Normalize type: convert dashes to underscores (mirrors safe_outputs_append.cjs)
-  const config = Object.fromEntries(
-    validatedOutput.items
-      .filter(item => item.type && typeof item.type === "string")
-      .map(item => [item.type.replace(/-/g, "_"), {}])
-  );
+  const config = Object.fromEntries(validatedOutput.items.filter(item => item.type && typeof item.type === "string").map(item => [item.type.replace(/-/g, "_"), {}]));
 
   core.info(`Handler config built from ${validatedOutput.items.length} item(s): ${Object.keys(config).join(", ")}`);
   return config;

actions/setup/js/model_costs.cjs

@@ -263,7 +263,7 @@ let _modelsJson = undefined;
  */
 function loadModelsJson() {
   if (_modelsJson !== undefined) {
-    return _modelsJson;
+    return _modelsJson ?? null;
   }
   try {
     const raw = fs.readFileSync(getModelsPath(), "utf8");
@@ -273,7 +273,7 @@ function loadModelsJson() {
   } catch {
     _modelsJson = null;
   }
-  return _modelsJson;
+  return _modelsJson ?? null;
 }
 
 function _resetModelCostsCache() {