Skip to content

fix(sbom): move non-eula packages to cpeMappings - #35793

Open
IlyaKhD wants to merge 3 commits into
DevExpress:mainfrom
IlyaKhD:khd/update-sbom-package-mappings
Open

IlyaKhD wants to merge 3 commits into
DevExpress:mainfrom
IlyaKhD:khd/update-sbom-package-mappings

Conversation

@IlyaKhD

@IlyaKhD IlyaKhD commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

@IlyaKhD IlyaKhD self-assigned this Oct 20, 2026
Copilot AI balanced review requested due to automatic review settings October 20, 2026 12:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused changes align package classifications with licensing and preserve the existing internal-package list.

No open issues 🎉

What changed in this PR

Updates SBOM package classification to separate non-EULA framework wrappers and reuse the internal-package registry.

Changes:

  • Moves Angular, React, and Vue wrappers to CPE mappings and adds devextreme-dist to the EULA package list.
  • Derives internal package names from the shared registry.
File Description
packages/​sbom/​scripts/​make.ts Updates classifications and reuses internal package names.
packages/​sbom/​scripts/​add-workspace-hashes.ts Exports the internal-package registry.

Review effort: Balanced

Copilot AI balanced review requested due to automatic review settings October 20, 2026 12:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused configuration changes align package classifications with licensing and preserve existing internal-package handling.

No open issues 🎉

Review effort: Balanced

This branch has not been deployed

No deployments
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.

3 participants