@leing2021/super-pi 0.23.2 → 0.23.3

package/extensions/ce-core/index.ts

@@ -310,6 +310,7 @@ const contextHandoffParams = Type.Object({
     Type.Literal("load"),
     Type.Literal("latest"),
     Type.Literal("status"),
+    Type.Literal("validate"),
   ], { description: "Handoff operation" }),
   repoRoot: Type.String({ description: "Repository root" }),
   currentStage: Type.Optional(Type.String({ description: "Current pipeline stage (e.g. 02-plan)" })),
@@ -393,18 +394,21 @@ export default function ceCoreExtension(pi: ExtensionAPI) {
     if (thinkingStrategy) {
       const targetThinking = thinkingStrategy[stageKey] ?? thinkingStrategy.default
       if (targetThinking) {
-        const levelMap: Record<string, "low" | "medium" | "high"> = {
+        const levelMap: Record<string, ReturnType<ExtensionAPI["getThinkingLevel"]>> = {
+          off: "off",
+          minimal: "minimal",
           low: "low",
           medium: "medium",
           high: "high",
+          xhigh: "xhigh",
           "0": "low",
           "1": "medium",
           "2": "high",
         }
         const normalized = levelMap[targetThinking.toLowerCase()] ?? "medium"
-        const currentLevel = ctx.getThinkingLevel?.() ?? "medium"
+        const currentLevel = pi.getThinkingLevel()
         if (currentLevel !== normalized) {
-          ctx.setThinkingLevel?.(normalized)
+          pi.setThinkingLevel(normalized)
           if (ctx.hasUI) {
             ctx.ui.notify(`Switched thinking level for ${stageKey}: ${normalized}`, "info")
           }
@@ -748,7 +752,7 @@ export default function ceCoreExtension(pi: ExtensionAPI) {
   pi.registerTool({
     name: contextHandoff.name,
     label: "Context Handoff",
-    description: "Manage cross-stage context handoffs with evidence-first templates. Supports save (write handoff + state), load (read handoff + state), latest (read latest dated handoff), and status (read current state).",
+    description: "Manage cross-stage context handoffs with evidence-first templates. Supports save (write handoff + state), load (read handoff + state), latest (read latest dated handoff), status (read current state), and validate (check continuation readiness with deterministic probes).",
     parameters: contextHandoffParams,
     async execute(_toolCallId, params) {
       const result = await contextHandoff.execute({

package/extensions/ce-core/tools/context-handoff.ts

@@ -5,8 +5,23 @@ import { normalizeSlug } from "../utils/name-utils"
 
 export type ContextHealth = "good" | "watch" | "heavy" | "critical"
 
+export type ContextHandoffRecommendedAction = "continue" | "save_handoff" | "fill_required_context"
+
+export interface ContextHandoffValidationCheck {
+  name: string
+  passed: boolean
+  reason: string
+}
+
+export interface ContextHandoffValidationProbes {
+  recall: boolean
+  continuation: boolean
+  artifact: boolean
+  decision: boolean
+}
+
 export interface ContextHandoffInput {
-  operation: "save" | "load" | "latest" | "status"
+  operation: "save" | "load" | "latest" | "status" | "validate"
   repoRoot: string
   currentStage?: string
   nextStage?: string
@@ -63,6 +78,13 @@ export interface ContextHandoffResult {
   recentlyAccessedFiles?: string[]
   compressionRisk?: string[]
   updatedAt?: string
+  // Validation fields
+  ok?: boolean
+  probes?: ContextHandoffValidationProbes
+  checks?: ContextHandoffValidationCheck[]
+  missing?: string[]
+  warnings?: string[]
+  recommendedAction?: ContextHandoffRecommendedAction
 }
 
 function ceDir(repoRoot: string): string {
@@ -268,6 +290,8 @@ export function createContextHandoffTool() {
           return latest(input)
         case "status":
           return status(input)
+        case "validate":
+          return validate(input)
         default:
           throw new Error(`Unknown operation: ${input.operation}`)
       }
@@ -434,6 +458,272 @@ async function latest(input: ContextHandoffInput): Promise<ContextHandoffResult>
   }
 }
 
+const PLACEHOLDER_VALUES = new Set(["n/a", "na", "not run", "none", "", "-"])
+const PLACEHOLDER_PREFIXES = ["- n/a", "- na", "- not run", "- none", "- "]
+
+function isPlaceholder(text: string): boolean {
+  const trimmed = text.trim().toLowerCase()
+  if (PLACEHOLDER_VALUES.has(trimmed)) return true
+  for (const prefix of PLACEHOLDER_PREFIXES) {
+    if (trimmed === prefix) return true
+  }
+  return false
+}
+
+function isMeaningfulText(value?: string): boolean {
+  return Boolean(value && !isPlaceholder(value))
+}
+
+function toPublicHandoffPath(repoRoot: string, filePath: string): string {
+  return path.isAbsolute(filePath) ? toRepoRelative(repoRoot, filePath) : filePath
+}
+
+function extractSection(markdown: string, heading: string): string {
+  const lines = markdown.split("\n")
+  const sectionLines: string[] = []
+  let inSection = false
+
+  for (const line of lines) {
+    if (line.startsWith("## ")) {
+      if (inSection) break
+      if (line.slice(3).trim() === heading) {
+        inSection = true
+      }
+      continue
+    }
+    if (inSection) {
+      sectionLines.push(line)
+    }
+  }
+
+  return sectionLines.join("\n").trim()
+}
+
+function sectionHasMeaningfulContent(markdown: string, heading: string): boolean {
+  const section = extractSection(markdown, heading)
+  if (!section) return false
+
+  const lines = section.split("\n")
+  for (const line of lines) {
+    const trimmed = line.trim()
+    if (!trimmed) continue
+    if (!isPlaceholder(trimmed)) return true
+  }
+  return false
+}
+
+function hasMeaningfulArray(arr: string[]): boolean {
+  return arr.some(item => isMeaningfulText(item))
+}
+
+function hasMeaningfulRecord(rec: Record<string, string | undefined>): boolean {
+  return Object.values(rec).some(isMeaningfulText)
+}
+
+function isMeaningfulStage(value?: string): boolean {
+  return Boolean(value && value.trim().length > 0 && value !== "unknown" && !isPlaceholder(value))
+}
+
+async function validate(input: ContextHandoffInput): Promise<ContextHandoffResult> {
+  const checks: ContextHandoffValidationCheck[] = []
+  const missing: string[] = []
+  const warnings: string[] = []
+
+  // Read state
+  const state = await readState(input.repoRoot)
+  const hasState = state !== null
+
+  // Read markdown
+  let markdown = ""
+  let hasMarkdown = false
+  let handoffFile = ""
+
+  if (input.handoffPath) {
+    const absolutePath = resolveRepoPath(input.repoRoot, input.handoffPath)
+    if (existsSync(absolutePath)) {
+      markdown = await readFile(absolutePath, "utf8")
+      hasMarkdown = markdown.trim().length > 0
+      handoffFile = toPublicHandoffPath(input.repoRoot, input.handoffPath)
+    }
+  }
+
+  if (!hasMarkdown && state?.latestHandoffPath) {
+    const absolutePath = resolveRepoPath(input.repoRoot, state.latestHandoffPath)
+    if (existsSync(absolutePath)) {
+      markdown = await readFile(absolutePath, "utf8")
+      hasMarkdown = markdown.trim().length > 0
+      handoffFile = toPublicHandoffPath(input.repoRoot, state.latestHandoffPath)
+    }
+  }
+
+  if (!hasMarkdown) {
+    const latestPath = latestHandoffPath(input.repoRoot)
+    if (existsSync(latestPath)) {
+      markdown = await readFile(latestPath, "utf8")
+      hasMarkdown = markdown.trim().length > 0
+      handoffFile = toRepoRelative(input.repoRoot, latestPath)
+    }
+  }
+
+  const found = hasState || hasMarkdown
+
+  checks.push({
+    name: "state_exists",
+    passed: hasState,
+    reason: hasState ? "Found context-state.json" : "No context-state.json found",
+  })
+
+  checks.push({
+    name: "handoff_exists",
+    passed: hasMarkdown,
+    reason: hasMarkdown ? "Found handoff markdown" : "No handoff markdown found",
+  })
+
+  // --- Recall probe ---
+  const hasCurrentTruth = state ? hasMeaningfulArray(state.currentTruth) : false
+  const hasCurrentStage = state ? isMeaningfulStage(state.currentStage) : false
+  const hasCurrentTaskSection = hasMarkdown && sectionHasMeaningfulContent(markdown, "Current Task")
+
+  const recallPass = hasCurrentTruth || (hasCurrentStage && (hasMarkdown || (state?.nextStage !== undefined))) || hasCurrentTaskSection
+
+  checks.push({
+    name: "recall_current_truth",
+    passed: hasCurrentTruth,
+    reason: hasCurrentTruth ? `Found ${state!.currentTruth.length} current truth entries` : "No current truth entries",
+  })
+
+  checks.push({
+    name: "recall_current_task",
+    passed: hasCurrentTaskSection,
+    reason: hasCurrentTaskSection ? "Current Task section has meaningful content" : "Current Task section is missing or placeholder",
+  })
+
+  checks.push({
+    name: "recall_current_stage",
+    passed: hasCurrentStage,
+    reason: hasCurrentStage ? `Current stage: ${state!.currentStage}` : "No meaningful current stage",
+  })
+
+  if (!recallPass) {
+    missing.push("recall: current task or goal evidence")
+  }
+
+  // --- Continuation probe ---
+  // Tightened: only actionable next-step evidence passes
+  const hasNextStage = state ? isMeaningfulStage(state.nextStage) : false
+  const hasNextMinimalStep = hasMarkdown && sectionHasMeaningfulContent(markdown, "Next Minimal Step")
+
+  const continuationPass = hasNextMinimalStep || hasNextStage
+
+  checks.push({
+    name: "continuation_next_stage",
+    passed: hasNextStage,
+    reason: hasNextStage ? `Next stage: ${state!.nextStage}` : "No meaningful next stage",
+  })
+
+  checks.push({
+    name: "continuation_next_minimal_step",
+    passed: hasNextMinimalStep,
+    reason: hasNextMinimalStep ? "Next Minimal Step has meaningful content" : "Next Minimal Step is missing or placeholder",
+  })
+
+  // Diagnostic checks (do not affect continuation pass)
+  const hasBlocker = state ? Boolean(state.blocker && !isPlaceholder(state.blocker)) : false
+  const hasVerification = state ? Boolean(state.verification && !isPlaceholder(state.verification)) : false
+  const hasVerificationSection = hasMarkdown && sectionHasMeaningfulContent(markdown, "Verification")
+
+  checks.push({
+    name: "continuation_blocker",
+    passed: hasBlocker,
+    reason: hasBlocker ? `Blocker: ${state!.blocker}` : "No blocker information",
+  })
+
+  checks.push({
+    name: "continuation_verification",
+    passed: hasVerification || hasVerificationSection,
+    reason: (hasVerification || hasVerificationSection) ? "Verification evidence present" : "No verification information",
+  })
+
+  if (!continuationPass) {
+    missing.push("continuation: next minimal step or next stage evidence")
+  }
+
+  // --- Artifact probe ---
+  const hasActiveFiles = state ? hasMeaningfulArray(state.activeFiles) : false
+  const hasRecentlyAccessed = state ? hasMeaningfulArray(state.recentlyAccessedFiles) : false
+  const hasArtifacts = state ? hasMeaningfulRecord(state.artifacts) : false
+  const hasActiveFilesSection = hasMarkdown && sectionHasMeaningfulContent(markdown, "Active Files")
+  const hasRecentlyAccessedSection = hasMarkdown && sectionHasMeaningfulContent(markdown, "Recently Accessed Files")
+  const hasArtifactsSection = hasMarkdown && sectionHasMeaningfulContent(markdown, "Artifacts")
+
+  const artifactPass = hasActiveFiles || hasRecentlyAccessed || hasArtifacts
+    || hasActiveFilesSection || hasRecentlyAccessedSection || hasArtifactsSection
+
+  checks.push({
+    name: "artifact_active_files",
+    passed: hasActiveFiles || hasActiveFilesSection,
+    reason: (hasActiveFiles || hasActiveFilesSection) ? "Active files present" : "No active files",
+  })
+
+  if (!artifactPass) {
+    warnings.push("artifact: active files or artifacts are missing")
+  }
+
+  // --- Decision probe ---
+  const hasOpenDecisions = state ? hasMeaningfulArray(state.openDecisions) : false
+  const hasInvalidatedAssumptions = state ? hasMeaningfulArray(state.invalidatedAssumptions) : false
+  const hasOpenDecisionsSection = hasMarkdown && sectionHasMeaningfulContent(markdown, "Open Decisions")
+  const hasInvalidatedSection = hasMarkdown && sectionHasMeaningfulContent(markdown, "Invalidated Assumptions")
+  const hasCurrentTruthSection = hasMarkdown && sectionHasMeaningfulContent(markdown, "Current Truth")
+
+  const decisionPass = hasOpenDecisions || hasInvalidatedAssumptions || hasCurrentTruth
+    || hasOpenDecisionsSection || hasInvalidatedSection || hasCurrentTruthSection
+
+  checks.push({
+    name: "decision_open_decisions",
+    passed: hasOpenDecisions || hasOpenDecisionsSection,
+    reason: (hasOpenDecisions || hasOpenDecisionsSection) ? "Open decisions present" : "No open decisions",
+  })
+
+  if (!decisionPass) {
+    warnings.push("decision: decisions or invalidated assumptions are missing")
+  }
+
+  // --- ok derivation ---
+  const ok = recallPass && continuationPass
+
+  // --- recommended action ---
+  let recommendedAction: ContextHandoffRecommendedAction
+  if (!found) {
+    recommendedAction = "save_handoff"
+  } else if (!ok) {
+    recommendedAction = "fill_required_context"
+  } else {
+    recommendedAction = "continue"
+  }
+
+  return {
+    operation: "validate",
+    found,
+    ok,
+    path: handoffFile || undefined,
+    probes: {
+      recall: recallPass,
+      continuation: continuationPass,
+      artifact: artifactPass,
+      decision: decisionPass,
+    },
+    checks,
+    missing,
+    warnings,
+    recommendedAction,
+    currentStage: state?.currentStage,
+    nextStage: state?.nextStage,
+    contextHealth: state?.contextHealth,
+    updatedAt: state?.updatedAt,
+  }
+}
+
 async function status(input: ContextHandoffInput): Promise<ContextHandoffResult> {
   const state = await readState(input.repoRoot)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@leing2021/super-pi",
-  "version": "0.23.2",
+  "version": "0.23.3",
   "private": false,
   "type": "module",
   "description": "Pi-native Compound Engineering package for iterative development workflows",