Skip to content

Commit 0b28afe

Browse files
committed
chore(slack): trim verbose comment
1 parent be61b2d commit 0b28afe

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

  • apps/sim/app/api/tools/slack/channels

apps/sim/app/api/tools/slack/channels/route.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,8 @@ const SCOPED_USER_ID_PATTERN =
3636
function parseScopedSlackUserId(accountId: string): string | null {
3737
const match = SCOPED_USER_ID_PATTERN.exec(accountId)
3838
if (match) return match[1]
39-
// A marker is present but the id didn't parse — surface it instead of
40-
// silently dropping to the bot `is_member` filter, which would bypass the
41-
// installer-scoped privacy check without any signal.
39+
// Marker present but unparseable — surface it rather than silently falling
40+
// back to the bot `is_member` filter and bypassing the privacy scope.
4241
if (accountId.includes('-usr_')) {
4342
logger.warn('Slack accountId carries usr_ marker but did not parse; using is_member fallback', {
4443
accountId,

0 commit comments

Comments
 (0)