refactor: extract markDeclNeedsUseThemeHook to shared types module

Consolidate the duplicated __runtimeCall needsUseThemeHook pattern
into a single shared helper in types.ts, removing local copies from
inline-style-props.ts and rule-interpolated-declaration.ts.

Co-authored-by: Kenneth Skovhus <skovhus@users.noreply.github.com>

Author: cursoragent

src/internal/lower-rules/inline-style-props.ts

@@ -21,7 +21,7 @@ import { buildPseudoMediaPropValue } from "./variant-utils.js";
 import { extractStaticPartsForDecl } from "./interpolations.js";
 import { cssDeclarationToStylexDeclarations } from "../css-prop-mapping.js";
 import { isStylexShorthandCamelCase } from "../stylex-shorthands.js";
-import { ensureShouldForwardPropDrop } from "./types.js";
+import { ensureShouldForwardPropDrop, markDeclNeedsUseThemeHook } from "./types.js";
 import { cloneAstNode, getFunctionBodyExpr } from "../utilities/jscodeshift-utils.js";
 import { cssPropertyToIdentifier, makeCssPropKey } from "./shared.js";
 import { styleKeyWithSuffix } from "../transform/helpers.js";
@@ -366,17 +366,3 @@ export function handleInlineStyleValueFromProps(ctx: InlineStyleFromPropsContext
 function hasSimpleIdentifierParam(expr: { params?: Array<{ type?: string }> }): boolean {
   return expr.params?.length === 1 && expr.params[0]?.type === "Identifier";
 }
-
-function markDeclNeedsUseThemeHook(decl: StyledDecl): void {
-  if (!decl.needsUseThemeHook) {
-    decl.needsUseThemeHook = [];
-  }
-  if (!decl.needsUseThemeHook.some((entry) => entry.themeProp === "__runtimeCall")) {
-    decl.needsUseThemeHook.push({
-      themeProp: "__runtimeCall",
-      trueStyleKey: null,
-      falseStyleKey: null,
-    });
-  }
-  decl.needsWrapperComponent = true;
-}

src/internal/lower-rules/rule-interpolated-declaration.ts

@@ -36,7 +36,11 @@ import {
   tryHandleInterpolatedStringValue,
   wrapExprWithStaticParts,
 } from "./interpolations.js";
-import { ensureShouldForwardPropDrop, literalToStaticValue } from "./types.js";
+import {
+  ensureShouldForwardPropDrop,
+  literalToStaticValue,
+  markDeclNeedsUseThemeHook,
+} from "./types.js";
 import {
   buildTemplateWithStaticParts,
   collectPropsFromArrowFn,
@@ -232,16 +236,7 @@ export function handleInterpolatedDeclaration(args: InterpolatedDeclarationConte
         : baseRuntimeExpr;
 
     if (hasThemeAccess) {
-      if (!decl.needsUseThemeHook) {
-        decl.needsUseThemeHook = [];
-      }
-      if (!decl.needsUseThemeHook.some((entry) => entry.themeProp === "__runtimeCall")) {
-        decl.needsUseThemeHook.push({
-          themeProp: "__runtimeCall",
-          trueStyleKey: null,
-          falseStyleKey: null,
-        });
-      }
+      markDeclNeedsUseThemeHook(decl);
     }
 
     const outs = cssDeclarationToStylexDeclarations(d);
@@ -2804,20 +2799,6 @@ function tryHandleMultiSlotTernary(ctx: DeclProcessingState, d: CssDeclarationIR
   return true;
 }
 
-function markDeclNeedsUseThemeHook(decl: StyledDecl): void {
-  if (!decl.needsUseThemeHook) {
-    decl.needsUseThemeHook = [];
-  }
-  if (!decl.needsUseThemeHook.some((entry) => entry.themeProp === "__runtimeCall")) {
-    decl.needsUseThemeHook.push({
-      themeProp: "__runtimeCall",
-      trueStyleKey: null,
-      falseStyleKey: null,
-    });
-  }
-  decl.needsWrapperComponent = true;
-}
-
 /** CSS shorthand properties that cannot be represented as a single var() custom property. */
 const UNSUPPORTED_CUSTOM_PROP_SHORTHANDS = new Set([
   "border",

src/internal/lower-rules/types.ts

@@ -6,6 +6,20 @@ import type { StyledDecl } from "../transform-types.js";
 
 export { literalToStaticValue } from "../utilities/jscodeshift-utils.js";
 
+export function markDeclNeedsUseThemeHook(decl: StyledDecl): void {
+  if (!decl.needsUseThemeHook) {
+    decl.needsUseThemeHook = [];
+  }
+  if (!decl.needsUseThemeHook.some((entry) => entry.themeProp === "__runtimeCall")) {
+    decl.needsUseThemeHook.push({
+      themeProp: "__runtimeCall",
+      trueStyleKey: null,
+      falseStyleKey: null,
+    });
+  }
+  decl.needsWrapperComponent = true;
+}
+
 export function ensureShouldForwardPropDrop(decl: StyledDecl, propName: string): void {
   // Ensure we generate a wrapper so we can consume the styling prop without forwarding it to DOM.
   decl.needsWrapperComponent = true;