Repository navigation
fix: sync dual-name snapshots for merged API sources - #81
Merged
Merged
Conversation
Match app 0292caf5 and SMTP d91bcfd0 generated A17 output; regenerate Postman normally. Fixes main snapshot drift in app run 41575368941 and SMTP run 41573928097. Refs qubrics/qubrics#904.
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one): 1
Please check the settings in the CodeRabbit UI or the
|
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.
Main CI fails because the app and Sending API sources emit the A17 dual-named contracts while docs main still contains the previous snapshots. App run 41575368941 fails at
scripts/check-openapi-snapshot.mjs:36; SMTP run 41573928097 fails atscripts/check-openapi-snapshot.js:35. Both successful PR runs selected docs #78, while their main pushes selected docs main12ef5105.Copy the exact generated app and Sending snapshots matching merged app
0292caf5and SMTPd91bcfd0, then regenerate their Postman collections normally. Four files, +55/-13. This is the intended qubrics/qubrics#904 A17 output: equal header twins, the optional mail correlation alias, and either upload-token header authenticated. Existing legacy names andsendmux.testremain. No check or product source changes.Validation: both owning
spec:checkcommands pass against this checkout; source trees match their qualified PR heads. Postman generation/drift, confidentiality, external links, Mint validation, broken links and whitespace checks pass. The exact OpenAPI bytes were already qualified in docs #78; this slice makes them available to main CI without its MCP guide/pin publication dependencies.Priority repair authorised by Ops Watch after app #915; no deployment. MCP examples, the protected release pin/snippet and the remaining docs #78 release notes stay in that held PR. After this snapshot merge, recheck the two failing main CI signals against the corrected docs input.