Skip to content

Fix empty bubbles - #34

Merged
rdehuyss merged 1 commit into
mainfrom
bugfix/empty-bubbles
Mar 26, 2026
Merged

rdehuyss merged 1 commit into
mainfrom
bugfix/empty-bubbles

Conversation

@auloin

@auloin auloin commented Mar 26, 2026

Copy link
Copy Markdown
Contributor

This simply filters out empty string from agent response. Empty string is returned by Spring AI when the agent is making a tool call.

See #23 for discussion around showing the actually tool calls in the web chat

@cla-bot cla-bot Bot added the cla-signed label Mar 26, 2026
@rdehuyss
rdehuyss merged commit 8bdffb7 into main Mar 26, 2026
1 check passed
@rdehuyss

Copy link
Copy Markdown
Collaborator

P.s. there is a branch that saves tool calls but it negatively impacts token usage in the long run

@auloin
auloin deleted the bugfix/empty-bubbles branch March 26, 2026 12:32
a-simeshin added a commit to a-simeshin/JavaClaw that referenced this pull request Apr 10, 2026
…s с 11 Playwright E2E тестами покрывающими E4 (SSE push → success/error/approval toasts), E5 (error card rendering), E9 (approval respond API + pending approvals + text reply), E10 (expired approval timeout + 404 on respond), E19 (SSE endpoint conversation-routing) — TypeScript/Vite build чистые, все 105 frontend и 537 backend тестов зелёные
a-simeshin added a commit to a-simeshin/JavaClaw that referenced this pull request Apr 11, 2026
…st per role — V33 миграция, SkillRoleAllowlist entity/repository, SkillVisibilityService, REST API endpoints для visibility и allowlist, 17 новых тестов, все 1046 тестов зелёные
arefbehboudi pushed a commit to arefbehboudi/JavaClaw that referenced this pull request Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants