Skip to content

refactor(ui): remove dead rail unarchive - #5190

Open
Phoenix500526 wants to merge 1 commit into
apache:mainfrom
Phoenix500526:refactore/remove-dead-codes
Open

refactor(ui): remove dead rail unarchive#5190
Phoenix500526 wants to merge 1 commit into
apache:mainfrom
Phoenix500526:refactore/remove-dead-codes

Conversation

@Phoenix500526

Copy link
Copy Markdown
Contributor

Summary

Remove the unreachable task-rail Unarchive path. Archived sessions are
excluded before rail projection, so SessionRowActions.onUnarchive and its
rendering branch could not run in production.

Keep the active restore paths unchanged: Settings > Archived Tasks, the
session controller and IPC boundary, archived projects, and Work Board.

Fixes #5189

Verification

  • npm exec biome lint -- <9 changed files> — passed
  • npm run format:check — passed (2,075 files)
  • npm --workspace @maka/ui run build — passed
  • npm --workspace @maka/ui run test:dist — passed (430 tests)
  • npm --workspace @maka/desktop run typecheck — passed
  • Storybook smoke was not run; this removes an unreachable rail branch and
    does not change a rendered reachable state.

AI use

Select exactly one:

  • No generative tool made a substantive contribution
  • Generative tooling made a substantive contribution

Tool(s) and scope: OpenAI Codex analyzed reachability, implemented the cleanup,
updated fixtures, ran verification, and drafted this pull request.

Checklist

  • Tests cover the change and fail without it
  • Lint, format, typecheck and the affected suites pass locally

Does this PR entail a change in behavior?

  • Yes — described under Summary above
  • No

Archived tasks are filtered before rail projection.
The restore branch was unreachable, so the rail contract no longer
exposes it.

Closes apache#5189
Generated-by: OpenAI Codex
Signed-off-by: Jiawei Zhao <Phoenix500526@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

effort/S Under 100 readable lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor(ui): remove the unreachable task-rail unarchive path

1 participant