Skip to content

feat: 스킬 시스템 및 직업별 스킬 구현 - #59

Merged
suti1110 merged 5 commits into
feature-GameLinkingfrom
feature-SkillCreate
Jul 8, 2026
Merged

suti1110 merged 5 commits into
feature-GameLinkingfrom
feature-SkillCreate

Conversation

@suti1110

@suti1110 suti1110 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

요약

feature-GameLinking 브랜치에 스킬 시스템 기반 코드와 Warrior/Archer/Wizard 스킬 구현을 병합합니다.

이번 PR은 develop 대상이 아니라 feature-GameLinking 작업 브랜치로 합치는 PR이므로, 별도 팀원 리뷰는 요청하지 않습니다. 스킬 구현 브랜치를 GameLinking 작업 흐름에 연결하기 위한 내부 병합용 PR입니다.

주요 변경 사항

공통 스킬 시스템

  • 액티브 스킬 실행기 ActiveSkillExecutor 추가
  • 액티브 스킬 대상 선택 방식 추가
    • 감지 대상
    • 전체 적
    • 자기 자신
  • 액티브 스킬 피해 카테고리 추가
    • None
    • SingleTarget
    • Area
  • 액티브 스킬 쿨타임 처리 추가
  • 스테이지 전환 시 1회 사용 제한 스킬 상태 초기화
  • 모바일 버튼 연결용 TryUseSkill(slotIndex) 기반 구조 추가

스탯/피해 시스템 확장

  • EntityStatModifier 기반 스탯 보정 확장
  • 공격 사거리, 공격 범위, 평타 타겟 수 보정 추가
  • 받는 피해 처리 전 이벤트 추가
  • 가하는 피해 후 이벤트 추가
  • 상태이상 및 디버프 처리를 위한 StatusEffectController 추가
  • 화상, 동상, 기절, 공격력 감소, 받는 피해 증가, 이동속도 감소 지원
  • 화상에 대상 최대 체력 비례 피해 지원 추가

Warrior 스킬 구현

  • 공통 피해형 스킬을 TargetedDamageActiveSkill로 통합
  • Warrior 특수 액티브 구현
    • 버티기
    • 잔심
    • 처형
    • 역류
    • 분신
    • 각성
  • Warrior 패시브 구현
    • 천 갑옷
    • 사전 준비
    • 브러진 칼
    • 슬로우 스타터
    • 부러진 갑옷
    • 단련 등

Archer 스킬 구현

  • 연속 사격
  • 정신집중
  • 부동자세
  • 추적
  • 헤드샷
  • 멀티샷
  • 공통 피해형 액티브 설정 가능 구조 반영

Wizard 스킬 구현

  • 불/얼음/바람 계열 패시브 및 액티브 구현 기반 추가
  • 화상, 4도 화상, 재, 폭발 등 불 계열 패시브 지원
  • 동상, 냉기강화, 얼음 갑옷, 영하 등 얼음 계열 패시브 지원
  • 칼바람, 순풍, 나락 등 바람 계열 패시브 지원
  • 장판형 스킬용 ArrowRainActiveSkill 추가
  • 공격 범위 증가 패시브가 평타/액티브/장판 범위에 반영되도록 처리

기타

  • 기존 Warrior 전용 피해형 액티브 클래스를 범용 TargetedDamageActiveSkill로 대체
  • 일부 Warrior 스킬 SO 경로 정리
  • 스킬 설정 가이드와 실제 코드 기준 불일치 수정
  • UTF-8 한글 인코딩 관련 프로젝트 규칙 추가

이번 PR에서 하지 않는 작업

  • 실제 스킬 SO 전체 생성
  • SkillNodeData 연결
  • SkillTreeData 구성
  • SkillTreePanel UI 배치
  • 직업별 스킬 버튼 프리팹 세팅
  • 서버 저장 연동
  • 팀원 코드 리뷰 요청

위 작업들은 이 브랜치의 범위가 아니라 후속 에디터 세팅/콘텐츠 구성 작업에서 진행합니다.

확인 필요

  • Unity Editor 컴파일 확인
  • 액티브 스킬 버튼에서 TryUseSkill(slotIndex) 호출 확인
  • 스킬 SO 생성 후 각 필드가 가이드와 맞는지 확인
  • 실제 Play Mode에서 상태이상, 쿨타임, 패시브 중첩 동작 확인

리뷰 정책

이 PR은 develop으로 바로 병합하는 PR이 아니라 feature-GameLinking에 기능 브랜치를 합치는 내부 병합용 PR입니다.

따라서 별도 팀원 리뷰는 받지 않고, GameLinking 작업 흐름에 필요한 코드 기반을 먼저 합치는 것을 목적으로 합니다.

@suti1110 suti1110 added the enhancement New feature or request label Jul 8, 2026
@suti1110 suti1110 linked an issue Jul 8, 2026 that may be closed by this pull request
10 tasks
@gemini-code-assist

Copy link
Copy Markdown

Warning

Gemini encountered an error creating the review. You can try again by commenting /gemini review.

@suti1110

suti1110 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

/gemini review

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread Fantasy-Grower/Assets/Scripts/Battle/Attack/SingleTargetAttackBehaviour.cs Outdated
Comment thread Fantasy-Grower/Assets/Scripts/Battle/AttackTargetsSensing.cs
Comment thread Fantasy-Grower/Assets/Scripts/Core/EntityStatModifierUtility.cs
Comment thread Fantasy-Grower/Assets/Scripts/UI/InDungeon/ActiveSkillButtonUI.cs Outdated
Comment thread Fantasy-Grower/Assets/Scripts/UI/InDungeon/ActiveSkillButtonUI.cs Outdated
Comment thread Fantasy-Grower/Assets/Scripts/Battle/Attack/SingleTargetAttackBehaviour.cs Outdated
Comment thread Fantasy-Grower/Assets/Scripts/Core/SkillTree/ActiveSkillExecutor.cs
Comment thread Fantasy-Grower/Assets/Scripts/Core/SkillTree/ActiveSkillExecutor.cs
@suti1110 suti1110 changed the title Feature skill create feat: 스킬 시스템 및 직업별 스킬 구현 Jul 8, 2026
@suti1110
suti1110 merged commit fe22290 into feature-GameLinking Jul 8, 2026
@github-project-automation github-project-automation Bot moved this from 계획됨 to 완료됨 in Fantasy-Client/Task Jul 8, 2026
@suti1110
suti1110 deleted the feature-SkillCreate branch July 8, 2026 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: 완료됨

Development

Successfully merging this pull request may close these issues.

[Feature] 게임 씬 연결 및 디자인 적용

1 participant