kcode-pi 0.1.24 → 0.1.30

package/src/harness/state.ts

@@ -100,7 +100,7 @@ export function addQuestion(
 	run: ActiveRun,
 	input: { question: string; reason?: string; choices?: string[]; blocking?: boolean },
 ): KdQuestion {
-	const existing = run.questions ?? [];
+	const existing = Array.isArray(run.questions) ? run.questions : [];
 	const question: KdQuestion = {
 		id: createQuestionId(existing.length + 1),
 		phase: run.phase,
@@ -118,7 +118,7 @@ export function addQuestion(
 }
 
 export function answerQuestion(cwd: string, run: ActiveRun, id: string, answer: string): KdQuestion | undefined {
-	const question = (run.questions ?? []).find((item) => item.id === id);
+	const question = (Array.isArray(run.questions) ? run.questions : []).find((item) => item.id === id);
 	if (!question) return undefined;
 	question.status = "answered";
 	question.answer = answer.trim();
@@ -129,7 +129,7 @@ export function answerQuestion(cwd: string, run: ActiveRun, id: string, answer:
 }
 
 export function openBlockingQuestions(run: ActiveRun): KdQuestion[] {
-	return (run.questions ?? []).filter((question) => question.status === "open" && question.blocking);
+	return (Array.isArray(run.questions) ? run.questions : []).filter((question) => question.status === "open" && question.blocking);
 }
 
 export function updateProductProfile(cwd: string, run: ActiveRun, productInput: string, version?: string): ActiveRun {
@@ -189,6 +189,21 @@ export function advanceRun(cwd: string, run: ActiveRun, requestedPhase?: KdPhase
 	return { run, message: `已推进 Kingdee run 到阶段：${target}` };
 }
 
+export function advanceRunIfReady(cwd: string, run: ActiveRun): { run: ActiveRun; advanced: boolean; message: string } {
+	const current = readRun(cwd, run.id) ?? run;
+	const target = nextPhase(current.phase);
+	if (!target) {
+		return { run: current, advanced: false, message: `Run 已处于最终阶段：${current.phase}` };
+	}
+
+	const result = advanceRun(cwd, current, target);
+	return {
+		run: result.run,
+		advanced: result.run.phase === target,
+		message: result.message,
+	};
+}
+
 export function refreshGate(cwd: string, run: ActiveRun): ActiveRun {
 	run.gate = inspectGate(cwd, run);
 	writeActiveRun(cwd, run);
@@ -202,19 +217,90 @@ function writeRunState(cwd: string, run: ActiveRun): void {
 }
 
 function hydrateRun(parsed: ActiveRun): ActiveRun | undefined {
-	if (!parsed.id || !isKdPhase(parsed.phase)) return undefined;
-	parsed.artifacts ??= {};
-	parsed.questions ??= [];
-	parsed.status ??= "active";
-	parsed.createdAt ??= parsed.updatedAt;
-	parsed.updatedAt ??= parsed.createdAt;
+	if (!parsed || typeof parsed !== "object") return undefined;
+	if (typeof parsed.id !== "string" || !parsed.id.trim()) return undefined;
+	if (typeof parsed.phase !== "string" || !isKdPhase(parsed.phase)) return undefined;
+	parsed.id = parsed.id.trim();
+	parsed.artifacts = isPlainObject(parsed.artifacts) ? sanitizeArtifacts(parsed.artifacts) : {};
+	parsed.questions = Array.isArray(parsed.questions) ? parsed.questions.map(sanitizeQuestion).filter((question): question is KdQuestion => Boolean(question)) : [];
+	parsed.status = parsed.status === "paused" || parsed.status === "done" ? parsed.status : "active";
+	parsed.goal = typeof parsed.goal === "string" ? parsed.goal : undefined;
+	parsed.version = typeof parsed.version === "string" ? parsed.version : undefined;
+	parsed.createdAt = typeof parsed.createdAt === "string" ? parsed.createdAt : typeof parsed.updatedAt === "string" ? parsed.updatedAt : undefined;
+	parsed.updatedAt = typeof parsed.updatedAt === "string" ? parsed.updatedAt : parsed.createdAt;
 	const legacyEdition = (parsed as ActiveRun & { edition?: string }).edition;
-	parsed.profile = parsed.profile ?? profileForProduct(parsed.product ?? resolveProductProfile(legacyEdition).product);
+	const product = typeof parsed.profile?.product === "string" ? parsed.profile.product : typeof parsed.product === "string" ? parsed.product : resolveProductProfile(typeof legacyEdition === "string" ? legacyEdition : undefined).product;
+	parsed.profile = profileForProduct(product);
 	parsed.product = parsed.profile.product;
-	parsed.gate ??= { passed: false, checkedAt: new Date().toISOString() };
+	parsed.riskAssessment = sanitizeRiskAssessment(parsed.riskAssessment);
+	parsed.repair = sanitizeRepairState(parsed.repair);
+	parsed.gate = sanitizeGate(parsed.gate);
 	return parsed;
 }
 
+function sanitizeArtifacts(value: Record<string, unknown>): Record<string, string> {
+	return Object.fromEntries(Object.entries(value).filter((entry): entry is [string, string] => typeof entry[0] === "string" && typeof entry[1] === "string"));
+}
+
+function sanitizeQuestion(value: unknown): KdQuestion | undefined {
+	if (!isPlainObject(value)) return undefined;
+	const phase = typeof value.phase === "string" && isKdPhase(value.phase) ? value.phase : undefined;
+	const question = typeof value.question === "string" ? value.question.trim() : "";
+	if (!phase || !question) return undefined;
+	const id = typeof value.id === "string" && value.id.trim() ? value.id.trim() : createQuestionId(1);
+	return {
+		id,
+		phase,
+		question,
+		reason: typeof value.reason === "string" && value.reason.trim() ? value.reason.trim() : undefined,
+		choices: Array.isArray(value.choices) ? value.choices.filter((choice): choice is string => typeof choice === "string" && Boolean(choice.trim())).map((choice) => choice.trim()) : undefined,
+		blocking: value.blocking !== false,
+		status: value.status === "answered" ? "answered" : "open",
+		answer: typeof value.answer === "string" ? value.answer : undefined,
+		createdAt: typeof value.createdAt === "string" ? value.createdAt : new Date().toISOString(),
+		answeredAt: typeof value.answeredAt === "string" ? value.answeredAt : undefined,
+	};
+}
+
+function sanitizeRiskAssessment(value: unknown): ActiveRun["riskAssessment"] {
+	if (!isPlainObject(value)) return undefined;
+	const level = value.level;
+	if (level !== "low" && level !== "medium" && level !== "high") return undefined;
+	return {
+		level,
+		reason: typeof value.reason === "string" ? value.reason : "",
+		source: value.source === "verify" || value.source === "ship" ? value.source : "manual",
+		updatedAt: typeof value.updatedAt === "string" ? value.updatedAt : new Date().toISOString(),
+	};
+}
+
+function sanitizeRepairState(value: unknown): ActiveRun["repair"] {
+	if (!isPlainObject(value)) return undefined;
+	const attempts = typeof value.attempts === "number" && Number.isFinite(value.attempts) ? Math.max(0, Math.trunc(value.attempts)) : 0;
+	const maxAttempts = typeof value.maxAttempts === "number" && Number.isFinite(value.maxAttempts) ? Math.max(1, Math.trunc(value.maxAttempts)) : 3;
+	return {
+		attempts,
+		maxAttempts,
+		lastFailureEvidence: typeof value.lastFailureEvidence === "string" ? value.lastFailureEvidence : undefined,
+		lastFailureSignature: typeof value.lastFailureSignature === "string" ? value.lastFailureSignature : undefined,
+		status: value.status === "repairing" || value.status === "blocked" ? value.status : "idle",
+		updatedAt: typeof value.updatedAt === "string" ? value.updatedAt : new Date().toISOString(),
+	};
+}
+
+function sanitizeGate(value: unknown): ActiveRun["gate"] {
+	if (!isPlainObject(value)) return { passed: false, checkedAt: new Date().toISOString() };
+	return {
+		passed: value.passed === true,
+		reason: typeof value.reason === "string" ? value.reason : undefined,
+		checkedAt: typeof value.checkedAt === "string" ? value.checkedAt : new Date().toISOString(),
+	};
+}
+
+function isPlainObject(value: unknown): value is Record<string, unknown> {
+	return Boolean(value) && typeof value === "object" && !Array.isArray(value);
+}
+
 function createRunId(goal: string): string {
 	const stamp = new Date().toISOString().replace(/[-:]/g, "").replace(/\..+/, "").replace("T", "-");
 	const slug = goal

package/src/harness/types.ts

@@ -18,6 +18,15 @@ export interface GateResult {
 	checkedAt: string;
 }
 
+export interface RepairState {
+	attempts: number;
+	maxAttempts: number;
+	lastFailureEvidence?: string;
+	lastFailureSignature?: string;
+	status: "idle" | "repairing" | "blocked";
+	updatedAt: string;
+}
+
 export interface KdQuestion {
 	id: string;
 	phase: KdPhase;
@@ -43,6 +52,7 @@ export interface ActiveRun {
 	version?: string;
 	profile?: ProductProfile;
 	riskAssessment?: RiskAssessment;
+	repair?: RepairState;
 	artifacts: Record<string, string>;
 	gate: GateResult;
 	questions?: KdQuestion[];