jettypod 4.1.2 → 4.1.4

package/features/skills-update/index.test.js

@@ -1,216 +0,0 @@
-const fs = require('fs');
-const path = require('path');
-const { createTestEnvironment } = require('../../lib/test-helpers');
-const { execSync } = require('child_process');
-
-/**
- * End-to-end tests for skills update functionality
- * Tests the BDD scenarios from features/always-update-skills.feature
- */
-describe('Skills Update on JettyPod Init', () => {
-  let testEnv;
-  let jettypodPath;
-  let skillsSourceDir;
-
-  beforeEach(() => {
-    testEnv = createTestEnvironment();
-    process.chdir(testEnv.testDir);
-
-    // Path to jettypod script (relative to test directory)
-    jettypodPath = path.join(__dirname, '..', '..', 'jettypod.js');
-
-    // SAFETY: Create mock skills source directory in /tmp/ test environment, NOT in actual project
-    skillsSourceDir = path.join(testEnv.testDir, 'mock-jettypod-skills');
-    if (!fs.existsSync(skillsSourceDir)) {
-      fs.mkdirSync(skillsSourceDir, { recursive: true });
-    }
-
-    // Create sample skills for testing
-    const speedModeSkillDir = path.join(skillsSourceDir, 'speed-mode');
-    fs.mkdirSync(speedModeSkillDir, { recursive: true });
-    fs.writeFileSync(
-      path.join(speedModeSkillDir, 'SKILL.md'),
-      '# Speed Mode Skill\nTest content for speed mode skill'
-    );
-
-    const stableModeSkillDir = path.join(skillsSourceDir, 'stable-mode');
-    fs.mkdirSync(stableModeSkillDir, { recursive: true });
-    fs.writeFileSync(
-      path.join(stableModeSkillDir, 'SKILL.md'),
-      '# Stable Mode Skill\nTest content for stable mode skill'
-    );
-
-    // Configure jettypod to use mock skills source
-    process.env.JETTYPOD_SKILLS_SOURCE_DIR = skillsSourceDir;
-  });
-
-  afterEach(() => {
-    // Clean up environment variable
-    delete process.env.JETTYPOD_SKILLS_SOURCE_DIR;
-    testEnv.cleanup();
-  });
-
-  describe('Scenario: Skills are updated when initializing existing project', () => {
-    test('should backup existing skills before updating', () => {
-      // Given: I have a project with old .claude/skills/ directory
-      const claudeDir = path.join(testEnv.testDir, '.claude');
-      const oldSkillsDir = path.join(claudeDir, 'skills');
-      fs.mkdirSync(oldSkillsDir, { recursive: true });
-
-      const oldSkillDir = path.join(oldSkillsDir, 'old-skill');
-      fs.mkdirSync(oldSkillDir, { recursive: true });
-      fs.writeFileSync(
-        path.join(oldSkillDir, 'SKILL.md'),
-        '# Old Skill\nThis is an old skill that should be backed up'
-      );
-
-      // When: I run jettypod in the project directory
-      execSync(`node ${jettypodPath}`, { stdio: 'pipe', env: { ...process.env } });
-
-      // Then: old skills are backed up before replacement
-      const backupDirs = fs.readdirSync(claudeDir).filter(name => name.startsWith('skills.backup-'));
-      expect(backupDirs.length).toBe(1);
-
-      const backupDir = path.join(claudeDir, backupDirs[0]);
-      expect(fs.existsSync(backupDir)).toBe(true);
-
-      // Verify old skill is in backup
-      const backedUpSkillFile = path.join(backupDir, 'old-skill', 'SKILL.md');
-      expect(fs.existsSync(backedUpSkillFile)).toBe(true);
-      const backedUpContent = fs.readFileSync(backedUpSkillFile, 'utf8');
-      expect(backedUpContent).toContain('This is an old skill that should be backed up');
-    });
-
-    test('should replace skills with latest from jettypod', () => {
-      // Given: I have a project with old .claude/skills/ directory
-      const claudeDir = path.join(testEnv.testDir, '.claude');
-      const oldSkillsDir = path.join(claudeDir, 'skills');
-      fs.mkdirSync(oldSkillsDir, { recursive: true });
-
-      const oldSkillDir = path.join(oldSkillsDir, 'old-skill');
-      fs.mkdirSync(oldSkillDir, { recursive: true });
-      fs.writeFileSync(path.join(oldSkillDir, 'SKILL.md'), '# Old Skill');
-
-      // When: I run jettypod in the project directory
-      execSync(`node ${jettypodPath}`, { stdio: 'pipe', env: { ...process.env } });
-
-      // Then: .claude/skills/ is replaced with latest skills from jettypod
-      const newSkillsDir = path.join(testEnv.testDir, '.claude', 'skills');
-      expect(fs.existsSync(newSkillsDir)).toBe(true);
-
-      // Old skill should be gone
-      expect(fs.existsSync(path.join(newSkillsDir, 'old-skill'))).toBe(false);
-
-      // New skills should be present
-      expect(fs.existsSync(path.join(newSkillsDir, 'speed-mode', 'SKILL.md'))).toBe(true);
-      expect(fs.existsSync(path.join(newSkillsDir, 'stable-mode', 'SKILL.md'))).toBe(true);
-
-      // Verify content is from jettypod source
-      const speedModeContent = fs.readFileSync(
-        path.join(newSkillsDir, 'speed-mode', 'SKILL.md'),
-        'utf8'
-      );
-      expect(speedModeContent).toContain('Test content for speed mode skill');
-    });
-
-    test('should handle concurrent inits with unique backup names', () => {
-      // Given: I have a project with .claude/skills/ directory
-      const claudeDir = path.join(testEnv.testDir, '.claude');
-      const skillsDir = path.join(claudeDir, 'skills');
-      fs.mkdirSync(skillsDir, { recursive: true });
-      fs.writeFileSync(path.join(skillsDir, 'test.md'), 'test');
-
-      // Manually create a backup with current timestamp format
-      const timestamp = new Date().toISOString().replace(/[:.]/g, '-');
-      const existingBackup = path.join(claudeDir, `skills.backup-${timestamp}`);
-      fs.mkdirSync(existingBackup, { recursive: true });
-      fs.writeFileSync(path.join(existingBackup, 'existing.md'), 'existing');
-
-      // Restore skills dir for the actual init
-      if (fs.existsSync(skillsDir)) {
-        fs.rmSync(skillsDir, { recursive: true, force: true });
-      }
-      fs.mkdirSync(skillsDir, { recursive: true });
-      fs.writeFileSync(path.join(skillsDir, 'test.md'), 'test');
-
-      // When: I run jettypod (which creates another backup with same timestamp)
-      execSync(`node ${jettypodPath}`, { stdio: 'pipe', env: { ...process.env } });
-
-      // Then: backup directory should have counter appended
-      const backupDirs = fs.readdirSync(claudeDir)
-        .filter(name => name.startsWith('skills.backup-'))
-        .sort();
-
-      // Should have at least 2 backups (the manual one and the new one)
-      expect(backupDirs.length).toBeGreaterThanOrEqual(2);
-    });
-  });
-
-  describe('Scenario: Skills are copied when initializing new project', () => {
-    test('should create .claude/skills/ with latest skills', () => {
-      // Given: I have a new project with no .claude/ directory
-      // (testEnv starts clean)
-
-      // When: I run jettypod to initialize
-      execSync(`node ${jettypodPath}`, { stdio: 'pipe', env: { ...process.env } });
-
-      // Then: .claude/skills/ is created with latest skills from jettypod
-      const skillsDir = path.join(testEnv.testDir, '.claude', 'skills');
-      expect(fs.existsSync(skillsDir)).toBe(true);
-
-      // And: all skills are ready to use
-      expect(fs.existsSync(path.join(skillsDir, 'speed-mode', 'SKILL.md'))).toBe(true);
-      expect(fs.existsSync(path.join(skillsDir, 'stable-mode', 'SKILL.md'))).toBe(true);
-
-      const speedModeContent = fs.readFileSync(
-        path.join(skillsDir, 'speed-mode', 'SKILL.md'),
-        'utf8'
-      );
-      expect(speedModeContent).toContain('Test content for speed mode skill');
-    });
-
-    test('should not create backup when no existing skills', () => {
-      // Given: I have a new project with no .claude/ directory
-      // When: I run jettypod to initialize
-      execSync(`node ${jettypodPath}`, { stdio: 'pipe', env: { ...process.env } });
-
-      // Then: no backup should be created
-      const claudeDir = path.join(testEnv.testDir, '.claude');
-      const backupDirs = fs.readdirSync(claudeDir).filter(name => name.startsWith('skills.backup-'));
-      expect(backupDirs.length).toBe(0);
-    });
-  });
-
-  describe('Error handling validation', () => {
-    test('should handle source directory missing gracefully', () => {
-      // Remove skills source directory
-      if (fs.existsSync(skillsSourceDir)) {
-        fs.rmSync(skillsSourceDir, { recursive: true, force: true });
-      }
-
-      // Should not throw, just warn
-      expect(() => {
-        execSync(`node ${jettypodPath}`, { stdio: 'pipe', env: { ...process.env } });
-      }).not.toThrow();
-
-      // .claude should still be created
-      expect(fs.existsSync(path.join(testEnv.testDir, '.claude'))).toBe(true);
-    });
-
-    test('should handle empty source directory gracefully', () => {
-      // Empty the skills source directory
-      if (fs.existsSync(skillsSourceDir)) {
-        fs.rmSync(skillsSourceDir, { recursive: true, force: true });
-      }
-      fs.mkdirSync(skillsSourceDir, { recursive: true });
-
-      // Should not throw, just warn
-      expect(() => {
-        execSync(`node ${jettypodPath}`, { stdio: 'pipe', env: { ...process.env } });
-      }).not.toThrow();
-
-      // .claude should still be created
-      expect(fs.existsSync(path.join(testEnv.testDir, '.claude'))).toBe(true);
-    });
-  });
-});

package/features/step_definitions/backlog-command.steps.js

@@ -1,37 +0,0 @@
-const { Given, Then } = require('@cucumber/cucumber');
-
-// Test data setup
-Given('I have work items in various statuses', async function () {
-  // Speed mode will implement: Create test work items with different statuses
-  // (backlog, todo, in_progress, done, cancelled)
-  this.testWorkItems = null; // Placeholder for speed mode
-});
-
-// Output assertions
-Then('I see only active work items \\(backlog, todo, in_progress)', function () {
-  // Speed mode will implement: Parse output and verify only active items shown
-  // Check output contains items with status: backlog, todo, in_progress
-  // Check output does NOT contain items with status: done, cancelled
-});
-
-Then('I see all work items regardless of status', function () {
-  // Speed mode will implement: Parse output and verify all items shown
-  // Check output contains items with all statuses
-});
-
-Then('I see only completed work items \\(done, cancelled)', function () {
-  // Speed mode will implement: Parse output and verify only completed items shown
-  // Check output contains items with status: done, cancelled
-  // Check output does NOT contain items with status: backlog, todo, in_progress
-});
-
-Then('I see the items in a tree hierarchy', function () {
-  // Speed mode will implement: Verify tree structure in output
-  // Check for indentation or tree symbols (├──, └──, etc.)
-  // Check parent-child relationships are preserved
-});
-
-Then('completed items are hidden', function () {
-  // Speed mode will implement: Verify no completed items in output
-  // Check output does NOT contain items with status: done, cancelled
-});

package/features/step_definitions/fix-text-wrapping.steps.js

@@ -1,271 +0,0 @@
-const { Given, When, Then, Before, After, setWorldConstructor } = require('@cucumber/cucumber');
-const assert = require('assert');
-const { execSync } = require('child_process');
-const path = require('path');
-const fs = require('fs');
-
-const jettypodPath = path.join(__dirname, '../../jettypod.js');
-
-// Global testDir for compatibility with work-commands steps
-global.testDir = undefined;
-
-// Setup test directory before each scenario
-Before({ tags: '@text-wrapping' }, async function () {
-  // Create unique test directory for this scenario
-  this.testDir = path.join('/tmp', 'jettypod-text-wrap-test-' + Date.now() + '-' + Math.random().toString(36).substring(7));
-  global.testDir = this.testDir;
-
-  // Clean up if it exists
-  if (fs.existsSync(this.testDir) && this.testDir.startsWith('/tmp/')) {
-    fs.rmSync(this.testDir, { recursive: true, force: true });
-  }
-  fs.mkdirSync(this.testDir, { recursive: true });
-
-  // Initialize jettypod in test directory
-  execSync(`node ${jettypodPath} init`, {
-    cwd: this.testDir,
-    env: { ...process.env, NODE_ENV: 'test' },
-    stdio: 'pipe'
-  });
-
-  this.terminalWidth = 80; // Default
-
-  // Helper to run jettypod commands in this test's directory
-  this.runJettypod = (args, termWidth = this.terminalWidth) => {
-    const env = {
-      ...process.env,
-      NODE_ENV: 'test',
-      COLUMNS: termWidth.toString()
-    };
-
-    try {
-      const output = execSync(`node ${jettypodPath} ${args}`, {
-        cwd: this.testDir,
-        encoding: 'utf-8',
-        env
-      });
-      this.commandOutput = output;
-      this.output = output;
-      return output;
-    } catch (err) {
-      const output = err.stdout || '';
-      this.commandOutput = output;
-      this.output = output;
-      return output;
-    }
-  };
-});
-
-// Cleanup after each scenario
-After({ tags: '@text-wrapping' }, function () {
-  if (this.testDir && fs.existsSync(this.testDir) && this.testDir.startsWith('/tmp/')) {
-    fs.rmSync(this.testDir, { recursive: true, force: true });
-  }
-  global.testDir = undefined;
-});
-
-// Test setup - terminal width and test items
-Given('I have a feature with a title longer than the terminal width', function () {
-  const longTitle = 'This is an extremely long feature title that will definitely exceed any reasonable terminal width and should wrap at word boundaries when displayed in the backlog output';
-  this.runJettypod(`work create feature "${longTitle}" "Test description"`);
-  this.terminalWidth = 80;
-});
-
-Given('I have a feature with a long description', function () {
-  const longDesc = 'This is a very detailed description that explains the feature in great depth and will definitely need to wrap across multiple lines when displayed in the terminal because it contains so much information about what the feature does and why it exists and how it should be implemented';
-  this.runJettypod(`work create feature "Test Feature" "${longDesc}"`);
-  this.terminalWidth = 80;
-});
-
-Given('I have nested items with long titles', function () {
-  // Create epic
-  const epicOutput = this.runJettypod('work create epic "Test Epic" "Epic description"');
-  const epicMatch = epicOutput.match(/Created epic #(\d+)/);
-  const epicId = epicMatch ? epicMatch[1] : '1';
-
-  const longTitle1 = 'This is a very long feature title that should wrap when displayed as a nested item under the epic';
-  const longTitle2 = 'Another extremely long feature title to test tree structure alignment with wrapped text';
-
-  this.runJettypod(`work create feature "${longTitle1}" "" --parent=${epicId}`);
-  this.runJettypod(`work create feature "${longTitle2}" "" --parent=${epicId}`);
-
-  this.epicId = epicId;
-});
-
-Given('the terminal width is {int} columns', function (width) {
-  this.terminalWidth = width;
-});
-
-Given('I have items with titles that wrap at {int} columns', function (columns) {
-  this.terminalWidth = columns;
-  const longTitle = 'A' + ' very'.repeat(20) + ' long title';
-  this.runJettypod(`work create feature "${longTitle}" ""`);
-});
-
-// Actions
-When(/^I run jettypod (.+) with current terminal width$/, function (args) {
-  this.runJettypod(args, this.terminalWidth);
-});
-
-When('the terminal is resized to {int} columns', function (columns) {
-  this.terminalWidth = columns;
-});
-
-// Assertions - word boundary wrapping
-Then('the title wraps at word boundaries', function () {
-  assert(this.output, 'Should have output');
-  const lines = this.output.split('\n');
-
-  for (let i = 0; i < lines.length - 1; i++) {
-    const currentLine = lines[i];
-    const nextLine = lines[i + 1];
-
-    if (currentLine && nextLine) {
-      // Check if the current line ends with a hyphen (mid-word break)
-      // or if when we remove trailing/leading spaces there's a word split
-      const trimmedCurrent = currentLine.trimEnd();
-      const trimmedNext = nextLine.trimStart();
-
-      if (trimmedCurrent && trimmedNext) {
-        const lastChar = trimmedCurrent[trimmedCurrent.length - 1];
-
-        // If line ends with hyphen, that's intentional for very long words
-        if (lastChar === '-') continue;
-
-        // If line ends with alphanumeric and next line starts with alphanumeric,
-        // but current line didn't have trailing space, then word was split
-        const hadTrailingSpace = currentLine !== trimmedCurrent;
-        const nextHasLeadingSpace = nextLine !== trimmedNext;
-
-        if (!hadTrailingSpace && !nextHasLeadingSpace) {
-          const isLastAlpha = /[a-zA-Z0-9]/.test(lastChar);
-          const firstChar = trimmedNext[0];
-          const isFirstAlpha = /[a-zA-Z0-9]/.test(firstChar);
-
-          assert(!(isLastAlpha && isFirstAlpha),
-            `Word split between lines ${i} and ${i+1}: "${trimmedCurrent}" | "${trimmedNext}"`);
-        }
-      }
-    }
-  }
-});
-
-Then('continuation lines align with the content indentation', function () {
-  assert(this.output, 'Should have output');
-  // Verified by implementation design
-});
-
-Then('no words are split mid-character', function () {
-  assert(this.output, 'Should have output');
-  const lines = this.output.split('\n');
-
-  for (let i = 0; i < lines.length - 1; i++) {
-    const currentLine = lines[i];
-    const nextLine = lines[i + 1];
-
-    if (currentLine && nextLine) {
-      const trimmedCurrent = currentLine.trimEnd();
-      const trimmedNext = nextLine.trimStart();
-
-      if (trimmedCurrent && trimmedNext) {
-        const lastChar = trimmedCurrent[trimmedCurrent.length - 1];
-        if (lastChar === '-') continue; // Hyphenation is allowed
-
-        const hadTrailingSpace = currentLine !== trimmedCurrent;
-        const nextHasLeadingSpace = nextLine !== trimmedNext;
-
-        if (!hadTrailingSpace && !nextHasLeadingSpace) {
-          const isLastAlpha = /[a-zA-Z0-9]/.test(lastChar);
-          const firstChar = trimmedNext[0];
-          const isFirstAlpha = /[a-zA-Z0-9]/.test(firstChar);
-
-          assert(!(isLastAlpha && isFirstAlpha), 'Words should not be split mid-character');
-        }
-      }
-    }
-  }
-});
-
-Then('the description wraps at word boundaries', function () {
-  assert(this.output, 'Should have output');
-  const lines = this.output.split('\n');
-
-  for (let i = 0; i < lines.length - 1; i++) {
-    const currentLine = lines[i];
-    const nextLine = lines[i + 1];
-
-    if (currentLine && nextLine && currentLine.includes('Description:')) {
-      const trimmedCurrent = currentLine.trimEnd();
-      const trimmedNext = nextLine.trimStart();
-
-      if (trimmedCurrent && trimmedNext) {
-        const lastChar = trimmedCurrent[trimmedCurrent.length - 1];
-        if (lastChar === '-') continue; // Hyphenation is allowed
-
-        const hadTrailingSpace = currentLine !== trimmedCurrent;
-        const nextHasLeadingSpace = nextLine !== trimmedNext;
-
-        if (!hadTrailingSpace && !nextHasLeadingSpace) {
-          const isLastAlpha = /[a-zA-Z0-9]/.test(lastChar);
-          const firstChar = trimmedNext[0];
-          const isFirstAlpha = /[a-zA-Z0-9]/.test(firstChar);
-
-          assert(!(isLastAlpha && isFirstAlpha), 'Description words should not be split');
-        }
-      }
-    }
-  }
-});
-
-Then('continuation lines have proper {string} prefix indentation', function (prefix) {
-  assert(this.output, 'Should have output');
-  // Verified by implementation design
-});
-
-Then('the text fits within terminal width', function () {
-  assert(this.output, 'Should have output');
-  const lines = this.output.split('\n');
-  const maxWidth = this.terminalWidth;
-
-  for (const line of lines) {
-    // Strip ANSI codes
-    const cleanLine = line.replace(/\x1b\[[0-9;]*m/g, '');
-    assert(cleanLine.length <= maxWidth,
-      `Line exceeds terminal width (${cleanLine.length} > ${maxWidth}): "${cleanLine.substring(0, 50)}..."`);
-  }
-});
-
-// Assertions - tree structure
-Then('tree connectors \\(├──, └──) display correctly', function () {
-  assert(this.output, 'Should have output');
-  const hasTreeConnectors = this.output.includes('├──') || this.output.includes('└──');
-  assert(hasTreeConnectors, 'Output should contain tree connectors');
-});
-
-Then('wrapped continuation lines align under the item content', function () {
-  assert(this.output, 'Should have output');
-  // Verified by implementation design
-});
-
-Then('tree structure remains visually clear', function () {
-  assert(this.output, 'Should have output');
-  const hasVerticalLines = this.output.includes('│');
-  const hasTreeConnectors = this.output.includes('├──') || this.output.includes('└──');
-  assert(hasVerticalLines || hasTreeConnectors, 'Tree structure should be present');
-});
-
-Then('text wraps to fit {int} columns', function (columns) {
-  assert(this.output, 'Should have output');
-  const lines = this.output.split('\n');
-
-  for (const line of lines) {
-    const cleanLine = line.replace(/\x1b\[[0-9;]*m/g, '');
-    assert(cleanLine.length <= columns,
-      `Line exceeds ${columns} columns: ${cleanLine.length}`);
-  }
-});
-
-Then('alignment adjusts to the new width', function () {
-  assert(this.output, 'Should have output');
-  // Tested by running command again with new width
-});