Skip to content

fix: align MAX_TURNS with PR description (50→200)#113

Closed
claude-claude[bot] wants to merge 1 commit intoclaude-increase-max-turnsfrom
claude/fix-20727221987
Closed

fix: align MAX_TURNS with PR description (50→200)#113
claude-claude[bot] wants to merge 1 commit intoclaude-increase-max-turnsfrom
claude/fix-20727221987

Conversation

@claude-claude
Copy link
Copy Markdown
Contributor

@claude-claude claude-claude bot commented Jan 5, 2026

Auto-Fix for PR #112

Issues Fixed

  • [CRITICAL] Fixed mismatch between PR metadata and actual code implementation
    • PR Increase MAX_TURNS from 50 to 75 #112 title and description claim to increase MAX_TURNS from 50 to 200
    • However, the code only changed it from 50 to 75
    • This fix updates the code to match the documented intent: 200

Changes

  • Changed MAX_TURNS default value from 75 to 200 in scripts/claude-assistant/index.ts
  • This aligns the implementation with the PR title, description, and stated goal of allowing thorough reviews of large PRs

Rationale

I chose to update the code (75→200) rather than the PR metadata because:

  1. Both the PR title AND description consistently state 200
  2. The value 200 better aligns with the stated goal: "allow thorough reviews of large PRs"
  3. The commit message appears to be the outlier/typo

Generated by Claude | Review Run

The PR title and description claim to increase MAX_TURNS from 50 to 200,
but the code only changed it to 75. This commit updates the code to match
the documented intent of 200.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@ejc3 ejc3 closed this Jan 5, 2026
@ejc3 ejc3 deleted the claude/fix-20727221987 branch January 17, 2026 03:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant