Skip to content

Introduce as PostHog Slack app in Slack-originated runs#2467

Merged
joshsny merged 3 commits into
mainfrom
vojtab/slack-app-identity-prompt
Jun 2, 2026
Merged

Introduce as PostHog Slack app in Slack-originated runs#2467
joshsny merged 3 commits into
mainfrom
vojtab/slack-app-identity-prompt

Conversation

@VojtechBartos
Copy link
Copy Markdown
Member

@VojtechBartos VojtechBartos commented Jun 2, 2026

Adds an identity block to the cloud system prompt so Slack-originated runs introduce themselves as "PostHog Slack app" instead of Claude. Scoped to origin === "slack" only.

Showcase

Screenshot 2026-06-02 at 22 56 12

@VojtechBartos VojtechBartos self-assigned this Jun 2, 2026
@VojtechBartos VojtechBartos marked this pull request as ready for review June 2, 2026 20:57
@VojtechBartos VojtechBartos requested review from joshsny, skoob13 and tatoalo and removed request for joshsny and skoob13 June 2, 2026 20:58
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Jun 2, 2026

Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
packages/agent/src/server/agent-server.ts:1686-1696
**Missing test coverage for identity block**

The existing Slack-origin tests in `agent-server.test.ts` (e.g., "returns auto-PR prompt for Slack-origin runs") don't assert that the new `# Identity` block and "PostHog Slack app" text are present when `origin === "slack"`, nor that the block is absent for non-Slack origins. A simple pair of parameterised checks would confirm the feature and catch accidental regressions — particularly important since the instruction explicitly suppresses model-disclosure language.

Reviews (1): Last reviewed commit: "Introduce as PostHog Slack app in Slack-..." | Re-trigger Greptile

Comment thread packages/agent/src/server/agent-server.ts
Copy link
Copy Markdown
Contributor

@joshsny joshsny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - left a small suggested change to the wording

Comment thread packages/agent/src/server/agent-server.ts Outdated
@joshsny joshsny enabled auto-merge (squash) June 2, 2026 21:53
@joshsny joshsny merged commit da82699 into main Jun 2, 2026
18 checks passed
@joshsny joshsny deleted the vojtab/slack-app-identity-prompt branch June 2, 2026 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants