pi-crew 0.1.44 → 0.1.46

package/skills/verification-before-done/SKILL.md

@@ -0,0 +1,57 @@
+---
+name: verification-before-done
+description: Use when about to claim work is complete, fixed, passing, reviewed, committed, or ready to hand off.
+---
+
+# verification-before-done
+
+Core principle: evidence before claims. A worker report, green-looking log, or previous run is not fresh verification.
+
+Distilled from detailed reads of agent-skill patterns for verification-before-completion, TDD, review reception, and QA workflows.
+
+## Gate Function
+
+Before any completion claim:
+
+1. Identify the command or inspection that proves the claim.
+2. Run the full command fresh, or explicitly state why a command cannot be run.
+3. Read the output, including exit code and failure counts.
+4. Compare the output to the claim.
+5. Report the claim only with the evidence.
+
+## Claim-to-Evidence Table
+
+| Claim | Requires | Not sufficient |
+|---|---|---|
+| Tests pass | Fresh test output with zero failures | Prior run, “should pass” |
+| Typecheck passes | Typecheck command exit 0 | Lint or targeted tests only |
+| Bug fixed | Original symptom/regression test passes | Code changed |
+| Requirements met | Checklist against request/plan | Generic test success |
+| Agent completed | Worker output plus artifact/diff/state inspection | Worker says DONE |
+| Safe to commit | Relevant checks pass and status reviewed | Partial local confidence |
+
+## Verification Ladder
+
+Choose the smallest reliable gate, then escalate when risk requires it:
+
+1. Read-only inspection for plans/reviews.
+2. Targeted unit test for touched behavior.
+3. Typecheck for TypeScript/schema/API changes.
+4. Integration test for runtime, subprocess, state, filesystem, UI, config, or session behavior.
+5. Full suite before commit/release or broad changes.
+6. Real Pi smoke only when safe and needed.
+
+## Done Report
+
+Include:
+
+- changed files or read-only status;
+- commands run and pass/fail result;
+- artifacts, run IDs, logs, or state paths inspected;
+- behavior actually verified;
+- skipped checks and why;
+- risks and rollback notes.
+
+## Red Flags
+
+Stop before saying done if you are using words like “should”, “probably”, “looks”, “seems”, “I think”, or if you are trusting an agent report without checking evidence.

package/skills/worktree-isolation/SKILL.md

@@ -0,0 +1,39 @@
+---
+name: worktree-isolation
+description: Conflict-safe git worktree workflow. Use when running parallel implementation workers, isolating risky edits, or cleaning up task worktrees.
+---
+
+# worktree-isolation
+
+Use this skill for worktree-based execution or cleanup.
+
+## Source patterns distilled
+
+- pi-subagents worktree runner and cleanup patterns
+- pi-crew worktrees: `src/worktree/worktree-manager.ts`, `src/worktree/cleanup.ts`, `src/worktree/branch-freshness.ts`
+- Team runner workspace mode: `src/runtime/team-runner.ts`, workflow/team resource fields
+
+## Rules
+
+- Use worktree mode for parallel or risky code-changing tasks when the repository is clean enough and merge ownership is clear.
+- Assign one owner per file/symbol/migration path to avoid conflict-heavy merges.
+- Name branches/worktrees deterministically from run/task IDs; avoid user-controlled path fragments without sanitization.
+- Before cleanup, check dirty state. Preserve dirty worktrees unless `force` is explicitly set.
+- Record worktree paths and branch metadata in artifacts/events so the operator can inspect or recover.
+- Do not run destructive git operations without explicit confirmation and evidence of target path containment.
+
+## Anti-patterns
+
+- Parallel editing the same file in multiple worktrees without a merge plan.
+- Force-removing dirty worktrees by default.
+- Reusing stale worktrees after the base branch has moved without freshness checks.
+- Storing worktrees outside the intended contained workspace root.
+
+## Verification
+
+```bash
+cd pi-crew
+npx tsc --noEmit
+node --experimental-strip-types --test test/integration/worktree-mode.test.ts test/unit/run-index.test.ts
+npm test
+```

package/src/agents/discover-agents.ts

@@ -1,7 +1,7 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
 import type { AgentConfig, ResourceSource } from "./agent-config.ts";
-import { loadConfig } from "../config/config.ts";
+import { loadConfig, type LoadedPiTeamsConfig } from "../config/config.ts";
 import { parseCsv, parseFrontmatter } from "../utils/frontmatter.ts";
 import { packageRoot, projectCrewRoot, userPiRoot } from "../utils/paths.ts";
 
@@ -36,9 +36,9 @@ function parseAgentFile(filePath: string, source: ResourceSource): AgentConfig |
 			source,
 			filePath,
 			systemPrompt: body.trim(),
-			model: frontmatter.model || undefined,
+			model: frontmatter.model === "false" ? undefined : frontmatter.model || undefined,
 			fallbackModels: parseCsv(frontmatter.fallbackModels),
-			thinking: frontmatter.thinking || undefined,
+			thinking: frontmatter.thinking === "false" ? undefined : frontmatter.thinking || undefined,
 			tools: parseCsv(frontmatter.tools),
 			extensions: frontmatter.extensions === "" ? [] : parseCsv(frontmatter.extensions),
 			skills: parseCsv(frontmatter.skills ?? frontmatter.skill),
@@ -63,12 +63,12 @@ function readAgentDir(dir: string, source: ResourceSource): AgentConfig[] {
 		.sort((a, b) => a.name.localeCompare(b.name));
 }
 
-function applyAgentOverrides(agents: AgentConfig[], cwd: string): AgentConfig[] {
-	const loaded = loadConfig(cwd);
-	const config = loaded.config.agents;
-	const overrides = config?.overrides ?? {};
+function applyAgentOverrides(agents: AgentConfig[], cwd: string, loadedConfig?: LoadedPiTeamsConfig): AgentConfig[] {
+	const loaded = loadedConfig ?? loadConfig(cwd);
+	const agentsConfig = loaded.config.agents;
+	const overrides = agentsConfig?.overrides ?? {};
 	return agents
-		.filter((agent) => !(config?.disableBuiltins && agent.source === "builtin"))
+		.filter((agent) => !(agentsConfig?.disableBuiltins && agent.source === "builtin"))
 		.map((agent) => {
 			const overrideEntry = Object.entries(overrides).find(([name]) => name.toLowerCase() === agent.name.toLowerCase());
 			if (!overrideEntry) return agent;
@@ -87,10 +87,11 @@ function applyAgentOverrides(agents: AgentConfig[], cwd: string): AgentConfig[]
 }
 
 export function discoverAgents(cwd: string): AgentDiscoveryResult {
+	const loaded = loadConfig(cwd);
 	return {
-		builtin: applyAgentOverrides(readAgentDir(path.join(packageRoot(), "agents"), "builtin"), cwd),
-		user: applyAgentOverrides(readAgentDir(path.join(userPiRoot(), "agents"), "user"), cwd),
-		project: applyAgentOverrides(readAgentDir(path.join(projectCrewRoot(cwd), "agents"), "project"), cwd),
+		builtin: applyAgentOverrides(readAgentDir(path.join(packageRoot(), "agents"), "builtin"), cwd, loaded),
+		user: applyAgentOverrides(readAgentDir(path.join(userPiRoot(), "agents"), "user"), cwd, loaded),
+		project: applyAgentOverrides(readAgentDir(path.join(projectCrewRoot(cwd), "agents"), "project"), cwd, loaded),
 	};
 }
 

package/src/config/config.ts

@@ -4,7 +4,7 @@ import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
 import { PiTeamsAutonomyProfileSchema, PiTeamsConfigSchema } from "../schema/config-schema.ts";
-import { projectCrewRoot } from "../utils/paths.ts";
+import { projectCrewRoot, projectPiRoot } from "../utils/paths.ts";
 
 export type PiTeamsAutonomyProfile = "manual" | "suggested" | "assisted" | "aggressive";
 
@@ -31,6 +31,7 @@ export interface CrewLimitsConfig {
 export type CrewRuntimeMode = "auto" | "scaffold" | "child-process" | "live-session";
 
 export type CompletionMutationGuardMode = "off" | "warn" | "fail";
+export type EffectivenessGuardMode = "off" | "warn" | "block" | "fail";
 
 export interface CrewRuntimeConfig {
 	mode?: CrewRuntimeMode;
@@ -44,6 +45,7 @@ export interface CrewRuntimeConfig {
 	groupJoinAckTimeoutMs?: number;
 	requirePlanApproval?: boolean;
 	completionMutationGuard?: CompletionMutationGuardMode;
+	effectivenessGuard?: EffectivenessGuardMode;
 }
 
 export interface CrewControlConfig {
@@ -182,6 +184,11 @@ export interface UpdateConfigOptions {
 }
 
 export function configPath(): string {
+	const home = process.env.PI_TEAMS_HOME?.trim() || os.homedir();
+	return path.join(home, ".pi", "agent", "pi-crew.json");
+}
+
+export function legacyConfigPath(): string {
 	const home = process.env.PI_TEAMS_HOME?.trim() || os.homedir();
 	return path.join(home, ".pi", "agent", "extensions", "pi-crew", "config.json");
 }
@@ -195,7 +202,7 @@ export function projectConfigPath(cwd: string): string {
  * This is a convenience path alongside the standard `config.json` in crewRoot.
  */
 export function projectPiCrewJsonPath(cwd: string): string {
-	return path.join(cwd, ".pi", "pi-crew.json");
+	return path.join(projectPiRoot(cwd), "pi-crew.json");
 }
 
 function withoutUndefined<T extends Record<string, unknown>>(value: T): Partial<T> {
@@ -509,6 +516,7 @@ function parseRuntimeConfig(value: unknown): CrewRuntimeConfig | undefined {
 		groupJoinAckTimeoutMs: parsePositiveInteger(obj.groupJoinAckTimeoutMs, 86_400_000),
 		requirePlanApproval: parseWithSchema(Type.Boolean(), obj.requirePlanApproval),
 		completionMutationGuard: parseWithSchema(Type.Union([Type.Literal("off"), Type.Literal("warn"), Type.Literal("fail")]), obj.completionMutationGuard),
+		effectivenessGuard: parseWithSchema(Type.Union([Type.Literal("off"), Type.Literal("warn"), Type.Literal("block"), Type.Literal("fail")]), obj.effectivenessGuard),
 	};
 	return Object.values(runtime).some((entry) => entry !== undefined) ? runtime : undefined;
 }
@@ -727,35 +735,51 @@ function readConfigRecord(filePath: string): Record<string, unknown> {
 	return raw as Record<string, unknown>;
 }
 
+function readOptionalConfig(filePath: string): { exists: boolean; config: PiTeamsConfig; warnings: string[] } {
+	if (!fs.existsSync(filePath)) return { exists: false, config: {}, warnings: [] };
+	try {
+		const raw = readConfigRecord(filePath);
+		const parsed = parseConfigWithWarnings(raw);
+		return { exists: true, config: parsed.config, warnings: parsed.warnings.map((warning) => `${filePath}: ${warning}`) };
+	} catch (error) {
+		const message = error instanceof Error ? error.message : String(error);
+		return { exists: true, config: {}, warnings: [`${filePath}: invalid config ignored: ${message}`] };
+	}
+}
+
 export function loadConfig(cwd?: string): LoadedPiTeamsConfig {
 	const filePath = configPath();
+	const legacyPath = legacyConfigPath();
 	const paths = cwd ? [filePath, projectConfigPath(cwd)] : [filePath];
-	try {
-		const userRaw = readConfigRecord(filePath);
-		const userConfig = parseConfigWithWarnings(userRaw);
-		let config = userConfig.config;
-		const warnings: string[] = userConfig.warnings.map((warning) => `${filePath}: ${warning}`);
-		if (cwd) {
-			const projectPath = projectConfigPath(cwd);
-			const projectConfig = parseConfigWithWarnings(readConfigRecord(projectPath));
+	const warnings: string[] = [];
+	const legacyConfig = readOptionalConfig(legacyPath);
+	if (legacyConfig.exists && legacyPath !== filePath) {
+		warnings.push(...legacyConfig.warnings);
+		paths.unshift(legacyPath);
+	}
+	const userConfig = readOptionalConfig(filePath);
+	warnings.push(...userConfig.warnings);
+	let config = mergeConfig(legacyConfig.exists && legacyPath !== filePath ? legacyConfig.config : {}, userConfig.config);
+	if (cwd) {
+		const projectPath = projectConfigPath(cwd);
+		const projectConfig = readOptionalConfig(projectPath);
+		if (projectConfig.exists) {
 			const projectSafeConfig = sanitizeProjectConfig(projectPath, config, projectConfig.config);
-			warnings.push(...projectConfig.warnings.map((warning) => `${projectPath}: ${warning}`), ...projectSafeConfig.warnings);
+			warnings.push(...projectConfig.warnings, ...projectSafeConfig.warnings);
 			config = mergeConfig(config, projectSafeConfig.config);
-			// Also load .pi/pi-crew.json from project root if it exists
-			const piCrewJsonPath = projectPiCrewJsonPath(cwd);
-			if (fs.existsSync(piCrewJsonPath)) {
-				const piCrewJsonConfig = parseConfigWithWarnings(readConfigRecord(piCrewJsonPath));
-				const piCrewJsonSafeConfig = sanitizeProjectConfig(piCrewJsonPath, config, piCrewJsonConfig.config);
-				warnings.push(...piCrewJsonConfig.warnings.map((warning) => `${piCrewJsonPath}: ${warning}`), ...piCrewJsonSafeConfig.warnings);
-				config = mergeConfig(config, piCrewJsonSafeConfig.config);
-				paths.push(piCrewJsonPath);
-			}
 		}
-		return { path: filePath, paths, config, warnings: warnings.length > 0 ? warnings : undefined };
-	} catch (error) {
-		const message = error instanceof Error ? error.message : String(error);
-		return { path: filePath, paths, config: {}, error: message };
+		// `.pi/pi-crew.json` is the project-owned override file. If present and valid,
+		// it may override all pi-crew config fields, including agents.overrides.
+		// If missing or invalid, it is ignored and defaults/user config remain effective.
+		const piCrewJsonPath = projectPiCrewJsonPath(cwd);
+		const piCrewJsonConfig = readOptionalConfig(piCrewJsonPath);
+		if (piCrewJsonConfig.exists) {
+			warnings.push(...piCrewJsonConfig.warnings);
+			config = mergeConfig(config, piCrewJsonConfig.config);
+			paths.push(piCrewJsonPath);
+		}
 	}
+	return { path: filePath, paths, config, warnings: warnings.length > 0 ? warnings : undefined };
 }
 
 export function updateConfig(patch: PiTeamsConfig, options: UpdateConfigOptions = {}): SavedPiTeamsConfig {

package/src/config/defaults.ts

@@ -53,6 +53,20 @@ export const DEFAULT_UI = {
 	refreshMs: 1000,
 	notifierIntervalMs: 5000,
 	widgetDefaultFrameMs: 1000,
+	widgetPlacement: "aboveEditor" as const,
+	widgetMaxLines: 8,
+	powerbar: true,
+	dashboardPlacement: "center" as const,
+	dashboardWidth: 72,
+	dashboardLiveRefreshMs: 1000,
+	autoOpenDashboard: false,
+	autoOpenDashboardForForegroundRuns: false,
+	showModel: true,
+	showTokens: true,
+	showTools: true,
+	transcriptTailBytes: 1024 * 1024,
+	mascotStyle: "cat" as const,
+	mascotEffect: "random" as const,
 };
 
 export const DEFAULT_NOTIFICATIONS = {

package/src/extension/project-init.ts

@@ -1,10 +1,13 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
-import { packageRoot, projectCrewRoot } from "../utils/paths.ts";
+import { configPath as globalConfigPath } from "../config/config.ts";
+import { DEFAULT_UI } from "../config/defaults.ts";
+import { packageRoot, projectCrewRoot, projectPiRoot } from "../utils/paths.ts";
 
 export interface ProjectInitOptions {
 	copyBuiltins?: boolean;
 	overwrite?: boolean;
+	configScope?: "global" | "project" | "none";
 }
 
 export interface ProjectInitResult {
@@ -13,6 +16,10 @@ export interface ProjectInitResult {
 	skippedFiles: string[];
 	gitignorePath: string;
 	gitignoreUpdated: boolean;
+	configPath: string;
+	configScope: "global" | "project" | "none";
+	configCreated: boolean;
+	configSkipped: boolean;
 }
 
 function ensureDir(dir: string, createdDirs: string[]): void {
@@ -24,6 +31,44 @@ function ensureDir(dir: string, createdDirs: string[]): void {
 	}
 }
 
+const DEFAULT_PI_CREW_CONFIG = {
+	// Keep generated config non-invasive: do not set runtime/limits defaults here.
+	// Those are provided by pi-crew internals and should not make a normal workflow block.
+	autonomous: {
+		enabled: true,
+		injectPolicy: true,
+		preferAsyncForLongTasks: false,
+		allowWorktreeSuggestion: true,
+	},
+	agents: {
+		overrides: {
+			explorer: { model: false, thinking: "off" },
+			writer: { model: false, thinking: "off" },
+			planner: { model: false, thinking: "medium" },
+			analyst: { model: false, thinking: "off" },
+			critic: { model: false, thinking: "low" },
+			executor: { model: false, thinking: "medium" },
+			reviewer: { model: false, thinking: "off" },
+			"security-reviewer": { model: false, thinking: "medium" },
+			"test-engineer": { model: false, thinking: "low" },
+			verifier: { model: false, thinking: "off" },
+		},
+	},
+	ui: {
+		widgetPlacement: DEFAULT_UI.widgetPlacement,
+		widgetMaxLines: DEFAULT_UI.widgetMaxLines,
+		powerbar: DEFAULT_UI.powerbar,
+		dashboardPlacement: DEFAULT_UI.dashboardPlacement,
+		dashboardWidth: DEFAULT_UI.dashboardWidth,
+		dashboardLiveRefreshMs: DEFAULT_UI.dashboardLiveRefreshMs,
+		autoOpenDashboard: DEFAULT_UI.autoOpenDashboard,
+		autoOpenDashboardForForegroundRuns: DEFAULT_UI.autoOpenDashboardForForegroundRuns,
+		showModel: DEFAULT_UI.showModel,
+		showTokens: DEFAULT_UI.showTokens,
+		showTools: DEFAULT_UI.showTools,
+	},
+};
+
 function copyBuiltinDir(kind: "agents" | "teams" | "workflows", targetDir: string, overwrite: boolean, copiedFiles: string[], skippedFiles: string[]): void {
 	const sourceDir = path.join(packageRoot(), kind);
 	if (!fs.existsSync(sourceDir)) return;
@@ -50,11 +95,26 @@ export function initializeProject(cwd: string, options: ProjectInitOptions = {})
 	const agentsDir = path.join(crewRoot, "agents");
 	const teamsDir = path.join(crewRoot, "teams");
 	const workflowsDir = path.join(crewRoot, "workflows");
+	const configScope = options.configScope ?? "global";
+	const configPath = configScope === "project" ? path.join(projectPiRoot(cwd), "pi-crew.json") : configScope === "global" ? globalConfigPath() : "";
 	ensureDir(agentsDir, createdDirs);
 	ensureDir(teamsDir, createdDirs);
 	ensureDir(workflowsDir, createdDirs);
 	ensureDir(path.join(crewRoot, "imports"), createdDirs);
 
+	let configCreated = false;
+	let configSkipped = false;
+	if (configPath) {
+		if (configScope === "project") ensureDir(path.dirname(configPath), createdDirs);
+		else fs.mkdirSync(path.dirname(configPath), { recursive: true });
+		if (!fs.existsSync(configPath) || options.overwrite === true) {
+			fs.writeFileSync(configPath, `${JSON.stringify(DEFAULT_PI_CREW_CONFIG, null, 2)}\n`, "utf-8");
+			configCreated = true;
+		} else {
+			configSkipped = true;
+		}
+	}
+
 	if (options.copyBuiltins) {
 		copyBuiltinDir("agents", agentsDir, options.overwrite === true, copiedFiles, skippedFiles);
 		copyBuiltinDir("teams", teamsDir, options.overwrite === true, copiedFiles, skippedFiles);
@@ -72,5 +132,5 @@ export function initializeProject(cwd: string, options: ProjectInitOptions = {})
 		gitignoreUpdated = true;
 	}
 
-	return { createdDirs, copiedFiles, skippedFiles, gitignorePath, gitignoreUpdated };
+	return { createdDirs, copiedFiles, skippedFiles, gitignorePath, gitignoreUpdated, configPath, configScope, configCreated, configSkipped };
 }

package/src/extension/register.ts

@@ -1,6 +1,7 @@
 import type { ExtensionAPI, ExtensionContext } from "@mariozechner/pi-coding-agent";
 import * as fs from "node:fs";
 import * as path from "node:path";
+import { fileURLToPath } from "node:url";
 import { loadConfig } from "../config/config.ts";
 import { registerAutonomousPolicy } from "./autonomous-policy.ts";
 import { startAsyncRunNotifier, stopAsyncRunNotifier, type AsyncNotifierState } from "./async-notifier.ts";
@@ -40,6 +41,7 @@ import { detectInterruptedRuns } from "../runtime/crash-recovery.ts";
 import { DeliveryCoordinator } from "../runtime/delivery-coordinator.ts";
 import { OverflowRecoveryTracker } from "../runtime/overflow-recovery.ts";
 import { tryRegisterSessionCleanup } from "../runtime/session-resources.ts";
+import { createSessionSnapshot } from "../runtime/session-snapshot.ts";
 import { initI18n } from "../i18n.ts";
 
 export { __test__subagentSpawnParams };
@@ -252,18 +254,18 @@ export function registerPiTeams(pi: ExtensionAPI): void {
 	const openLiveSidebar = (ctx: ExtensionContext, runId: string): void => {
 		const uiConfig = loadConfig(ctx.cwd).config.ui;
 		const autoOpen = uiConfig?.autoOpenDashboard === true;
-		const foregroundAutoOpen = uiConfig?.autoOpenDashboardForForegroundRuns !== false;
-		if (!ctx.hasUI || !autoOpen || !foregroundAutoOpen || (uiConfig?.dashboardPlacement ?? "right") !== "right") return;
+		const foregroundAutoOpen = uiConfig?.autoOpenDashboardForForegroundRuns ?? DEFAULT_UI.autoOpenDashboardForForegroundRuns;
+		if (!ctx.hasUI || !autoOpen || !foregroundAutoOpen || (uiConfig?.dashboardPlacement ?? DEFAULT_UI.dashboardPlacement) !== "right") return;
 		if (liveSidebarRunId === runId) return;
 		liveSidebarRunId = runId;
-		const widgetPlacement = uiConfig?.widgetPlacement ?? "aboveEditor";
+		const widgetPlacement = uiConfig?.widgetPlacement ?? DEFAULT_UI.widgetPlacement;
 		setExtensionWidget(ctx, "pi-crew", undefined, { placement: widgetPlacement });
 		setExtensionWidget(ctx, "pi-crew-active", undefined, { placement: widgetPlacement });
 		widgetState.lastVisibility = "hidden";
 		widgetState.lastPlacement = widgetPlacement;
 		widgetState.lastKey = "pi-crew-active";
 		widgetState.model = undefined;
-		const width = Math.min(90, Math.max(40, uiConfig?.dashboardWidth ?? 56));
+		const width = Math.min(90, Math.max(40, uiConfig?.dashboardWidth ?? DEFAULT_UI.dashboardWidth));
 		void showCustom<undefined>(ctx, (_tui, theme, _keybindings, done) => new LiveRunSidebar({ cwd: ctx.cwd, runId, done, theme, config: uiConfig, snapshotCache: getRunSnapshotCache(ctx.cwd) }), {
 			overlay: true,
 			overlayOptions: { width, minWidth: 40, maxHeight: "100%", anchor: "top-right", offsetX: 0, offsetY: 0, margin: { top: 0, right: 0, bottom: 0, left: 0 }, visible: (termWidth: number) => termWidth >= 100 },
@@ -397,7 +399,7 @@ export function registerPiTeams(pi: ExtensionAPI): void {
 		configureNotifications(ctx);
 		configureObservability(ctx);
 		configureDeliveryCoordinator();
-		const sessionId = (ctx as unknown as Record<string, unknown>).sessionId;
+		const sessionId = ctx.sessionManager?.getSessionId?.() ?? (ctx as unknown as Record<string, unknown>).sessionId;
 		if (typeof sessionId === "string" && sessionId) deliveryCoordinator?.activate(sessionId);
 		tryRegisterSessionCleanup(pi, () => { terminateActiveChildPiProcesses(); cleanupRuntime(); });
 		registerPiCrewPowerbarSegments(pi.events, loadedConfig.config.ui);
@@ -459,7 +461,7 @@ export function registerPiTeams(pi: ExtensionAPI): void {
 			const snapshotCache = lastFrameSnapshotCache ?? getRunSnapshotCache(currentCtx.cwd);
 			const manifests = lastPreloadedManifests.length > 0 ? lastPreloadedManifests : activeCache.list(20);
 			if (liveSidebarRunId) {
-				const placement = config?.widgetPlacement ?? "aboveEditor";
+				const placement = config?.widgetPlacement ?? DEFAULT_UI.widgetPlacement;
 				if (widgetState.lastVisibility !== "hidden" || widgetState.lastPlacement !== placement) {
 					setExtensionWidget(currentCtx, "pi-crew", undefined, { placement });
 					setExtensionWidget(currentCtx, "pi-crew-active", undefined, { placement });
@@ -499,7 +501,7 @@ export function registerPiTeams(pi: ExtensionAPI): void {
 			}
 		};
 
-		const fallbackMs = loadedConfig.config.ui?.dashboardLiveRefreshMs ?? 250;
+		const fallbackMs = loadedConfig.config.ui?.dashboardLiveRefreshMs ?? DEFAULT_UI.refreshMs;
 		renderScheduler = new RenderScheduler(pi.events, renderTick, {
 			fallbackMs,
 			onInvalidate: () => getRunSnapshotCache(ctx.cwd).invalidate(),
@@ -509,8 +511,14 @@ export function registerPiTeams(pi: ExtensionAPI): void {
 	});
 	pi.on("session_before_switch", () => {
 		sessionGeneration++;
-		// Phase 11b: Capture state before session switch
 		const pendingCount = deliveryCoordinator?.getPendingCount() ?? 0;
+		try {
+			const activeRuns = currentCtx ? getManifestCache(currentCtx.cwd).list(50).filter((run) => run.status === "running" || run.status === "queued" || run.status === "blocked") : [];
+			const snapshot = createSessionSnapshot(activeRuns, pendingCount, sessionGeneration);
+			if (pendingCount > 0 || snapshot.activeRunIds.length > 0) logInternalError("register.session-before-switch", undefined, JSON.stringify(snapshot));
+		} catch (error) {
+			logInternalError("register.session-before-switch.snapshot", error);
+		}
 		if (pendingCount > 0) {
 			logInternalError("register.session-before-switch", `Switching session with ${pendingCount} pending deliveries`);
 		}
@@ -523,8 +531,9 @@ export function registerPiTeams(pi: ExtensionAPI): void {
 	// Phase 11a: Dynamic resource discovery — inject pi-crew skill paths.
 	try {
 		pi.on("resources_discover", () => {
-			const skillDir = path.resolve(process.cwd(), "skills");
-			const extSkillDir = path.resolve(__dirname, "..", "..", "skills");
+			const sessionCwd = currentCtx?.cwd ?? process.cwd();
+			const skillDir = path.resolve(sessionCwd, "skills");
+			const extSkillDir = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "..", "..", "skills");
 			const paths: string[] = [];
 			if (fs.existsSync(extSkillDir)) paths.push(extSkillDir);
 			if (skillDir !== extSkillDir && fs.existsSync(skillDir)) paths.push(skillDir);