Merge pull request #2 from AJAmit17/merge-fix

Merge fix

Author: AJAmit17

README.md

@@ -360,6 +360,7 @@ Adapters are used by both `export` and `run`. Available adapters:
 | `gemini` | Google Gemini CLI (GEMINI.md + settings.json) |
 | `openclaw` | OpenClaw format |
 | `nanobot` | Nanobot format |
+| `cursor` | Cursor `.cursor/rules/*.mdc` files |
 
 ```bash
 # Export to system prompt

src/adapters/cursor.test.ts

@@ -0,0 +1,230 @@
+/**
+ * Tests for the Cursor adapter (export + enhanced import).
+ *
+ * Uses Node.js built-in test runner (node --test).
+ */
+import { test, describe } from 'node:test';
+import assert from 'node:assert/strict';
+import { mkdtempSync, writeFileSync, mkdirSync } from 'node:fs';
+import { join } from 'node:path';
+import { tmpdir } from 'node:os';
+
+import {
+  exportToCursor,
+  exportToCursorString,
+  parseMdcFile,
+  readCursorRules,
+} from './cursor.js';
+
+// ---------------------------------------------------------------------------
+// Helpers
+// ---------------------------------------------------------------------------
+
+/** Create a minimal gitagent directory in a temp folder. */
+function makeAgentDir(opts: {
+  name?: string;
+  description?: string;
+  soul?: string;
+  rules?: string;
+  skills?: Array<{ name: string; description: string; instructions: string; globs?: string }>;
+}): string {
+  const dir = mkdtempSync(join(tmpdir(), 'gitagent-cursor-test-'));
+
+  const manifest = {
+    spec_version: '0.1.0',
+    name: opts.name ?? 'test-agent',
+    version: '0.1.0',
+    description: opts.description ?? 'A test agent',
+  };
+
+  writeFileSync(
+    join(dir, 'agent.yaml'),
+    `spec_version: '0.1.0'\nname: ${manifest.name}\nversion: '0.1.0'\ndescription: '${manifest.description}'\n`,
+    'utf-8',
+  );
+
+  if (opts.soul !== undefined) {
+    writeFileSync(join(dir, 'SOUL.md'), opts.soul, 'utf-8');
+  }
+
+  if (opts.rules !== undefined) {
+    writeFileSync(join(dir, 'RULES.md'), opts.rules, 'utf-8');
+  }
+
+  if (opts.skills) {
+    for (const skill of opts.skills) {
+      const skillDir = join(dir, 'skills', skill.name);
+      mkdirSync(skillDir, { recursive: true });
+      const metadataLine = skill.globs ? `metadata:\n  globs: ${skill.globs}\n` : '';
+      writeFileSync(
+        join(skillDir, 'SKILL.md'),
+        `---\nname: ${skill.name}\ndescription: '${skill.description}'\n${metadataLine}---\n\n${skill.instructions}\n`,
+        'utf-8',
+      );
+    }
+  }
+
+  return dir;
+}
+
+// ---------------------------------------------------------------------------
+// parseMdcFile
+// ---------------------------------------------------------------------------
+
+describe('parseMdcFile', () => {
+  test('parses valid frontmatter + body', () => {
+    const content = `---\ndescription: "Hello"\nalwaysApply: true\n---\n\n# Body\n\nSome content.\n`;
+    const result = parseMdcFile(content);
+    assert.equal(result.frontmatter.description, 'Hello');
+    assert.equal(result.frontmatter.alwaysApply, true);
+    assert.match(result.body, /# Body/);
+  });
+
+  test('handles missing frontmatter gracefully', () => {
+    const content = `Just plain markdown, no frontmatter.`;
+    const result = parseMdcFile(content);
+    assert.deepEqual(result.frontmatter, {});
+    assert.equal(result.body, content);
+  });
+
+  test('parses array globs', () => {
+    const content = `---\nglobs:\n  - "*.ts"\n  - "src/**"\nalwaysApply: false\n---\n\nbody\n`;
+    const result = parseMdcFile(content);
+    assert.deepEqual(result.frontmatter.globs, ['*.ts', 'src/**']);
+  });
+});
+
+// ---------------------------------------------------------------------------
+// readCursorRules
+// ---------------------------------------------------------------------------
+
+describe('readCursorRules', () => {
+  test('returns empty array when no .cursor/rules dir exists', () => {
+    const dir = mkdtempSync(join(tmpdir(), 'empty-'));
+    const rules = readCursorRules(dir);
+    assert.deepEqual(rules, []);
+  });
+
+  test('reads .mdc files from .cursor/rules/', () => {
+    const dir = mkdtempSync(join(tmpdir(), 'cursor-rules-'));
+    const rulesDir = join(dir, '.cursor', 'rules');
+    mkdirSync(rulesDir, { recursive: true });
+    writeFileSync(join(rulesDir, 'rule-a.mdc'), `---\nalwaysApply: true\n---\n\nbody a\n`, 'utf-8');
+    writeFileSync(join(rulesDir, 'rule-b.mdc'), `---\nalwaysApply: false\n---\n\nbody b\n`, 'utf-8');
+    // Non-.mdc file should be ignored
+    writeFileSync(join(rulesDir, 'ignored.md'), 'should be ignored', 'utf-8');
+
+    const rules = readCursorRules(dir);
+    assert.equal(rules.length, 2);
+    const filenames = rules.map(r => r.filename).sort();
+    assert.deepEqual(filenames, ['rule-a.mdc', 'rule-b.mdc']);
+  });
+});
+
+// ---------------------------------------------------------------------------
+// exportToCursor — global rule
+// ---------------------------------------------------------------------------
+
+describe('exportToCursor — global rule', () => {
+  test('emits alwaysApply rule when SOUL.md exists', () => {
+    const dir = makeAgentDir({ soul: '# I am a soul', description: 'My agent' });
+    const exp = exportToCursor(dir);
+    const global = exp.rules.find(r => r.content.includes('alwaysApply: true'));
+    assert.ok(global, 'Expected an alwaysApply: true rule');
+    assert.match(global!.content, /I am a soul/);
+  });
+
+  test('includes RULES.md content in global rule', () => {
+    const dir = makeAgentDir({ soul: '# Soul', rules: '# Never lie' });
+    const exp = exportToCursor(dir);
+    const global = exp.rules.find(r => r.content.includes('alwaysApply: true'));
+    assert.ok(global);
+    assert.match(global!.content, /Never lie/);
+  });
+
+  test('no global rule emitted when neither SOUL.md nor RULES.md exists', () => {
+    const dir = makeAgentDir({});
+    const exp = exportToCursor(dir);
+    const global = exp.rules.filter(r => r.content.includes('alwaysApply: true'));
+    assert.equal(global.length, 0);
+  });
+});
+
+// ---------------------------------------------------------------------------
+// exportToCursor — skill rules
+// ---------------------------------------------------------------------------
+
+describe('exportToCursor — skill rules', () => {
+  test('emits one skill rule per skill', () => {
+    const dir = makeAgentDir({
+      skills: [
+        { name: 'code-review', description: 'Reviews code', instructions: 'Check for bugs.' },
+        { name: 'docs-writer', description: 'Writes docs', instructions: 'Write clear docs.' },
+      ],
+    });
+    const exp = exportToCursor(dir);
+    const skillRules = exp.rules.filter(r => !r.content.includes('alwaysApply: true'));
+    assert.equal(skillRules.length, 2);
+  });
+
+  test('skill rule filename is slugified skill name', () => {
+    const dir = makeAgentDir({
+      skills: [{ name: 'my-skill', description: 'Skill', instructions: 'Do stuff.' }],
+    });
+    const exp = exportToCursor(dir);
+    const rule = exp.rules.find(r => r.filename === 'my-skill.mdc');
+    assert.ok(rule, 'Expected my-skill.mdc');
+  });
+
+  test('skill rule includes globs when metadata.globs is set', () => {
+    const dir = makeAgentDir({
+      skills: [
+        {
+          name: 'api-handler',
+          description: 'API handler review',
+          instructions: 'Check endpoints.',
+          globs: 'src/api/** *.route.ts',
+        },
+      ],
+    });
+    const exp = exportToCursor(dir);
+    const rule = exp.rules.find(r => r.filename === 'api-handler.mdc');
+    assert.ok(rule);
+    assert.match(rule!.content, /globs:/);
+    assert.match(rule!.content, /src\/api\/\*\*/);
+    assert.match(rule!.content, /\*\.route\.ts/);
+  });
+
+  test('skill rule has alwaysApply: false', () => {
+    const dir = makeAgentDir({
+      skills: [{ name: 'linter', description: 'Lints', instructions: 'Lint everything.' }],
+    });
+    const exp = exportToCursor(dir);
+    const rule = exp.rules.find(r => r.filename === 'linter.mdc');
+    assert.ok(rule);
+    assert.match(rule!.content, /alwaysApply: false/);
+  });
+});
+
+// ---------------------------------------------------------------------------
+// exportToCursorString
+// ---------------------------------------------------------------------------
+
+describe('exportToCursorString', () => {
+  test('returns string with file path headers', () => {
+    const dir = makeAgentDir({
+      soul: '# Soul',
+      skills: [{ name: 'test-skill', description: 'Test', instructions: 'Do test.' }],
+    });
+    const output = exportToCursorString(dir);
+    assert.match(output, /# === \.cursor\/rules\//);
+    assert.match(output, /\.mdc ===/);
+  });
+
+  test('output is non-empty string', () => {
+    const dir = makeAgentDir({ soul: '# Soul' });
+    const output = exportToCursorString(dir);
+    assert.ok(output.length > 0);
+    assert.equal(typeof output, 'string');
+  });
+});