feat: 스킬 시스템 및 직업별 스킬 구현 - #59
Conversation
|
Warning Gemini encountered an error creating the review. You can try again by commenting |
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request introduces a comprehensive active and passive skill tree system, including an active skill executor, a status effect controller, and various specific active and passive skill implementations. It also updates entity stats, attack behaviors, and sensing colliders to dynamically scale with stats and modifiers. The review feedback highlights several critical issues: a potential bug where non-positive target counts could cause attacks to hit all targets, an incorrect collider offset calculation that expands the sensing area backwards, a stat leak in the kill-stack passive, a memory leak risk from a static executor reference in the UI, and missing field updates or null checks in utility and executor classes.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
요약
feature-GameLinking브랜치에 스킬 시스템 기반 코드와 Warrior/Archer/Wizard 스킬 구현을 병합합니다.이번 PR은
develop대상이 아니라feature-GameLinking작업 브랜치로 합치는 PR이므로, 별도 팀원 리뷰는 요청하지 않습니다. 스킬 구현 브랜치를 GameLinking 작업 흐름에 연결하기 위한 내부 병합용 PR입니다.주요 변경 사항
공통 스킬 시스템
ActiveSkillExecutor추가NoneSingleTargetAreaTryUseSkill(slotIndex)기반 구조 추가스탯/피해 시스템 확장
EntityStatModifier기반 스탯 보정 확장StatusEffectController추가Warrior 스킬 구현
TargetedDamageActiveSkill로 통합Archer 스킬 구현
Wizard 스킬 구현
ArrowRainActiveSkill추가기타
TargetedDamageActiveSkill로 대체이번 PR에서 하지 않는 작업
SkillNodeData연결SkillTreeData구성SkillTreePanelUI 배치위 작업들은 이 브랜치의 범위가 아니라 후속 에디터 세팅/콘텐츠 구성 작업에서 진행합니다.
확인 필요
TryUseSkill(slotIndex)호출 확인리뷰 정책
이 PR은
develop으로 바로 병합하는 PR이 아니라feature-GameLinking에 기능 브랜치를 합치는 내부 병합용 PR입니다.따라서 별도 팀원 리뷰는 받지 않고, GameLinking 작업 흐름에 필요한 코드 기반을 먼저 합치는 것을 목적으로 합니다.