feat: add Codex AI provider and refactor shared provider logic#121
Open
Savid wants to merge 2 commits into
Open
feat: add Codex AI provider and refactor shared provider logic#121Savid wants to merge 2 commits into
Savid wants to merge 2 commits into
Conversation
Add a new Codex AI provider backed by codex-agent-sdk-go, registered alongside the existing Claude provider in the factory. Extract shared streaming state, error types, stream event parsing, and tool summary formatting into provider_shared.go so both providers reuse common logic. Update the Dashboard to persist the selected AI provider in localStorage and restart active sessions on provider change.
mattevans
approved these changes
Feb 26, 2026
* master: chore(deps): bump the go-dependencies group across 1 directory with 8 updates chore(deps): bump the actions group across 1 directory with 4 updates feat(clickhouse): wire external Xatu credentials through init and infra (#131) chore(dependencies): update tool versions, Go modules, and dependencies for improved functionality and security refactor(seeddata): remove unused 'spec' variable from multiple functions and structures to simplify code and enhance clarity refactor(commands): remove 'spec' command flag and related logic from transformation generation commands to streamline user experience and focus on essential parameters refactor(api_stack.go): simplify switch statement for stack status handling to improve readability and maintainability docs(assertions.go): update assertion instructions with ClickHouse SQL rules for better clarity and guidance feat(assertions.go): add new assertions for validating row count and negative gas values in SQL queries feat(discovery): integrate AI engine into range detection and assertion generation processes to improve accuracy and flexibility
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Add a new Codex AI provider backed by codex-agent-sdk-go, registered alongside the existing Claude provider in the factory. Extract shared streaming state, error types, stream event parsing, and tool summary formatting into provider_shared.go so both providers reuse common logic.
Update the Dashboard to persist the selected AI provider in localStorage and restart active sessions on provider change.
might need to run
sorry not sorry