Include search:read.users in the Slack manifest scopes

Rhys Sullivan committed 4 months ago

Commit 63fbb9e1 parent 4cda3233 files · +4 −1Browse files

Unified · Split

3 files

e2e/selfhost/auth-methods-ui.test.ts +1 −0
packages/react/src/components/oauth-app-setup.ts +1 −0
packages/react/src/components/oauth-client-form.test.ts +2 −1

e2e/selfhost/auth-methods-ui.test.ts+1−0

@@ -45,6 +45,7 @@ const SLACK_SCOPES = [ · 44 unmodified lines
45	45	"search:read.mpim",
46	46	"search:read.im",
47	47	"search:read.files",
	48	+ "search:read.users",
48	49	"chat:write",
49	50	"channels:history",
50	51	"groups:history",

packages/react/src/components/oauth-app-setup.ts+1−0

@@ -35,6 +35,7 @@ export const slackMcpUserScopes = [ · 34 unmodified lines
35	35	"search:read.mpim",
36	36	"search:read.im",
37	37	"search:read.files",
	38	+ "search:read.users",
38	39	"chat:write",
39	40	"channels:history",
40	41	"groups:history",

packages/react/src/components/oauth-client-form.test.ts+2−1

@@ -151,8 +151,9 @@ describe("oauthAppSetupFor", () => { · 150 unmodified lines
151	151
152	152	expect(manifest.settings.is_mcp_enabled).toBe(true);
153	153	expect(manifest.oauth_config.redirect_urls).toEqual(["https://executor.sh/api/oauth/callback"]);
154		− expect(manifest.oauth_config.scopes.user.length).toBe(25);
	154	+ expect(manifest.oauth_config.scopes.user.length).toBe(26);
155	155	expect(manifest.oauth_config.scopes.user).toContain("search:read.private");
	156	+ expect(manifest.oauth_config.scopes.user).toContain("search:read.users");
156	157	expect(manifest.oauth_config.scopes.user).toContain("chat:write");
157	158	expect(manifest.settings.agent_view).toBeUndefined();
158	159	});