Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(cli): Add generator config, oauth and addtional properties to dynamic IR #6143

Open
wants to merge 23 commits into
base: main
Choose a base branch
from

Conversation

amckinney
Copy link
Member

@amckinney amckinney commented Feb 23, 2025

This adds a few new fields to the dynamic IR:

  1. Add the GeneratorConfig as an optional property to the dynamic IR.
    • This will make it simpler to generate dynamic snippets in the front-end.
  2. OAuth configuration for the client credentials flow.
    • For now, we reserve the OAuth name, but it's easy for us to roll out additional OAuth configuration options later (e.g. OAuthAuthorizationCode, etc).
  3. Types now know whether or not they support additionalProperties (which is based on the extra-properties setting).
    • This opts to use the new naming convention given that we plan to update the extra-properties name in the future.
  4. Add pathParameters to the top-level of the dynamic IR.
    • For now, all path parameters are included in the endpoint, but we'll update this behavior so that we only include the service-level and endpoint-level path parameters there.
    • Several test definitions will temporarily be broken from this change - these failures will be resolved in a follow-up when we migrate each dynamic snippet generator to the latest IR.

@amckinney amckinney changed the title feat(cli): Add OAuth and addtional properties to dynamic IR feat(cli): Add generator config, OAuth and addtional properties to dynamic IR Mar 4, 2025
@amckinney amckinney changed the title feat(cli): Add generator config, OAuth and addtional properties to dynamic IR feat(cli): Add generator config, oauth and addtional properties to dynamic IR Mar 4, 2025
@amckinney amckinney marked this pull request as ready for review March 4, 2025 02:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants