Skip to content

Deliver Slack research sources and complete tool continuations - #8

Merged
jerelvelarde merged 8 commits into
mainfrom
jerel/slack-research-completion
Sep 11, 2026
Merged

jerelvelarde merged 8 commits into
mainfrom
jerel/slack-research-completion

Conversation

@jerelvelarde

@jerelvelarde jerelvelarde commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Slack research now posts native source buttons directly from validated Exa URLs, makes search failures visible, and retains the evidence for the agent's answer. A channel-only wrapper creates a fresh inner agent for each tool-continuation run, preserving transcript, state and cancellation while avoiding the pinned runtime's already-running guard.

The app uses managed Slack's default hidden tool progress. With the optional progress stream enabled, live runs delivered cards but dropped final prose and left the working indicator visible. The default configuration passed the full live flow. The search tool also tells the agent that returned sources have already been posted as native cards.

Validation at bb4e0dc:

  • Root typecheck and verification pass: 50 root tests, 22 channel tests, 34 web tests. Independent source and regression reviews have no remaining findings. Tests reproduce the real BuiltInAgent continuation guard and cover transcript/state, clone, cancellation and stream cleanup; removing cleanup makes its regression fail.
  • Fresh managed Slack/OpenAI/Exa run passed: thread-aware incident card, native source cards, an AWS source button opening its PDF, final explanation with two next checks, and cleared working indicator. No closed-path or handler-failure events were captured.
  • Current main's three-app layout, web persistence, React Native, credits guidance and CI/Auth0 cleanup are preserved. No removed directories are restored; shared web/mobile runtime behavior and dependency pins are unchanged.

Live OpenRouter calls remain unverified. Temporary diagnostic logging used during investigation is excluded from this PR.

@jerelvelarde
jerelvelarde force-pushed the jerel/slack-research-completion branch from 0372cca to a833f2d Compare September 11, 2026 20:07
@jerelvelarde
jerelvelarde changed the base branch from jerel/template-readiness to main September 11, 2026 20:07
@jerelvelarde
jerelvelarde force-pushed the jerel/slack-research-completion branch from a833f2d to b90997a Compare September 11, 2026 20:24
@jerelvelarde
jerelvelarde force-pushed the jerel/slack-research-completion branch from b90997a to 281189d Compare September 11, 2026 21:09
@jerelvelarde jerelvelarde changed the title fix(slack): publish search sources in the thread Deliver Slack research sources and preserve tool continuation Sep 11, 2026
@jerelvelarde jerelvelarde changed the title Deliver Slack research sources and preserve tool continuation Deliver Slack research sources and complete tool continuations Sep 11, 2026
@jerelvelarde
jerelvelarde marked this pull request as ready for review September 11, 2026 21:28
@jerelvelarde
jerelvelarde merged commit 95b3fe0 into main Sep 11, 2026
@jerelvelarde
jerelvelarde deleted the jerel/slack-research-completion branch September 12, 2026 16:44
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