Skip to content

Conversation

@piiq
Copy link
Owner

@piiq piiq commented Sep 15, 2025

Additionally this PR bumps openbb-ai to the latest version to resolve citations not rendering correctly

@piiq piiq requested a review from Copilot September 15, 2025 14:25
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds configuration validation on application startup to ensure all required external services are properly configured before the app begins serving requests. Additionally, it bumps the openbb-ai dependency from version 1.5.0 to 1.7.4 to resolve citation rendering issues.

Key changes:

  • Added comprehensive validation module to check OpenRouter, S3/local storage, and FMP API credentials
  • Integrated startup validation into the FastAPI application lifecycle
  • Updated openbb-ai dependency to latest version

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pyproject.toml Added aiohttp dependency and bumped openbb-ai from 1.5.0 to 1.7.4
allocator_bot/validation.py New module providing validation functions for external service credentials
allocator_bot/utils.py Added blank line for improved formatting
allocator_bot/api.py Integrated startup validation using FastAPI lifespan context manager
.env.example Added VALIDATION_SKIP environment variable for development bypass

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@piiq piiq merged commit 2a096c7 into main Sep 15, 2025
2 checks passed
@piiq piiq deleted the feature/startup-validation branch September 15, 2025 14:28
@piiq piiq mentioned this pull request Sep 16, 2025
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.

2 participants