feat: add automated testing infrastructure, code cleanup, and update default model #23
Open
coderashed wants to merge 13 commits intoCerebras:mainfrom
Open
feat: add automated testing infrastructure, code cleanup, and update default model #23coderashed wants to merge 13 commits intoCerebras:mainfrom
coderashed wants to merge 13 commits intoCerebras:mainfrom
Conversation
- Add vitest for unit testing with coverage support - Add eslint 9 with flat config for linting - Add prettier for code formatting - Add npm scripts: test, lint, format - Create tests/ directory structure mirroring src/ - Add sample rate-limiter tests Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Rewrite all principle docs from Python to JavaScript - Add JSDoc documentation guide (replaces type hints) - Add ESLint complexity guide (replaces radon) - Update GoF patterns with JS-idiomatic recommendations - Mark over-engineered patterns to avoid (Singleton, Strategy classes, etc.) - Update TDD workflow for npm/vitest/eslint - Add docs/testing.md with vitest usage guide - Add .ai/intuitions/performance.md with optimization ideas Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add diff-formatter.test.js for diff generation - Add response-formatter.test.js for edit/create responses - Add syntax-highlighter.test.js for code highlighting Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Fix async timing in API tests with vi.waitFor() for existingContent reads - Update constants test to match v1.3.3 debugLog behavior (always logs) - Remove OpenRouter timeout test (not implemented in v1.3.3) - Add caughtErrorsIgnorePattern to eslint for unused catch variables - Exclude interactive-config.js from linting (upstream code) - Fix minor lint issues: remove unused imports, use const where appropriate All 125 tests passing, lint clean.
- Allow OpenRouter model to be configured via OPENROUTER_MODEL env var - Default remains 'qwen/qwen3-coder' for backwards compatibility - Add environment variables documentation table to README
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add integration test suite exercising Cerebras and OpenRouter providers - Add fallback test that verifies Cerebras → OpenRouter failover path - Add test helpers for provider isolation and fallback forcing - Add vitest.integration.config.js with 30s timeout for API latency - Add test:integration npm script - Exclude integration tests from unit test runs Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add integration test section covering prerequisites, test helpers, fallback testing strategy, and vitest integration config. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Exclude interactive-config.js from coverage (untestable CLI wizard) - Add getClineRulesPath platform branch tests (win32/linux) - Add config env var branch tests (CEREBRAS_MODEL, MAX_TOKENS, OPENROUTER_MODEL) - Add router no-fallback path tests (OpenRouter-only provider) - Add response-formatter IDE variant tests (cursor, crush, cline, vscode) - Add response-formatter typescript file and truncation tests - Add diff-formatter removal edge case test - Add API context file error, filtering, and empty content tests - Add tool-handlers empty file edge case test 155 tests passing, 99.27% statements, 95.37% branches, 100% functions. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove unused imports (beforeEach, afterEach, vi, cleanCodeResponse) and unused variable assignment in tool-handlers test. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…andling - Edit existing file (per provider): seed file, modify via API, verify diff response - Context files: pass context_files param, verify API uses context - Nested directory creation: file_path with non-existent parents - Error handling: missing prompt, missing file_path, invalid API key - All return structured responses (no uncaught throws) 10 integration tests passing (was 3). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Error tests (missing prompt, missing file_path, invalid key) don't require real API keys. They now run even when env vars are not set. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
src/modules at 99%+ coverageOPENROUTER_MODELenv var support for configurable model selectiondocs/testing.md)Test results
Test plan
npm test— all 155 unit tests passnpm run test:coverage— exceeds 80% target across all metricsnpm run test:integration— all 10 integration tests pass (Cerebras, OpenRouter, fallback, edit path, context files, error handling)npm run lint— cleanAll functionality has been hand tested in Claude Code.