Skip to content

doc: add assistant_view in features#67650

Open
insomnius wants to merge 2 commits intoopenclaw:mainfrom
insomnius:patch-3
Open

doc: add assistant_view in features#67650
insomnius wants to merge 2 commits intoopenclaw:mainfrom
insomnius:patch-3

Conversation

@insomnius
Copy link
Copy Markdown

@insomnius insomnius commented Apr 16, 2026

Summary

image image

Add assistant_view in sample manifest

  • Problem: Slack required assistant_view listed in features so that we can use of assistant:write oauth scope
  • Why it matters: it matters so that assistant:write oauth access can be used
  • What changed: adding assistant_view in list of features
  • What did NOT change (scope boundary): -

Change Type (select all)

  • Bug fix
  • Feature
  • Refactor required for the fix
  • Docs
  • Security hardening
  • Chore/infra

Scope (select all touched areas)

  • Gateway / orchestration
  • Skills / tool execution
  • Auth / tokens
  • Memory / storage
  • Integrations
  • API / contracts
  • UI / DX
  • CI/CD / infra

Linked Issue/PR

Not applicable

Root Cause (if applicable)

For bug fixes or regressions, explain why this happened, not just what changed. Otherwise write N/A. If the cause is unclear, write Unknown.

  • Root cause: assistant_view feature should be listed when assistant:write is in oauth scope
  • Missing detection / guardrail:
  • Contributing context (if known):

Regression Test Plan (if applicable)

Not applicable

User-visible / Behavior Changes

No behavior changes

Diagram (if applicable)

Not applicable

Security Impact (required)

  • New permissions/capabilities? No
  • Secrets/tokens handling changed? No
  • New/changed network calls? No
  • Command/tool execution surface changed? No
  • Data access scope changed? No
  • If any Yes, explain risk + mitigation: No

Repro + Verification

Not applicable

Evidence

Not applicable

Human Verification (required)

I verified by creating an app with the manifest I just changed, and it doesn't show any warning

Review Conversations

  • I replied to or resolved every bot review conversation I addressed in this PR.
  • I left unresolved only the conversations that still need reviewer or maintainer judgment.

If a bot review conversation is addressed by this PR, resolve that conversation yourself. Do not leave bot review conversation cleanup for maintainers.

Compatibility / Migration

  • Backward compatible? Yes
  • Config/env changes? No
  • Migration needed? No
  • If yes, exact upgrade steps: No

Risks and Mitigations

No risk

@openclaw-barnacle openclaw-barnacle bot added docs Improvements or additions to documentation channel: slack Channel integration: slack size: XS labels Apr 16, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 16, 2026

Greptile Summary

This PR adds the assistant_view feature block to the Slack app manifest examples in docs/channels/slack.md for both Socket Mode and HTTP Request URL variants. The change is needed because Slack requires assistant_view to be declared in features when the assistant:write OAuth scope is used, preventing a validation warning when creating the app from the manifest.

Confidence Score: 5/5

Safe to merge — docs-only change that adds a required Slack manifest field, verified by the author.

Single documentation change adding a missing manifest field consistently across both manifest variants. No logic, no code, and the author confirmed it resolves the Slack validation warning.

No files require special attention.

Reviews (1): Last reviewed commit: "Add assistant_view in features" | Re-trigger Greptile

@insomnius insomnius changed the title Add assistant_view in features doc: add assistant_view in features Apr 16, 2026
@zimeg
Copy link
Copy Markdown
Member

zimeg commented Apr 19, 2026

👋 Hey @insomnius! Thanks for taking note of this! 👾

We're aware that these warnings appear in Slack app settings at the moment, but we're holding off on this feature for now with some considerations to the current default experience across the Slack UI.

AFAICT the warning shouldn't cause problems in interactions that use the recommended defaults but please let me know of other findings! 🔭

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

Labels

channel: slack Channel integration: slack docs Improvements or additions to documentation size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants