Repository navigation
Conversation
Economy.md promises that defence platforms go offline in a power deficit, but Simulation/Combat had no notion of power: whether a tower fires was decided by IsArmed alone, so an attack on the enemy power plant was only half effective. CombatSystem now takes the EconomySystem (all callers already passed it) and skips building roles whose owner is low on power: no target acquisition, no fire, a held target is dropped; the cooldown keeps running so fire resumes the tick power returns. Units are never affected. HUD warnings name the silent towers. Fixes #67. Decision D-122 (draft; binding with the merge). No golden baseline and no pin moves; RulesHash64 is unchanged (the gap from #108). Sim tests 782/782; Unity EditMode mirrors not run. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
EditMode 667/667 and PlayMode 15/15 measured on this branch with Unity 6000.5.4f1 via the lane script from #110; no failures, clean tree. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Was: Gebäuderollen (MS-1: Verteidigungsplattform) im Strommangel des Besitzers (
PlayerEconomyState.IsLowPower) erfassen nichts, feuern nicht und verwerfen ihr Ziel; Cooldown läuft weiter, bei Rückkehr des Stroms sofort wieder Feuer. Einheiten nie betroffen.CombatSystembekommt dasEconomySystemals dritten Parameter (alle Aufrufer übergaben es schon). Ressourcenleiste und Baubar nennen „Türme aus“.Warum: Economy.md verlangt „Verteidigungsplattformen offline“; ohne das bleibt der Angriff aufs Kraftwerk halb wirksam. Fixes #67, Entscheidung D-122 (Entwurf).
Bereiche:
Simulation/Combat/,Gameplay/UI/(zwei Presenter), Tests beider Spuren, Buildings.md, Economy.md, DemoRunbook.md, DecisionLog, CHANGELOG.Changelog:
[Unreleased]→ Geändert.Tests:
dotnet test782/782 (vier neue, Negativkontrolle rot ohne Gate);check_docs.pyOK; Guard-Selbsttest OK. Kein Pin, keine Baseline,RulesHash64unverändert (Hash-Lücke wie #108, siehe D-122).Unity-Spuren: EditMode 667/667, PlayMode 15/15.
Spielabnahme: nicht gespielt — Restrisiko: Balance (Kraftwerksausfall schaltet die gesamte Verteidigung ab), Spielerfalle durch nicht reservierte Baustellen, HUD-Textbreite unter Unity ungeprüft.
Berichte:
reports/v8.6.0/issue-lauf-2026-10-17/03-issue-67-*.md,03b-review-1-issue-67.md.🤖 Generated with Claude Code