Skip to content

Add Slack app screenshot to the Neo Slack integration page#19901

Open
aminix wants to merge 1 commit into
masterfrom
add-slack-app-screenshot
Open

Add Slack app screenshot to the Neo Slack integration page#19901
aminix wants to merge 1 commit into
masterfrom
add-slack-app-screenshot

Conversation

@aminix

@aminix aminix commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Proposed changes

Adds a screenshot of a Neo conversation in a Slack thread to the Slack integration docs page, placed right before the "Setting up the integration" section. The image is co-located as a page resource and has a 1px grey border for visual clarity, consistent with other AI docs screenshots.

Unreleased product version (optional)

N/A

Related issues (optional)

N/A

Adds a bordered screenshot of a Neo conversation in a Slack thread,
placed right before the "Setting up the integration" section.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added review:triaging Claude Triage is currently classifying the PR domain:docs PR touches technical docs review:in-progress Claude review is currently running and removed review:triaging Claude Triage is currently classifying the PR labels Jun 26, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Pre-merge Review — Last updated 2026-06-26T16:51:11Z

Tip

Summary: This PR adds a single explanatory screenshot to the existing Neo Slack integration docs page (content/docs/ai/integrations/slack/), placed under the paragraph describing how Neo answers in a Slack thread. The likeliest failure modes — a broken image reference or a misleading caption — both check out: the image uses the same relative page-bundle path convention as its sibling AI docs pages (e.g. pull-requests/, which references pr-example.png the same way), it's a 345 KB PNG under the size limit with descriptive alt text, and the surrounding claim about Slack-thread visibility verified against the page text. One optional polish: the screenshot has no 1px gray border, so it may blend into the white page background — running /add-borders would set it off cleanly. Passes run: claim verification, frontmatter sweep, and an image/convention check.

Review confidence:

Dimension Level Notes
mechanics HIGH
facts HIGH
Investigation log
  • Cross-sibling reads: not run (not in a templated section)
  • External claim verification: 1 of 1 claims verified (0 unverifiable, 0 contradicted) · 4 specialists (numerical, cross-reference, capability, framing); 0 cross-specialist corroborations · routed: 0 inline, 1 Pass 1, 0 Pass 2, 0 Pass 3.
  • Cited-claim spot-checks: not run (no cited claims)
  • Frontmatter sweep: ran on body + meta_desc
  • Temporal-trigger sweep: ran (recency words present in diff; spot-check in-review)
  • Code execution: not run (no static/programs/ change)
  • Code-examples checks: not run (no fenced code blocks in content files)
  • Editorial-balance pass: not run (not under content/blog/)
🚨 Outstanding ⚠️ Low-confidence 💡 Pre-existing ✅ Resolved
0 0 0 0

🔍 Verification trail

1 claims extracted · 1 verified · 0 unverifiable · 0 contradicted
  • L19 in content/docs/ai/integrations/slack/_index.md "The Pulumi Neo Slack integration surfaces conversations in a Slack thread rather than the Pulumi Cloud console, giving the rest of the channel visibility into…" → ✅ verified (evidence: The file at line ~19 states: "the conversation happens in a Slack thread instead of the Pulumi Cloud console, which means the rest of the channel has visibility into what was asked and what Neo found." This is an exact match to the claim.; source: content/docs/ai/integrations/slack/_index.md)

🚨 Outstanding in this PR

No outstanding findings in this PR.

⚠️ Low-confidence

No low-confidence findings.

💡 Pre-existing issues in touched files (optional)

No pre-existing issues in touched files.

✅ Resolved since last review

No items resolved since the last review.

📜 Review history

  • 2026-06-26T16:51:11Z — Clean image addition to the Neo Slack docs page; image convention, format, alt text, and the one surrounding claim all check out (optional: add a 1px border). (1a44ab8)

Need a re-review? Want to dispute a finding? Mention @claude and include #update-review.
(For ad-hoc questions or fixes, just @claude — no hashtag.)

@github-actions github-actions Bot added review:no-blockers Claude review completed cleanly; outstanding is empty and removed review:in-progress Claude review is currently running labels Jun 26, 2026
@pulumi-bot

Copy link
Copy Markdown
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain:docs PR touches technical docs review:no-blockers Claude review completed cleanly; outstanding is empty

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants