ci(docs): dispatch convoy-java SDK generation - #2739
Merged
Conversation
adds convoy-java to the SDK regeneration matrix. the repo implements the same sdk_generation.yaml contract (openapi generator, java native library); docs updated for the third client and the PAT scope note.
|
Current version of PR was reviewed by /review-bugbot on Jul 19, 22:30 GMT+1. It flagged 0 findings. Bugbot on commit |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
convoy-javato the SDK regeneration dispatch matrix inspeakeasy-sdk.yml. The repo implements the samesdk_generation.yamlcontract as convoy.js/convoy-python (feat: add OpenAPI-generated API client convoy-java#1), generating via OpenAPI Generator (java, native library).docs/md/SPEAKEASY_SDK.mdfor the third client and notes thatSDK_REPOS_PAT/SDK_BOT_PATmust coverconvoy-java.Ops prerequisite before the first dispatched run: extend
SDK_REPOS_PAT(on this repo) and addSDK_BOT_PAT(on convoy-java) to cover frain-dev/convoy-java. Missing tokens fail the dispatch step loudly; they do not create a security gap.Test plan
sdk_generation.yamlvalidated end to end locally (generation, compile, 25 tests)Note
Low Risk
CI and documentation only, reusing the existing dispatch pattern; token scope must be updated before the first Java run or dispatch will fail loudly.
Overview
Extends the
speakeasy-sdk.ymldispatcher so OpenAPI changes onmainalso triggerfrain-dev/convoy-javavia the samesdk_generation.yamlcontract as the JS and Python repos (Java uses OpenAPI Generator locally in that repo).Workflow comments now describe the per-repo generator model and that
SDK_REPOS_PATmust cover all three SDK repos. The optionalfeature_branchguard addsrefs/*so names likerefs/heads/maincannot slip past themainblock and affect force-push behavior downstream.docs/md/SPEAKEASY_SDK.mddocumentsconvoy-java, expands scope/secrets/pipeline diagrams, and drops Java from the “future languages” list.Reviewed by Cursor Bugbot for commit f7a9aaf. Bugbot is set up for automated code reviews on this repo. Configure here.