Skip to content

fix(gameplay): surface blockers and clear attack targets - #83

Merged
cubetribe merged 2 commits into
mainfrom
codex/fix/16-10-decision-feedback
Aug 10, 2026
Merged

fix(gameplay): surface blockers and clear attack targets#83
cubetribe merged 2 commits into
mainfrom
codex/fix/16-10-decision-feedback

Conversation

@cubetribe

@cubetribe cubetribe commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

Was

  • leitet für die Baubar vor der Zielzellenwahl den ersten sichtbaren Blocker in der HUD-Priorität Voraussetzung → AE → Energie → globales Baustellenlimit her
  • erhält beim Zusammenführen mit feat(construction): enforce all building prerequisites #81 sämtliche Mehrfachvoraussetzungen und zeigt fehlende Gebäudenamen stabil und vollständig an
  • zeigt die Strombilanz dauerhaft an der Baubar sowie Bedarf oder Erzeugung beim Hover und auf der gewählten Gebäude-Befehlskarte
  • lässt Energiemangel den Platzierungsmodus bewusst erreichbar, während Voraussetzung, AE und Baustellenlimit den Knopf sperren
  • löscht beim bestehenden Stop-Befehl zusätzlich AttackTarget, ohne CommandKind, StateVersion oder Zustandslayout zu ändern
  • schließt die Sprint-16-Statusdokumentation für die technisch implementierten Pakete 16.3–16.10 ab

Warum

Die Simulation bündelt mehrere Platzierungsursachen im selben ResultCode; die Oberfläche muss deshalb den für die Entscheidung relevanten Grund aus demselben Zustand herleiten. Die HUD-Priorität beschreibt dabei ausdrücklich die Prüfung vor Auswahl einer konkreten Zielzelle und nicht die globale Reihenfolge im CommandExecutor. Strom gehört an den Entscheidungspunkt. Stoppen räumte bisher Bewegung, Ernte und Reparatur auf, ließ aber das Angriffsziel stehen.

Nachweise auf aktuellem Head e69aee3

  • vollständige .NET-Suite: 707/707 grün
  • vollständige Unity-EditMode-Suite: 591/591 grün
  • fokussierter Graybox-PlayMode-Batch: 3/3 grün
  • Doku-Prüfung: 181 Markdown-Dateien + 5 Quality-JSON-Dateien grün
  • git diff --check: grün; keine Konfliktmarker
  • Baseline-Guard und External-Contributor-Review jeweils lokal per Self-Test grün
  • unabhängiges Read-only-Finalreview: keine offenen P1–P3-Befunde, FINAL GO
  • geschützte Golden-/Determinismus-Baselines und ProjectSettings unverändert
  • keine Änderung an AI-/Hash-Pins, CommandKind, StateVersion, Zustandslayout oder Rules Hash gegenüber main

Manuelle Abnahme und Restrisiko

Nicht manuell gespielt. Der Maintainer konnte die Runde nicht interaktiv abnehmen; stattdessen wurden die vollständigen automatisierten Suites und der fokussierte Graybox-PlayMode-Batch auf dem finalen Head ausgeführt. Verbleibendes Restrisiko: Lesbarkeit, Hover-/Hitbox-Gefühl und das subjektive Stop-/Auto-Reacquire-Verhalten wurden nicht durch einen Menschen im laufenden Spiel beurteilt. Ein echtes Halte-Feuer bleibt außerhalb dieses Pakets, weil D-087 im nächsten Combat-Tick wieder ein Ziel erfassen darf.

Governance und Merge-Gates

Changelog

Genau ein Eintrag unter [Unreleased] / Behoben für #45, #47 und #48; keine neue D-ID.

@cubetribe
cubetribe requested review from travelhawk and removed request for travelhawk August 9, 2026 22:10
@cubetribe
cubetribe marked this pull request as ready for review August 10, 2026 11:17
@cubetribe
cubetribe merged commit 75973d3 into main Aug 10, 2026
9 of 10 checks passed
@cubetribe
cubetribe deleted the codex/fix/16-10-decision-feedback branch August 10, 2026 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant