Skip to content

Feat: ExperienceStatus 분기 추가#332

Merged
hyoinkang merged 3 commits intodevfrom
feat/add-generate-failed-#331
Mar 11, 2026
Merged

Feat: ExperienceStatus 분기 추가#332
hyoinkang merged 3 commits intodevfrom
feat/add-generate-failed-#331

Conversation

@hyoinkang
Copy link
Contributor

@hyoinkang hyoinkang commented Mar 11, 2026

Summary

경험 정리 분기를 명확화하기 위한 status 분기 추가

Changes

  • enum 상태 추가
  • 로직 수정

Type of Change

해당하는 항목에 체크해주세요:

  • Bug fix (기존 기능을 수정하는 변경)
  • New feature (새로운 기능 추가)
  • Breaking change (기존 기능에 영향을 주는 변경)
  • Refactoring (기능 변경 없이 코드 개선)
  • Documentation (문서 변경)
  • Chore (빌드, 설정 등)

Target Environment

배포 대상 브랜치를 선택해주세요:

  • Dev (dev)
  • Prod (main)

Related Issues

관련 이슈를 연결해주세요:

Testing

테스트 방법을 작성해주세요:

  • Postman/Swagger로 API 호출 확인
  • 단위 테스트 통과
  • E2E 테스트 통과

Checklist

PR 생성 전 확인사항:

  • 코드 컨벤션을 준수했습니다 (docs/development/CODE_STYLE.md)
  • Git 컨벤션을 준수했습니다 (docs/development/GIT_CONVENTIONS.md)
  • 네이밍 컨벤션을 준수했습니다 (docs/development/NAMING_CONVENTIONS.md)
  • 로컬에서 빌드가 성공합니다 (pnpm run build)
  • 로컬에서 린트가 통과합니다 (pnpm run lint)
  • (API 변경 시) Swagger 문서가 업데이트되었습니다
  • (필요 시) 테스트 코드를 작성했습니다

Screenshots (Optional)

UI 변경이 있다면 스크린샷을 첨부해주세요.

Additional Notes

리뷰어에게 전달할 추가 정보가 있다면 작성해주세요.

- 포트폴리오 생성 실패 시 ON_CHAT으로 롤백하는 대신 사용할 enum을 추가함
@hyoinkang hyoinkang self-assigned this Mar 11, 2026
@hyoinkang hyoinkang added the ✨ feature New feature or request label Mar 11, 2026
@hyoinkang hyoinkang linked an issue Mar 11, 2026 that may be closed by this pull request
10 tasks
@gemini-code-assist
Copy link

Summary of Changes

Hello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 경험 생성 과정에서 발생할 수 있는 실패 상태를 명확하게 관리하기 위해 ExperienceStatus enum에 GENERATE_FAILED 상태를 도입합니다. 이는 경험의 상태 관리를 세분화하여 시스템이 생성 실패를 보다 정확하게 반영할 수 있도록 합니다. 변경 사항은 enum 업데이트, 서비스 로직 수정, 관련 호출 지점 조정 및 데이터베이스 마이그레이션을 포함합니다.

Highlights

  • 새로운 경험 상태 추가: 경험(Experience)의 상태를 나타내는 ExperienceStatus enum에 GENERATE_FAILED라는 새로운 상태가 추가되었습니다.
  • 상태 전환 로직 개선: ExperienceService 내의 revertToOnChat 메서드가 transitionToGenerateFailed로 이름이 변경되었으며, 실패 시 ON_CHAT 대신 GENERATE_FAILED 상태로 전환하도록 수정되었습니다.
  • 관련 서비스 호출 업데이트: InternalPortfolioFacadeInterviewFacade에서 ExperienceService의 상태 전환 메서드를 호출하는 로직이 새로운 transitionToGenerateFailed 메서드를 사용하도록 업데이트되었습니다.
  • 데이터베이스 스키마 마이그레이션: 데이터베이스의 experience_status_enum 타입에 GENERATE_FAILED 값을 추가하는 SQL 마이그레이션 스크립트가 포함되었습니다.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Changelog
  • src/modules/experience/application/services/experience.service.ts
    • revertToOnChat 메서드가 transitionToGenerateFailed로 이름이 변경되었고, ExperienceStatus.ON_CHAT 대신 ExperienceStatus.GENERATE_FAILED로 상태를 설정하도록 수정되었습니다.
  • src/modules/experience/domain/enums/experience-status.enum.ts
    • ExperienceStatus enum에 GENERATE_FAILED 상태가 추가되었습니다.
  • src/modules/internal/application/facades/internal-portfolio.facade.ts
    • experienceService.revertToOnChat 호출이 experienceService.transitionToGenerateFailed로 업데이트되었습니다.
  • src/modules/interview/application/facades/interview.facade.ts
    • experienceService.revertToOnChat 호출이 experienceService.transitionToGenerateFailed로 업데이트되었습니다.
  • supabase/migrations/20260311100000_add_generate_failed_to_experience_status.sql
    • experience_status_enumGENERATE_FAILED 값을 추가하는 SQL 마이그레이션 스크립트가 추가되었습니다.
Activity
  • 이 PR에 대한 활동 내역은 제공되지 않았습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

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

Choose a reason for hiding this comment

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

Code Review

포트폴리오 생성 실패 시 상태를 명확히 하기 위해 GENERATE_FAILED 상태를 추가하신 점은 좋은 개선입니다. 관련 서비스와 데이터베이스 마이그레이션까지 일관성 있게 변경이 적용되었습니다. 다만, GENERATE_FAILED 상태가 된 이후 사용자가 재시도를 할 수 있는 경로가 현재 코드상으로는 보이지 않는다는 점이 우려됩니다. 이로 인해 사용자가 더 이상 진행할 수 없는 상태에 빠질 수 있으므로, 관련 로직의 보완이 필요해 보입니다.

@hyoinkang hyoinkang merged commit 2a1ff2b into dev Mar 11, 2026
2 checks passed
@hyoinkang hyoinkang deleted the feat/add-generate-failed-#331 branch March 11, 2026 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task] ExpeirenceStatus에 GENRATE_FAILED 상태 추가

1 participant