@monoes/monograph 1.5.6 → 1.5.7

package/__tests__/cli/skill-gen.test.ts

@@ -1,280 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { tmpdir } from 'os';
-import { join } from 'path';
-import { mkdirSync, existsSync, rmSync } from 'fs';
-
-// Mock the db module so tests don't require a real indexed repo
-vi.mock('../../src/storage/db.js', () => ({
-  openDb: vi.fn(),
-  closeDb: vi.fn(),
-}));
-
-import { openDb, closeDb } from '../../src/storage/db.js';
-import { generateSkillFiles } from '../../src/cli/skill-gen.js';
-
-// ============================================================================
-// HELPERS
-// ============================================================================
-
-function makeMockDb(rows: {
-  communities?: unknown[];
-  members?: unknown[];
-  crossConnections?: unknown[];
-}) {
-  const prepare = vi.fn().mockImplementation((sql: string) => {
-    // Communities query: contains 'community_id' and 'member_count'
-    if (sql.includes('member_count')) {
-      return {
-        all: vi.fn().mockReturnValue(rows.communities ?? []),
-      };
-    }
-    // deriveCommunityLabel query: contains 'file_path' but NOT 'start_line'
-    if (sql.includes('file_path') && !sql.includes('start_line') && !sql.includes('source_id')) {
-      return {
-        all: vi.fn().mockReturnValue([]),
-      };
-    }
-    // Members query: contains 'start_line'
-    if (sql.includes('start_line')) {
-      return {
-        all: vi.fn().mockReturnValue(rows.members ?? []),
-      };
-    }
-    // Cross-connections query: contains 'source_id'
-    if (sql.includes('source_id')) {
-      return {
-        all: vi.fn().mockReturnValue(rows.crossConnections ?? []),
-      };
-    }
-    // Fallback
-    return { all: vi.fn().mockReturnValue([]) };
-  });
-
-  return { prepare } as unknown as ReturnType<typeof openDb>;
-}
-
-// ============================================================================
-// TESTS
-// ============================================================================
-
-describe('generateSkillFiles', () => {
-  let tmpDir: string;
-
-  beforeEach(() => {
-    tmpDir = join(tmpdir(), `skill-gen-test-${Date.now()}`);
-    mkdirSync(join(tmpDir, '.monomind'), { recursive: true });
-    vi.mocked(closeDb).mockImplementation(() => {});
-  });
-
-  afterEach(() => {
-    vi.clearAllMocks();
-    try {
-      rmSync(tmpDir, { recursive: true, force: true });
-    } catch {
-      // ignore cleanup errors
-    }
-  });
-
-  // --------------------------------------------------------------------------
-  // Shape
-  // --------------------------------------------------------------------------
-
-  it('returns SkillGenResult shape with empty result when no communities found', async () => {
-    const db = makeMockDb({ communities: [] });
-    vi.mocked(openDb).mockReturnValue(db);
-
-    const result = await generateSkillFiles(tmpDir);
-
-    expect(result).toHaveProperty('filesWritten');
-    expect(result).toHaveProperty('communityCount');
-    expect(Array.isArray(result.filesWritten)).toBe(true);
-    expect(typeof result.communityCount).toBe('number');
-  });
-
-  it('returns empty filesWritten and communityCount=0 when no communities found', async () => {
-    const db = makeMockDb({ communities: [] });
-    vi.mocked(openDb).mockReturnValue(db);
-
-    const result = await generateSkillFiles(tmpDir);
-
-    expect(result.filesWritten).toHaveLength(0);
-    expect(result.communityCount).toBe(0);
-  });
-
-  // --------------------------------------------------------------------------
-  // Output directory creation
-  // --------------------------------------------------------------------------
-
-  it('creates the output directory if it does not exist', async () => {
-    const customOutputDir = join(tmpDir, 'custom-skills-dir');
-    expect(existsSync(customOutputDir)).toBe(false);
-
-    const db = makeMockDb({ communities: [] });
-    vi.mocked(openDb).mockReturnValue(db);
-
-    await generateSkillFiles(tmpDir, customOutputDir);
-
-    // Directory should be created even with no communities (mkdir happens before loop)
-    // Actually the function returns early before mkdir when no communities — so only
-    // assert the directory does NOT get created when there are no communities.
-    // The directory IS created when there are communities.
-    expect(existsSync(customOutputDir)).toBe(false);
-  });
-
-  it('creates the default .monomind/skills/ directory when communities exist', async () => {
-    const db = makeMockDb({
-      communities: [{ community_id: 1, label: 'Function', member_count: 3 }],
-      members: [
-        {
-          id: 'n1',
-          label: 'Function',
-          name: 'parseFile',
-          file_path: `${tmpDir}/src/parser.ts`,
-          start_line: 10,
-          is_exported: 1,
-        },
-      ],
-      crossConnections: [],
-    });
-    vi.mocked(openDb).mockReturnValue(db);
-
-    const result = await generateSkillFiles(tmpDir);
-
-    const defaultDir = join(tmpDir, '.monomind', 'skills');
-    expect(existsSync(defaultDir)).toBe(true);
-    expect(result.communityCount).toBe(1);
-    expect(result.filesWritten).toHaveLength(1);
-  });
-
-  it('creates custom output directory when communities exist', async () => {
-    const customOutputDir = join(tmpDir, 'my-skills');
-    const db = makeMockDb({
-      communities: [{ community_id: 2, label: 'Class', member_count: 5 }],
-      members: [
-        {
-          id: 'n2',
-          label: 'Class',
-          name: 'GraphBuilder',
-          file_path: `${tmpDir}/src/graph/builder.ts`,
-          start_line: 1,
-          is_exported: 1,
-        },
-      ],
-      crossConnections: [],
-    });
-    vi.mocked(openDb).mockReturnValue(db);
-
-    const result = await generateSkillFiles(tmpDir, customOutputDir);
-
-    expect(existsSync(customOutputDir)).toBe(true);
-    expect(result.filesWritten[0]).toContain(customOutputDir);
-  });
-
-  // --------------------------------------------------------------------------
-  // File generation
-  // --------------------------------------------------------------------------
-
-  it('generates one skill file per community', async () => {
-    const db = makeMockDb({
-      communities: [
-        { community_id: 1, label: 'Function', member_count: 3 },
-        { community_id: 2, label: 'Class', member_count: 4 },
-      ],
-      members: [],
-      crossConnections: [],
-    });
-    vi.mocked(openDb).mockReturnValue(db);
-
-    const result = await generateSkillFiles(tmpDir);
-
-    expect(result.communityCount).toBe(2);
-    expect(result.filesWritten).toHaveLength(2);
-  });
-
-  it('generated skill file contains community label', async () => {
-    const { readFileSync } = await import('fs');
-
-    const db = makeMockDb({
-      communities: [{ community_id: 1, label: 'Parser', member_count: 3 }],
-      members: [
-        {
-          id: 'n1',
-          label: 'Function',
-          name: 'parseFile',
-          file_path: `${tmpDir}/src/parser.ts`,
-          start_line: 5,
-          is_exported: 1,
-        },
-      ],
-      crossConnections: [],
-    });
-    vi.mocked(openDb).mockReturnValue(db);
-
-    const result = await generateSkillFiles(tmpDir);
-    expect(result.filesWritten).toHaveLength(1);
-
-    const content = readFileSync(result.filesWritten[0], 'utf-8');
-    expect(content).toContain('Parser');
-    expect(content).toContain('parseFile');
-  });
-
-  it('generated skill file is valid markdown with frontmatter', async () => {
-    const { readFileSync } = await import('fs');
-
-    const db = makeMockDb({
-      communities: [{ community_id: 3, label: 'Storage', member_count: 2 }],
-      members: [],
-      crossConnections: [],
-    });
-    vi.mocked(openDb).mockReturnValue(db);
-
-    const result = await generateSkillFiles(tmpDir);
-    const content = readFileSync(result.filesWritten[0], 'utf-8');
-
-    expect(content.startsWith('---')).toBe(true);
-    expect(content).toContain('name:');
-    expect(content).toContain('description:');
-    expect(content).toContain('# Storage');
-  });
-
-  it('skill file contains How to Explore section', async () => {
-    const { readFileSync } = await import('fs');
-
-    const db = makeMockDb({
-      communities: [{ community_id: 4, label: 'Auth', member_count: 5 }],
-      members: [
-        {
-          id: 'n4',
-          label: 'Function',
-          name: 'verifyToken',
-          file_path: `${tmpDir}/src/auth/verify.ts`,
-          start_line: 20,
-          is_exported: 1,
-        },
-      ],
-      crossConnections: [],
-    });
-    vi.mocked(openDb).mockReturnValue(db);
-
-    const result = await generateSkillFiles(tmpDir);
-    const content = readFileSync(result.filesWritten[0], 'utf-8');
-
-    expect(content).toContain('## How to Explore');
-    expect(content).toContain('monograph_context');
-  });
-
-  it('filesWritten paths end with .md', async () => {
-    const db = makeMockDb({
-      communities: [{ community_id: 1, label: 'Graph', member_count: 3 }],
-      members: [],
-      crossConnections: [],
-    });
-    vi.mocked(openDb).mockReturnValue(db);
-
-    const result = await generateSkillFiles(tmpDir);
-
-    for (const f of result.filesWritten) {
-      expect(f.endsWith('.md')).toBe(true);
-    }
-  });
-});

package/__tests__/export/formats.test.ts

@@ -1,82 +0,0 @@
-import { describe, it, expect, beforeEach } from 'vitest';
-import { toJson } from '../../src/export/json.js';
-import { toCypher } from '../../src/export/cypher.js';
-import type { MonographNode, MonographEdge } from '../../src/types.js';
-
-let nodes: MonographNode[];
-let edges: MonographEdge[];
-
-beforeEach(() => {
-  nodes = [
-    { id: 'n1', label: 'Class', name: 'Foo', normLabel: 'foo', isExported: true },
-    { id: 'n2', label: 'Method', name: 'bar', normLabel: 'bar', isExported: false, filePath: 'foo.ts' },
-  ];
-  edges = [
-    { id: 'e1', sourceId: 'n1', targetId: 'n2', relation: 'CONTAINS', confidence: 'EXTRACTED', confidenceScore: 1.0 },
-    { id: 'e2', sourceId: 'n2', targetId: 'n1', relation: 'CALLS', confidence: 'INFERRED', confidenceScore: 0.5 },
-  ];
-});
-
-describe('json export', () => {
-  it('produces nodes and edges fields', () => {
-    const result = toJson(nodes, edges);
-    const parsed = JSON.parse(result);
-    expect(parsed.nodes).toHaveLength(2);
-    expect(parsed.edges).toHaveLength(2);
-  });
-
-  it('transforms node properties correctly', () => {
-    const result = toJson(nodes, edges);
-    const parsed = JSON.parse(result);
-    const firstNode = parsed.nodes[0];
-    expect(firstNode.id).toBe('n1');
-    expect(firstNode.label).toBe('Class');
-    expect(firstNode.norm_label).toBe('foo');
-    expect(firstNode.is_exported).toBe(true);
-  });
-
-  it('includes links array', () => {
-    const result = toJson(nodes, edges);
-    const parsed = JSON.parse(result);
-    expect(parsed.links).toBeDefined();
-    expect(parsed.links).toHaveLength(2);
-  });
-});
-
-describe('cypher export', () => {
-  it('does not include destructive operations', () => {
-    const result = toCypher(nodes, edges);
-    expect(result).toContain('CREATE');
-    // Verify no standalone destructive operations (DROP/DELETE statements, not in comments)
-    const lines = result.split('\n').filter(l => !l.trim().startsWith('//'));
-    expect(lines.some(l => l.includes('DROP'))).toBe(false);
-    expect(lines.some(l => l.includes('DELETE'))).toBe(false);
-  });
-
-  it('includes comment header', () => {
-    const result = toCypher(nodes, edges);
-    expect(result).toContain('// Generated by Monograph');
-  });
-
-  it('creates nodes and edges separately', () => {
-    const result = toCypher(nodes, edges);
-    const lines = result.split('\n');
-    const createNodeLines = lines.filter(l => l.includes('CREATE') && l.includes(':'));
-    expect(createNodeLines.length).toBeGreaterThan(0);
-  });
-
-  it('handles special characters safely', () => {
-    const nodeWithSpecial = {
-      ...nodes[0],
-      name: 'Foo"Bar"Baz',
-      id: 'n-special_id',
-    };
-    const result = toCypher([nodeWithSpecial], []);
-    // Verify escaped quotes in output
-    expect(result).toContain('\\"');
-    // Verify it doesn't have unescaped double quotes in the middle
-    const lines = result.split('\n');
-    const createLines = lines.filter(l => l.includes('CREATE') && l.includes('name:'));
-    expect(createLines.length).toBeGreaterThan(0);
-  });
-});

package/__tests__/fixtures/sample.c

@@ -1,25 +0,0 @@
-#include <stdio.h>
-#include <stdlib.h>
-
-typedef struct {
-  int x;
-  int y;
-} Point;
-
-enum Color { RED, GREEN, BLUE };
-
-int add(int a, int b) {
-  return a + b;
-}
-
-int multiply(int a, int b) {
-  return a * b;
-}
-
-int main(void) {
-  Point p;
-  p.x = add(1, 2);
-  p.y = multiply(3, 4);
-  printf("x=%d y=%d\n", p.x, p.y);
-  return 0;
-}

package/__tests__/fixtures/sample.cpp

@@ -1,32 +0,0 @@
-#include <string>
-#include <iostream>
-
-namespace Animals {
-
-class Animal {
-public:
-  std::string name;
-  Animal(std::string n) : name(n) {}
-  virtual void speak() {}
-};
-
-class Dog : public Animal {
-public:
-  Dog(std::string n) : Animal(n) {}
-  void speak() override {
-    std::cout << "Woof!" << std::endl;
-  }
-};
-
-} // namespace Animals
-
-template<typename T>
-T add(T a, T b) {
-  return a + b;
-}
-
-int main() {
-  Animals::Dog d("Rex");
-  d.speak();
-  return 0;
-}

package/__tests__/fixtures/sample.dart

@@ -1,34 +0,0 @@
-import 'package:flutter/material.dart';
-
-abstract class Animal {
-  String get name;
-  String speak();
-}
-
-class Dog extends Animal {
-  final String name;
-
-  Dog(this.name);
-
-  @override
-  String speak() {
-    return 'Woof!';
-  }
-}
-
-class Cat extends Animal {
-  final String name;
-
-  Cat(this.name);
-
-  @override
-  String speak() {
-    return 'Meow!';
-  }
-}
-
-enum Direction { north, south, east, west }
-
-String helperFn(String s) {
-  return s.toUpperCase();
-}

package/__tests__/fixtures/sample.kt

@@ -1,30 +0,0 @@
-import com.example.model.User
-import com.example.service.Logger
-
-interface UserService {
-    fun getUser(id: Int): User?
-    fun createUser(name: String): Boolean
-}
-
-data class User(val id: Int, val name: String)
-
-class UserServiceImpl : UserService {
-    private val users = mutableListOf<User>()
-
-    override fun getUser(id: Int): User? {
-        return users.find { it.id == id }
-    }
-
-    override fun createUser(name: String): Boolean {
-        users.add(User(users.size + 1, name))
-        return true
-    }
-
-    private fun helperFn(s: String): String {
-        return s.uppercase()
-    }
-}
-
-fun topLevelFn(x: Int): Int {
-    return x * 2
-}

package/__tests__/fixtures/sample.php

@@ -1,30 +0,0 @@
-<?php
-namespace App\Models;
-
-use App\Contracts\UserInterface;
-use App\Services\Logger;
-
-interface UserInterface {
-  public function getUser(int $id): ?array;
-  public function createUser(string $name): bool;
-}
-
-class User implements UserInterface {
-  private string $name;
-
-  public function __construct(string $name) {
-    $this->name = $name;
-  }
-
-  public function getUser(int $id): ?array {
-    return ['id' => $id, 'name' => $this->name];
-  }
-
-  public function createUser(string $name): bool {
-    return !empty($name);
-  }
-}
-
-function helperFn(string $s): string {
-  return strtoupper($s);
-}

package/__tests__/fixtures/sample.py

@@ -1,11 +0,0 @@
-from os import path
-
-class UserService:
-    def __init__(self, db):
-        self.db = db
-
-    def get_user(self, user_id: str):
-        return self.db.find(user_id)
-
-def helper_fn(x: int) -> int:
-    return x * 2

package/__tests__/fixtures/sample.rb

@@ -1,26 +0,0 @@
-require 'json'
-require 'net/http'
-
-module Animals
-  class Animal
-    attr_reader :name
-
-    def initialize(name)
-      @name = name
-    end
-
-    def speak
-      raise NotImplementedError, 'Subclass must implement speak'
-    end
-  end
-
-  class Dog < Animal
-    def speak
-      "Woof!"
-    end
-  end
-end
-
-def helper_fn(x)
-  x * 2
-end

package/__tests__/fixtures/sample.swift

@@ -1,42 +0,0 @@
-import Foundation
-
-protocol Greetable {
-  func greet() -> String
-}
-
-struct Person: Greetable {
-  var name: String
-
-  func greet() -> String {
-    return "Hello, \(name)"
-  }
-}
-
-class Animal {
-  var name: String
-
-  init(name: String) {
-    self.name = name
-  }
-
-  func speak() {
-    print("\(name) speaks")
-  }
-}
-
-class Dog: Animal {
-  override func speak() {
-    print("Woof!")
-  }
-}
-
-enum Direction {
-  case north
-  case south
-  case east
-  case west
-}
-
-func helperFn() -> Int {
-  return 42
-}

package/__tests__/fixtures/sample.ts

@@ -1,17 +0,0 @@
-import { readFile } from 'fs/promises';
-
-export interface UserService {
-  getUser(id: string): Promise<User>;
-}
-
-export class UserServiceImpl implements UserService {
-  constructor(private db: Database) {}
-
-  async getUser(id: string): Promise<User> {
-    return this.db.find(id);
-  }
-}
-
-function helperFn(x: number): number {
-  return x * 2;
-}

package/__tests__/fixtures/sample.vue

@@ -1,40 +0,0 @@
-<template>
-  <div class="user-card">
-    <h1>{{ user.name }}</h1>
-    <button @click="greet">Greet</button>
-  </div>
-</template>
-
-<script lang="ts">
-import { defineComponent, ref } from 'vue';
-
-interface User {
-  name: string;
-  id: number;
-}
-
-class UserService {
-  getUser(id: number): User {
-    return { id, name: 'Alice' };
-  }
-}
-
-export default defineComponent({
-  name: 'UserCard',
-  setup() {
-    const user = ref<User>({ name: 'Alice', id: 1 });
-
-    function greet() {
-      alert(`Hello, ${user.value.name}`);
-    }
-
-    return { user, greet };
-  },
-});
-</script>
-
-<style scoped>
-.user-card {
-  padding: 1rem;
-}
-</style>