Add Yahoo Finance integration as free data provider option #424
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This PR introduces yfinance as a free alternative to the existing Financial Datasets API, making the platform more accessible to users who don't have premium API access. ## Key Changes ### Data Provider Infrastructure - Add `src/data/providers.py` with configurable data source management - Add `src/utils/financial_data.py` for safe financial data access utilities - Integrate data provider selection into CLI workflow ### Yahoo Finance Integration - Add yfinance dependency to pyproject.toml - Implement `get_prices_yfinance()` function in api.py - Add fallback logic for price data fetching - Update all 16 investment analyst agents to support dynamic data provider selection ### Enhanced Reliability - Add safe field access utilities for financial data - Implement proper error handling for missing/invalid data - Add working capital and shareholders equity calculation helpers - Validate required fields before processing ### CLI Improvements - Add data provider selection to main CLI workflow - Default to Yahoo Finance as free option - Maintain backward compatibility with existing Financial Datasets API 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Use getattr() to safely access dividends_and_other_cash_distributions attribute to prevent AttributeError when the attribute doesn't exist on financial line items. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR introduces yfinance as a free alternative to the existing Financial Datasets API, making the platform more accessible to users who don't have premium API access.
Key Changes
Data Provider Infrastructure
src/data/providers.pywith configurable data source managementsrc/utils/financial_data.pyfor safe financial data access utilitiesYahoo Finance Integration
get_prices_yfinance()function in api.pyEnhanced Reliability
CLI Improvements
🤖 Generated with Claude Code