vinny/CMOnet
Watch
1
0
Fork
You've already forked CMOnet
 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity Actions

Scenario Builder: a donor's last battery can move to a base #34

Manually merged
vinny merged 0 commits from vinny/builder-battery into master 2026-10-14 00:33:07 -04:00 AGit
Conversation 0 Commits - Files changed -
vinny commented 2026-10-14 00:32:14 -04:00
Owner
Copy link

A Scenario Builder deployment no longer fails when a base's air defence takes the last battery of a donor package.

  • Found live (2026-10-14): the Baltic laboratory plan (#7) failed at "placing the forces" with 1 validation error for RoleCount count: Input should be greater than or equal to 1.
  • Cause: moved_battery decremented the donor item's count before removing an emptied item. The contracts validate on assignment, so count = 0 raised first.
  • Fix: the last battery now leaves its package, as intended. New test test_a_donors_last_battery_moves_and_its_package_goes.

🤖 Generated with Claude Code

A Scenario Builder deployment no longer fails when a base's air defence takes the last battery of a donor package. - **Found live (2026-10-14):** the Baltic laboratory plan (#7) failed at "placing the forces" with `1 validation error for RoleCount count: Input should be greater than or equal to 1`. - **Cause:** `moved_battery` decremented the donor item's count before removing an emptied item. The contracts validate on assignment, so `count = 0` raised first. - **Fix:** the last battery now leaves its package, as intended. New test `test_a_donors_last_battery_moves_and_its_package_goes`. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Base air defence moves a battery from a crowded site to a base with none. When
the donor package held a single battery, item.count -= 1 assigned 0, which the
RoleCount contract (validated on assignment, count >= 1) refused, and the whole
deployment failed ("Input should be greater than or equal to 1"), live on
2026-10-14 for the Baltic laboratory plan. The last battery now leaves its
package instead.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Version 0.7.18 (builder-battery)
Some checks failed
check / check (ubuntu-latest) (push) Has been cancelled
Details
check / check (windows-latest) (push) Has been cancelled
Details
check / check (ubuntu-latest) (pull_request) Has been cancelled
Details
check / check (windows-latest) (pull_request) Has been cancelled
Details
0fd901f717
vinny manually merged commit 0fd901f717 into master 2026-10-14 00:33:07 -04:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
vinny/CMOnet!34
No description provided.