guettli/sharedinbox
1
0
Fork 0
Code Issues 21 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

feat: add undo log detail view #461

Merged
guettlibot merged 1 commits from refs/pull/461/head into main 2026-06-06 03:43:18 +00:00
Conversation 0 Commits 1 Files Changed 6
+582 -4664
guettlibot commented 2026-06-05 22:24:18 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

Summary

  • Tapping a row in the Undo Log list opens a new UndoLogDetailScreen
  • Detail screen shows: account ID, action type (with icon/colour), timestamp, source folder, destination folder (move only), and a list of all emails in the transaction (subject + sender)
  • Navigation uses go_router nested route /accounts/undo-log/:actionId with state.extra to pass the UndoAction object
  • AppBar has an Undo button that calls the existing undo service and pops back

Also fixed

  • flake.nix: replaced the broken dagger/nix 0.20.8 Nix wrapper (infinite self-exec loop) with a direct 0.21.4 fetchurl derivation; wired DAGGER_HOST so the pre-commit dart-check hook can reach the running engine
  • pubspec.lock: bumped meta 1.17→1.18 and test 1.30→1.31 to match what the CI resolver picks up (eliminates spurious generated-files drift in CI)

Verification

  • task test — all 492 unit/widget tests pass
  • dart analyze --fatal-infos — clean (no warnings or infos)
  • Pre-commit hooks (including dart-check via Dagger) — all passed on commit

Closes #450

## Summary - Tapping a row in the Undo Log list opens a new `UndoLogDetailScreen` - Detail screen shows: account ID, action type (with icon/colour), timestamp, source folder, destination folder (move only), and a list of all emails in the transaction (subject + sender) - Navigation uses go_router nested route `/accounts/undo-log/:actionId` with `state.extra` to pass the `UndoAction` object - AppBar has an **Undo** button that calls the existing undo service and pops back ## Also fixed - `flake.nix`: replaced the broken dagger/nix 0.20.8 Nix wrapper (infinite self-exec loop) with a direct 0.21.4 `fetchurl` derivation; wired `DAGGER_HOST` so the pre-commit `dart-check` hook can reach the running engine - `pubspec.lock`: bumped `meta` 1.17→1.18 and `test` 1.30→1.31 to match what the CI resolver picks up (eliminates spurious generated-files drift in CI) ## Verification - `task test` — all 492 unit/widget tests pass - `dart analyze --fatal-infos` — clean (no warnings or infos) - Pre-commit hooks (including `dart-check` via Dagger) — all passed on commit Closes #450
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
NeedSupervisor
Issue escalated to a human supervisor; agentloop will skip it until cleared.
 State/InProgress
State/Later
State/Planned
automerge
Eligible for automatic merge by CI
 ci-failure
Issue opened by agentloop to track a failing CI workflow; used for deduplication.
 do-not-merge
Plan PR — review only, do not merge.
 loop/code
Add to run the built-in "code" prompt; override at prompts/code.md.
 loop/code-ci-pending
Prompt "code" finished; waiting for the PR's CI to pass before advancing.
 loop/code-done
Prompt "code" finished successfully.
 loop/code-in-process
Agent for the "code" prompt is currently running on this issue.
 loop/merge
Managed by agentloop
 loop/merge-done
Managed by agentloop
 loop/merge-in-process
Managed by agentloop
 loop/plan
Add to run the built-in "plan" prompt; override at prompts/plan.md.
 loop/plan-done
Prompt "plan" finished successfully.
 loop/plan-in-process
Agent for the "plan" prompt is currently running on this issue.
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
guettlibot guettli
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: guettli/sharedinbox#461