Merge branch 'main' into fix/js-jest30-loop-runner

Resolved conflicts by:
1. Accepting origin/main's refactored verify_requirements() in support.py
   - Uses centralized find_node_modules_with_package() from init_javascript.py
   - Cleaner monorepo dependency detection

2. Accepting origin/main's refactored Jest parsing in parse_test_output.py
   - Jest-specific parsing moved to new codeflash/languages/javascript/parse.py
   - parse_test_xml() now routes to _parse_jest_test_xml() for JavaScript

3. Fixed TestType.GENERATED_PERFORMANCE bug in new parse.py
   - Changed to TestType.GENERATED_REGRESSION (performance tests are regression tests)
   - This was part of the original fixes in this branch

The merge preserves all the infrastructure fixes from this branch while
adopting the cleaner code organization from main.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: mohammedahmed18

.github/workflows/claude.yml

@@ -54,20 +54,28 @@ jobs:
             PR NUMBER: ${{ github.event.pull_request.number }}
             EVENT: ${{ github.event.action }}
 
-            ## STEP 1: Run pre-commit checks and fix issues
+            ## STEP 1: Run prek and mypy checks, fix issues
 
-            First, run `uv run prek run --from-ref origin/main` to check for linting/formatting issues on files changed in this PR.
+            First, run these checks on files changed in this PR:
+            1. `uv run prek run --from-ref origin/main` - linting/formatting issues
+            2. `uv run mypy <changed_files>` - type checking issues
 
-            If there are any issues:
+            If there are prek issues:
             - For SAFE auto-fixable issues (formatting, import sorting, trailing whitespace, etc.), run `uv run prek run --from-ref origin/main` again to auto-fix them
+
+            If there are mypy issues:
+            - Fix type annotation issues (missing return types, Optional/None unions, import errors for type hints, incorrect types)
+            - Do NOT add `type: ignore` comments - always fix the root cause
+
+            After fixing issues:
             - Stage the fixed files with `git add`
-            - Commit with message "style: auto-fix linting issues"
+            - Commit with message "style: auto-fix linting issues" or "fix: resolve mypy type errors" as appropriate
             - Push the changes with `git push`
 
             Do NOT attempt to fix:
-            - Type errors that require logic changes
-            - Complex refactoring suggestions
-            - Anything that could change behavior
+            - Type errors that require logic changes or refactoring
+            - Complex generic type issues
+            - Anything that could change runtime behavior
 
             ## STEP 2: Review the PR
 
@@ -85,7 +93,6 @@ jobs:
             - Only create NEW inline comments for HIGH-PRIORITY issues found in changed files.
             - Limit to 5-7 NEW comments maximum per review.
             - Use CLAUDE.md for project-specific guidance.
-            - Use `gh pr comment` for summary-level feedback.
             - Use `mcp__github_inline_comment__create_inline_comment` sparingly for critical code issues only.
 
             ## STEP 3: Coverage analysis
@@ -122,7 +129,45 @@ jobs:
             - New implementations/files: Must have ≥75% test coverage
             - Modified code: Changed lines should be exercised by existing or new tests
             - No coverage regressions: Overall coverage should not decrease
-          claude_args: '--allowedTools "mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*),Bash(gh issue view:*),Bash(gh issue list:*),Bash(gh api:*),Bash(uv run prek *),Bash(uv run coverage *),Bash(uv run pytest *),Bash(git status*),Bash(git add *),Bash(git commit *),Bash(git push*),Bash(git diff *),Bash(git checkout *),Read,Glob,Grep"'
+
+            ## STEP 4: Post ONE consolidated summary comment
+
+            CRITICAL: You must post exactly ONE summary comment containing ALL results (pre-commit, review, coverage).
+            DO NOT post multiple separate comments. Use this format:
+
+            ```
+            ## PR Review Summary
+
+            ### Prek Checks
+            [status and any fixes made]
+
+            ### Code Review
+            [critical issues found, if any]
+
+            ### Test Coverage
+            [coverage table and analysis]
+
+            ---
+            *Last updated: <timestamp>*
+            ```
+
+            To ensure only ONE comment exists:
+            1. Find existing claude[bot] comment: `gh api repos/${{ github.repository }}/issues/${{ github.event.pull_request.number }}/comments --jq '.[] | select(.user.login == "claude[bot]") | .id' | head -1`
+            2. If found, UPDATE it: `gh api --method PATCH repos/${{ github.repository }}/issues/comments/<ID> -f body="<content>"`
+            3. If not found, CREATE: `gh pr comment ${{ github.event.pull_request.number }} --body "<content>"`
+            4. Delete any OTHER claude[bot] comments to clean up duplicates: `gh api repos/${{ github.repository }}/issues/${{ github.event.pull_request.number }}/comments --jq '.[] | select(.user.login == "claude[bot]") | .id' | tail -n +2 | xargs -I {} gh api --method DELETE repos/${{ github.repository }}/issues/comments/{}`
+
+            ## STEP 5: Merge pending codeflash optimization PRs
+
+            Check for open optimization PRs from codeflash and merge if CI passes:
+
+            1. List open PRs from codeflash bot:
+               `gh pr list --author "codeflash-ai[bot]" --state open --json number,title,headRefName`
+
+            2. For each optimization PR:
+               - Check if CI is passing: `gh pr checks <number>`
+               - If all checks pass, merge it: `gh pr merge <number> --squash --delete-branch`
+          claude_args: '--allowedTools "mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*),Bash(gh pr checks:*),Bash(gh pr merge:*),Bash(gh issue view:*),Bash(gh issue list:*),Bash(gh api:*),Bash(uv run prek *),Bash(uv run mypy *),Bash(uv run coverage *),Bash(uv run pytest *),Bash(git status*),Bash(git add *),Bash(git commit *),Bash(git push*),Bash(git diff *),Bash(git checkout *),Read,Glob,Grep,Edit"'
           additional_permissions: |
             actions: read
         env:

CLAUDE.md

@@ -24,8 +24,8 @@ uv run mypy codeflash/           # Type check
 uv run ruff check codeflash/     # Lint
 uv run ruff format codeflash/    # Format
 
-# Pre-commit (run before committing)
-uv run pre-commit run --all-files
+# Linting (run before committing)
+uv run prek run --from-ref origin/main
 
 # Running the CLI
 uv run codeflash --help
@@ -69,7 +69,7 @@ codeflash/
 
 - **Line length**: 120 characters
 - **Python**: 3.9+ syntax
-- **Tooling**: Ruff for linting/formatting, mypy strict mode, pre-commit hooks
+- **Tooling**: Ruff for linting/formatting, mypy strict mode, prek for pre-commit checks
 - **Comments**: Minimal - only explain "why", not "what"
 - **Docstrings**: Do not add unless explicitly requested
 - **Naming**: NEVER use leading underscores (`_function_name`) - Python has no true private functions, use public names

codeflash/cli_cmds/init_javascript.py

@@ -146,6 +146,32 @@ def determine_js_package_manager(project_root: Path) -> JsPackageManager:
     return JsPackageManager.UNKNOWN
 
 
+def find_node_modules_with_package(project_root: Path, package_name: str) -> Path | None:
+    """Find node_modules directory containing a specific package.
+
+    Searches from project_root up to filesystem root for node_modules containing
+    the specified package. This supports monorepo setups where dependencies are
+    hoisted to the workspace root.
+
+    Args:
+        project_root: Starting directory for the search.
+        package_name: Name of the package to look for (e.g., "jest", "vitest").
+
+    Returns:
+        Path to the node_modules directory containing the package, or None if not found.
+
+    """
+    current_dir = project_root.resolve()
+    while current_dir != current_dir.parent:
+        node_modules = current_dir / "node_modules"
+        if node_modules.exists():
+            package_path = node_modules / package_name
+            if package_path.exists():
+                return node_modules
+        current_dir = current_dir.parent
+    return None
+
+
 def get_package_install_command(project_root: Path, package: str, dev: bool = True) -> list[str]:
     """Get the correct install command for the project's package manager.