Skip to content

Comments

chore(status):add version status#63

Merged
agustingroh merged 1 commit intomainfrom
chore/add-version-status
Feb 24, 2026
Merged

chore(status):add version status#63
agustingroh merged 1 commit intomainfrom
chore/add-version-status

Conversation

@agustingroh
Copy link
Contributor

@agustingroh agustingroh commented Feb 23, 2026

Summary by CodeRabbit

  • New Features
    • Introduced a new error code to handle scenarios where a component version is not found, expanding error handling capabilities in the API.

@coderabbitai
Copy link

coderabbitai bot commented Feb 23, 2026

Warning

Rate limit exceeded

@agustingroh has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 10 minutes and 37 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 7ab10dc and 51d48a0.

⛔ Files ignored due to path filters (1)
  • api/commonv2/scanoss-common.pb.go is excluded by !**/*.pb.go
📒 Files selected for processing (4)
  • CHANGELOG.md
  • protobuf/scanoss/api/common/v2/scanoss-common.proto
  • protobuf/scanoss/api/cryptography/v2/scanoss-cryptography.swagger.json
  • protobuf/scanoss/api/vulnerabilities/v2/scanoss-vulnerabilities.swagger.json
📝 Walkthrough

Walkthrough

These changes introduce a new VERSION_NOT_FOUND error code to the ErrorCode enum across protobuf definitions and OpenAPI/Swagger schema files, with corresponding documentation updates to the changelog.

Changes

Cohort / File(s) Summary
Changelog
CHANGELOG.md
Added 0.30.0 unreleased section documenting the introduction of the new VERSION_NOT_FOUND error code.
Protobuf Definition
protobuf/scanoss/api/common/v2/scanoss-common.proto
Extended ErrorCode enum with new member VERSION_NOT_FOUND = 4.
OpenAPI/Swagger Schemas
protobuf/scanoss/api/cryptography/v2/scanoss-cryptography.swagger.json, protobuf/scanoss/api/vulnerabilities/v2/scanoss-vulnerabilities.swagger.json
Updated v2ErrorCode enum definitions and descriptions in both schema files to include the new VERSION_NOT_FOUND error code with explanatory text.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 A version lost, but now it's found,
With error codes all gathered 'round,
The enum grows, the schemas align,
Four codes become five—how fine! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'chore(status):add version status' is somewhat vague and generic. While it mentions adding version status, it doesn't clearly convey that the change introduces a new VERSION_NOT_FOUND error code to the ErrorCode enum across multiple files and protobuf definitions. Consider a more specific title like 'chore(status): add VERSION_NOT_FOUND error code' to clearly communicate the primary change being introduced.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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 unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/add-version-status

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

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

Copy link
Contributor

@scanoss-qg scanoss-qg left a comment

Choose a reason for hiding this comment

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

LGTM

@agustingroh agustingroh force-pushed the chore/add-version-status branch from 7ab10dc to 51d48a0 Compare February 24, 2026 11:38
@agustingroh agustingroh merged commit c0d4e0a into main Feb 24, 2026
4 checks passed
@agustingroh agustingroh deleted the chore/add-version-status branch February 24, 2026 11:39
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