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

[Bug] Commands like codegate version fail to be processed #1362

Open
blkt opened this issue Apr 22, 2025 · 0 comments · May be fixed by #1392
Open

[Bug] Commands like codegate version fail to be processed #1362

blkt opened this issue Apr 22, 2025 · 0 comments · May be fixed by #1392
Labels
bug

Comments

@blkt
Copy link
Contributor

blkt commented Apr 22, 2025

Describe the issue

Commands like codegate version are forwarded to the LLM rather than bypassing it.

The root cause is that the code implementing the short-circuit logic was removed during refactoring and never added back again. Note that the linked code would not work as-is and must be rewritten to behave in the same way.

Steps to Reproduce

Using any assistant, open a chat window and enter codegate version.

Operating System

Other

IDE and Version

Any

Extension and Version

Any

Provider

Other

Model

Any

Codegate version

9c51b4d0134a

Logs

No response

Additional Context

No response
blkt added a commit that referenced this issue Apr 29, 2025
@blkt
Fix codegate version and similar commands.
c42c633 
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 linked a pull request May 1, 2025 that will close this issue
Draft
blkt added a commit that referenced this issue May 1, 2025
@blkt
Fix codegate version and similar commands.
460529b 
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 added a commit that referenced this issue May 1, 2025
@blkt
Fix codegate version and similar commands.
0fd788a 
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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix codegate version and similar commands. stacklok/codegate
2 participants
@blkt @jhrozek