Skip to content

chore: remove dead StatusMessage component #69

Description

@tech-adrian

Area
frontend/src/components/ui/StatusMessage.tsx, frontend/src/components/ui/index.ts

Problem
StatusMessage (and its inferTone helper) is exported from the ui barrel but has zero usages anywhere in frontend/src/app/** — it appears to be a superseded precursor to the current toast-based status pattern (Toast.tsx, which duplicates the same inferTone-style logic at lines 26-37). Dead, unused code with a duplicated helper is easy to accidentally diverge from the real (toast) implementation if someone edits one copy and not the other.

Proposed change
Delete StatusMessage.tsx and its barrel export if truly unused (double-check with a repo-wide search first), and consolidate inferTone-style logic into a single shared helper used by Toast.tsx.

Acceptance Criteria

  • Confirmed no usages of StatusMessage anywhere in the app before deleting
  • Dead component and its export removed
  • Any duplicated tone-inference logic consolidated into one shared helper
  • Relevant tests (just test, pnpm test, cargo test, or nargo test as applicable) pass locally and in CI
  • A screenshot or short screen recording is attached to the PR showing the change working (pnpm build succeeding with the component removed) — required before this can be merged.

Open your PR against the dev branch, not main. All active development merges into dev.

Discuss this issue / coordinate work: join the DShield contributor Telegram group: https://t.me/+SiGHH24No9U2MDJk

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions