Skip to content

change default values, improve prompt building#13

Merged
radetsky merged 1 commit into
mainfrom
fixing/documentation_and_parameters
Mar 27, 2026
Merged

change default values, improve prompt building#13
radetsky merged 1 commit into
mainfrom
fixing/documentation_and_parameters

Conversation

@radetsky

Copy link
Copy Markdown
Owner

Added

  • max_tokens input in GitHub Action to override max tokens per LLM request per workflow run.
  • timeout input in GitHub Action to override LLM request timeout per workflow run.
  • LLM_MAX_RESPONSE_TOKENS environment variable to control max tokens in LLM response (default: 16384).
  • llm.max_response_tokens config key (default: 16384).

Changed

  • Default llm.timeout increased from 180 to 600 seconds to accommodate slow providers and large diffs with chunking.
  • Default max_tokens_per_request increased to 32768 (see v0.1.3 notes).
  • LLM requests now use system + user message roles instead of a single user message, giving reviewer instructions higher priority in the model's context window.
  • max_tokens is now explicitly passed in every LLM API call, preventing unbounded response generation.

@github-actions

Copy link
Copy Markdown

⚠️ LLM Code Review

Status: Model Unavailable (static analysis used)

Category Count
Critical Issues 0
Warnings 1
Suggestions 0

Warnings

  • ⚠️ 🔴 LLM model unavailable: Error code: 401 - {'detail': 'Your session has expired or the token is invalid. Please sign in again.'}

Generated by LLM Code Review

@radetsky radetsky merged commit fb84022 into main Mar 27, 2026
1 check passed
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