@clanker-code/pi-subagents 0.10.5

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (130) hide show
  1. package/.plans/PLAN-next-changes.md +183 -0
  2. package/.plans/README.md +14 -0
  3. package/AGENTS.md +31 -0
  4. package/CHANGELOG.md +583 -0
  5. package/CLAUDE.md +1 -0
  6. package/LICENSE +21 -0
  7. package/README.md +630 -0
  8. package/RELEASE.md +39 -0
  9. package/dist/abort-resend.d.ts +35 -0
  10. package/dist/abort-resend.js +71 -0
  11. package/dist/agent-details.d.ts +17 -0
  12. package/dist/agent-details.js +22 -0
  13. package/dist/agent-manager.d.ts +132 -0
  14. package/dist/agent-manager.js +493 -0
  15. package/dist/agent-runner.d.ts +165 -0
  16. package/dist/agent-runner.js +732 -0
  17. package/dist/agent-tool-description.d.ts +9 -0
  18. package/dist/agent-tool-description.js +147 -0
  19. package/dist/agent-types.d.ts +60 -0
  20. package/dist/agent-types.js +157 -0
  21. package/dist/context.d.ts +12 -0
  22. package/dist/context.js +56 -0
  23. package/dist/cross-extension-rpc.d.ts +46 -0
  24. package/dist/cross-extension-rpc.js +76 -0
  25. package/dist/custom-agents.d.ts +14 -0
  26. package/dist/custom-agents.js +149 -0
  27. package/dist/default-agents.d.ts +7 -0
  28. package/dist/default-agents.js +119 -0
  29. package/dist/enabled-models.d.ts +49 -0
  30. package/dist/enabled-models.js +145 -0
  31. package/dist/env.d.ts +6 -0
  32. package/dist/env.js +28 -0
  33. package/dist/group-join.d.ts +32 -0
  34. package/dist/group-join.js +116 -0
  35. package/dist/index.d.ts +36 -0
  36. package/dist/index.js +1918 -0
  37. package/dist/invocation-config.d.ts +25 -0
  38. package/dist/invocation-config.js +19 -0
  39. package/dist/memory.d.ts +49 -0
  40. package/dist/memory.js +151 -0
  41. package/dist/model-resolver.d.ts +19 -0
  42. package/dist/model-resolver.js +62 -0
  43. package/dist/notifications.d.ts +6 -0
  44. package/dist/notifications.js +107 -0
  45. package/dist/output-file.d.ts +24 -0
  46. package/dist/output-file.js +86 -0
  47. package/dist/peek.d.ts +37 -0
  48. package/dist/peek.js +121 -0
  49. package/dist/prompts.d.ts +40 -0
  50. package/dist/prompts.js +95 -0
  51. package/dist/schedule-store.d.ts +38 -0
  52. package/dist/schedule-store.js +155 -0
  53. package/dist/schedule.d.ts +109 -0
  54. package/dist/schedule.js +338 -0
  55. package/dist/settings.d.ts +135 -0
  56. package/dist/settings.js +168 -0
  57. package/dist/skill-loader.d.ts +24 -0
  58. package/dist/skill-loader.js +93 -0
  59. package/dist/status-note.d.ts +13 -0
  60. package/dist/status-note.js +24 -0
  61. package/dist/types.d.ts +184 -0
  62. package/dist/types.js +7 -0
  63. package/dist/ui/agent-tool-rendering.d.ts +34 -0
  64. package/dist/ui/agent-tool-rendering.js +154 -0
  65. package/dist/ui/agent-widget-tree.d.ts +33 -0
  66. package/dist/ui/agent-widget-tree.js +130 -0
  67. package/dist/ui/agent-widget.d.ts +156 -0
  68. package/dist/ui/agent-widget.js +408 -0
  69. package/dist/ui/conversation-viewer.d.ts +47 -0
  70. package/dist/ui/conversation-viewer.js +290 -0
  71. package/dist/ui/menu-select.d.ts +20 -0
  72. package/dist/ui/menu-select.js +46 -0
  73. package/dist/ui/schedule-menu.d.ts +16 -0
  74. package/dist/ui/schedule-menu.js +99 -0
  75. package/dist/ui/viewer-keys.d.ts +20 -0
  76. package/dist/ui/viewer-keys.js +17 -0
  77. package/dist/usage.d.ts +50 -0
  78. package/dist/usage.js +49 -0
  79. package/dist/wait.d.ts +10 -0
  80. package/dist/wait.js +37 -0
  81. package/dist/worktree.d.ts +45 -0
  82. package/dist/worktree.js +160 -0
  83. package/docs/design/default-extension-tool-exposure.md +56 -0
  84. package/docs/superpowers/plans/2026-06-19-recursive-subagent-widget.md +600 -0
  85. package/docs/superpowers/specs/2026-06-19-recursive-subagent-widget-design.md +189 -0
  86. package/examples/agent-tool-description.md +45 -0
  87. package/package.json +56 -0
  88. package/reviews/proposal-structured-output-schema.md +135 -0
  89. package/reviews/recursive-subagent-widget-preview-rev2.png +0 -0
  90. package/reviews/recursive-subagent-widget-preview.html +137 -0
  91. package/reviews/recursive-subagent-widget-preview.png +0 -0
  92. package/reviews/subagent-features-comparison.md +350 -0
  93. package/src/abort-resend.ts +75 -0
  94. package/src/agent-details.ts +31 -0
  95. package/src/agent-manager.ts +596 -0
  96. package/src/agent-runner.ts +872 -0
  97. package/src/agent-tool-description.ts +163 -0
  98. package/src/agent-types.ts +189 -0
  99. package/src/context.ts +58 -0
  100. package/src/cross-extension-rpc.ts +122 -0
  101. package/src/custom-agents.ts +160 -0
  102. package/src/default-agents.ts +123 -0
  103. package/src/enabled-models.ts +180 -0
  104. package/src/env.ts +33 -0
  105. package/src/group-join.ts +141 -0
  106. package/src/index.ts +2115 -0
  107. package/src/invocation-config.ts +42 -0
  108. package/src/memory.ts +165 -0
  109. package/src/model-resolver.ts +81 -0
  110. package/src/notifications.ts +120 -0
  111. package/src/output-file.ts +96 -0
  112. package/src/peek.ts +155 -0
  113. package/src/prompts.ts +129 -0
  114. package/src/schedule-store.ts +153 -0
  115. package/src/schedule.ts +365 -0
  116. package/src/settings.ts +289 -0
  117. package/src/skill-loader.ts +102 -0
  118. package/src/status-note.ts +25 -0
  119. package/src/types.ts +195 -0
  120. package/src/ui/agent-tool-rendering.ts +175 -0
  121. package/src/ui/agent-widget-tree.ts +169 -0
  122. package/src/ui/agent-widget.ts +497 -0
  123. package/src/ui/conversation-viewer.ts +297 -0
  124. package/src/ui/menu-select.ts +68 -0
  125. package/src/ui/schedule-menu.ts +105 -0
  126. package/src/ui/viewer-keys.ts +39 -0
  127. package/src/usage.ts +60 -0
  128. package/src/wait.ts +44 -0
  129. package/src/worktree.ts +191 -0
  130. package/vitest.config.ts +25 -0
package/src/prompts.ts ADDED
@@ -0,0 +1,129 @@
1
+ /**
2
+ * prompts.ts — System prompt builder for agents.
3
+ */
4
+
5
+ import type { AgentConfig, EnvInfo } from "./types.js";
6
+
7
+ /** Extra sections to inject into the system prompt (memory, skills, etc.). */
8
+ export interface PromptExtras {
9
+ /** Persistent memory content to inject (first 200 lines of MEMORY.md + instructions). */
10
+ memoryBlock?: string;
11
+ /** Preloaded skill contents to inject. */
12
+ skillBlocks?: { name: string; content: string }[];
13
+ /** Manager-assigned agent id, used for recursive subagent metadata. */
14
+ agentId?: string;
15
+ /** Parent subagent id when this agent was spawned recursively. */
16
+ parentAgentId?: string;
17
+ /** Recursive subagent depth. */
18
+ depth?: number;
19
+ /** Maximum recursive subagent depth. */
20
+ maxDepth?: number;
21
+ }
22
+
23
+ function xmlAttr(value: string | number | undefined): string | undefined {
24
+ if (value == null) return undefined;
25
+ return String(value)
26
+ .replace(/&/g, "&")
27
+ .replace(/"/g, """)
28
+ .replace(/</g, "&lt;")
29
+ .replace(/>/g, "&gt;");
30
+ }
31
+
32
+ /**
33
+ * Build the system prompt for an agent from its config.
34
+ *
35
+ * - "replace" mode: env header + config.systemPrompt (full control, no parent identity)
36
+ * - "append" mode: parent system prompt + sub-agent context + env header + config.systemPrompt
37
+ * - "append" with empty systemPrompt: pure parent clone
38
+ *
39
+ * Both modes include an `<active_agent name="${config.name}"/>` tag so downstream
40
+ * extensions (e.g. permission/policy systems) can resolve per-agent policy
41
+ * inside the child session by parsing the system prompt. In replace mode the tag
42
+ * is prepended; in append mode it follows the shared inherited content so the
43
+ * parent prompt forms an identical, cacheable byte prefix with the parent
44
+ * session (the LLM's KV cache can then reuse those tokens across every spawn).
45
+ *
46
+ * @param parentSystemPrompt The parent agent's effective system prompt (for append mode).
47
+ * @param extras Optional extra sections to inject (memory, preloaded skills).
48
+ */
49
+ export function buildAgentPrompt(
50
+ config: AgentConfig,
51
+ cwd: string,
52
+ env: EnvInfo,
53
+ parentSystemPrompt?: string,
54
+ extras?: PromptExtras,
55
+ ): string {
56
+ const activeAgentAttrs = [
57
+ ["name", config.name],
58
+ ["agent_id", extras?.agentId],
59
+ ["parent_agent_id", extras?.parentAgentId],
60
+ ["depth", extras?.depth],
61
+ ["max_depth", extras?.maxDepth],
62
+ ]
63
+ .map(([name, value]) => {
64
+ const escaped = xmlAttr(value);
65
+ return escaped == null ? undefined : `${name}="${escaped}"`;
66
+ })
67
+ .filter(Boolean)
68
+ .join(" ");
69
+ const activeAgentTag = `<active_agent ${activeAgentAttrs}/>\n\n`;
70
+
71
+ const envBlock = `# Environment
72
+ Working directory: ${cwd}
73
+ ${env.isGitRepo ? `Git repository: yes\nBranch: ${env.branch}` : "Not a git repository"}
74
+ Platform: ${env.platform}`;
75
+
76
+ // Build optional extras suffix
77
+ const extraSections: string[] = [];
78
+ if (extras?.memoryBlock) {
79
+ extraSections.push(extras.memoryBlock);
80
+ }
81
+ if (extras?.skillBlocks?.length) {
82
+ for (const skill of extras.skillBlocks) {
83
+ extraSections.push(`\n# Preloaded Skill: ${skill.name}\n${skill.content}`);
84
+ }
85
+ }
86
+ const extrasSuffix = extraSections.length > 0 ? "\n\n" + extraSections.join("\n") : "";
87
+
88
+ if (config.promptMode === "append") {
89
+ const identity = parentSystemPrompt || genericBase;
90
+
91
+ const bridge = `<sub_agent_context>
92
+ You are operating as a sub-agent invoked to handle a specific task.
93
+ - Use the read tool instead of cat/head/tail
94
+ - Use the edit tool instead of sed/awk
95
+ - Use the write tool instead of echo/heredoc
96
+ - Use the find tool instead of bash find/ls for file search
97
+ - Use the grep tool instead of bash grep/rg for content search
98
+ - Make independent tool calls in parallel
99
+ - Use absolute file paths
100
+ - Do not use emojis
101
+ - Be concise but complete
102
+ </sub_agent_context>`;
103
+
104
+ const customSection = config.systemPrompt?.trim()
105
+ ? `\n\n<agent_instructions>\n${config.systemPrompt}\n</agent_instructions>`
106
+ : "";
107
+
108
+ // Place shared/stable content first so the LLM's KV cache can reuse the
109
+ // inherited prefix across all subagent invocations. The parent prompt is
110
+ // placed verbatim (no wrapper tag) so it forms an identical byte prefix
111
+ // with the parent session, maximising KV cache hits. The <active_agent>
112
+ // tag and env block vary per call and are placed after the cached prefix.
113
+ return identity + "\n\n" + bridge + "\n\n" + activeAgentTag + envBlock + customSection + extrasSuffix;
114
+ }
115
+
116
+ // "replace" mode — env header + the config's full system prompt
117
+ const replaceHeader = `You are a pi coding agent sub-agent.
118
+ You have been invoked to handle a specific task autonomously.
119
+
120
+ ${envBlock}`;
121
+
122
+ return activeAgentTag + replaceHeader + "\n\n" + config.systemPrompt + extrasSuffix;
123
+ }
124
+
125
+ /** Fallback base prompt when parent system prompt is unavailable in append mode. */
126
+ const genericBase = `# Role
127
+ You are a general-purpose coding agent for complex, multi-step tasks.
128
+ You have full access to read, write, edit files, and execute commands.
129
+ Do what has been asked; nothing more, nothing less.`;
@@ -0,0 +1,153 @@
1
+ /**
2
+ * schedule-store.ts — File-backed store for scheduled subagents.
3
+ *
4
+ * Session-scoped: each pi session owns its own schedules at
5
+ * `<cwd>/.pi/subagent-schedules/<sessionId>.json`. `/new` starts a fresh
6
+ * empty store; `/resume` reloads.
7
+ *
8
+ * Concurrency model lifted from pi-chonky-tasks/src/task-store.ts: every
9
+ * mutation acquires a PID-based exclusion lock, re-reads the latest state
10
+ * from disk, applies the change, atomic-writes via temp+rename, releases.
11
+ */
12
+
13
+ import { existsSync, mkdirSync, readFileSync, renameSync, unlinkSync, writeFileSync } from "node:fs";
14
+ import { dirname, join } from "node:path";
15
+ import type { ScheduledSubagent, ScheduleStoreData } from "./types.js";
16
+
17
+ const LOCK_RETRY_MS = 50;
18
+ const LOCK_MAX_RETRIES = 100;
19
+
20
+ function isProcessRunning(pid: number): boolean {
21
+ try { process.kill(pid, 0); return true; } catch { return false; }
22
+ }
23
+
24
+ function acquireLock(lockPath: string): void {
25
+ for (let i = 0; i < LOCK_MAX_RETRIES; i++) {
26
+ try {
27
+ writeFileSync(lockPath, `${process.pid}`, { flag: "wx" });
28
+ return;
29
+ } catch (e: any) {
30
+ if (e.code === "EEXIST") {
31
+ try {
32
+ const pid = parseInt(readFileSync(lockPath, "utf-8"), 10);
33
+ if (pid && !isProcessRunning(pid)) {
34
+ unlinkSync(lockPath);
35
+ continue;
36
+ }
37
+ } catch { /* ignore — try again */ }
38
+ const start = Date.now();
39
+ while (Date.now() - start < LOCK_RETRY_MS) { /* busy wait */ }
40
+ continue;
41
+ }
42
+ throw e;
43
+ }
44
+ }
45
+ throw new Error(`Failed to acquire schedule lock: ${lockPath}`);
46
+ }
47
+
48
+ function releaseLock(lockPath: string): void {
49
+ try { unlinkSync(lockPath); } catch { /* ignore */ }
50
+ }
51
+
52
+ /** Resolve the storage path for a session-scoped store. */
53
+ export function resolveStorePath(cwd: string, sessionId: string): string {
54
+ return join(cwd, ".pi", "subagent-schedules", `${sessionId}.json`);
55
+ }
56
+
57
+ export class ScheduleStore {
58
+ private filePath: string;
59
+ private lockPath: string;
60
+ private jobs = new Map<string, ScheduledSubagent>();
61
+
62
+ constructor(filePath: string) {
63
+ this.filePath = filePath;
64
+ this.lockPath = filePath + ".lock";
65
+ this.load();
66
+ }
67
+
68
+ /** Create the backing directory lazily — only when we're about to persist. */
69
+ private ensureDir(): void {
70
+ mkdirSync(dirname(this.filePath), { recursive: true });
71
+ }
72
+
73
+ /** Load from disk into the in-memory cache. Silent on parse errors. */
74
+ private load(): void {
75
+ if (!existsSync(this.filePath)) return;
76
+ try {
77
+ const data: ScheduleStoreData = JSON.parse(readFileSync(this.filePath, "utf-8"));
78
+ this.jobs.clear();
79
+ for (const j of data.jobs ?? []) this.jobs.set(j.id, j);
80
+ } catch { /* corrupt — start fresh, next save rewrites */ }
81
+ }
82
+
83
+ /** Atomic write via temp file + rename (POSIX-atomic). */
84
+ private save(): void {
85
+ const data: ScheduleStoreData = { version: 1, jobs: [...this.jobs.values()] };
86
+ const tmp = this.filePath + ".tmp";
87
+ writeFileSync(tmp, JSON.stringify(data, null, 2));
88
+ renameSync(tmp, this.filePath);
89
+ }
90
+
91
+ /** Acquire lock → reload → mutate → save → release. */
92
+ private withLock<T>(fn: () => T): T {
93
+ this.ensureDir();
94
+ acquireLock(this.lockPath);
95
+ try {
96
+ this.load();
97
+ const result = fn();
98
+ this.save();
99
+ return result;
100
+ } finally {
101
+ releaseLock(this.lockPath);
102
+ }
103
+ }
104
+
105
+ /** Read-only — returns a snapshot of the in-memory cache. */
106
+ list(): ScheduledSubagent[] {
107
+ return [...this.jobs.values()];
108
+ }
109
+
110
+ /** Read-only check — uses the cache. */
111
+ hasName(name: string, exceptId?: string): boolean {
112
+ for (const j of this.jobs.values()) {
113
+ if (j.id !== exceptId && j.name === name) return true;
114
+ }
115
+ return false;
116
+ }
117
+
118
+ get(id: string): ScheduledSubagent | undefined {
119
+ return this.jobs.get(id);
120
+ }
121
+
122
+ add(job: ScheduledSubagent): void {
123
+ this.withLock(() => {
124
+ this.jobs.set(job.id, job);
125
+ });
126
+ }
127
+
128
+ update(id: string, patch: Partial<ScheduledSubagent>): ScheduledSubagent | undefined {
129
+ // No-op fast path — an unknown id changes nothing, so don't lock or touch
130
+ // disk (which would otherwise lazily create the backing directory).
131
+ if (!this.jobs.has(id)) return undefined;
132
+ return this.withLock(() => {
133
+ const existing = this.jobs.get(id);
134
+ if (!existing) return undefined;
135
+ const updated = { ...existing, ...patch };
136
+ this.jobs.set(id, updated);
137
+ return updated;
138
+ });
139
+ }
140
+
141
+ remove(id: string): boolean {
142
+ // No-op fast path — see update().
143
+ if (!this.jobs.has(id)) return false;
144
+ return this.withLock(() => this.jobs.delete(id));
145
+ }
146
+
147
+ /** Delete the backing file (used when no jobs remain, optional cleanup). */
148
+ deleteFileIfEmpty(): void {
149
+ if (this.jobs.size === 0 && existsSync(this.filePath)) {
150
+ try { unlinkSync(this.filePath); } catch { /* ignore */ }
151
+ }
152
+ }
153
+ }
@@ -0,0 +1,365 @@
1
+ /**
2
+ * schedule.ts — `SubagentScheduler`: timer-driven dispatcher of scheduled subagents.
3
+ *
4
+ * Mirrors the engine shape of pi-cron-schedule/src/scheduler.ts:
5
+ * - two-Map split (jobs = croner Cron, intervals = setInterval/setTimeout)
6
+ * - addJob/removeJob/updateJob/scheduleJob/unscheduleJob/executeJob
7
+ * - static parsers for cron / "+10m" / "5m" / ISO formats
8
+ *
9
+ * Differences vs pi-cron-schedule:
10
+ * - Persistence is via ScheduleStore (PID-locked, session-scoped, atomic).
11
+ * - `executeJob` calls `manager.spawn(..., { bypassQueue: true })` instead
12
+ * of dispatching a user message — schedule fires bypass maxConcurrent so
13
+ * a 5-minute interval can't be deferred behind 4 long-running agents.
14
+ * - Result delivery is implicit: spawn → background completion → existing
15
+ * steering-style `subagent-notification` path. No new delivery code.
16
+ */
17
+
18
+ import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
19
+ import { Cron } from "croner";
20
+ import { nanoid } from "nanoid";
21
+ import type { AgentManager } from "./agent-manager.js";
22
+ import { resolveModel } from "./model-resolver.js";
23
+ import type { ScheduleStore } from "./schedule-store.js";
24
+ import type { IsolationMode, ScheduledSubagent, SubagentType, ThinkingLevel } from "./types.js";
25
+
26
+ /** Event emitted on `pi.events` for cross-extension consumers. */
27
+ export type ScheduleChangeEvent =
28
+ | { type: "added"; job: ScheduledSubagent }
29
+ | { type: "removed"; jobId: string }
30
+ | { type: "updated"; job: ScheduledSubagent }
31
+ | { type: "fired"; jobId: string; agentId: string; name: string }
32
+ | { type: "error"; jobId: string; error: string };
33
+
34
+ /** Params accepted at job creation — ID, timestamps, and state are derived. */
35
+ export interface NewJobInput {
36
+ name: string;
37
+ description: string;
38
+ schedule: string;
39
+ subagent_type: SubagentType;
40
+ prompt: string;
41
+ model?: string;
42
+ thinking?: ThinkingLevel;
43
+ max_turns?: number;
44
+ isolated?: boolean;
45
+ isolation?: IsolationMode;
46
+ }
47
+
48
+ export class SubagentScheduler {
49
+ private jobs = new Map<string, Cron>();
50
+ private intervals = new Map<string, NodeJS.Timeout>();
51
+ private store: ScheduleStore | undefined;
52
+ private pi: ExtensionAPI | undefined;
53
+ private ctx: ExtensionContext | undefined;
54
+ private manager: AgentManager | undefined;
55
+
56
+ /** Start the scheduler: bind to a session's store and arm enabled jobs. */
57
+ start(pi: ExtensionAPI, ctx: ExtensionContext, manager: AgentManager, store: ScheduleStore): void {
58
+ this.pi = pi;
59
+ this.ctx = ctx;
60
+ this.manager = manager;
61
+ this.store = store;
62
+
63
+ for (const job of store.list()) {
64
+ if (job.enabled) this.scheduleJob(job);
65
+ }
66
+ }
67
+
68
+ /** Stop all timers; drop refs. Safe to call repeatedly. */
69
+ stop(): void {
70
+ for (const cron of this.jobs.values()) cron.stop();
71
+ this.jobs.clear();
72
+ for (const t of this.intervals.values()) clearTimeout(t);
73
+ this.intervals.clear();
74
+ this.store = undefined;
75
+ this.pi = undefined;
76
+ this.ctx = undefined;
77
+ this.manager = undefined;
78
+ }
79
+
80
+ /** True if start() has bound a store and the scheduler is active. */
81
+ isActive(): boolean {
82
+ return this.store !== undefined;
83
+ }
84
+
85
+ list(): ScheduledSubagent[] {
86
+ return this.store?.list() ?? [];
87
+ }
88
+
89
+ /**
90
+ * Build a `ScheduledSubagent` from user input. Validates the schedule
91
+ * format and tags `scheduleType`. Throws on invalid input.
92
+ */
93
+ buildJob(input: NewJobInput): ScheduledSubagent {
94
+ const detected = SubagentScheduler.detectSchedule(input.schedule);
95
+ return {
96
+ id: nanoid(10),
97
+ name: input.name,
98
+ description: input.description,
99
+ schedule: detected.normalized,
100
+ scheduleType: detected.type,
101
+ intervalMs: detected.intervalMs,
102
+ subagent_type: input.subagent_type,
103
+ prompt: input.prompt,
104
+ model: input.model,
105
+ thinking: input.thinking,
106
+ max_turns: input.max_turns,
107
+ isolated: input.isolated,
108
+ isolation: input.isolation,
109
+ enabled: true,
110
+ createdAt: new Date().toISOString(),
111
+ runCount: 0,
112
+ };
113
+ }
114
+
115
+ /** Add a job, persist, and arm if enabled. Returns the stored job. */
116
+ addJob(input: NewJobInput): ScheduledSubagent {
117
+ const store = this.requireStore();
118
+ if (store.hasName(input.name)) {
119
+ throw new Error(`A scheduled job named "${input.name}" already exists.`);
120
+ }
121
+ const job = this.buildJob(input);
122
+ store.add(job);
123
+ if (job.enabled) this.scheduleJob(job);
124
+ this.emit({ type: "added", job });
125
+ return job;
126
+ }
127
+
128
+ removeJob(id: string): boolean {
129
+ const store = this.requireStore();
130
+ if (!store.get(id)) return false;
131
+ this.unscheduleJob(id);
132
+ const ok = store.remove(id);
133
+ if (ok) this.emit({ type: "removed", jobId: id });
134
+ return ok;
135
+ }
136
+
137
+ /** Toggle / mutate a job. Re-arms based on the new `enabled` state. */
138
+ updateJob(id: string, patch: Partial<ScheduledSubagent>): ScheduledSubagent | undefined {
139
+ const store = this.requireStore();
140
+ const updated = store.update(id, patch);
141
+ if (!updated) return undefined;
142
+ this.unscheduleJob(id);
143
+ if (updated.enabled) this.scheduleJob(updated);
144
+ this.emit({ type: "updated", job: updated });
145
+ return updated;
146
+ }
147
+
148
+ /** Next-run time as ISO, or undefined if not currently armed. */
149
+ getNextRun(jobId: string): string | undefined {
150
+ const cron = this.jobs.get(jobId);
151
+ if (cron) return cron.nextRun()?.toISOString();
152
+ const job = this.store?.get(jobId);
153
+ if (!job?.enabled) return undefined;
154
+ if (job.scheduleType === "once") return job.schedule;
155
+ if (job.scheduleType === "interval" && job.intervalMs) {
156
+ // Before the first fire there's no `lastRun`, so fall back to "now" —
157
+ // accurate at create time (setInterval was just armed) and within
158
+ // intervalMs of correct in any pre-first-fire view.
159
+ const base = job.lastRun ? new Date(job.lastRun).getTime() : Date.now();
160
+ return new Date(base + job.intervalMs).toISOString();
161
+ }
162
+ return undefined;
163
+ }
164
+
165
+ // ── Scheduling primitives ────────────────────────────────────────────
166
+
167
+ private scheduleJob(job: ScheduledSubagent): void {
168
+ const store = this.store;
169
+ if (!store) return;
170
+ try {
171
+ if (job.scheduleType === "interval" && job.intervalMs) {
172
+ const t = setInterval(() => this.executeJob(job.id), job.intervalMs);
173
+ this.intervals.set(job.id, t);
174
+ } else if (job.scheduleType === "once") {
175
+ const target = new Date(job.schedule).getTime();
176
+ const delay = target - Date.now();
177
+ if (delay > 0) {
178
+ const t = setTimeout(() => {
179
+ this.executeJob(job.id);
180
+ // Auto-disable one-shots after they fire (mirrors pi-cron-schedule)
181
+ store.update(job.id, { enabled: false });
182
+ const updated = store.get(job.id);
183
+ if (updated) this.emit({ type: "updated", job: updated });
184
+ }, delay);
185
+ this.intervals.set(job.id, t);
186
+ } else {
187
+ // Past timestamp — disable, mark error, never fire
188
+ store.update(job.id, { enabled: false, lastStatus: "error" });
189
+ this.emit({ type: "error", jobId: job.id, error: `Scheduled time ${job.schedule} is in the past` });
190
+ }
191
+ } else {
192
+ const cron = new Cron(job.schedule, () => this.executeJob(job.id));
193
+ this.jobs.set(job.id, cron);
194
+ }
195
+ } catch (err) {
196
+ this.emit({ type: "error", jobId: job.id, error: err instanceof Error ? err.message : String(err) });
197
+ }
198
+ }
199
+
200
+ private unscheduleJob(id: string): void {
201
+ const cron = this.jobs.get(id);
202
+ if (cron) {
203
+ cron.stop();
204
+ this.jobs.delete(id);
205
+ }
206
+ const t = this.intervals.get(id);
207
+ if (t) {
208
+ clearTimeout(t);
209
+ clearInterval(t);
210
+ this.intervals.delete(id);
211
+ }
212
+ }
213
+
214
+ /**
215
+ * Fire a job: persist running state, spawn (bypassing the concurrency
216
+ * queue), persist completion. Fire-and-forget: the timer tick returns
217
+ * immediately so other jobs keep firing.
218
+ */
219
+ private executeJob(id: string): void {
220
+ const store = this.store;
221
+ const pi = this.pi;
222
+ const ctx = this.ctx;
223
+ const manager = this.manager;
224
+ if (!store || !pi || !ctx || !manager) return;
225
+ const job = store.get(id);
226
+ if (!job?.enabled) return;
227
+
228
+ store.update(id, { lastStatus: "running" });
229
+
230
+ // Resolve model at fire time — registry contents may have changed since the
231
+ // job was created (auth added/removed). Fall back silently to spawn-default
232
+ // if resolution fails; the spawn path handles undefined model gracefully.
233
+ let resolvedModel: any | undefined;
234
+ if (job.model) {
235
+ const r = resolveModel(job.model, ctx.modelRegistry);
236
+ if (typeof r !== "string") resolvedModel = r;
237
+ }
238
+
239
+ let agentId: string;
240
+ try {
241
+ agentId = manager.spawn(pi, ctx, job.subagent_type, job.prompt, {
242
+ description: job.description,
243
+ isBackground: true,
244
+ bypassQueue: true,
245
+ model: resolvedModel,
246
+ maxTurns: job.max_turns,
247
+ isolated: job.isolated,
248
+ thinkingLevel: job.thinking,
249
+ isolation: job.isolation,
250
+ });
251
+ } catch (err) {
252
+ const error = err instanceof Error ? err.message : String(err);
253
+ store.update(id, { lastRun: new Date().toISOString(), lastStatus: "error" });
254
+ this.emit({ type: "error", jobId: id, error });
255
+ return;
256
+ }
257
+
258
+ this.emit({ type: "fired", jobId: id, agentId, name: job.name });
259
+
260
+ const record = manager.getRecord(agentId);
261
+ const finalize = (status: "success" | "error") => {
262
+ const next = this.getNextRun(id);
263
+ const current = store.get(id);
264
+ store.update(id, {
265
+ lastRun: new Date().toISOString(),
266
+ lastStatus: status,
267
+ runCount: (current?.runCount ?? 0) + 1,
268
+ nextRun: next,
269
+ });
270
+ };
271
+
272
+ // AgentManager's promise resolves either way (its .catch returns ""), so we
273
+ // can't infer success/failure from the promise — read record.status instead.
274
+ // Terminal states: completed/steered = success; error/aborted/stopped = error.
275
+ if (record?.promise) {
276
+ record.promise
277
+ .then(() => {
278
+ const r = manager.getRecord(agentId);
279
+ const failed = r?.status === "error" || r?.status === "aborted" || r?.status === "stopped";
280
+ finalize(failed ? "error" : "success");
281
+ })
282
+ .catch(() => finalize("error"));
283
+ } else {
284
+ // Spawn returned without a promise (defensive — bypassQueue path always sets one).
285
+ finalize("success");
286
+ }
287
+ }
288
+
289
+ private emit(event: ScheduleChangeEvent): void {
290
+ if (this.pi) this.pi.events.emit("subagents:scheduled", event);
291
+ }
292
+
293
+ private requireStore(): ScheduleStore {
294
+ if (!this.store) throw new Error("Scheduler not started — no active session.");
295
+ return this.store;
296
+ }
297
+
298
+ // ── Format detection / parsers (statics — pure) ──────────────────────
299
+
300
+ /**
301
+ * Sniff a schedule string and tag its type. Throws on invalid input.
302
+ * Order matters: relative ("+10m") and interval ("5m") both match digit+unit;
303
+ * relative requires the leading "+" to disambiguate.
304
+ */
305
+ static detectSchedule(s: string): { type: "cron" | "once" | "interval"; intervalMs?: number; normalized: string } {
306
+ const trimmed = s.trim();
307
+ // "+10m" — relative one-shot
308
+ const rel = SubagentScheduler.parseRelativeTime(trimmed);
309
+ if (rel !== null) return { type: "once", normalized: rel };
310
+ // "5m" — interval
311
+ const ivl = SubagentScheduler.parseInterval(trimmed);
312
+ if (ivl !== null) return { type: "interval", intervalMs: ivl, normalized: trimmed };
313
+ // ISO timestamp — one-shot. Reject past timestamps upfront so we never
314
+ // create a dead-on-arrival record (scheduleJob's safety net still catches
315
+ // micro-races from `+0s`-style relatives).
316
+ if (/^\d{4}-\d{2}-\d{2}T/.test(trimmed)) {
317
+ const d = new Date(trimmed);
318
+ if (!Number.isNaN(d.getTime())) {
319
+ if (d.getTime() <= Date.now()) {
320
+ throw new Error(`Scheduled time ${d.toISOString()} is in the past.`);
321
+ }
322
+ return { type: "once", normalized: d.toISOString() };
323
+ }
324
+ }
325
+ // Cron — 6-field
326
+ const cronCheck = SubagentScheduler.validateCronExpression(trimmed);
327
+ if (cronCheck.valid) return { type: "cron", normalized: trimmed };
328
+ throw new Error(
329
+ `Invalid schedule "${s}". Use 6-field cron (e.g. "0 0 9 * * 1" — 9am every Monday), interval ("5m"/"1h"), or one-shot ("+10m" / ISO).`
330
+ );
331
+ }
332
+
333
+ /** 6-field cron — 'second minute hour dom month dow'. */
334
+ static validateCronExpression(expr: string): { valid: boolean; error?: string } {
335
+ const fields = expr.trim().split(/\s+/);
336
+ if (fields.length !== 6) {
337
+ return {
338
+ valid: false,
339
+ error: `Cron must have 6 fields (second minute hour dom month dow), got ${fields.length}. Example: "0 0 9 * * 1" for 9am every Monday.`,
340
+ };
341
+ }
342
+ try {
343
+ // Croner validates by construction.
344
+ new Cron(expr, () => {});
345
+ return { valid: true };
346
+ } catch (e) {
347
+ return { valid: false, error: e instanceof Error ? e.message : "Invalid cron expression" };
348
+ }
349
+ }
350
+
351
+ /** "+10s"/"+5m"/"+1h"/"+2d" → ISO timestamp. */
352
+ static parseRelativeTime(s: string): string | null {
353
+ const m = s.match(/^\+(\d+)(s|m|h|d)$/);
354
+ if (!m) return null;
355
+ const ms = parseInt(m[1], 10) * { s: 1000, m: 60_000, h: 3_600_000, d: 86_400_000 }[m[2] as "s" | "m" | "h" | "d"];
356
+ return new Date(Date.now() + ms).toISOString();
357
+ }
358
+
359
+ /** "10s"/"5m"/"1h"/"2d" → milliseconds. */
360
+ static parseInterval(s: string): number | null {
361
+ const m = s.match(/^(\d+)(s|m|h|d)$/);
362
+ if (!m) return null;
363
+ return parseInt(m[1], 10) * { s: 1000, m: 60_000, h: 3_600_000, d: 86_400_000 }[m[2] as "s" | "m" | "h" | "d"];
364
+ }
365
+ }