@pushpalsdev/cli 1.0.29 → 1.0.30

package/dist/pushpals-cli.js

@@ -3953,8 +3953,45 @@ function formatSessionEventLine(event) {
   }
   return null;
 }
+function buildSessionEventReplayFingerprint(event) {
+  const type = String(event.type ?? "").trim().toLowerCase();
+  if (type !== "status")
+    return null;
+  const payload = event.payload ?? {};
+  const source = String(event.from ?? payload.agentId ?? "status").trim().toLowerCase();
+  const state = String(payload.state ?? "").trim().toLowerCase();
+  const detail = String(payload.detail ?? "").trim().toLowerCase();
+  const message = String(payload.message ?? "").trim().toLowerCase();
+  return {
+    source,
+    fingerprint: `${type}:${source}:${state}:${detail}:${message}`
+  };
+}
+function createSessionEventReplayFilter() {
+  const seenEventIds = new Set;
+  const lastStatusFingerprintBySource = new Map;
+  return {
+    shouldRender(event) {
+      const eventId = String(event.id ?? "").trim();
+      if (eventId) {
+        if (seenEventIds.has(eventId))
+          return false;
+        seenEventIds.add(eventId);
+      }
+      const replayStatus = buildSessionEventReplayFingerprint(event);
+      if (!replayStatus)
+        return true;
+      const previous = lastStatusFingerprintBySource.get(replayStatus.source);
+      if (previous === replayStatus.fingerprint)
+        return false;
+      lastStatusFingerprintBySource.set(replayStatus.source, replayStatus.fingerprint);
+      return true;
+    }
+  };
+}
 async function runSessionStream(serverUrl, sessionId, client, print, signal) {
   let cursor = 0;
+  const replayFilter = createSessionEventReplayFilter();
   while (!signal.aborted) {
     try {
       const response = await fetchWithTimeout(`${serverUrl}/sessions/${encodeURIComponent(sessionId)}/events${buildClientTransportQuery(cursor, client)}`, {}, 15000);
@@ -4004,6 +4041,8 @@ async function runSessionStream(serverUrl, sessionId, client, print, signal) {
           cursor = Math.max(cursor, blockCursor, serverCursor);
           if (!parsed.envelope)
             continue;
+          if (!replayFilter.shouldRender(parsed.envelope))
+            continue;
           const line = formatSessionEventLine(parsed.envelope);
           if (line)
             print(line);
@@ -4506,6 +4545,7 @@ export {
   ensureWorkerpalDockerImageReady,
   ensureRuntimeBinaries,
   downloadRuntimeAssetsFromSourceTag,
+  createSessionEventReplayFilter,
   copyTrackedRepoPath,
   cleanupLingeringWorkerpalWarmContainers,
   cleanupLingeringPushPalsGitWorktrees,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushpalsdev/cli",
-  "version": "1.0.29",
+  "version": "1.0.30",
   "description": "PushPals terminal CLI for LocalBuddy -> RemoteBuddy orchestration",
   "license": "MIT",
   "repository": {

package/runtime/sandbox/apps/workerpals/src/execute_job.ts

@@ -130,14 +130,51 @@ export function redactSensitiveText(value: string): string {
 }
 
 export function buildCriticRevisionIssues(
-  critic: { score: number; mustFix: string[] } | null | undefined,
+  critic:
+    | {
+        score: number;
+        findings?: string[];
+        mustFix?: string[];
+        revisionGuidance?: string;
+      }
+    | null
+    | undefined,
   qualityCriticMinScore: number,
 ): string[] {
   if (!critic) return [];
   if (critic.score >= qualityCriticMinScore) return [];
-  return [
+  const issues = [
     `Critic score ${critic.score.toFixed(1)} is below required threshold ${qualityCriticMinScore}.`,
   ];
+  const mustFix = Array.isArray(critic.mustFix) ? critic.mustFix : [];
+  const findings = Array.isArray(critic.findings) ? critic.findings : [];
+  const revisionGuidance = String(critic.revisionGuidance ?? "").trim();
+  const actionableItems = (mustFix.length > 0 ? mustFix : findings)
+    .map((entry) => toSingleLine(entry, 180))
+    .filter(Boolean)
+    .slice(0, 3);
+  for (const item of actionableItems) {
+    issues.push(mustFix.length > 0 ? `Critic must-fix: ${item}` : `Critic finding: ${item}`);
+  }
+  if (revisionGuidance) {
+    issues.push(`Critic revision guidance: ${toSingleLine(revisionGuidance, 220)}`);
+  }
+  return issues;
+}
+
+export function buildQualityGateRevisionIssues(
+  qualityIssues: string[],
+  critic: CriticReview | null,
+  qualityCriticMinScore: number,
+): string[] {
+  const normalizedQualityIssues = qualityIssues
+    .map((entry) => String(entry ?? "").trim())
+    .filter(Boolean);
+  if (!critic || critic.score >= qualityCriticMinScore) {
+    return [...normalizedQualityIssues];
+  }
+  const merged = [...normalizedQualityIssues, ...buildCriticRevisionIssues(critic, qualityCriticMinScore)];
+  return [...new Set(merged)];
 }
 
 export function resolveReviewFixCompletionBranch(
@@ -3006,9 +3043,10 @@ export async function executeJob(
       : executor === "openai_codex"
         ? await runCodexCriticReview(repo, attemptParams, quality, runtimeConfig, onLog)
         : await runTaskCriticReview(repo, attemptParams, quality, runtimeConfig, onLog);
+    const deterministicRequiresRevision = !quality.ok;
     const criticRequiresRevision = Boolean(critic && critic.score < qualityCriticMinScore);
 
-    if (!criticRequiresRevision) {
+    if (!deterministicRequiresRevision && !criticRequiresRevision) {
       if (critic) {
         onLog?.(
           "stdout",
@@ -3018,10 +3056,11 @@ export async function executeJob(
       return result;
     }
 
-    const issues: string[] = [];
-    if (criticRequiresRevision && critic) {
-      issues.push(...buildCriticRevisionIssues(critic, qualityCriticMinScore));
-    }
+    const issues = buildQualityGateRevisionIssues(
+      quality.issues,
+      critic,
+      qualityCriticMinScore,
+    );
     const issueSummary = issues.map((entry) => toSingleLine(entry, 180)).join(" | ");
     if (revisionAttempt >= qualityMaxAutoRevisions) {
       if (qualitySoftPassOnExhausted) {