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

add execution failureSummary#160

Open
chao0kang wants to merge 4 commits intodevfrom
feature/workflow-execution-fail
Open

add execution failureSummary#160
chao0kang wants to merge 4 commits intodevfrom
feature/workflow-execution-fail

Conversation

@chao0kang
Copy link
Copy Markdown
Contributor

No description provided.

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 22, 2025

Codecov Report

❌ Patch coverage is 97.22222% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...tar.GAgents.GroupChat/WorkflowCoordinatorGAgent.cs 87.50% 0 Missing and 1 partial ⚠️
...GAgents.GroupChat/WorkflowExecutionRecordGAgent.cs 95.65% 0 Missing and 1 partial ⚠️
Files with missing lines Coverage Δ
....GAgents.GroupChat.Core/Events/CoordinatorEvent.cs 100.00% <100.00%> (+41.66%) ⬆️
...upChat.Core/States/WorkflowExecutionRecordState.cs 100.00% <100.00%> (ø)
...s.GroupChat.GroupMember/GAgent/MemberGAgentBase.cs 64.06% <100.00%> (+64.06%) ⬆️
...ent/Coordinator/WorkflowView/WorkflowViewGAgent.cs 73.33% <100.00%> (+0.25%) ⬆️
...Aevatar.GAgents.GroupChat/GroupMemberGAgentBase.cs 98.82% <100.00%> (+57.97%) ⬆️
...tar.GAgents.GroupChat/WorkflowCoordinatorGAgent.cs 87.76% <87.50%> (+6.32%) ⬆️
...GAgents.GroupChat/WorkflowExecutionRecordGAgent.cs 95.83% <95.65%> (+1.02%) ⬆️

... 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.

arg-foo pushed a commit to aevatarAI/aevatar-station that referenced this pull request Sep 2, 2025
…ents

Migrated from aevatarAI/aevatar-gagents#160

This migration maintains exactly 23 files changed (matching original PR):
- GroupChat workflow coordination improvements
- PsiOmni GAgent enhancements and mode updates
- Test suite updates
- Remove deprecated PsiOmniGAgent_Review.cs

Files: 23 files changed (22 modified + 1 deleted)
- 18 source files in gagents/src/
- 4 test files in gagents/test/
- 1 file deletion (PsiOmniGAgent_Review.cs)
arg-foo pushed a commit to aevatarAI/aevatar-station that referenced this pull request Sep 11, 2025
Migrated from aevatarAI/aevatar-gagents#160

This migration includes exactly 12 files as per the original PR:
- Added failure summary tracking to workflow execution
- Enhanced member and coordinator failure handling
- Added comprehensive unit tests for failure scenarios

Changes:
- 3 files in GroupChat.Core (events, models, states)
- 1 file in GroupChat.GroupMember
- 4 files in GroupChat (coordinator and record agents)
- 4 test files (csproj, worker agent, and test cases)
chao0kang pushed a commit to aevatarAI/aevatar-station that referenced this pull request Sep 11, 2025
…les only) (#1426)

Migrated from aevatarAI/aevatar-gagents#160

## Summary
This is the **correct** migration of PR #160 with exactly 12 files as
per the original PR.

**Previous PR #1380 incorrectly included 23 files including unrelated
PsiOmni changes.**

## Changes in this PR (12 files total)
### Core Changes (3 files)
-  - Added FailureSummary property
-  - Added Failed status
-  - Added failure tracking

### Implementation (5 files)
-  - Failure handling
-  - Added ConfirmEvents
-  - Enhanced failure response
-  - Failure summary propagation
-  - Failure recording

### Tests (4 files)
-  - Test dependencies
-  - Test worker with failure simulation
-  - Failure scenario tests
-  - Unit tests

## Testing
- ✅ Build successful
- ✅ All 27 GroupChat tests pass
- ✅ Verified exactly 12 files changed

Closes #1380 (incorrect migration)
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.

1 participant