Part of Phase 1 (1b): Archive Notifier migrated from Redux to Zustand

[Parship12]

Which problem is this PR solving?

• Resolves part of Tracking: ADR 0004 Migrate from Redux to Zustand + TanStack React Query #3657

Description of the changes

• New files:

  • src/stores/archive-store.ts: Zustand store owning archives: Record<string, TraceArchive>, plus submitArchiveTrace(traceId) (async, calls JaegerAPI directly, handles loading/success/error transitions) and acknowledge(traceId).
  • src/stores/archive-store.test.ts

• Modified:

  • TracePage/index.tsx
  • configure-store.ts: removed archive reducer from combineReducers.
  • types/index.ts: removed archive: TracesArchive from ReduxState.
  • TracePage/index.test.jsx

• Deleted:

  • ArchiveNotifier/duck.ts
  • ArchiveNotifier/duck.test.js

How was this change tested?

• Unit tests
• Manually verified Archive button visibile... with local jaeger-ui.config.json (archiveStorage: true), clicked Archive on a trace, and confirmed the client issues POST /api/archive/<traceId>; backend returned 500 with “archive span storage was not configured” on my local Query (expected without archive storage).

Checklist

• I have read https://github.com/jaegertracing/jaeger/blob/main/CONTRIBUTING_GUIDELINES.md
• I have signed all commits
• I have added unit tests for the new functionality
• I have run lint and test steps successfully: make lint test

AI Usage in this PR (choose one)

See AI Usage Policy.

• None: No AI tools were used in creating this PR
• Light: AI provided minor assistance (formatting, simple suggestions)
• Moderate: AI helped with code generation or debugging specific parts
• Heavy: AI generated most or all of the code changes

[Copilot]

Pull request overview

This pull request completes Phase 1b of the Redux-to-Zustand migration by moving the Archive Notifier state management from Redux to a dedicated Zustand store. The archive functionality (submitting trace archives and acknowledging completion) is now managed client-side through the new useArchiveStore hook, with all Redux-related code cleanly removed.

Changes:

• Created new Zustand store (src/stores/archive-store.ts) managing archive state with submitArchiveTrace and acknowledge actions
• Updated TracePage component to use the new Zustand store via hooks instead of Redux dispatch/state
• Removed Redux archive duck (ArchiveNotifier/duck.ts and its tests)
• Cleaned up Redux configuration and type definitions
• Updated all related tests to use the new store

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/stores/archive-store.ts	New Zustand store managing archive state and async submission logic
src/stores/archive-store.test.ts	Comprehensive tests for store actions and state transitions
src/components/TracePage/index.tsx	Migrated from Redux dispatch/selectors to Zustand hooks
src/components/TracePage/index.test.jsx	Updated tests to mock Zustand store instead of Redux actions
src/utils/configure-store.ts	Removed archive reducer from Redux combineReducers
src/types/index.ts	Removed archive field from ReduxState type
docs/adr/0004-state-management-strategy.md	Marked Phase 1b as complete

---

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.21%. Comparing base (f9226f8) to head (bb79a89).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3715      +/-   ##
==========================================
- Coverage   89.21%   89.21%   -0.01%     
==========================================
  Files         330      330              
  Lines        9900     9889      -11     
  Branches     2570     2566       -4     
==========================================
- Hits         8832     8822      -10     
+ Misses        926      925       -1     
  Partials      142      142              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

Pull request overview

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

---

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

[Copilot]

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated no new comments.

---

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