@apo-ai/sdk 0.2.0 → 0.2.2

package/dist/agent-task/public.d.ts

@@ -61,13 +61,15 @@ type AgentTurnResult = {
 };
 /**
  * The adapter-reported identity of the agent under test for one Task Run
- *. The adapter resolves its configuration (env vars, aliases,
+ * The adapter resolves its configuration (env vars, aliases,
  * defaults), constructs the agent from that same resolved object, and reports
  * the resolved values here.
  *
  * - `model` is the exact identifier the adapter passed to its runtime.
- * - `effort` is the exact short value that runtime used (`low`/`medium`/...
- *   or `"default"`). Omitted means "the adapter did not report effort".
+ * - `effort` is the exact short value that the selected model/provider applied
+ *   (`low`/`medium`/... or `"default"`). Omit it when effort is unsupported,
+ *   ignored, or cannot be verified. A configured default or accepted request
+ *   parameter is not an applied effort by itself.
  *
  * The whole configuration is absent (`undefined`) when the adapter does not
  * support reporting it. Values are descriptive — they never affect task

package/dist/agent-task/public.js

@@ -979,13 +979,13 @@ async function loadTaskDefinition(taskFilePath) {
     const task2 = registeredTask ?? loaded.default;
     if (!task2 || typeof task2 !== "object") {
       throw new Error(
-        "No task definition found \u2014 call task(name, config) or default export defineTask(adapter, config)"
+        "No task definition found. The .eval.ts file must call task(name, { adapter, deliverables, ... }) at module top level. If it does, the CLI and the .eval.ts may be loading different @apo-ai/sdk instances (check for a stale packages/sdk/dist build or a version mismatch between the CLI and node_modules)."
       );
     }
     const adapter = getTaskAdapterDefinition(task2);
     if (!adapter) {
       throw new Error(
-        registeredTask ? "task() config must include an adapter" : "Task module must default export defineTask(adapter, {...})"
+        registeredTask ? "task() config must include an adapter" : "Task module must call task(name, { adapter, ... }) \u2014 the config is missing an adapter."
       );
     }
     return {
@@ -1109,15 +1109,15 @@ function fileArtifact(path, options = {}) {
   try {
     const lstat = lstatSync(path);
     if (lstat.isSymbolicLink()) {
-      throw new Error(`fileArtifact: '${path}' is a symbolic link, not a regular file`);
+      throw new Error("fileArtifact: path is a symbolic link, not a regular file");
     }
     stat = statSync2(path);
   } catch (error) {
-    if (!existsLike(error)) throw error;
-    throw new Error(`fileArtifact: '${path}' is not a regular file`);
+    if (error instanceof Error && error.message.startsWith("fileArtifact:")) throw error;
+    throw new Error("fileArtifact: path is not a regular file or is inaccessible");
   }
   if (!stat.isFile()) {
-    throw new Error(`fileArtifact: '${path}' is not a regular file`);
+    throw new Error("fileArtifact: path is not a regular file");
   }
   const displayFilename = options.displayFilename ?? basename3(path);
   if (!displayFilename) {
@@ -1133,21 +1133,19 @@ function fileArtifact(path, options = {}) {
 function isFileArtifact(value) {
   return typeof value === "object" && value !== null && value.kind === "apo.file-artifact" && typeof value.path === "string";
 }
-function existsLike(error) {
-  if (!(error instanceof Error)) return false;
-  const code = error.code;
-  return code === "ENOENT";
-}
 
 // src/agent-task/deliverables/upload.ts
 import { createHash } from "crypto";
 import { createReadStream } from "fs";
-import { basename as basename4 } from "path";
 import { Readable } from "stream";
 var DEFAULT_FETCH = typeof fetch === "function" ? fetch : void 0;
 async function persistFileArtifacts(deliverables, config) {
   const jsonDeliverables = {};
   const artifactUploads = [];
+  const artifactNames = Object.entries(deliverables).filter(([, v]) => isFileArtifact(v)).map(([name]) => name);
+  if (artifactNames.length > 0) {
+    validateUploadConfig(config, artifactNames[0]);
+  }
   for (const [name, value] of Object.entries(deliverables)) {
     if (isFileArtifact(value)) {
       const summary = await uploadOne(name, value, config);
@@ -1158,13 +1156,37 @@ async function persistFileArtifacts(deliverables, config) {
   }
   return { jsonDeliverables, artifactUploads };
 }
+function validateUploadConfig(config, firstName) {
+  if (!config.taskRunId) {
+    throw new Error(
+      `Artifact '${firstName}' cannot be uploaded: missing task run ID`
+    );
+  }
+  if (!config.authToken) {
+    throw new Error(
+      `Artifact '${firstName}' cannot be uploaded: missing auth token`
+    );
+  }
+  if (!config.baseUrl) {
+    throw new Error(
+      `Artifact '${firstName}' cannot be uploaded: missing backend base URL`
+    );
+  }
+}
 async function uploadOne(name, artifact, config) {
-  const { size, sha256 } = await hashFile(artifact.path);
-  const intent = await createIntent(name, artifact, size, sha256, config);
-  const body = Readable.toWeb(createReadStream(artifact.path));
-  const summary = await putBytes(intent.upload_url, body, size, config);
-  void basename4;
-  return summary;
+  try {
+    const { size, sha256 } = await hashFile(artifact.path);
+    const intent = await createIntent(name, artifact, size, sha256, config);
+    const body = Readable.toWeb(createReadStream(artifact.path));
+    const summary = await putBytes(intent.upload_url, body, size, config);
+    return summary;
+  } catch (err) {
+    if (err instanceof Error && err.code) {
+      const code = err.code ?? "UNKNOWN";
+      throw new Error(`Artifact '${name}' file read failed: ${code}`);
+    }
+    throw err;
+  }
 }
 async function hashFile(path) {
   return new Promise((resolve4, reject) => {