@galda/cli 0.10.106 → 0.10.108

package/engine/lib.mjs

@@ -1896,6 +1896,51 @@ export function canStartMore(runningCount, limit) {
   return Number.isFinite(limit) && limit > 0 && (runningCount ?? 0) < limit;
 }
 
+// A goal sitting in `stacked` is not being worked on and did not fail — it is
+// waiting. Say why, and say what has to happen for it to start, because "nothing
+// is happening" is indistinguishable from "it is broken" from the outside.
+//
+// Keyed by GOAL id, not task id: a stacked goal has no task yet. That is the whole
+// point of the slot accounting (#541) — a goal holds its slot for as long as it is
+// in Doing, so the ones beyond the cap stay `stacked` instead of quietly planning
+// and making the board claim four things are being worked on when two are.
+//
+// The implementation of this shipped once (goal 058a11c, 2026-07-26) and was lost
+// when PR #396 integrated the stability roadmap: the test came across, the server
+// and app halves did not. main has carried a red test for it ever since.
+export function buildQueueWaits({ goals = [], projectPaused = () => false, projectCap = 2, globalCap = 4 } = {}) {
+  const waits = {};
+  const isActive = (g) => g.status === 'planning' || g.status === 'running';
+  const activeEverywhere = goals.filter(isActive);
+  for (const goal of goals) {
+    if (goal?.status !== 'stacked') continue;
+    const active = activeEverywhere.filter((g) => g.projectId === goal.projectId);
+    if (projectPaused(goal.projectId)) {
+      waits[goal.id] = {
+        reason: 'this project is paused',
+        condition: 'you resume the project',
+        targetGoalIds: [],
+      };
+    } else if (active.length < projectCap && activeEverywhere.length >= globalCap) {
+      // This project has room; the machine does not. Naming the other projects'
+      // goals here would point at work the person may not even be looking at, so
+      // the target stays the global limit itself.
+      waits[goal.id] = {
+        reason: `all ${globalCap} slots across every project are already Doing`,
+        condition: 'one of them finishes, fails, or moves to Review',
+        targetGoalIds: activeEverywhere.map((g) => g.id),
+      };
+    } else {
+      waits[goal.id] = {
+        reason: `all ${projectCap} project slots are already Doing`,
+        condition: 'one of them finishes, fails, or moves to Review',
+        targetGoalIds: active.map((g) => g.id),
+      };
+    }
+  }
+  return waits;
+}
+
 // Pick up to `limit` tasks from the front of `waiting` that are safe to start
 // RIGHT NOW, preserving same-goal ordering: a goal already represented in
 // `runningGoalIds` (has an in-flight task) contributes nothing this round,
@@ -3609,6 +3654,44 @@ export function pickChromePath(env = {}, exists = () => false) {
   return chromeCandidates(env).find((c) => exists(c)) ?? null;
 }
 
+// What THIS machine can and cannot do, stated before the person spends an hour
+// reading the absence as a product bug. Two capabilities are host-dependent and
+// silently degrade today: without Chrome nothing can take a screenshot or verify
+// in a browser, and without a native picker "Open folder…" cannot open Finder.
+//
+// Every check here is ADVISORY. None of them stop anything — the app works with
+// neither. That is deliberately NOT testEnvironmentPreflight's blocking/canRun
+// model (which answers "can `npm test` even start"): mixing the two is what made
+// a missing puppeteer read as a failed verification.
+// (`existsFn`/`env` are injected so this stays a pure function under test.)
+export function hostReadiness({ platform = 'darwin', env = {}, existsFn = () => false } = {}) {
+  const chrome = pickChromePath(env, existsFn);
+  const onPath = (cmd) => isCommandOnPath(cmd, {
+    pathEnv: env.PATH ?? '', sep: platform === 'win32' ? ';' : ':', pathExt: env.PATHEXT ?? '', existsFn,
+  });
+  // macOS and Windows ship theirs; Linux has two common ones and neither is a given.
+  const picker = platform === 'darwin' ? 'osascript'
+    : platform === 'win32' ? 'powershell.exe'
+      : (onPath('zenity') ? 'zenity' : 'kdialog');
+  const hasPicker = onPath(picker);
+  return [
+    {
+      id: 'chrome',
+      ok: Boolean(chrome),
+      detail: chrome
+        ? `Chrome/Chromium: ${chrome}`
+        : 'No Chrome/Chromium found — screenshots and browser checks are unavailable',
+    },
+    {
+      id: 'folder-picker',
+      ok: hasPicker,
+      detail: hasPicker
+        ? `Native folder picker: ${picker}`
+        : `No native folder picker (${picker} is not installed) — Open folder… asks you to type the path`,
+    },
+  ];
+}
+
 // `node <manager>/engine/shot.mjs <url|file> <out.png> [--width N] [--height N]
 //  [--full] [--wait MS] [--wait-for SELECTOR] [--element SELECTOR] [--clip x,y,w,h]`
 // Returns { error } instead of throwing, so the CLI can print one clear line.

package/engine/server.mjs

@@ -20,8 +20,8 @@ import { resolve, dirname, join, basename } from 'node:path';
 import { homedir } from 'node:os';
 import { fileURLToPath } from 'node:url';
 import { pathToFileURL } from 'node:url';
-import { needsProjectFolder, folderLabel, chooseFolderScript, parseChosenFolder, buildFolderQuestion, resolveFolderAnswer, classifyFolderTarget, buildFolderInitConfirm, isFolderInitConfirmed, needsReviewPreference, buildReviewPreferenceQuestion, resolveReviewPreferenceAnswer, routeQuestionAnswer, notUnderstoodNote, parseRelocalizedQuestion, classifyAuthProbe, authBannerText, makeSigninChallenge, parseClaimResponse, isCommandOnPath, parseRequestedWorkspaceDir } from './lib.mjs';
-import { parseStreamEvents, parseCodexEvents, buildCodexArgs, parsePlan, refinePlanTasks, canEditGoal, canDeleteGoal, shouldAskClarification, workerExitReason, classifyWorkerFailure, isForcedStop, taskStatusAfterVerify, workerResultText, taskCountsAsComplete, resolveWantsPR, resolveGoalSource, buildGoalSummary, buildReviewAttemptLedger, latestGoalOutcomeTask, buildDirtyWorkspacePrBlock, buildGoalProofMd, buildGoalPrBody, buildReviewSummaryPrompt, parseReviewSummary, buildReviewRuleSection, nextGoalStatus, isPrMerged, isPrApproved, reviewToDoneStatus, advanceReviewGoal, revertReviewGoal, approveGoal, dismissGoal, rejectGoal, reopenGoal, permissionModeFor, isPlanReview, nextAfterPlanApprove, GOAL_MODES, parseSkillFrontmatter, collectSkills, retestGoal, cancelRetestGoal, computeRetestOutcome, revertGoal, planRevertActions, archiveGoal, unarchiveGoal, undismissGoal, parseNumstat, truncateDiffText, sumUsage, resolveEntryUrl, rebasePreviewUrl, shouldCreateGoalPR, normalizePullRequestUrl, parseGitLog, diffNewCommits, replayQueueLog, reconcileOrphanGoals, reorderQueue, sortQueueByPriority, TASK_PRIORITIES, validateWorkflowColumns, DEFAULT_WORKFLOW_COLUMNS, validateReviewDefinition, DEFAULT_REVIEW_DEFINITION, resolveTestGate, buildEphemeralSeedLog, buildEphemeralSeedProjects, trimTaskActivityForState, buildAskContext, WORKER_TOOLS, workerPrompt, verifyCfAccessJwt, resolveIdentity, goalVisibleTo, clampParallelLimit, canStartMore, nextRunnableTasks, goalsConflict, detectConflicts, pickFoldTarget, autoFoldReviewsEnabled, parseGitUnifiedDiffLocations, hasTestRelevantChanges, parseFailingTestNames, classifyTestGate, classifyPrSafety, isInconclusiveTestRun, countInfraFlakes, classifyRunFailures, classifyVerifyGate, buildVerificationInfraError, verificationInfraErrorFrom, buildExecutionPlan, buildContextHandoffSummary, buildFailurePostmortem, appendFailureMemory, latestFailurePolicy, classifyChangeRisk, checkRunBudget, usageBudgetTokens, isRateLimited, nextResumeDelay, checkFreeTierLimit, resolveEntitlement, resolveCachedEntitlement, FREE_TIER_LIMITS, QUEUED_GOAL_STATUSES, verifyLicenseToken, licenseTokenPayload, shouldRefreshLicense, shouldEmitSetupCompleted, pickAnalyticsUid, shouldEmitFreeExhausted, detectRequestLanguage, testFailureReason, manualTestRetryPrompt, isNothingVerifiable, isNothingVerifiableForPR, isSuspiciouslyIncompleteDone, classifyComposerIntentHeuristic, detectPauseIntent, buildIntentPrompt, parseIntentResponse, buildBoardSnapshot, validateBoardSnapshot, boardIsEmpty, decideBoardPull, resolveConnectedAgents, pickAvailableAgent, pickUtilityAgent, utilityModel, liveTakeoverDecision, shouldOpenBrowserOnBoot, goalTaskTitle, buildGoalTask, shouldUsePlannerForGoal, RUN_DECL_FILE, REQUIREMENT_EVIDENCE_FILE, parseRequirementEvidenceDocument, parseRunDeclaration, SHOT_DIR, collectShotNames, parseGoalAddress, REPLY_OUTCOMES, buildGoalMessage, serializeGoalMessage, parseGoalMessages, computeInternalQualityMetrics, classifyInternalQualityTaskError, resolveWorkspaceMode, goalSessionFor, setGoalAgentSession, switchGoalAgentSession, parseApprovalRequest } from './lib.mjs';
+import { needsProjectFolder, folderLabel, chooseFolderScript, parseChosenFolder, buildFolderQuestion, resolveFolderAnswer, classifyFolderTarget, buildFolderInitConfirm, isFolderInitConfirmed, needsReviewPreference, buildReviewPreferenceQuestion, resolveReviewPreferenceAnswer, routeQuestionAnswer, notUnderstoodNote, parseRelocalizedQuestion, classifyAuthProbe, authBannerText, makeSigninChallenge, parseClaimResponse, isCommandOnPath, hostReadiness, parseRequestedWorkspaceDir } from './lib.mjs';
+import { parseStreamEvents, parseCodexEvents, buildCodexArgs, parsePlan, refinePlanTasks, canEditGoal, canDeleteGoal, shouldAskClarification, workerExitReason, classifyWorkerFailure, isForcedStop, taskStatusAfterVerify, workerResultText, taskCountsAsComplete, resolveWantsPR, resolveGoalSource, buildGoalSummary, buildReviewAttemptLedger, latestGoalOutcomeTask, buildDirtyWorkspacePrBlock, buildGoalProofMd, buildGoalPrBody, buildReviewSummaryPrompt, parseReviewSummary, buildReviewRuleSection, nextGoalStatus, isPrMerged, isPrApproved, reviewToDoneStatus, advanceReviewGoal, revertReviewGoal, approveGoal, dismissGoal, rejectGoal, reopenGoal, permissionModeFor, isPlanReview, nextAfterPlanApprove, GOAL_MODES, parseSkillFrontmatter, collectSkills, retestGoal, cancelRetestGoal, computeRetestOutcome, revertGoal, planRevertActions, archiveGoal, unarchiveGoal, undismissGoal, parseNumstat, truncateDiffText, sumUsage, resolveEntryUrl, rebasePreviewUrl, shouldCreateGoalPR, normalizePullRequestUrl, parseGitLog, diffNewCommits, replayQueueLog, reconcileOrphanGoals, reorderQueue, sortQueueByPriority, TASK_PRIORITIES, validateWorkflowColumns, DEFAULT_WORKFLOW_COLUMNS, validateReviewDefinition, DEFAULT_REVIEW_DEFINITION, resolveTestGate, buildEphemeralSeedLog, buildEphemeralSeedProjects, trimTaskActivityForState, buildAskContext, WORKER_TOOLS, workerPrompt, verifyCfAccessJwt, resolveIdentity, goalVisibleTo, clampParallelLimit, canStartMore, buildQueueWaits, nextRunnableTasks, goalsConflict, detectConflicts, pickFoldTarget, autoFoldReviewsEnabled, parseGitUnifiedDiffLocations, hasTestRelevantChanges, parseFailingTestNames, classifyTestGate, classifyPrSafety, isInconclusiveTestRun, countInfraFlakes, classifyRunFailures, classifyVerifyGate, buildVerificationInfraError, verificationInfraErrorFrom, buildExecutionPlan, buildContextHandoffSummary, buildFailurePostmortem, appendFailureMemory, latestFailurePolicy, classifyChangeRisk, checkRunBudget, usageBudgetTokens, isRateLimited, nextResumeDelay, checkFreeTierLimit, resolveEntitlement, resolveCachedEntitlement, FREE_TIER_LIMITS, QUEUED_GOAL_STATUSES, verifyLicenseToken, licenseTokenPayload, shouldRefreshLicense, shouldEmitSetupCompleted, pickAnalyticsUid, shouldEmitFreeExhausted, detectRequestLanguage, testFailureReason, manualTestRetryPrompt, isNothingVerifiable, isNothingVerifiableForPR, isSuspiciouslyIncompleteDone, classifyComposerIntentHeuristic, detectPauseIntent, buildIntentPrompt, parseIntentResponse, buildBoardSnapshot, validateBoardSnapshot, boardIsEmpty, decideBoardPull, resolveConnectedAgents, pickAvailableAgent, pickUtilityAgent, utilityModel, liveTakeoverDecision, shouldOpenBrowserOnBoot, goalTaskTitle, buildGoalTask, shouldUsePlannerForGoal, RUN_DECL_FILE, REQUIREMENT_EVIDENCE_FILE, parseRequirementEvidenceDocument, parseRunDeclaration, SHOT_DIR, collectShotNames, parseGoalAddress, REPLY_OUTCOMES, buildGoalMessage, serializeGoalMessage, parseGoalMessages, computeInternalQualityMetrics, classifyInternalQualityTaskError, resolveWorkspaceMode, goalSessionFor, setGoalAgentSession, switchGoalAgentSession, parseApprovalRequest } from './lib.mjs';
 import { migrateRequirementModel, validateRequirementModel } from './requirement-model.mjs';
 import { buildManagerVerificationChecks, buildRequirementVerificationEvidence, mergeRequirementVerificationEvidence } from './requirement-verification.mjs';
 import { createSerialQueue } from './lib.mjs';
@@ -3335,6 +3335,14 @@ function testNodePath(dir, dependencySourceDir = null) {
     .filter(Boolean).filter(existsSync);
   return [...new Set([...local, ...(process.env.NODE_PATH ? process.env.NODE_PATH.split(sep) : [])])].join(sep);
 }
+// Advisory host capabilities for /api/state (see hostReadiness in lib.mjs). This
+// touches the filesystem, and /api/state is polled every couple of seconds, so
+// answer once: installing Chrome mid-session is not a thing to poll for, and a
+// restart is already how this app picks up a new environment.
+let _hostReadiness = null;
+function hostReadinessNow() {
+  return (_hostReadiness ??= hostReadiness({ platform: process.platform, env: process.env, existsFn: existsSync }));
+}
 function testEnvironmentPreflight(dir, dependencySourceDir = null) {
   const nodePath = testNodePath(dir, dependencySourceDir);
   const sources = nodePath ? nodePath.split(process.platform === 'win32' ? ';' : ':') : [];
@@ -4367,7 +4375,7 @@ const server = createServer(async (req, res) => {
       const dirty = projectDirtyInfo(p.dir);
       return { ...p, ...(branch ? { branch } : {}), ...dirty };
     });
-    return json(res, 200, { projects: projectsWithBranch, goals: visibleGoals, tasks: trimTaskActivityForState(visibleTasks), models: MODELS, agentModels: { 'claude-code': MODELS, codex: CODEX_MODELS }, agentEfforts: { 'claude-code': CLAUDE_EFFORTS, codex: CODEX_EFFORTS }, agents: AVAILABLE_AGENTS, connectCommand, queueOrder, externalActivity, workflowColumns: projectColumns, reviewDefinitions: projectReviewDefs, runningCounts, parallelLimits, uiVersion: uiVersion(), billing, auth: publicAuth() });
+    return json(res, 200, { projects: projectsWithBranch, goals: visibleGoals, tasks: trimTaskActivityForState(visibleTasks), models: MODELS, agentModels: { 'claude-code': MODELS, codex: CODEX_MODELS }, agentEfforts: { 'claude-code': CLAUDE_EFFORTS, codex: CODEX_EFFORTS }, agents: AVAILABLE_AGENTS, connectCommand, queueOrder, externalActivity, workflowColumns: projectColumns, reviewDefinitions: projectReviewDefs, runningCounts, parallelLimits, uiVersion: uiVersion(), billing, hostReadiness: hostReadinessNow(), queueWaits: buildQueueWaits({ goals: visibleGoals, projectPaused: isProjectPaused, projectCap: projectParallelCap(), globalCap: GLOBAL_MAX_PARALLEL }), auth: publicAuth() });
   }
 
   // Read-only stability snapshot: separates implementation failures from

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@galda/cli",
-  "version": "0.10.106",
+  "version": "0.10.108",
   "type": "module",
   "description": "Galda - hand off work to Claude Code or Codex, get proof back. Runs on your existing subscription, no extra API cost.",
   "scripts": {