Skip to content

Conversation

@Bori-github
Copy link
Collaborator

📌 요약

  • playground 폴더 구조 변경 및 불필요한 코드 제거

📝 작업 내용

  • 기존 page/playground-page.tsx 제거 및 page/playground-groups.tsx > page/playground-page.tsx 로 네이밍 수정
  • playground store 테스트 코드 추가
  • feature/playground 폴더구조 수정: model 폴더 생성 후 파일 위치 변경
  • widgets/playground 폴더구조 수정: ui 폴더 생성 후 파일 위치 변경

🔍 스크린샷

  • 없음

💡 참고 사항

  • 없음

@Bori-github Bori-github self-assigned this Nov 6, 2025
@ohah ohah requested a review from Copilot November 6, 2025 03:40
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the playground structure by consolidating duplicate code and reorganizing the file structure. The main objective is to merge PlaygroundGroups functionality into PlaygroundPage and extract the playground UI logic into a reusable PlaygroundWidget component.

  • Extracted playground UI rendering logic into a new PlaygroundWidget component in the widgets layer
  • Consolidated PlaygroundGroups component functionality into PlaygroundPage
  • Created a unified usePlaygroundStore that manages both tabs and playground execution state
  • Reorganized the features/playground module structure with a dedicated model subdirectory

Reviewed Changes

Copilot reviewed 9 out of 12 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
apps/executeJS/src/widgets/playground/ui/playground-widget.tsx New widget component extracted from playground-page, handles UI rendering for a single playground instance
apps/executeJS/src/widgets/playground/ui/index.ts Export barrel for playground widget UI
apps/executeJS/src/widgets/playground/index.ts Updated to export from ui subdirectory
apps/executeJS/src/pages/playground/playground-page.tsx Refactored to incorporate PlaygroundGroups functionality and use new PlaygroundWidget
apps/executeJS/src/pages/playground/playground-groups.tsx Deleted file (functionality merged into playground-page)
apps/executeJS/src/pages/playground/index.ts Removed export of deleted PlaygroundGroups component
apps/executeJS/src/features/playground/model/store.ts New unified store managing tabs and playground state
apps/executeJS/src/features/playground/model/store.test.ts Comprehensive test suite for the playground store
apps/executeJS/src/features/playground/model/index.ts Export barrel for model layer
apps/executeJS/src/features/playground/model/const.ts Constants for playground feature
apps/executeJS/src/features/playground/index.ts Updated to export from model subdirectory
apps/executeJS/src/app/index.tsx Updated to use PlaygroundPage instead of PlaygroundGroups

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Bori-github Bori-github merged commit 88f7429 into main Nov 7, 2025
9 checks passed
ohah pushed a commit that referenced this pull request Dec 12, 2025
* refactor: 기존 playground-page 컴포넌트 제거 및 playground-groups > playground-page로 네이밍 변경

* feat: playground store 테스트 코드 추가

* refactor: feature/playground 폴더구조 수정
- model 폴더 생성

* refactor: widgets/playground 폴더구조 수정
- ui 폴더 생성
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants