@plaited/development-skills 0.6.4 → 0.7.0

package/{.plaited → .agents}/rules/accuracy.md

@@ -28,4 +28,4 @@
 - Code review: Read files before commenting
 - Patterns: Confirm examples reflect actual usage
 
-See .plaited/rules/testing.md for verification in test contexts.
+See .agents/rules/testing.md for verification in test contexts.

package/{.plaited → .agents}/rules/testing.md

@@ -14,7 +14,7 @@
 
 **Use real dependencies** - Prefer installed packages over mocks when testing module resolution  
 *Verify:* Review test imports for fake paths  
-*Fix:* Use actual package like `@modelcontextprotocol/sdk/client`
+*Fix:* Use actual package like `typescript`
 
 **Organize with describe** - Group related tests in `describe('feature', () => {...})`  
 *Verify:* Check for flat test structure  

package/{.plaited → .agents}/skills/scaffold-rules/SKILL.md

@@ -34,7 +34,7 @@ bunx @plaited/development-skills scaffold-rules
 ```
 
 This will:
-1. Copy rules to `.plaited/rules/` (canonical location)
+1. Copy rules to `.agents/rules/` (canonical location)
 2. Create symlinks in `.claude/rules` and `.cursor/rules` (if those directories exist)
 3. Fallback: append links to `AGENTS.md` if no agent directories found
 
@@ -52,23 +52,23 @@ Tell the user what was created based on the `actions` output.
 ## How It Works
 
 ```
-.plaited/rules/          ← Canonical location (files copied here)
+.agents/rules/          ← Canonical location (files copied here)
     ├── testing.md
     ├── bun.md
     └── ...
 
-.claude/rules -> ../.plaited/rules   ← Symlink (if .claude/ exists)
-.cursor/rules -> ../.plaited/rules   ← Symlink (if .cursor/ exists)
+.claude/rules -> ../.agents/rules   ← Symlink (if .claude/ exists)
+.cursor/rules -> ../.agents/rules   ← Symlink (if .cursor/ exists)
 ```
 
 | Project has... | Copy | Symlinks | AGENTS.md |
 |----------------|------|----------|-----------|
-| `.plaited/` only | ✓ | None | No |
+| `.agents/` only | ✓ | None | No |
 | `.claude/` only | ✓ | `.claude/rules` | No |
 | `.cursor/` only | ✓ | `.cursor/rules` | No |
-| `.plaited/` + `.claude/` | ✓ | `.claude/rules` | No |
-| `.plaited/` + `.cursor/` | ✓ | `.cursor/rules` | No |
-| `.plaited/` + `.claude/` + `.cursor/` | ✓ | Both | No |
+| `.agents/` + `.claude/` | ✓ | `.claude/rules` | No |
+| `.agents/` + `.cursor/` | ✓ | `.cursor/rules` | No |
+| `.agents/` + `.claude/` + `.cursor/` | ✓ | Both | No |
 | None of the above | ✓ | None | ✓ Append links |
 
 ## Related Skills

package/README.md

@@ -42,11 +42,11 @@ LSP understands re-exports, aliases, and type relationships.
 ## Install for AI Agents
 
 ```bash
-curl -fsSL https://raw.githubusercontent.com/plaited/skills-installer/main/install.sh | bash -s -- --agent <agent-name> --project development-skills
+npx skills add plaited/development-skills
+# or
+bunx skills add plaited/development-skills
 ```
 
-**Agents:** `claude` · `cursor` · `copilot` · `codex` · `gemini` · `windsurf` · `opencode` · `amp` · `goose` · `factory`
-
 ## Commands
 
 | Command | What it does |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plaited/development-skills",
-  "version": "0.6.4",
+  "version": "0.7.0",
   "description": "Development skills for Claude Code - TypeScript LSP, code documentation, and validation tools",
   "license": "ISC",
   "engines": {
@@ -21,7 +21,7 @@
   "files": [
     "bin/",
     "src/",
-    ".plaited/"
+    ".agents/"
   ],
   "publishConfig": {
     "access": "public"
@@ -48,7 +48,6 @@
   },
   "devDependencies": {
     "@biomejs/biome": "2.3.11",
-    "@modelcontextprotocol/sdk": "1.22.0",
     "@types/bun": "1.3.6",
     "format-package": "7.0.0",
     "lint-staged": "16.2.7",

package/src/scaffold-rules.ts

@@ -2,7 +2,7 @@
 /**
  * Scaffold development rules - Copy bundled rules and create symlinks
  *
- * Copies rules from the package to `.plaited/rules/` (canonical location),
+ * Copies rules from the package to `.agents/rules/` (canonical location),
  * creates symlinks for `.claude/` and `.cursor/` agent directories,
  * and falls back to appending links in `AGENTS.md` if no agent dirs exist.
  *
@@ -15,18 +15,18 @@ import { mkdir, readdir, readlink, stat, symlink } from 'node:fs/promises'
 import { join } from 'node:path'
 import { parseArgs } from 'node:util'
 
-/** Agents that get symlinks to .plaited/rules (not .plaited itself) */
+/** Agents that get symlinks to .agents/rules (not .agents itself) */
 const SYMLINK_AGENTS = ['.claude', '.cursor'] as const
 
-/** All supported agent directories (including .plaited which gets direct copy) */
-const ALL_AGENTS = ['.plaited', ...SYMLINK_AGENTS] as const
+/** All supported agent directories (including .agents which gets direct copy) */
+const ALL_AGENTS = ['.agents', ...SYMLINK_AGENTS] as const
 
 /** Canonical rules location */
-const TARGET_RULES = '.plaited/rules' as const
+const TARGET_RULES = '.agents/rules' as const
 
 /**
  * NOTE: This tool only scaffolds RULES, not skills.
- * Skills symlinks (.claude/skills -> ../.plaited/skills) are managed separately
+ * Skills symlinks (.claude/skills -> ../.agents/skills) are managed separately
  * via the skills-installer or manual setup.
  */
 
@@ -59,7 +59,7 @@ export const scaffoldRules = async (args: string[]): Promise<void> => {
   const dryRun = values['dry-run'] as boolean | undefined
   const listOnly = values.list as boolean | undefined
 
-  const sourceRules = join(import.meta.dir, '../.plaited/rules')
+  const sourceRules = join(import.meta.dir, '../.agents/rules')
   const cwd = process.cwd()
 
   // Get available rules
@@ -74,7 +74,7 @@ export const scaffoldRules = async (args: string[]): Promise<void> => {
 
   const actions: string[] = []
 
-  // Check for agent directories BEFORE copying (since copy creates .plaited/)
+  // Check for agent directories BEFORE copying (since copy creates .agents/)
   // This determines whether to fall back to AGENTS.md append
   let hadAgentDirBeforeScaffold = false
   for (const agent of ALL_AGENTS) {
@@ -84,7 +84,7 @@ export const scaffoldRules = async (args: string[]): Promise<void> => {
     }
   }
 
-  // 1. Copy rules to .plaited/rules/ (canonical location, serves .plaited agent)
+  // 1. Copy rules to .agents/rules/ (canonical location, serves .agents agent)
   const targetDir = join(cwd, TARGET_RULES)
   if (!dryRun) {
     await mkdir(targetDir, { recursive: true })
@@ -108,7 +108,7 @@ export const scaffoldRules = async (args: string[]): Promise<void> => {
       // Check if symlink already exists and points to right place
       try {
         const existing = await readlink(rulesLink)
-        if (existing === '../.plaited/rules') {
+        if (existing === '../.agents/rules') {
           actions.push(`skip: ${agent}/rules (symlink exists)`)
           continue
         }
@@ -117,9 +117,9 @@ export const scaffoldRules = async (args: string[]): Promise<void> => {
       }
 
       if (!dryRun) {
-        await symlink('../.plaited/rules', rulesLink)
+        await symlink('../.agents/rules', rulesLink)
       }
-      actions.push(`symlink: ${agent}/rules -> ../.plaited/rules`)
+      actions.push(`symlink: ${agent}/rules -> ../.agents/rules`)
     }
   }
 
@@ -130,7 +130,7 @@ export const scaffoldRules = async (args: string[]): Promise<void> => {
 
     if (await agentsMd.exists()) {
       const content = await agentsMd.text()
-      if (content.includes('.plaited/rules')) {
+      if (content.includes('.agents/rules')) {
         actions.push('skip: AGENTS.md (already has rules)')
       } else {
         const links = rules.map((f) => `- [${f.replace('.md', '')}](${TARGET_RULES}/${f})`).join('\n')

package/src/tests/resolve-file-path.spec.ts

@@ -75,27 +75,6 @@ describe('resolveFilePath', () => {
   })
 
   describe('scoped package specifiers', () => {
-    test('resolves scoped package subpath export', () => {
-      // @modelcontextprotocol/sdk/client is a defined export
-      const result = resolveFilePath('@modelcontextprotocol/sdk/client')
-      expect(result).toContain('node_modules/@modelcontextprotocol/sdk')
-      expect(result).toContain('client')
-    })
-
-    test('resolves scoped package server subpath', () => {
-      const result = resolveFilePath('@modelcontextprotocol/sdk/server')
-      expect(result).toContain('node_modules/@modelcontextprotocol/sdk')
-      expect(result).toContain('server')
-    })
-
-    test('resolves scoped package deep subpath with extension', () => {
-      // Deep subpath with .js extension via wildcard export "./*"
-      const path = '@modelcontextprotocol/sdk/server/auth/middleware/bearerAuth.js'
-      const result = resolveFilePath(path)
-      expect(result).toContain('node_modules/@modelcontextprotocol/sdk')
-      expect(result).toContain('bearerAuth')
-    })
-
     test('falls back to cwd for non-existent scoped package', () => {
       const scopedPkg = '@nonexistent/pkg/src/file.ts'
       const result = resolveFilePath(scopedPkg)

package/src/tests/scaffold-rules.spec.ts

@@ -63,7 +63,7 @@ describe('scaffold-rules', () => {
       const result: ScaffoldOutput = await $`cd ${testDir} && bun ${binDir}/cli.ts scaffold-rules --dry-run`.json()
 
       expect(result.dryRun).toBe(true)
-      expect(result.targetRules).toBe('.plaited/rules')
+      expect(result.targetRules).toBe('.agents/rules')
       expect(result.actions).toBeArray()
     })
 
@@ -81,7 +81,7 @@ describe('scaffold-rules', () => {
     test('does not create files in dry-run mode', async () => {
       await $`cd ${testDir} && bun ${binDir}/cli.ts scaffold-rules --dry-run`.json()
 
-      const rulesDir = Bun.file(join(testDir, '.plaited/rules'))
+      const rulesDir = Bun.file(join(testDir, '.agents/rules'))
       expect(await rulesDir.exists()).toBe(false)
     })
 
@@ -93,10 +93,10 @@ describe('scaffold-rules', () => {
   })
 
   describe('copy behavior', () => {
-    test('copies rules to .plaited/rules/', async () => {
+    test('copies rules to .agents/rules/', async () => {
       await $`cd ${testDir} && bun ${binDir}/cli.ts scaffold-rules`.quiet()
 
-      const coreRule = Bun.file(join(testDir, '.plaited/rules/core.md'))
+      const coreRule = Bun.file(join(testDir, '.agents/rules/core.md'))
       expect(await coreRule.exists()).toBe(true)
 
       const content = await coreRule.text()
@@ -109,15 +109,15 @@ describe('scaffold-rules', () => {
       const expectedRules = ['accuracy', 'bun', 'core', 'documentation', 'modules', 'testing', 'workflow']
 
       for (const rule of expectedRules) {
-        const ruleFile = Bun.file(join(testDir, `.plaited/rules/${rule}.md`))
+        const ruleFile = Bun.file(join(testDir, `.agents/rules/${rule}.md`))
         expect(await ruleFile.exists()).toBe(true)
       }
     })
 
-    test('creates .plaited/rules directory if missing', async () => {
+    test('creates .agents/rules directory if missing', async () => {
       await $`cd ${testDir} && bun ${binDir}/cli.ts scaffold-rules`.quiet()
 
-      const rulesDir = await Bun.file(join(testDir, '.plaited/rules/core.md')).exists()
+      const rulesDir = await Bun.file(join(testDir, '.agents/rules/core.md')).exists()
       expect(rulesDir).toBe(true)
     })
   })
@@ -131,7 +131,7 @@ describe('scaffold-rules', () => {
 
       // Check symlink exists and points to right place
       const linkTarget = await readlink(join(testDir, '.claude/rules'))
-      expect(linkTarget).toBe('../.plaited/rules')
+      expect(linkTarget).toBe('../.agents/rules')
     })
 
     test('creates symlink for .cursor/rules when .cursor exists', async () => {
@@ -141,7 +141,7 @@ describe('scaffold-rules', () => {
       await $`cd ${testDir} && bun ${binDir}/cli.ts scaffold-rules`.quiet()
 
       const linkTarget = await readlink(join(testDir, '.cursor/rules'))
-      expect(linkTarget).toBe('../.plaited/rules')
+      expect(linkTarget).toBe('../.agents/rules')
     })
 
     test('skips symlink if already exists with correct target', async () => {
@@ -174,22 +174,22 @@ describe('scaffold-rules', () => {
 
       const content = await Bun.file(join(testDir, 'AGENTS.md')).text()
       expect(content).toContain('## Rules')
-      expect(content).toContain('.plaited/rules/')
+      expect(content).toContain('.agents/rules/')
     })
 
     test('does not append if agent dir exists', async () => {
       await Bun.write(join(testDir, 'AGENTS.md'), '# AGENTS\n\nSome content\n')
-      await mkdir(join(testDir, '.plaited'), { recursive: true })
+      await mkdir(join(testDir, '.agents'), { recursive: true })
 
       await $`cd ${testDir} && bun ${binDir}/cli.ts scaffold-rules`.quiet()
 
       const content = await Bun.file(join(testDir, 'AGENTS.md')).text()
-      // Should not have appended rules section (since .plaited exists)
+      // Should not have appended rules section (since .agents exists)
       expect(content).not.toContain('## Rules')
     })
 
     test('skips if AGENTS.md already has rules', async () => {
-      await Bun.write(join(testDir, 'AGENTS.md'), '# AGENTS\n\nSee .plaited/rules/ for rules\n')
+      await Bun.write(join(testDir, 'AGENTS.md'), '# AGENTS\n\nSee .agents/rules/ for rules\n')
 
       const result: ScaffoldOutput = await $`cd ${testDir} && bun ${binDir}/cli.ts scaffold-rules`.json()
 
@@ -207,7 +207,7 @@ describe('scaffold-rules', () => {
       expect(result).toHaveProperty('actions')
 
       expect(result.dryRun).toBe(false)
-      expect(result.targetRules).toBe('.plaited/rules')
+      expect(result.targetRules).toBe('.agents/rules')
       expect(result.actions).toBeArray()
     })
   })
@@ -248,7 +248,7 @@ describe('scaffold-rules', () => {
     test('core.md contains TypeScript conventions', async () => {
       await $`cd ${testDir} && bun ${binDir}/cli.ts scaffold-rules`.quiet()
 
-      const content = await Bun.file(join(testDir, '.plaited/rules/core.md')).text()
+      const content = await Bun.file(join(testDir, '.agents/rules/core.md')).text()
       expect(content).toContain('Type over interface')
       expect(content).toContain('Arrow functions')
     })
@@ -256,7 +256,7 @@ describe('scaffold-rules', () => {
     test('testing.md contains test conventions', async () => {
       await $`cd ${testDir} && bun ${binDir}/cli.ts scaffold-rules`.quiet()
 
-      const content = await Bun.file(join(testDir, '.plaited/rules/testing.md')).text()
+      const content = await Bun.file(join(testDir, '.agents/rules/testing.md')).text()
       expect(content).toContain('test not it')
       expect(content).toContain('No conditional assertions')
     })
@@ -264,7 +264,7 @@ describe('scaffold-rules', () => {
     test('rules include verification patterns', async () => {
       await $`cd ${testDir} && bun ${binDir}/cli.ts scaffold-rules`.quiet()
 
-      const content = await Bun.file(join(testDir, '.plaited/rules/core.md')).text()
+      const content = await Bun.file(join(testDir, '.agents/rules/core.md')).text()
       expect(content).toContain('*Verify:*')
       expect(content).toContain('*Fix:*')
     })
@@ -272,7 +272,7 @@ describe('scaffold-rules', () => {
     test('rules are compressed (no verbose examples)', async () => {
       await $`cd ${testDir} && bun ${binDir}/cli.ts scaffold-rules`.quiet()
 
-      const content = await Bun.file(join(testDir, '.plaited/rules/core.md')).text()
+      const content = await Bun.file(join(testDir, '.agents/rules/core.md')).text()
 
       // Should not have verbose code blocks with Good/Avoid patterns
       expect(content).not.toContain('// ✅ Good')