@superblocksteam/sdk 2.0.141-next.0 → 2.0.141-next.2

package/src/dev-utils/optimize-deps-config.mts

@@ -0,0 +1,96 @@
+import type { UserConfig } from "vite";
+
+/**
+ * Shared `optimizeDeps` config for both the pre-warm and activation Vite
+ * servers (see `dev-server.mts`).
+ *
+ * Vite includes the stringified `optimizeDeps` config in its dep cache hash, so
+ * both servers must use identical config or activation will discard the
+ * pre-warmed cache and re-run full dependency optimization.
+ *
+ * Lives in its own module (rather than inline in `dev-server.mts`) so the
+ * dep-pre-bundling contract can be asserted in tests without importing the
+ * entire dev-server module graph (vite + express + file-sync + telemetry).
+ * See `dev-server.optimize-deps.test.mts`.
+ */
+export const OPTIMIZE_DEPS_CONFIG: UserConfig["optimizeDeps"] = {
+  // Crawl app source (not just the HTML entry graph) during the cold-start dep
+  // scan so lazily/dynamically imported deps (`@radix-ui/*`, `lucide-react/*`
+  // icons behind route/code-splits) are pre-bundled up front. Without this,
+  // Vite discovers them at runtime and forces an "optimized dependencies
+  // changed. reloading" full-page reload that drops the editor websocket
+  // (close 1001) mid Clark generation (APPS-4576).
+  //
+  // `entries` is NOT part of Vite's optimizeDeps config hash (verified against
+  // Vite 6.4.2: `getConfigHash` in `dist/node/chunks/dep-*.js` hashes only
+  // `optimizeDeps.{include,exclude,esbuildOptions}`, never `entries`), so adding
+  // it cannot desync the pre-warm vs activation dep cache. Re-verify after a
+  // Vite upgrade: if `entries` ever enters the hash, the pre-warm cache would be
+  // discarded on activation whenever the two servers scan different source trees.
+  // `node_modules`/`outDir` are always excluded by Vite.
+  //
+  // The source globs cover every extension a generated app can import a heavy
+  // dep from (`.ts` utilities, not just `.tsx`/`.jsx` components), so a future
+  // `.ts` module importing a tree-shakeable dep is still pre-bundled. `**/*.html`
+  // preserves the default HTML-entry discovery; the `!` patterns keep the scan
+  // off the externalized custom-component dir and the drafts dir (both
+  // watch-ignored elsewhere) so churny user content can't slow or fail the
+  // cold-start scan.
+  //
+  // Node-side files must never become browser dep-scan entries. A generated
+  // app's `vite.config.ts` imports `@tailwindcss/vite`, and a fullstack app's
+  // `server/**` imports Node/native packages; when those files are scan entries
+  // esbuild follows their imports into build-time-only packages and aborts the
+  // whole browser optimize — `@tailwindcss/oxide` has no `.node` loader (see
+  // esbuildOptions below) and `lightningcss` fails to resolve its generated
+  // `../pkg` binding ("Could not resolve \"../pkg\""). Either crashes cold-start
+  // pre-warm for *every* tailwind-v4 app, so exclude config files and the server
+  // tree up front rather than trying to make esbuild tolerate every native dep.
+  entries: [
+    "**/*.html",
+    "**/*.{js,ts,jsx,tsx}",
+    "!custom/**",
+    "!.superblocks/**",
+    "!**/*.config.{js,ts,mjs,mts,cjs,cts}",
+    "!server/**",
+  ],
+  // Force-bundle deps the cold-start scan structurally cannot discover, so they
+  // aren't found at runtime and don't trigger their own reload (verified on an
+  // EE warm-pool box: without this, a freshly-generated app still reloads once
+  // on first render for the JSX runtime — APPS-4576).
+  //
+  // The scan's esbuild uses the DEFAULT (react) JSX runtime, so it emits
+  // `react/jsx-runtime` imports and pre-bundles those — but never
+  // `@superblocksteam/library/jsx-dev-runtime`, even though every generated app
+  // sets `jsxImportSource: "@superblocksteam/library"` and pulls that runtime in
+  // at serve time. Force-including it pre-bundles it up front. An unresolvable
+  // `include` entry is a no-op (Vite warns and skips), so an app that somehow
+  // lacks the library never fails cold-start over this.
+  include: ["@superblocksteam/library/jsx-dev-runtime"],
+  // `@superblocksteam/sdk-api` is a VIRTUAL module the `sb-sdk-api-entry-point`
+  // plugin serves (not a real package), imported by generated app code. Left to
+  // the optimizer it gets discovered as a "new dependency" on first import and
+  // forces a reload; excluding it keeps the dep optimizer from ever trying to
+  // pre-bundle or re-optimize it (the plugin serves it as-is). `include` cannot
+  // fix this one — there is no real file for esbuild to pre-bundle.
+  exclude: ["@superblocksteam/sdk-api"],
+  esbuildOptions: {
+    supported: {
+      destructuring: true,
+    },
+    // Tolerate native `.node` addons reached during the cold-start scan. The
+    // widened `entries` glob crawls Node-side app files too (a fullstack app's
+    // `vite.config.ts` imports `@tailwindcss/vite`, which pulls in the native
+    // `@tailwindcss/oxide` `tailwindcss-oxide.<platform>.node`). esbuild has no
+    // loader for `.node`, so bundling such a dep aborts the whole optimize with
+    // "Build failed ... No loader is configured for \".node\" files", which
+    // stops the dev server from serving and leaves the editor websocket stuck
+    // "released" after a respawn (APPS-4576, PR #19858 CI). A native addon
+    // reached during the *browser* dep optimize is always build-time-only and
+    // never executed in the browser, so emptying it is safe and keeps the
+    // cold-start scan from crashing on any native dependency, not just oxide.
+    loader: {
+      ".node": "empty",
+    },
+  },
+};

package/src/dev-utils/prepare-in-place-restart.mts

@@ -3,11 +3,11 @@ import { RESTART_REASON_FLIP_RESTART } from "@superblocksteam/library-shared/res
 /** Upper bound on the pre-exit IPC handshake to the parent supervisor. */
 const RESTART_REASON_IPC_TIMEOUT_MS = 1000;
 
-export interface PrepareInPlaceRestartLogger {
+interface PrepareInPlaceRestartLogger {
   warn(message: string, meta?: Record<string, unknown>): void;
 }
 
-export interface PrepareInPlaceRestartServices {
+interface PrepareInPlaceRestartServices {
   aiService?: {
     chatSessionStore: { invalidateCache(): void };
   };
@@ -17,7 +17,7 @@ export interface PrepareInPlaceRestartServices {
   };
 }
 
-export interface PrepareInPlaceRestartOptions {
+interface PrepareInPlaceRestartOptions {
   flipRestart?: true;
   logger?: PrepareInPlaceRestartLogger;
 }

package/src/dev-utils/vite-dev-server-diagnostics.mts

@@ -6,7 +6,7 @@
 /** Maximum characters of raw Vite/esbuild output stored on the error meta stack field. */
 export const VITE_ERROR_RAW_LOG_MAX_CHARS = 12_000;
 
-export type ViteBuildErrorDiagnostic =
+type ViteBuildErrorDiagnostic =
   | {
       category: "module_resolve";
       /** Package specifier esbuild could not resolve, when detected. */

package/src/index.ts

@@ -70,13 +70,6 @@ export {
 
 export { AUTO_UPGRADE_EXIT_CODE } from "./cli-replacement/automatic-upgrades.js";
 export { RESTART_EXIT_CODE } from "./dev-utils/dev-server.mjs";
-// Surfaces the canonical restart-contract markers alongside RESTART_EXIT_CODE so
-// the builtins-only CLI parent copies can be drift-checked against it in tests.
-export {
-  RESTART_REASON_ENV,
-  RESTART_REASON_FLIP_RESTART,
-  RESTART_SUPERVISED_ENV,
-} from "@superblocksteam/library-shared/restart-contract";
 
 export { createDevServer, preWarmViteCache } from "./dev-utils/dev-server.mjs";
 export {

package/src/parse-sdk-registry.mts

@@ -6,8 +6,7 @@ import {
   type RegistryKeyToEntry,
 } from "@superblocksteam/vite-plugin-file-sync/parsing";
 
-export type RegistryKeyToPath = Map<string, string>;
-export type { RegistryKeyToEntry };
+type RegistryKeyToPath = Map<string, string>;
 
 type PathExists = (p: string) => Promise<boolean>;
 const RESOLUTION_EXTENSIONS = [".ts", ".tsx", ".js"] as const;

package/src/sdk.ts

@@ -110,11 +110,6 @@ import type { DeploymentHistoryEntryDto, UserMeDto } from "./types/index.js";
 
 // Exporting here instead of index.ts because only sdk.ts is exposed outside in package.json
 export * from "./errors.js";
-export {
-  RESTART_REASON_ENV,
-  RESTART_REASON_FLIP_RESTART,
-  RESTART_SUPERVISED_ENV,
-} from "@superblocksteam/library-shared/restart-contract";
 
 export class SuperblocksSdk {
   token = "";

package/src/telemetry/attributes.ts

@@ -1,5 +1,3 @@
-// NOTE: @joeyagreco - this is how the "env" facet is determined in datadog: https://docs.datadoghq.com/opentelemetry/setup/collector_exporter/#3---configure-your-application
-export const ATTR_DEPLOYMENT_ENVIRONMENT = "deployment.environment";
 // NOTE: @joeyagreco - this can be used to determine if we are using mock-csb, staging-csb, prod-csb, etc
 export const ATTR_SUPERBLOCKS_BASE_URL = "superblocks.base_url";
 export const ATTR_SUPERBLOCKS_IS_CSB = "superblocks.is_csb";

package/src/telemetry/local-obs.ts

@@ -34,7 +34,7 @@ import { DeploymentTypeEnum } from "@superblocksteam/shared";
 const DEFAULT_LOCAL_OTEL_ENDPOINT = "http://localhost:4318";
 const DEFAULT_ML_APP_NAME = "superblocks-ai-code-gen";
 
-export interface LocalObsConfig {
+interface LocalObsConfig {
   /** Whether local observability mode is enabled */
   enabled: boolean;
   /** OTEL endpoint for traces, metrics, logs */
@@ -85,7 +85,7 @@ export function isLocalObsEnabled(): boolean {
  *
  * @returns LocalObsConfig if local obs is enabled, null otherwise
  */
-export function getLocalObsConfig(): LocalObsConfig | null {
+function getLocalObsConfig(): LocalObsConfig | null {
   if (!isLocalObsEnabled()) {
     return null;
   }

package/src/telemetry/logging.ts

@@ -73,7 +73,7 @@ export interface ErrorMeta {
  * facets), so values must stay low-cardinality-friendly (enums/ids/numbers),
  * not free-form text.
  */
-export type LogAttributes = Record<string, string | number | boolean>;
+type LogAttributes = Record<string, string | number | boolean>;
 
 export interface Logger {
   debug: (...messages: unknown[]) => void;