* fix(db): remove unique constraint on folder to support multi-channel agents * ci: implement automated skill drift detection and self-healing PRs * fix: align registration logic with Gavriel's feedback and fix build/test issues from Daniel Mi * style: conform to prettier standards for CI validation * test: fix branch naming inconsistency in CI (master vs main) * fix(ci): robust module resolution by removing file extensions in scripts * refactor(ci): simplify skill validation by removing redundant combination tests * style: conform skills-engine to prettier, unify logging in index.ts and cleanup unused imports * refactor: extract multi-channel DB changes to separate branch Move channel column, folder suffix logic, and related migrations to feat/multi-channel-db-v2 for independent review. This PR now contains only CI/CD optimizations, Prettier formatting, and logging improvements. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
150 lines
4.0 KiB
TypeScript
150 lines
4.0 KiB
TypeScript
import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|
|
|
// Mock logger
|
|
vi.mock('./logger.js', () => ({
|
|
logger: {
|
|
debug: vi.fn(),
|
|
info: vi.fn(),
|
|
warn: vi.fn(),
|
|
error: vi.fn(),
|
|
},
|
|
}));
|
|
|
|
// Mock child_process — store the mock fn so tests can configure it
|
|
const mockExecSync = vi.fn();
|
|
vi.mock('child_process', () => ({
|
|
execSync: (...args: unknown[]) => mockExecSync(...args),
|
|
}));
|
|
|
|
import {
|
|
CONTAINER_RUNTIME_BIN,
|
|
readonlyMountArgs,
|
|
stopContainer,
|
|
ensureContainerRuntimeRunning,
|
|
cleanupOrphans,
|
|
} from './container-runtime.js';
|
|
import { logger } from './logger.js';
|
|
|
|
beforeEach(() => {
|
|
vi.clearAllMocks();
|
|
});
|
|
|
|
// --- Pure functions ---
|
|
|
|
describe('readonlyMountArgs', () => {
|
|
it('returns -v flag with :ro suffix', () => {
|
|
const args = readonlyMountArgs('/host/path', '/container/path');
|
|
expect(args).toEqual(['-v', '/host/path:/container/path:ro']);
|
|
});
|
|
});
|
|
|
|
describe('stopContainer', () => {
|
|
it('returns stop command using CONTAINER_RUNTIME_BIN', () => {
|
|
expect(stopContainer('nanoclaw-test-123')).toBe(
|
|
`${CONTAINER_RUNTIME_BIN} stop nanoclaw-test-123`,
|
|
);
|
|
});
|
|
});
|
|
|
|
// --- ensureContainerRuntimeRunning ---
|
|
|
|
describe('ensureContainerRuntimeRunning', () => {
|
|
it('does nothing when runtime is already running', () => {
|
|
mockExecSync.mockReturnValueOnce('');
|
|
|
|
ensureContainerRuntimeRunning();
|
|
|
|
expect(mockExecSync).toHaveBeenCalledTimes(1);
|
|
expect(mockExecSync).toHaveBeenCalledWith(`${CONTAINER_RUNTIME_BIN} info`, {
|
|
stdio: 'pipe',
|
|
timeout: 10000,
|
|
});
|
|
expect(logger.debug).toHaveBeenCalledWith(
|
|
'Container runtime already running',
|
|
);
|
|
});
|
|
|
|
it('throws when docker info fails', () => {
|
|
mockExecSync.mockImplementationOnce(() => {
|
|
throw new Error('Cannot connect to the Docker daemon');
|
|
});
|
|
|
|
expect(() => ensureContainerRuntimeRunning()).toThrow(
|
|
'Container runtime is required but failed to start',
|
|
);
|
|
expect(logger.error).toHaveBeenCalled();
|
|
});
|
|
});
|
|
|
|
// --- cleanupOrphans ---
|
|
|
|
describe('cleanupOrphans', () => {
|
|
it('stops orphaned nanoclaw containers', () => {
|
|
// docker ps returns container names, one per line
|
|
mockExecSync.mockReturnValueOnce(
|
|
'nanoclaw-group1-111\nnanoclaw-group2-222\n',
|
|
);
|
|
// stop calls succeed
|
|
mockExecSync.mockReturnValue('');
|
|
|
|
cleanupOrphans();
|
|
|
|
// ps + 2 stop calls
|
|
expect(mockExecSync).toHaveBeenCalledTimes(3);
|
|
expect(mockExecSync).toHaveBeenNthCalledWith(
|
|
2,
|
|
`${CONTAINER_RUNTIME_BIN} stop nanoclaw-group1-111`,
|
|
{ stdio: 'pipe' },
|
|
);
|
|
expect(mockExecSync).toHaveBeenNthCalledWith(
|
|
3,
|
|
`${CONTAINER_RUNTIME_BIN} stop nanoclaw-group2-222`,
|
|
{ stdio: 'pipe' },
|
|
);
|
|
expect(logger.info).toHaveBeenCalledWith(
|
|
{ count: 2, names: ['nanoclaw-group1-111', 'nanoclaw-group2-222'] },
|
|
'Stopped orphaned containers',
|
|
);
|
|
});
|
|
|
|
it('does nothing when no orphans exist', () => {
|
|
mockExecSync.mockReturnValueOnce('');
|
|
|
|
cleanupOrphans();
|
|
|
|
expect(mockExecSync).toHaveBeenCalledTimes(1);
|
|
expect(logger.info).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it('warns and continues when ps fails', () => {
|
|
mockExecSync.mockImplementationOnce(() => {
|
|
throw new Error('docker not available');
|
|
});
|
|
|
|
cleanupOrphans(); // should not throw
|
|
|
|
expect(logger.warn).toHaveBeenCalledWith(
|
|
expect.objectContaining({ err: expect.any(Error) }),
|
|
'Failed to clean up orphaned containers',
|
|
);
|
|
});
|
|
|
|
it('continues stopping remaining containers when one stop fails', () => {
|
|
mockExecSync.mockReturnValueOnce('nanoclaw-a-1\nnanoclaw-b-2\n');
|
|
// First stop fails
|
|
mockExecSync.mockImplementationOnce(() => {
|
|
throw new Error('already stopped');
|
|
});
|
|
// Second stop succeeds
|
|
mockExecSync.mockReturnValueOnce('');
|
|
|
|
cleanupOrphans(); // should not throw
|
|
|
|
expect(mockExecSync).toHaveBeenCalledTimes(3);
|
|
expect(logger.info).toHaveBeenCalledWith(
|
|
{ count: 2, names: ['nanoclaw-a-1', 'nanoclaw-b-2'] },
|
|
'Stopped orphaned containers',
|
|
);
|
|
});
|
|
});
|