feat(slack): App Home quick-start tab - #596
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
The latest updates on your projects. Learn more about Unkey Deploy
|
Greptile SummaryThe PR adds a static Slack App Home quick-start view and publishes it when users open the Home tab.
Confidence Score: 4/5The PR appears safe to merge, with a non-blocking typing issue in the new App Home view that should be cleaned up. The manifest, command names, listener registration, and production Slack client path are aligned; the remaining concern is that generic unknown-based types bypass compile-time validation of the Block Kit payload. Files Needing Attention: apps/slack/src/slack/app-home.ts, apps/slack/src/slack/listeners.ts, apps/slack/src/slack/app-home.test.ts Important Files Changed
Sequence DiagramsequenceDiagram
participant U as Slack user
participant S as Slack
participant L as App listener
participant V as views.publish
U->>S: Open Databuddy Home tab
S->>L: app_home_opened
L->>L: Build static quick-start view
L->>V: Publish view for event.user
V-->>U: Render App Home
Reviews (1): Last reviewed commit: "feat(slack): publish an App Home tab wit..." | Re-trigger Greptile |
| @@ -0,0 +1,43 @@ | |||
| import { SLACK_SUGGESTED_PROMPTS } from "@/slack/messages"; | |||
|
|
|||
| export function buildAppHomeView(): Record<string, unknown> { | |||
There was a problem hiding this comment.
Use an explicit Slack view type
Returning Record<string, unknown> forces the listener to double-cast the result before publishing and prevents the compiler from validating future Block Kit changes. Type the builder with Slack's Home view contract and use the same concrete block types in the test.
Context Used: Basic guidelines for the project so vibe coders do... (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Publishes a Home tab (the app's dashboard page in Slack) with a quick-start guide: what Databuddy can do, example questions, and the slash commands. Enabled via
app_home_opened+views.publish.Stacked on #595 (drill-downs) — will auto-retarget to staging when that merges.
Testing
Follow-up
v1 is static guidance. Next: enrich with live data (open investigations count, connected sites) and quick-action buttons.
Summary by cubic
Publishes a Slack App Home tab with a quick-start guide to help users get started in Slack. The Home tab renders on open via
app_home_openedand publishes the view withviews.publish.app_home_openedin the manifest and registers a listener to publish the Home view.buildAppHomeViewusing Block Kit (header, sections, dividers, context) with suggested prompts and slash commands.views.publish.Written for commit 11f8c73. Summary will update on new commits.