perf(react): hoist static render values

[j15z]

Summary

Four values that do not depend on a render or request are now created once at module load. This removes repeated allocation and gives consumers a stable reference without changing their values.

The two remaining React Doctor warnings are intentional: scrollingBoxes and frozenBoxes are filled during each render, so hoisting them would leak stale collaborator selections.

Metric	Before	This PR
Static-value warnings	6	2 confirmed false positives

Type of Change

• Bug fix
• New feature
• Breaking change
• Documentation
• Other: Performance and maintainability improvement

Testing

• npx react-doctor@latest --verbose: only the 2 mutable-array false positives remain
• bun run check:api-validation passed
• Biome passed for the 4 files changed in this PR
• Patch equivalence was verified after rebasing this PR onto staging

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Screenshots/Videos

Not applicable. The hoisted values and their runtime contents are unchanged.

Post-Deploy Monitoring & Validation

No additional operational monitoring is required. These constants preserve their previous values and do not change user-visible behavior.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
docs	Ready	Preview	Aug 19, 2026 6:15pm

[cursor]

PR Summary

Low Risk
Refactor-only constant hoisting with identical runtime values; no auth, data, or API logic changes beyond moving existing maps/objects to module scope.

Overview
Hoists static values that do not depend on render or request to module-level constants so they are allocated once and consumers get stable references.

Docs search API — LOCALE_MAP replaces a per-request locale→PostgreSQL text-search config map (including ja/zh → simple).

Comparisons landing page — BREADCRUMB_JSON_LD replaces breadcrumb JSON-LD built inside the page component.

Logs line chart — CHART_PADDING replaces an inline padding object in the chart component.

Workflow tag dropdown — EMPTY_VARIABLE_INFO_MAP replaces a per-useMemo empty map in tag computation early returns (aligned with existing EMPTY_VARIABLES).

No user-visible or API behavior changes; this targets React Doctor static-value warnings and minor render-time allocation.

^{Reviewed by Cursor Bugbot for commit fc23bed. Bugbot is set up for automated code reviews on this repo. Configure here.}

[greptile-apps]

Greptile Summary

The PR hoists static render and request values to module scope to avoid recreating them during execution.

• Hoists the documentation search locale map.
• Hoists comparison-page breadcrumb structured data.
• Reuses stable chart padding and empty variable metadata constants.

Confidence Score: 4/5

The PR appears safe to merge, with only a non-blocking TypeScript convention issue affecting the newly hoisted object constants.

The hoisted values preserve existing runtime behavior and are not mutated, but three new object constants lack the repository-required const assertions.

Files Needing Attention: apps/sim/app/(landing)/comparisons/page.tsx, apps/docs/app/api/search/route.ts, and tag-dropdown.tsx

Important Files Changed

Filename	Overview
apps/docs/app/api/search/route.ts	Hoists the locale-to-PostgreSQL-search-configuration map without changing lookup behavior, but omits the required const assertion.
apps/sim/app/(landing)/comparisons/page.tsx	Hoists static breadcrumb JSON-LD while preserving rendered data, but the constant lacks the repository-required const assertion.
apps/sim/app/workspace/[workspaceId]/logs/components/dashboard/components/line-chart/line-chart.tsx	Hoists immutable chart padding with an appropriate const assertion; all consumers remain read-only.
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/tag-dropdown/tag-dropdown.tsx	Safely shares an empty read-only-in-practice metadata map, though its declaration omits the required const assertion.

_{Reviews (1): Last reviewed commit: "perf(react): hoist static render values" | Re-trigger Greptile}

[greptile-apps]

Const assertions omitted

The newly hoisted BREADCRUMB_JSON_LD object lacks the repository-required as const assertion, leaving its inferred properties unnecessarily widened and mutable. The same changed-code pattern appears in LOCALE_MAP and EMPTY_VARIABLE_INFO_MAP.

Context Used: CLAUDE.md (source)

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!