skills-package-manager 0.1.1

package/test/install.test.ts

@@ -0,0 +1,169 @@
+import { describe, expect, it } from '@rstest/core'
+import { mkdtempSync, existsSync, lstatSync, readFileSync, writeFileSync } from 'node:fs'
+import { tmpdir } from 'node:os'
+import path from 'node:path'
+import YAML from 'yaml'
+import { installSkills } from '../src/install/installSkills'
+import { writeSkillsManifest } from '../src/config/writeSkillsManifest'
+import { writeSkillsLock } from '../src/config/writeSkillsLock'
+
+describe('installSkills', () => {
+  it('installs a local skill and creates symlinks', async () => {
+    const root = mkdtempSync(path.join(tmpdir(), 'skills-pm-install-'))
+    await writeSkillsManifest(root, {
+      installDir: '.agents/skills',
+      linkTargets: ['.cursor/skills'],
+      skills: {
+        'hello-skill': `file:${path.resolve(__dirname, 'fixtures/local-source')}#path:/skills/hello-skill`,
+      },
+    })
+    await writeSkillsLock(root, {
+      lockfileVersion: '0.1',
+      installDir: '.agents/skills',
+      linkTargets: ['.cursor/skills'],
+      skills: {
+        'hello-skill': {
+          specifier: `file:${path.resolve(__dirname, 'fixtures/local-source')}#path:/skills/hello-skill`,
+          resolution: {
+            type: 'file',
+            path: path.resolve(__dirname, 'fixtures/local-source'),
+          },
+          digest: 'test-digest',
+        },
+      },
+    })
+
+    await installSkills(root)
+
+    const installedSkill = path.join(root, '.agents/skills/hello-skill/SKILL.md')
+    const linkedSkill = path.join(root, '.cursor/skills/hello-skill')
+    expect(existsSync(installedSkill)).toBe(true)
+    expect(lstatSync(linkedSkill).isSymbolicLink()).toBe(true)
+    expect(readFileSync(installedSkill, 'utf8')).toContain('Hello skill')
+  })
+
+  it('installs a git skill from a local git repository', async () => {
+    const root = mkdtempSync(path.join(tmpdir(), 'skills-pm-install-git-'))
+    const gitRepo = mkdtempSync(path.join(tmpdir(), 'skills-pm-git-source-'))
+
+    require('node:fs').mkdirSync(path.join(gitRepo, 'skills/hello-git-skill'), { recursive: true })
+    require('node:fs').writeFileSync(path.join(gitRepo, 'skills/hello-git-skill/SKILL.md'), '# Hello from git\n')
+    require('node:child_process').execSync('git init', { cwd: gitRepo, stdio: 'ignore' })
+    require('node:child_process').execSync('git config user.email test@example.com', { cwd: gitRepo, stdio: 'ignore' })
+    require('node:child_process').execSync('git config user.name test', { cwd: gitRepo, stdio: 'ignore' })
+    require('node:child_process').execSync('git add .', { cwd: gitRepo, stdio: 'ignore' })
+    require('node:child_process').execSync('git commit -m init', { cwd: gitRepo, stdio: 'ignore' })
+
+    await writeSkillsManifest(root, {
+      installDir: '.agents/skills',
+      linkTargets: [],
+      skills: {
+        'hello-git-skill': `${gitRepo}#HEAD&path:/skills/hello-git-skill`,
+      },
+    })
+    await writeSkillsLock(root, {
+      lockfileVersion: '0.1',
+      installDir: '.agents/skills',
+      linkTargets: [],
+      skills: {
+        'hello-git-skill': {
+          specifier: `${gitRepo}#HEAD&path:/skills/hello-git-skill`,
+          resolution: {
+            type: 'git',
+            url: gitRepo,
+            commit: 'HEAD',
+            path: '/skills/hello-git-skill',
+          },
+          digest: 'test-git-digest',
+        },
+      },
+    })
+
+    await installSkills(root)
+
+    const installedSkill = path.join(root, '.agents/skills/hello-git-skill/SKILL.md')
+    expect(existsSync(installedSkill)).toBe(true)
+    expect(readFileSync(installedSkill, 'utf8')).toContain('Hello from git')
+  })
+
+  it('updates stale lock entries from manifest before installing', async () => {
+    const root = mkdtempSync(path.join(tmpdir(), 'skills-pm-install-stale-lock-'))
+    const gitRepo = mkdtempSync(path.join(tmpdir(), 'skills-pm-git-stale-source-'))
+
+    require('node:fs').mkdirSync(path.join(gitRepo, 'skills/fixed-skill'), { recursive: true })
+    require('node:fs').writeFileSync(path.join(gitRepo, 'skills/fixed-skill/SKILL.md'), '# Fixed skill\n')
+    require('node:child_process').execSync('git init', { cwd: gitRepo, stdio: 'ignore' })
+    require('node:child_process').execSync('git config user.email test@example.com', { cwd: gitRepo, stdio: 'ignore' })
+    require('node:child_process').execSync('git config user.name test', { cwd: gitRepo, stdio: 'ignore' })
+    require('node:child_process').execSync('git add .', { cwd: gitRepo, stdio: 'ignore' })
+    require('node:child_process').execSync('git commit -m init', { cwd: gitRepo, stdio: 'ignore' })
+
+    await writeSkillsManifest(root, {
+      installDir: '.agents/skills',
+      linkTargets: [],
+      skills: {
+        'fixed-skill': `${gitRepo}#HEAD&path:/skills/fixed-skill`,
+      },
+    })
+
+    writeFileSync(
+      path.join(root, 'skills-lock.yaml'),
+      YAML.stringify({
+        lockfileVersion: '0.1',
+        installDir: '.agents/skills',
+        linkTargets: [],
+        skills: {
+          '': {
+            specifier: gitRepo,
+            resolution: {
+              type: 'git',
+              url: gitRepo,
+              commit: 'HEAD',
+              path: '/',
+            },
+            digest: 'bad-digest',
+          },
+        },
+      }),
+    )
+
+    await installSkills(root)
+
+    const installedSkill = path.join(root, '.agents/skills/fixed-skill/SKILL.md')
+    const rewrittenLock = YAML.parse(readFileSync(path.join(root, 'skills-lock.yaml'), 'utf8'))
+
+    expect(existsSync(installedSkill)).toBe(true)
+    expect(readFileSync(installedSkill, 'utf8')).toContain('Fixed skill')
+    expect(rewrittenLock.skills['fixed-skill'].specifier).toBe(`${gitRepo}#HEAD&path:/skills/fixed-skill`)
+    expect(rewrittenLock.skills['']).toBeUndefined()
+  })
+
+  it('removes managed skills that are no longer declared', async () => {
+    const root = mkdtempSync(path.join(tmpdir(), 'skills-pm-prune-'))
+
+    await writeSkillsManifest(root, {
+      installDir: '.agents/skills',
+      linkTargets: ['.cursor/skills'],
+      skills: {
+        'hello-skill': `file:${path.resolve(__dirname, 'fixtures/local-source')}#path:/skills/hello-skill`,
+        'obsolete-skill': `file:${path.resolve(__dirname, 'fixtures/local-source')}#path:/skills/hello-skill`,
+      },
+    })
+
+    await installSkills(root)
+
+    await writeSkillsManifest(root, {
+      installDir: '.agents/skills',
+      linkTargets: ['.cursor/skills'],
+      skills: {
+        'hello-skill': `file:${path.resolve(__dirname, 'fixtures/local-source')}#path:/skills/hello-skill`,
+      },
+    })
+
+    await installSkills(root)
+
+    expect(existsSync(path.join(root, '.agents/skills/obsolete-skill'))).toBe(false)
+    expect(existsSync(path.join(root, '.cursor/skills/obsolete-skill'))).toBe(false)
+    expect(existsSync(path.join(root, '.agents/skills/hello-skill'))).toBe(true)
+  })
+})

package/test/manifest.test.ts

@@ -0,0 +1,19 @@
+import { describe, expect, it } from '@rstest/core'
+import { mkdtempSync } from 'node:fs'
+import { tmpdir } from 'node:os'
+import path from 'node:path'
+import { writeSkillsManifest } from '../src/config/writeSkillsManifest'
+import { readSkillsManifest } from '../src/config/readSkillsManifest'
+
+describe('manifest io', () => {
+  it('writes default manifest shape', async () => {
+    const root = mkdtempSync(path.join(tmpdir(), 'skills-pm-'))
+    await writeSkillsManifest(root, { skills: { hello: 'file:./skills/hello' } })
+    const manifest = await readSkillsManifest(root)
+    expect(manifest).toEqual({
+      installDir: '.agents/skills',
+      linkTargets: [],
+      skills: { hello: 'file:./skills/hello' },
+    })
+  })
+})

package/test/specifiers.test.ts

@@ -0,0 +1,43 @@
+import { describe, expect, it } from '@rstest/core'
+import { normalizeSpecifier } from '../src/specifiers/normalizeSpecifier'
+
+describe('normalizeSpecifier', () => {
+  it('parses git path specifier', () => {
+    expect(normalizeSpecifier('https://github.com/acme/skills.git#main&path:/skills/hello')).toEqual({
+      type: 'git',
+      source: 'https://github.com/acme/skills.git',
+      ref: 'main',
+      path: '/skills/hello',
+      normalized: 'https://github.com/acme/skills.git#main&path:/skills/hello',
+      skillName: 'hello',
+    })
+  })
+
+  it('parses file path specifier', () => {
+    expect(normalizeSpecifier('file:./fixtures/local-source#path:/skills/hello-skill')).toEqual({
+      type: 'file',
+      source: 'file:./fixtures/local-source',
+      ref: null,
+      path: '/skills/hello-skill',
+      normalized: 'file:./fixtures/local-source#path:/skills/hello-skill',
+      skillName: 'hello-skill',
+    })
+  })
+
+  it('parses git specifier without ref', () => {
+    expect(normalizeSpecifier('https://github.com/acme/skills.git#path:/skills/world')).toEqual({
+      type: 'git',
+      source: 'https://github.com/acme/skills.git',
+      ref: null,
+      path: '/skills/world',
+      normalized: 'https://github.com/acme/skills.git#path:/skills/world',
+      skillName: 'world',
+    })
+  })
+
+  it('rejects duplicate path fragments', () => {
+    expect(() =>
+      normalizeSpecifier('https://github.com/acme/skills.git#path:/skills/world#path:/skills/world'),
+    ).toThrow('Invalid specifier: multiple # fragments are not supported')
+  })
+})

package/tsconfig.json

@@ -0,0 +1,8 @@
+{
+  "extends": "../../tsconfig.json",
+  "compilerOptions": {
+    "rootDir": ".",
+    "outDir": "dist"
+  },
+  "include": ["src", "test"]
+}