gitspace 0.2.0-rc.20 → 0.2.0-rc.21

package/src/commands/__tests__/connect-key.test.ts

@@ -1,10 +0,0 @@
-import { describe, expect, it } from 'bun:test'
-import { buildRemoteBackendKey } from '../connect-key'
-
-describe('buildRemoteBackendKey', () => {
-  it('builds canonical remote backend key with relay URL and machine id', () => {
-    expect(buildRemoteBackendKey('ws://relay.test/ws', 'machine-1')).toBe(
-      'remote:ws://relay.test/ws:machine-1'
-    )
-  })
-})

package/src/commands/__tests__/events.test.ts

@@ -1,201 +0,0 @@
-/**
- * Events command tests
- */
-
-import { describe, expect, it, mock, beforeEach, afterAll } from 'bun:test';
-import { mkdtempSync, mkdirSync, rmSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-import * as realConfig from '../../core/config.js';
-import * as realEventPaths from '../../lib/events/paths.js';
-import * as realEventReader from '../../lib/events/reader.js';
-import { SpacesError } from '../../types/errors.js';
-
-const tempDirs: string[] = [];
-
-function makeTempDir(prefix: string): string {
-  const dir = mkdtempSync(join(tmpdir(), prefix));
-  tempDirs.push(dir);
-  return dir;
-}
-
-let baseDir = makeTempDir('gssh-events-base-');
-
-const mockGetProjectWorkspacesDir = mock<(projectName: string) => string>((projectName: string) =>
-  join(baseDir, projectName, 'workspaces')
-);
-
-mock.module('../../core/config.js', () => ({
-  ...realConfig,
-  getProjectWorkspacesDir: mockGetProjectWorkspacesDir,
-}));
-
-const mockGetProcessEventsDir = mock<(workspacePath: string, processName: string) => string>(
-  (workspacePath: string, processName: string) =>
-    realEventPaths.getProcessEventsDir(workspacePath, processName)
-);
-const mockListProcessEventsDirs = mock<(workspacePath: string) => string[]>(
-  (workspacePath: string) => realEventPaths.listProcessEventsDirs(workspacePath)
-);
-
-mock.module('../../lib/events/paths.js', () => ({
-  ...realEventPaths,
-  getProcessEventsDir: mockGetProcessEventsDir,
-  listProcessEventsDirs: mockListProcessEventsDirs,
-}));
-
-const mockReadWideEvents = mock<(...args: unknown[]) => unknown[]>((...args: unknown[]) =>
-  realEventReader.readWideEvents(...(args as Parameters<typeof realEventReader.readWideEvents>))
-);
-
-mock.module('../../lib/events/reader.js', () => ({
-  ...realEventReader,
-  readWideEvents: mockReadWideEvents,
-}));
-
-const { listEvents, showEvent, tailEvents } = await import('../events.js');
-
-function makeWorkspace(projectName: string, workspaceName: string): string {
-  const workspacePath = join(mockGetProjectWorkspacesDir(projectName), workspaceName);
-  mkdirSync(workspacePath, { recursive: true });
-  return workspacePath;
-}
-
-afterAll(() => {
-  while (tempDirs.length > 0) {
-    const dir = tempDirs.pop();
-    if (dir) {
-      rmSync(dir, { recursive: true, force: true });
-    }
-  }
-});
-
-beforeEach(() => {
-  baseDir = makeTempDir('gssh-events-base-');
-  mockGetProcessEventsDir.mockReset();
-  mockListProcessEventsDirs.mockReset();
-  mockReadWideEvents.mockReset();
-  mockGetProjectWorkspacesDir.mockImplementation((projectName: string) =>
-    join(baseDir, projectName, 'workspaces')
-  );
-  mockListProcessEventsDirs.mockImplementation(() => []);
-  mockReadWideEvents.mockImplementation(() => []);
-});
-
-describe('events command requires explicit project/workspace', () => {
-  it('throws when --project is missing', async () => {
-    try {
-      await listEvents({ workspace: 'ws-1' });
-      expect.unreachable('should throw');
-    } catch (err) {
-      expect(err).toBeInstanceOf(SpacesError);
-      expect((err as SpacesError).message).toContain('Provide project');
-    }
-  });
-
-  it('throws when --workspace is missing', async () => {
-    try {
-      await listEvents({ project: 'my-project' });
-      expect.unreachable('should throw');
-    } catch (err) {
-      expect(err).toBeInstanceOf(SpacesError);
-      expect((err as SpacesError).message).toContain('Provide workspace');
-    }
-  });
-
-  it('throws when explicit workspace does not exist', async () => {
-    try {
-      await listEvents({ project: 'my-project', workspace: 'missing-ws' });
-      expect.unreachable('should throw');
-    } catch (err) {
-      expect(err).toBeInstanceOf(SpacesError);
-      expect((err as SpacesError).message).toContain('Workspace not found');
-    }
-  });
-});
-
-describe('events command directory resolution', () => {
-  it('throws when no process events directory exists', async () => {
-    makeWorkspace('my-project', 'ws-1');
-    mockListProcessEventsDirs.mockImplementation(() => []);
-
-    try {
-      await listEvents({ project: 'my-project', workspace: 'ws-1' });
-      expect.unreachable('should throw');
-    } catch (err) {
-      expect(err).toBeInstanceOf(SpacesError);
-      expect((err as SpacesError).message).toContain('No process events directory');
-    }
-  });
-
-  it('uses processName filter to resolve process-specific events dir', async () => {
-    const workspacePath = makeWorkspace('my-project', 'ws-1');
-    const eventsDir = makeTempDir('gssh-events-dir-');
-    mockGetProcessEventsDir.mockImplementation(() => eventsDir);
-
-    await listEvents({
-      project: 'my-project',
-      workspace: 'ws-1',
-      filter: 'processName=web',
-    });
-
-    expect(mockGetProcessEventsDir).toHaveBeenCalledWith(workspacePath, 'web');
-  });
-
-  it('aggregates events across all process dirs when processName is not provided', async () => {
-    makeWorkspace('my-project', 'ws-1');
-    const eventsDirA = makeTempDir('gssh-events-dir-a-');
-    const eventsDirB = makeTempDir('gssh-events-dir-b-');
-    mockListProcessEventsDirs.mockImplementation(() => [eventsDirA, eventsDirB]);
-
-    await listEvents({ project: 'my-project', workspace: 'ws-1' });
-
-    expect(mockReadWideEvents).toHaveBeenCalledTimes(2);
-    expect(mockReadWideEvents).toHaveBeenCalledWith({ eventsDir: eventsDirA, filter: {} });
-    expect(mockReadWideEvents).toHaveBeenCalledWith({ eventsDir: eventsDirB, filter: {} });
-  });
-});
-
-describe('showEvent validations', () => {
-  it('throws when eventId filter is missing', async () => {
-    try {
-      await showEvent({ project: 'my-project', workspace: 'ws-1' });
-      expect.unreachable('should throw');
-    } catch (err) {
-      expect(err).toBeInstanceOf(SpacesError);
-      expect((err as SpacesError).message).toContain('eventId');
-    }
-  });
-
-  it('throws when filter is not eventId-prefixed', async () => {
-    try {
-      await showEvent({ project: 'my-project', workspace: 'ws-1', filter: 'level=error' });
-      expect.unreachable('should throw');
-    } catch (err) {
-      expect(err).toBeInstanceOf(SpacesError);
-      expect((err as SpacesError).message).toContain('eventId');
-    }
-  });
-});
-
-describe('events command happy path', () => {
-  it('listEvents succeeds when events dir exists', async () => {
-    makeWorkspace('my-project', 'ws-1');
-    const eventsDir = makeTempDir('gssh-events-dir-');
-    mockListProcessEventsDirs.mockImplementation(() => [eventsDir]);
-    mockReadWideEvents.mockImplementation(() => []);
-
-    await listEvents({ project: 'my-project', workspace: 'ws-1' });
-    expect(mockReadWideEvents).toHaveBeenCalledTimes(1);
-  });
-
-  it('tailEvents succeeds when events dir exists', async () => {
-    makeWorkspace('my-project', 'ws-1');
-    const eventsDir = makeTempDir('gssh-events-dir-');
-    mockListProcessEventsDirs.mockImplementation(() => [eventsDir]);
-    mockReadWideEvents.mockImplementation(() => []);
-
-    await tailEvents({ project: 'my-project', workspace: 'ws-1' });
-    expect(mockReadWideEvents).toHaveBeenCalledTimes(1);
-  });
-});

package/src/commands/__tests__/notifications.test.ts

@@ -1,349 +0,0 @@
-import { describe, expect, it, beforeEach, afterEach } from 'bun:test';
-import { mkdtempSync, rmSync, writeFileSync, readFileSync, existsSync } from 'fs';
-import { join } from 'path';
-import { tmpdir } from 'os';
-import {
-  isHookInstalled,
-  installHook,
-  uninstallHook,
-  MARKER_START_EXPORT as MARKER_START,
-  MARKER_END_EXPORT as MARKER_END,
-  BASH_ZSH_HOOK_EXPORT as BASH_ZSH_HOOK,
-  FISH_HOOK_EXPORT as FISH_HOOK,
-} from '../notifications';
-import { logger } from '../../utils/logger';
-import { spyOn } from 'bun:test';
-
-// ============================================================================
-// Test Setup
-// ============================================================================
-
-let tempDir: string;
-
-beforeEach(() => {
-  tempDir = mkdtempSync(join(tmpdir(), 'gssh-notifications-test-'));
-});
-
-afterEach(() => {
-  rmSync(tempDir, { recursive: true, force: true });
-});
-
-// ============================================================================
-// isHookInstalled
-// ============================================================================
-
-describe('isHookInstalled', () => {
-  it('should return false for non-existent file', () => {
-    const filePath = join(tempDir, 'nonexistent');
-    expect(isHookInstalled(filePath)).toBe(false);
-  });
-
-  it('should return false for file without hook', () => {
-    const filePath = join(tempDir, '.zshrc');
-    writeFileSync(filePath, '# My zsh config\nexport PATH="/usr/local/bin:$PATH"\n');
-
-    expect(isHookInstalled(filePath)).toBe(false);
-  });
-
-  it('should return true for file with hook marker', () => {
-    const filePath = join(tempDir, '.zshrc');
-    writeFileSync(filePath, `# My config\n${MARKER_START}\n# hook content\n${MARKER_END}\n`);
-
-    expect(isHookInstalled(filePath)).toBe(true);
-  });
-
-  it('should detect hook anywhere in file', () => {
-    const filePath = join(tempDir, '.bashrc');
-    const content = `
-# Existing config
-export FOO=bar
-
-${MARKER_START}
-# hook content here
-${MARKER_END}
-
-# More config after
-alias ll='ls -la'
-`;
-    writeFileSync(filePath, content);
-
-    expect(isHookInstalled(filePath)).toBe(true);
-  });
-});
-
-// ============================================================================
-// installHook
-// ============================================================================
-
-describe('installHook', () => {
-  it('should create file and install hook if file does not exist', () => {
-    const filePath = join(tempDir, '.zshrc');
-
-    const result = installHook(filePath, BASH_ZSH_HOOK);
-
-    expect(result.installed).toBe(true);
-    expect(result.created).toBe(true);
-    expect(existsSync(filePath)).toBe(true);
-
-    const content = readFileSync(filePath, 'utf-8');
-    expect(content).toContain(MARKER_START);
-    expect(content).toContain(MARKER_END);
-    expect(content).toContain('__gitspace_preexec');
-  });
-
-  it('should append hook to existing file', () => {
-    const filePath = join(tempDir, '.zshrc');
-    const existingContent = '# My existing config\nexport PATH="/usr/local/bin:$PATH"\n';
-    writeFileSync(filePath, existingContent);
-
-    const result = installHook(filePath, BASH_ZSH_HOOK);
-
-    expect(result.installed).toBe(true);
-    expect(result.created).toBe(false);
-
-    const content = readFileSync(filePath, 'utf-8');
-    expect(content).toContain('My existing config');
-    expect(content).toContain(MARKER_START);
-  });
-
-  it('should be idempotent - not install twice', () => {
-    const filePath = join(tempDir, '.zshrc');
-    writeFileSync(filePath, '');
-
-    // First install
-    const result1 = installHook(filePath, BASH_ZSH_HOOK);
-    expect(result1.installed).toBe(true);
-
-    const contentAfterFirst = readFileSync(filePath, 'utf-8');
-    const markerCount1 = (contentAfterFirst.match(new RegExp(MARKER_START, 'g')) || []).length;
-    expect(markerCount1).toBe(1);
-
-    // Second install - should not add duplicate
-    const result2 = installHook(filePath, BASH_ZSH_HOOK);
-    expect(result2.installed).toBe(false);
-    expect(result2.created).toBe(false);
-
-    const contentAfterSecond = readFileSync(filePath, 'utf-8');
-    const markerCount2 = (contentAfterSecond.match(new RegExp(MARKER_START, 'g')) || []).length;
-    expect(markerCount2).toBe(1);
-  });
-
-  it('should create parent directories if needed', () => {
-    const nestedPath = join(tempDir, '.config', 'fish', 'config.fish');
-
-    const result = installHook(nestedPath, FISH_HOOK);
-
-    expect(result.installed).toBe(true);
-    expect(result.created).toBe(true);
-    expect(existsSync(nestedPath)).toBe(true);
-
-    const content = readFileSync(nestedPath, 'utf-8');
-    expect(content).toContain('__gitspace_preexec');
-    expect(content).toContain('fish_preexec');
-  });
-
-  it('should warn if existing DEBUG trap is found in bashrc', () => {
-    const filePath = join(tempDir, '.bashrc');
-    const existingContent = 'trap "echo debug" DEBUG\n';
-    writeFileSync(filePath, existingContent);
-
-    const warnSpy = spyOn(logger, 'warning');
-    installHook(filePath, BASH_ZSH_HOOK);
-
-    expect(warnSpy).toHaveBeenCalled();
-    expect(warnSpy.mock.calls[0][0]).toContain('Existing DEBUG trap found');
-    warnSpy.mockRestore();
-  });
-});
-
-// ============================================================================
-// uninstallHook
-// ============================================================================
-
-describe('uninstallHook', () => {
-  it('should return false for non-existent file', () => {
-    const filePath = join(tempDir, 'nonexistent');
-    expect(uninstallHook(filePath)).toBe(false);
-  });
-
-  it('should return false for file without hook', () => {
-    const filePath = join(tempDir, '.zshrc');
-    writeFileSync(filePath, '# My config\nexport FOO=bar\n');
-
-    expect(uninstallHook(filePath)).toBe(false);
-
-    // Content should be unchanged
-    const content = readFileSync(filePath, 'utf-8');
-    expect(content).toBe('# My config\nexport FOO=bar\n');
-  });
-
-  it('should remove hook block from file', () => {
-    const filePath = join(tempDir, '.zshrc');
-    const contentWithHook = `# Before hook
-export FOO=bar
-
-${BASH_ZSH_HOOK}
-
-# After hook
-export BAZ=qux
-`;
-    writeFileSync(filePath, contentWithHook);
-
-    const result = uninstallHook(filePath);
-
-    expect(result).toBe(true);
-
-    const content = readFileSync(filePath, 'utf-8');
-    expect(content).not.toContain(MARKER_START);
-    expect(content).not.toContain(MARKER_END);
-    expect(content).not.toContain('__gitspace_preexec');
-    expect(content).toContain('FOO=bar');
-    expect(content).toContain('BAZ=qux');
-  });
-
-  it('should be idempotent - only remove once', () => {
-    const filePath = join(tempDir, '.zshrc');
-    writeFileSync(filePath, `# Config\n${BASH_ZSH_HOOK}\n# End\n`);
-
-    // First uninstall
-    expect(uninstallHook(filePath)).toBe(true);
-
-    // Second uninstall - nothing to remove
-    expect(uninstallHook(filePath)).toBe(false);
-  });
-
-  it('should handle hook at start of file', () => {
-    const filePath = join(tempDir, '.bashrc');
-    writeFileSync(filePath, `${BASH_ZSH_HOOK}\n# Rest of config\n`);
-
-    const result = uninstallHook(filePath);
-
-    expect(result).toBe(true);
-    const content = readFileSync(filePath, 'utf-8');
-    expect(content).not.toContain(MARKER_START);
-    expect(content).toContain('Rest of config');
-  });
-
-  it('should handle hook at end of file', () => {
-    const filePath = join(tempDir, '.bashrc');
-    writeFileSync(filePath, `# Start of config\nexport PATH="/bin"\n${BASH_ZSH_HOOK}`);
-
-    const result = uninstallHook(filePath);
-
-    expect(result).toBe(true);
-    const content = readFileSync(filePath, 'utf-8');
-    expect(content).not.toContain(MARKER_START);
-    expect(content).toContain('Start of config');
-  });
-
-  it('should preserve file content outside hook block', () => {
-    const filePath = join(tempDir, '.zshrc');
-    const beforeHook = `# ZSH Configuration
-export EDITOR=vim
-export PATH="/usr/local/bin:$PATH"
-
-# Aliases
-alias gs='git status'
-alias gd='git diff'
-`;
-    const afterHook = `
-# More config
-alias ll='ls -la'
-export NODE_ENV=development
-`;
-    writeFileSync(filePath, `${beforeHook}${BASH_ZSH_HOOK}${afterHook}`);
-
-    uninstallHook(filePath);
-
-    const content = readFileSync(filePath, 'utf-8');
-    expect(content).toContain('export EDITOR=vim');
-    expect(content).toContain("alias gs='git status'");
-    expect(content).toContain("alias ll='ls -la'");
-    expect(content).toContain('export NODE_ENV=development');
-  });
-});
-
-// ============================================================================
-// Round-trip tests (install then uninstall)
-// ============================================================================
-
-describe('install/uninstall round-trip', () => {
-  it('should restore file to near-original state after round-trip', () => {
-    const filePath = join(tempDir, '.zshrc');
-    const originalContent = `# My ZSH config
-export PATH="/usr/local/bin:$PATH"
-alias ll='ls -la'
-`;
-    writeFileSync(filePath, originalContent);
-
-    // Install
-    installHook(filePath, BASH_ZSH_HOOK);
-    const afterInstall = readFileSync(filePath, 'utf-8');
-    expect(afterInstall).toContain(MARKER_START);
-
-    // Uninstall
-    uninstallHook(filePath);
-    const afterUninstall = readFileSync(filePath, 'utf-8');
-
-    // Should contain original content
-    expect(afterUninstall).toContain('My ZSH config');
-    expect(afterUninstall).toContain('PATH="/usr/local/bin');
-    expect(afterUninstall).toContain("alias ll='ls -la'");
-    expect(afterUninstall).not.toContain(MARKER_START);
-  });
-
-  it('should work with fish shell config', () => {
-    const filePath = join(tempDir, 'config.fish');
-    const originalContent = `# Fish config
-set -x PATH /usr/local/bin $PATH
-`;
-    writeFileSync(filePath, originalContent);
-
-    installHook(filePath, FISH_HOOK);
-    expect(isHookInstalled(filePath)).toBe(true);
-
-    uninstallHook(filePath);
-    expect(isHookInstalled(filePath)).toBe(false);
-
-    const content = readFileSync(filePath, 'utf-8');
-    expect(content).toContain('Fish config');
-  });
-});
-
-// ============================================================================
-// Hook content validation
-// ============================================================================
-
-describe('hook content validation', () => {
-  it('BASH_ZSH_HOOK should contain required markers', () => {
-    expect(BASH_ZSH_HOOK).toContain(MARKER_START);
-    expect(BASH_ZSH_HOOK).toContain(MARKER_END);
-  });
-
-  it('BASH_ZSH_HOOK should check for TMUX_LITE env var', () => {
-    expect(BASH_ZSH_HOOK).toContain('TMUX_LITE');
-  });
-
-  it('BASH_ZSH_HOOK should emit OSC 133 sequences', () => {
-    expect(BASH_ZSH_HOOK).toContain('133;C');
-    expect(BASH_ZSH_HOOK).toContain('133;D');
-  });
-
-  it('BASH_ZSH_HOOK should emit OSC 777 for exit notifications', () => {
-    expect(BASH_ZSH_HOOK).toContain('777;exit');
-  });
-
-  it('FISH_HOOK should contain required markers', () => {
-    expect(FISH_HOOK).toContain(MARKER_START);
-    expect(FISH_HOOK).toContain(MARKER_END);
-  });
-
-  it('FISH_HOOK should check for TMUX_LITE env var', () => {
-    expect(FISH_HOOK).toContain('set -q TMUX_LITE');
-  });
-
-  it('FISH_HOOK should use fish event handlers', () => {
-    expect(FISH_HOOK).toContain('--on-event fish_preexec');
-    expect(FISH_HOOK).toContain('--on-event fish_postexec');
-  });
-});