Repository navigation
refactor: rename tilde api with dollar - #4086
Conversation
|
|
🦋 Changeset detectedLatest commit: d5ca286 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
@hey-api/codegen-core
@hey-api/json-schema-ref-parser
@hey-api/nuxt
@hey-api/openapi-ts
@hey-api/shared
@hey-api/spec-types
@hey-api/types
@hey-api/vite-plugin
commit: |
There was a problem hiding this comment.
Reviewed changes — renames ~hooks to $hooks and ~resolvers to $resolvers across plugin config types, runtime code, and documentation, with @deprecated annotations on the old keys for backward compatibility.
- Add
$hooksand$resolverstoPluginBaseConfigandPlugin.Resolverstypes — new fields inpackages/shared/src/plugins/types.tswith@deprecatedon~hooksand~resolvers - Update all resolver access sites — every
toAstfunction and validator resolver across typescript, zod (mini/v3/v4), valibot, and pydantic now checks$resolversfirst, falling back to~resolvers - Update
PluginInstancehook reading —buildEventHooks,getHooks, andisOperationKindinpackages/shared/src/plugins/shared/utils/instance.tstry$hooksfirst - Update plugin export types —
Required<...>['$resolvers']inindex.tsfor bothopenapi-tsandopenapi-python - Update documentation — parser, resolvers, custom plugin, and SDK docs reference
$hooks/$resolvers - Add changesets — patch for
@hey-api/openapi-tsand@hey-api/shared
ℹ️ Nitpicks
packages/shared/src/plugins/shared/utils/__tests__/instance.test.tsstill uses'~hooks'in test config — worth migrating to$hooks(and/or adding coverage for the$hookspath) in a follow-up.
DeepSeek Pro (free via Pullfrog for OSS) | 𝕏
| </TabItem> | ||
| </Tabs> | ||
|
|
||
| ## Parameters |
There was a problem hiding this comment.
This new ## Parameters section (~70 lines documenting paramsStructure) is unrelated to the ~ → $ rename. Consider moving it to a separate PR to keep this diff focused on the rename.
0096ae5 to
d5ca286
Compare

No description provided.