pi-crew 0.9.16 → 0.9.17

package/src/config/defaults.ts

@@ -30,7 +30,7 @@ export const DEFAULT_CHILD_PI: Readonly<{
 
 export const DEFAULT_LIVE_SESSION = {
 	/** Maximum wall-clock time for a single live-session task before abort (ms). */
-	responseTimeoutMs: 10 * 60_000,  // 10 minutes - increased from 5min for complex verification
+	responseTimeoutMs: 10 * 60_000, // 10 minutes - increased from 5min for complex verification
 	/** Maximum yield reminder attempts before accepting no-yield. */
 	maxYieldRetries: 3,
 	/** Polling interval for session idle check during yield enforcement (ms). */
@@ -56,13 +56,39 @@ export const DEFAULT_CONCURRENCY = {
 };
 
 export const DEFAULT_EVENT_LOG = {
-	terminalEventTypes: ["run.blocked", "run.completed", "run.failed", "run.cancelled", "task.completed", "task.failed", "task.skipped", "task.cancelled", "task.needs_attention"],
+	terminalEventTypes: [
+		"run.blocked",
+		"run.completed",
+		"run.failed",
+		"run.cancelled",
+		"task.completed",
+		"task.failed",
+		"task.skipped",
+		"task.cancelled",
+		"task.needs_attention",
+	],
 };
 
 export const DEFAULT_ARTIFACT_CLEANUP = {
 	maxAgeDays: 7,
 };
 
+/** Round 25 (L6): thresholds for task-output-context inline-bytes budget.
+ *  Single source of truth for the per-dep cap, total dep cap, and tee
+ *  recovery multiplier. Previously hardcoded in
+ *  src/runtime/task-output-context.ts; moved here so that operators can
+ *  override them via config (Phase 5 ship) without touching runtime code. */
+export const DEFAULT_OUTPUT_CONTEXT = {
+	/** Per-dep inline-bytes cap (chars). Set by plan §5 L4 (96 KB total). */
+	maxResultInlineBytes: 32_000,
+	/** Total inline-bytes budget across all deps for one downstream worker. */
+	maxTotalDepInlineBytes: 96_000,
+	/** Tee recovery threshold: only when file > TEE_THRESHOLD_MULTIPLIER *
+	 *  maxResultInlineBytes, the truncated inline is also written to a
+	 *  tee file (R2: 1.25x = 40 KB). */
+	teeThresholdMultiplier: 1.25,
+} as const;
+
 export const DEFAULT_PATHS = {
 	state: {
 		runsSubdir: "state/runs",

package/src/config/drift-detector.ts

@@ -1,5 +1,5 @@
-import type { PiTeamsConfig } from "./config.ts";
 import { PiTeamsConfigSchema } from "../schema/config-schema.ts";
+import type { PiTeamsConfig } from "./config.ts";
 
 /**
  * A single drift finding between discovered resources and config expectations.
@@ -53,9 +53,7 @@ export interface ConfigReferences {
 /**
  * Known top-level config keys allowed by the schema.
  */
-const SCHEMA_KEYS: ReadonlySet<string> = new Set<string>(
-	Object.keys(PiTeamsConfigSchema.properties ?? {}),
-);
+const SCHEMA_KEYS: ReadonlySet<string> = new Set<string>(Object.keys(PiTeamsConfigSchema.properties ?? {}));
 
 /**
  * Extract all agent, team, and workflow names referenced in a raw config object.
@@ -70,13 +68,15 @@ function extractConfigReferences(config: Record<string, unknown>): ConfigReferen
 	const workflows = new Set<string>();
 
 	// agents.overrides keys are explicit agent references
-	const agentsObj = typeof config.agents === "object" && config.agents !== null && !Array.isArray(config.agents)
-		? config.agents as Record<string, unknown>
-		: undefined;
-	if (agentsObj) {
-		const overrides = typeof agentsObj.overrides === "object" && agentsObj.overrides !== null && !Array.isArray(agentsObj.overrides)
-			? agentsObj.overrides as Record<string, unknown>
+	const agentsObj =
+		typeof config.agents === "object" && config.agents !== null && !Array.isArray(config.agents)
+			? (config.agents as Record<string, unknown>)
 			: undefined;
+	if (agentsObj) {
+		const overrides =
+			typeof agentsObj.overrides === "object" && agentsObj.overrides !== null && !Array.isArray(agentsObj.overrides)
+				? (agentsObj.overrides as Record<string, unknown>)
+				: undefined;
 		if (overrides) {
 			for (const name of Object.keys(overrides)) {
 				agents.add(name);
@@ -103,10 +103,7 @@ function extractConfigReferences(config: Record<string, unknown>): ConfigReferen
  * For teams/workflows: the config currently doesn't have explicit team/workflow references,
  * so missing/extra checks rely solely on what resources reference each other.
  */
-export function detectDrift(
-	discovered: DiscoveredResources,
-	config: PiTeamsConfig,
-): DriftReport {
+export function detectDrift(discovered: DiscoveredResources, config: PiTeamsConfig): DriftReport {
 	const items: DriftItem[] = [];
 
 	const configRaw = config as Record<string, unknown>;

package/src/config/markers.ts

@@ -10,21 +10,21 @@ export const MARKER_END = "<!-- PI-CREW:GUIDANCE:END -->";
 
 /** A single guidance section to inject between markers. */
 export interface GuidanceBlock {
-  /**
-   * Unique identifier for this guidance section (e.g. "team-tool-usage").
-   * Must match `/^[a-zA-Z0-9_-]+$/`.
-   */
-  id: string;
-  /**
-   * Markdown content to inject.
-   *
-   * **WARNING:** Content must be from trusted sources only. Guidance blocks
-   * are injected into markdown files that AI agents read. Untrusted content
-   * could contain prompt-injection payloads that manipulate agent behavior.
-   */
-  content: string;
-  /** Higher priority = appears first. Default 0. */
-  priority?: number;
+	/**
+	 * Unique identifier for this guidance section (e.g. "team-tool-usage").
+	 * Must match `/^[a-zA-Z0-9_-]+$/`.
+	 */
+	id: string;
+	/**
+	 * Markdown content to inject.
+	 *
+	 * **WARNING:** Content must be from trusted sources only. Guidance blocks
+	 * are injected into markdown files that AI agents read. Untrusted content
+	 * could contain prompt-injection payloads that manipulate agent behavior.
+	 */
+	content: string;
+	/** Higher priority = appears first. Default 0. */
+	priority?: number;
 }
 
 /** Regex for valid guidance block IDs. */
@@ -45,26 +45,26 @@ const VALID_BLOCK_ID = /^[a-zA-Z0-9_-]+$/;
  * prevent all bypasses (e.g., base64, ROT13, semantic tricks).
  */
 export function sanitizeGuidanceContent(content: string): string {
-  let sanitized = content;
-  // Strip zero-width and invisible Unicode characters (potential bypass vectors)
-  sanitized = sanitized.replace(/[\u200B-\u200F\u2028-\u202F\u2060-\u206F\uFEFF]/g, "");
-  // Strip HTML comments (potential instruction hiding)
-  sanitized = sanitized.replace(/<!--[\s\S]*?-->/g, "");
-  // Strip lines that look like system directives (e.g. "SYSTEM:", "INSTRUCTION:", "IGNORE PREVIOUS")
-  sanitized = sanitized.replace(/^\s*(?:SYSTEM|INSTRUCTION|IGNORE\s+PREVIOUS|OVERRIDE)\s*:.*$/gim, "");
-  // Collapse multiple blank lines left by removals
-  sanitized = sanitized.replace(/\n{3,}/g, "\n\n");
-  return sanitized.trim();
+	let sanitized = content;
+	// Strip zero-width and invisible Unicode characters (potential bypass vectors)
+	sanitized = sanitized.replace(/[\u200B-\u200F\u2028-\u202F\u2060-\u206F\uFEFF]/g, "");
+	// Strip HTML comments (potential instruction hiding)
+	sanitized = sanitized.replace(/<!--[\s\S]*?-->/g, "");
+	// Strip lines that look like system directives (e.g. "SYSTEM:", "INSTRUCTION:", "IGNORE PREVIOUS")
+	sanitized = sanitized.replace(/^\s*(?:SYSTEM|INSTRUCTION|IGNORE\s+PREVIOUS|OVERRIDE)\s*:.*$/gim, "");
+	// Collapse multiple blank lines left by removals
+	sanitized = sanitized.replace(/\n{3,}/g, "\n\n");
+	return sanitized.trim();
 }
 
 /** Result of an injection or removal operation. */
 export interface InjectionResult {
-  path: string;
-  modified: boolean;
-  /** Block IDs that were added or updated. */
-  added: string[];
-  /** Block IDs that were removed (content was empty). */
-  removed: string[];
+	path: string;
+	modified: boolean;
+	/** Block IDs that were added or updated. */
+	added: string[];
+	/** Block IDs that were removed (content was empty). */
+	removed: string[];
 }
 
 /**
@@ -75,10 +75,8 @@ export interface InjectionResult {
  * parse individual blocks back out.
  */
 function renderBlocks(blocks: GuidanceBlock[]): string {
-  const sorted = [...blocks].sort((a, b) => (b.priority ?? 0) - (a.priority ?? 0));
-  return sorted
-    .map((block) => `<!-- PI-CREW:BLOCK:${block.id} -->\n${block.content}\n<!-- PI-CREW:/BLOCK:${block.id} -->`)
-    .join("\n\n");
+	const sorted = [...blocks].sort((a, b) => (b.priority ?? 0) - (a.priority ?? 0));
+	return sorted.map((block) => `<!-- PI-CREW:BLOCK:${block.id} -->\n${block.content}\n<!-- PI-CREW:/BLOCK:${block.id} -->`).join("\n\n");
 }
 
 /**
@@ -101,8 +99,7 @@ export function extractGuidanceIds(content: string): string[] {
  */
 function parseExistingBlocks(inner: string): Map<string, GuidanceBlock> {
 	const map = new Map<string, GuidanceBlock>();
-	const regex =
-		/<!-- PI-CREW:BLOCK:([^\s>]+) -->\n([\s\S]*?)<!-- PI-CREW:\/BLOCK:\1 -->/g;
+	const regex = /<!-- PI-CREW:BLOCK:([^\s>]+) -->\n([\s\S]*?)<!-- PI-CREW:\/BLOCK:\1 -->/g;
 	let match: RegExpExecArray | null = regex.exec(inner);
 	while (match !== null) {
 		const id = match[1];
@@ -119,8 +116,8 @@ function parseExistingBlocks(inner: string): Map<string, GuidanceBlock> {
  * Build the full marker section string.
  */
 function buildMarkerSection(blocks: GuidanceBlock[]): string {
-  const body = renderBlocks(blocks);
-  return `\n${MARKER_START}\n${body}\n${MARKER_END}\n`;
+	const body = renderBlocks(blocks);
+	return `\n${MARKER_START}\n${body}\n${MARKER_END}\n`;
 }
 
 /**
@@ -137,87 +134,88 @@ function buildMarkerSection(blocks: GuidanceBlock[]): string {
  * - Content outside the markers is preserved exactly.
  */
 export function injectGuidance(filePath: string, blocks: GuidanceBlock[]): InjectionResult {
-  // Validate block IDs before processing (M1)
-  for (const block of blocks) {
-    if (!VALID_BLOCK_ID.test(block.id)) {
-      throw new Error(
-        `Invalid guidance block ID '${block.id}': must match /^[a-zA-Z0-9_-]+$/.`,
-      );
-    }
-  }
-
-  const original = fs.existsSync(filePath) ? fs.readFileSync(filePath, "utf-8") : "";
-
-  const startIdx = original.indexOf(MARKER_START);
-  const endIdx = original.indexOf(MARKER_END);
-
-  let existingBlocks = new Map<string, GuidanceBlock>();
-  let before = original;
-  let after = "";
-
-  if (startIdx !== -1 && endIdx !== -1 && endIdx > startIdx) {
-    // Extract content around markers
-    before = original.slice(0, startIdx);
-    after = original.slice(endIdx + MARKER_END.length);
-
-    const inner = original.slice(startIdx + MARKER_START.length, endIdx);
-    existingBlocks = parseExistingBlocks(inner);
-  } else {
-    // No markers: whole file is "before"
-    before = original;
-    after = "";
-  }
-
-  // Merge: apply new blocks on top of existing
-  const added: string[] = [];
-  const removed: string[] = [];
-
-  for (const block of blocks) {
-    if (block.content === "") {
-      // Empty content signals removal
-      if (existingBlocks.has(block.id)) {
-        existingBlocks.delete(block.id);
-        removed.push(block.id);
-      }
-    } else {
-      existingBlocks.set(block.id, { ...block, content: sanitizeGuidanceContent(block.content) });
-      added.push(block.id);
-    }
-  }
-
-  const merged = [...existingBlocks.values()];
-
-  // Nothing to write if there are no blocks and no markers existed
-  const hadMarkers = startIdx !== -1;
-  if (merged.length === 0 && !hadMarkers) {
-    return { path: filePath, modified: false, added: [], removed: [] };
-  }
-
-  // If all blocks removed and markers existed, remove the whole section
-  if (merged.length === 0 && hadMarkers) {
-    const newContent = trimTrailingNewlines(before) + after;
-    if (newContent === original) {
-      return { path: filePath, modified: false, added: [], removed };
-    }
-    fs.writeFileSync(filePath, newContent, "utf-8");
-    return { path: filePath, modified: true, added: [], removed };
-  }
-
-  const section = buildMarkerSection(merged);
-  const newContent = trimTrailingNewlines(before) + section + trimLeadingNewlines(after);
-
-  if (newContent === original) {
-    return { path: filePath, modified: false, added: [], removed };
-  }
-
-  // Ensure parent directory exists
-  const dir = filePath.substring(0, Math.max(filePath.lastIndexOf("/"), filePath.lastIndexOf("\\")));
-  if (dir && !fs.existsSync(dir)) {
-    fs.mkdirSync(dir, { recursive: true });
-  }
-
-  fs.writeFileSync(filePath, newContent, "utf-8");
-  return { path: filePath, modified: true, added, removed };
+	// Validate block IDs before processing (M1)
+	for (const block of blocks) {
+		if (!VALID_BLOCK_ID.test(block.id)) {
+			throw new Error(`Invalid guidance block ID '${block.id}': must match /^[a-zA-Z0-9_-]+$/.`);
+		}
+	}
+
+	const original = fs.existsSync(filePath) ? fs.readFileSync(filePath, "utf-8") : "";
+
+	const startIdx = original.indexOf(MARKER_START);
+	const endIdx = original.indexOf(MARKER_END);
+
+	let existingBlocks = new Map<string, GuidanceBlock>();
+	let before = original;
+	let after = "";
+
+	if (startIdx !== -1 && endIdx !== -1 && endIdx > startIdx) {
+		// Extract content around markers
+		before = original.slice(0, startIdx);
+		after = original.slice(endIdx + MARKER_END.length);
+
+		const inner = original.slice(startIdx + MARKER_START.length, endIdx);
+		existingBlocks = parseExistingBlocks(inner);
+	} else {
+		// No markers: whole file is "before"
+		before = original;
+		after = "";
+	}
+
+	// Merge: apply new blocks on top of existing
+	const added: string[] = [];
+	const removed: string[] = [];
+
+	for (const block of blocks) {
+		if (block.content === "") {
+			// Empty content signals removal
+			if (existingBlocks.has(block.id)) {
+				existingBlocks.delete(block.id);
+				removed.push(block.id);
+			}
+		} else {
+			existingBlocks.set(block.id, {
+				...block,
+				content: sanitizeGuidanceContent(block.content),
+			});
+			added.push(block.id);
+		}
+	}
+
+	const merged = [...existingBlocks.values()];
+
+	// Nothing to write if there are no blocks and no markers existed
+	const hadMarkers = startIdx !== -1;
+	if (merged.length === 0 && !hadMarkers) {
+		return { path: filePath, modified: false, added: [], removed: [] };
+	}
+
+	// If all blocks removed and markers existed, remove the whole section
+	if (merged.length === 0 && hadMarkers) {
+		const newContent = trimTrailingNewlines(before) + after;
+		if (newContent === original) {
+			return { path: filePath, modified: false, added: [], removed };
+		}
+		fs.writeFileSync(filePath, newContent, "utf-8");
+		return { path: filePath, modified: true, added: [], removed };
+	}
+
+	const section = buildMarkerSection(merged);
+	const newContent = trimTrailingNewlines(before) + section + trimLeadingNewlines(after);
+
+	if (newContent === original) {
+		return { path: filePath, modified: false, added: [], removed };
+	}
+
+	// Ensure parent directory exists
+	const dir = filePath.substring(0, Math.max(filePath.lastIndexOf("/"), filePath.lastIndexOf("\\")));
+	if (dir && !fs.existsSync(dir)) {
+		fs.mkdirSync(dir, { recursive: true });
+	}
+
+	fs.writeFileSync(filePath, newContent, "utf-8");
+	return { path: filePath, modified: true, added, removed };
 }
 
 /**
@@ -229,58 +227,58 @@ export function injectGuidance(filePath: string, blocks: GuidanceBlock[]): Injec
  * - Content outside markers is preserved exactly.
  */
 export function removeGuidance(filePath: string, ids?: string[]): InjectionResult {
-  if (!fs.existsSync(filePath)) {
-    return { path: filePath, modified: false, added: [], removed: [] };
-  }
-
-  const original = fs.readFileSync(filePath, "utf-8");
-  const startIdx = original.indexOf(MARKER_START);
-  const endIdx = original.indexOf(MARKER_END);
-
-  if (startIdx === -1 || endIdx === -1 || endIdx <= startIdx) {
-    return { path: filePath, modified: false, added: [], removed: [] };
-  }
-
-  const before = original.slice(0, startIdx);
-  const after = original.slice(endIdx + MARKER_END.length);
-  const inner = original.slice(startIdx + MARKER_START.length, endIdx);
-
-  if (ids === undefined) {
-    // Remove entire marker section
-    const allIds = extractGuidanceIds(inner);
-    const newContent = trimTrailingNewlines(before) + after;
-    if (newContent === original) {
-      return { path: filePath, modified: false, added: [], removed: [] };
-    }
-    fs.writeFileSync(filePath, newContent, "utf-8");
-    return { path: filePath, modified: true, added: [], removed: allIds };
-  }
-
-  // Remove specific block IDs
-  const existingBlocks = parseExistingBlocks(inner);
-  const removed: string[] = [];
-  for (const id of ids) {
-    if (existingBlocks.delete(id)) {
-      removed.push(id);
-    }
-  }
-
-  if (removed.length === 0) {
-    return { path: filePath, modified: false, added: [], removed: [] };
-  }
-
-  const merged = [...existingBlocks.values()];
-
-  if (merged.length === 0) {
-    const newContent = trimTrailingNewlines(before) + after;
-    fs.writeFileSync(filePath, newContent, "utf-8");
-    return { path: filePath, modified: true, added: [], removed };
-  }
-
-  const section = buildMarkerSection(merged);
-  const newContent = trimTrailingNewlines(before) + section + trimLeadingNewlines(after);
-  fs.writeFileSync(filePath, newContent, "utf-8");
-  return { path: filePath, modified: true, added: [], removed };
+	if (!fs.existsSync(filePath)) {
+		return { path: filePath, modified: false, added: [], removed: [] };
+	}
+
+	const original = fs.readFileSync(filePath, "utf-8");
+	const startIdx = original.indexOf(MARKER_START);
+	const endIdx = original.indexOf(MARKER_END);
+
+	if (startIdx === -1 || endIdx === -1 || endIdx <= startIdx) {
+		return { path: filePath, modified: false, added: [], removed: [] };
+	}
+
+	const before = original.slice(0, startIdx);
+	const after = original.slice(endIdx + MARKER_END.length);
+	const inner = original.slice(startIdx + MARKER_START.length, endIdx);
+
+	if (ids === undefined) {
+		// Remove entire marker section
+		const allIds = extractGuidanceIds(inner);
+		const newContent = trimTrailingNewlines(before) + after;
+		if (newContent === original) {
+			return { path: filePath, modified: false, added: [], removed: [] };
+		}
+		fs.writeFileSync(filePath, newContent, "utf-8");
+		return { path: filePath, modified: true, added: [], removed: allIds };
+	}
+
+	// Remove specific block IDs
+	const existingBlocks = parseExistingBlocks(inner);
+	const removed: string[] = [];
+	for (const id of ids) {
+		if (existingBlocks.delete(id)) {
+			removed.push(id);
+		}
+	}
+
+	if (removed.length === 0) {
+		return { path: filePath, modified: false, added: [], removed: [] };
+	}
+
+	const merged = [...existingBlocks.values()];
+
+	if (merged.length === 0) {
+		const newContent = trimTrailingNewlines(before) + after;
+		fs.writeFileSync(filePath, newContent, "utf-8");
+		return { path: filePath, modified: true, added: [], removed };
+	}
+
+	const section = buildMarkerSection(merged);
+	const newContent = trimTrailingNewlines(before) + section + trimLeadingNewlines(after);
+	fs.writeFileSync(filePath, newContent, "utf-8");
+	return { path: filePath, modified: true, added: [], removed };
 }
 
 /**
@@ -290,43 +288,43 @@ export function removeGuidance(filePath: string, ids?: string[]): InjectionResul
  * @param version — current pi-crew package version for version-aware updates
  */
 export function standardGuidanceBlocks(version: string): GuidanceBlock[] {
-  return [
-    {
-      id: "pi-crew-overview",
-      priority: 10,
-      content: [
-        "## pi-crew",
-        "",
-        `> Managed by **pi-crew v${version}** — do not edit this section manually.`,
-        "",
-        "pi-crew is a Pi extension for coordinated AI agent teams, workflows,",
-        "worktrees, and async task orchestration.",
-      ].join("\n"),
-    },
-    {
-      id: "pi-crew-commands",
-      priority: 5,
-      content: [
-        "### Quick Commands",
-        "",
-        "| Command | Description |",
-        "|---|---|",
-        "| `team action='init'` | Initialize pi-crew for this project |",
-        "| `team action='run'` | Start a team run |",
-        "| `team action='status'` | Check run status |",
-        "| `team action='list'` | List available teams/agents/workflows |",
-        "| `team action='recommend'` | Get team/workflow recommendations |",
-      ].join("\n"),
-    },
-  ];
+	return [
+		{
+			id: "pi-crew-overview",
+			priority: 10,
+			content: [
+				"## pi-crew",
+				"",
+				`> Managed by **pi-crew v${version}** — do not edit this section manually.`,
+				"",
+				"pi-crew is a Pi extension for coordinated AI agent teams, workflows,",
+				"worktrees, and async task orchestration.",
+			].join("\n"),
+		},
+		{
+			id: "pi-crew-commands",
+			priority: 5,
+			content: [
+				"### Quick Commands",
+				"",
+				"| Command | Description |",
+				"|---|---|",
+				"| `team action='init'` | Initialize pi-crew for this project |",
+				"| `team action='run'` | Start a team run |",
+				"| `team action='status'` | Check run status |",
+				"| `team action='list'` | List available teams/agents/workflows |",
+				"| `team action='recommend'` | Get team/workflow recommendations |",
+			].join("\n"),
+		},
+	];
 }
 
 /** Remove trailing newlines from a string, ensuring at most one trailing newline. */
 function trimTrailingNewlines(s: string): string {
-  return s.replace(/\n+$/, "");
+	return s.replace(/\n+$/, "");
 }
 
 /** Remove leading newlines from a string. */
 function trimLeadingNewlines(s: string): string {
-  return s.replace(/^\n+/, "");
+	return s.replace(/^\n+/, "");
 }

package/src/config/resilient-parser.ts

@@ -2,14 +2,11 @@
  * Per-field resilient config parsing — continues on error, collects all field-level issues.
  */
 
-import { PiTeamsConfigSchema } from "../schema/config-schema.ts";
 import type { TSchema } from "@sinclair/typebox";
 import { Value } from "@sinclair/typebox/value";
+import { PiTeamsConfigSchema } from "../schema/config-schema.ts";
+import { type PiTeamsConfig, parseConfig } from "./config.ts";
 import { suggestConfigKey } from "./suggestions.ts";
-import {
-	parseConfig,
-	type PiTeamsConfig,
-} from "./config.ts";
 
 // ---------------------------------------------------------------------------
 // Types
@@ -56,7 +53,18 @@ export function parseConfigResilient(raw: unknown): ResilientParseResult {
 	const warnings: string[] = [];
 
 	if (!isObject(raw)) {
-		return { config: {}, errors: [{ field: "config", message: "config must be an object", value: raw }], warnings, valid: false };
+		return {
+			config: {},
+			errors: [
+				{
+					field: "config",
+					message: "config must be an object",
+					value: raw,
+				},
+			],
+			warnings,
+			valid: false,
+		};
 	}
 
 	// Collect unknown top-level keys with suggestions

package/src/config/role-tools.ts

@@ -102,7 +102,7 @@ export function getToolConfig(role: string): RoleToolConfig {
  */
 export function hasToolRestrictions(role: string): boolean {
 	const config = getToolConfig(role);
-	return (config.tools !== undefined) || (config.excludeTools !== undefined);
+	return config.tools !== undefined || config.excludeTools !== undefined;
 }
 
 /**
@@ -110,6 +110,6 @@ export function hasToolRestrictions(role: string): boolean {
  */
 export function getRestrictedRoles(): string[] {
 	return Object.entries(ROLE_TOOL_CONFIGS)
-		.filter(([, config]) => (config.tools !== undefined) || (config.excludeTools !== undefined))
+		.filter(([, config]) => config.tools !== undefined || config.excludeTools !== undefined)
 		.map(([role]) => role);
-}
+}