From cf2062284f7fc9d46f8a9df3b3ae9f4238f60fb9 Mon Sep 17 00:00:00 2001
From: Spencer Judge <spencer@temporal.io>
Date: Tue, 31 Mar 2026 11:56:42 -0700
Subject: [PATCH 1/2] Claude fix all docstring problems & add check to CI

---
 .github/workflows/ci.yml                      | 26 ++++++++++++++++++-
 .gitignore                                    |  4 ++-
 gradle/java.gradle                            |  5 ++--
 .../io/temporal/envconfig/ClientConfig.java   |  2 +-
 .../codec/AbstractRemoteDataEncoderCodec.java |  2 --
 .../io/temporal/activity/ActivityInfo.java    |  4 +--
 .../temporal/activity/ActivityInterface.java  |  2 --
 .../io/temporal/client/WorkflowClient.java    | 15 +++++------
 .../temporal/client/WorkflowClientPlugin.java |  8 +++---
 .../client/WorkflowExecutionDescription.java  |  8 +++---
 .../io/temporal/client/WorkflowOptions.java   |  2 +-
 .../java/io/temporal/client/WorkflowStub.java | 15 +++++------
 .../ScheduleActionStartWorkflow.java          |  6 ++---
 .../WorkflowInboundCallsInterceptor.java      |  6 ++---
 .../WorkflowOutboundCallsInterceptor.java     |  4 +--
 .../io/temporal/internal/package-info.java    |  4 +--
 .../internal/task/VirtualThreadDelegate.java  |  2 +-
 .../main/java/io/temporal/worker/Worker.java  |  4 ---
 .../temporal/worker/WorkerFactoryOptions.java |  6 ++---
 .../java/io/temporal/worker/WorkerPlugin.java |  4 +--
 .../worker/WorkflowTaskDispatchHandle.java    |  3 ++-
 .../worker/tuning/SlotReserveContext.java     |  2 +-
 .../java/io/temporal/workflow/Workflow.java   | 15 +++++------
 .../io/temporal/workflow/WorkflowInfo.java    | 12 ++++-----
 .../temporal/workflow/WorkflowInterface.java  |  2 --
 .../io/temporal/workflow/WorkflowLocal.java   | 18 ++++++-------
 .../workflow/WorkflowThreadLocal.java         | 18 ++++++-------
 .../workflow/unsafe/WorkflowUnsafe.java       |  4 +--
 .../temporal/internal/BackoffThrottler.java   |  2 --
 .../CheckedExceptionWrapper.java              |  4 +--
 .../testservice/TestWorkflowService.java      |  2 +-
 31 files changed, 109 insertions(+), 102 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 964d0d9883..8846e4e0a4 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -191,7 +191,7 @@ jobs:
       - name: Set up Java
         uses: actions/setup-java@v5
         with:
-          java-version: "11"
+          java-version: "23"
           distribution: "temurin"
 
       - name: Set up Gradle
@@ -200,6 +200,30 @@ jobs:
       - name: Run copyright and code format checks
         run: ./gradlew --no-daemon spotlessCheck
 
+  javadoc:
+    name: Javadoc
+    runs-on: ubuntu-latest
+    timeout-minutes: 20
+    steps:
+      - name: Checkout repo
+        uses: actions/checkout@v5
+        with:
+          fetch-depth: 0
+          submodules: recursive
+          ref: ${{ github.event.pull_request.head.sha }}
+
+      - name: Set up Java
+        uses: actions/setup-java@v5
+        with:
+          java-version: "23"
+          distribution: "temurin"
+
+      - name: Set up Gradle
+        uses: gradle/actions/setup-gradle@v5
+
+      - name: Run javadoc
+        run: ./gradlew --no-daemon javadoc
+
   build_native_images:
     name: Build native test server
     uses: ./.github/workflows/build-native-image.yml
diff --git a/.gitignore b/.gitignore
index ef1d5524bd..619d53f221 100644
--- a/.gitignore
+++ b/.gitignore
@@ -18,4 +18,6 @@ src/main/idls/*
 .settings
 .vscode/
 */bin
-/.claude
\ No newline at end of file
+/.claude
+mise.local.toml
+**/.factorypath
diff --git a/gradle/java.gradle b/gradle/java.gradle
index d968e5ff1d..fbf17a925a 100644
--- a/gradle/java.gradle
+++ b/gradle/java.gradle
@@ -32,9 +32,8 @@ subprojects {
 
     javadoc {
         options.encoding = 'UTF-8'
-        if (JavaVersion.current().isJava8Compatible()) {
-            options.addStringOption('Xdoclint:none', '-quiet')
-        }
+        options.addStringOption('Xdoclint:reference', '-quiet')
+        options.addBooleanOption('Werror', true)
         if (JavaVersion.current().isJava9Compatible()) {
             options.addBooleanOption('html5', true)
         }
diff --git a/temporal-envconfig/src/main/java/io/temporal/envconfig/ClientConfig.java b/temporal-envconfig/src/main/java/io/temporal/envconfig/ClientConfig.java
index 30dfa9b436..50051b57a7 100644
--- a/temporal-envconfig/src/main/java/io/temporal/envconfig/ClientConfig.java
+++ b/temporal-envconfig/src/main/java/io/temporal/envconfig/ClientConfig.java
@@ -172,7 +172,7 @@ public static ClientConfig fromToml(byte[] tomlData, ClientConfigFromTomlOptions
    *
    * @param config the client config to convert
    * @return the TOML data as bytes
-   * @apiNote The output will not be identical to the input if the config was loaded from a file
+   *     <p>Note: The output will not be identical to the input if the config was loaded from a file
    *     because comments and formatting are not preserved.
    */
   public static byte[] toTomlAsBytes(ClientConfig config) throws IOException {
diff --git a/temporal-remote-data-encoder/src/main/java/io/temporal/payload/codec/AbstractRemoteDataEncoderCodec.java b/temporal-remote-data-encoder/src/main/java/io/temporal/payload/codec/AbstractRemoteDataEncoderCodec.java
index a70f651bb1..cf02f24f35 100644
--- a/temporal-remote-data-encoder/src/main/java/io/temporal/payload/codec/AbstractRemoteDataEncoderCodec.java
+++ b/temporal-remote-data-encoder/src/main/java/io/temporal/payload/codec/AbstractRemoteDataEncoderCodec.java
@@ -11,8 +11,6 @@
 /**
  * Performs encoding/decoding of the payloads via the Remote Data Encoder (RDE) available over http.
  *
- * <p>
- *
  * <h2>Remote Data Encoder Http Server specification</h2>
  *
  * <p>RDE Server must:
diff --git a/temporal-sdk/src/main/java/io/temporal/activity/ActivityInfo.java b/temporal-sdk/src/main/java/io/temporal/activity/ActivityInfo.java
index 40d8d78af7..d161c8361c 100644
--- a/temporal-sdk/src/main/java/io/temporal/activity/ActivityInfo.java
+++ b/temporal-sdk/src/main/java/io/temporal/activity/ActivityInfo.java
@@ -126,8 +126,8 @@ public interface ActivityInfo {
   /**
    * Return the priority of the activity task.
    *
-   * @apiNote If unset or on an older server version, this method will return {@link
-   *     Priority#getDefaultInstance()}.
+   * <p>Note: If unset or on an older server version, this method will return {@link
+   * Priority#getDefaultInstance()}.
    */
   @Experimental
   @Nonnull
diff --git a/temporal-sdk/src/main/java/io/temporal/activity/ActivityInterface.java b/temporal-sdk/src/main/java/io/temporal/activity/ActivityInterface.java
index 0a7b01e87d..9cfb5ee975 100644
--- a/temporal-sdk/src/main/java/io/temporal/activity/ActivityInterface.java
+++ b/temporal-sdk/src/main/java/io/temporal/activity/ActivityInterface.java
@@ -42,8 +42,6 @@
  * When <code>CImpl</code> instance is registered with the {@link io.temporal.worker.Worker} the
  * following activities are registered:
  *
- * <p>
- *
  * <ul>
  *   <li>B_a
  *   <li>B_b
diff --git a/temporal-sdk/src/main/java/io/temporal/client/WorkflowClient.java b/temporal-sdk/src/main/java/io/temporal/client/WorkflowClient.java
index f73ffc92f2..ef0c615a5a 100644
--- a/temporal-sdk/src/main/java/io/temporal/client/WorkflowClient.java
+++ b/temporal-sdk/src/main/java/io/temporal/client/WorkflowClient.java
@@ -158,9 +158,8 @@ static WorkflowClient newInstance(WorkflowServiceStubs service, WorkflowClientOp
    * @param workflowId Workflow id.
    * @param runId Run id of the workflow execution.
    * @return Stub that implements workflowInterface and can be used to signal, update, or query it.
-   * @deprecated Use {@link #newWorkflowStub(Class, WorkflowTargetOptions)} instead.
-   * @apiNote This method is deprecated because the returned stub does not properly account for the
-   *     runId.
+   * @deprecated Use {@link #newWorkflowStub(Class, WorkflowTargetOptions)} instead. This method is
+   *     deprecated because the returned stub does not properly account for the runId.
    */
   @Deprecated
   <T> T newWorkflowStub(Class<T> workflowInterface, String workflowId, Optional<String> runId);
@@ -205,9 +204,8 @@ static WorkflowClient newInstance(WorkflowServiceStubs service, WorkflowClientOp
    *     workflowId is assumed.
    * @param workflowType type of the workflow. Optional as it is used for error reporting only.
    * @return Stub that can be used to start workflow and later to signal or query it.
-   * @deprecated Use {@link #newUntypedWorkflowStub(WorkflowTargetOptions, Optional)} instead.
-   * @apiNote This method is deprecated because the returned stub does not properly account for the
-   *     runId.
+   * @deprecated Use {@link #newUntypedWorkflowStub(WorkflowTargetOptions, Optional)} instead. This
+   *     method is deprecated because the returned stub does not properly account for the runId.
    */
   @Deprecated
   WorkflowStub newUntypedWorkflowStub(
@@ -220,9 +218,8 @@ WorkflowStub newUntypedWorkflowStub(
    * @param execution workflow id and optional run id for execution
    * @param workflowType type of the workflow. Optional as it is used for error reporting only.
    * @return Stub that can be used to start workflow and later to signal or query it.
-   * @deprecated Use {@link #newUntypedWorkflowStub(WorkflowTargetOptions, Optional)} instead.
-   * @apiNote This method is deprecated because the returned stub does not properly account for the
-   *     runId.
+   * @deprecated Use {@link #newUntypedWorkflowStub(WorkflowTargetOptions, Optional)} instead. This
+   *     method is deprecated because the returned stub does not properly account for the runId.
    */
   WorkflowStub newUntypedWorkflowStub(WorkflowExecution execution, Optional<String> workflowType);
 
diff --git a/temporal-sdk/src/main/java/io/temporal/client/WorkflowClientPlugin.java b/temporal-sdk/src/main/java/io/temporal/client/WorkflowClientPlugin.java
index 702fa141bf..44d7a4957c 100644
--- a/temporal-sdk/src/main/java/io/temporal/client/WorkflowClientPlugin.java
+++ b/temporal-sdk/src/main/java/io/temporal/client/WorkflowClientPlugin.java
@@ -27,9 +27,9 @@
 /**
  * Plugin interface for customizing Temporal workflow client configuration.
  *
- * <p>This interface is separate from {@link
- * io.temporal.serviceclient.WorkflowServiceStubs.ServiceStubsPlugin} to allow plugins that only
- * need to configure the workflow client without affecting the underlying gRPC connection.
+ * <p>This interface is separate from {@link io.temporal.serviceclient.WorkflowServiceStubsPlugin}
+ * to allow plugins that only need to configure the workflow client without affecting the underlying
+ * gRPC connection.
  *
  * <p>Plugins that implement both {@code ServiceStubsPlugin} and {@code WorkflowClientPlugin} will
  * have their service stubs configuration applied when creating the service stubs, and their client
@@ -54,7 +54,7 @@
  * }
  * }</pre>
  *
- * @see io.temporal.serviceclient.WorkflowServiceStubs.ServiceStubsPlugin
+ * @see io.temporal.serviceclient.WorkflowServiceStubsPlugin
  * @see io.temporal.worker.WorkerPlugin
  * @see SimplePlugin
  */
diff --git a/temporal-sdk/src/main/java/io/temporal/client/WorkflowExecutionDescription.java b/temporal-sdk/src/main/java/io/temporal/client/WorkflowExecutionDescription.java
index 18bc890599..f4b29a0af7 100644
--- a/temporal-sdk/src/main/java/io/temporal/client/WorkflowExecutionDescription.java
+++ b/temporal-sdk/src/main/java/io/temporal/client/WorkflowExecutionDescription.java
@@ -22,8 +22,8 @@ public WorkflowExecutionDescription(
   /**
    * Get the fixed summary for this workflow execution.
    *
-   * @apiNote Will be decoded on each invocation, so it is recommended to cache the result if it is
-   *     used multiple times.
+   * <p>Note: Will be decoded on each invocation, so it is recommended to cache the result if it is
+   * used multiple times.
    */
   @Experimental
   @Nullable
@@ -45,8 +45,8 @@ public String getStaticSummary() {
   /**
    * Get the details summary for this workflow execution.
    *
-   * @apiNote Will be decoded on each invocation, so it is recommended to cache the result if it is
-   *     used multiple times.
+   * <p>Note: Will be decoded on each invocation, so it is recommended to cache the result if it is
+   * used multiple times.
    */
   @Experimental
   @Nullable
diff --git a/temporal-sdk/src/main/java/io/temporal/client/WorkflowOptions.java b/temporal-sdk/src/main/java/io/temporal/client/WorkflowOptions.java
index 1747e888f7..628aa1a61f 100644
--- a/temporal-sdk/src/main/java/io/temporal/client/WorkflowOptions.java
+++ b/temporal-sdk/src/main/java/io/temporal/client/WorkflowOptions.java
@@ -162,7 +162,7 @@ public Builder setWorkflowId(String workflowId) {
     /**
      * Specifies server behavior if a completed workflow with the same id exists. Note that under no
      * conditions Temporal allows two workflows with the same namespace and workflow id run
-     * simultaneously. See {@line setWorkflowIdConflictPolicy} for handling a workflow id
+     * simultaneously. See {@link #setWorkflowIdConflictPolicy} for handling a workflow id
      * duplication with a <b>Running</b> workflow.
      *
      * <p>Default value if not set: <b>AllowDuplicate</b>
diff --git a/temporal-sdk/src/main/java/io/temporal/client/WorkflowStub.java b/temporal-sdk/src/main/java/io/temporal/client/WorkflowStub.java
index 203921110a..e432454a7e 100644
--- a/temporal-sdk/src/main/java/io/temporal/client/WorkflowStub.java
+++ b/temporal-sdk/src/main/java/io/temporal/client/WorkflowStub.java
@@ -10,7 +10,6 @@
 import java.lang.reflect.Type;
 import java.util.Optional;
 import java.util.concurrent.CompletableFuture;
-import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.TimeoutException;
 import javax.annotation.Nullable;
@@ -273,8 +272,7 @@ <R> R getResult(long timeout, TimeUnit unit, Class<R> resultClass, Type resultTy
    * @param <R> type of the workflow return value
    * @return future completed with workflow return value or an exception
    * @see #getResult(Class) as a sync version of this method for detailed information about
-   *     exceptions that may be thrown from {@link CompletableFuture#get()} wrapped by {@link
-   *     ExecutionException}
+   *     exceptions that may be thrown from CompletableFuture.get() wrapped by ExecutionException
    */
   <R> CompletableFuture<R> getResultAsync(Class<R> resultClass);
 
@@ -289,8 +287,7 @@ <R> R getResult(long timeout, TimeUnit unit, Class<R> resultClass, Type resultTy
    * @param <R> type of the workflow return value
    * @return future completed with workflow return value or an exception
    * @see #getResult(Class, Type) as a sync version of this method for detailed information about
-   *     exceptions that may be thrown from {@link CompletableFuture#get()} wrapped by {@link
-   *     ExecutionException}
+   *     exceptions that may be thrown from CompletableFuture.get() wrapped by ExecutionException
    */
   <R> CompletableFuture<R> getResultAsync(Class<R> resultClass, Type resultType);
 
@@ -305,8 +302,8 @@ <R> R getResult(long timeout, TimeUnit unit, Class<R> resultClass, Type resultTy
    * @param <R> type of the workflow return value
    * @return future completed with workflow return value or an exception
    * @see #getResult(long, TimeUnit, Class) as a sync version of this method for detailed
-   *     information about exceptions that may be thrown from {@link CompletableFuture#get()}
-   *     wrapped by {@link ExecutionException}
+   *     information about exceptions that may be thrown from CompletableFuture.get() wrapped by
+   *     ExecutionException
    */
   <R> CompletableFuture<R> getResultAsync(long timeout, TimeUnit unit, Class<R> resultClass);
 
@@ -323,8 +320,8 @@ <R> R getResult(long timeout, TimeUnit unit, Class<R> resultClass, Type resultTy
    * @param <R> type of the workflow return value
    * @return future completed with workflow return value or an exception
    * @see #getResult(long, TimeUnit, Class, Type) as a sync version of this method for detailed
-   *     information about exceptions that may be thrown from {@link CompletableFuture#get()}
-   *     wrapped by {@link ExecutionException}
+   *     information about exceptions that may be thrown from CompletableFuture.get() wrapped by
+   *     ExecutionException
    */
   <R> CompletableFuture<R> getResultAsync(
       long timeout, TimeUnit unit, Class<R> resultClass, Type resultType);
diff --git a/temporal-sdk/src/main/java/io/temporal/client/schedules/ScheduleActionStartWorkflow.java b/temporal-sdk/src/main/java/io/temporal/client/schedules/ScheduleActionStartWorkflow.java
index 3a67abe7e6..64e31baaa0 100644
--- a/temporal-sdk/src/main/java/io/temporal/client/schedules/ScheduleActionStartWorkflow.java
+++ b/temporal-sdk/src/main/java/io/temporal/client/schedules/ScheduleActionStartWorkflow.java
@@ -51,9 +51,9 @@ public <T> Builder setWorkflowType(Class<T> workflowInterface) {
     /**
      * Set the workflow options to use when starting a workflow action.
      *
-     * @note ID and TaskQueue are required. Some options like ID reuse policy, cron schedule, and
-     *     start signal cannot be set or an error will occur. Schedules requires the use of typed
-     *     search attributes and untyped search attributes will be ignored.
+     * <p>Note: ID and TaskQueue are required. Some options like ID reuse policy, cron schedule, and
+     * start signal cannot be set or an error will occur. Schedules requires the use of typed search
+     * attributes and untyped search attributes will be ignored.
      */
     public Builder setOptions(WorkflowOptions options) {
       this.options = options;
diff --git a/temporal-sdk/src/main/java/io/temporal/common/interceptors/WorkflowInboundCallsInterceptor.java b/temporal-sdk/src/main/java/io/temporal/common/interceptors/WorkflowInboundCallsInterceptor.java
index 1c014d4508..0d50573bef 100644
--- a/temporal-sdk/src/main/java/io/temporal/common/interceptors/WorkflowInboundCallsInterceptor.java
+++ b/temporal-sdk/src/main/java/io/temporal/common/interceptors/WorkflowInboundCallsInterceptor.java
@@ -22,7 +22,7 @@
  * parameters.
  *
  * @see WorkerInterceptor#interceptWorkflow(WorkflowInboundCallsInterceptor) for a definition of
- *     "next" {@link WorkflowInboundCallsInterceptor}
+ *     "next" WorkflowInboundCallsInterceptor
  */
 @Experimental
 public interface WorkflowInboundCallsInterceptor {
@@ -168,8 +168,8 @@ public Object getResult() {
    *
    * @param outboundCalls an existing interceptor instance to be proxied by the interceptor created
    *     inside this method
-   * @see WorkerInterceptor#interceptWorkflow for the definition of "next" {@link
-   *     WorkflowInboundCallsInterceptor}
+   * @see WorkerInterceptor#interceptWorkflow for the definition of "next"
+   *     WorkflowInboundCallsInterceptor
    */
   void init(WorkflowOutboundCallsInterceptor outboundCalls);
 
diff --git a/temporal-sdk/src/main/java/io/temporal/common/interceptors/WorkflowOutboundCallsInterceptor.java b/temporal-sdk/src/main/java/io/temporal/common/interceptors/WorkflowOutboundCallsInterceptor.java
index d515817b29..357df1c4da 100644
--- a/temporal-sdk/src/main/java/io/temporal/common/interceptors/WorkflowOutboundCallsInterceptor.java
+++ b/temporal-sdk/src/main/java/io/temporal/common/interceptors/WorkflowOutboundCallsInterceptor.java
@@ -33,8 +33,8 @@
  * implementation must forward all the calls to the outbound interceptor passed as a {@code
  * outboundCalls} parameter to the {@code init} call.
  *
- * @see WorkerInterceptor#interceptWorkflow for the definition of "next" {@link
- *     WorkflowInboundCallsInterceptor}.
+ * @see WorkerInterceptor#interceptWorkflow for the definition of "next"
+ *     WorkflowInboundCallsInterceptor.
  */
 @Experimental
 public interface WorkflowOutboundCallsInterceptor {
diff --git a/temporal-sdk/src/main/java/io/temporal/internal/package-info.java b/temporal-sdk/src/main/java/io/temporal/internal/package-info.java
index be6ac366f8..3c4710c05f 100644
--- a/temporal-sdk/src/main/java/io/temporal/internal/package-info.java
+++ b/temporal-sdk/src/main/java/io/temporal/internal/package-info.java
@@ -1,7 +1,7 @@
 /**
  * This package and its subpackages contain implementation classes of the Temporal SDK.
  *
- * <p><strong></strong>Do not use them directly in any application code! There is absolutely no
- * guarantee about backwards compatibility of APIs in the internal packages.</strong>
+ * <p><strong>Do not use them directly in any application code! There is absolutely no guarantee
+ * about backwards compatibility of APIs in the internal packages.</strong>
  */
 package io.temporal.internal;
diff --git a/temporal-sdk/src/main/java/io/temporal/internal/task/VirtualThreadDelegate.java b/temporal-sdk/src/main/java/io/temporal/internal/task/VirtualThreadDelegate.java
index 48446725ab..e4ce2d4c5b 100644
--- a/temporal-sdk/src/main/java/io/temporal/internal/task/VirtualThreadDelegate.java
+++ b/temporal-sdk/src/main/java/io/temporal/internal/task/VirtualThreadDelegate.java
@@ -4,7 +4,7 @@
 
 /**
  * Internal delegate for virtual thread handling on JDK 21. This is a dummy version for reachability
- * on JDK <21.
+ * on JDK &lt;21.
  */
 public final class VirtualThreadDelegate {
   public static ExecutorService newVirtualThreadExecutor(ThreadConfigurator configurator) {
diff --git a/temporal-sdk/src/main/java/io/temporal/worker/Worker.java b/temporal-sdk/src/main/java/io/temporal/worker/Worker.java
index 5ffd300e9e..143825dc45 100644
--- a/temporal-sdk/src/main/java/io/temporal/worker/Worker.java
+++ b/temporal-sdk/src/main/java/io/temporal/worker/Worker.java
@@ -254,8 +254,6 @@ public <R> void addWorkflowImplementationFactory(
    * <p>This method is misaligned with other workflow implementation registration methods in this
    * aspect.
    *
-   * <p></font>
-   *
    * @deprecated Use {@link #registerWorkflowImplementationFactory(Class, Func,
    *     WorkflowImplementationOptions)} with {@code
    *     WorkflowImplementationOptions.newBuilder().setFailWorkflowExceptionTypes(Throwable.class).build()}
@@ -304,8 +302,6 @@ public <R> void addWorkflowImplementationFactory(Class<R> workflowInterface, Fun
    * instances is allowed. This way, the configuration is persisted into the history and maintained
    * same during replay.
    *
-   * <p></font>
-   *
    * @param workflowInterface Workflow interface that this factory implements
    * @param factory should create a new instance of the workflow implementation object every time
    *     it's called
diff --git a/temporal-sdk/src/main/java/io/temporal/worker/WorkerFactoryOptions.java b/temporal-sdk/src/main/java/io/temporal/worker/WorkerFactoryOptions.java
index ee24188110..07063416c6 100644
--- a/temporal-sdk/src/main/java/io/temporal/worker/WorkerFactoryOptions.java
+++ b/temporal-sdk/src/main/java/io/temporal/worker/WorkerFactoryOptions.java
@@ -107,9 +107,9 @@ public Builder setWorkerInterceptors(WorkerInterceptor... workerInterceptors) {
      * Sets the worker plugins to use with workers created by this factory. Plugins can modify
      * worker configuration and wrap worker lifecycle.
      *
-     * <p>Note: Plugins that implement both {@link io.temporal.client.ClientPlugin} and {@link
-     * WorkerPlugin} are automatically propagated from the client. Use this method for worker-only
-     * plugins that don't need client-side configuration.
+     * <p>Note: Plugins that implement both {@link io.temporal.client.WorkflowClientPlugin} and
+     * {@link WorkerPlugin} are automatically propagated from the client. Use this method for
+     * worker-only plugins that don't need client-side configuration.
      *
      * @param plugins the worker plugins to use
      * @return this builder for chaining
diff --git a/temporal-sdk/src/main/java/io/temporal/worker/WorkerPlugin.java b/temporal-sdk/src/main/java/io/temporal/worker/WorkerPlugin.java
index 8f0a2e3fbc..ab4c223709 100644
--- a/temporal-sdk/src/main/java/io/temporal/worker/WorkerPlugin.java
+++ b/temporal-sdk/src/main/java/io/temporal/worker/WorkerPlugin.java
@@ -30,7 +30,7 @@
 /**
  * Plugin interface for customizing Temporal worker configuration and lifecycle.
  *
- * <p>Plugins that also implement {@link io.temporal.client.ClientPlugin} are automatically
+ * <p>Plugins that also implement {@link io.temporal.client.WorkflowClientPlugin} are automatically
  * propagated from the client to workers created from that client.
  *
  * <p>Example implementation:
@@ -61,7 +61,7 @@
  * }
  * }</pre>
  *
- * @see io.temporal.client.ClientPlugin
+ * @see io.temporal.client.WorkflowClientPlugin
  * @see SimplePlugin
  */
 @Experimental
diff --git a/temporal-sdk/src/main/java/io/temporal/worker/WorkflowTaskDispatchHandle.java b/temporal-sdk/src/main/java/io/temporal/worker/WorkflowTaskDispatchHandle.java
index 0ec6397564..e768b615e3 100644
--- a/temporal-sdk/src/main/java/io/temporal/worker/WorkflowTaskDispatchHandle.java
+++ b/temporal-sdk/src/main/java/io/temporal/worker/WorkflowTaskDispatchHandle.java
@@ -78,7 +78,8 @@ public interface DispatchCallback extends Function<WorkflowTask, Boolean> {
      *
      * @param workflowTask WorkflowTask to be dispatched
      * @return true if the dispatch was successful and false otherwise
-     * @throws IllegalArgumentException if {@code workflowTask} doesn't belong to the task queue of the Worker that provided the {@link WorkflowTaskDispatchHandle
+     * @throws IllegalArgumentException if {@code workflowTask} doesn't belong to the task queue of
+     *     the Worker that provided the {@link WorkflowTaskDispatchHandle}
      */
     @Override
     Boolean apply(WorkflowTask workflowTask) throws IllegalArgumentException;
diff --git a/temporal-sdk/src/main/java/io/temporal/worker/tuning/SlotReserveContext.java b/temporal-sdk/src/main/java/io/temporal/worker/tuning/SlotReserveContext.java
index 5c8f39e585..f4a1c73f4c 100644
--- a/temporal-sdk/src/main/java/io/temporal/worker/tuning/SlotReserveContext.java
+++ b/temporal-sdk/src/main/java/io/temporal/worker/tuning/SlotReserveContext.java
@@ -9,7 +9,7 @@ public interface SlotReserveContext<SI extends SlotInfo> {
   String getTaskQueue();
 
   /**
-   * @return A read-only & safe for concurrent access mapping of slot permits to the information
+   * @return A read-only &amp; safe for concurrent access mapping of slot permits to the information
    *     associated with the in-use slot. This map is changed internally any time new slots are
    *     used.
    */
diff --git a/temporal-sdk/src/main/java/io/temporal/workflow/Workflow.java b/temporal-sdk/src/main/java/io/temporal/workflow/Workflow.java
index 61c787757f..0378b8b6c2 100644
--- a/temporal-sdk/src/main/java/io/temporal/workflow/Workflow.java
+++ b/temporal-sdk/src/main/java/io/temporal/workflow/Workflow.java
@@ -525,8 +525,9 @@ public static <E> Promise<E> newFailedPromise(Exception failure) {
   /**
    * Creates a {@link WorkflowLock} implementation that can be used from workflow code.
    *
-   * @apiNote The lock returned is not reentrant. If a workflow thread tries to acquire a lock that
-   *     it already holds, the call will block indefinitely.
+   * <p>Note: The lock returned is not reentrant. If a workflow thread tries to acquire a lock that
+   * it already holds, the call will block indefinitely.
+   *
    * @return new instance of {@link WorkflowLock}
    */
   public static WorkflowLock newWorkflowLock() {
@@ -664,8 +665,6 @@ public static void retry(
    * exceptions. So as exceptions are always wrapped adding checked ones to method signature causes
    * more pain than benefit.
    *
-   * <p>
-   *
    * <pre>
    * try {
    *     return someCall();
@@ -1557,10 +1556,10 @@ public static String getCurrentDetails() {
   /**
    * Get the currently running workflow instance.
    *
-   * @apiNote The instance is only available after it has been initialized. This function will
-   *     return null if called before the workflow has been initialized. For example, this could
-   *     happen if the function is called from a {@link WorkflowInit} constructor or {@link
-   *     io.temporal.common.interceptors.WorkflowInboundCallsInterceptor#init(WorkflowOutboundCallsInterceptor)}.
+   * <p>Note: The instance is only available after it has been initialized. This function will
+   * return null if called before the workflow has been initialized. For example, this could happen
+   * if the function is called from a {@link WorkflowInit} constructor or {@link
+   * io.temporal.common.interceptors.WorkflowInboundCallsInterceptor#init(WorkflowOutboundCallsInterceptor)}.
    */
   @Experimental
   @Nullable
diff --git a/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowInfo.java b/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowInfo.java
index e49929e18d..c31c714c27 100644
--- a/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowInfo.java
+++ b/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowInfo.java
@@ -122,15 +122,15 @@ public interface WorkflowInfo {
 
   /**
    * @return Workflow ID of the root Workflow
-   * @apiNote On server versions prior to v1.27.0, this method will be empty. Otherwise, it will be
-   *     empty if the workflow is its own root.
+   *     <p>Note: On server versions prior to v1.27.0, this method will be empty. Otherwise, it will
+   *     be empty if the workflow is its own root.
    */
   Optional<String> getRootWorkflowId();
 
   /**
    * @return Run ID of the root Workflow
-   * @apiNote On server versions prior to v1.27.0, this method will be empty. Otherwise, it will be
-   *     empty if the workflow is its own root.
+   *     <p>Note: On server versions prior to v1.27.0, this method will be empty. Otherwise, it will
+   *     be empty if the workflow is its own root.
    */
   Optional<String> getRootRunId();
 
@@ -191,8 +191,8 @@ public interface WorkflowInfo {
   /**
    * Return the priority of the workflow task.
    *
-   * @apiNote If unset or on an older server version, this method will return {@link
-   *     Priority#getDefaultInstance()}.
+   * <p>Note: If unset or on an older server version, this method will return {@link
+   * Priority#getDefaultInstance()}.
    */
   @Experimental
   @Nonnull
diff --git a/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowInterface.java b/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowInterface.java
index c5a338ea6d..c70b460099 100644
--- a/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowInterface.java
+++ b/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowInterface.java
@@ -76,8 +76,6 @@
  * When <code>CImpl</code> instance is registered with the {@link io.temporal.worker.Worker} the
  * following is registered:
  *
- * <p>
- *
  * <ul>
  *   <li>a signal handler
  *   <li>b signal handler
diff --git a/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowLocal.java b/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowLocal.java
index 4944d7c2e0..78951dbe92 100644
--- a/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowLocal.java
+++ b/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowLocal.java
@@ -44,14 +44,14 @@ public WorkflowLocal() {
 
   /**
    * Create an instance that returns the value returned by the given {@code Supplier} when {@link
-   * #set(S)} has not yet been called in the Workflow. Note that the value returned by the {@code
-   * Supplier} is not stored in the {@code WorkflowLocal} implicitly; repeatedly calling {@link
-   * #get()} will always re-execute the {@code Supplier} until you call {@link #set(S)} for the
-   * first time. If you want the value returned by the {@code Supplier} to be stored in the {@code
-   * WorkflowLocal}, use {@link #withCachedInitial(Supplier)} instead.
+   * #set(Object)} has not yet been called in the Workflow. Note that the value returned by the
+   * {@code Supplier} is not stored in the {@code WorkflowLocal} implicitly; repeatedly calling
+   * {@link #get()} will always re-execute the {@code Supplier} until you call {@link #set(Object)}
+   * for the first time. If you want the value returned by the {@code Supplier} to be stored in the
+   * {@code WorkflowLocal}, use {@link #withCachedInitial(Supplier)} instead.
    *
    * @param supplier Callback that will be executed whenever {@link #get()} is called, until {@link
-   *     #set(S)} is called for the first time.
+   *     #set(Object)} is called for the first time.
    * @return A {@code WorkflowLocal} instance.
    * @param <S> The type stored in the {@code WorkflowLocal}.
    * @deprecated Because the non-caching behavior of this API is typically not desirable, it's
@@ -64,11 +64,11 @@ public static <S> WorkflowLocal<S> withInitial(Supplier<? extends S> supplier) {
 
   /**
    * Create an instance that returns the value returned by the given {@code Supplier} when {@link
-   * #set(S)} has not yet been called in the Workflow, and then stores the returned value inside the
-   * {@code WorkflowLocal}.
+   * #set(Object)} has not yet been called in the Workflow, and then stores the returned value
+   * inside the {@code WorkflowLocal}.
    *
    * @param supplier Callback that will be executed when {@link #get()} is called for the first
-   *     time, if {@link #set(S)} has not already been called.
+   *     time, if {@link #set(Object)} has not already been called.
    * @return A {@code WorkflowLocal} instance.
    * @param <S> The type stored in the {@code WorkflowLocal}.
    */
diff --git a/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowThreadLocal.java b/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowThreadLocal.java
index 1a9aea9cb1..ef21018d8c 100644
--- a/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowThreadLocal.java
+++ b/temporal-sdk/src/main/java/io/temporal/workflow/WorkflowThreadLocal.java
@@ -22,15 +22,15 @@ public WorkflowThreadLocal() {
 
   /**
    * Create an instance that returns the value returned by the given {@code Supplier} when {@link
-   * #set(S)} has not yet been called in the thread. Note that the value returned by the {@code
+   * #set(Object)} has not yet been called in the thread. Note that the value returned by the {@code
    * Supplier} is not stored in the {@code WorkflowThreadLocal} implicitly; repeatedly calling
-   * {@link #get()} will always re-execute the {@code Supplier} until you call {@link #set(S)} for
-   * the first time. This differs from the behavior of {@code ThreadLocal}. If you want the value
-   * returned by the {@code Supplier} to be stored in the {@code WorkflowThreadLocal}, which matches
-   * the behavior of {@code ThreadLocal}, use {@link #withCachedInitial(Supplier)} instead.
+   * {@link #get()} will always re-execute the {@code Supplier} until you call {@link #set(Object)}
+   * for the first time. This differs from the behavior of {@code ThreadLocal}. If you want the
+   * value returned by the {@code Supplier} to be stored in the {@code WorkflowThreadLocal}, which
+   * matches the behavior of {@code ThreadLocal}, use {@link #withCachedInitial(Supplier)} instead.
    *
    * @param supplier Callback that will be executed whenever {@link #get()} is called, until {@link
-   *     #set(S)} is called for the first time.
+   *     #set(Object)} is called for the first time.
    * @return A {@code WorkflowThreadLocal} instance.
    * @param <S> The type stored in the {@code WorkflowThreadLocal}.
    * @deprecated Because the non-caching behavior of this API is typically not desirable, it's
@@ -43,11 +43,11 @@ public static <S> WorkflowThreadLocal<S> withInitial(Supplier<? extends S> suppl
 
   /**
    * Create an instance that returns the value returned by the given {@code Supplier} when {@link
-   * #set(S)} has not yet been called in the Workflow, and then stores the returned value inside the
-   * {@code WorkflowThreadLocal}.
+   * #set(Object)} has not yet been called in the Workflow, and then stores the returned value
+   * inside the {@code WorkflowThreadLocal}.
    *
    * @param supplier Callback that will be executed when {@link #get()} is called for the first
-   *     time, if {@link #set(S)} has not already been called.
+   *     time, if {@link #set(Object)} has not already been called.
    * @return A {@code WorkflowThreadLocal} instance.
    * @param <S> The type stored in the {@code WorkflowThreadLocal}.
    */
diff --git a/temporal-sdk/src/main/java/io/temporal/workflow/unsafe/WorkflowUnsafe.java b/temporal-sdk/src/main/java/io/temporal/workflow/unsafe/WorkflowUnsafe.java
index 43ed87f006..1a67b4e8a6 100644
--- a/temporal-sdk/src/main/java/io/temporal/workflow/unsafe/WorkflowUnsafe.java
+++ b/temporal-sdk/src/main/java/io/temporal/workflow/unsafe/WorkflowUnsafe.java
@@ -75,8 +75,8 @@ public static void deadlockDetectorOff(Functions.Proc proc) {
    *
    * <p><b>Warning!</b> <br>
    * Never make workflow code depend on this flag. Workflow code that runs into deadlock detector is
-   * implemented incorrectly (see {@link }). The intended use of this execution mode is blocking
-   * calls and IO in {@link io.temporal.payload.codec.PayloadCodec}, {@link
+   * implemented incorrectly. The intended use of this execution mode is blocking calls and IO in
+   * {@link io.temporal.payload.codec.PayloadCodec}, {@link
    * io.temporal.common.converter.PayloadConverter} or {@link
    * io.temporal.common.interceptors.WorkerInterceptor} implementations.
    *
diff --git a/temporal-serviceclient/src/main/java/io/temporal/internal/BackoffThrottler.java b/temporal-serviceclient/src/main/java/io/temporal/internal/BackoffThrottler.java
index b2dd0b1600..eede68707c 100644
--- a/temporal-serviceclient/src/main/java/io/temporal/internal/BackoffThrottler.java
+++ b/temporal-serviceclient/src/main/java/io/temporal/internal/BackoffThrottler.java
@@ -12,8 +12,6 @@
  *
  * <p>The formula used to calculate the next sleep interval is:
  *
- * <p>
- *
  * <pre>
  * jitter = random number in the range [-maxJitterCoefficient, +maxJitterCoefficient];
  * sleepTime = min(pow(backoffCoefficient, failureCount - 1) * initialSleep * (1 + jitter), maxSleep);
diff --git a/temporal-serviceclient/src/main/java/io/temporal/serviceclient/CheckedExceptionWrapper.java b/temporal-serviceclient/src/main/java/io/temporal/serviceclient/CheckedExceptionWrapper.java
index e61ff9f05b..42a86eb2dd 100644
--- a/temporal-serviceclient/src/main/java/io/temporal/serviceclient/CheckedExceptionWrapper.java
+++ b/temporal-serviceclient/src/main/java/io/temporal/serviceclient/CheckedExceptionWrapper.java
@@ -3,8 +3,8 @@
 import java.lang.reflect.InvocationTargetException;
 
 /**
- * Do not reference directly by the application level code. Use {@link
- * io.temporal.workflow.Workflow#wrap(Exception)} inside a workflow code and {@link
+ * Do not reference directly by the application level code. Use {@code
+ * io.temporal.workflow.Workflow#wrap(Exception)} inside a workflow code and {@code
  * io.temporal.activity.Activity#wrap(Throwable)} inside an activity code instead.
  */
 public final class CheckedExceptionWrapper extends RuntimeException {
diff --git a/temporal-test-server/src/main/java/io/temporal/internal/testservice/TestWorkflowService.java b/temporal-test-server/src/main/java/io/temporal/internal/testservice/TestWorkflowService.java
index e73e6e7a80..f7076de77b 100644
--- a/temporal-test-server/src/main/java/io/temporal/internal/testservice/TestWorkflowService.java
+++ b/temporal-test-server/src/main/java/io/temporal/internal/testservice/TestWorkflowService.java
@@ -65,7 +65,7 @@
 /**
  * In memory implementation of the Workflow Service. To be used for testing purposes only.
  *
- * <p>Do not use directly, instead use {@link io.temporal.testing.TestWorkflowEnvironment}.
+ * <p>Do not use directly, instead use {@code io.temporal.testing.TestWorkflowEnvironment}.
  */
 public final class TestWorkflowService extends WorkflowServiceGrpc.WorkflowServiceImplBase
     implements Closeable {

From e27cd0a1f3a85bb0a06563ac235e0980981a6f25 Mon Sep 17 00:00:00 2001
From: Spencer Judge <spencer@temporal.io>
Date: Thu, 2 Apr 2026 09:28:25 -0700
Subject: [PATCH 2/2] Fix removed links

---
 .../java/io/temporal/client/WorkflowStub.java | 27 ++++++++++++-------
 .../WorkflowInboundCallsInterceptor.java      |  9 ++++---
 2 files changed, 22 insertions(+), 14 deletions(-)

diff --git a/temporal-sdk/src/main/java/io/temporal/client/WorkflowStub.java b/temporal-sdk/src/main/java/io/temporal/client/WorkflowStub.java
index e432454a7e..6bc93b5e9d 100644
--- a/temporal-sdk/src/main/java/io/temporal/client/WorkflowStub.java
+++ b/temporal-sdk/src/main/java/io/temporal/client/WorkflowStub.java
@@ -10,6 +10,7 @@
 import java.lang.reflect.Type;
 import java.util.Optional;
 import java.util.concurrent.CompletableFuture;
+import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.TimeoutException;
 import javax.annotation.Nullable;
@@ -268,11 +269,13 @@ <R> R getResult(long timeout, TimeUnit unit, Class<R> resultClass, Type resultTy
    * to complete. Behind the scenes this call performs long polls the Temporal Server waiting for
    * workflow completion.
    *
+   * <p>See {@link #getResult(Class)} as a sync version of this method for detailed information
+   * about exceptions that may be thrown from {@link CompletableFuture#get()} wrapped by {@link
+   * ExecutionException}.
+   *
    * @param resultClass class of the workflow return value
    * @param <R> type of the workflow return value
    * @return future completed with workflow return value or an exception
-   * @see #getResult(Class) as a sync version of this method for detailed information about
-   *     exceptions that may be thrown from CompletableFuture.get() wrapped by ExecutionException
    */
   <R> CompletableFuture<R> getResultAsync(Class<R> resultClass);
 
@@ -281,13 +284,15 @@ <R> R getResult(long timeout, TimeUnit unit, Class<R> resultClass, Type resultTy
    * to complete. Behind the scene this call performs long poll on Temporal service waiting for
    * workflow completion notification.
    *
+   * <p>See {@link #getResult(Class, Type)} as a sync version of this method for detailed
+   * information about exceptions that may be thrown from {@link CompletableFuture#get()} wrapped by
+   * {@link ExecutionException}.
+   *
    * @param resultClass class of the workflow return value
    * @param resultType type of the workflow return value. Differs from {@code resultClass} for
    *     generic types.
    * @param <R> type of the workflow return value
    * @return future completed with workflow return value or an exception
-   * @see #getResult(Class, Type) as a sync version of this method for detailed information about
-   *     exceptions that may be thrown from CompletableFuture.get() wrapped by ExecutionException
    */
   <R> CompletableFuture<R> getResultAsync(Class<R> resultClass, Type resultType);
 
@@ -296,14 +301,15 @@ <R> R getResult(long timeout, TimeUnit unit, Class<R> resultClass, Type resultTy
    * to complete. Behind the scene this call performs long poll on Temporal service waiting for
    * workflow completion notification.
    *
+   * <p>See {@link #getResult(long, TimeUnit, Class)} as a sync version of this method for detailed
+   * information about exceptions that may be thrown from {@link CompletableFuture#get()} wrapped by
+   * {@link ExecutionException}.
+   *
    * @param timeout maximum time to wait and perform a background long poll
    * @param unit unit of timeout
    * @param resultClass class of the workflow return value
    * @param <R> type of the workflow return value
    * @return future completed with workflow return value or an exception
-   * @see #getResult(long, TimeUnit, Class) as a sync version of this method for detailed
-   *     information about exceptions that may be thrown from CompletableFuture.get() wrapped by
-   *     ExecutionException
    */
   <R> CompletableFuture<R> getResultAsync(long timeout, TimeUnit unit, Class<R> resultClass);
 
@@ -312,6 +318,10 @@ <R> R getResult(long timeout, TimeUnit unit, Class<R> resultClass, Type resultTy
    * to complete. Behind the scene this call performs long poll on Temporal service waiting for
    * workflow completion notification.
    *
+   * <p>See {@link #getResult(long, TimeUnit, Class, Type)} as a sync version of this method for
+   * detailed information about exceptions that may be thrown from {@link CompletableFuture#get()}
+   * wrapped by {@link ExecutionException}.
+   *
    * @param timeout maximum time to wait and perform a background long poll
    * @param unit unit of timeout
    * @param resultClass class of the workflow return value
@@ -319,9 +329,6 @@ <R> R getResult(long timeout, TimeUnit unit, Class<R> resultClass, Type resultTy
    *     generic types.
    * @param <R> type of the workflow return value
    * @return future completed with workflow return value or an exception
-   * @see #getResult(long, TimeUnit, Class, Type) as a sync version of this method for detailed
-   *     information about exceptions that may be thrown from CompletableFuture.get() wrapped by
-   *     ExecutionException
    */
   <R> CompletableFuture<R> getResultAsync(
       long timeout, TimeUnit unit, Class<R> resultClass, Type resultType);
diff --git a/temporal-sdk/src/main/java/io/temporal/common/interceptors/WorkflowInboundCallsInterceptor.java b/temporal-sdk/src/main/java/io/temporal/common/interceptors/WorkflowInboundCallsInterceptor.java
index 0d50573bef..5e69a0b14e 100644
--- a/temporal-sdk/src/main/java/io/temporal/common/interceptors/WorkflowInboundCallsInterceptor.java
+++ b/temporal-sdk/src/main/java/io/temporal/common/interceptors/WorkflowInboundCallsInterceptor.java
@@ -21,8 +21,8 @@
  * <p>The implementation must forward all the calls to {@code next}, but it may change the input
  * parameters.
  *
- * @see WorkerInterceptor#interceptWorkflow(WorkflowInboundCallsInterceptor) for a definition of
- *     "next" WorkflowInboundCallsInterceptor
+ * <p>See {@link WorkerInterceptor#interceptWorkflow(WorkflowInboundCallsInterceptor)} for a
+ * definition of "next" {@link WorkflowInboundCallsInterceptor}.
  */
 @Experimental
 public interface WorkflowInboundCallsInterceptor {
@@ -166,10 +166,11 @@ public Object getResult() {
    *
    * <p>The instance should be passed into the {next.init(newWorkflowOutboundCallsInterceptor)}.
    *
+   * <p>See {@link WorkerInterceptor#interceptWorkflow} for the definition of "next" {@link
+   * WorkflowInboundCallsInterceptor}.
+   *
    * @param outboundCalls an existing interceptor instance to be proxied by the interceptor created
    *     inside this method
-   * @see WorkerInterceptor#interceptWorkflow for the definition of "next"
-   *     WorkflowInboundCallsInterceptor
    */
   void init(WorkflowOutboundCallsInterceptor outboundCalls);