Update builder extension docs with new template #181
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Contributor
|
📝 Documentation updates detected! New suggestion: Add Java SDK builder extensibility documentation with Template Method pattern |
Contributor
dsinghvi
reviewed
Jul 25, 2025
| ## Adding custom client configuration | ||
|
|
||
| When you need to intercept and transform client configuration before the SDK client is created, you can extend the generated builder classes. This allows you to add custom code and logic during the client initialization process. | ||
| The Java SDK generator now supports builder extensibility through a template method pattern. By extending the generated builder classes and overriding protected methods, you can customize how your SDK client is configured without modifying generated code. |
Member
There was a problem hiding this comment.
nit: lets delete now since these docs will be forever
dsinghvi
reviewed
Jul 25, 2025
| ``` | ||
|
|
||
| ### Create a custom builder with your transformation logic | ||
| ### Create a custom builder |
Member
There was a problem hiding this comment.
nit: lets add class Create a custom builder class
dsinghvi
reviewed
Jul 25, 2025
|
|
||
| ### Update `.fernignore` | ||
|
|
||
| Add the `MyClient.java` and `MyClientBuilder.java` to `.fernignore`. |
Member
There was a problem hiding this comment.
By adding these two files to .fernignore, fern will not update them on new generations.
dsinghvi
reviewed
Jul 25, 2025
| @Override | ||
| protected void setEnvironment(ClientOptions.Builder builder) { | ||
| String baseUrl = this.environment.getUrl(); | ||
| String tenantUrl = baseUrl.replace("/api/", "/api/tenants/" + tenantId + "/"); |
Member
There was a problem hiding this comment.
nit: lets get rid of the duplicative api
dsinghvi
approved these changes
Jul 25, 2025
Contributor
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
New pattern introduced here which we need to update fern-api/fern#8372