Skip to content

fix: support for WORKSPACE_PREFIX env var - #20

Merged
NachoCamposMarti merged 1 commit into
mainfrom
ASPMAIN-5246_workspace-prefix-fix
Jun 17, 2026
Merged

fix: support for WORKSPACE_PREFIX env var#20
NachoCamposMarti merged 1 commit into
mainfrom
ASPMAIN-5246_workspace-prefix-fix

Conversation

@NachoCamposMarti

Copy link
Copy Markdown
Contributor

No description provided.

@NachoCamposMarti
NachoCamposMarti merged commit 81fd5ad into main Jun 17, 2026
1 check passed

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request refactors the TradingView-specific Grype OCI label transformation logic by moving it out of the core GrypeTranslator and into a modular client extension structure (client_extensions/tradingview). The core GrypeTranslator now exposes a generic label_value_transforms property, and the phoenix_multi_scanner_enhanced script dynamically applies the TradingView transforms if requested. Additionally, the required WORKSPACE_PREFIX environment variable configuration has been added to the Docker Compose and example environment files. Feedback was provided regarding the type check in apply_tv_tags_to_grype_translator, suggesting the use of isinstance instead of a string-based class name comparison to make the check more robust.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant