chore(import):SP-4180 remove mandatory fields form scan result validator

[agustingroh]

Summary by CodeRabbit

• Changed
  • Scan result validation requires fewer mandatory fields: release_date, url, and url_hash are no longer treated as required non-empty strings. Core validations (component, file, hashes, licenses, purl and required arrays) remain enforced.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b5926986-3e78-4b59-8237-4bf9de49fdab

📥 Commits

Reviewing files that changed from the base of the PR and between 871b96b and e2c8f87.

📒 Files selected for processing (2)

• CHANGELOG.md
• src/main/modules/validator/ScanossResultValidator.ts

💤 Files with no reviewable changes (1)

• src/main/modules/validator/ScanossResultValidator.ts

✅ Files skipped from review due to trivial changes (1)

• CHANGELOG.md

---

📝 Walkthrough

Walkthrough

Validator and changelog updated: release_date, url, and url_hash are no longer enforced as required non-empty string fields in scan file result validation; CHANGELOG.md records this unreleased change.

Changes

Cohort / File(s)	Summary
Validator change src/main/modules/validator/ScanossResultValidator.ts	Removed release_date, url, and url_hash from the set of required string fields in validateFileResult(). Their absence or non-string values no longer produce validation errors; other required fields unchanged.
Changelog update CHANGELOG.md	Added an Unreleased entry noting that release_date, url, and url_hash are no longer mandatory string fields for scan result validation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 I hop through lines of code at night,

I tuck three keys out of the light,
A lighter check, a softer nudge,
The changelog whispers, "No more judge",
I bounce along — small changes, bright.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly summarizes the main change: removing mandatory fields from the scan result validator, which aligns with the modifications in both CHANGELOG.md and ScanossResultValidator.ts.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/SP-4180-remove-required-fields-from-result-validator

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}