Backport #29181 task and incident utility split to 1.13

[shah-harshit]

Summary

• Direct stacked backport for OSS sequence seq37: perf(ui): split task, incident, and data quality utilities #29181.
• Uses the already reviewed 1.13 backport resolution from Backport task and incident utility split to 1.13 #30793.
• This PR is stacked on the previous sequence branch so it can be reviewed and merged in ticket order.

Tracking

• Source PR(s): perf(ui): split task, incident, and data quality utilities #29181
• Source commit(s): 5ae863e
• Common branch backport PR: Backport task and incident utility split to 1.13 #30793
• Cherry-picked commit: 33b3451
• Depends on: ui/direct-1.13-seq35-36-glossary-service-lazy

Testing

• git diff --check HEAD~1..HEAD

[gitar-bot]

💡 Quality: Task form schema cache uses bare Map instead of lru-cache

taskFormSchemaCache is a bare Map used as a module-level cache keyed by ${taskType}::${taskCategory} with no eviction bound. The project's caching guideline requires lru-cache over a bare Map/plain object. Impact is low here since the key space (task type × category) is finite, but consider switching to lru-cache for consistency with the codebase convention.

Use lru-cache with a bounded size instead of a bare Map.:

import { LRUCache } from 'lru-cache';

const taskFormSchemaCache = new LRUCache<
  string,
  Promise<TaskFormSchema | undefined>
>({ max: 100 });

• Apply fix

_{Check the box to apply the fix or reply for a change | Was this helpful? React with 👍 / 👎}

[gitar-bot]

Code Review 👍 Approved with suggestions 0 resolved / 1 findings

Backports the task and incident utility split from #29181 to branch 1.13. Consider replacing the bare Map in the task form schema cache with lru-cache to prevent unbounded memory growth.

💡 Quality: Task form schema cache uses bare Map instead of lru-cache

📄 openmetadata-ui/src/main/resources/ui/src/utils/TaskFormSchemaUtils.ts:59-62

taskFormSchemaCache is a bare Map used as a module-level cache keyed by ${taskType}::${taskCategory} with no eviction bound. The project's caching guideline requires lru-cache over a bare Map/plain object. Impact is low here since the key space (task type × category) is finite, but consider switching to lru-cache for consistency with the codebase convention.

Use lru-cache with a bounded size instead of a bare Map.

import { LRUCache } from 'lru-cache';

const taskFormSchemaCache = new LRUCache<
  string,
  Promise<TaskFormSchema | undefined>
>({ max: 100 });

🤖 Prompt for agents

Code Review: Backports the task and incident utility split from #29181 to branch 1.13. Consider replacing the bare Map in the task form schema cache with lru-cache to prevent unbounded memory growth.

1. 💡 Quality: Task form schema cache uses bare Map instead of lru-cache
   Files: openmetadata-ui/src/main/resources/ui/src/utils/TaskFormSchemaUtils.ts:59-62

   `taskFormSchemaCache` is a bare `Map` used as a module-level cache keyed by `${taskType}::${taskCategory}` with no eviction bound. The project's caching guideline requires `lru-cache` over a bare `Map`/plain object. Impact is low here since the key space (task type × category) is finite, but consider switching to `lru-cache` for consistency with the codebase convention.

   Fix (Use lru-cache with a bounded size instead of a bare Map.):
   import { LRUCache } from 'lru-cache';
   
   const taskFormSchemaCache = new LRUCache<
     string,
     Promise<TaskFormSchema | undefined>
   >({ max: 100 });

Options

Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Compact

gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar | Powered by Gitar — free for open source}

[github-actions]

❌ UI Checkstyle Failed

❌ ESLint + Prettier + Organise Imports (src)

One or more source files have linting or formatting issues.

❌ Licence Header

One or more files are missing or have an outdated Apache 2.0 licence header.

Affected files

• openmetadata-ui/src/main/resources/ui/src/components/ActivityFeed/TaskFeedCard/TaskFeedCardFromTask.component.test.tsx
  • openmetadata-ui/src/main/resources/ui/src/components/NotificationBox/NotificationFeedCard.test.tsx
  • openmetadata-ui/src/main/resources/ui/src/pages/TasksPage/shared/TaskPayloadSchemaFields.tsx
  • openmetadata-ui/src/main/resources/ui/src/utils/TaskFormSchemaUtils.ts

❌ Tailwind Audit

Hardcoded Tailwind values found. Use a design-system utility (run yarn tw-audit:report for the token each value maps to).

❌ Antd + Less Deprecation Guard

A new antd import or new .less file was added. Use UntitledUI + Tailwind for new work.

Affected files

at Function._resolveFilename (node:internal/modules/cjs/loader:1401:15)
at defaultResolveImpl (node:internal/modules/cjs/loader:1057:19)
at resolveForCJSWithHooks (node:internal/modules/cjs/loader:1062:22)
at Function._load (node:internal/modules/cjs/loader:1211:37)
at TracingChannel.traceSync (node:diagnostics_channel:322:14)
at wrapModuleLoad (node:internal/modules/cjs/loader:235:24)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:171:5)
at node:internal/main/run_main_module:36:49 {

---

Fix locally (fast - only checks files changed in this branch):

make ui-checkstyle-changed