Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(website): remove chat package and merge #242

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Conversation

sunrabbit123
Copy link
Collaborator

@sunrabbit123 sunrabbit123 commented Mar 26, 2025

This PR removes the unused chat package and integrates the playground into the website.

After this task, we plan to recreate the standalone project in a modern style. Based on that, the playground and React template will be created.


This pull request introduces several new components and enhancements to the website/app/_components/playground directory, specifically focusing on the AgenticaChatMovie and its associated message and side components. The main changes include new components for handling markdown rendering, chat messages, and chat side movies, along with improvements to the existing Page component.

New Components and Enhancements:

Markdown Rendering:

Chat Movie Components:

Chat Message Components:

Chat Side Components:

Page Component Update:

@sunrabbit123 sunrabbit123 marked this pull request as draft March 26, 2025 10:48
Copy link

pkg-pr-new bot commented Mar 26, 2025

Open in Stackblitz

@agentica/benchmark

npm i https://pkg.pr.new/wrtnlabs/agentica/@agentica/benchmark@242

agentica

npm i https://pkg.pr.new/wrtnlabs/agentica@242

@agentica/core

npm i https://pkg.pr.new/wrtnlabs/agentica/@agentica/core@242

create-agentica

npm i https://pkg.pr.new/wrtnlabs/agentica/create-agentica@242

@agentica/pg-vector-selector

npm i https://pkg.pr.new/wrtnlabs/agentica/@agentica/pg-vector-selector@242

@agentica/rpc

npm i https://pkg.pr.new/wrtnlabs/agentica/@agentica/rpc@242

commit: c6060a1

@sunrabbit123 sunrabbit123 linked an issue Mar 26, 2025 that may be closed by this pull request
Copy link
Contributor

@Copilot 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 removes the outdated chat package and related example files, consolidating the implementation into the new website components. Key changes include:

  • Removal of all chat examples (shopping and BBS) along with their assets.
  • Clean-up of build, configuration, and deployment scripts related to the removed package.
  • Elimination of legacy documentation files for the chat package.

Reviewed Changes

Copilot reviewed 50 out of 56 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/chat/src/examples/shopping/script.md Deleted obsolete shopping chat example script.
packages/chat/src/examples/shopping/ShoppingChatApplication.tsx Removed the shopping chat application component.
packages/chat/src/examples/index.tsx Removed the index file for chat examples.
packages/chat/src/examples/bbs/script.md Deleted obsolete BBS chat example script.
packages/chat/src/examples/bbs/index.tsx Removed the BBS chat application entry file.
packages/chat/src/examples/bbs/IBbsArticle.ts Removed BBS article interface definitions.
packages/chat/src/examples/bbs/BbsChatApplication.tsx Deleted the BBS chat application component.
packages/chat/src/examples/bbs/BbsArticleService.ts Removed the service handling BBS articles.
packages/chat/src/README.md Removed legacy README for the chat package.
packages/chat/src/AgenticaChatApplication.tsx Deleted the Agentica chat application wrapper.
packages/chat/rollup.config.cjs Removed build configuration for the chat package.
packages/chat/eslint.config.mts Removed ESLint configuration related to the package.
packages/chat/build/deploy.mjs Removed deployment script for the chat package.
packages/chat/README.md Removed the top-level README for the chat package.
Files not reviewed (6)
  • packages/chat/.gitignore: Language not supported
  • packages/chat/LICENSE: Language not supported
  • packages/chat/bbs/index.html: Language not supported
  • packages/chat/index.html: Language not supported
  • packages/chat/package.json: Language not supported
  • packages/chat/shopping/index.html: Language not supported

Signed-off-by: sunrabbit123 <[email protected]>
sunrabbit123 and others added 5 commits March 27, 2025 17:43
Signed-off-by: sunrabbit123 <[email protected]>
Signed-off-by: sunrabbit123 <[email protected]>
Signed-off-by: sunrabbit123 <[email protected]>
@sunrabbit123 sunrabbit123 marked this pull request as ready for review March 27, 2025 09:36
Signed-off-by: sunrabbit123 <[email protected]>
Copy link
Member

@ryoppippi ryoppippi left a comment

Choose a reason for hiding this comment

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

LGTM for me.
Aside this PR, we need to mark chat library on npm as deprecated.

Waiting for @samchon 's approvement

@sunrabbit123
Copy link
Collaborator Author

remine alert
@samchon

Copy link
Contributor

@samchon samchon left a comment

Choose a reason for hiding this comment

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

Wait until standalone package comes.

@sunrabbit123
Copy link
Collaborator Author

ok good, maybe it is my task... sorry

We have to go through an eslint github action failure while the project is completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

merge packages/chat to website
3 participants