Skip to content
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

Fix codegate version and similar commands. #1392

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

blkt
Copy link
Contributor

@blkt blkt commented May 1, 2025

While refactoring I removed three lines of coding managing
short-circuited requests. We short-circuit requests to implement
codegate version, codegate workspace, and similar commands.

Given we now have a provider-native representation of the messages, it
is necessary to produce the right message for the given request and
provider, so some code must be added in provider-specific modules to
handle that.

Fixes #1362

@blkt blkt force-pushed the fix/fix-codegate-inline-commands branch from c42c633 to 460529b Compare May 1, 2025 14:47
While refactoring I removed three lines of coding managing
short-circuited requests. We short-circuit requests to implement
`codegate version`, `codegate workspace`, and similar commands.

Given we now have a provider-native representation of the messages, it
is necessary to produce the right message for the given request and
provider, so some code must be added in provider-specific modules to
handle that.

Fixes #1362
@blkt blkt force-pushed the fix/fix-codegate-inline-commands branch from 460529b to 0fd788a Compare May 1, 2025 14:56
@blkt blkt marked this pull request as draft May 1, 2025 14:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Commands like codegate version fail to be processed
2 participants