chore: Remove dashboard variables feature toggle

[pulpdrew]

Summary

This PR removes the dashboard variables feature toggle, so that the feature is enabled everywhere.

Screenshots or video

How to test on Vercel preview

Check that the feature is enabled in the preview environment.

References

• Linear Issue: Closes HDX-5148

[changeset-bot]

🦋 Changeset detected

Latest commit: 2b114dd

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages

Name	Type
@hyperdx/app	Patch
@hyperdx/api	Patch
@hyperdx/otel-collector	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
hyperdx-oss	Ready	Preview	Aug 27, 2026 4:27pm
hyperdx-storybook	Ready	Preview	Aug 27, 2026 4:27pm

[greptile-apps]

Greptile Summary

This PR removes the dashboard-variable feature toggle and enables dashboard variables unconditionally.

• Removes the environment/config flag and its associated hook and unit tests.
• Passes dashboard variables directly to dashboard tiles and editing interfaces.
• Updates Playwright configuration, E2E documentation, and release metadata.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename	Overview
packages/app/src/DBDashboardPage.tsx	Removes feature-flag gating, always supplies dashboard variables, and updates the tooltip to sentence case.
packages/app/src/config.ts	Removes the obsolete dashboard-variable configuration constant.
packages/app/playwright.config.ts	Removes the dashboard-variable setting from the development and build environments.
packages/app/src/hooks/useIsVariablesEnabled.ts	Deletes the hook made obsolete by unconditional feature availability.
packages/app/tests/e2e/features/dashboard-filter-variables.spec.ts	Updates test documentation to reflect unconditional dashboard-variable availability.

_{Reviews (4): Last reviewed commit: "Merge branch 'main' into cursor/remove-d..." | Re-trigger Greptile}

[github-actions]

E2E Test Results

✅ All tests passed • 323 passed • 1 skipped • 1320s

Status	Count
✅ Passed	323
❌ Failed	0
⚠️ Flaky	0
⏭️ Skipped	1

Tests ran across 4 shards in parallel.

View full report →

[github-actions]

🔵 Tier 2 — Low Risk

Small, isolated change with no API route or data model modifications.

Why this tier:

• Standard feature/fix — introduces new logic or modifies core functionality

Review process: AI review + quick human skim (target: 5–15 min). Reviewer validates AI assessment and checks for domain-specific concerns.
SLA: Resolve within 4 business hours.

Stats

• Production files changed: 5
• Production lines changed: 46 (+ 51 in test files, excluded from tier calculation)
• Branch: cursor/remove-dashboard-variables-toggle-5e76
• Author: pulpdrew

To override this classification, remove the review/tier-2 label and apply a different review/tier-* label. Manual overrides are preserved on subsequent pushes.

[github-actions]

Deep Review

✅ No critical issues found.

This is a clean, mechanical removal of the dashboard-variables feature toggle. All consumers of the variables prop are typed to accept it optionally and previously handled undefined gracefully, so passing variables unconditionally is type-safe and behavior-preserving. The renderTileComponent useMemo dependency array correctly drops the removed showFilterVariableOptions while retaining variables, and grep confirms no dangling references to any removed symbol (IS_DASHBOARD_VARIABLES_ENABLED, useIsVariablesEnabled, showFilterVariableOptions). The showVariableOptions prop on DashboardFiltersModal is not vestigial — ServicesDashboardPage.tsx still passes showVariableOptions={false}. The prior tooltip title-case comment is addressed (label="Edit filters and variables").

🔵 P3 nitpicks (1)

• packages/app/playwright.config.ts:109 — the non-USE_DEV branch still passes NEXT_PUBLIC_ENABLE_DASHBOARD_VARIABLES=true in the yarn start segment, an incomplete removal that is now dead config since config.ts no longer reads the env var (it was already stripped from the dev branch and the yarn build segment).
  • Fix: Remove the leftover NEXT_PUBLIC_ENABLE_DASHBOARD_VARIABLES=true token from the yarn start portion of the build-branch command.
  • _{maintainability, correctness, testing, project-standards}

---

Reviewers (6): correctness, maintainability, testing, project-standards, kieran-typescript, previous-comments.

Testing gaps: The deleted useIsVariablesEnabled unit test covered deleted code, and the always-on path is exercised by the existing @full-stack e2e spec dashboard-filter-variables.spec.ts; no new coverage is required.