fix(slack): add nested profile object to get_user output so users.info-shaped references resolve#4954
fix(slack): add nested profile object to get_user output so users.info-shaped references resolve#4954waleedlatif1 wants to merge 90 commits into
Conversation
…ership workflow edits via sockets, ui improvements
…ng improvements, posthog, secrets mutations
…ration, signup method feature flags, SSO improvements
…nts, secrets performance, polling refactors, drag resources in mothership
…y invalidation, HITL docs
…endar triggers, docs updates, integrations/models pages improvements
…ions, jira forms endpoints
…mat, logs performance improvements fix(csp): add missing analytics domains, remove unsafe-eval, fix workspace CSP gap (#4179) fix(landing): return 404 for invalid dynamic route slugs (#4182) improvement(seo): optimize sitemaps, robots.txt, and core web vitals across sim and docs (#4170) fix(gemini): support structured output with tools on Gemini 3 models (#4184) feat(brightdata): add Bright Data integration with 8 tools (#4183) fix(mothership): fix superagent credentials (#4185) fix(logs): close sidebar when selected log disappears from filtered list; cleanup (#4186)
v0.6.46: mothership streaming fixes, brightdata integration
…m integration, atlassian triggers
…s trimmed v0.6.84: redis pub/sub SNI override, security hardening, copilot reads trimmed
…il integrations, markdown google docs creation
…d new relic integrations
…oogle slides endpoints, DB access pattern improvements
…er, search & replace UX, kb connectors multi-select, mcp negative cache
…fal-ai hosted key
… support, executor code cleanup
…ter, hosted-key queueing v0.6.92: enrichment table column type, table run fixes, scheduled jitter, hosted-key queueing
… messages table, user stats hot path improvements, RB2B
v0.6.95: data enrichment block, nullable workflow description fix
…ables UI improvements, new slack scopes, model-level denylists, object storage tracespans
…ble, integration icons audit
…ion, together AI, Baseten, Ollama Cloud, Linq integrations, kb doc ownership bindings
…dgebase connector, SSO provider ID allowlist, singleton memory leak fix
…rs, clickhouse integration
…ration, smooth streaming, security hardening, db fixes
…o-shaped references resolve
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview The Reviewed by Cursor Bugbot for commit d562353. Configure here. |
|
| GitGuardian id | GitGuardian status | Secret | Commit | Filename | |
|---|---|---|---|---|---|
| 29606901 | Triggered | Generic High Entropy Secret | a54dcbe | apps/sim/providers/utils.test.ts | View secret |
| 32763747 | Triggered | Generic Password | 3e9849b | helm/sim/tests/validators_test.yaml | View secret |
| 33881454 | Triggered | Generic Password | 7ffc495 | apps/sim/app/api/chat/utils.test.ts | View secret |
🛠 Guidelines to remediate hardcoded secrets
- Understand the implications of revoking this secret by investigating where it is used in your code.
- Replace and store your secrets safely. Learn here the best practices.
- Revoke and rotate these secrets.
- If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.
To avoid such incidents in the future consider
- following these best practices for managing and storing secrets including API keys and other credentials
- install secret detection on pre-commit to catch secret before it leaves your machine and ease remediation.
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
Greptile SummaryThis PR fixes a
Confidence Score: 5/5Safe to merge — the change is purely additive, all new fields fall back to empty strings or null, and existing flat fields on the output object are untouched. The change adds a well-scoped nested No files require special attention. Important Files Changed
Sequence DiagramsequenceDiagram
participant U as User/Block
participant T as slackGetUserTool
participant S as Slack users.info API
U->>T: userId
T->>S: "GET /api/users.info?user=<id>"
S-->>T: "{ ok: true, user: { id, name, ..., profile: { email, display_name, ... } } }"
T->>T: transformResponse()
Note over T: Flat fields (user.email, user.display_name, ...)<br/>Nested profile (user.profile.email, user.profile.display_name, ...)
T-->>U: "{ success: true, output: { user: { ...flatFields, profile: { ...profileFields } } } }"
Reviews (1): Last reviewed commit: "fix(slack): add nested profile object to..." | Re-trigger Greptile |
Summary
profileobject to the Slack Get User tool output, mirroring Slack'susers.infoAPI shape, so references like<block.user.profile.email>resolve instead of throwingInvalidFieldErrorUSER_PROFILE_OUTPUT_PROPERTIESschema intoUSER_OUTPUT_PROPERTIESand add aSlackUserProfiletypeuser.email, etc.) are unchanged for backward compatibilityType of Change
Testing
Typecheck passes with no slack-related errors; block-outputs/block-reference/slack test suites pass (34 tests)
Checklist