@orderful/droid 0.2.0 → 0.3.0

package/src/lib/skills.test.ts

@@ -1,5 +1,5 @@
 import { describe, it, expect, beforeEach, afterEach } from 'bun:test';
-import { existsSync, mkdirSync, rmSync, writeFileSync, readFileSync } from 'fs';
+import { existsSync, mkdirSync, rmSync, writeFileSync, readFileSync, readdirSync } from 'fs';
 import { join } from 'path';
 import { tmpdir } from 'os';
 import { homedir } from 'os';
@@ -13,6 +13,19 @@ import {
   getBundledSkillsDir,
 } from './skills.js';
 
+/**
+ * Parse YAML frontmatter from markdown content
+ */
+function parseFrontmatter(content: string): Record<string, unknown> | null {
+  const match = content.match(/^---\n([\s\S]*?)\n---/);
+  if (!match) return null;
+  try {
+    return YAML.parse(match[1]);
+  } catch {
+    return null;
+  }
+}
+
 describe('getSkillsInstallPath', () => {
   it('should return Claude Code path', () => {
     const path = getSkillsInstallPath(AITool.ClaudeCode);
@@ -136,3 +149,64 @@ describe('skill manifest parsing', () => {
     expect(parsed.provides_output).toBeUndefined();
   });
 });
+
+describe('bundled skills validation', () => {
+  it('all skills should have SKILL.md with valid frontmatter', () => {
+    const skillsDir = getBundledSkillsDir();
+    const skillDirs = readdirSync(skillsDir, { withFileTypes: true })
+      .filter((d) => d.isDirectory())
+      .map((d) => d.name);
+
+    for (const skillName of skillDirs) {
+      const skillMdPath = join(skillsDir, skillName, 'SKILL.md');
+      expect(existsSync(skillMdPath)).toBe(true);
+
+      const content = readFileSync(skillMdPath, 'utf-8');
+      const frontmatter = parseFrontmatter(content);
+
+      expect(frontmatter).not.toBeNull();
+      expect(frontmatter?.name).toBe(skillName);
+      expect(typeof frontmatter?.description).toBe('string');
+    }
+  });
+
+  it('all skills should have SKILL.yaml manifest', () => {
+    const skillsDir = getBundledSkillsDir();
+    const skillDirs = readdirSync(skillsDir, { withFileTypes: true })
+      .filter((d) => d.isDirectory())
+      .map((d) => d.name);
+
+    for (const skillName of skillDirs) {
+      const yamlPath = join(skillsDir, skillName, 'SKILL.yaml');
+      expect(existsSync(yamlPath)).toBe(true);
+
+      const content = readFileSync(yamlPath, 'utf-8');
+      const manifest = YAML.parse(content);
+
+      expect(manifest.name).toBe(skillName);
+      expect(typeof manifest.description).toBe('string');
+      expect(typeof manifest.version).toBe('string');
+    }
+  });
+
+  it('SKILL.md frontmatter should match SKILL.yaml', () => {
+    const skillsDir = getBundledSkillsDir();
+    const skillDirs = readdirSync(skillsDir, { withFileTypes: true })
+      .filter((d) => d.isDirectory())
+      .map((d) => d.name);
+
+    for (const skillName of skillDirs) {
+      const mdPath = join(skillsDir, skillName, 'SKILL.md');
+      const yamlPath = join(skillsDir, skillName, 'SKILL.yaml');
+
+      const mdContent = readFileSync(mdPath, 'utf-8');
+      const yamlContent = readFileSync(yamlPath, 'utf-8');
+
+      const frontmatter = parseFrontmatter(mdContent);
+      const manifest = YAML.parse(yamlContent);
+
+      expect(frontmatter?.name).toBe(manifest.name);
+      expect(frontmatter?.description).toBe(manifest.description);
+    }
+  });
+});

package/src/lib/types.ts

@@ -45,6 +45,11 @@ export interface InstalledSkill {
   installed_at: string;
 }
 
+export interface SkillExample {
+  title: string;
+  code: string;
+}
+
 export interface SkillManifest {
   name: string;
   description: string;
@@ -54,6 +59,8 @@ export interface SkillManifest {
   config_schema?: Record<string, ConfigOption>;
   // Skills can declare they provide an output target
   provides_output?: boolean;
+  // Usage examples to show in the TUI
+  examples?: SkillExample[];
 }
 
 export interface ConfigOption {

package/src/lib/version.test.ts

@@ -1,5 +1,5 @@
 import { describe, it, expect } from 'bun:test';
-import { getVersion } from './version.js';
+import { getVersion, compareSemver } from './version.js';
 
 describe('getVersion', () => {
   it('should return a version string', () => {
@@ -21,3 +21,22 @@ describe('getVersion', () => {
     expect(version).not.toBe('0.0.0');
   });
 });
+
+describe('compareSemver', () => {
+  it('should return 1 when first version is greater', () => {
+    expect(compareSemver('1.0.0', '0.9.9')).toBe(1);
+    expect(compareSemver('0.2.1', '0.2.0')).toBe(1);
+    expect(compareSemver('2.0.0', '1.9.9')).toBe(1);
+  });
+
+  it('should return -1 when first version is lesser', () => {
+    expect(compareSemver('0.9.9', '1.0.0')).toBe(-1);
+    expect(compareSemver('0.2.0', '0.2.1')).toBe(-1);
+    expect(compareSemver('1.9.9', '2.0.0')).toBe(-1);
+  });
+
+  it('should return 0 when versions are equal', () => {
+    expect(compareSemver('1.0.0', '1.0.0')).toBe(0);
+    expect(compareSemver('0.2.1', '0.2.1')).toBe(0);
+  });
+});

package/src/lib/version.ts

@@ -19,6 +19,23 @@ export function getVersion(): string {
   }
 }
 
+/**
+ * Compare two semver versions
+ * Returns: 1 if a > b, -1 if a < b, 0 if equal
+ */
+export function compareSemver(a: string, b: string): number {
+  const partsA = a.split('.').map(Number);
+  const partsB = b.split('.').map(Number);
+
+  for (let i = 0; i < 3; i++) {
+    const numA = partsA[i] || 0;
+    const numB = partsB[i] || 0;
+    if (numA > numB) return 1;
+    if (numA < numB) return -1;
+  }
+  return 0;
+}
+
 /**
  * Check for updates (non-blocking)
  * Shows a message if a new version is available
@@ -33,7 +50,8 @@ export async function checkForUpdates(): Promise<void> {
       timeout: 3000,
     }).trim();
 
-    if (latestVersion && latestVersion !== currentVersion) {
+    // Only show update message if latest is actually newer
+    if (latestVersion && compareSemver(latestVersion, currentVersion) > 0) {
       console.log(
         chalk.yellow(
           `\n⚠️  A new version of droid is available (${currentVersion} → ${latestVersion})`

package/src/skills/comments/SKILL.md

@@ -1,3 +1,11 @@
+---
+name: comments
+description: Inline conversation in any file via @droid/@user markers
+globs:
+  - "**/*"
+alwaysApply: false
+---
+
 # Comments Skill
 
 Enable inline conversation in any file using `> @droid` and `> @{user}` markers.

package/src/skills/comments/SKILL.yaml

@@ -16,3 +16,35 @@ config_schema:
     type: boolean
     description: Keep original comments after addressing (vs removing them)
     default: false
+examples:
+  - title: "Action request"
+    code: |
+      // @droid use PascalCase for enum keys
+      enum Status {
+        PENDING = 'pending'
+      }
+  - title: "Ask a question"
+    code: |
+      > @droid Should we cache this?
+
+      > @fry Yes, add Redis caching here...
+  - title: "TODO with context"
+    code: |
+      // @droid TODO: refactor to async/await
+      // The callback pattern is unwieldy
+      function fetchData(callback) {
+        api.get('/data', callback);
+      }
+  - title: "Code review note"
+    code: |
+      // @droid potential memory leak here
+      // Clear interval on unmount?
+      useEffect(() => {
+        setInterval(poll, 1000);
+      }, []);
+  - title: "Multi-line discussion"
+    code: |
+      > @droid Best approach for pagination?
+      > We have ~10k records.
+
+      > @fry Use cursor-based, limit 50.

package/tsconfig.json

@@ -1,8 +1,8 @@
 {
   "compilerOptions": {
     "target": "ES2022",
-    "module": "NodeNext",
-    "moduleResolution": "NodeNext",
+    "module": "ESNext",
+    "moduleResolution": "bundler",
     "outDir": "./dist",
     "rootDir": "./src",
     "strict": true,
@@ -12,7 +12,9 @@
     "declaration": true,
     "declarationMap": true,
     "sourceMap": true,
-    "resolveJsonModule": true
+    "resolveJsonModule": true,
+    "jsx": "react-jsx",
+    "jsxImportSource": "react"
   },
   "include": ["src/**/*"],
   "exclude": ["node_modules", "dist", "**/*.test.ts"]