event-catalog / event-catalog/generators

Remove x-parser-schema-id from generated schemas

Open
#14 0 comments 1 reaction 0 assignees View on GitHub
generator-asyncapi
Dominant language
TypeScript
Stars
15
Forks
16
Avg merge
25m
Merged PRs (30d)
4

Description

### Use Case

We are trying to generate our schemas and service definitions from AsyncAPI, and it works pretty well. But whenever we run the generator, it updates _ALL_ the generated schema files that have nested schemas, with a new `x-parser-schema-id`.

As a result, the commit diff ends up showing nearly all our schemas changing, but the actual change is something like this, which is just not useful.

![image](https://github.com/user-attachments/assets/0b585d36-77fa-44d4-bb1e-b9638c8f7ff5)

### Proposed Solution

Is the x-parser-schema-id really needed in this? Or is it an implementation detail of how the parser works?

If its the latter, it would be great if we could just drop that property.

Or could try to keep it consistent and only update when necessary.

My goal is to just minimize the git diff.

### Implementation Notes

_No response_

### Community Notes

* Please vote by adding a 👍 reaction to the issue to help us prioritize.
* If you are interested to work on this issue, please leave a comment.
* If this issue is labeled **needs-discussion**, it means the spec has not been finalized yet. Please reach out in the [EventCatalog Discord](https://discord.com/invite/3rjaZMmrAm).

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.