feat(ui): extract TablePureUtils and TagsPureUtils#28673
Conversation
Move 43 pure functions from TableUtils.tsx to TablePureUtils.ts and 5 functions from TagsUtils.tsx to TagsPureUtils.ts. Both original files re-export all moved symbols for backward compatibility. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
🟡 Playwright Results — all passed (13 flaky)✅ 4260 passed · ❌ 0 failed · 🟡 13 flaky · ⏭️ 88 skipped
🟡 13 flaky test(s) (passed on retry)
How to debug locally# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip # view trace |
Convert interface, entity type, and other type-only symbols to import type in TablePureUtils.ts, TableUtils.tsx, TagsPureUtils.ts, TagsUtils.tsx. Runtime enum members (DataType, LabelType, State, TagSource, SettledStatus, SearchIndex) remain value imports. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Code Review ✅ ApprovedRefactors 48 pure functions from OptionsDisplay: compact → Showing less information. Comment with these commands to change:
Was this helpful? React with 👍 / 👎 | Gitar |



Summary
TableUtils.tsxinto newTablePureUtils.ts(tier/certification tag helpers, column utils, schema analysis, data type utils, table joins, field update helpers)TagsUtils.tsxinto newTagsPureUtils.ts(tag query filters, permission-based index exclusion, tag redirect links)TableUtils.tsxreduced from 1891 to ~1047 lines;TagsUtils.tsxreduced from 677 to ~380 linesRef: https://github.com/open-metadata/openmetadata-collate/issues/4230
Test plan
npx tsc --noEmitpasses (verified locally, 0 new errors)🤖 Generated with Claude Code