chore: address CodeRabbit and PR #27 review feedback by sheepdestroyer · Pull Request #28 · sheepdestroyer/LLM-Routing

Configure gated Ollama routing and set llm-routing-ollama as free tier fallback

sheepdestroyer

committed

Address code review comments: Fix annotations race condition and handle null content safely

sheepdestroyer

committed

Address new code reviews: Break Ollama fallback loop, use env key, and fix SAST/lint comments

sheepdestroyer

committed

fix: sanitize triage router 429 detail and force immediate LiteLLM cooldown for llm-routing-ollama on rate limit

sheepdestroyer

committed

docs: update fallback diagrams and cooldown behavior for Ollama models

sheepdestroyer

committed

fix: implement router-side Ollama cooldown to prevent crashloop

sheepdestroyer

committed

chore: tidy up repository, remove hello world dummies, move and document verification scripts

sheepdestroyer

committed

fix: resolve hardcoded worktree leaks and LiteLLM auth errors

sheepdestroyer

committed

Address PR comments: robust httpx client teardown, dynamic path escaping in sed, expected model asserts in tests, synced metrics documentation

sheepdestroyer

committed

Implement Valkey global cooldown cache sync, standard HTTPX client lifecycles, and handle transient exception status codes

sheepdestroyer

committed

Address PR#11 code reviews: multimodal message extraction, concurrent breaker Valkey calls, generic error detail messages, SSE stream token counting, secure YAML rendering, and robust verification …

sheepdestroyer

committed

Address PR #12 code review comments

sheepdestroyer

committed

feat: expose model capabilities, token limits, and costs in Model Hub Table

sheepdestroyer

committed

chore: address review feedback on PR #14 (DRY purge helper, safety-net capabilities, align returned context lengths, and refactor verify scripts to httpx)

sheepdestroyer

committed

chore: address PR #15 code review fixes and fix CI workflow triggers

sheepdestroyer

committed

chore(deps): adjust dependabot root docker update time to 02:55 UTC (04:55 local)

sheepdestroyer

committed

chore: address code review feedback (YAML list indentation, raise_for_status in metrics fetch, detailed HTTP routing diagnostics, and defensive config type checking)

sheepdestroyer

committed

chore: trigger CI

sheepdestroyer

committed

Address code reviews for PR #25

sheepdestroyer

committed

Address new PR reviews and CodeRabbit feedback

sheepdestroyer

committed

Address Gemini Code Assist review feedback on null text handling and empty choices fallback

sheepdestroyer

committed

chore: address PR review feedback on DATABASE_URL, fallback password, and max_tokens clamping

sheepdestroyer

committed

chore: address CodeRabbit and PR #27 review feedback

sheepdestroyer

committed

chore: address Gemini Code Assist review feedback on PR #28

sheepdestroyer

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: address CodeRabbit and PR #27 review feedback#28

chore: address CodeRabbit and PR #27 review feedback#28
sheepdestroyer wants to merge 24 commits into
masterfrom
feat/pr27-review-fixes

Commits on Jun 20, 2026

Configure gated Ollama routing and set llm-routing-ollama as free tier fallback

Address code review comments: Fix annotations race condition and handle null content safely

Address new code reviews: Break Ollama fallback loop, use env key, and fix SAST/lint comments

fix: sanitize triage router 429 detail and force immediate LiteLLM cooldown for llm-routing-ollama on rate limit

docs: update fallback diagrams and cooldown behavior for Ollama models

fix: implement router-side Ollama cooldown to prevent crashloop

chore: tidy up repository, remove hello world dummies, move and document verification scripts

fix: resolve hardcoded worktree leaks and LiteLLM auth errors

Address PR comments: robust httpx client teardown, dynamic path escaping in sed, expected model asserts in tests, synced metrics documentation

Implement Valkey global cooldown cache sync, standard HTTPX client lifecycles, and handle transient exception status codes

Address PR#11 code reviews: multimodal message extraction, concurrent breaker Valkey calls, generic error detail messages, SSE stream token counting, secure YAML rendering, and robust verification …

Address PR #12 code review comments

feat: expose model capabilities, token limits, and costs in Model Hub Table

chore: address review feedback on PR #14 (DRY purge helper, safety-net capabilities, align returned context lengths, and refactor verify scripts to httpx)

chore: address PR #15 code review fixes and fix CI workflow triggers

chore(deps): adjust dependabot root docker update time to 02:55 UTC (04:55 local)

chore: address code review feedback (YAML list indentation, raise_for_status in metrics fetch, detailed HTTP routing diagnostics, and defensive config type checking)

chore: trigger CI

Address code reviews for PR #25

Address new PR reviews and CodeRabbit feedback

Address Gemini Code Assist review feedback on null text handling and empty choices fallback

chore: address PR review feedback on DATABASE_URL, fallback password, and max_tokens clamping

chore: address CodeRabbit and PR #27 review feedback

chore: address Gemini Code Assist review feedback on PR #28