Skip to content

feat: allTick&ozon seller toolset #299

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: allTick&ozon seller toolset #299

wants to merge 2 commits into from

Conversation

@ZUJIAJIA
Copy link

@ZUJIAJIA ZUJIAJIA commented Nov 19, 2025

looong,这是 huang的pr

@cla-assistant
Copy link

cla-assistant bot commented Nov 19, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@cla-assistant
Copy link

cla-assistant bot commented Nov 19, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions
Copy link

github-actions bot commented Nov 19, 2025 •
edited
Loading

✅ Build Successful - Preview Images for this PR:

registry.cn-hangzhou.aliyuncs.com/fastgpt/fastgpt-plugin-pr:f53016d94094180d5ec5ba46251a3c2bdbb1fd4e

Changed packages:
allTick ozon

@c121914yu c121914yu requested a review from Copilot November 20, 2025 06:59
Copilot finished reviewing on behalf of c121914yu November 20, 2025 07:00
Copilot AI reviewed Nov 20, 2025
View reviewed changes
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 adds two new toolsets for FastGPT: allTick (financial market data) and Ozon Seller (e-commerce product management). The changes include complete implementations with configurations, tools, tests, and documentation templates.

Key changes:

  • Added allTick toolset with 4 tools for financial market data (trade tick, static info, k-line, depth tick)
  • Added Ozon Seller toolset with 10 tools for e-commerce product management (warehouse, stock, pricing, product creation, etc.)
  • Added numerous dependencies to root package.json for database, document processing, and API integrations

Reviewed Changes

Copilot reviewed 67 out of 85 changed files in this pull request and generated no comments.

File Description
package.json Added extensive dependencies for DB drivers, document processing (docx, excel, pptx), scraping, and AI integrations
modules/tool/packages/ozon/* Complete Ozon Seller toolset implementation with 10 child tools for product/warehouse management
modules/tool/packages/allTick/* Complete allTick toolset implementation with 4 child tools for financial market data queries

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

FinleyGe
FinleyGe requested changes Nov 21, 2025
View reviewed changes
Copy link
Collaborator

@FinleyGe FinleyGe left a comment

Choose a reason for hiding this comment

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

  1. 子工具的 logo，如果没有就删，会拿父工具的，否则默认上传 fastgpt logo
  2. DESIGN.md 没有用就删了
  3. README.md 写一下
  4. 依赖不要装外面，最外层的 bun.lock 和 package.json 不会更新

package.json
Copy link
Collaborator

@FinleyGe FinleyGe Nov 21, 2025

Choose a reason for hiding this comment

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

最外面的 package.json 不应该发生更新

bun.lock
Copy link
Collaborator

@FinleyGe FinleyGe Nov 21, 2025

Choose a reason for hiding this comment

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

这个 lock 文件也不应该发生更新

modules/tool/packages/ozon/README.md
Copy link
Collaborator

@FinleyGe FinleyGe Nov 21, 2025

Choose a reason for hiding this comment

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

写个 README.md

modules/tool/packages/ozon/package.json
Copy link
Collaborator

@FinleyGe FinleyGe Nov 21, 2025

Choose a reason for hiding this comment

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

工具内部的依赖需要装在内部的 package.json 中

modules/tool/packages/ozon/DESIGN.md Outdated
Copy link
Collaborator

@FinleyGe FinleyGe Nov 21, 2025

Choose a reason for hiding this comment

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

没用这个就删掉

modules/tool/packages/allTick/DESIGN.md Outdated
Copy link
Collaborator

@FinleyGe FinleyGe Nov 21, 2025

Choose a reason for hiding this comment

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

没写就删掉

modules/tool/packages/allTick/children/tradeTick/logo.svg Outdated
Copy link
Collaborator

@FinleyGe FinleyGe Nov 21, 2025

Choose a reason for hiding this comment

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

删

modules/tool/packages/allTick/children/staticInfo/logo.svg Outdated
Copy link
Collaborator

@FinleyGe FinleyGe Nov 21, 2025

Choose a reason for hiding this comment

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

删

modules/tool/packages/allTick/children/kline/logo.svg Outdated
Copy link
Collaborator

@FinleyGe FinleyGe Nov 21, 2025

Choose a reason for hiding this comment

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

删

modules/tool/packages/allTick/children/depthTick/logo.svg Outdated
Copy link
Collaborator

@FinleyGe FinleyGe Nov 21, 2025

Choose a reason for hiding this comment

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

删

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@FinleyGe FinleyGe FinleyGe requested changes

Assignees

No one assigned

Labels

size/XXL

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ZUJIAJIA @FinleyGe