deepagents 1.7.3 → 1.7.5

package/dist/index.cjs

@@ -1856,6 +1856,7 @@ function createMemoryMiddleware(options) {
 const MAX_SKILL_FILE_SIZE = 10 * 1024 * 1024;
 const MAX_SKILL_NAME_LENGTH = 64;
 const MAX_SKILL_DESCRIPTION_LENGTH = 1024;
+const MAX_SKILL_COMPATIBILITY_LENGTH = 500;
 /**
 * Zod schema for a single skill metadata entry.
 */
@@ -1943,6 +1944,22 @@ Remember: Skills are tools to make you more capable and consistent. When in doub
 `;
 /**
 * Validate skill name per Agent Skills specification.
+*
+* Constraints per Agent Skills specification:
+*
+* - 1-64 characters
+* - Unicode lowercase alphanumeric and hyphens only (`a-z` and `-`).
+* - Must not start or end with `-`
+* - Must not contain consecutive `--`
+* - Must match the parent directory name containing the `SKILL.md` file
+*
+* Unicode lowercase alphanumeric means any lowercase or decimal digit, which
+* covers accented Latin characters (e.g., `'café'`, `'über-tool'`) and other
+* scripts.
+*
+* @param name - The skill name from YAML frontmatter
+* @param directoryName - The parent directory name
+* @returns `{ valid, error }` tuple. Error is empty string if valid.
 */
 function validateSkillName$1(name, directoryName) {
 	if (!name) return {
@@ -1953,10 +1970,18 @@ function validateSkillName$1(name, directoryName) {
 		valid: false,
 		error: "name exceeds 64 characters"
 	};
-	if (!/^[a-z0-9]+(-[a-z0-9]+)*$/.test(name)) return {
+	if (name.startsWith("-") || name.endsWith("-") || name.includes("--")) return {
 		valid: false,
 		error: "name must be lowercase alphanumeric with single hyphens only"
 	};
+	for (const c of name) {
+		if (c === "-") continue;
+		if (/\p{Ll}/u.test(c) || /\p{Nd}/u.test(c)) continue;
+		return {
+			valid: false,
+			error: "name must be lowercase alphanumeric with single hyphens only"
+		};
+	}
 	if (name !== directoryName) return {
 		valid: false,
 		error: `name '${name}' must match directory name '${directoryName}'`
@@ -1967,7 +1992,52 @@ function validateSkillName$1(name, directoryName) {
 	};
 }
 /**
-* Parse YAML frontmatter from SKILL.md content.
+* Validate and normalize the metadata field from YAML frontmatter.
+*
+* YAML parsing can return any type for the `metadata` key. This ensures the
+* value in {@link SkillMetadata} is always a `Record<string, string>` by
+* coercing via `String()` and rejecting non-object inputs.
+*
+* @param raw - Raw value from `frontmatterData.metadata`.
+* @param skillPath - Path to the `SKILL.md` file (for warning messages).
+* @returns A validated `Record<string, string>`.
+*/
+function validateMetadata(raw, skillPath) {
+	if (typeof raw !== "object" || raw === null || Array.isArray(raw)) {
+		if (raw) console.warn(`Ignoring non-object metadata in ${skillPath} (got ${typeof raw})`);
+		return {};
+	}
+	const result = {};
+	for (const [k, v] of Object.entries(raw)) result[String(k)] = String(v);
+	return result;
+}
+/**
+* Build a parenthetical annotation string from optional skill fields.
+*
+* Combines license and compatibility into a comma-separated string for
+* display in the system prompt skill listing.
+*
+* @param skill - Skill metadata to extract annotations from.
+* @returns Annotation string like `'License: MIT, Compatibility: Python 3.10+'`,
+*   or empty string if neither field is set.
+*/
+function formatSkillAnnotations(skill) {
+	const parts = [];
+	if (skill.license) parts.push(`License: ${skill.license}`);
+	if (skill.compatibility) parts.push(`Compatibility: ${skill.compatibility}`);
+	return parts.join(", ");
+}
+/**
+* Parse YAML frontmatter from `SKILL.md` content.
+*
+* Extracts metadata per Agent Skills specification from YAML frontmatter
+* delimited by `---` markers at the start of the content.
+*
+* @param content - Content of the `SKILL.md` file
+* @param skillPath - Path to the `SKILL.md` file (for error messages and metadata)
+* @param directoryName - Name of the parent directory containing the skill
+* @returns `SkillMetadata` if parsing succeeds, `null` if parsing fails or
+*   validation errors occur
 */
 function parseSkillMetadataFromContent(content, skillPath, directoryName) {
 	if (content.length > MAX_SKILL_FILE_SIZE) {
@@ -1991,28 +2061,36 @@ function parseSkillMetadataFromContent(content, skillPath, directoryName) {
 		console.warn(`Skipping ${skillPath}: frontmatter is not a mapping`);
 		return null;
 	}
-	const name = frontmatterData.name;
-	const description = frontmatterData.description;
+	const name = String(frontmatterData.name ?? "").trim();
+	const description = String(frontmatterData.description ?? "").trim();
 	if (!name || !description) {
 		console.warn(`Skipping ${skillPath}: missing required 'name' or 'description'`);
 		return null;
 	}
-	const validation = validateSkillName$1(String(name), directoryName);
+	const validation = validateSkillName$1(name, directoryName);
 	if (!validation.valid) console.warn(`Skill '${name}' in ${skillPath} does not follow Agent Skills specification: ${validation.error}. Consider renaming for spec compliance.`);
-	let descriptionStr = String(description).trim();
+	let descriptionStr = description;
 	if (descriptionStr.length > MAX_SKILL_DESCRIPTION_LENGTH) {
 		console.warn(`Description exceeds ${MAX_SKILL_DESCRIPTION_LENGTH} characters in ${skillPath}, truncating`);
 		descriptionStr = descriptionStr.slice(0, MAX_SKILL_DESCRIPTION_LENGTH);
 	}
-	const allowedToolsStr = frontmatterData["allowed-tools"];
-	const allowedTools = allowedToolsStr ? allowedToolsStr.split(" ") : [];
+	const rawTools = frontmatterData["allowed-tools"];
+	let allowedTools;
+	if (rawTools) if (Array.isArray(rawTools)) allowedTools = rawTools.map((t) => String(t).trim()).filter(Boolean);
+	else allowedTools = String(rawTools).split(/\s+/).filter(Boolean);
+	else allowedTools = [];
+	let compatibilityStr = String(frontmatterData.compatibility ?? "").trim() || null;
+	if (compatibilityStr && compatibilityStr.length > MAX_SKILL_COMPATIBILITY_LENGTH) {
+		console.warn(`Compatibility exceeds ${MAX_SKILL_COMPATIBILITY_LENGTH} characters in ${skillPath}, truncating`);
+		compatibilityStr = compatibilityStr.slice(0, MAX_SKILL_COMPATIBILITY_LENGTH);
+	}
 	return {
-		name: String(name),
+		name,
 		description: descriptionStr,
 		path: skillPath,
-		metadata: frontmatterData.metadata || {},
-		license: typeof frontmatterData.license === "string" ? frontmatterData.license.trim() || null : null,
-		compatibility: typeof frontmatterData.compatibility === "string" ? frontmatterData.compatibility.trim() || null : null,
+		metadata: validateMetadata(frontmatterData.metadata ?? {}, skillPath),
+		license: String(frontmatterData.license ?? "").trim() || null,
+		compatibility: compatibilityStr,
 		allowedTools
 	};
 }
@@ -2076,7 +2154,10 @@ function formatSkillsList(skills, sources) {
 	if (skills.length === 0) return `(No skills available yet. You can create skills in ${sources.map((s) => `\`${s}\``).join(" or ")})`;
 	const lines = [];
 	for (const skill of skills) {
-		lines.push(`- **${skill.name}**: ${skill.description}`);
+		const annotations = formatSkillAnnotations(skill);
+		let descLine = `- **${skill.name}**: ${skill.description}`;
+		if (annotations) descLine += ` (${annotations})`;
+		lines.push(descLine);
 		if (skill.allowedTools && skill.allowedTools.length > 0) lines.push(`  → Allowed tools: ${skill.allowedTools.join(", ")}`);
 		lines.push(`  → Read \`${skill.path}\` for full instructions`);
 	}
@@ -2136,11 +2217,10 @@ function createSkillsMiddleware(options) {
 			const skillsLocations = formatSkillsLocations(sources);
 			const skillsList = formatSkillsList(skillsMetadata, sources);
 			const skillsSection = SKILLS_SYSTEM_PROMPT.replace("{skills_locations}", skillsLocations).replace("{skills_list}", skillsList);
-			const currentSystemPrompt = request.systemPrompt || "";
-			const newSystemPrompt = currentSystemPrompt ? `${currentSystemPrompt}\n\n${skillsSection}` : skillsSection;
+			const newSystemMessage = request.systemMessage.concat(skillsSection);
 			return handler({
 				...request,
-				systemPrompt: newSystemPrompt
+				systemMessage: newSystemMessage
 			});
 		}
 	});
@@ -2198,9 +2278,25 @@ function createSkillsMiddleware(options) {
 * from `langchain` directly.
 */
 /**
+* Zod schema for a summarization event that tracks what was summarized and
+* where the cutoff is.
+*
+* Instead of rewriting LangGraph state with `RemoveMessage(REMOVE_ALL_MESSAGES)`,
+* the middleware stores this event and uses it to reconstruct the effective message
+* list on subsequent calls.
+*/
+const SummarizationEventSchema = zod.z.object({
+	cutoffIndex: zod.z.number(),
+	summaryMessage: zod.z.instanceof(langchain.HumanMessage),
+	filePath: zod.z.string().nullable()
+});
+/**
 * State schema for summarization middleware.
 */
-const SummarizationStateSchema = zod.z.object({ _summarizationSessionId: zod.z.string().optional() });
+const SummarizationStateSchema = zod.z.object({
+	_summarizationSessionId: zod.z.string().optional(),
+	_summarizationEvent: SummarizationEventSchema.optional()
+});
 
 //#endregion
 //#region src/backends/store.ts