@dalzoubi/dev-agents-sync 1.0.9 → 1.0.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dalzoubi/dev-agents-sync",
-  "version": "1.0.9",
+  "version": "1.0.10",
   "type": "module",
   "description": "CLI that syncs managed dev-agent prompts into consumer repos (.claude/ and/or .cursor/).",
   "license": "UNLICENSED",

package/src/lockfile.mjs

@@ -14,7 +14,9 @@ export const DEFAULT_SOURCE = 'github:dalzoubi/dev-agents';
 export const SCHEMA_URL =
   'https://raw.githubusercontent.com/dalzoubi/dev-agents/v1/schema/lockfile.schema.json';
 
-const VALID_TARGETS = new Set(['claude', 'cursor']);
+// TODO(ask-first follow-up): simplify targets — once all consumers adopt Copilot,
+// adding 'cursor' could auto-include 'copilot' so callers don't need to list both.
+const VALID_TARGETS = new Set(['claude', 'cursor', 'copilot']);
 
 // Field order is load-bearing for determinism.
 const FIELD_ORDER = [

package/src/writer.mjs

@@ -15,6 +15,7 @@ import { hasMarker } from './marker.mjs';
 const TARGET_PREFIX = {
   claude: '.claude',
   cursor: '.cursor',
+  copilot: '.github',
 };
 
 // Subdirectories that uniquely identify a target. Used to normalize
@@ -39,7 +40,7 @@ export function normalizeFileMap(fileMap) {
   for (const [key, content] of Object.entries(fileMap)) {
     const norm = key.replace(/\\/g, '/');
     const first = norm.split('/')[0];
-    if (first === 'claude' || first === 'cursor') {
+    if (first === 'claude' || first === 'cursor' || first === 'copilot') {
       out[norm] = content;
       continue;
     }

package/tests/fixtures/release-v1.0.0/copilot/prompts/define.prompt.md

@@ -0,0 +1,17 @@
+---
+mode: "agent"
+description: "Planning and analysis agent for spec-driven development."
+tools: ["Read", "Grep", "Glob", "WebFetch", "WebSearch", "Agent", "TodoWrite", "Write"]
+---
+<!-- managed-by: dev-agents-sync v1.0.0 -->
+You are the **Define** agent: planning and analysis for spec-driven development.
+
+## Your posture
+
+Senior product-minded engineer. You do not write code. You produce a spec that is concrete enough for the Implement agent to execute.
+
+## Hard rules
+
+- **Read-only on source code.** You may only write the final spec markdown.
+- **One question at a time.** Never ask multi-part questions.
+- Honor the project's `CLAUDE.md` at the repo root.

package/tests/fixtures/release-v1.0.0/copilot/prompts/preflight.prompt.md

@@ -0,0 +1,7 @@
+---
+mode: "agent"
+description: "Pre-task orientation command. Run at the start of any session."
+tools: ["Read", "Grep", "Glob"]
+---
+<!-- managed-by: dev-agents-sync v1.0.0 -->
+Read CLAUDE.md and any AGENTS.md files. Orient yourself to the codebase before taking any action.

package/tests/lockfile.test.mjs

@@ -86,6 +86,16 @@ describe('validateLockfile', () => {
     assert.doesNotThrow(() => validateLockfile({ ...VALID_LOCKFILE, targets: ['cursor'] }));
   });
 
+  it('accepts targets: ["copilot"]', () => {
+    assert.doesNotThrow(() => validateLockfile({ ...VALID_LOCKFILE, targets: ['copilot'] }));
+  });
+
+  it('accepts targets: ["claude", "cursor", "copilot"]', () => {
+    assert.doesNotThrow(() =>
+      validateLockfile({ ...VALID_LOCKFILE, targets: ['claude', 'cursor', 'copilot'] }),
+    );
+  });
+
   it('rejects when source is missing', () => {
     const bad = { ...VALID_LOCKFILE };
     delete bad.source;

package/tests/paths.test.mjs

@@ -37,9 +37,9 @@ import { resolveConsumerPath } from '../src/writer.mjs';
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const FIXTURE_DIR = path.join(__dirname, 'fixtures', 'release-v1.0.0');
 
-function buildFixtureFileMap() {
+function buildFixtureFileMap(targets = ['claude', 'cursor', 'copilot']) {
   const map = {};
-  for (const target of ['claude', 'cursor']) {
+  for (const target of targets) {
     const targetDir = path.join(FIXTURE_DIR, target);
     if (!existsSync(targetDir)) continue;
     collectFiles(targetDir, targetDir, map, target);
@@ -113,6 +113,20 @@ describe('resolveConsumerPath', () => {
     assert.equal(result, expected);
   });
 
+  it('translates "copilot/prompts/define.prompt.md" to .github/prompts/define.prompt.md', () => {
+    const consumerRoot = '/some/repo';
+    const result = resolveConsumerPath(consumerRoot, 'copilot/prompts/define.prompt.md');
+    const expected = path.join(consumerRoot, '.github', 'prompts', 'define.prompt.md');
+    assert.equal(result, expected);
+  });
+
+  it('translates "copilot/instructions/foo.instructions.md" to .github/instructions/foo.instructions.md', () => {
+    const consumerRoot = '/some/repo';
+    const result = resolveConsumerPath(consumerRoot, 'copilot/instructions/foo.instructions.md');
+    const expected = path.join(consumerRoot, '.github', 'instructions', 'foo.instructions.md');
+    assert.equal(result, expected);
+  });
+
   it('works with a Windows-style absolute consumer root path', () => {
     // Simulate a Windows-style path
     const consumerRoot = 'C:\\Users\\developer\\projects\\my-app';
@@ -184,6 +198,20 @@ describe('init — platform-correct file paths', () => {
     );
   });
 
+  it('writes .github/prompts/define.prompt.md using the OS path separator', async () => {
+    await runInit(consumerDir, {
+      targets: ['copilot'],
+      fetcher: makeFixtureFetcher(),
+      availableTags: ['v1.0.0', 'v1.2.0'],
+    });
+
+    const expectedPath = path.join(consumerDir, '.github', 'prompts', 'define.prompt.md');
+    assert.ok(
+      existsSync(expectedPath),
+      `file must exist at OS-correct path: ${expectedPath}`,
+    );
+  });
+
   it('written file content is readable via path.join on this platform', async () => {
     await runInit(consumerDir, {
       targets: ['claude'],

package/tests/update.test.mjs

@@ -42,9 +42,9 @@ function readFixtureContent(relativePath) {
  * Builds a FileMap from the fixture directory.
  * Keys are relative paths using forward slashes.
  */
-function buildFixtureFileMap() {
+function buildFixtureFileMap(targets = ['claude', 'cursor', 'copilot']) {
   const map = {};
-  for (const target of ['claude', 'cursor']) {
+  for (const target of targets) {
     const targetDir = path.join(FIXTURE_DIR, target);
     if (!existsSync(targetDir)) continue;
     collectFiles(targetDir, targetDir, map, target);
@@ -320,3 +320,114 @@ describe('update — lockfile required', () => {
     }
   });
 });
+
+// ---------------------------------------------------------------------------
+// Copilot target (.github/)
+// ---------------------------------------------------------------------------
+
+describe('update — copilot target writes .github/', () => {
+  let consumerDir;
+
+  beforeEach(() => {
+    consumerDir = makeTmpDir();
+  });
+
+  afterEach(() => {
+    rmSync(consumerDir, { recursive: true, force: true });
+  });
+
+  it('writes .github/prompts/ files when targets includes copilot', async () => {
+    writeLockfile(consumerDir, {
+      source: 'github:dalzoubi/dev-agents',
+      range: '^1',
+      resolvedVersion: '1.0.0',
+      targets: ['copilot'],
+      lastUpdated: '2026-04-01T00:00:00Z',
+    });
+
+    await runUpdate(consumerDir, {
+      fetcher: makeV120Fetcher(),
+      availableTags: ['v1.0.0', 'v1.2.0'],
+    });
+
+    const expectedPath = path.join(consumerDir, '.github', 'prompts', 'define.prompt.md');
+    assert.ok(existsSync(expectedPath), `.github/prompts/define.prompt.md must be written`);
+    const content = readFileSync(expectedPath, 'utf8');
+    assert.ok(hasMarker(content), 'written file must have the managed-by marker');
+  });
+
+  it('does not write .claude/ or .cursor/ files when targets is only copilot', async () => {
+    writeLockfile(consumerDir, {
+      source: 'github:dalzoubi/dev-agents',
+      range: '^1',
+      resolvedVersion: '1.0.0',
+      targets: ['copilot'],
+      lastUpdated: '2026-04-01T00:00:00Z',
+    });
+
+    await runUpdate(consumerDir, {
+      fetcher: makeV120Fetcher(),
+      availableTags: ['v1.0.0', 'v1.2.0'],
+    });
+
+    assert.ok(
+      !existsSync(path.join(consumerDir, '.claude')),
+      '.claude/ must not be created for copilot-only target',
+    );
+    assert.ok(
+      !existsSync(path.join(consumerDir, '.cursor')),
+      '.cursor/ must not be created for copilot-only target',
+    );
+  });
+
+  it('writes all three target dirs when targets includes claude, cursor, and copilot', async () => {
+    writeLockfile(consumerDir, {
+      source: 'github:dalzoubi/dev-agents',
+      range: '^1',
+      resolvedVersion: '1.0.0',
+      targets: ['claude', 'cursor', 'copilot'],
+      lastUpdated: '2026-04-01T00:00:00Z',
+    });
+
+    await runUpdate(consumerDir, {
+      fetcher: makeV120Fetcher(),
+      availableTags: ['v1.0.0', 'v1.2.0'],
+    });
+
+    assert.ok(
+      existsSync(path.join(consumerDir, '.claude', 'agents', 'define.md')),
+      '.claude/agents/define.md must exist',
+    );
+    assert.ok(
+      existsSync(path.join(consumerDir, '.cursor', 'rules', 'define.mdc')),
+      '.cursor/rules/define.mdc must exist',
+    );
+    assert.ok(
+      existsSync(path.join(consumerDir, '.github', 'prompts', 'define.prompt.md')),
+      '.github/prompts/define.prompt.md must exist',
+    );
+  });
+
+  it('refuses to overwrite an unmanaged .github/ file without --force', async () => {
+    writeLockfile(consumerDir, {
+      source: 'github:dalzoubi/dev-agents',
+      range: '^1',
+      resolvedVersion: '1.0.0',
+      targets: ['copilot'],
+      lastUpdated: '2026-04-01T00:00:00Z',
+    });
+
+    // Pre-create an unmanaged file at the target path
+    const promptsDir = path.join(consumerDir, '.github', 'prompts');
+    mkdirSync(promptsDir, { recursive: true });
+    writeFileSync(path.join(promptsDir, 'define.prompt.md'), '# Unmanaged', 'utf8');
+
+    await assert.rejects(
+      runUpdate(consumerDir, {
+        fetcher: makeV120Fetcher(),
+        availableTags: ['v1.0.0', 'v1.2.0'],
+      }),
+      /unmanaged|marker|force/i,
+    );
+  });
+});