@tduyng/pi-goal 0.0.1 → 0.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tduyng/pi-goal",
-  "version": "0.0.1",
+  "version": "0.0.2",
   "description": "Long-running objectives that survive reload and tree navigation, plus a timed or self-paced loop to drive them.",
   "keywords": [
     "pi",

package/src/goal.ts

@@ -510,7 +510,7 @@ export function goalContextMessages<T>(messages: T[]): T[] {
 }
 
 export function goalStopStatusForAssistantError(message: { errorMessage?: string } | undefined): GoalStatus {
-    return PROVIDER_LIMIT_PATTERN.test(message?.errorMessage ?? '') ? 'usageLimited' : 'blocked'
+    return PROVIDER_LIMIT_PATTERN.test(message?.errorMessage ?? '') ? 'usageLimited' : 'paused'
 }
 
 const RoleContentSchema = t.object(
@@ -952,7 +952,11 @@ export default function goalExtension(pi: ExtensionAPI) {
             }
 
             const checkArg = /^check(?:\s+([\s\S]+))?$/i.exec(trimmed)
-            if (checkArg !== null && goal !== null) {
+            if (checkArg !== null) {
+                if (goal === null) {
+                    showGoalMessage('No goal is currently set.')
+                    return
+                }
                 const raw = checkArg[1]?.trim() ?? ''
                 if (raw === '') {
                     showGoalMessage('Usage: /goal check <command>, or /goal check off')

package/src/loop/index.ts

@@ -31,6 +31,7 @@ export function parseLoopState(value: JsonValue): LoopState | null {
 export default function loopExtension(pi: ExtensionAPI): void {
     let loop: LoopState | null = null
     let timer: ReturnType<typeof setTimeout> | null = null
+    let lastError: string | undefined
 
     function clearTimer(): void {
         if (timer) {
@@ -142,8 +143,27 @@ export default function loopExtension(pi: ExtensionAPI): void {
 
     pi.on('session_shutdown', async () => clearTimer())
 
+    pi.on('agent_end', async (event) => {
+        const assistant = event.messages.findLast((message) => message.role === 'assistant')
+        lastError =
+            assistant?.stopReason === 'error'
+                ? (assistant.errorMessage ?? 'the last turn ended with an error')
+                : undefined
+    })
+
     pi.on('agent_settled', async (event, ctx) => {
-        if (!loop || loop.intervalMs !== null) return
+        if (!loop) return
+        if (lastError !== undefined) {
+            const reason = lastError
+            lastError = undefined
+            clearTimer()
+            loop = null
+            persist()
+            updateStatus(ctx)
+            ctx.ui.notify(`Loop stopped: ${reason}`, 'warning')
+            return
+        }
+        if (loop.intervalMs !== null) return
         if (event.aborted) return
         clearTimer()
         timer = setTimeout(() => fireSafely(ctx), SELF_PACE_DELAY_MS)

package/src/loop/logic.ts

@@ -14,6 +14,7 @@ export type ParsedLoopArgs =
 const INTERVAL_PATTERN = /^(\d+)(ms|s|m|h|d)$/i
 const MS_PER_UNIT = { ms: 1, s: 1_000, m: 60_000, h: 3_600_000, d: 86_400_000 } satisfies Record<string, number>
 export const MAX_INTERVAL_MS = 2_147_483_647
+export const MIN_INTERVAL_MS = 10_000
 const STOP_WORDS = new Set(['stop', 'cancel', 'clear', 'off'])
 
 export function parseInterval(token: string): number | null {
@@ -23,7 +24,7 @@ export function parseInterval(token: string): number | null {
     const unit = (match[2] ?? '').toLowerCase()
     const msPerUnit = MS_PER_UNIT[unit as keyof typeof MS_PER_UNIT]
     if (!Number.isFinite(amount) || amount <= 0 || msPerUnit === undefined) return null
-    return Math.min(amount * msPerUnit, MAX_INTERVAL_MS)
+    return Math.min(Math.max(amount * msPerUnit, MIN_INTERVAL_MS), MAX_INTERVAL_MS)
 }
 
 export function parseLoopArgs(input: string): ParsedLoopArgs {