fix(next-js): disable health check during build phase

Author: NathanFlurry

.github/workflows/claude-code-review.yml

@@ -51,6 +51,24 @@ jobs:
 
             Use `gh pr comment` with your Bash tool to leave your review as a comment on the PR.
 
+          append_system_prompt: |
+            When formatting your output in GitHub comments, use collapsible sections with <details> tags instead of markdown headings.
+
+            Format each major section as:
+            ```
+            <details>
+            <summary>Section Title</summary>
+
+            Section content here. You can include:
+            - Lists
+            - Code blocks
+            - Any other markdown
+
+            </details>
+            ```
+
+            This makes the output more compact and easier to navigate.
+
           # See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md
           # or https://docs.claude.com/en/docs/claude-code/cli-reference for available options
           claude_args: '--allowed-tools "Bash(gh issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*)"'

.github/workflows/claude.yml

@@ -40,6 +40,24 @@ jobs:
           additional_permissions: |
             actions: read
 
+          append_system_prompt: |
+            When formatting your output in GitHub comments, use collapsible sections with <details> tags instead of markdown headings.
+
+            Format each major section as:
+            ```
+            <details>
+            <summary>Section Title</summary>
+
+            Section content here. You can include:
+            - Lists
+            - Code blocks
+            - Any other markdown
+
+            </details>
+            ```
+
+            This makes the output more compact and easier to navigate.
+
           # Optional: Give a custom prompt to Claude. If this is not specified, Claude will perform the instructions specified in the comment that tagged it.
           # prompt: 'Update the pull request description to include a summary of changes.'
 

packages/rivetkit/src/client/client.ts

@@ -4,6 +4,7 @@ import type { Encoding } from "@/actor/protocol/serde";
 import type { ManagerDriver } from "@/driver-helpers/mod";
 import type { ActorQuery } from "@/manager/protocol/query";
 import type { Registry } from "@/registry/mod";
+import { getEnvUniversal } from "@/utils";
 import type { ActorActionFunction } from "./actor-common";
 import {
 	type ActorConn,

packages/rivetkit/src/remote-manager-driver/mod.ts

@@ -16,7 +16,7 @@ import type {
 } from "@/driver-helpers/mod";
 import type { Encoding, UniversalWebSocket } from "@/mod";
 import { uint8ArrayToBase64 } from "@/serde";
-import { combineUrlPath } from "@/utils";
+import { combineUrlPath, getEnvUniversal } from "@/utils";
 import { sendHttpRequestToActor } from "./actor-http-client";
 import {
 	buildWebSocketProtocols,
@@ -56,6 +56,14 @@ export class RemoteManagerDriver implements ManagerDriver {
 	#metadataPromise: Promise<void> | undefined;
 
 	constructor(runConfig: ClientConfig) {
+		// Disable health check if in Next.js build phase since there is no `/metadata` endpoint
+		//
+		// See https://github.com/vercel/next.js/blob/5e6b008b561caf2710ab7be63320a3d549474a5b/packages/next/shared/lib/constants.ts#L19-L23
+		if (getEnvUniversal("NEXT_PHASE") === "phase-production-build") {
+			logger().info("detected next.js build phase, disabling health check");
+			runConfig.disableHealthCheck = true;
+		}
+
 		this.#config = runConfig;
 
 		// Perform metadata check if enabled