Skip to content

initial commit#63

Merged
boyney123 merged 2 commits intomainfrom
schema-update-3
Jun 4, 2025
Merged

initial commit#63
boyney123 merged 2 commits intomainfrom
schema-update-3

Conversation

@boyney123
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 3, 2025

Invalid license key for EventCatalog GitHub Action, please check your license key or get a 14 day free trial at https://eventcatalog.cloud/.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 3, 2025

EventCatalog: Schema Review

The following schemas were modified in this pull request:

Order amended (0.0.1) (🚨 Danger)

Schema Format Risk Score
JSONDraft7 30/100

The new schema removes the “totalPrice” property (and omits definitions for productId, productName, newQuantity) from amendedItems, making it backward-incompatible for existing consumers. Additionally, the event lacks a version field, violating EDA best practices. This change poses immediate risks for downstream validation and processing.

Potential Effected Consumers

  • Inventory Service (0.0.2) - This schema change removes the required “totalPrice” field (and omits productId, productName, newQuantity definitions) from amendedItems. Your service’s JSON validation and any price-related calculations will fail. You must coordinate with the producer to either reintroduce these fields in a new versioned event or update your consumer logic to handle the missing properties.

Using Model: o4-mini | Provider: openai

@boyney123 boyney123 merged commit 8ba80f0 into main Jun 4, 2025
1 check failed
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.

1 participant