Register Mech tokens through the shared spawn path - #4
Merged
Merged
Conversation
Contributor
Author
|
Mirrored rules PR: abysl/agni-rfb#3 . Kai integration will pin the combined audit fixes; no main branch has been merged by this task. |
This was referenced Sep 22, 2026
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
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.
Problem
The rules engine creates Mechs through a handwritten fallback but does not register Mech as a canonical token name. Three token-name diagnostics are ignored for this known gap. Ferrous Forerunner's existing fallback already creates its printed tokens; this is a registry/integration fix, not a claim that Deathknell previously did nothing.
Change
Token::Mech, domainless 3-Might face and shared name registration.MECH_TOKENalias and update other Mech consumers.Validation
Passed in both repositories:
cargo fmt --all -- --check, pinned Nixtreefmt --ci,bash ci/check.sh.The portable Wasm artifact was not built by this task; no GUI/network playtest claimed. The remaining ignored tests are not counted as verified.
Integration
Companion PR mirrors the still-duplicated rules source; Kai currently consumes Agni. The app dependency/module must be updated to deliver the fix to players.