Claude/fix dropdown class options 011 c up hr laq eg62k gyh sd8e g - #6
Merged
RobGruhl merged 2 commits intoNov 5, 2025
Merged
Conversation
Issue: Balcor had the 'machine' class which is intended for robots/AI characters, but Balcor is an organic Nerind'ar engineer. Changes: - Replaced 'machine' class with 'tech' class in Balcor's classBoards - Removed redundant 'impervious' skill specification (already provided by support class) - Balcor now has access to engineering-appropriate skills: engineering, regulate, hackAndGrab, assist, camouflage, hardToHit, and distraction This makes thematic sense for Balcor's character description as a "Skilled Nerind'ar engineer with modified prosthetic arm."
Issue: Characters like Balcor and Wade could only see 3-4 class options in the dropdown, when they should be able to choose from all available classes. Root Cause: The resolveAvailableClasses() function was limiting characters to only the classes listed in their classBoards array, regardless of whether they were machine-restricted characters or not. Changes: - Added machine-only character detection (checks if all classBoards are machine/machineTech) - Machine characters (Hopper, XL) still get only their restricted classes - Non-machine characters now see ALL global classes (all 14 classes) - Character-specific skill customizations from classBoards are merged into global classes - Characters without classBoards continue to get all classes (unchanged) Result: - Balcor now sees all 14 classes instead of just 3 - Wade now sees all 14 classes instead of just 4 - Balcor's custom Marine/Support/Tech skill lists are preserved - Wade's custom skill lists for his classes are preserved - Machine characters (Hopper, XL) remain restricted to machine classes only
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.
No description provided.