Skip to content

chore: rename shouldGenerateWebsocketClients to generateWebSocketClients#4284

Merged
fern-support merged 3 commits intomainfrom
devin/1773634597-rename-websocket-clients-docs
Mar 16, 2026
Merged

chore: rename shouldGenerateWebsocketClients to generateWebSocketClients#4284
fern-support merged 3 commits intomainfrom
devin/1773634597-rename-websocket-clients-docs

Conversation

@Swimburger
Copy link
Member

@Swimburger Swimburger commented Mar 16, 2026

Summary

Updates documentation to reflect the renamed TypeScript SDK generator config option: shouldGenerateWebsocketClientsgenerateWebSocketClients. This is the docs companion to fern-api/fern#13562, which renamed the config flag in the generator code (keeping the old name as a deprecated fallback).

Files changed:

  • configuration.mdx — Updated the <ParamField> path from shouldGenerateWebsocketClients to generateWebSocketClients, and added a note that the old name is still accepted as a deprecated alias
  • changelog/2025-03-06.mdx — Updated the inline code reference to use the new config name

Review & Testing Checklist for Human

  • Decide whether the changelog entry (2025-03-06) should keep the original name shouldGenerateWebsocketClients since it's a historical record of what was released at that time, or use the new name for discoverability
  • Verify the deprecation note wording in configuration.mdx reads well on the rendered docs site

Notes

The generator-side change preserves backwards compatibility: generateWebSocketClients is preferred, but shouldGenerateWebsocketClients still works as a fallback. The configuration page now documents this fallback.

Link to Devin session: https://app.devin.ai/sessions/d8ce068101844e7b8b41db4d4546b912
Requested by: @Swimburger


Open with Devin

…nts in docs

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions
Copy link
Contributor

github-actions bot commented Mar 16, 2026

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

</ParamField>

<ParamField path="shouldGenerateWebsocketClients" type="boolean" toc={true}>
<ParamField path="generateWebSocketClients" type="boolean" toc={true}>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mention the old name, just in case users are on an older generator version.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done — added a note mentioning shouldGenerateWebsocketClients as a deprecated alias.

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@fern-support fern-support merged commit aa3dd52 into main Mar 16, 2026
2 checks passed
@fern-support fern-support deleted the devin/1773634597-rename-websocket-clients-docs branch March 16, 2026 04:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants