address reviewer feedback

Author: Eric Oliver

docs/product-stories/cli-utility/dev-prompt.md

@@ -1,11 +1,19 @@
-we are ready to work on issue #15 (docs/product-stories/cli-utility/story-15-mcp-server-support.md) in repo https://github.com/sakamotopaya/code-agent.
+we are ready to work on issue #18 (docs/product-stories/cli-utility/story-18-update-documentation.md) in repo https://github.com/sakamotopaya/code-agent.
 follow the normal git flow. create a new local branch for the story.
 code the tasks and unit tests that prove the task are complete.
 if you need information about prior stories, you can find them locally here docs/product-stories/cli-utility
 we often get rejected trying to push our changes. make sure and run a build and lint prior to trying to push
 when you are finished with the code and tests, update the issue with a new comment describing your work and then
 push your branch and create a pull request for this branch against main
+you should use regular git command but it might be best to create the PR using the mcp server
 
-We need to resume work on issue #7 (docs/product-stories/cli-utility/story-07-cli-configuration-management.md) in repo https://github.com/sakamotopaya/code-agent.
-review the documents and complete the story. when you are finished with the code and tests, update the issue with a new comment describing your work and then
-push your branch and create a pull request for this branch against main
+We need to resume work on issue #17 (docs/product-stories/cli-utility/story-17-comprehensive-cli-testing.md) in repo https://github.com/sakamotopaya/code-agent.
+
+you were creating a pr when you filled up context. here is where you were on the pr creation:
+{
+"owner": "sakamotopaya",
+"repo": "code-agent",
+"title": "feat: Comprehensive CLI Testing Framework (Story 17)",
+"body": "## Summary\n\nImplements Story 17: Comprehensive CLI Testing with a complete testing framework for the CLI utility.\n\n## Changes Made\n\n### Testing Infrastructure\n- **Jest Configuration**: CLI-specific Jest config with 90% coverage threshold\n- **Test Utilities**: `TestHelpers` class with workspace management, CLI command execution, and performance measurement\n- **Mock Services**: Comprehensive mocking for UI, browser, session, and MCP services\n- **Test Setup**: Global test configuration with automatic cleanup\n\n### Test Categories Implemented\n\n#### Unit Tests (29 tests)\n- Console output handling\n- Basic functionality validation\n- Error handling and recovery\n- Async operations\n- Data types and validation\n- Environment and platform detection\n- Performance and memory operations\n- JSON and data processing\n- Edge cases and error conditions\n- Concurrency and timing\n\n#### Integration Tests\n- File operations (create, read, write, delete)\n- Directory operations and traversal\n- Project structure management\n- Configuration management\n- Output formatting validation (JSON, YAML, table, plain text)\n- Session management\n- Performance characteristics\n- Cross-platform compatibility\n- Concurrent operations\n- Resource cleanup\n\n#### End-to-End Tests\n- New user onboarding workflows\n- Complete development workflows\n- Project creation (simple, React, Node.js)\n- File management operations\n- Data processing workflows\n- Session lifecycle management\n- Error recovery scenarios\n- Configuration workflows\n\n#### Performance Tests\n- Startup time benchmarking (< 2 seconds)\n- Memory usage profiling (< 100MB increase)\n- Large file processing (< 5 seconds for 10MB)\n- Concurrent operation testing\n- Resource cleanup validation\n- Memory pressure testing\n\n#### Cross-Platform Tests\n- Platform detection (Windows, macOS, Linux)\n- File path handling\n- Environment variable management\n- Process operations\n- File system permissions\n- Text encoding and line endings\n- Node.js version compatibility\n\n### Test Scripts Added\n`bash\nnpm run test:cli              # All CLI tests\nnpm run test:cli:unit         # Unit tests only\nnpm run test:cli:integration  # Integration tests only\nnpm run test:cli:e2e          # End-to-end tests only\nnpm run test:cli:performance  # Performance tests only\nnpm run test:cli:platform     # Cross-platform tests only\nnpm run test:cli:coverage     # With coverage report\nnpm run test:cli:watch        # Watch mode for development\n`\n\n### Coverage Requirements\n- **Branches**: 90%\n- **Functions**: 90%\n- **Lines**: 90%\n- **Statements**: 90%\n\n## Test Results\n- ✅ All unit tests passing (29/29)\n- ✅ Lint passing with zero warnings\n- ✅ TypeScript compilation successful\n- ✅ Test infrastructure functional\n\n## Quality Assurance\n- All code follows ESLint rules\n- TypeScript strict mode compliance\n- Comprehensive error handling\n- Resource cleanup in all tests\n- Platform-agnostic test design\n- Performance benchmarks established\n\n## Files Added\n- `src/cli/jest.config.mjs` - Jest configuration for CLI testing\n- `src/cli/__tests__/setup.ts` - Global test setup and teardown\n- `src/cli/__tests__/utils/TestHelpers.ts` - Core test utilities\n- `src/cli/__tests__/utils/MockServices.ts` - Mock service implementations\n- `src/cli/__tests__/unit/services/CLIUIService.test.ts` - Unit tests\n- `src/cli/**tests**/integration/CLIIntegration.test.
+
+finish the PR and then add your comment to the issue. be sure and mark the issue as closes.

src/cli/__tests__/utils/TestHelpers.ts

@@ -51,7 +51,7 @@ export class TestHelpers {
 				reject(new Error(`Command timed out after ${timeout}ms`))
 			}, timeout)
 
-			const child = spawn("node", ["../dist/cli/index.js", ...args], {
+			const child = spawn("node", [path.resolve(__dirname, "../../index.js"), ...args], {
 				stdio: "pipe",
 				cwd,
 				env: { ...process.env, ...env },