fix(cli): propagate Venus selfHosted flag to IR for local and remote generation #10766
+2
−0
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.
Description
Refs https://buildwithfern.slack.com/archives/C05J55L8JES/p1763673405911359
Fixes a bug where the
selfHostedflag in the IR was alwaysfalseeven when Venus returnsselfHostedSDKs: truefor an organization. This was causing issues with README generation for organizations like Bloomberg that have self-hosted SDKs enabled.Changes Made
selfHostedflag from Venus organization data to IR in local generation path (runLocalGenerationForWorkspace.ts)selfHostedflag from Venus organization data to IR in remote generation path (runRemoteGenerationForGenerator.ts)whiteLabelflag propagationRoot Cause
The IR was hardcoded with
selfHosted: falseingenerateIntermediateRepresentation(), and even though the Venus API returns organization data withselfHostedSDKs, this value was never being used to update the IR after generation.Testing
pnpm run check)selfHostedSDKs: true(e.g., Bloomberg) - Reviewer should verify this works as expectedReview Checklist
selfHostedSdKsmatches the Venus SDK type definition (it's the TypeScript field name that maps to wire formatselfHostedSDKs)whiteLabelflag is propagated (same pattern, same location)ir.selfHostedfor README generation or other purposes (TypeScript generator confirmed to use it)Link to Devin run: https://app.devin.ai/sessions/dde07ad03651451a95658926abd372a1
Requested by: [email protected] (@aditya-arolkar-swe)