Merge branch 'main' into feat/subagent-low-effort-defaults

Author: aseembits93

.claude/rules/code-style.md

@@ -9,3 +9,5 @@
 - **Naming**: NEVER use leading underscores (`_function_name`) - Python has no true private functions, use public names
 - **Paths**: Always use absolute paths
 - **Encoding**: Always pass `encoding="utf-8"` to `open()`, `read_text()`, `write_text()`, etc. in new or changed code — Windows defaults to `cp1252` which breaks on non-ASCII content. Don't flag pre-existing code that lacks it unless you're already modifying that line.
+- **Pre-commit**: Run `uv run prek` before committing — fix any issues before creating the commit
+- **Pre-push**: Before pushing, run `uv run prek run --from-ref origin/<base>` to check all changed files against the PR base — this matches CI behavior and catches issues that per-commit prek misses. To detect the base branch: `gh pr view --json baseRefName -q .baseRefName 2>/dev/null || echo main`

.github/workflows/claude.yml

@@ -4,6 +4,17 @@ on:
   workflow_dispatch:
   pull_request:
     types: [opened, synchronize, ready_for_review, reopened]
+    paths-ignore:
+      - '.github/workflows/**'
+      - '*.md'
+      - 'docs/**'
+      - 'demos/**'
+      - 'experiments/**'
+      - 'LICENSE'
+      - '.tessl/**'
+      - 'code_to_optimize/**'
+      - 'codeflash.code-workspace'
+      - 'uv.lock'
   issue_comment:
     types: [created]
   pull_request_review_comment:
@@ -23,7 +34,7 @@ jobs:
     if: |
       (
         github.event_name == 'pull_request' &&
-        github.actor != 'claude[bot]' &&
+        github.event.sender.login != 'claude[bot]' &&
         github.event.pull_request.head.repo.full_name == github.repository
       ) ||
       github.event_name == 'workflow_dispatch'
@@ -77,6 +88,20 @@ jobs:
             Post all review findings in a single summary comment only — never as inline PR review comments.
             </commitment>
 
+            <step name="triage">
+            Before doing any work, assess the PR scope:
+
+            1. Run `gh pr diff ${{ github.event.pull_request.number }} --name-only` to get changed files.
+            2. Classify as TRIVIAL if ALL changed files are:
+               - Config/CI files (.github/, .tessl/, *.toml, *.lock, *.json, *.yml, *.yaml)
+               - Documentation (*.md, docs/)
+               - Non-production code (demos/, experiments/, code_to_optimize/)
+               - Only whitespace, formatting, or comment changes
+
+            If TRIVIAL: post a single comment "No substantive code changes to review." and stop — do not execute any further steps.
+            Otherwise: continue with the full review below.
+            </step>
+
             <step name="lint_and_typecheck">
             Run checks on files changed in this PR and auto-fix what you can.
 
@@ -167,12 +192,6 @@ jobs:
             *Last updated: <timestamp>*
             </step>
 
-            <step name="simplify">
-            Run /simplify to review recently changed code for reuse, quality, and efficiency opportunities.
-            If improvements are found, commit with "refactor: simplify <description>" and push.
-            Only make behavior-preserving changes.
-            </step>
-
             <step name="merge_optimization_prs">
             Check for open PRs from codeflash-ai[bot]:
             `gh pr list --author "codeflash-ai[bot]" --state open --json number,title,headRefName,createdAt,mergeable`
@@ -194,7 +213,7 @@ jobs:
             - All findings are in a single summary comment (no inline review comments were created)
             - If fixes were made, they were verified with prek
             </verification>
-          claude_args: '--model us.anthropic.claude-opus-4-6-v1 --max-turns 25 --allowedTools "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,Skill"'
+          claude_args: '--model us.anthropic.claude-sonnet-4-6 --allowedTools "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 pr close:*),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
 
@@ -272,6 +291,6 @@ jobs:
         uses: anthropics/claude-code-action@v1
         with:
           use_bedrock: "true"
-          claude_args: '--model us.anthropic.claude-opus-4-6-v1 --allowedTools "Read,Edit,Write,Glob,Grep,Bash(git status*),Bash(git diff*),Bash(git add *),Bash(git commit *),Bash(git push*),Bash(git log*),Bash(git merge*),Bash(git fetch*),Bash(git checkout*),Bash(git branch*),Bash(uv run prek *),Bash(prek *),Bash(uv run ruff *),Bash(uv run pytest *),Bash(uv run mypy *),Bash(uv run coverage *),Bash(gh pr comment*),Bash(gh pr view*),Bash(gh pr diff*),Bash(gh pr merge*),Bash(gh pr close*)"'
+          claude_args: '--model us.anthropic.claude-sonnet-4-6 --allowedTools "Read,Edit,Write,Glob,Grep,Bash(git status*),Bash(git diff*),Bash(git add *),Bash(git commit *),Bash(git push*),Bash(git log*),Bash(git merge*),Bash(git fetch*),Bash(git checkout*),Bash(git branch*),Bash(uv run prek *),Bash(prek *),Bash(uv run ruff *),Bash(uv run pytest *),Bash(uv run mypy *),Bash(uv run coverage *),Bash(gh pr comment*),Bash(gh pr view*),Bash(gh pr diff*),Bash(gh pr merge*),Bash(gh pr close*)"'
           additional_permissions: |
             actions: read

code_to_optimize/js/code_to_optimize_mocha/codeflash.yaml

@@ -0,0 +1,5 @@
+# Codeflash Configuration for Mocha CJS JavaScript Project
+module_root: "."
+tests_root: "tests"
+test_framework: "mocha"
+formatter_cmds: []

code_to_optimize/js/code_to_optimize_mocha/fibonacci.js

@@ -0,0 +1,60 @@
+/**
+ * Fibonacci implementations - CommonJS module
+ * Intentionally inefficient for optimization testing.
+ */
+
+/**
+ * Calculate the nth Fibonacci number using naive recursion.
+ * This is intentionally slow to demonstrate optimization potential.
+ * @param {number} n - The index of the Fibonacci number to calculate
+ * @returns {number} The nth Fibonacci number
+ */
+function fibonacci(n) {
+    if (n <= 1) {
+        return n;
+    }
+    return fibonacci(n - 1) + fibonacci(n - 2);
+}
+
+/**
+ * Check if a number is a Fibonacci number.
+ * @param {number} num - The number to check
+ * @returns {boolean} True if num is a Fibonacci number
+ */
+function isFibonacci(num) {
+    // A number is Fibonacci if one of (5*n*n + 4) or (5*n*n - 4) is a perfect square
+    const check1 = 5 * num * num + 4;
+    const check2 = 5 * num * num - 4;
+    return isPerfectSquare(check1) || isPerfectSquare(check2);
+}
+
+/**
+ * Check if a number is a perfect square.
+ * @param {number} n - The number to check
+ * @returns {boolean} True if n is a perfect square
+ */
+function isPerfectSquare(n) {
+    const sqrt = Math.sqrt(n);
+    return sqrt === Math.floor(sqrt);
+}
+
+/**
+ * Generate an array of Fibonacci numbers up to n.
+ * @param {number} n - The number of Fibonacci numbers to generate
+ * @returns {number[]} Array of Fibonacci numbers
+ */
+function fibonacciSequence(n) {
+    const result = [];
+    for (let i = 0; i < n; i++) {
+        result.push(fibonacci(i));
+    }
+    return result;
+}
+
+// CommonJS exports
+module.exports = {
+    fibonacci,
+    isFibonacci,
+    isPerfectSquare,
+    fibonacciSequence,
+};