feat(ui): extract remaining pure utils

[shah-harshit]

Summary

Extract pure (no-React, no-side-effects) functions from 12 large utility files into dedicated *PureUtils.ts modules. All original files re-export every moved symbol for full backward compatibility.

New pure util files

New File	Extracted From	Key Exports
AgentsStatusWidgetPureUtils.ts	AgentsStatusWidgetUtils.tsx	getAgentLabelFromType, getAgentStatusLabelFromStatus, getAgentStatusSummary
Alerts/AlertsUtilPure.ts	Alerts/AlertsUtil.tsx	20 pure alert helpers (formatters, validators, name generators)
BlockEditorPureUtils.ts	BlockEditorUtils.ts	Pure block editor utils
ContainerDetailPureUtils.ts	ContainerDetailUtils.tsx	Pure container detail utils
ContextCenterPureUtils.ts	ContextCenterUtils.tsx	Pure context center utils
CuratedAssetsPureUtils.ts	CuratedAssetsUtils.tsx	Pure curated assets utils
CustomizableLandingPagePureUtils.ts	CustomizableLandingPageUtils.tsx	Pure landing page utils
CustomizePage/CustomizePagePureUtils.ts	CustomizePage/CustomizePageUtils.ts	Pure customize page utils
DataAssetSummaryPanelPureUtils.ts	DataAssetSummaryPanelUtils.tsx	getTableFieldsFromTableDetails, getUsageData
CSV/CSVPureUtils.ts	CSV/CSV.utils.tsx	Pure CSV processing utils
Lineage/LineagePureUtils.ts	Lineage/LineageUtils.tsx	Pure lineage utils
MetricEntityUtils/MetricPureUtils.ts	MetricEntityUtils/MetricUtils.tsx	Pure metric utils
UsersPureUtils.ts	Users.util.tsx	Pure user utils

Guidelines followed

• Pure utils: no React imports, no JSX, no API calls, no singletons
• All original files re-export moved symbols (export { fn } from './XxxPureUtils')
• import type used for all type-only imports
• No consumer changes required (re-exports ensure backward compat)

Test plan

• npx tsc --noEmit passes (no new errors in changed files)
• UI checkstyle passes
• Existing tests continue to pass (no import paths broken)

🤖 Generated with Claude Code

[github-actions]

❌ PR checklist incomplete

This PR cannot be merged until the following are addressed on its linked issue:

• No GitHub issue is linked. Add a closing reference such as Fixes #12345 to the PR description (accepted keywords: Fixes, Closes, Resolves).

The fields live on the linked issue in the Shipping project (open the issue → right sidebar → Projects). After you set them, re-run this check (or push a commit) — issue/project changes do not re-trigger it automatically.

Maintainers can bypass this check by adding the skip-pr-checks label.

[github-actions]

Hi there 👋 Thanks for your contribution!

The OpenMetadata team will review the PR shortly! Once it has been labeled as safe to test, the CI workflows
will start executing and we'll be able to make sure everything is working as expected.

Let us know if you need any help!

[github-actions]

Jest test Coverage

UI tests summary

Lines	Statements	Branches	Functions
	63.17% (67593/106993)	44.09% (37112/84155)	46.51% (11217/24115)

[sonarqubecloud]

Quality Gate passed for 'open-metadata-ui'

Issues
1453 New issues
0 Accepted issues

Measures
26 Security Hotspots
0.0% Coverage on New Code
3.6% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

🟡 Playwright Results — all passed (13 flaky)

✅ 4270 passed · ❌ 0 failed · 🟡 13 flaky · ⏭️ 88 skipped

Shard	Passed	Failed	Flaky	Skipped
✅ Shard 1	301	0	0	4
🟡 Shard 2	804	0	2	9
🟡 Shard 3	801	0	3	8
🟡 Shard 4	843	0	4	12
🟡 Shard 5	720	0	1	47
🟡 Shard 6	801	0	3	8

🟡 13 flaky test(s) (passed on retry)

• Features/DataQuality/DataQuality.spec.ts › TestCase filters (shard 2, 1 retry)
• Features/Glossary/GlossaryWorkflow.spec.ts › should start term as Draft when glossary has reviewers (shard 2, 2 retries)
• Features/RTL.spec.ts › Verify Following widget functionality (shard 3, 1 retry)
• Features/Table.spec.ts › Table pagination with sorting should works (shard 3, 1 retry)
• Features/UserProfileOnlineStatus.spec.ts › Should show online status badge on user profile for active users (shard 3, 1 retry)
• Flow/PersonaFlow.spec.ts › Set default persona for team should work properly (shard 4, 1 retry)
• Pages/CustomProperties.spec.ts › Date Time (shard 4, 1 retry)
• Pages/CustomProperties.spec.ts › Set & Update all CP types on glossaryTerm (shard 4, 1 retry)
• Pages/CustomProperties.spec.ts › Should clear search and show all properties for apiCollection in right panel (shard 4, 1 retry)
• Pages/ExplorePageRightPanel_KnowledgeCenter.spec.ts › Should remove user owner for knowledgeCenter (shard 5, 1 retry)
• Pages/Lineage/LineageFilters.spec.ts › Verify lineage service type filter selection (shard 6, 1 retry)
• Pages/Lineage/LineageFilters.spec.ts › Verify lineage schema filter selection (shard 6, 1 retry)
• Pages/ODCSImportExport.spec.ts › Multi-object ODCS contract - object selector shows all schema objects (shard 6, 1 retry)

📦 Download artifacts

How to debug locally

# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip    # view trace

[gitar-bot]

Code Review ✅ Approved 1 resolved / 1 findings

Refactors 13 utility files by extracting pure logic into dedicated modules with backward-compatible re-exports. Address the duplicate and dead code introduced in BlockEditorPureUtils to ensure consistency with the other modules.

✅ 1 resolved

✅ Quality: BlockEditorPureUtils adds duplicate/divergent + dead code

📄 openmetadata-ui/src/main/resources/ui/src/utils/BlockEditorPureUtils.ts:56-70 📄 openmetadata-ui/src/main/resources/ui/src/utils/BlockEditorPureUtils.ts:92-106 📄 openmetadata-ui/src/main/resources/ui/src/utils/BlockEditorPureUtils.ts:131-145 📄 openmetadata-ui/src/main/resources/ui/src/utils/BlockEditorUtils.ts:14-17 📄 openmetadata-ui/src/main/resources/ui/src/utils/BlockEditorUtils.ts:187
Unlike the other extractions (which move a function and re-export it for backward compatibility), BlockEditorPureUtils.ts introduces a new copy of isHTMLString while the original isHTMLString remains defined in BlockEditorUtils.ts:187 and is NOT re-exported from the pure file. So there are now two independent implementations of isHTMLString that can drift apart. Additionally, formatContentForClient and the private _convertMarkdownFormatToHtmlString introduced in the pure file are not re-exported by BlockEditorUtils.ts and a codebase search finds no consumers of formatContentForClient — they are effectively dead code (and _convertMarkdownFormatToHtmlString duplicates the markdown-to-HTML logic already living in BlockEditorUtils.ts's formatContent). This contradicts the PR's stated guideline that original files re-export every moved symbol, and creates maintenance risk where the two copies diverge. Recommend either: (a) have BlockEditorUtils.ts import/re-export isHTMLString from the pure module and delete the local copy, and wire up formatContentForClient to its intended caller; or (b) drop the currently-unused formatContentForClient/isHTMLString/_convertMarkdownFormatToHtmlString from the pure file until they are actually consumed.

Options

Display: compact → Showing less information.

Comment with these commands to change:

Compact

gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar}