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

Conversation

@therealnb
Copy link
Contributor

@therealnb therealnb commented Feb 21, 2025

This re-enables suspicious commands. Closes #1038

The main difference is in these lines https://github.com/stacklok/codegate/pull/1151/files#diff-0658dbd5db53af1b59ad53ac8ce9dc0ba40042abea62cce7f6b04eebf3fa4cffR56-R59

It should no longer mess with edits mode on copilot. Also, it should no longer apply if the language is any of

        "python",
        "javascript",
        "typescript",
        "go",
        "rust",
        "java",

I see the secrets warning occasionally, as per #1038, but it is not going through the suspicious commands code.

I tested this quite a lot on co-pilot, but not the other assistants. The accuracy for unfamiliar commands needs improvement.

@yrobla yrobla merged commit e3f9e9a into main Feb 24, 2025
11 checks passed
@yrobla yrobla deleted the codegate-1038 branch February 24, 2025 09:45
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.

Suspicious commands are routing via secrets pipeline

3 participants