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

feat: syncLog add Copy button, stack trace, isPermanent, Android device info (#237) #241

Closed
guettlibot wants to merge 7 commits from issue-237-fix into main
pull from: issue-237-fix
merge into: :main
Conversation 0 Commits 7 Files Changed 25
+740 -85
guettlibot commented 2026-05-25 06:35:40 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

Summary

  • Schema v33: adds error_stack_trace and is_permanent columns to sync_logs; IMAP and JMAP sync loops now store the full stack trace string and isPermanent flag on error entries
  • New lib/ui/utils/about_markdown.dart utility (shared between AboutScreen and sync log copy) that builds the markdown table including Android device info via device_info_plus
  • AboutScreen updated to use the utility and show Android manufacturer, model, and OS version
  • SyncLogScreen: subtitle shows "Error (permanent)" for permanent errors; expanded view shows stack trace in red monospace; each tile has a Copy button that copies a markdown summary of the entry plus the full About section

Test plan

  • Unit: migration_test.dart — verifies error_stack_trace and is_permanent columns exist after migration from v1, v22, and on fresh install
  • Unit: sync_log_repository_impl_test.dart — new test verifies stackTrace and isPermanent are stored and retrieved correctly
  • All 451 existing tests (unit + widget) continue to pass

🤖 Generated with Claude Code

## Summary - Schema v33: adds `error_stack_trace` and `is_permanent` columns to `sync_logs`; IMAP and JMAP sync loops now store the full stack trace string and `isPermanent` flag on error entries - New `lib/ui/utils/about_markdown.dart` utility (shared between AboutScreen and sync log copy) that builds the markdown table including Android device info via `device_info_plus` - AboutScreen updated to use the utility and show Android manufacturer, model, and OS version - SyncLogScreen: subtitle shows "Error (permanent)" for permanent errors; expanded view shows stack trace in red monospace; each tile has a Copy button that copies a markdown summary of the entry plus the full About section ## Test plan - [ ] Unit: `migration_test.dart` — verifies `error_stack_trace` and `is_permanent` columns exist after migration from v1, v22, and on fresh install - [ ] Unit: `sync_log_repository_impl_test.dart` — new test verifies `stackTrace` and `isPermanent` are stored and retrieved correctly - [ ] All 451 existing tests (unit + widget) continue to pass 🤖 Generated with [Claude Code](https://claude.com/claude-code)

Pull request closed

Please reopen this pull request to perform a merge.
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#241