humanish 0.93.1 → 0.95.0

package/dist/run.d.ts

@@ -6,6 +6,7 @@ import { type CapturedGitState } from "./core/git-state.js";
 import type { E2BDesktopModule } from "./e2b-desktop-launch.js";
 import { type PreparedRunArtifactPaths } from "./run-paths.js";
 import { type RunLabProvenance } from "./run-status.js";
+import { type DetectLocalAgentsOptions } from "./local-agent-cli.js";
 export declare const RUN_BUNDLE_SCHEMA = "humanish.run-bundle.v1";
 export declare const SHARED_WORLD_SCHEMA = "humanish.shared-world.v1";
 export declare const REVIEW_SCHEMA = "humanish.review.v1";
@@ -1226,7 +1227,11 @@ export declare function readReview(cwdInput: string, runInput: string): Promise<
 export declare const DESKTOP_SDK_FLOOR = "2.3.1";
 /** The advisory `doctor` attaches to an installed desktop SDK older than the floor, else undefined. */
 export declare function desktopSdkAdvisory(version: string | undefined): string | undefined;
-export declare function doctor(cwdInput: string): Promise<DoctorResult>;
+export declare function doctor(cwdInput: string, options?: {
+    lab?: string;
+    env?: NodeJS.ProcessEnv;
+    localAgents?: DetectLocalAgentsOptions;
+}): Promise<DoctorResult>;
 /** Resolve "latest" or an explicit run id to its prepared artifact paths. Exported for the
  *  reclaim command (#358), which must locate a run WITHOUT trusting anything but the managed dir. */
 export declare function resolveRunPath(cwd: string, runInput: string): Promise<PreparedRunArtifactPaths | null>;

package/dist/run.js

@@ -30,6 +30,7 @@ import { probeKeySources } from "./key-resolution.js";
 import { beginRunStatus, withRunStatusScope } from "./run-status.js";
 import { nodeSupportsTui, terminalSurfaceMessage, tuiBundleUrl } from "./tui-contract.js";
 import { detectLocalAgents, localAgentDoctorMessage } from "./local-agent-cli.js";
+import { labSetupChecks } from "./doctor-lab.js";
 import { assertPreparedSelectedOutputDirectory, assertSafeOutputPathSegment, bindExistingManagedHumanishOutputDirectory, prepareContainedOutputDirectory, prepareContainedOutputDirectoryRoot, prepareContainedOutputFile, prepareSelectedOutputDirectory, readContainedRegularFile, writeContainedOutputFile, writePreparedRunLatestPointer } from "./selected-output-paths.js";
 export const RUN_BUNDLE_SCHEMA = "humanish.run-bundle.v1";
 export const SHARED_WORLD_SCHEMA = "humanish.shared-world.v1";
@@ -3122,7 +3123,7 @@ async function installedDesktopSdkVersion() {
         return undefined;
     }
 }
-export async function doctor(cwdInput) {
+export async function doctor(cwdInput, options = {}) {
     const cwd = path.resolve(cwdInput);
     const cwdOk = await validateCwd(cwd).then((error) => error === null).catch(() => false);
     if (!cwdOk) {
@@ -3155,17 +3156,32 @@ export async function doctor(cwdInput) {
             return false;
         }
     };
+    const env = options.env ?? process.env;
+    const agents = await detectLocalAgents({ ...options.localAgents, env });
+    const probes = await probeKeySources(["OPENAI_API_KEY", "E2B_API_KEY", "GH_TOKEN", "CODEX_API_KEY"], { cwd, env });
+    const setup = options.lab ? await labSetupChecks({ cwd, lab: options.lab, env, agents,
+        keyPresent: name => probes.some(probe => probe.name === name && probe.source !== null) }) : undefined;
     const checks = [
         {
             name: "target cwd",
             ok: true,
             message: "target directory exists"
         },
-        {
-            name: "package.json",
-            ok: await safeCheck(async () => await readImplicitProjectFile(projectRoot, "package.json") !== null),
-            message: "package.json is present and safe to read"
-        },
+        await (async () => {
+            try {
+                const contents = await readImplicitProjectFile(projectRoot, "package.json");
+                return {
+                    name: "package.json",
+                    ok: true,
+                    message: contents === null
+                        ? "package.json is absent; it is optional for Humanish, so npm-script integration is skipped"
+                        : "package.json is present and safe to read"
+                };
+            }
+            catch {
+                return { name: "package.json", ok: false, message: "package.json could not be safely read" };
+            }
+        })(),
         {
             name: "humanish source",
             ok: await safeCheck(() => implicitProjectDirectoryExists(projectRoot, "humanish")),
@@ -3194,10 +3210,11 @@ export async function doctor(cwdInput) {
             const advisory = desktopSdkAdvisory(version);
             return {
                 name: "e2b desktop sdk",
-                ok: present,
+                ok: present || setup?.desktop === false,
                 message: present
-                    ? `optional peer @e2b/desktop ${version ?? "(version unread)"} is installed; live desktop lanes can launch${advisory === undefined ? "" : `. ${advisory}`}`
-                    : "optional peer @e2b/desktop is NOT installed — dry runs work, but any live desktop lane will fail closed. Install it with `npm i -D @e2b/desktop`."
+                    ? `optional peer @e2b/desktop ${version ?? "(version unread)"} is installed; provider access is not tested${advisory === undefined ? "" : `. ${advisory}`}`
+                    : setup?.desktop === false ? "optional peer @e2b/desktop is absent; not required by the selected route"
+                        : "optional peer @e2b/desktop is NOT installed — dry runs work, but any live desktop lane will fail closed. Install it with `npm i -D @e2b/desktop`."
             };
         })(),
         // The stakeholder surface (#455). Reported as capability, never as a gate: the TUI is optional,
@@ -3230,32 +3247,29 @@ export async function doctor(cwdInput) {
         // "go make an API key" is where most people trying humanish stop, and a developer very often
         // already has one of these signed in.
         ...await (async () => {
-            const found = await detectLocalAgents();
-            return [{ name: "local agents", ok: true, message: localAgentDoctorMessage(found) }];
+            return [{ name: "local agents", ok: true, message: localAgentDoctorMessage(agents) }];
         })(),
         // Provider-key discovery (#436): which source supplies each live-run key, through the same
         // chain a live command resolves (env/--env-file, project overlay, vendor stores, the
         // humanish user store). Values never appear; sources and fill commands do.
         ...await (async () => {
-            const probes = await probeKeySources(["OPENAI_API_KEY", "E2B_API_KEY", "GH_TOKEN"], {
-                cwd,
-                env: process.env
-            });
             return probes.map((probe) => {
                 const present = probe.source !== null;
                 // GH_TOKEN is needed only for private clone subjects, so its absence is informational.
-                const required = probe.name !== "GH_TOKEN";
+                const required = setup ? setup.keys.includes(probe.name) : probe.name === "E2B_API_KEY"
+                    || probe.name === "OPENAI_API_KEY" && !agents.some(agent => agent.authStatus === "authenticated");
                 return {
                     name: `key ${probe.name}`,
                     ok: present || !required,
                     message: present
-                        ? `supplied by ${probe.source}`
+                        ? `supplied by ${probe.source}; presence only, validity not tested`
                         : required
                             ? `missing from every source — ${probe.hint}`
-                            : `missing (needed only for private clone subjects) — ${probe.hint}`
+                            : `not required for ${setup ? "the selected participant route" : "every route"}; ${probe.hint}`
                 };
             });
-        })()
+        })(),
+        ...(setup?.checks ?? [{ name: "setup route", ok: true, message: "General capabilities only. Use humanish doctor --lab <lab> for the selected participant's requirements and separate analysis readiness." }])
     ];
     return {
         schema: DOCTOR_SCHEMA,