Skip to content

fix: sync dual-name snapshots for merged API sources - #81

Merged
roshanjonah merged 1 commit into
mainfrom
agent/rebrand-qubrics-904-main-contracts
Nov 3, 2026
Merged

roshanjonah merged 1 commit into
mainfrom
agent/rebrand-qubrics-904-main-contracts

Conversation

@roshanjonah

Copy link
Copy Markdown
Member

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 at scripts/check-openapi-snapshot.js:35. Both successful PR runs selected docs #78, while their main pushes selected docs main 12ef5105.

Copy the exact generated app and Sending snapshots matching merged app 0292caf5 and SMTP d91bcfd0, 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 and sendmux.test remain. No check or product source changes.

Validation: both owning spec:check commands 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.

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.
@coderabbitai

coderabbitai Bot commented Nov 3, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one): 1
  • major

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Team
  • Run ID: 9cef358c-90e5-4b7c-80ca-e31cf6121bc1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.

@roshanjonah
roshanjonah merged commit 5f0f7c8 into main Nov 3, 2026
3 checks passed
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