Skip to content

fix(slack): refresh active thread status on Slack activity - #1534

Merged
Aaronontheweb merged 5 commits into
netclaw-dev:devfrom
Aaronontheweb:issue-1533-slack-processing-state
Jul 1, 2026
Merged

fix(slack): refresh active thread status on Slack activity#1534
Aaronontheweb merged 5 commits into
netclaw-dev:devfrom
Aaronontheweb:issue-1533-slack-processing-state

Conversation

@Aaronontheweb

@Aaronontheweb Aaronontheweb commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • keep ProcessingStateOutput as semantic busy/idle state from the session actor
  • refresh Slack Assistant Thread status locally when Slack thread activity can clear it while processing is still active
  • add Slack binding regressions for inbound user messages and bot replies while processing status is active

Fixes #1533

Validation

  • dotnet test src/Netclaw.Actors.Tests/Netclaw.Actors.Tests.csproj --filter FullyQualifiedName~SlackSessionBindingContractTests
  • dotnet test src/Netclaw.Actors.Tests/Netclaw.Actors.Tests.csproj
  • dotnet slopwatch analyze
  • pwsh ./scripts/Add-FileHeaders.ps1 -Verify
  • git diff --check

@Aaronontheweb Aaronontheweb added channels Discord, Slack, and other channels. UX/DX UI / UX / DX friction issue or user-facing annoyances. labels Jun 30, 2026
@Aaronontheweb
Aaronontheweb force-pushed the issue-1533-slack-processing-state branch from ca650ba to 5d1b055 Compare June 30, 2026 21:33
@Aaronontheweb Aaronontheweb changed the title fix(slack): refresh processing status during tool loops fix(slack): refresh active thread status on Slack activity Jun 30, 2026
@Aaronontheweb
Aaronontheweb marked this pull request as ready for review June 30, 2026 21:49

@Aaronontheweb Aaronontheweb left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

LGTM (afaik - have to test to validate with live Slack instance)

@Aaronontheweb
Aaronontheweb merged commit 5a2ee69 into netclaw-dev:dev Jul 1, 2026
15 checks passed
@Aaronontheweb
Aaronontheweb deleted the issue-1533-slack-processing-state branch July 1, 2026 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

channels Discord, Slack, and other channels. UX/DX UI / UX / DX friction issue or user-facing annoyances.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Slack processing indicator clears mid-turn after first response

1 participant