Skip to content

feat: Chocolatey Package Identification in Discovery step#329

Closed
sumanthkb44 wants to merge 158 commits intofeature/choco_logging_for_devfrom
feature/ChocoPackages_Discovery
Closed

feat: Chocolatey Package Identification in Discovery step#329
sumanthkb44 wants to merge 158 commits intofeature/choco_logging_for_devfrom
feature/ChocoPackages_Discovery

Conversation

@sumanthkb44
Copy link
Collaborator

Changes

  1. Modified the package identifier logic to specifically target and parse choco.config/*.choco.config files for Chocolatey dependency information.
  2. Implemented filtering to ensure only direct dependencies explicitly listed in input files are discovered and reported.

This commit introduces support for Choco package management in the application. Key changes include:

- Added a `Choco` property to `CommonAppSettings.cs` for configuration.
- Updated project type mappings in `CommonHelper.cs` to recognize "CHOCO".
- Enhanced `BomCreator.cs` to handle the "CHOCO" project type.
- Modified `DisplayInformation.cs` to include "CHOCO" in include/exclude mappings.
- Updated `NugetProcessor.cs` to identify internal components for Choco packages.
- Introduced `ChocoProcessor.cs` to parse Choco configuration files.

These changes integrate Choco management seamlessly with existing functionalities.
@sumanthkb44 sumanthkb44 changed the base branch from main to development September 3, 2025 04:54
@sumanthkb44 sumanthkb44 changed the title feat : Chocolatey Package Identification in Discovery step feat: Chocolatey Package Identification in Discovery step Sep 3, 2025
K B, Sumanth (FT D AA IN SGI EA EBT BE) added 3 commits September 3, 2025 19:49
Updated `AddSiemensDirectProperty` method to be instance-based in `NugetParserTests.cs` for better encapsulation. Introduced a new method in `ChocoProcessor.cs` to add SiemensDirect properties to BOM components, including handling for null properties and duplicate entries. Changed the method in `NugetProcessor.cs` from static to virtual to allow for overriding in derived classes.
Modified the `CommonIdentiferHelper` class to append the "_Bom.cdx.json" suffix to the BOM file name. Updated the corresponding unit test to assert the new file name format, ensuring consistency in the expected output.
NullReferenceExceptions when checking the project type.
@adityanarayanp adityanarayanp added enhancement New feature or request work in progress started working on the issue and is in progress Importance Low Minor Impact labels Sep 7, 2025
@adityanarayanp adityanarayanp added this to the v8.3.0 milestone Sep 7, 2025
HARSHITA Singh and others added 18 commits September 10, 2025 12:13
feat: Choco packages testcase updated for Choco.
Modified the `ChocoProcessor` constructor to include an additional `null` parameter in the call to the base class `NugetProcessor`. The `ISpdxBomParser spdxBomParser` parameter remains unchanged, reflecting an update in the expected parameters for the base class.
@crvreddy
Copy link
Collaborator

crvreddy commented Feb 5, 2026

This PR is not required

@crvreddy crvreddy closed this Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants