Skip to content

Conversation

@IISweetHeartII
Copy link
Contributor

Summary

회사별 인사이트 조회 API를 추가해 회사 ID 기준 인사이트 데이터를 조회할 수 있도록 구현했습니다.

Changes

  • GET /portfolio-correction/company-insights/:companyId 엔드포인트 추가
  • Company Insight 응답 DTO 변환 로직 추가
  • PortfolioCorrectionService 조회 메서드 추가
  • 컨트롤러 응답 타입/Swagger 스펙 반영

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 테스트 통과
  • pnpm run lint 통과
  • pnpm run build 통과
  • pnpm exec jest --passWithNoTests 통과

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)

N/A

Additional Notes

도메인 계층 분리를 유지하며 조회 전용 로직으로 구현했습니다.

@IISweetHeartII IISweetHeartII self-assigned this Feb 12, 2026
@IISweetHeartII IISweetHeartII added ✨ feature New feature or request 덕환 labels Feb 12, 2026
@gemini-code-assist
Copy link

Summary of Changes

Hello @IISweetHeartII, 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은 회사별 인사이트 데이터를 조회하기 위한 새로운 기능을 도입합니다. 기존 PortfolioCorrection 엔티티를 활용하여 특정 회사 ID에 해당하는 인사이트 정보를 효율적으로 가져올 수 있도록 API, 서비스, DTO 계층에 걸쳐 필요한 로직을 구현했습니다. 이를 통해 사용자들은 회사별 인사이트를 쉽게 확인할 수 있게 됩니다.

Highlights

  • 회사별 인사이트 조회 API 추가: 회사 ID를 기준으로 특정 회사의 인사이트 데이터를 조회할 수 있는 새로운 API 엔드포인트가 추가되었습니다.
  • 응답 DTO 변환 로직 구현: PortfolioCorrection 엔티티를 UpdateCompanyInsightResDTO로 변환하는 정적 팩토리 메서드가 DTO에 추가되었습니다.
  • 서비스 계층 조회 메서드 구현: PortfolioCorrectionService에 getCompanyInsight 메서드가 추가되어 회사 인사이트 조회 로직을 캡슐화합니다.
  • 컨트롤러 및 Swagger 스펙 업데이트: PortfolioCorrectionController에 새로운 API 엔드포인트가 구현되었으며, Swagger 문서에 응답 타입이 반영되었습니다.

🧠 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/portfolio-correction/application/dtos/company-insight.dto.ts
    • PortfolioCorrection 엔티티를 임포트했습니다.
    • UpdateCompanyInsightResDTO에 PortfolioCorrection 엔티티로부터 DTO 인스턴스를 생성하는 정적 from 메서드를 추가했습니다.
  • src/modules/portfolio-correction/application/services/portfolio-correction.service.ts
    • UpdateCompanyInsightResDTO를 임포트했습니다.
    • 주어진 correctionId와 userId를 사용하여 회사 인사이트를 조회하고 UpdateCompanyInsightResDTO로 반환하는 비동기 getCompanyInsight 메서드를 추가했습니다.
  • src/modules/portfolio-correction/presentation/portfolio-correction.controller.ts
    • @user('sub') userId와 @param('correctionId', ParseIntPipe) correctionId를 인자로 받아 회사 인사이트를 조회하는 getCompanyInsight 엔드포인트를 구현했습니다.
    • 기존의 NOT_IMPLEMENTED 예외 처리를 제거하고 portfolioCorrectionService.getCompanyInsight 호출로 대체했습니다.
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

이 PR은 회사별 인사이트 조회 API를 구현합니다. 새로운 API 엔드포인트가 추가되었으나, 이에 대한 단위 테스트나 E2E 테스트 코드가 누락되었습니다. 레포지토리 스타일 가이드의 '우선순위 중간' 항목인 '테스트 누락'에 해당하므로, 코드 안정성과 유지보수성 향상을 위해 테스트 코드를 추가하는 것을 권장합니다.

@IISweetHeartII
Copy link
Contributor Author

Correction note (formatted):

  • Added explicit validation to UpdateCompanyInsightReqDTO (@IsOptional, @IsString, @MaxLength) so request handling is consistent with global ValidationPipe behavior.
  • Updated nullable response typing for companyInsight/highlightPoint and correction item JSON fields so DTO contracts match nullable entity columns.
  • Commit: 9cfc86f

Verification:

  • lsp_diagnostics on modified files: clean
  • pnpm run lint: pass
  • pnpm exec jest --passWithNoTests: pass (no test files)
  • pnpm run build: fails due to pre-existing repository-wide src/metadata.ts TS2307 resolution issue (not introduced by this change)

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.

[Feat] 기업 분석 정보 조회 API 구현

1 participant