@galda/cli 0.10.106 → 0.10.107

package/engine/lib.mjs

@@ -3609,6 +3609,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,7 +20,7 @@ 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 { 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, 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';
@@ -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(), 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.107",
   "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": {