Skip to content

Refactor project scaffolding around FsTreeNode - #1943

Merged
aidandaly24 merged 1 commit into
refactorfrom
tighten-up
Aug 7, 2026
Merged

Refactor project scaffolding around FsTreeNode#1943
aidandaly24 merged 1 commit into
refactorfrom
tighten-up

Conversation

@AlexanderRichey

Copy link
Copy Markdown
Contributor

Clean up project scaffolding a bit.

@github-actions github-actions Bot added the agentcore-harness-reviewing AgentCore Harness review in progress label Aug 7, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 99.29078% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 96.41%. Comparing base (ae6e810) to head (ae1fba7).
⚠️ Report is 5 commits behind head on refactor.

Files with missing lines Patch % Lines
src/core/project/fsUtils.ts 90.90% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           refactor    #1943      +/-   ##
============================================
+ Coverage     96.28%   96.41%   +0.13%     
============================================
  Files           245      270      +25     
  Lines         12105    13330    +1225     
============================================
+ Hits          11655    12852    +1197     
- Misses          450      478      +28     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

@github-actions github-actions Bot removed the agentcore-harness-reviewing AgentCore Harness review in progress label Aug 7, 2026

@tejaskash tejaskash left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approving, great refactor and I like the AsyncGenerator pattern more than cb's

@aidandaly24

Copy link
Copy Markdown
Contributor

Makes sense to me I really like how clean the fsTree class and functions are

@aidandaly24
aidandaly24 merged commit 64058e3 into refactor Aug 7, 2026
10 checks passed
@aidandaly24
aidandaly24 deleted the tighten-up branch August 7, 2026 16:30
This was referenced Sep 3, 2026
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.

4 participants