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

Fix CI of Chaos monkey pr #456

New Issue
Closed
opened 2026-06-05 20:50:03 +00:00 by guettli · 1 comment
guettli commented 2026-06-05 20:50:03 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

https://codeberg.org/guettli/sharedinbox/pulls/455

https://codeberg.org/guettli/sharedinbox/pulls/455
guettlibot commented 2026-06-05 21:35:02 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

The PR is open at https://codeberg.org/guettli/sharedinbox/pulls/458.

What I did:

The CI failures for PR #455 (chaos monkey backend test) were caused by dart analyze --fatal-infos rejecting three categories of lint violations in test/backend/chaos_monkey_test.dart:

  1. avoid_print — 5 print(...) calls replaced with stdout.writeln(...) (dart:io was already imported)
  2. avoid_redundant_argument_values — _env('CHAOS_SEED', '') simplified to _env('CHAOS_SEED') since '' is the function's default
  3. dart format — trailing commas added and long connectToServer(...) calls wrapped to fit the 80-char limit

After the fixes, fvm dart analyze --fatal-infos reports No issues found! and the formatter confirms the file is clean.

The PR is open at https://codeberg.org/guettli/sharedinbox/pulls/458. --- **What I did:** The CI failures for PR #455 (chaos monkey backend test) were caused by `dart analyze --fatal-infos` rejecting three categories of lint violations in `test/backend/chaos_monkey_test.dart`: 1. **`avoid_print`** — 5 `print(...)` calls replaced with `stdout.writeln(...)` (`dart:io` was already imported) 2. **`avoid_redundant_argument_values`** — `_env('CHAOS_SEED', '')` simplified to `_env('CHAOS_SEED')` since `''` is the function's default 3. **`dart format`** — trailing commas added and long `connectToServer(...)` calls wrapped to fit the 80-char limit After the fixes, `fvm dart analyze --fatal-infos` reports `No issues found!` and the formatter confirms the file is clean.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
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 loop/code-done
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#456