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

docs: add dart format and import ordering to AGENTS.md #385

Closed
guettlibot wants to merge 1 commits from issue-384-fix-open-prs into main
pull from: issue-384-fix-open-prs
merge into: :main
Conversation 0 Commits 1 Files Changed 1
+2
guettlibot commented 2026-06-03 22:46:58 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

Summary

Three open PRs were failing CI due to dart format and dart analyze --fatal-infos violations:

  • PR #381 (issue-367-prefetch-next-email): lib/di.dart needed reformatting
  • PR #378 (issue-377-remember-show-images): test/unit/migration_test.dart needed reformatting
  • PR #379 (issue-376-combined-inbox): lib/data/repositories/email_repository_impl.dart needed reformatting, lib/ui/router.dart had imports out of alphabetical order, and stale mock formatting in test/unit/account_sync_manager_test.mocks.dart

Formatting fix commits have been pushed directly to each PR branch so their CI can re-run.

This PR adds explicit requirements to AGENTS.md so future agents know to run dart format before committing and to keep imports sorted.

Closes #384

Test plan

  • dart format --output=none --set-exit-if-changed lib/ test/ — no changes
  • dart analyze --fatal-infos lib/ test/ — no issues

🤖 Generated with Claude Code

## Summary Three open PRs were failing CI due to `dart format` and `dart analyze --fatal-infos` violations: - **PR #381** (`issue-367-prefetch-next-email`): `lib/di.dart` needed reformatting - **PR #378** (`issue-377-remember-show-images`): `test/unit/migration_test.dart` needed reformatting - **PR #379** (`issue-376-combined-inbox`): `lib/data/repositories/email_repository_impl.dart` needed reformatting, `lib/ui/router.dart` had imports out of alphabetical order, and stale mock formatting in `test/unit/account_sync_manager_test.mocks.dart` Formatting fix commits have been pushed directly to each PR branch so their CI can re-run. This PR adds explicit requirements to `AGENTS.md` so future agents know to run `dart format` before committing and to keep imports sorted. Closes #384 ## Test plan - [x] `dart format --output=none --set-exit-if-changed lib/ test/` — no changes - [x] `dart analyze --fatal-infos lib/ test/` — no issues 🤖 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#385