Skip to content

Commit d1f584c

Browse files
fix(slack): remove redundant conversation pages output
1 parent fb4aac0 commit d1f584c

6 files changed

Lines changed: 3 additions & 13 deletions

File tree

‎apps/docs/content/docs/integrations/slack.mdx‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -985,7 +985,6 @@ List one page of accessible public and private Slack channels. Pass the returned
985985
| `count` | number | Number of conversations returned in this page |
986986
| `hasMore` | boolean | Whether a next cursor is available to fetch more Slack conversations |
987987
| `nextCursor` | string | Cursor to fetch the next page; null when there are no more pages |
988-
| `pages` | number | Number of Slack conversation pages fetched in this invocation \(always 1\) |
989988

990989
### Slack List Channel Members
991990

‎apps/sim/lib/internal/slack/operations/list-conversations.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,6 @@ export const executeSlackListConversationsOperation: InternalToolOperationImplem
153153
count: channels.length,
154154
hasMore: Boolean(nextCursor),
155155
nextCursor,
156-
pages: 1,
157156
},
158157
}
159158
}

‎apps/sim/tools/generated/tool-outputs.ts‎

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

‎apps/sim/tools/slack/list_channels.test.ts‎

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ describe('Slack list channels', () => {
3838
expect(slackListChannelsTool.request).toBeUndefined()
3939
expect(slackListChannelsTool.oauth?.requiredScopes).toEqual([])
4040
expect(slackListChannelsTool.params).not.toHaveProperty('maxPages')
41+
expect(slackListChannelsTool.outputs).not.toHaveProperty('pages')
4142
})
4243

4344
it('remains available through the caller’s own Assistant account without model-supplied tokens', () => {
@@ -125,7 +126,6 @@ describe('Slack list channels', () => {
125126
count: 3,
126127
hasMore: true,
127128
nextCursor: 'cursor-2',
128-
pages: 1,
129129
},
130130
})
131131
})
@@ -178,7 +178,6 @@ describe('Slack list channels', () => {
178178
count: 1,
179179
hasMore: true,
180180
nextCursor: 'cursor-2',
181-
pages: 1,
182181
})
183182
if (typeof first.output.nextCursor !== 'string') throw new Error('Expected a next cursor')
184183

@@ -197,7 +196,6 @@ describe('Slack list channels', () => {
197196
count: 1,
198197
hasMore: false,
199198
nextCursor: null,
200-
pages: 1,
201199
})
202200
})
203201

@@ -214,7 +212,6 @@ describe('Slack list channels', () => {
214212
count: 0,
215213
hasMore: true,
216214
nextCursor: 'cursor-2',
217-
pages: 1,
218215
})
219216
})
220217

@@ -228,7 +225,7 @@ describe('Slack list channels', () => {
228225
const result = await executeSlackListConversationsOperation(BASE_PARAMS)
229226

230227
expect(fetchMock).toHaveBeenCalledTimes(1)
231-
expect(result.output).toMatchObject({ hasMore: false, nextCursor: null, pages: 1 })
228+
expect(result.output).toMatchObject({ hasMore: false, nextCursor: null })
232229
}
233230
)
234231

‎apps/sim/tools/slack/list_channels.ts‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -107,9 +107,5 @@ export const slackListChannelsTool: InternalToolConfig<
107107
description: 'Cursor to fetch the next page; null when there are no more pages',
108108
optional: true,
109109
},
110-
pages: {
111-
type: 'number',
112-
description: 'Number of Slack conversation pages fetched in this invocation (always 1)',
113-
},
114110
},
115111
}

‎apps/sim/tools/slack/types.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1163,7 +1163,6 @@ export interface SlackListChannelsResponse extends ToolResponse {
11631163
count: number
11641164
hasMore: boolean
11651165
nextCursor: string | null
1166-
pages: number
11671166
}
11681167
}
11691168

0 commit comments

Comments
 (0)