Skip to content

fix: report LLM failures and preserve complete history - #5

Merged
justgithubaccount merged 2 commits into
mainfrom
fix/llm-errors-and-history
Oct 29, 2026
Merged

justgithubaccount merged 2 commits into
mainfrom
fix/llm-errors-and-history

Conversation

@justgithubaccount

Copy link
Copy Markdown
Owner

What changed

  • return 502 Bad Gateway for explicit and malformed LLM provider failures
  • persist project history only after a successful completion, including the assistant reply
  • keep the production container startup fully offline by invoking the installed uvicorn binary
  • add regression and container-runtime checks to CI
  • document the corrected API behavior

Verification

  • uv run ruff check .
  • uv run mypy app
  • uv run pytest --cov=app --cov-report=term-missing -q — 22 passed
  • docker build -t app-poly-gitops-fastapi:test .
  • bash tests/test_container_runtime.sh app-poly-gitops-fastapi:test — /health passed with --network none

Independent exact-head review: approved for operator merge at 370a5a9f82e03a5ddc9b79d26b303a8cb1173044.

@justgithubaccount
justgithubaccount merged commit ab70440 into main Oct 29, 2026
2 checks 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