Skip to content

refact: extract the core logic into core subfolder #1774

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
Draft
wants to merge 25 commits into
base: fa
Choose a base branch
from
Draft

refact: extract the core logic into core subfolder #1774

wants to merge 25 commits into from

Conversation

@WannaTen
Copy link
Contributor

Features

  • extract the core logic of metagpt into core subfolder, making whole project more clear.

@WannaTen
refact: extract the core logic into core subfolder
7567141 
Signed-off-by: Lin Yi <teroincn@gmail.com>
better629
better629 reviewed Mar 21, 2025
View reviewed changes
metagpt/core/tools/base.py
@@ -0,0 +1,13 @@
from pydantic import BaseModel

Copy link
Member

Choose a reason for hiding this comment

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

seems missing tool_*.py

metagpt/core/roles/base.py Outdated
from typing import Optional, Union

from metagpt.core.base.base_serialization import BaseSerialization

Copy link
Member

Choose a reason for hiding this comment

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

seems missing RoleZero

metagpt/core/actions/base.py Outdated
@@ -0,0 +1,29 @@
#!/usr/bin/env python
# -*- coding: utf-8 -*-
"""
Copy link
Member

Choose a reason for hiding this comment

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

seems missing action_*.py

@WannaTen WannaTen deployed to pre-commit July 25, 2025 06:01 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@better629 better629 better629 left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@WannaTen @better629 @openhands-agent