@drawcall/market 0.1.26 → 0.1.28

package/tests/resolve-skills.test.ts

@@ -0,0 +1,87 @@
+import assert from 'node:assert/strict'
+import test from 'node:test'
+import { resolve, ResolutionError } from '../src/resolve.js'
+
+interface FakeAsset {
+  latestVersion: string
+  approved?: boolean
+  npmDependencies?: Record<string, string>
+  assetDependencies?: Record<string, string>
+  skillDependencies?: Record<string, string>
+}
+
+function clientWith(assets: Record<string, FakeAsset>) {
+  return {
+    exact: async ({ name }: { name: string }) => {
+      const asset = assets[name]
+      if (!asset) return null
+      return {
+        name,
+        latestVersion: asset.latestVersion,
+        approved: asset.approved ?? true,
+        npmDependencies: JSON.stringify(asset.npmDependencies ?? {}),
+        assetDependencies: JSON.stringify(asset.assetDependencies ?? {}),
+        skillDependencies: JSON.stringify(asset.skillDependencies ?? {}),
+      }
+    },
+  } as never
+}
+
+test('resolve collects skill dependencies across transitive assets', async () => {
+  const client = clientWith({
+    root: {
+      latestVersion: '1.0.0',
+      assetDependencies: { dep: '*' },
+      skillDependencies: { 'web-design-guidelines': 'vercel-labs/agent-skills' },
+    },
+    dep: {
+      latestVersion: '2.0.0',
+      skillDependencies: { 'local-skill': './public/skills/local-skill' },
+    },
+  })
+
+  const result = await resolve(client, [{ name: 'root', range: '*' }])
+
+  assert.deepEqual(result.skillDependencies, {
+    'web-design-guidelines': 'vercel-labs/agent-skills',
+    'local-skill': './public/skills/local-skill',
+  })
+})
+
+test('resolve unions identical skill sources without complaint', async () => {
+  const client = clientWith({
+    root: {
+      latestVersion: '1.0.0',
+      assetDependencies: { dep: '*' },
+      skillDependencies: { shared: 'vercel-labs/agent-skills' },
+    },
+    dep: {
+      latestVersion: '1.0.0',
+      skillDependencies: { shared: 'vercel-labs/agent-skills' },
+    },
+  })
+
+  const result = await resolve(client, [{ name: 'root', range: '*' }])
+
+  assert.deepEqual(result.skillDependencies, { shared: 'vercel-labs/agent-skills' })
+})
+
+test('resolve rejects the same skill label with conflicting sources', async () => {
+  const client = clientWith({
+    root: {
+      latestVersion: '1.0.0',
+      assetDependencies: { dep: '*' },
+      skillDependencies: { shared: 'vercel-labs/agent-skills' },
+    },
+    dep: {
+      latestVersion: '1.0.0',
+      skillDependencies: { shared: 'other-org/other-skills' },
+    },
+  })
+
+  await assert.rejects(
+    resolve(client, [{ name: 'root', range: '*' }]),
+    (err: unknown) =>
+      err instanceof ResolutionError && /skill dependency conflict for "shared"/u.test(err.message),
+  )
+})

package/tests/upload-deps.test.ts

@@ -0,0 +1,44 @@
+import assert from 'node:assert/strict'
+import test from 'node:test'
+import { parseSkillDeps, parseVersionedDeps } from '../src/commands/upload.js'
+
+test('parseVersionedDeps parses name@range and defaults a missing range to *', () => {
+  assert.deepEqual(parseVersionedDeps(['three@^0.178.0', 'gsap'], 'npm'), {
+    three: '^0.178.0',
+    gsap: '*',
+  })
+})
+
+test('parseVersionedDeps keeps the scope marker on scoped npm packages', () => {
+  assert.deepEqual(parseVersionedDeps(['@react-three/drei@^9.0.0', '@types/node'], 'npm'), {
+    '@react-three/drei': '^9.0.0',
+    '@types/node': '*',
+  })
+})
+
+test('parseVersionedDeps rejects duplicates', () => {
+  assert.throws(() => parseVersionedDeps(['three@^1', 'three@^2'], 'npm'), /Duplicate npm dependency/u)
+})
+
+test('parseSkillDeps splits label=source on the first = only', () => {
+  assert.deepEqual(
+    parseSkillDeps([
+      'web-design=vercel-labs/agent-skills/tree/main/skills/web-design-guidelines',
+      'local=./public/skills/local',
+    ]),
+    {
+      'web-design': 'vercel-labs/agent-skills/tree/main/skills/web-design-guidelines',
+      local: './public/skills/local',
+    },
+  )
+})
+
+test('parseSkillDeps rejects specs without a usable label=source', () => {
+  assert.throws(() => parseSkillDeps(['no-separator']), /Use label=source/u)
+  assert.throws(() => parseSkillDeps(['=missing-label']), /Use label=source/u)
+  assert.throws(() => parseSkillDeps(['missing-source=']), /Use label=source/u)
+})
+
+test('parseSkillDeps rejects duplicate labels', () => {
+  assert.throws(() => parseSkillDeps(['a=one', 'a=two']), /Duplicate skill dependency "a"/u)
+})