From 515ca05c2fc7acf091e060c6dfcdd436de93bdbb Mon Sep 17 00:00:00 2001 From: ashish-kumar2-glean Date: Tue, 18 Aug 2026 14:21:44 +0000 Subject: [PATCH] test: allow the triggers top-level SDK namespace The Platform Triggers API went Public in askscio/scio#275355, so all ten trigger operations now reach the transform carrying x-speakeasy-group: triggers. post_transform_smoke allowlists the platform top-level SDK namespaces and did not include triggers, so Transform OpenAPI Specs has failed on every run since that merge, leaving specs/final without the triggers endpoints and never dispatching the developer-site redeploy. Verified by injecting the ten transformed trigger paths into overlayed_specs/glean-merged-spec.yaml: the assertion reproduces the CI failure without this change and passes with it. --- tests/post_transform_smoke.test.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/tests/post_transform_smoke.test.js b/tests/post_transform_smoke.test.js index 8afddb56..e8484953 100644 --- a/tests/post_transform_smoke.test.js +++ b/tests/post_transform_smoke.test.js @@ -237,7 +237,13 @@ describe('Post-transformation smoke tests', () => { // without a group silently falls back to its `tags` and leaks a method to // the SDK top level. The top level is reserved for the Platform API, so // client/indexing operations must be nested under `client.*` / `indexing.*`. - const platformSegments = new Set(['agents', 'chat', 'search', 'skills']); + const platformSegments = new Set([ + 'agents', + 'chat', + 'search', + 'skills', + 'triggers', + ]); const allowedTopLevelSegments = new Set([ 'client', 'indexing',