@byfriends/sdk 0.2.3 → 0.2.5

package/dist/index.d.mts

@@ -393,6 +393,7 @@ export declare type BackgroundConfig = z.infer<typeof BackgroundConfigSchema>;
 
 declare const BackgroundConfigSchema: z.ZodObject<{
     maxRunningTasks: z.ZodOptional<z.ZodNumber>;
+    maxConcurrentSubagents: z.ZodOptional<z.ZodNumber>;
     keepAliveOnExit: z.ZodOptional<z.ZodBoolean>;
     killGracePeriodMs: z.ZodOptional<z.ZodNumber>;
     agentTaskTimeoutS: z.ZodOptional<z.ZodNumber>;
@@ -1217,6 +1218,7 @@ declare const ByfConfigPatchSchema: z.ZodObject<{
     }, z.core.$strip>>;
     background: z.ZodOptional<z.ZodObject<{
         maxRunningTasks: z.ZodOptional<z.ZodOptional<z.ZodNumber>>;
+        maxConcurrentSubagents: z.ZodOptional<z.ZodOptional<z.ZodNumber>>;
         keepAliveOnExit: z.ZodOptional<z.ZodOptional<z.ZodBoolean>>;
         killGracePeriodMs: z.ZodOptional<z.ZodOptional<z.ZodNumber>>;
         agentTaskTimeoutS: z.ZodOptional<z.ZodOptional<z.ZodNumber>>;
@@ -1354,6 +1356,7 @@ declare const ByfConfigSchema: z.ZodObject<{
     }, z.core.$strip>>;
     background: z.ZodOptional<z.ZodObject<{
         maxRunningTasks: z.ZodOptional<z.ZodNumber>;
+        maxConcurrentSubagents: z.ZodOptional<z.ZodNumber>;
         keepAliveOnExit: z.ZodOptional<z.ZodBoolean>;
         killGracePeriodMs: z.ZodOptional<z.ZodNumber>;
         agentTaskTimeoutS: z.ZodOptional<z.ZodNumber>;
@@ -4384,10 +4387,13 @@ declare class SessionSubagentHost {
     private readonly ownerAgentId;
     readonly backgroundTaskTimeoutMs?: number | undefined;
     private readonly activeChildren;
-    constructor(session: Session_2, ownerAgentId: string, backgroundTaskTimeoutMs?: number | undefined);
+    /** Maximum concurrent subagents; exposed for testing and config diagnostics. */
+    readonly maxConcurrentSubagents: number;
+    constructor(session: Session_2, ownerAgentId: string, backgroundTaskTimeoutMs?: number | undefined, maxConcurrentSubagents?: number | undefined);
     spawn(profileName: string, options: RunSubagentOptions): Promise<SubagentHandle>;
     resume(agentId: string, options: RunSubagentOptions): Promise<SubagentHandle>;
     cancelAll(): void;
+    private assertCanSpawn;
     getProfileName(agentId: string): string | undefined;
     private resolveProfile;
     private runChild;

package/dist/index.mjs

@@ -56166,7 +56166,7 @@ var agent_background_disabled_default = "Background agent execution is disabled
 var agent_background_enabled_default = "When `run_in_background=true`, the subagent runs detached from this turn. The completion arrives in a later turn as a synthetic user-role message containing its result — you do not need to poll, sleep, or check on its progress. Continue with other work or respond to the user. Never fabricate or predict what the result will say.\n\nFor a background task, when `timeout` is omitted it falls back to the operator-configured background timeout, if one is set. If the operator has not configured a background timeout, an omitted `timeout` means the task runs with no time limit.\n";
 //#endregion
 //#region ../agent-core/src/tools/builtin/collaboration/agent.md
-var agent_default = "Launch a subagent to handle a task. The subagent starts with zero context — it has not seen this conversation. Brief it with the goal, what you already know, and exact paths or commands.\n\n- When the task continues earlier work a subagent already did, prefer resuming that agent (pass its `resume` id) over spawning a fresh instance.\n- A subagent's result is only visible to you, not to the user. Summarize the relevant parts yourself when the user needs to see them.\n- Skip delegation for trivial work you can do directly — reading a known file, searching a small set of files, or any one-step task.\n- Once a subagent is running, do not redo its searches or reads in parallel, and do not abandon it midway and finish the job manually.\n";
+var agent_default = "Launch a subagent to handle a task. The subagent starts with zero context — it has not seen this conversation. Brief it with the goal, what you already know, and exact paths or commands.\n\n- When the task continues earlier work a subagent already did, prefer resuming that agent (pass its `resume` id) over spawning a fresh instance.\n- A subagent's result is only visible to you, not to the user. Summarize the relevant parts yourself when the user needs to see them.\n- Skip delegation for trivial work you can do directly — reading a known file, searching a small set of files, or any one-step task.\n- Once a subagent is running, do not redo its searches or reads in parallel, and do not abandon it midway and finish the job manually.\n- There is a concurrency limit on parallel subagents. If the limit is reached, wait for a running subagent to complete before launching another.\n";
 //#endregion
 //#region ../agent-core/src/tools/builtin/collaboration/agent.ts
 /**
@@ -70841,7 +70841,7 @@ var SkillManager = class {
 		const skill = this.registry.getSkill(input.name);
 		if (skill === void 0) throw new ByfError(ErrorCodes.SKILL_NOT_FOUND, `Skill "${input.name}" was not found`);
 		if (!isUserActivatableSkillType(skill.metadata.type)) throw new ByfError(ErrorCodes.SKILL_TYPE_UNSUPPORTED, `Skill "${skill.name}" cannot be activated by the user`);
-		this.recordActivation({
+		const origin = {
 			kind: "skill_activation",
 			activationId: randomUUID(),
 			skillName: skill.name,
@@ -70850,10 +70850,13 @@ var SkillManager = class {
 			skillPath: skill.path,
 			skillSource: skill.source,
 			skillArgs: input.args
-		}, [{
+		};
+		const skillContent = this.registry.renderSkillPrompt(skill, input.args ?? "");
+		this.recordActivation(origin, [{
 			type: "text",
-			text: this.registry.renderSkillPrompt(skill, input.args ?? "")
+			text: skillContent
 		}]);
+		this.agent.context.appendSystemReminder(`<byf-skill-loaded name="${skill.name}">\n${skillContent}\n</byf-skill-loaded>`, origin);
 	}
 	recordActivation(origin, input) {
 		this.agent.emitEvent({
@@ -78851,9 +78854,8 @@ function createChatStreamingCallbacks(deps) {
 * enforcement, usage aggregation, optional continuation after non-tool stops,
 * and final `TurnResult` mapping. One-step execution lives in `turn-step.ts`.
 */
-const DEFAULT_MAX_STEPS = 1e3;
 async function runTurn(input) {
-	const { turnId, signal, llm, buildMessages, dispatchEvent, tools, hooks, log, maxSteps = DEFAULT_MAX_STEPS, maxRetryAttempts } = input;
+	const { turnId, signal, llm, buildMessages, dispatchEvent, tools, hooks, log, maxSteps, maxRetryAttempts } = input;
 	let usage = emptyUsage();
 	let steps = 0;
 	let stopReason = "end_turn";
@@ -78864,7 +78866,7 @@ async function runTurn(input) {
 	try {
 		while (true) {
 			signal.throwIfAborted();
-			if (steps >= maxSteps) throw createMaxStepsExceededError(maxSteps);
+			if (maxSteps !== void 0 && steps >= maxSteps) throw createMaxStepsExceededError(maxSteps);
 			steps += 1;
 			activeStep = steps;
 			const stepResult = await executeLoopStep({
@@ -84517,7 +84519,7 @@ var Agent = class {
 		this.usage = new UsageRecorder(this);
 		this.tools = new ToolManager(this);
 		this.background = new BackgroundManager(this, {
-			maxRunningTasks: config.backgroundMaxRunningTasks,
+			maxRunningTasks: config.backgroundMaxRunningTasks ?? 10,
 			sessionDir: config.backgroundSessionDir
 		});
 		this.replayBuilder = new ReplayBuilder(this);
@@ -84889,6 +84891,7 @@ const LoopControlSchema = z.object({
 });
 const BackgroundConfigSchema = z.object({
 	maxRunningTasks: z.number().int().min(1).optional(),
+	maxConcurrentSubagents: z.number().int().min(1).optional(),
 	keepAliveOnExit: z.boolean().optional(),
 	killGracePeriodMs: z.number().int().min(0).optional(),
 	agentTaskTimeoutS: z.number().int().min(1).optional(),
@@ -89035,18 +89038,27 @@ const SUMMARY_MAX_LENGTH = 8e3;
 const SUMMARY_CONTINUATION_ATTEMPTS = 1;
 const HOOK_TEXT_PREVIEW_LENGTH = 500;
 const SUBAGENT_MAX_TOKENS_ERROR = "Subagent turn failed before completing its final summary: reason=max_tokens";
+/**
+* Maximum number of concurrently running subagents per parent.
+* Prevents cascading subagent proliferation that exhausts LLM bandwidth.
+*/
+const DEFAULT_MAX_CONCURRENT_SUBAGENTS = 5;
 var SessionSubagentHost = class {
 	session;
 	ownerAgentId;
 	backgroundTaskTimeoutMs;
 	activeChildren = /* @__PURE__ */ new Map();
-	constructor(session, ownerAgentId, backgroundTaskTimeoutMs) {
+	/** Maximum concurrent subagents; exposed for testing and config diagnostics. */
+	maxConcurrentSubagents;
+	constructor(session, ownerAgentId, backgroundTaskTimeoutMs, maxConcurrentSubagents = void 0) {
 		this.session = session;
 		this.ownerAgentId = ownerAgentId;
 		this.backgroundTaskTimeoutMs = backgroundTaskTimeoutMs;
+		this.maxConcurrentSubagents = maxConcurrentSubagents ?? DEFAULT_MAX_CONCURRENT_SUBAGENTS;
 	}
 	async spawn(profileName, options) {
 		options.signal.throwIfAborted();
+		this.assertCanSpawn();
 		const parent = this.session.agents.get(this.ownerAgentId);
 		if (parent === void 0) throw new Error(`Parent agent "${this.ownerAgentId}" was not found`);
 		const profile = this.resolveProfile(parent, profileName);
@@ -89076,6 +89088,7 @@ var SessionSubagentHost = class {
 	}
 	async resume(agentId, options) {
 		options.signal.throwIfAborted();
+		this.assertCanSpawn();
 		const parent = this.session.agents.get(this.ownerAgentId);
 		if (parent === void 0) throw new Error(`Parent agent "${this.ownerAgentId}" was not found`);
 		const child = this.session.agents.get(agentId);
@@ -89114,6 +89127,9 @@ var SessionSubagentHost = class {
 			child.controller.abort();
 		}
 	}
+	assertCanSpawn() {
+		if (this.activeChildren.size >= this.maxConcurrentSubagents) throw new Error(`Too many concurrent subagents (${this.activeChildren.size} running, maximum ${this.maxConcurrentSubagents}).`);
+	}
 	getProfileName(agentId) {
 		const metadata = this.session.metadata.agents[agentId];
 		if (metadata?.type !== "sub" || metadata.parentAgentId !== this.ownerAgentId) return;
@@ -89511,7 +89527,7 @@ var Session$1 = class {
 			providerManager: this.config.providerManager,
 			sessionId: this.config.id,
 			hookEngine: config.hookEngine ?? this.hookEngine,
-			subagentHost: config.subagentHost ?? new SessionSubagentHost(this, id, this.backgroundTaskTimeoutMs()),
+			subagentHost: config.subagentHost ?? new SessionSubagentHost(this, id, this.backgroundTaskTimeoutMs(), this.config.background?.maxConcurrentSubagents),
 			mcp: this.mcp,
 			backgroundMaxRunningTasks: this.config.background?.maxRunningTasks,
 			backgroundSessionDir: homedir,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@byfriends/sdk",
-  "version": "0.2.3",
+  "version": "0.2.5",
   "description": "Public TypeScript SDK for BYF agents and sessions",
   "license": "Proprietary",
   "author": "ByronFinn",
@@ -49,10 +49,10 @@
   },
   "devDependencies": {
     "@types/yazl": "^2.4.6",
-    "@byfriends/kosong": "^0.2.2",
+    "@byfriends/agent-core": "^0.2.5",
     "@byfriends/oauth": "^0.1.1",
-    "@byfriends/kaos": "^0.2.2",
-    "@byfriends/agent-core": "^0.2.3"
+    "@byfriends/kosong": "^0.2.2",
+    "@byfriends/kaos": "^0.2.2"
   },
   "scripts": {
     "build": "tsdown && pnpm run build:dts",