Skip to content

Conversation

@daverod24
Copy link

Description

This PR enhances the LLM provider configuration and project documentation, including:

  • Updated dependencies for Google and Groq support
  • Detailed Docker and LLM configuration documentation
  • Updated company ticker map
  • General project documentation improvements

Key Changes

  • Added @ai-sdk/google and @ai-sdk/groq as dependencies
  • Updated @types/node, @types/react, and @types/react-dom versions
  • Improved README.md documentation
  • Added Docker setup instructions

Impact

These changes improve the project infrastructure and make it easier to configure and use different LLM providers, while providing better documentation for future developers.

David Rodriguez added 2 commits June 27, 2025 19:21
- Added @ai-sdk/google and @ai-sdk/groq dependencies.
- Updated @types/node, @types/react, and @types/react-dom to specific patch versions.
…E; create company ticker map documentation
Copy link

@anthonywu anthonywu left a comment

Choose a reason for hiding this comment

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

Fellow PR contributor here. I submitted #10 and going into it I was hoping someone would create a provider pattern like this PR is trying to do

The spanish in company-ticket-map.md seems like a vibe coded thing. It's inconsistent with the rest of the English docs.

The CSS changes are distracting from the main goal of the LLM provider change and I think should be scoped out of this change.

The addition of crypto tickers... should also be its own proposal/PR.

Just my two cents.

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