Skip to content

chore(components):SP-4329 replace error_message/error_code witn info_…#77

Merged
agustingroh merged 1 commit intomainfrom
chore/SP-4329-replace-error-message-by-info-message
Apr 20, 2026
Merged

chore(components):SP-4329 replace error_message/error_code witn info_…#77
agustingroh merged 1 commit intomainfrom
chore/SP-4329-replace-error-message-by-info-message

Conversation

@agustingroh
Copy link
Copy Markdown
Contributor

@agustingroh agustingroh commented Apr 20, 2026

…massage/info_code in component responses

Summary by CodeRabbit

  • Breaking Changes
    • ComponentStatusResponse now uses info_message and info_code as the definitive outcome fields. The previous error_message and error_code fields have been removed. Clients must migrate their implementations to use the new field names for both version and component status responses to continue receiving status information.

@agustingroh agustingroh requested a review from scanoss-qg April 20, 2026 17:11
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f9f576be-250e-4601-b3b7-fd73573e5b65

📥 Commits

Reviewing files that changed from the base of the PR and between 511e37c and 0a6bff9.

⛔ Files ignored due to path filters (1)
  • api/componentsv2/scanoss-components.pb.go is excluded by !**/*.pb.go
📒 Files selected for processing (3)
  • CHANGELOG.md
  • protobuf/scanoss/api/components/v2/scanoss-components.proto
  • protobuf/scanoss/api/components/v2/scanoss-components.swagger.json

📝 Walkthrough

Walkthrough

This PR documents and implements a breaking change for version 0.42.0, replacing error_message/error_code fields with info_message/info_code in ComponentStatusResponse messages across Protobuf definitions, OpenAPI schema, and documentation.

Changes

Cohort / File(s) Summary
Documentation
CHANGELOG.md
Added release notes for version 0.42.0 documenting the breaking change: removal of error_message/error_code fields and introduction of info_message/info_code fields in ComponentStatusResponse messages.
Protobuf Definitions
protobuf/scanoss/api/components/v2/scanoss-components.proto
Replaced error_message and error_code (fields 6, 7) with info_message and info_code (fields 8, 9) in both ComponentStatusResponse.VersionStatus and ComponentStatusResponse.ComponentStatus. Old field numbers and names marked as reserved. Documented possible info_code values for each message context.
OpenAPI Schema
protobuf/scanoss/api/components/v2/scanoss-components.swagger.json
Updated schema definitions for ComponentStatusResponseComponentStatus and ComponentStatusResponseVersionStatus to replace error_message/error_code with info_message/info_code. Removed v2ErrorCode definition reference; info_code now uses inline string values with operation-specific enumerations.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • scanoss-qg
  • isasmendiagus
  • eeisegn

Poem

🐰 Field numbers dance in reservation's grace,
Old names retired to memory's space,
Info arrives where error once stood,
Messages evolve as schemas should,
Breaking change, forward we go! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title is truncated and contains a typo ('witn' instead of 'with'), making it unclear and incomplete. Complete the truncated title and fix the typo. Suggested: 'chore(components): SP-4329 replace error_message/error_code with info_message/info_code'.
✅ 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
  • Commit unit tests in branch chore/SP-4329-replace-error-message-by-info-message

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
Copy Markdown
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/SP-4329-replace-error-message-by-info-message branch from c65aa5f to 0a6bff9 Compare April 20, 2026 17:32
@agustingroh agustingroh merged commit d1476be into main Apr 20, 2026
4 checks passed
@agustingroh agustingroh deleted the chore/SP-4329-replace-error-message-by-info-message branch April 20, 2026 17:38
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.

2 participants