Add skill tree tab with keystones, weapon/crafting skills UI#48
Open
derivagral wants to merge 2 commits intomainfrom
Open
Add skill tree tab with keystones, weapon/crafting skills UI#48derivagral wants to merge 2 commits intomainfrom
derivagral wants to merge 2 commits intomainfrom
Conversation
New Skills tab displays skill tree data extracted from save files: - Auto-detected weapon stances (94 skills) and crafting/elven tree (40 skills) grouped by type/branch with collapsible sections - Manual keystone checklist for 14 main-tree keystones that can't be auto-detected from opaque node IDs (proximity, mastery, affinity, utility) - Crystal cards section (16 cards, visual only, effects TBD) - Main tree summary (node count by type) Each skill with a statId gets a user-editable value input so users can enter actual stat contributions. Paragon skills multiply value by level with editable level overrides. All skills can be toggled on/off. Skill tree stats feed into the derived stats calculation engine: - Weapon/crafting skill values merge into base stat aggregation - Affinity keystones set edpsAD.affinityDamage config override - Proximity keystones set distance flags on edpsEMulti New files: - useSkillTreeStore hook (state, overrides, computed stats/configs) - SkillTreeTab + KeystoneChecklist, WeaponSkillsSection, CraftingSkillsSection, CardsSection, SkillRow components - 17 new tests for store logic https://claude.ai/code/session_01GoRqnUjSxx3Nj71jEW2NQZ
- Rename tab label from "Skills" to "Skill Tree" - Look up STAT_REGISTRY.isPercent for each skill's statId to determine whether to show percent or flat input (step=10 for flat values) - Update CLAUDE.md: add skill tree tab docs, store state flow, eDPS integration notes, new test file references - Add tests for flat vs percent stat handling and mixed scenarios https://claude.ai/code/session_01GoRqnUjSxx3Nj71jEW2NQZ
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.
Summary
Adds a complete Skill Tree tab to the character builder, enabling users to view auto-detected skills from save files and manually configure keystones, weapon stances, and crafting skills. Integrates skill tree stats into the derived stats calculation engine.
Key Changes
New Components
State Management
Integration
skillTreeStatsandskillTreeConfigOverridesparameters, merging skill tree contributions into final stat calculationsTesting
Documentation
Implementation Details
https://claude.ai/code/session_01GoRqnUjSxx3Nj71jEW2NQZ