Skip to content

Polish agentic chat and move agent to Vertex#487

Merged
dtrn2048 merged 5 commits intotestingfrom
feat/better-agentic-chat
Mar 19, 2026
Merged

Polish agentic chat and move agent to Vertex#487
dtrn2048 merged 5 commits intotestingfrom
feat/better-agentic-chat

Conversation

@dtrn2048
Copy link
Copy Markdown
Contributor

Summary

  • move the agentic service from Google AI Studio usage to GCP-backed Vertex configuration and reuse the existing service account setup
  • bring agentic chat UI in line with the main chat experience, including title updates, copy actions, timestamps, and tighter tool-call presentation
  • add transcript chunk deep links so agent citations can jump directly to the referenced transcript block

Testing

  • uv run python -m pytest tests/test_settings.py tests/test_agent_graph.py tests/test_agent_tools.py tests/test_main_routes.py
  • uv run python -m pytest tests/api/test_agentic_api.py
  • pnpm exec tsc --noEmit
  • pnpm exec biome check /Users/dattran/Development/echo/echo/frontend/src/components/chat/AgenticChatPanel.tsx /Users/dattran/Development/echo/echo/frontend/src/components/chat/agenticToolActivity.ts /Users/dattran/Development/echo/echo/frontend/src/components/chat/ChatHistoryMessage.tsx /Users/dattran/Development/echo/echo/frontend/src/components/common/Markdown.tsx /Users/dattran/Development/echo/echo/cypress/e2e/suites/36-agentic-chat-local.cy.js
  • npx cypress run --spec e2e/suites/36-agentic-chat-local.cy.js --env version=local --config video=false

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 18, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: b229e3e9-99c5-49cc-87ba-3b3c030b958a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/better-agentic-chat
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

CodeRabbit can enforce grammar and style rules using `languagetool`.

Configure the reviews.tools.languagetool setting to enable/disable rules and categories. Refer to the LanguageTool Community to learn more.

…tic-chat-pr

# Conflicts:
#	echo/frontend/src/locales/de-DE.ts
#	echo/frontend/src/locales/en-US.ts
#	echo/frontend/src/locales/es-ES.ts
#	echo/frontend/src/locales/fr-FR.ts
#	echo/frontend/src/locales/it-IT.ts
#	echo/frontend/src/locales/nl-NL.ts
# Conflicts:
#	echo/frontend/src/components/chat/ChatHistoryMessage.tsx
#	echo/frontend/src/locales/de-DE.po
#	echo/frontend/src/locales/de-DE.ts
#	echo/frontend/src/locales/en-US.po
#	echo/frontend/src/locales/en-US.ts
#	echo/frontend/src/locales/es-ES.po
#	echo/frontend/src/locales/es-ES.ts
#	echo/frontend/src/locales/fr-FR.po
#	echo/frontend/src/locales/fr-FR.ts
#	echo/frontend/src/locales/it-IT.po
#	echo/frontend/src/locales/it-IT.ts
#	echo/frontend/src/locales/nl-NL.po
#	echo/frontend/src/locales/nl-NL.ts
@dtrn2048 dtrn2048 merged commit 9c1c343 into testing Mar 19, 2026
6 checks passed
ussaama added a commit that referenced this pull request Mar 19, 2026
- Added S3 connectivity check functionality in the `ParticipantConversationAudio` component to ensure audio can be saved before recording starts.
- Introduced a modal to inform users of S3 connectivity issues and provide a reconnect option.
- Updated analytics events to track S3 connectivity failures and reconnect attempts.
- Implemented a new API endpoint to generate a presigned PUT URL for S3 reachability checks.

This change enhances user experience by preventing audio loss due to connectivity issues and providing clear feedback on connection status.
ussaama added a commit that referenced this pull request Mar 19, 2026
…491)

- Added S3 connectivity check functionality in the
`ParticipantConversationAudio` component to ensure audio can be saved
before recording starts.
- Introduced a modal to inform users of S3 connectivity issues and
provide a reconnect option.
- Updated analytics events to track S3 connectivity failures and
reconnect attempts.
- Implemented a new API endpoint to generate a presigned PUT URL for S3
reachability checks.

This change enhances user experience by preventing audio loss due to
connectivity issues and providing clear feedback on connection status.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **New Features**
  * Added S3 connectivity verification before audio recording begins
* Record button shows loading/disabled state while connectivity is
checked
* Blocking, non-dismissible modal appears on connectivity failure with a
“Reconnect” action
* Added analytics events for connectivity check failures and reconnect
attempts

* **Localization**
* Added localized strings for the connectivity modal, message,
suggestions, and reconnect button across languages
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
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