@agent-facets/core 0.1.2 → 0.2.0

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @agent-facets/core
 
+## 0.2.0
+
+### Minor Changes
+
+- [#35](https://github.com/agent-facets/facets/pull/35) [`6350718`](https://github.com/agent-facets/facets/commit/63507188f1bb3a7276cd4812f69f7d16d1778fd6) Thanks [@eXamadeus](https://github.com/eXamadeus)! - Ensure proper release isolation
+
+### Patch Changes
+
+- [#33](https://github.com/agent-facets/facets/pull/33) [`540e126`](https://github.com/agent-facets/facets/commit/540e126e677de98a9b3d4e39542df37de8756b73) Thanks [@eXamadeus](https://github.com/eXamadeus)! - Ensure CI runs tests before release and notify Slack when failures occur.
+
+## 0.1.3
+
+### Patch Changes
+
+- [`098fd08`](https://github.com/agent-facets/facets/commit/098fd08bf5d9970babc5c57bee6a155bffcecd97) Thanks [@eXamadeus](https://github.com/eXamadeus)! - Better CLI parameter validation
+
+- [`5262cbe`](https://github.com/agent-facets/facets/commit/5262cbe66df02c625430309878e6061ccde183de) Thanks [@eXamadeus](https://github.com/eXamadeus)! - Fix publishing by properly categorizing dev dependencies
+
+- [`d3b9439`](https://github.com/agent-facets/facets/commit/d3b9439466e0eb65687901426e2ebd6c5a333c60) Thanks [@eXamadeus](https://github.com/eXamadeus)! - Use better github attribution for changesets
+
 ## 0.1.2
 
 ### Patch Changes

package/package.json

@@ -5,25 +5,28 @@
     "url": "https://github.com/agent-facets/facets",
     "directory": "packages/core"
   },
-  "version": "0.1.2",
+  "version": "0.2.0",
   "type": "module",
   "exports": {
     ".": "./src/index.ts"
   },
   "scripts": {
+    "prepack": "bun ../../scripts/prepack.ts",
+    "postpack": "bun ../../scripts/postpack.ts",
     "types": "tsc --noEmit",
     "test": "bun test"
   },
   "dependencies": {
     "arktype": "2.1.29",
     "comment-json": "^4.2.5",
-    "nanotar": "0.3.0"
+    "nanotar": "0.3.0",
+    "yaml": "2.8.3"
   },
   "devDependencies": {
     "@types/bun": "1.3.10"
   },
   "peerDependencies": {
-    "typescript": "^5"
+    "typescript": "^5 || ^6"
   },
   "publishConfig": {
     "access": "public",

package/src/__tests__/build-pipeline.test.ts

@@ -2,6 +2,7 @@ import { afterAll, beforeAll, describe, expect, test } from 'bun:test'
 import { mkdtemp, rm } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
 import { join } from 'node:path'
+import dedent from 'dedent'
 import { detectNamingCollisions } from '../build/detect-collisions.ts'
 import { runBuildPipeline } from '../build/pipeline.ts'
 import { validateCompactFacets } from '../build/validate-facets.ts'
@@ -206,7 +207,7 @@ describe('validatePlatformConfigs', () => {
 describe('runBuildPipeline', () => {
   test('successful build with valid facet', async () => {
     const dir = await createFixtureDir('valid-build')
-    await Bun.write(join(dir, 'skills/example.md'), '# Example skill')
+    await Bun.write(join(dir, 'skills/example/SKILL.md'), '# Example skill')
     await Bun.write(
       join(dir, 'facet.json'),
       JSON.stringify({
@@ -230,8 +231,8 @@ describe('runBuildPipeline', () => {
       expect(result.archiveFilename).toBe('test-facet-1.0.0.facet')
       expect(result.archiveBytes.length).toBeGreaterThan(0)
       expect(Object.keys(result.assetHashes)).toContain('facet.json')
-      expect(Object.keys(result.assetHashes)).toContain('skills/example.md')
-      expect(result.assetHashes['skills/example.md']).toMatchInlineSnapshot(
+      expect(Object.keys(result.assetHashes)).toContain('skills/example/SKILL.md')
+      expect(result.assetHashes['skills/example/SKILL.md']).toMatchInlineSnapshot(
         `"sha256:ded8057927e03783371d0d929e4a6e92da66eb9dd164377ad6845a5a1c0cb5ba"`,
       )
       expect(result.integrity).toMatch(/^sha256:[a-f0-9]{64}$/)
@@ -263,13 +264,13 @@ describe('runBuildPipeline', () => {
     expect(result.ok).toBe(false)
     if (!result.ok) {
       expect(result.errors[0]?.path).toBe('skills.example')
-      expect(result.errors[0]?.message).toContain('skills/example.md')
+      expect(result.errors[0]?.message).toContain('skills/example/SKILL.md')
     }
   })
 
   test('build succeeds with cross-type name sharing', async () => {
     const dir = await createFixtureDir('cross-type')
-    await Bun.write(join(dir, 'skills/review.md'), '# Review skill')
+    await Bun.write(join(dir, 'skills/review/SKILL.md'), '# Review skill')
     await Bun.write(join(dir, 'commands/review.md'), '# Review command')
     await Bun.write(
       join(dir, 'facet.json'),
@@ -291,8 +292,8 @@ describe('runBuildPipeline', () => {
 
   test('build with all asset types includes all hashes', async () => {
     const dir = await createFixtureDir('all-types')
-    await Bun.write(join(dir, 'skills/alpha.md'), '# Alpha skill')
-    await Bun.write(join(dir, 'skills/beta.md'), '# Beta skill')
+    await Bun.write(join(dir, 'skills/alpha/SKILL.md'), '# Alpha skill')
+    await Bun.write(join(dir, 'skills/beta/SKILL.md'), '# Beta skill')
     await Bun.write(join(dir, 'agents/helper.md'), '# Helper agent')
     await Bun.write(join(dir, 'commands/deploy.md'), '# Deploy command')
     await Bun.write(
@@ -322,15 +323,15 @@ describe('runBuildPipeline', () => {
         'agents/helper.md',
         'commands/deploy.md',
         'facet.json',
-        'skills/alpha.md',
-        'skills/beta.md',
+        'skills/alpha/SKILL.md',
+        'skills/beta/SKILL.md',
       ])
     }
   })
 
   test('build fails on malformed compact facets entry', async () => {
     const dir = await createFixtureDir('bad-facets')
-    await Bun.write(join(dir, 'skills/x.md'), '# Skill')
+    await Bun.write(join(dir, 'skills/x/SKILL.md'), '# Skill')
     await Bun.write(
       join(dir, 'facet.json'),
       JSON.stringify({
@@ -351,12 +352,96 @@ describe('runBuildPipeline', () => {
   })
 })
 
+// --- Content validation ---
+
+describe('content validation', () => {
+  test('build fails on file with YAML front matter', async () => {
+    const dir = await createFixtureDir('front-matter')
+    await Bun.write(
+      join(dir, 'skills/review/SKILL.md'),
+      dedent`
+        ---
+        name: Review
+        description: A review skill
+        ---
+        # Review
+        Review all code.
+      `,
+    )
+    await Bun.write(
+      join(dir, 'facet.json'),
+      JSON.stringify({
+        name: 'test-facet',
+        version: '1.0.0',
+        skills: {
+          review: { description: 'A review skill' },
+        },
+      }),
+    )
+
+    const result = await runBuildPipeline(dir)
+    expect(result.ok).toBe(false)
+    if (!result.ok) {
+      expect(result.errors).toHaveLength(1)
+      expect(result.errors[0]?.path).toBe('skills.review')
+      expect(result.errors[0]?.message).toContain('front matter')
+      expect(result.errors[0]?.message).toContain('skills/review/SKILL.md')
+    }
+  })
+
+  test('build fails on empty content file', async () => {
+    const dir = await createFixtureDir('empty-file')
+    await Bun.write(join(dir, 'skills/empty/SKILL.md'), '')
+    await Bun.write(
+      join(dir, 'facet.json'),
+      JSON.stringify({
+        name: 'test-facet',
+        version: '1.0.0',
+        skills: {
+          empty: { description: 'An empty skill' },
+        },
+      }),
+    )
+
+    const result = await runBuildPipeline(dir)
+    expect(result.ok).toBe(false)
+    if (!result.ok) {
+      expect(result.errors).toHaveLength(1)
+      expect(result.errors[0]?.path).toBe('skills.empty')
+      expect(result.errors[0]?.message).toContain('empty')
+    }
+  })
+
+  test('build fails on whitespace-only content file', async () => {
+    const dir = await createFixtureDir('whitespace-file')
+    await Bun.write(join(dir, 'agents/blank.md'), '   \n\n  \n')
+    await Bun.write(
+      join(dir, 'facet.json'),
+      JSON.stringify({
+        name: 'test-facet',
+        version: '1.0.0',
+        agents: {
+          blank: { description: 'A blank agent' },
+        },
+      }),
+    )
+
+    const result = await runBuildPipeline(dir)
+    expect(result.ok).toBe(false)
+    if (!result.ok) {
+      expect(result.errors).toHaveLength(1)
+      expect(result.errors[0]?.path).toBe('agents.blank')
+      expect(result.errors[0]?.message).toContain('empty')
+    }
+  })
+})
+
 // --- Build output generation ---
 
 describe('writeBuildOutput', () => {
   test('writes archive and build manifest to dist/', async () => {
     const dir = await createFixtureDir('write-output')
-    await Bun.write(join(dir, 'skills/example.md'), '# Resolved content')
+    await Bun.write(join(dir, 'skills/example/SKILL.md'), '# Resolved content')
     await Bun.write(
       join(dir, 'facet.json'),
       JSON.stringify({
@@ -385,7 +470,7 @@ describe('writeBuildOutput', () => {
     expect(manifest.archive).toBe('test-facet-1.0.0.facet')
     expect(manifest.integrity).toMatch(/^sha256:[a-f0-9]{64}$/)
     expect(manifest.assets['facet.json']).toMatch(/^sha256:[a-f0-9]{64}$/)
-    expect(manifest.assets['skills/example.md']).toMatch(/^sha256:[a-f0-9]{64}$/)
+    expect(manifest.assets['skills/example/SKILL.md']).toMatch(/^sha256:[a-f0-9]{64}$/)
 
     // No loose files
     const looseManifest = await Bun.file(join(dir, 'dist/facet.json')).exists()
@@ -394,7 +479,7 @@ describe('writeBuildOutput', () => {
 
   test('cleans previous dist/ before writing', async () => {
     const dir = await createFixtureDir('clean-dist')
-    await Bun.write(join(dir, 'skills/x.md'), '# Skill')
+    await Bun.write(join(dir, 'skills/x/SKILL.md'), '# Skill')
     await Bun.write(
       join(dir, 'facet.json'),
       JSON.stringify({

package/src/__tests__/content-hash.test.ts

@@ -63,7 +63,7 @@ describe('collectArchiveEntries', () => {
 
     expect(entries).toHaveLength(4)
     expect(entries.map((e) => e.path)).toContain('facet.json')
-    expect(entries.map((e) => e.path)).toContain('skills/review.md')
+    expect(entries.map((e) => e.path)).toContain('skills/review/SKILL.md')
     expect(entries.map((e) => e.path)).toContain('agents/helper.md')
     expect(entries.map((e) => e.path)).toContain('commands/deploy.md')
   })
@@ -84,7 +84,7 @@ describe('collectArchiveEntries', () => {
     const entries = collectArchiveEntries(resolved, 'manifest content')
     const paths = entries.map((e) => e.path)
 
-    expect(paths).toEqual(['agents/b-agent.md', 'facet.json', 'skills/a-skill.md', 'skills/z-skill.md'])
+    expect(paths).toEqual(['agents/b-agent.md', 'facet.json', 'skills/a-skill/SKILL.md', 'skills/z-skill/SKILL.md'])
   })
 
   test('handles manifest with no optional asset types', () => {
@@ -105,7 +105,7 @@ describe('computeAssetHashes', () => {
   test('returns correct hash for each entry', () => {
     const entries = [
       { path: 'facet.json', content: '{"name":"test"}' },
-      { path: 'skills/review.md', content: '# Review' },
+      { path: 'skills/review/SKILL.md', content: '# Review' },
     ]
 
     const hashes = computeAssetHashes(entries)
@@ -114,7 +114,7 @@ describe('computeAssetHashes', () => {
     expect(hashes['facet.json']).toMatchInlineSnapshot(
       `"sha256:7d9fd2051fc32b32feab10946fab6bb91426ab7e39aa5439289ed892864aa91d"`,
     )
-    expect(hashes['skills/review.md']).toMatchInlineSnapshot(
+    expect(hashes['skills/review/SKILL.md']).toMatchInlineSnapshot(
       `"sha256:f1a9d9d60fba2e67d82d788760d147d95461a58456411e205bf33a6dbdc3497f"`,
     )
   })
@@ -134,7 +134,7 @@ describe('assembleTar', () => {
   test('produces a valid tar archive', () => {
     const entries = [
       { path: 'facet.json', content: '{"name":"test","version":"1.0.0"}' },
-      { path: 'skills/review.md', content: '# Review skill' },
+      { path: 'skills/review/SKILL.md', content: '# Review skill' },
     ]
 
     const tar = assembleTar(entries)
@@ -147,7 +147,7 @@ describe('assembleTar', () => {
 
     const names = parsed.map((f) => f.name)
     expect(names).toContain('facet.json')
-    expect(names).toContain('skills/review.md')
+    expect(names).toContain('skills/review/SKILL.md')
   })
 
   test('tar contains correct file contents', () => {
@@ -205,7 +205,7 @@ describe('compressArchive', () => {
   test('compressed archive can be decompressed to recover original tar', async () => {
     const entries = [
       { path: 'facet.json', content: '{"name":"test","version":"1.0.0"}' },
-      { path: 'skills/review.md', content: '# Review skill' },
+      { path: 'skills/review/SKILL.md', content: '# Review skill' },
     ]
 
     const tar = assembleTar(entries)
@@ -220,7 +220,7 @@ describe('compressArchive', () => {
 
     const names = parsed.map((f) => f.name)
     expect(names).toContain('facet.json')
-    expect(names).toContain('skills/review.md')
-    expect(parsed.find((f) => f.name === 'skills/review.md')?.text).toBe('# Review skill')
+    expect(names).toContain('skills/review/SKILL.md')
+    expect(parsed.find((f) => f.name === 'skills/review/SKILL.md')?.text).toBe('# Review skill')
   })
 })

package/src/__tests__/edit.test.ts

@@ -0,0 +1,190 @@
+import { describe, expect, test } from 'bun:test'
+import { mkdir } from 'node:fs/promises'
+import { tmpdir } from 'node:os'
+import { join } from 'node:path'
+import { writeManifest } from '../edit/manifest-writer.ts'
+import { reconcile } from '../edit/reconcile.ts'
+import type { DiscoveredAsset } from '../edit/scanner.ts'
+import { scanAssets } from '../edit/scanner.ts'
+import type { FacetManifest } from '../schemas/facet-manifest.ts'
+
+async function createFixtureDir(name: string): Promise<string> {
+  const dir = join(tmpdir(), `facets-edit-test-${name}-${Date.now()}`)
+  await mkdir(dir, { recursive: true })
+  return dir
+}
+
+// --- Scanner ---
+
+describe('scanAssets', () => {
+  test('discovers skills in directory convention', async () => {
+    const dir = await createFixtureDir('scan-skills')
+    await mkdir(join(dir, 'skills/review'), { recursive: true })
+    await mkdir(join(dir, 'skills/code-fix'), { recursive: true })
+    await Bun.write(join(dir, 'skills/review/SKILL.md'), '# Review')
+    await Bun.write(join(dir, 'skills/code-fix/SKILL.md'), '# Code Fix')
+
+    const assets = await scanAssets(dir)
+    expect(assets.filter((a) => a.type === 'skills')).toHaveLength(2)
+    expect(assets.find((a) => a.name === 'review')).toBeDefined()
+    expect(assets.find((a) => a.name === 'code-fix')).toBeDefined()
+  })
+
+  test('discovers agents and commands in flat convention', async () => {
+    const dir = await createFixtureDir('scan-flat')
+    await mkdir(join(dir, 'agents'), { recursive: true })
+    await mkdir(join(dir, 'commands'), { recursive: true })
+    await Bun.write(join(dir, 'agents/reviewer.md'), '# Reviewer')
+    await Bun.write(join(dir, 'commands/deploy.md'), '# Deploy')
+
+    const assets = await scanAssets(dir)
+    expect(assets.find((a) => a.type === 'agents' && a.name === 'reviewer')).toBeDefined()
+    expect(assets.find((a) => a.type === 'commands' && a.name === 'deploy')).toBeDefined()
+  })
+
+  test('skips non-kebab-case names', async () => {
+    const dir = await createFixtureDir('scan-skip')
+    await mkdir(join(dir, 'skills/InvalidName'), { recursive: true })
+    await mkdir(join(dir, 'agents'), { recursive: true })
+    await Bun.write(join(dir, 'skills/InvalidName/SKILL.md'), '# Bad')
+    await Bun.write(join(dir, 'agents/NOT_VALID.md'), '# Bad')
+
+    const assets = await scanAssets(dir)
+    expect(assets).toHaveLength(0)
+  })
+
+  test('returns empty for directory with no assets', async () => {
+    const dir = await createFixtureDir('scan-empty')
+    const assets = await scanAssets(dir)
+    expect(assets).toHaveLength(0)
+  })
+
+  test('returns sorted results', async () => {
+    const dir = await createFixtureDir('scan-sorted')
+    await mkdir(join(dir, 'skills/zebra'), { recursive: true })
+    await mkdir(join(dir, 'agents'), { recursive: true })
+    await mkdir(join(dir, 'skills/alpha'), { recursive: true })
+    await Bun.write(join(dir, 'skills/zebra/SKILL.md'), '# Z')
+    await Bun.write(join(dir, 'skills/alpha/SKILL.md'), '# A')
+    await Bun.write(join(dir, 'agents/beta.md'), '# B')
+
+    const assets = await scanAssets(dir)
+    const names = assets.map((a) => `${a.type}:${a.name}`)
+    expect(names).toEqual(['agents:beta', 'skills:alpha', 'skills:zebra'])
+  })
+})
+
+// --- Reconciliation ---
+
+describe('reconcile', () => {
+  test('identifies additions (on disk, not in manifest)', () => {
+    const manifest: FacetManifest = {
+      name: 'test',
+      version: '1.0.0',
+      skills: { review: { description: 'Review skill' } },
+    }
+    const discovered: DiscoveredAsset[] = [
+      { type: 'skills', name: 'review', path: 'skills/review/SKILL.md' },
+      { type: 'skills', name: 'new-skill', path: 'skills/new-skill/SKILL.md' },
+      { type: 'agents', name: 'helper', path: 'agents/helper.md' },
+    ]
+
+    const result = reconcile(manifest, discovered)
+    expect(result.additions).toHaveLength(2)
+    expect(result.additions.find((a) => a.name === 'new-skill')).toBeDefined()
+    expect(result.additions.find((a) => a.name === 'helper')).toBeDefined()
+  })
+
+  test('identifies missing files (in manifest, not on disk)', () => {
+    const manifest: FacetManifest = {
+      name: 'test',
+      version: '1.0.0',
+      skills: {
+        review: { description: 'Review' },
+        gone: { description: 'Gone' },
+      },
+    }
+    const discovered: DiscoveredAsset[] = [{ type: 'skills', name: 'review', path: 'skills/review/SKILL.md' }]
+
+    const result = reconcile(manifest, discovered)
+    expect(result.missing).toHaveLength(1)
+    expect(result.missing[0]?.name).toBe('gone')
+    expect(result.missing[0]?.expectedPath).toBe('skills/gone/SKILL.md')
+  })
+
+  test('identifies matched assets', () => {
+    const manifest: FacetManifest = {
+      name: 'test',
+      version: '1.0.0',
+      agents: { reviewer: { description: 'Review agent' } },
+    }
+    const discovered: DiscoveredAsset[] = [{ type: 'agents', name: 'reviewer', path: 'agents/reviewer.md' }]
+
+    const result = reconcile(manifest, discovered)
+    expect(result.matched).toHaveLength(1)
+    expect(result.matched[0]?.name).toBe('reviewer')
+    expect(result.additions).toHaveLength(0)
+    expect(result.missing).toHaveLength(0)
+  })
+
+  test('handles manifest with no assets', () => {
+    const manifest: FacetManifest = {
+      name: 'test',
+      version: '1.0.0',
+      facets: ['base@1.0.0'],
+    }
+    const discovered: DiscoveredAsset[] = [{ type: 'skills', name: 'extra', path: 'skills/extra/SKILL.md' }]
+
+    const result = reconcile(manifest, discovered)
+    expect(result.additions).toHaveLength(1)
+    expect(result.missing).toHaveLength(0)
+    expect(result.matched).toHaveLength(0)
+  })
+
+  test('missing agent has flat file expected path', () => {
+    const manifest: FacetManifest = {
+      name: 'test',
+      version: '1.0.0',
+      agents: { gone: { description: 'Gone' } },
+    }
+
+    const result = reconcile(manifest, [])
+    expect(result.missing[0]?.expectedPath).toBe('agents/gone.md')
+  })
+})
+
+// --- Manifest Writer ---
+
+describe('writeManifest', () => {
+  test('writes valid JSON to facet.json', async () => {
+    const dir = await createFixtureDir('write-manifest')
+    const manifest: FacetManifest = {
+      name: 'test-facet',
+      version: '1.0.0',
+      skills: { review: { description: 'A review skill' } },
+    }
+
+    await writeManifest(manifest, dir)
+
+    const content = await Bun.file(join(dir, 'facet.json')).text()
+    const parsed = JSON.parse(content)
+    expect(parsed.name).toBe('test-facet')
+    expect(parsed.version).toBe('1.0.0')
+    expect(parsed.skills.review.description).toBe('A review skill')
+  })
+
+  test('output is 2-space indented', async () => {
+    const dir = await createFixtureDir('write-indent')
+    const manifest: FacetManifest = {
+      name: 'test',
+      version: '1.0.0',
+      skills: { x: { description: 'X' } },
+    }
+
+    await writeManifest(manifest, dir)
+
+    const content = await Bun.file(join(dir, 'facet.json')).text()
+    // 2-space indent means lines like '  "name": "test"'
+    expect(content).toContain('  "name"')
+  })
+})

package/src/__tests__/facet-loader.test.ts

@@ -2,6 +2,7 @@ import { afterAll, beforeAll, describe, expect, test } from 'bun:test'
 import { mkdtemp, rm } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
 import { join } from 'node:path'
+import dedent from 'dedent'
 import { loadManifest, resolvePrompts } from '../loaders/facet.ts'
 
 let testDir: string
@@ -174,9 +175,30 @@ describe('loadManifest', () => {
 describe('resolvePrompts', () => {
   test('prompt content is resolved from conventional file paths', async () => {
     const dir = await createFixtureDir('resolve-convention')
-    await writeFixture(dir, 'skills/review.md', '# Code Review\nReview all code.')
-    await writeFixture(dir, 'agents/reviewer.md', '# Reviewer\nReview this code.')
-    await writeFixture(dir, 'commands/deploy.md', '# Deploy\nDeploy the code.')
+    await writeFixture(
+      dir,
+      'skills/review/SKILL.md',
+      dedent`
+        # Code Review
+        Review all code.
+      `,
+    )
+    await writeFixture(
+      dir,
+      'agents/reviewer.md',
+      dedent`
+        # Reviewer
+        Review this code.
+      `,
+    )
+    await writeFixture(
+      dir,
+      'commands/deploy.md',
+      dedent`
+        # Deploy
+        Deploy the code.
+      `,
+    )
 
     const manifest = {
       name: 'test',
@@ -195,15 +217,31 @@ describe('resolvePrompts', () => {
     const result = await resolvePrompts(manifest, dir)
     expect(result.ok).toBe(true)
     if (result.ok) {
-      expect(result.data.skills?.review?.prompt).toBe('# Code Review\nReview all code.')
-      expect(result.data.agents?.reviewer?.prompt).toBe('# Reviewer\nReview this code.')
-      expect(result.data.commands?.deploy?.prompt).toBe('# Deploy\nDeploy the code.')
+      expect(result.data.skills?.review?.prompt).toBe(dedent`
+        # Code Review
+        Review all code.
+      `)
+      expect(result.data.agents?.reviewer?.prompt).toBe(dedent`
+        # Reviewer
+        Review this code.
+      `)
+      expect(result.data.commands?.deploy?.prompt).toBe(dedent`
+        # Deploy
+        Deploy the code.
+      `)
     }
   })
 
   test('file-based prompt is resolved from agents/<name>.md', async () => {
     const dir = await createFixtureDir('resolve-file')
-    await writeFixture(dir, 'agents/reviewer.md', '# Review\nCheck all code.')
+    await writeFixture(
+      dir,
+      'agents/reviewer.md',
+      dedent`
+        # Review
+        Check all code.
+      `,
+    )
 
     const manifest = {
       name: 'test',
@@ -216,7 +254,10 @@ describe('resolvePrompts', () => {
     const result = await resolvePrompts(manifest, dir)
     expect(result.ok).toBe(true)
     if (result.ok) {
-      expect(result.data.agents?.reviewer?.prompt).toBe('# Review\nCheck all code.')
+      expect(result.data.agents?.reviewer?.prompt).toBe(dedent`
+        # Review
+        Check all code.
+      `)
     }
   })
 
@@ -242,7 +283,14 @@ describe('resolvePrompts', () => {
 
   test('manifest without agents or commands resolves successfully', async () => {
     const dir = await createFixtureDir('resolve-skills-only')
-    await writeFixture(dir, 'skills/x.md', '# Skill X\nDo x.')
+    await writeFixture(
+      dir,
+      'skills/x/SKILL.md',
+      dedent`
+        # Skill X
+        Do x.
+      `,
+    )
 
     const manifest = {
       name: 'test',
@@ -258,7 +306,10 @@ describe('resolvePrompts', () => {
     expect(result.ok).toBe(true)
     if (result.ok) {
       expect(result.data.name).toBe('test')
-      expect(result.data.skills?.x?.prompt).toBe('# Skill X\nDo x.')
+      expect(result.data.skills?.x?.prompt).toBe(dedent`
+        # Skill X
+        Do x.
+      `)
     }
   })
 })