@vibe-forge/workspace-assets 3.0.0 → 3.1.0

package/__tests__/adapter-asset-plan.spec.ts

@@ -394,7 +394,7 @@ describe('buildAdapterAssetPlan', () => {
     ])
   })
 
-  it('builds copilot native skill overlays and translated runtime diagnostics', async () => {
+  it('builds copilot native skill overlays and native hook diagnostics', async () => {
     const workspace = await createWorkspace()
 
     await installPluginPackage(workspace, '@vibe-forge/plugin-logger', {
@@ -489,7 +489,7 @@ describe('buildAdapterAssetPlan', () => {
       expect.objectContaining({
         assetId: loggerHookPluginId,
         adapter: 'copilot',
-        status: 'translated'
+        status: 'native'
       }),
       expect.objectContaining({
         assetId: docsMcpId,

package/__tests__/skill-dependencies-cli.spec.ts

@@ -113,6 +113,141 @@ describe('skills CLI dependency resolution', () => {
     })
   })
 
+  it('blocks missing dependency installs when auto downloads are disabled', async () => {
+    const workspace = await createWorkspace()
+
+    await writeDocument(
+      join(workspace, '.ai/skills/app-builder/SKILL.md'),
+      [
+        '---',
+        'name: app-builder',
+        'description: Build apps',
+        'dependencies:',
+        '  - name: frontend-design',
+        '    source: anthropics/skills',
+        '    registry: https://dependency-registry.example.test',
+        '---',
+        'Build the app.'
+      ].join('\n')
+    )
+
+    const bundle = await resolveWorkspaceAssetBundle({
+      cwd: workspace,
+      configs: [{
+        skills: {
+          autoDownloadDependencies: false
+        }
+      }, undefined],
+      useDefaultVibeForgeMcpServer: false
+    })
+
+    await expect(buildAdapterAssetPlan({
+      adapter: 'opencode',
+      bundle,
+      options: {
+        skills: {
+          include: ['app-builder']
+        }
+      }
+    })).rejects.toThrow('Skill dependency automatic downloads are disabled; cache not found')
+
+    expect(mocks.findSkillsCli).not.toHaveBeenCalled()
+    expect(mocks.installSkillsCliRefToTemp).not.toHaveBeenCalled()
+    expect(mocks.installSkillsCliSkillToTemp).not.toHaveBeenCalled()
+    expect(bundle.skills.map(asset => asset.name)).toEqual(['app-builder'])
+  })
+
+  it('blocks bare-name dependency searches when auto downloads are disabled', async () => {
+    const workspace = await createWorkspace()
+
+    await writeDocument(
+      join(workspace, '.ai/skills/app-builder/SKILL.md'),
+      [
+        '---',
+        'name: app-builder',
+        'description: Build apps',
+        'dependencies:',
+        '  - frontend-design',
+        '---',
+        'Build the app.'
+      ].join('\n')
+    )
+
+    const bundle = await resolveWorkspaceAssetBundle({
+      cwd: workspace,
+      configs: [{
+        skills: {
+          autoDownloadDependencies: false
+        }
+      }, undefined],
+      useDefaultVibeForgeMcpServer: false
+    })
+
+    await expect(buildAdapterAssetPlan({
+      adapter: 'opencode',
+      bundle,
+      options: {
+        skills: {
+          include: ['app-builder']
+        }
+      }
+    })).rejects.toThrow(
+      'Skill dependency automatic downloads are disabled; cannot resolve frontend-design without a source'
+    )
+
+    expect(mocks.findSkillsCli).not.toHaveBeenCalled()
+    expect(mocks.installSkillsCliRefToTemp).not.toHaveBeenCalled()
+    expect(mocks.installSkillsCliSkillToTemp).not.toHaveBeenCalled()
+  })
+
+  it('reuses source-qualified dependency caches when auto downloads are disabled', async () => {
+    const workspace = await createWorkspace()
+    await writeDocument(
+      join(
+        workspace,
+        '.ai/caches/skill-dependencies/skills-cli/skills/latest/default/anthropics/skills/latest/frontend-design/SKILL.md'
+      ),
+      '---\nname: frontend-design\ndescription: Cached UI guidance\n---\nUse the cached dependency.\n'
+    )
+    await writeDocument(
+      join(workspace, '.ai/skills/app-builder/SKILL.md'),
+      [
+        '---',
+        'name: app-builder',
+        'description: Build apps',
+        'dependencies:',
+        '  - anthropics/skills@frontend-design',
+        '---',
+        'Build the app.'
+      ].join('\n')
+    )
+
+    const bundle = await resolveWorkspaceAssetBundle({
+      cwd: workspace,
+      configs: [{
+        skills: {
+          autoDownloadDependencies: false
+        }
+      }, undefined],
+      useDefaultVibeForgeMcpServer: false
+    })
+
+    await buildAdapterAssetPlan({
+      adapter: 'opencode',
+      bundle,
+      options: {
+        skills: {
+          include: ['app-builder']
+        }
+      }
+    })
+
+    expect(mocks.findSkillsCli).not.toHaveBeenCalled()
+    expect(mocks.installSkillsCliRefToTemp).not.toHaveBeenCalled()
+    expect(mocks.installSkillsCliSkillToTemp).not.toHaveBeenCalled()
+    expect(bundle.skills.map(asset => asset.name).sort()).toEqual(['app-builder', 'frontend-design'])
+  })
+
   it('parses registry and version from dependency specs', async () => {
     const workspace = await createWorkspace()
     const installedSkillDir = join(installWorkspace, '.agents', 'skills', 'frontend-design')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vibe-forge/workspace-assets",
-  "version": "3.0.0",
+  "version": "3.1.0",
   "description": "Workspace asset resolution and adapter asset planning for Vibe Forge",
   "imports": {
     "#~/*.js": {
@@ -29,10 +29,10 @@
     "fast-glob": "^3.3.3",
     "front-matter": "^4.0.2",
     "js-yaml": "^4.1.1",
-    "@vibe-forge/types": "3.0.0",
-    "@vibe-forge/definition-core": "3.0.0",
-    "@vibe-forge/utils": "3.0.0",
-    "@vibe-forge/config": "3.0.0"
+    "@vibe-forge/types": "3.1.0",
+    "@vibe-forge/config": "3.1.0",
+    "@vibe-forge/utils": "3.1.0",
+    "@vibe-forge/definition-core": "3.1.0"
   },
   "devDependencies": {
     "@types/js-yaml": "^4.0.9"

package/src/adapter-asset-plan.ts

@@ -78,7 +78,7 @@ export async function buildAdapterAssetPlan(params: {
   params.bundle.hookPlugins.forEach((asset) => {
     pushDiagnostic(asset, {
       adapter: params.adapter,
-      status: params.adapter === 'copilot' ? 'translated' : 'native',
+      status: 'native',
       reason: params.adapter === 'claude-code'
         ? 'Mapped into the Claude Code native hooks bridge.'
         : params.adapter === 'codex'
@@ -86,7 +86,7 @@ export async function buildAdapterAssetPlan(params: {
         : params.adapter === 'gemini'
         ? 'Mapped into the Gemini native hooks bridge.'
         : params.adapter === 'copilot'
-        ? 'Handled by the Vibe Forge task hook bridge.'
+        ? 'Mapped into the Copilot CLI native hooks bridge.'
         : params.adapter === 'kimi'
         ? 'Mapped into the Kimi native hooks bridge.'
         : 'Mapped into the OpenCode native hooks bridge.'

package/src/skills-cli-dependency.ts

@@ -14,17 +14,33 @@ import {
   withInstallLock
 } from './skills-cli-dependency-helpers'
 
+const resolveAutoDownloadDependenciesEnabled = (
+  projectConfig: Config | undefined,
+  userConfig: Config | undefined
+) => userConfig?.skills?.autoDownloadDependencies ?? projectConfig?.skills?.autoDownloadDependencies ?? true
+
 export const installSkillsCliDependency = async (params: {
   cwd: string
   configs: [Config?, Config?]
   dependency: NormalizedSkillDependency
 }) => {
-  const resolvedTarget = params.dependency.source != null
-    ? {
-      skill: params.dependency.name,
-      source: params.dependency.source
+  const [projectConfig, userConfig] = params.configs
+  const autoDownloadDependenciesEnabled = resolveAutoDownloadDependenciesEnabled(projectConfig, userConfig)
+  const resolvedTarget = await (async () => {
+    if (params.dependency.source != null) {
+      return {
+        skill: params.dependency.name,
+        source: params.dependency.source
+      }
+    }
+
+    if (!autoDownloadDependenciesEnabled) {
+      throw new Error(
+        `Skill dependency automatic downloads are disabled; cannot resolve ${params.dependency.ref} without a source`
+      )
     }
-    : await (async () => {
+
+    return await (async () => {
       const searchResults = await findSkillsCli({
         registry: params.dependency.registry,
         query: params.dependency.name
@@ -40,6 +56,7 @@ export const installSkillsCliDependency = async (params: {
         source: selected.source
       }
     })()
+  })()
 
   const installDir = buildInstallDir({
     cwd: params.cwd,
@@ -58,6 +75,10 @@ export const installSkillsCliDependency = async (params: {
       }
     }
 
+    if (!autoDownloadDependenciesEnabled) {
+      throw new Error(`Skill dependency automatic downloads are disabled; cache not found for ${params.dependency.ref}`)
+    }
+
     const tempInstallDir = `${installDir}.tmp-${process.pid}-${Date.now()}-${Math.random().toString(36).slice(2)}`
     await rm(tempInstallDir, { recursive: true, force: true })
     await mkdir(tempInstallDir, { recursive: true })