chore(redux-utils): remove unused notImplementedOriginalReduxThunk factory

[mroz22]

Summary

Removes the unused notImplementedOriginalReduxThunk factory from @suite-common/redux-utils. Confirmed no callers across the monorepo.

Split out from the staging dead-code PR #27551 (suite-common/* batch).

Related PRs

Sibling PRs in this batch (each removes one piece of dead code from suite-common/*):

• chore(wallet-utils): remove unused readUtxoOutpoint function #27836 — chore(wallet-utils): remove unused readUtxoOutpoint function
• chore(suite-utils): remove unused PollingController.isScheduled method #27837 — chore(suite-utils): remove unused PollingController.isScheduled method
• chore(connect-popup): remove unused selectWalletConnectAppPermissions selector #27838 — chore(connect-popup): remove unused selectWalletConnectAppPermissions selector
• chore(suite-types): remove unused GitHub API response types #27839 — chore(suite-types): remove unused GitHub API response types
• chore(analytics): drop unnecessary export from ANALYTICS_EVENT_NAME_KEBAB_SEGMENT #27840 — chore(analytics): drop unnecessary export from ANALYTICS_EVENT_NAME_KEBAB_SEGMENT
• chore(wallet-config): remove unused hasNetworkSettlementLayer and isProdStakingNetworkSymbol #27841 — chore(wallet-config): remove unused hasNetworkSettlementLayer and isProdStakingNetworkSymbol

Parent staging PR: #27551

Test plan

• CI typecheck and tests pass

🤖 LLM Test Recommendations

Summary: The changed file suite-common/redux-utils/src/notImplemented.ts is a low-level utility in the redux-utils package that likely provides a helper for marking unimplemented functionality (e.g., throwing errors for abstract methods). No tests statically map to this file, and none of the LLM-analyzed E2E tests reference notImplemented or redux-utils in their source hints. This is a foundational utility whose changes would most likely manifest as build-time or unit-test failures rather than E2E behavioral regressions. No E2E tests are recommended; unit tests and type checks should provide sufficient coverage.

Changed files (1)

• suite-common/redux-utils/src/notImplemented.ts

No test recommendations found.

⚠️ Changes with no test coverage (1)

• suite-common/redux-utils/src/notImplemented.ts

Updated: 2026-05-17T06:43:25.218Z

🔍 Currents Test Results

🔍 Suite desktop test results: View in Currents

🔍 Suite web test results: View in Currents

🔍 Suite native android test results: View in Currents

🔒 Quarantined E2E Tests

Trezor Suite (web) — 4 test(s)

Test	Type
Quarantine test: "Onboarding - create wallet,Success (basic)"	🙋 manual
Quarantine test: "Database migration,Db migration between: release/22.5/web => develop/web"	🙋 manual
Quarantine test: "Multiple sessions,Session overtaken by another"	🙋 manual
Quarantine test: "Recovery T2T1 - dry run,Recovery after partial recovery"	🙋 manual

Updated: 2026-05-17T06:48:19.950Z • 4 test(s) total

Trezor Suite (desktop) — 2 test(s)

Test	Type
Onboarding - create wallet > Success (basic)	🤖 auto
Quarantine test: "Multiple sessions,Session overtaken by another"	🙋 manual

Updated: 2026-05-17T06:47:22.461Z • 2 test(s) total

🌐 Preview deployments

🌐 Suite Web preview: https://dev.suite.sldev.cz/suite-web/mroz22/dead-code-redux-utils/web/

[Copilot]

Pull request overview

Removes an unused Redux utility factory from @suite-common/redux-utils, reducing dead code in a shared internal package.

Changes:

• Deleted notImplementedOriginalReduxThunk from notImplemented.ts.
• Verified no remaining references to the removed export in the repository.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.