Draft
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## dev #213 +/- ##
==========================================
- Coverage 81.87% 81.86% -0.02%
==========================================
Files 786 786
Lines 50533 50533
Branches 6642 6642
==========================================
- Hits 41376 41370 -6
- Misses 6317 6323 +6
Partials 2840 2840
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This draft PR implements the Phase A scope of #205 only.
It integrates the browser-based voice UI into the existing CLI web app so
aevatar voice --agent <actorId>can ensure the local UI host is running, point it at the selected backend, and open/voice?agent=<actorId>.What Changed
aevatar voiceCLI entrypoint for launching the Phase A voice pagechatandvoiceuse the same probe/start/proxy-target flow/voiceReact page with microphone capture, websocket session handling, playout metrics, and provider/sample-rate controlsChronoAIProject/voice-presence/ws/voice/{actorId}in both workflow and mainnet hostsWhy
Issue #205 Phase A requires a browser voice UI first, with the CLI acting as the entrypoint. The Phase A goal is to reuse the existing web UI shell instead of introducing a parallel standalone host, while keeping the voice websocket path available from the backend hosts.
User / Developer Impact
aevatar voice --agent <actorId>to open the browser voice UI directly/ws/voice/{actorId}traffic to the configured backendValidation
npm run buildintools/Aevatar.Tools.Cli/Frontenddotnet test test/Aevatar.Tools.Cli.Tests/Aevatar.Tools.Cli.Tests.csproj --nologo -m:1 -p:UseSharedCompilation=false --filter 'FullyQualifiedName~VoiceCommandTests'dotnet test test/Aevatar.Tools.Cli.Tests/Aevatar.Tools.Cli.Tests.csproj --nologo -m:1 -p:UseSharedCompilation=false --filter 'FullyQualifiedName~RootCommandFactoryTests'dotnet test test/Aevatar.Tools.Cli.Tests/Aevatar.Tools.Cli.Tests.csproj --nologo -m:1 -p:UseSharedCompilation=false --filter 'FullyQualifiedName~AppPlaygroundHostTests'dotnet build src/workflow/Aevatar.Workflow.Host.Api/Aevatar.Workflow.Host.Api.csproj --nologo -m:1 -p:UseSharedCompilation=falsedotnet build src/Aevatar.Mainnet.Host.Api/Aevatar.Mainnet.Host.Api.csproj --nologo -m:1 -p:UseSharedCompilation=falseNotes
WorkflowStudioBranchCoverageTestsstill fails in the fullAevatar.Tools.Cli.Testsproject run; it is outside this diff and was not changed hereChronoAIProject/voice-presence