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

CodeGate seems to be raising more alerts than expected #875

Closed
wright-io opened this issue Jan 31, 2025 · 1 comment · Fixed by #989
Closed

CodeGate seems to be raising more alerts than expected #875

wright-io opened this issue Jan 31, 2025 · 1 comment · Fixed by #989
Assignees
Labels

Comments

@wright-io
Copy link
Contributor

Describe the issue

The latest release of CodeGate seems to be raising more alerts than expected . When the copliot response says there are 2 secrets protected, 6 are showing up in the dashboard (including firing chrome alerts, if configured).

Steps to Reproduce

Image

Operating System

MacOS (Arm)

IDE and Version

Extension and Version

Provider

OpenAI

Model

GPT-4o

Codegate version

v0.1.15

Logs

No response

Additional Context

No response

@yrobla
Copy link
Contributor

yrobla commented Feb 7, 2025

that is because actually github copilot generates internal messages , splitting the request in two. I added some logic to group those requests in one, i will do the same for alerts

yrobla added a commit that referenced this issue Feb 7, 2025
Sometimes the different client tools generate multiple requests when
the user requests a task. This generates what it looks as a duplicate
alert, but is not really a total duplicate as it belongs to different
request.
But for the user does not provide so much value to have it, so proceed
with deduplicating those alerts, based on the code snippet and details
of the alert

Also remove dogecoin regex as it's giving false positives

Closes: #875
aponcedeleonch pushed a commit that referenced this issue Feb 12, 2025
Sometimes the different client tools generate multiple requests when
the user requests a task. This generates what it looks as a duplicate
alert, but is not really a total duplicate as it belongs to different
request.
But for the user does not provide so much value to have it, so proceed
with deduplicating those alerts, based on the code snippet and details
of the alert

Also remove dogecoin regex as it's giving false positives

Closes: #875
aponcedeleonch added a commit that referenced this issue Feb 12, 2025
* feat: remove duplicated alerts

Sometimes the different client tools generate multiple requests when
the user requests a task. This generates what it looks as a duplicate
alert, but is not really a total duplicate as it belongs to different
request.
But for the user does not provide so much value to have it, so proceed
with deduplicating those alerts, based on the code snippet and details
of the alert

Also remove dogecoin regex as it's giving false positives

Closes: #875

* Moved function to remove duplicate alerts to v1_processing

* Restore dogecoin

---------

Co-authored-by: Alejandro Ponce <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants