GPT-5 models do not support custom temperature or the system role,
similar to reasoning models. Unified detection via _is_restricted_model()
for all models requiring max_completion_tokens and developer role.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
fix: use max_completion_tokens for GPT-5 (replaces deprecated max_tokens)
- OpenAI models: gpt-5 and gpt-5-mini only (removed GPT-4 series)
- Default max_tokens raised from 2048 to 8192 across all providers
- Added o4-mini to reasoning model detection
- Updated setup wizard fallback lists and defaults
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
fix: verification prompt now allows SQL data summarization across rows
The verifier was rejecting valid responses for SQL-heavy queries because
check #2 demanded an EXACT data point match for each claim. When the LLM
summarizes across 49 SQL rows (e.g., "scores ranged from 3 to 5"), no
single row contains that statement, causing repeated verification failure
and strict-mode refusal.
Updated verification checklist:
- Check #2: SQL rows may be legitimately summarized; verify underlying
data points exist across rows, not that one row matches verbatim
- Check #10: SQL citations may cover an entire dataset/table
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- H1: validate_citations regex now matches References:/Reference/Sources (Primary)
- H2: _strip_sql_comments rewritten as state machine, preserves --//**/ inside strings
- H3: SQL citation count fixed: 1 source per query, not 1 per row
- M1: Context budget now accounts for web context size before local allocation
- M2: max_context_chars default aligned to 12000 (was 0 in code vs 12000 in docs)
10 agents dispatched: 5 audit + 3 fix + 2 cross-check. 175/175 tests passing.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>