feat: add OpenSpec change proposal for archive command

openspec/changes/add-archive-command/proposal.md

@@ -0,0 +1,12 @@
+## Why
+Need a command to archive completed changes to the archive folder with proper date prefixing, following OpenSpec conventions. Currently changes must be manually moved and renamed.
+
+## What Changes
+- Add new `archive` command to CLI that moves changes to `changes/archive/YYYY-MM-DD-[change-name]/`
+- Check for incomplete tasks before archiving and warn user
+- Allow interactive selection of change to archive
+- Prevent archiving if target directory already exists
+
+## Impact
+- Affected specs: cli-archive (new)
+- Affected code: src/cli/index.ts, src/core/archive.ts (new)

openspec/changes/add-archive-command/specs/cli-archive/spec.md

@@ -0,0 +1,60 @@
+# CLI Archive Command Specification
+
+## Purpose
+The archive command moves completed changes from the active changes directory to the archive folder with date-based naming, following OpenSpec conventions.
+
+## Command Syntax
+```bash
+openspec archive [change-name]
+```
+
+## Behavior
+
+### Change Selection
+WHEN no change-name is provided
+THEN display interactive list of available changes (excluding archive/)
+AND allow user to select one
+
+WHEN change-name is provided
+THEN use that change directly
+AND validate it exists
+
+### Task Completion Check
+The command SHALL scan the change's tasks.md file for incomplete tasks (marked with `- [ ]`)
+
+WHEN incomplete tasks are found
+THEN display all incomplete tasks to the user
+AND prompt for confirmation to continue
+AND default to "No" for safety
+
+WHEN all tasks are complete OR no tasks.md exists
+THEN proceed with archiving without prompting
+
+### Archive Process
+The archive operation SHALL:
+1. Create archive/ directory if it doesn't exist
+2. Generate target name as `YYYY-MM-DD-[change-name]` using current date
+3. Check if target directory already exists
+4. Move the entire change directory to the archive location
+
+WHEN target archive already exists
+THEN fail with error message
+AND do not overwrite existing archive
+
+WHEN move succeeds
+THEN display success message with archived name
+
+## Error Handling
+
+SHALL handle the following error conditions:
+- Missing openspec/changes/ directory
+- Change not found
+- Archive target already exists
+- File system permissions issues
+
+## Why These Decisions
+
+**Interactive selection**: Reduces typing and helps users see available changes
+**Task checking**: Prevents accidental archiving of incomplete work
+**Date prefixing**: Maintains chronological order and prevents naming conflicts
+**No overwrite**: Preserves historical archives and prevents data loss

openspec/changes/add-archive-command/tasks.md

@@ -0,0 +1,30 @@
+# Implementation Tasks
+
+## 1. Core Implementation
+- [ ] 1.1 Create `src/core/archive.ts` with ArchiveCommand class
+  - [ ] 1.1.1 Implement change selection (interactive if not provided)
+  - [ ] 1.1.2 Implement incomplete task checking from tasks.md
+  - [ ] 1.1.3 Implement confirmation prompt for incomplete tasks
+  - [ ] 1.1.4 Implement archive move with date prefixing
+
+## 2. CLI Integration
+- [ ] 2.1 Add archive command to `src/cli/index.ts`
+  - [ ] 2.1.1 Import ArchiveCommand
+  - [ ] 2.1.2 Register command with commander
+  - [ ] 2.1.3 Add proper error handling
+
+## 3. Error Handling
+- [ ] 3.1 Handle missing openspec/changes/ directory
+- [ ] 3.2 Handle change not found
+- [ ] 3.3 Handle archive target already exists
+- [ ] 3.4 Handle user cancellation
+
+## 4. Testing
+- [ ] 4.1 Test with fully completed change
+- [ ] 4.2 Test with incomplete tasks (warning shown)
+- [ ] 4.3 Test interactive selection mode
+- [ ] 4.4 Test duplicate archive prevention
+
+## 5. Build and Validation
+- [ ] 5.1 Ensure TypeScript compilation succeeds
+- [ ] 5.2 Test command execution

src/cli/index.ts

@@ -5,7 +5,6 @@ import { promises as fs } from 'fs';
 import { InitCommand } from '../core/init.js';
 import { UpdateCommand } from '../core/update.js';
 import { DiffCommand } from '../core/diff.js';
-import { ListCommand } from '../core/list.js';
 
 const program = new Command();
 
@@ -76,18 +75,4 @@ program
     }
   });
 
-program
-  .command('list')
-  .description('List all active changes with their task status')
-  .action(async () => {
-    try {
-      const listCommand = new ListCommand();
-      await listCommand.execute();
-    } catch (error) {
-      console.log(); // Empty line for spacing
-      ora().fail(`Error: ${(error as Error).message}`);
-      process.exit(1);
-    }
-  });
-
 program.parse();

src/core/list.ts

@@ -1,90 +0,0 @@
-import { promises as fs } from 'fs';
-import path from 'path';
-
-interface ChangeInfo {
-  name: string;
-  completedTasks: number;
-  totalTasks: number;
-}
-
-export class ListCommand {
-  async execute(targetPath: string = '.'): Promise<void> {
-    const changesDir = path.join(targetPath, 'openspec', 'changes');
-    
-    // Check if changes directory exists
-    try {
-      await fs.access(changesDir);
-    } catch {
-      throw new Error("No OpenSpec changes directory found. Run 'openspec init' first.");
-    }
-
-    // Get all directories in changes (excluding archive)
-    const entries = await fs.readdir(changesDir, { withFileTypes: true });
-    const changeDirs = entries
-      .filter(entry => entry.isDirectory() && entry.name !== 'archive')
-      .map(entry => entry.name);
-
-    if (changeDirs.length === 0) {
-      console.log('No active changes found.');
-      return;
-    }
-
-    // Collect information about each change
-    const changes: ChangeInfo[] = [];
-    
-    for (const changeDir of changeDirs) {
-      const tasksPath = path.join(changesDir, changeDir, 'tasks.md');
-      let completedTasks = 0;
-      let incompleteTasks = 0;
-      
-      try {
-        const content = await fs.readFile(tasksPath, 'utf-8');
-        const lines = content.split('\n');
-        
-        for (const line of lines) {
-          if (line.includes('- [x]')) {
-            completedTasks++;
-          } else if (line.includes('- [ ]')) {
-            incompleteTasks++;
-          }
-        }
-      } catch {
-        // No tasks.md file
-        changes.push({
-          name: changeDir,
-          completedTasks: 0,
-          totalTasks: 0
-        });
-        continue;
-      }
-      
-      changes.push({
-        name: changeDir,
-        completedTasks,
-        totalTasks: completedTasks + incompleteTasks
-      });
-    }
-
-    // Sort alphabetically by name
-    changes.sort((a, b) => a.name.localeCompare(b.name));
-
-    // Display results
-    console.log('Changes:');
-    for (const change of changes) {
-      const padding = '  ';
-      const nameWidth = Math.max(...changes.map(c => c.name.length));
-      const paddedName = change.name.padEnd(nameWidth);
-      
-      let status: string;
-      if (change.totalTasks === 0) {
-        status = 'No tasks';
-      } else if (change.completedTasks === change.totalTasks) {
-        status = '✓ Complete';
-      } else {
-        status = `${change.completedTasks}/${change.totalTasks} tasks`;
-      }
-      
-      console.log(`${padding}${paddedName}     ${status}`);
-    }
-  }
-}

test/core/list.test.ts

@@ -1,165 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { promises as fs } from 'fs';
-import path from 'path';
-import os from 'os';
-import { ListCommand } from '../../src/core/list.js';
-
-describe('ListCommand', () => {
-  let tempDir: string;
-  let originalLog: typeof console.log;
-  let logOutput: string[] = [];
-
-  beforeEach(async () => {
-    // Create temp directory
-    tempDir = path.join(os.tmpdir(), `openspec-list-test-${Date.now()}`);
-    await fs.mkdir(tempDir, { recursive: true });
-
-    // Mock console.log to capture output
-    originalLog = console.log;
-    console.log = (...args: any[]) => {
-      logOutput.push(args.join(' '));
-    };
-    logOutput = [];
-  });
-
-  afterEach(async () => {
-    // Restore console.log
-    console.log = originalLog;
-
-    // Clean up temp directory
-    await fs.rm(tempDir, { recursive: true, force: true });
-  });
-
-  describe('execute', () => {
-    it('should handle missing openspec/changes directory', async () => {
-      const listCommand = new ListCommand();
-      
-      await expect(listCommand.execute(tempDir)).rejects.toThrow(
-        "No OpenSpec changes directory found. Run 'openspec init' first."
-      );
-    });
-
-    it('should handle empty changes directory', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      await fs.mkdir(changesDir, { recursive: true });
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir);
-
-      expect(logOutput).toEqual(['No active changes found.']);
-    });
-
-    it('should exclude archive directory', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      await fs.mkdir(path.join(changesDir, 'archive'), { recursive: true });
-      await fs.mkdir(path.join(changesDir, 'my-change'), { recursive: true });
-      
-      // Create tasks.md with some tasks
-      await fs.writeFile(
-        path.join(changesDir, 'my-change', 'tasks.md'),
-        '- [x] Task 1\n- [ ] Task 2\n'
-      );
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir);
-
-      expect(logOutput).toContain('Changes:');
-      expect(logOutput.some(line => line.includes('my-change'))).toBe(true);
-      expect(logOutput.some(line => line.includes('archive'))).toBe(false);
-    });
-
-    it('should count tasks correctly', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      await fs.mkdir(path.join(changesDir, 'test-change'), { recursive: true });
-      
-      await fs.writeFile(
-        path.join(changesDir, 'test-change', 'tasks.md'),
-        `# Tasks
-- [x] Completed task 1
-- [x] Completed task 2
-- [ ] Incomplete task 1
-- [ ] Incomplete task 2
-- [ ] Incomplete task 3
-Regular text that should be ignored
-`
-      );
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir);
-
-      expect(logOutput.some(line => line.includes('2/5 tasks'))).toBe(true);
-    });
-
-    it('should show complete status for fully completed changes', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      await fs.mkdir(path.join(changesDir, 'completed-change'), { recursive: true });
-      
-      await fs.writeFile(
-        path.join(changesDir, 'completed-change', 'tasks.md'),
-        '- [x] Task 1\n- [x] Task 2\n- [x] Task 3\n'
-      );
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir);
-
-      expect(logOutput.some(line => line.includes('✓ Complete'))).toBe(true);
-    });
-
-    it('should handle changes without tasks.md', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      await fs.mkdir(path.join(changesDir, 'no-tasks'), { recursive: true });
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir);
-
-      expect(logOutput.some(line => line.includes('no-tasks') && line.includes('No tasks'))).toBe(true);
-    });
-
-    it('should sort changes alphabetically', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      await fs.mkdir(path.join(changesDir, 'zebra'), { recursive: true });
-      await fs.mkdir(path.join(changesDir, 'alpha'), { recursive: true });
-      await fs.mkdir(path.join(changesDir, 'middle'), { recursive: true });
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir);
-
-      const changeLines = logOutput.filter(line => 
-        line.includes('alpha') || line.includes('middle') || line.includes('zebra')
-      );
-      
-      expect(changeLines[0]).toContain('alpha');
-      expect(changeLines[1]).toContain('middle');
-      expect(changeLines[2]).toContain('zebra');
-    });
-
-    it('should handle multiple changes with various states', async () => {
-      const changesDir = path.join(tempDir, 'openspec', 'changes');
-      
-      // Complete change
-      await fs.mkdir(path.join(changesDir, 'completed'), { recursive: true });
-      await fs.writeFile(
-        path.join(changesDir, 'completed', 'tasks.md'),
-        '- [x] Task 1\n- [x] Task 2\n'
-      );
-
-      // Partial change
-      await fs.mkdir(path.join(changesDir, 'partial'), { recursive: true });
-      await fs.writeFile(
-        path.join(changesDir, 'partial', 'tasks.md'),
-        '- [x] Done\n- [ ] Not done\n- [ ] Also not done\n'
-      );
-
-      // No tasks
-      await fs.mkdir(path.join(changesDir, 'no-tasks'), { recursive: true });
-
-      const listCommand = new ListCommand();
-      await listCommand.execute(tempDir);
-
-      expect(logOutput).toContain('Changes:');
-      expect(logOutput.some(line => line.includes('completed') && line.includes('✓ Complete'))).toBe(true);
-      expect(logOutput.some(line => line.includes('partial') && line.includes('1/3 tasks'))).toBe(true);
-      expect(logOutput.some(line => line.includes('no-tasks') && line.includes('No tasks'))).toBe(true);
-    });
-  });
-});