Skip to content

feat(slack): add explicit bounded history access - #79796

Open
mats-i wants to merge 1 commit into
NousResearch:mainfrom
mats-i:codex/slack-history-54535-compat
Open

feat(slack): add explicit bounded history access#79796
mats-i wants to merge 1 commit into
NousResearch:mainfrom
mats-i:codex/slack-history-54535-compat

Conversation

@mats-i

@mats-i mats-i commented Aug 6, 2026

Copy link
Copy Markdown

Summary

  • add an explicit, one-shot Slack history tool scoped to the active workspace, channel, and thread
  • require deterministic fail-closed user authorization and bound reads to 12 messages without pagination
  • prevent raw history from leaking into persistence, hooks, traces, fallback providers, Relay, MoA, or automatic thread hydration
  • document the consent and retention contract

Verification

  • 750 passed, 4 skipped across the affected agent, gateway, Slack, persistence, fallback, and streaming suites
  • Ruff clean
  • git diff --check clean
  • fresh-context adversarial review: ACCEPT

Notes

  • prior Slack content is never injected automatically; only an explicit direct request authorizes one read for the current turn
  • the final assistant answer remains normally visible and persistable

@alt-glitch alt-glitch added type/feature New feature or request comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/gateway Gateway runner, session dispatch, delivery comp/plugins Plugin system and bundled plugins comp/tools Tool registry, model_tools, toolsets platform/slack Slack app adapter P3 Low — cosmetic, nice to have needs-decision Awaiting maintainer decision before any implementation sweeper:risk-session-state Sweeper risk: may lose/corrupt/mis-associate session or context state sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages labels Aug 6, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

This was generated by AI during triage.

Related: #54535 and #29489 are open Slack history implementations for #6345. This PR adds a stricter one-shot authorization and ephemeral-retention design; maintainers should choose the preferred scope rather than treat the implementations as duplicates.

@andrexibiza

Copy link
Copy Markdown
Contributor

Campaign interlock

Related: #6345. Related: #79772. Collision candidate; retain contributor credit while adjudicating #6345.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/gateway Gateway runner, session dispatch, delivery comp/plugins Plugin system and bundled plugins comp/tools Tool registry, model_tools, toolsets needs-decision Awaiting maintainer decision before any implementation P3 Low — cosmetic, nice to have platform/slack Slack app adapter sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages sweeper:risk-session-state Sweeper risk: may lose/corrupt/mis-associate session or context state type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants