Add feature flags for Pinecone, Cloudinary, and GitLab OAuth detectors

[camgunz]

Description:

Puts the Pinecone, Cloudinary, and GitLab OAuth detectors behind feature flags.

Checklist:

• Tests passing (make test-community)?
• Lint passing (make lint this requires golangci-lint)?

[amanfcp]

Just wondering if it would be better to add Enabled suffix since this is a bool

[camgunz]

Sure!

[amanfcp]

Just trying to understand the motivation: what prompted flagging these three specifically?

Another thing from OSS perspective is that since atomic.Bool defaults to false and main.go does not .Store(true) these (the way it does for EnableAPKHandler, UseGitMirror, etc.), OSS users on main will silently lose these three detectors with no CLI way to turn them back on. Might be worth a release-note callout, or wiring a flag.

[camgunz]

Another thing from OSS perspective is that since atomic.Bool defaults to false and main.go does not .Store(true) these
Ooh I did forget that; will work it up, thanks

[amanfcp]

LGTM. We just need to find a way to exclude them from the test that is failing

[mustansir14]

LGTM. For the failing test, you will need to add these detectors to excludedFromDefaultList in the test, ideally in a separate section with a comment explaining why it was added.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit e59c77d. Configure here.}

[camgunz]

Yeah I wanted to try and minimize the number of changes needed to feature flag a detector, but I think this is what we have to do for now.