Skip to content

[codex] Clean non-production assets#858

Open
eanzhao wants to merge 2 commits into
devfrom
chore/2026-05-23_mainline-code-audit
Open

[codex] Clean non-production assets#858
eanzhao wants to merge 2 commits into
devfrom
chore/2026-05-23_mainline-code-audit

Conversation

@eanzhao
Copy link
Copy Markdown
Contributor

@eanzhao eanzhao commented May 23, 2026

Summary

  • remove non-production demos, root workflow samples, local tool apps, generated logs, and their now-orphaned tests
  • keep AI-facing assets and the apps/aevatar-console-web frontend workspace intact
  • restore CI guard assets under tools/ci and tools/docs, while making guards tolerate the removed demo/legacy CLI paths
  • update docs, solution filters, workflow catalog loading, and affected tests so backend/frontend production paths still build and run

Validation

  • dotnet restore aevatar.slnx --nologo -p:NuGetAudit=false
  • dotnet build aevatar.slnx --no-restore --nologo -p:NuGetAudit=false
  • bash tools/ci/architecture_guards.sh
  • bash tools/ci/test_solution_ownership_guard.sh
  • bash tools/ci/test_stability_guards.sh
  • bash tools/ci/projection_route_mapping_guard.sh
  • bash tools/ci/slow_test_guards.sh
  • bash test/tools/test_docs_tools.sh
  • pnpm --dir apps/aevatar-console-web install --frozen-lockfile
  • pnpm --dir apps/aevatar-console-web tsc
  • pnpm --dir apps/aevatar-console-web test --runInBand
  • pnpm --dir apps/aevatar-console-web build

Note: full backend dotnet test aevatar.slnx --no-build --nologo -p:NuGetAudit=false was run during cleanup. The only remaining failure observed there was an external OpenAI Realtime integration timeout when running in the full suite; the affected OpenAI test project passed when rerun separately.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.51%. Comparing base (fccb80d) to head (4a02998).

@@            Coverage Diff             @@
##              dev     #858      +/-   ##
==========================================
- Coverage   83.06%   82.51%   -0.56%     
==========================================
  Files         981      981              
  Lines       61936    61933       -3     
  Branches     8069     8068       -1     
==========================================
- Hits        51447    51102     -345     
- Misses       7009     7376     +367     
+ Partials     3480     3455      -25     
Flag Coverage Δ
ci 82.51% <ø> (-0.56%) ⬇️

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

Files with missing lines Coverage Δ
...n/WorkflowCapabilityServiceCollectionExtensions.cs 100.00% <ø> (ø)
...ructure/Workflows/FileBackedWorkflowCatalogPort.cs 69.91% <ø> (-2.58%) ⬇️

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

@eanzhao eanzhao marked this pull request as ready for review May 23, 2026 03:11
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.

1 participant