* refactor: multi-channel infrastructure with explicit channel/is_group tracking - Add channels[] array and findChannel() routing in index.ts, replacing hardcoded whatsapp.* calls with channel-agnostic callbacks - Add channel TEXT and is_group INTEGER columns to chats table with COALESCE upsert to protect existing values from null overwrites - is_group defaults to 0 (safe: unknown chats excluded from groups) - WhatsApp passes explicit channel='whatsapp' and isGroup to onChatMetadata - getAvailableGroups filters on is_group instead of JID pattern matching - findChannel logs warnings instead of silently dropping unroutable JIDs - Migration backfills channel/is_group from JID patterns for existing DBs Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat: skills engine v0.1 — deterministic skill packages with rerere resolution Three-way merge engine for applying skill packages on top of a core codebase. Skills declare which files they add/modify, and the engine uses git merge-file for conflict detection with git rerere for automatic resolution of previously-seen conflicts. Key components: - apply: three-way merge with backup/rollback safety net - replay: clean-slate replay for uninstall and rebase - update: core version updates with deletion detection - rebase: bake applied skills into base (one-way) - manifest: validation with path traversal protection - resolution-cache: pre-computed rerere resolutions - structured: npm deps, env vars, docker-compose merging - CI: per-skill test matrix with conflict detection 151 unit tests covering merge, rerere, backup, replay, uninstall, update, rebase, structured ops, and edge cases. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat: add Discord and Telegram skill packages Skill packages for adding Discord and Telegram channels to NanoClaw. Each package includes: - Channel implementation (add/src/channels/) - Three-way merge targets for index.ts, config.ts, routing.test.ts - Intent docs explaining merge invariants - Standalone integration tests - manifest.yaml with dependency/conflict declarations Applied via: npx tsx scripts/apply-skill.ts .claude/skills/add-discord These are inert until applied — no runtime impact. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * remove unused docs (skills-system-status, implementation-guide) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
98 lines
3.2 KiB
TypeScript
98 lines
3.2 KiB
TypeScript
import { execSync } from 'child_process';
|
|
import { describe, it, expect, beforeEach, afterEach } from 'vitest';
|
|
import fs from 'fs';
|
|
import path from 'path';
|
|
import { isGitRepo, mergeFile, setupRerereAdapter } from '../merge.js';
|
|
import { createTempDir, initGitRepo, cleanup } from './test-helpers.js';
|
|
|
|
describe('merge', () => {
|
|
let tmpDir: string;
|
|
const originalCwd = process.cwd();
|
|
|
|
beforeEach(() => {
|
|
tmpDir = createTempDir();
|
|
process.chdir(tmpDir);
|
|
});
|
|
|
|
afterEach(() => {
|
|
process.chdir(originalCwd);
|
|
cleanup(tmpDir);
|
|
});
|
|
|
|
it('isGitRepo returns true in a git repo', () => {
|
|
initGitRepo(tmpDir);
|
|
expect(isGitRepo()).toBe(true);
|
|
});
|
|
|
|
it('isGitRepo returns false outside a git repo', () => {
|
|
expect(isGitRepo()).toBe(false);
|
|
});
|
|
|
|
describe('mergeFile', () => {
|
|
beforeEach(() => {
|
|
initGitRepo(tmpDir);
|
|
});
|
|
|
|
it('clean merge with no overlapping changes', () => {
|
|
const base = path.join(tmpDir, 'base.txt');
|
|
const current = path.join(tmpDir, 'current.txt');
|
|
const skill = path.join(tmpDir, 'skill.txt');
|
|
|
|
fs.writeFileSync(base, 'line1\nline2\nline3\n');
|
|
fs.writeFileSync(current, 'line1-modified\nline2\nline3\n');
|
|
fs.writeFileSync(skill, 'line1\nline2\nline3-modified\n');
|
|
|
|
const result = mergeFile(current, base, skill);
|
|
expect(result.clean).toBe(true);
|
|
expect(result.exitCode).toBe(0);
|
|
|
|
const merged = fs.readFileSync(current, 'utf-8');
|
|
expect(merged).toContain('line1-modified');
|
|
expect(merged).toContain('line3-modified');
|
|
});
|
|
|
|
it('setupRerereAdapter cleans stale MERGE_HEAD before proceeding', () => {
|
|
// Simulate a stale MERGE_HEAD from a previous crash
|
|
const gitDir = execSync('git rev-parse --git-dir', {
|
|
cwd: tmpDir,
|
|
encoding: 'utf-8',
|
|
}).trim();
|
|
const headHash = execSync('git rev-parse HEAD', {
|
|
cwd: tmpDir,
|
|
encoding: 'utf-8',
|
|
}).trim();
|
|
fs.writeFileSync(path.join(gitDir, 'MERGE_HEAD'), headHash + '\n');
|
|
fs.writeFileSync(path.join(gitDir, 'MERGE_MSG'), 'stale merge\n');
|
|
|
|
// Write a file for the adapter to work with
|
|
fs.writeFileSync(path.join(tmpDir, 'test.txt'), 'conflicted content');
|
|
|
|
// setupRerereAdapter should not throw despite stale MERGE_HEAD
|
|
expect(() =>
|
|
setupRerereAdapter('test.txt', 'base', 'ours', 'theirs'),
|
|
).not.toThrow();
|
|
|
|
// MERGE_HEAD should still exist (newly written by setupRerereAdapter)
|
|
expect(fs.existsSync(path.join(gitDir, 'MERGE_HEAD'))).toBe(true);
|
|
});
|
|
|
|
it('conflict with overlapping changes', () => {
|
|
const base = path.join(tmpDir, 'base.txt');
|
|
const current = path.join(tmpDir, 'current.txt');
|
|
const skill = path.join(tmpDir, 'skill.txt');
|
|
|
|
fs.writeFileSync(base, 'line1\nline2\nline3\n');
|
|
fs.writeFileSync(current, 'line1-ours\nline2\nline3\n');
|
|
fs.writeFileSync(skill, 'line1-theirs\nline2\nline3\n');
|
|
|
|
const result = mergeFile(current, base, skill);
|
|
expect(result.clean).toBe(false);
|
|
expect(result.exitCode).toBeGreaterThan(0);
|
|
|
|
const merged = fs.readFileSync(current, 'utf-8');
|
|
expect(merged).toContain('<<<<<<<');
|
|
expect(merged).toContain('>>>>>>>');
|
|
});
|
|
});
|
|
});
|