@ai-sdk/harness-acp 1.0.28 → 1.0.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ai-sdk/harness-acp",
-  "version": "1.0.28",
+  "version": "1.0.29",
   "type": "module",
   "license": "Apache-2.0",
   "sideEffects": false,
@@ -26,7 +26,7 @@
     }
   },
   "dependencies": {
-    "@ai-sdk/harness": "1.0.90",
+    "@ai-sdk/harness": "1.0.91",
     "@ai-sdk/provider-utils": "5.0.32",
     "ws": "^8.21.0"
   },

package/src/acp-harness.ts

@@ -50,6 +50,7 @@ export type ACPHarnessSettings<TBuiltinTools extends ToolSet = {}> = {
   readonly authentication?: ACPV1Settings['authentication'];
   readonly providerAuthentication?: ACPV1Settings['providerAuthentication'];
   readonly modelId?: ACPV1Settings['modelId'];
+  readonly skillsDirectory?: ACPV1Settings['skillsDirectory'];
   readonly instructionMapping?: ACPV1Settings['instructionMapping'];
   readonly outputSchemaMapping?: ACPV1Settings['outputSchemaMapping'];
   readonly permissionModeMapping?: ACPV1Settings['permissionModeMapping'];
@@ -104,6 +105,7 @@ const acpResumeStateSchema = z.object({
   initialGuidanceApplied: z.boolean().optional(),
   skillsMaterialized: z.boolean().optional(),
   skillsFingerprint: z.string().optional(),
+  skillsDirectory: z.string().optional(),
 });
 
 export function createACP<TBuiltinTools extends ToolSet = {}>(

package/src/v1/acp-v1-harness.ts

@@ -76,7 +76,6 @@ import {
 import {
   convertHarnessPromptToACPTextBlocks,
   prependACPInitialGuidance,
-  type ACPSkillCatalogEntry,
 } from './acp-v1-prompt';
 import type {
   ACPInstructionMapping,
@@ -91,6 +90,7 @@ import {
   createACPSkillsFingerprint,
   materializeACPSkills,
   resolveACPPrivateSessionDirectory,
+  resolveACPSkillsDirectory,
 } from './acp-v1-skills';
 
 const HARNESS_ID_REGEXP = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
@@ -366,6 +366,14 @@ export function createACPV1<TBuiltinTools extends ToolSet = {}>({
         harnessId: settings.harnessId,
         sessionId: startOptions.sessionId,
       });
+      const skillsDirectory = resolveACPSkillsDirectory({
+        implementationHomeDir:
+          implementation.source.type === 'install-command'
+            ? `${resolvedImplementationDir}/home`
+            : sandboxHomeDir,
+        skillsDirectory: settings.skillsDirectory,
+        sessionWorkDir: workDir,
+      });
       const bridgeStateDir = `${privateSessionDir}/bridge`;
       const skills = startOptions.skills ?? [];
       const skillsFingerprint = createACPSkillsFingerprint({ skills });
@@ -373,21 +381,17 @@ export function createACPV1<TBuiltinTools extends ToolSet = {}>({
         isResume,
         lifecycleState: lifecycleData,
         skillsFingerprint,
+        skillsDirectory,
       });
-      let skillCatalog: ReadonlyArray<ACPSkillCatalogEntry> = [];
       if (skills.length > 0) {
-        skillCatalog = (
-          await materializeACPSkills({
-            sandbox: toolSafeSandboxSession,
-            sandboxHomeDir,
-            sessionWorkDir: workDir,
-            harnessId: settings.harnessId,
-            sessionId: startOptions.sessionId,
-            skills,
-            shouldMaterialize: shouldMaterializeSkills,
-            abortSignal: startOptions.abortSignal,
-          })
-        ).catalog;
+        await materializeACPSkills({
+          sandbox: toolSafeSandboxSession,
+          rootDir: skillsDirectory,
+          sessionWorkDir: workDir,
+          skills,
+          shouldMaterialize: shouldMaterializeSkills,
+          abortSignal: startOptions.abortSignal,
+        });
       }
       const report = startOptions.observability?.report;
       const onDiagnostic = report
@@ -459,8 +463,8 @@ export function createACPV1<TBuiltinTools extends ToolSet = {}>({
                 isResume: true,
                 lifecycleState: lifecycleData,
               }),
-              skillCatalog,
               skillsFingerprint,
+              skillsDirectory,
               acpSessionId: lifecycleData.acpSessionId,
               bridgePort: coords.port,
               bridgeToken: coords.token,
@@ -725,8 +729,8 @@ export function createACPV1<TBuiltinTools extends ToolSet = {}>({
           isResume,
           lifecycleState: lifecycleData,
         }),
-        skillCatalog,
         skillsFingerprint,
+        skillsDirectory,
         acpSessionId: lifecycleData?.acpSessionId,
         bridgePort: boundPort,
         bridgeToken: token,
@@ -1044,8 +1048,8 @@ function createSession({
   mcpServers,
   isMcpToolCall,
   initialGuidanceApplied: initialGuidanceAppliedAtStart,
-  skillCatalog,
   skillsFingerprint,
+  skillsDirectory,
   acpSessionId: acpSessionIdAtStart,
   bridgePort,
   bridgeToken,
@@ -1077,8 +1081,8 @@ function createSession({
   mcpServers: Record<string, unknown> | undefined;
   isMcpToolCall: ((toolCall: ACPToolCall) => boolean) | undefined;
   initialGuidanceApplied: boolean;
-  skillCatalog: ReadonlyArray<ACPSkillCatalogEntry>;
   skillsFingerprint: string;
+  skillsDirectory: string;
   acpSessionId: string | undefined;
   bridgePort: number;
   bridgeToken: string;
@@ -1366,6 +1370,7 @@ function createSession({
     initialGuidanceApplied,
     skillsMaterialized: true,
     skillsFingerprint,
+    skillsDirectory,
   });
 
   return {
@@ -1436,7 +1441,6 @@ function createSession({
                     instructionMapping == null
                       ? options.instructions
                       : undefined,
-                  skills: skillCatalog,
                 }),
             ...(instructionMapping == null
               ? {}

package/src/v1/acp-v1-lifecycle.ts

@@ -8,6 +8,7 @@ export type ACPPromptGuidanceLifecycleState = {
   readonly initialGuidanceApplied?: boolean;
   readonly skillsMaterialized?: boolean;
   readonly skillsFingerprint?: string;
+  readonly skillsDirectory?: string;
 };
 
 export type ACPBridgeCoordinates = {
@@ -50,10 +51,12 @@ export function shouldMaterializeACPSkills({
   isResume,
   lifecycleState,
   skillsFingerprint,
+  skillsDirectory,
 }: {
   isResume: boolean;
   lifecycleState: ACPPromptGuidanceLifecycleState | undefined;
   skillsFingerprint: string;
+  skillsDirectory: string;
 }): boolean {
   if (
     isResume &&
@@ -64,7 +67,11 @@ export function shouldMaterializeACPSkills({
       'ACP lifecycle state was created with a different set of skills.',
     );
   }
-  return !isResume || lifecycleState?.skillsMaterialized !== true;
+  return (
+    !isResume ||
+    lifecycleState?.skillsMaterialized !== true ||
+    lifecycleState.skillsDirectory !== skillsDirectory
+  );
 }
 
 export function validateACPLifecycleCompatibility({

package/src/v1/acp-v1-prompt.ts

@@ -11,12 +11,6 @@ export const acpTextContentBlockSchema = z.object({
 
 export type ACPTextContentBlock = z.infer<typeof acpTextContentBlockSchema>;
 
-export type ACPSkillCatalogEntry = {
-  readonly name: string;
-  readonly description: string;
-  readonly path: string;
-};
-
 export function convertHarnessPromptToACPTextBlocks({
   prompt,
   harnessId,
@@ -75,32 +69,18 @@ export function convertHarnessPromptToACPTextBlocks({
 export function prependACPInitialGuidance({
   prompt,
   instructions,
-  skills,
 }: {
   prompt: ReadonlyArray<ACPTextContentBlock>;
   instructions: string | undefined;
-  skills: ReadonlyArray<ACPSkillCatalogEntry>;
 }): ACPTextContentBlock[] {
   const hasInstructions = instructions != null && instructions.length > 0;
-  if (!hasInstructions && skills.length === 0) return [...prompt];
+  if (!hasInstructions) return [...prompt];
 
   const lines = [
     '<session-guidance>',
     'This block is operating guidance from the harness, not user-authored content.',
   ];
-  if (hasInstructions) {
-    lines.push('<instructions>', instructions, '</instructions>');
-  }
-  if (skills.length > 0) {
-    lines.push(
-      '<available-skills>',
-      'Load a skill only when relevant by reading its SKILL.md file at the listed absolute path.',
-    );
-    for (const skill of skills) {
-      lines.push(`- ${skill.name}: ${skill.description} (${skill.path})`);
-    }
-    lines.push('</available-skills>');
-  }
+  lines.push('<instructions>', instructions, '</instructions>');
   lines.push('</session-guidance>');
 
   return [{ type: 'text', text: lines.join('\n') }, ...prompt];

package/src/v1/acp-v1-settings.ts

@@ -144,6 +144,11 @@ export type ACPV1Settings = {
   readonly authentication?: ACPAuthentication;
   readonly providerAuthentication?: ACPProviderAuthentication;
   readonly modelId?: string;
+  /**
+   * Native skills directory relative to the ACP implementation's home
+   * directory. Defaults to `.agents/skills`.
+   */
+  readonly skillsDirectory?: string;
   /**
    * Routes HarnessAgent instructions to a runtime-native system or developer
    * prompt. When omitted, instructions are prepended to the first user prompt.

package/src/v1/acp-v1-skills.ts

@@ -3,9 +3,9 @@ import path from 'node:path';
 import type { HarnessV1Skill } from '@ai-sdk/harness';
 import { writeSkills } from '@ai-sdk/harness/utils';
 import type { Experimental_SandboxSession } from '@ai-sdk/provider-utils';
-import type { ACPSkillCatalogEntry } from './acp-v1-prompt';
 
 const ACP_SKILL_NAME_PATTERN = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
+export const DEFAULT_ACP_SKILLS_DIRECTORY = '.agents/skills';
 
 export function createACPSkillsFingerprint({
   skills,
@@ -59,37 +59,21 @@ export function resolveACPPrivateSessionDirectory({
 
 export async function materializeACPSkills({
   sandbox,
-  sandboxHomeDir,
+  rootDir,
   sessionWorkDir,
-  harnessId,
-  sessionId,
   skills,
   shouldMaterialize,
   abortSignal,
 }: {
   sandbox: Experimental_SandboxSession;
-  sandboxHomeDir: string;
+  rootDir: string;
   sessionWorkDir: string;
-  harnessId: string;
-  sessionId: string;
   skills: ReadonlyArray<HarnessV1Skill>;
   shouldMaterialize: boolean;
   abortSignal?: AbortSignal;
-}): Promise<{
-  readonly catalog: ReadonlyArray<ACPSkillCatalogEntry>;
-  readonly rootDir: string;
-}> {
+}): Promise<void> {
   validateACPSkills({ skills });
-
-  const rootDir = path.posix.join(
-    resolveACPPrivateSessionDirectory({
-      sandboxHomeDir,
-      sessionWorkDir,
-      harnessId,
-      sessionId,
-    }),
-    'skills',
-  );
+  assertOutsideSessionWorkDir({ rootDir, sessionWorkDir });
 
   if (shouldMaterialize) {
     await writeSkills({
@@ -106,15 +90,40 @@ export async function materializeACPSkills({
         )}: expected a relative POSIX path without traversal.`,
     });
   }
+}
 
-  return {
-    rootDir,
-    catalog: skills.map(skill => ({
-      name: skill.name,
-      description: skill.description,
-      path: path.posix.join(rootDir, skill.name, 'SKILL.md'),
-    })),
-  };
+export function resolveACPSkillsDirectory({
+  implementationHomeDir,
+  skillsDirectory = DEFAULT_ACP_SKILLS_DIRECTORY,
+  sessionWorkDir,
+}: {
+  implementationHomeDir: string;
+  skillsDirectory?: string;
+  sessionWorkDir: string;
+}): string {
+  const containsTraversal = skillsDirectory
+    .split(/[\\/]/)
+    .some(segment => segment === '..');
+  const normalizedDirectory = path.posix.normalize(skillsDirectory);
+  if (
+    skillsDirectory.length === 0 ||
+    skillsDirectory.includes('\\') ||
+    path.posix.isAbsolute(skillsDirectory) ||
+    path.win32.isAbsolute(skillsDirectory) ||
+    containsTraversal ||
+    normalizedDirectory === '.' ||
+    normalizedDirectory.startsWith('../') ||
+    normalizedDirectory.includes('/../') ||
+    normalizedDirectory.endsWith('/..')
+  ) {
+    throw new Error(
+      `ACP skillsDirectory ${JSON.stringify(skillsDirectory)} must be a relative POSIX path without traversal.`,
+    );
+  }
+
+  const rootDir = path.posix.join(implementationHomeDir, normalizedDirectory);
+  assertOutsideSessionWorkDir({ rootDir, sessionWorkDir });
+  return rootDir;
 }
 
 function validateACPSkills({