Skip to content

CEXT-6138: SDK Capability Discovery#427

Open
iivvaannxx wants to merge 2 commits into
mainfrom
poc/openapi-generation-router
Open

CEXT-6138: SDK Capability Discovery#427
iivvaannxx wants to merge 2 commits into
mainfrom
poc/openapi-generation-router

Conversation

@iivvaannxx
Copy link
Copy Markdown
Collaborator

@iivvaannxx iivvaannxx commented May 6, 2026

No description provided.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 6, 2026 •
edited
Loading

⚠️ No Changeset found

Latest commit: 1bedecb

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions Bot added without-changeset The PR does not contain a Changeset file pkg: aio-commerce-lib-core Includes changes in `packages/aio-commerce-lib-core` pkg: aio-commerce-lib-app Includes changes in `packages/aio-commerce-lib-app` private-pkg: common-utils labels May 6, 2026
@iivvaannxx iivvaannxx marked this pull request as draft May 6, 2026 12:51
@github-actions github-actions Bot removed the pkg: aio-commerce-lib-core Includes changes in `packages/aio-commerce-lib-core` label May 6, 2026
@iivvaannxx iivvaannxx marked this pull request as ready for review May 13, 2026 14:05
@iivvaannxx iivvaannxx changed the title PoC: OpenAPI CEXT-6138: SDK Capability Discovery May 13, 2026
obarcelonap
obarcelonap reviewed May 14, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@obarcelonap obarcelonap left a comment

Choose a reason for hiding this comment

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

Don't forget to flag the spec CEXT-6138 as implemented.
Also, we need to document from user perspective that filtered openapi spec is available as new endpoint, usage.md in aio-commerce-lib-app?

Comment thread packages/aio-commerce-lib-app/generated/openapi.gen.json
Copy link
Copy Markdown
Member

@obarcelonap obarcelonap May 14, 2026

Choose a reason for hiding this comment

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

is this the final content? I am missing all the endpoints
additionally, I guess we will need to filter the content in pre-app-build hook

Copy link
Copy Markdown
Collaborator Author

@iivvaannxx iivvaannxx May 14, 2026

Choose a reason for hiding this comment

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

Content filtering is pending yeah, thought it wasn't necessary but it is, will add. The final content I'll push just before merge, it only adds noise to diff count of PR and I didn't want to scare people with a 6K lines PR

jcuerdo
jcuerdo reviewed May 15, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@jcuerdo jcuerdo left a comment

Choose a reason for hiding this comment

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

Changeset is missing too.

Comment thread packages-private/common-utils/source/actions/http/openapi.ts
Copy link
Copy Markdown
Collaborator

@jcuerdo jcuerdo May 15, 2026

Choose a reason for hiding this comment

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

Missing test for this file. Check others

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jcuerdo jcuerdo jcuerdo left review comments

@obarcelonap obarcelonap obarcelonap left review comments

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

pkg: aio-commerce-lib-app Includes changes in `packages/aio-commerce-lib-app` private-pkg: common-utils without-changeset The PR does not contain a Changeset file

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@iivvaannxx @jcuerdo @obarcelonap