Skip to content
This repository was archived by the owner on Mar 31, 2026. It is now read-only.

Migrate PR #163: Complete migration with all 14 file changes#1397

Draft
arg-foo wants to merge 1 commit intodevfrom
feature/migrated-pr-163-complete
Draft

Migrate PR #163: Complete migration with all 14 file changes#1397
arg-foo wants to merge 1 commit intodevfrom
feature/migrated-pr-163-complete

Conversation

@arg-foo
Copy link
Copy Markdown
Contributor

@arg-foo arg-foo commented Sep 4, 2025

Overview

Migrated from aevatarAI/aevatar-gagents#163

This PR is the complete migration of all changes from PR-163, replacing the incomplete migration in #1396.

What's Included (13 files, excluding .sln)

New Files (2):

  • gagents/src/Aevatar.GAgents.Basic/Common/GenericMetaAttribute.cs - Flexible attribute system for metadata
  • gagents/src/Aevatar.GAgents.Twitter/GAgents/ChatAIAgent/ChatAISystemLLMEnum.cs - LLM provider enum with structured metadata

Modified Files (11) - Added Description attributes:

  • gagents/src/Aevatar.GAgents.AIGAgent/Dtos/LLMConfigDto.cs
  • gagents/src/Aevatar.GAgents.ChatAgent/ChatConfigDto.cs
  • gagents/src/Aevatar.GAgents.GroupChat.Core/Configs/GroupMemberConfigDto.cs
  • gagents/src/Aevatar.GAgents.GroupChat.Core/Configs/WorkflowCoordinatorConfigDto.cs
  • gagents/src/Aevatar.GAgents.GroupChat.GroupMember/GAgent/Dto/MemberConfigDto.cs
  • gagents/src/Aevatar.GAgents.GroupChat/GAgent/Coordinator/WorkflowView/Dto/WorkflowViewConfigDto.cs
  • gagents/src/Aevatar.GAgents.InputGAgent/Dto/InputConfigDto.cs
  • gagents/src/Aevatar.GAgents.MCP.Core/Options/MCPGAgentConfig.cs
  • gagents/src/Aevatar.GAgents.MCP.Core/Options/MCPServerConfig.cs
  • gagents/src/Aevatar.GAgents.Twitter/GAgents/ChatAIAgent/ChatAIGAgent.cs
  • gagents/src/Aevatar.GAgents.Twitter/GAgents/ChatAIAgent/ChatAIGAgentConfigDto.cs

Migration Details

  • All ConfigDto classes now have comprehensive Description attributes for better documentation
  • GenericMetaAttribute enables flexible metadata for enums and other types
  • ChatAISystemLLMEnum provides structured metadata for LLM providers

Testing

✅ Build completed successfully with no errors

Note

The solution file (AevatarGAgents.sln) was excluded from this migration to avoid potential conflicts.

…tar-gagents

- Add GenericMetaAttribute and ChatAISystemLLMEnum for metadata framework
- Add Description attributes to all ConfigDto properties across the project
- Cover all major configuration classes including:
  * MemberConfigDto and GroupMemberConfigDto
  * LLMConfigDto and SelfLLMConfig
  * ChatConfigDto and ChatAIGAgentConfigDto
  * InputConfigDto and PsiOmniGAgentConfig
  * MCPGAgentConfig and WorkflowCoordinatorConfigDto
  * WorkflowUnitDto and related workflow classes
- Migrated from aevatarAI/aevatar-gagents#163
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 4, 2025

Codecov Report

❌ Patch coverage is 22.00000% with 39 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...vatar.GAgents.Basic/Common/GenericMetaAttribute.cs 0.00% 35 Missing ⚠️
.../src/Aevatar.GAgents.AIGAgent/Dtos/LLMConfigDto.cs 0.00% 3 Missing ⚠️
...Agents.Twitter/GAgents/ChatAIAgent/ChatAIGAgent.cs 0.00% 1 Missing ⚠️

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #1397      +/-   ##
==========================================
- Coverage   40.00%   39.76%   -0.24%     
==========================================
  Files         637      638       +1     
  Lines       25647    25748     +101     
  Branches     2399     2408       +9     
==========================================
- Hits        10259    10238      -21     
- Misses      14795    14920     +125     
+ Partials      593      590       -3     
Flag Coverage Δ
unittests 39.76% <22.00%> (-0.24%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...nts/src/Aevatar.GAgents.ChatAgent/ChatConfigDto.cs 0.00% <ø> (ø)
...nts.GroupChat.Core/Configs/GroupMemberConfigDto.cs 100.00% <100.00%> (ø)
...pChat.Core/Configs/WorkflowCoordinatorConfigDto.cs 100.00% <100.00%> (ø)
...roupChat.GroupMember/GAgent/Dto/MemberConfigDto.cs 100.00% <ø> (ø)
...rdinator/WorkflowView/Dto/WorkflowViewConfigDto.cs 100.00% <ø> (ø)
.../Aevatar.GAgents.InputGAgent/Dto/InputConfigDto.cs 100.00% <ø> (ø)
...evatar.GAgents.MCP.Core/Options/MCPGAgentConfig.cs 100.00% <ø> (ø)
...evatar.GAgents.MCP.Core/Options/MCPServerConfig.cs 62.50% <ø> (ø)
...itter/GAgents/ChatAIAgent/ChatAIGAgentConfigDto.cs 100.00% <100.00%> (ø)
...Agents.Twitter/GAgents/ChatAIAgent/ChatAIGAgent.cs 10.00% <0.00%> (ø)
... and 2 more

... and 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Charles-aelf Charles-aelf self-assigned this Sep 26, 2025
@Charles-aelf Charles-aelf marked this pull request as draft September 26, 2025 06:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants