feat(review): Add thinking effort to code reviews#732

Merged

alex-alecu merged 32 commits intomainfrom

feat/thinking-effort-code-reviews

Mar 4, 2026

Commits on Mar 2, 2026

feat: add thinking effort dropdown to code review settings (WIP)
alex-alecu
committed
fix: add regex validation to variant schemas and improve xhigh label
alex-alecu
committed
Merge branch 'main' into feat/thinking-effort-code-reviews
alex-alecu
authored

Commits on Mar 3, 2026

Merge branch 'main' into feat/thinking-effort-code-reviews
alex-alecu
authored
Merge branch 'main' into feat/thinking-effort-code-reviews
alex-alecu
authored
fix: add regex validation to thinking_effort in CodeReviewAgentConfigSchema
alex-alecu
committed
test: add sync test for model-variants vs recommended-models
alex-alecu
committed
refactor: simplify useEffect for thinking effort reset
alex-alecu
committed
fix: validate variant before shell interpolation in streaming.ts
alex-alecu
committed
refactor: delegate getAvailableThinkingEfforts to getModelVariants, removing duplicated variant logic
alex-alecu
committed
fix: Fix duplicate detection
alex-alecu
committed
fix: Replaced the curl -f approach with curl -w '\n%{http_code}' so that when POST /session fails, the response body is captured and logged at error level. This prevents the opaque "HTTP error from…
alex-alecu
committed
fix: Fix: replace autoApproval with permission format
alex-alecu
committed
fix: Fix cli_session_id db type
alex-alecu
committed
Merge remote-tracking branch 'origin/main' into feat/thinking-effort-code-reviews
alex-alecu
committed
feat: Increase cloud-agent-next timeout to 30 minutes
alex-alecu
committed
fix: correct import path for getModelVariants (recommended-models -> model-settings)
alex-alecu
committed
refactor: replace 'as string | null' with 'satisfies' in code-reviews-router
alex-alecu
committed
refactor: replace 'as string | null' with 'satisfies' in organization-code-reviews-router
alex-alecu
committed
refactor: remove 'as Record<string, unknown>' cast for configContent.permission
alex-alecu
committed
refactor: remove redundant images/variant properties already included via spread
alex-alecu
committed
Merge remote-tracking branch 'origin/main' into feat/thinking-effort-code-reviews
alex-alecu
committed
feat: Put cloud review thinking behind feature flag
alex-alecu
committed
fix: revert autoCommit .default(false) to preserve undefined semantics
alex-alecu
committed
fix: use explicit fallback for CLI_TIMEOUT_SECONDS parsing
alex-alecu
committed
docs: add rationale comment for 30-min CLI timeout default
alex-alecu
committed
docs: clarify startup timeout is an upper bound, not a delay
alex-alecu
committed
fix: correct migration 0041 timestamp to be after 0040
alex-alecu
committed
docs: clarify permission model parity with autoApproval and glob resolution
alex-alecu
committed
fix: Fix migrations after merge
alex-alecu
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(review): Add thinking effort to code reviews#732

feat(review): Add thinking effort to code reviews#732
alex-alecu merged 32 commits intomainfrom
feat/thinking-effort-code-reviews

Commits on Mar 2, 2026

feat: add thinking effort dropdown to code review settings (WIP)

fix: add regex validation to variant schemas and improve xhigh label

Merge branch 'main' into feat/thinking-effort-code-reviews

Commits on Mar 3, 2026

Merge branch 'main' into feat/thinking-effort-code-reviews

Merge branch 'main' into feat/thinking-effort-code-reviews

fix: add regex validation to thinking_effort in CodeReviewAgentConfigSchema

test: add sync test for model-variants vs recommended-models

refactor: simplify useEffect for thinking effort reset

fix: validate variant before shell interpolation in streaming.ts

refactor: delegate getAvailableThinkingEfforts to getModelVariants, removing duplicated variant logic

fix: Fix duplicate detection

fix: Replaced the curl -f approach with curl -w '\n%{http_code}' so that when POST /session fails, the response body is captured and logged at error level. This prevents the opaque "HTTP error from…

fix: Fix: replace autoApproval with permission format

fix: Fix cli_session_id db type

Merge remote-tracking branch 'origin/main' into feat/thinking-effort-code-reviews

feat: Increase cloud-agent-next timeout to 30 minutes

fix: correct import path for getModelVariants (recommended-models -> model-settings)

refactor: replace 'as string | null' with 'satisfies' in code-reviews-router

refactor: replace 'as string | null' with 'satisfies' in organization-code-reviews-router

refactor: remove 'as Record<string, unknown>' cast for configContent.permission

refactor: remove redundant images/variant properties already included via spread

Merge remote-tracking branch 'origin/main' into feat/thinking-effort-code-reviews

feat: Put cloud review thinking behind feature flag

fix: revert autoCommit .default(false) to preserve undefined semantics

fix: use explicit fallback for CLI_TIMEOUT_SECONDS parsing

docs: add rationale comment for 30-min CLI timeout default

docs: clarify startup timeout is an upper bound, not a delay

fix: correct migration 0041 timestamp to be after 0040

docs: clarify permission model parity with autoApproval and glob resolution

fix: Fix migrations after merge

Commits on Mar 4, 2026

Merge remote-tracking branch 'origin/main' into feat/thinking-effort-code-reviews

refactor: simplify V2 timeout — use DEFAULT_INFLIGHT_TIMEOUT_MS directly