refactor(FR-2479): standardize confirmation UX with BAIConfirmModalWithInput and Popconfirm

[ironAiken2]

Resolves #6437 (FR-2479)

Summary

• Phase 1: Upgrade 6 destructive actions from modal.confirm() to BAIConfirmModalWithInput (typed-input confirmation)
  • ResourcePresetList.tsx — preset deletion
  • EndpointList.tsx — endpoint deletion
  • ShellScriptEditModal.tsx — script deletion
  • AIAgentPage.tsx — agent deletion and reset
  • ManageAppsModal.tsx — image reinstallation confirmation
  • ManageImageResourceLimitModal.tsx — image reinstallation confirmation
• Phase 2: Convert 2 TOTP toggle confirmations from modal.confirm() to Popconfirm
  • UserSettingModal.tsx — TOTP deactivation toggle
  • UserProfileSettingModal.tsx — TOTP deactivation toggle
• Phase 3: Remove orphaned PopConfirmWithInput.tsx (0 imports)

Test plan

• Delete a resource preset — typed confirmation modal appears, must type preset name
• Delete an endpoint — typed confirmation modal appears, must type endpoint name
• Delete a shell script — typed confirmation modal appears
• Delete/reset an AI agent — typed confirmation modal appears
• Modify apps on installed image — typed confirmation modal appears
• Modify resource limits on installed image — typed confirmation modal appears
• Toggle TOTP off (UserSettingModal) — Popconfirm appears inline
• Toggle TOTP off (UserProfileSettingModal) — Popconfirm appears inline
• Verify PopConfirmWithInput.tsx is removed

[github-actions]

Coverage report for ./react

St.❔	Category	Percentage	Covered / Total
🔴	Statements	8.64% (-0% 🔻)	1738/20107
🔴	Branches	7.84% (-0.01% 🔻)	1104/14085
🔴	Functions	5.19% (-0% 🔻)	283/5453
🔴	Lines	8.35% (-0% 🔻)	1630/19522

Test suite run success

847 tests passing in 38 suites.

Report generated by 🧪jest coverage report action from 500a0f7

[Copilot]

Pull request overview

This PR standardizes destructive-action confirmations across the WebUI by migrating modal.confirm() usage to BAIConfirmModalWithInput (typed-input confirmations) and switching TOTP deactivation confirmations to inline Popconfirm, while removing the now-orphaned PopConfirmWithInput component.

Changes:

• Replaced several destructive modal.confirm() flows with BAIConfirmModalWithInput across agents, endpoints, resource presets, shell scripts, and image modification modals.
• Converted TOTP “turn off” confirmation from modal-based confirm to inline Popconfirm in two settings modals.
• Deleted the unused PopConfirmWithInput.tsx component.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 15 comments.

Show a summary per file

File	Description
react/src/pages/AIAgentPage.tsx	Replace agent delete/reset confirmation with typed-input confirm modal.
react/src/components/EndpointList.tsx	Replace endpoint deletion confirmation with typed-input confirm modal.
react/src/components/ResourcePresetList.tsx	Replace resource preset deletion confirmation with typed-input confirm modal.
react/src/components/ShellScriptEditModal.tsx	Replace script deletion confirmation with typed-input confirm modal.
react/src/components/ManageAppsModal.tsx	Add typed-input confirmation when installed image modifications require reinstall.
react/src/components/ManageImageResourceLimitModal.tsx	Add typed-input confirmation when installed image resource limit changes require reinstall.
react/src/components/UserSettingModal.tsx	Switch TOTP deactivation confirmation to inline Popconfirm.
react/src/components/UserProfileSettingModal.tsx	Switch TOTP deactivation confirmation to inline Popconfirm.
react/src/components/PopConfirmWithInput.tsx	Remove orphaned duplicate confirmation component.

[ironAiken2]

• refactor(FR-2479): standardize confirmation UX with BAIConfirmModalWithInput and Popconfirm #6471 👈 (View in Graphite)
• main

---

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

• flow:merge-queue - adds this PR to the back of the merge queue
• flow:hotfix - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

[github-actions]

Coverage report for ./packages/backend.ai-ui

St.❔	Category	Percentage	Covered / Total
🟢	Statements	81.87%	429/524
🟡	Branches	70.43%	362/514
🟡	Functions	76.86%	93/121
🟢	Lines	83.19%	391/470

Test suite run success

319 tests passing in 13 suites.

Report generated by 🧪jest coverage report action from 500a0f7

[nowgnuesLee]

If this is our policy going forward, could you add it to .claude/rules?