Skip to content

Disclose host-added attribution before Slack sends - #117

Open
christj99 wants to merge 1 commit into
slackapi:mainfrom
christj99:chris/disclose-message-attribution
Open

Disclose host-added attribution before Slack sends#117
christj99 wants to merge 1 commit into
slackapi:mainfrom
christj99:chris/disclose-message-attribution

Conversation

@christj99

Copy link
Copy Markdown

Summary

Add a connector-attribution preflight to slack-messaging so an agent:

  • tells the user when its current host adds sender attribution
  • pauses for an explicit choice before a real or scheduled post
  • offers final text for manual paste when attribution-free delivery matters
  • does not interrupt compose-only, rewrite-only, or draft-only work

This avoids surprising someone who asked the agent to post “as me,” especially for mass mentions or customer-facing messages.

Preview

Documentation and behavioral contract tests only.

Testing

  • make lint
  • make test-unit (20 passed)
  • make typecheck

The LLM eval suite was not run locally because it requires GEMINI_API_KEY and SLACK_MCP_TOKEN.

Notes

The attribution is added by the host rather than the reviewed message body, so the skill does not promise that an edit can remove it.

Requirements

@christj99
christj99 requested a review from a team as a code owner August 11, 2026 15:51
@salesforce-cla

Copy link
Copy Markdown

Thanks for the contribution! Before we can merge this, we need @christj99 to sign the Salesforce Inc. Contributor License Agreement.

@changeset-bot

changeset-bot Bot commented Aug 11, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 4682fc4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
slack Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

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.

1 participant