Skip to content

feat: Generate feature flags from OpenAPI spec - #1635

Draft
gjtorikian wants to merge 2 commits into
mainfrom
oagen/own-feature-flags
Draft

gjtorikian wants to merge 2 commits into
mainfrom
oagen/own-feature-flags

Conversation

@gjtorikian

Copy link
Copy Markdown
Contributor

Description

This PR generates the feature flags category in the Node SDK from the OpenAPI spec.

It changes a few methods from positional args to more modern object args:

 ┌────────────────────┬────────────────┬─────────────────────────────┐
  │       Method       │     Before     │            After            │
  ├────────────────────┼────────────────┼─────────────────────────────┤
  │ getFeatureFlag     │ (slug: string) │ (options: { slug: string }) │
  ├────────────────────┼────────────────┼─────────────────────────────┤
  │ enableFeatureFlag  │ (slug: string) │ (options: { slug: string }) │
  ├────────────────────┼────────────────┼─────────────────────────────┤
  │ disableFeatureFlag │ (slug: string) │ (options: { slug: string }) │
  └────────────────────┴────────────────┴─────────────────────────────┘
  └────────────────────────┴────────────────────────────┘

As well:

  • createdAt / updatedAt are now Date objects, not ISO strings.

Because of this, the PR should be treated as a breaking change.

@gjtorikian gjtorikian added the autogenerated Autogenerated code or content label Jun 18, 2026
Regenerate the feature flags surface now that the openapi-spec policy
renames the upstream `Flag`/`FlagList` schemas to
`FeatureFlag`/`FeatureFlagList`, matching the type names the SDK already
exposes. `getFeatureFlag` returns `FeatureFlag` again, so all four
methods share one consistent type.

Removes the stale generated `Flag`/`FlagOwner` interface, serializer,
and fixture files (and their manifest entries) left behind by the prior
generation.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autogenerated Autogenerated code or content breaking change Contains a breaking change

Development

Successfully merging this pull request may close these issues.

1 participant