@hackerrank/astra-cli 0.1.9 → 0.1.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hackerrank/astra-cli",
-  "version": "0.1.9",
+  "version": "0.1.11",
   "description": "Minimal zero-dependency AI coding agent for the HackerRank AI Gateway.",
   "type": "module",
   "bin": {

package/src/report.js

@@ -20,7 +20,11 @@ export function refreshReport(root) {
   if (!fs.existsSync(rootDir)) {
     throw new Error(`bench root not found: ${rootDir}`);
   }
-  const runs = scanRuns(rootDir);
+  // Task versions are immutable benchmark definitions. A task output directory
+  // may retain old attempts for auditability, but combining versions changes
+  // the measured task and makes the dashboard misleading. Show the newest
+  // version for each task while preserving unversioned legacy runs.
+  const runs = latestTaskRuns(scanRuns(rootDir));
   const summary = buildSummary(runs);
 
   for (const run of runs) {
@@ -82,6 +86,21 @@ export function scanRuns(rootDir) {
   return runs;
 }
 
+function latestTaskRuns(runs) {
+  const latestVersionByTask = new Map();
+  for (const run of runs) {
+    const taskId = run.project?.id;
+    const version = Number(run.project?.version);
+    if (!taskId || !Number.isFinite(version)) continue;
+    latestVersionByTask.set(taskId, Math.max(latestVersionByTask.get(taskId) ?? version, version));
+  }
+  return runs.filter((run) => {
+    const taskId = run.project?.id;
+    const version = Number(run.project?.version);
+    return !taskId || !Number.isFinite(version) || latestVersionByTask.get(taskId) === version;
+  });
+}
+
 export function buildSummary(runs) {
   runs = runs.map((run) => ({ ...run, criteria: flattenCriteria(run.criteria) }));
   const verifierCriteria = summarizeCriteria(runs);

package/src/verifier-runner.js

@@ -53,6 +53,18 @@ async function acquireVerifierLock(projectRoot, timeoutSeconds) {
       };
     } catch (error) {
       if (error?.code !== "EEXIST") throw error;
+      // A killed benchmark worker cannot run its finally block. Reclaim only
+      // a lock whose recorded owner is no longer alive; active workers keep
+      // exclusive ownership of fixed Docker ports.
+      try {
+        const owner = Number(fs.readFileSync(lockPath, "utf8").trim());
+        process.kill(owner, 0);
+      } catch (ownerError) {
+        if (ownerError?.code === "ESRCH" || !Number.isInteger(Number(fs.readFileSync(lockPath, "utf8").trim()))) {
+          try { fs.rmSync(lockPath, { force: true }); } catch {}
+          continue;
+        }
+      }
       await new Promise((resolve) => setTimeout(resolve, 100));
     }
   }