Skip to content

Add screenshot-dev-app agent skill#2527

Open
Twixes wants to merge 2 commits into
mainfrom
add-screenshot-dev-app-skill
Open

Add screenshot-dev-app agent skill#2527
Twixes wants to merge 2 commits into
mainfrom
add-screenshot-dev-app-skill

Conversation

@Twixes
Copy link
Copy Markdown
Member

@Twixes Twixes commented Jun 8, 2026

Summary

  • Adds .claude/skills/screenshot-dev-app/SKILL.md for visually verifying PostHog Code UI changes via the Vite web preview (localhost:5173/?previewMode=true).
  • Documents navigate → snapshot → screenshot workflow using cursor-ide-browser (no Electron / screencapture).
  • Includes route map, mock report ids, and where to extend preview fixtures in apps/code/index.html.

Test plan

  • Skill file renders correctly in the repo
  • Follow skill: open http://localhost:5173/?previewMode=true#/code/inbox/pulls with dev server running
  • Confirm browser_take_screenshot captures the inbox list with mock data

Documents how agents navigate the Vite preview (?previewMode=true), capture screenshots via cursor-ide-browser, and extend mocks in index.html when fixtures are insufficient.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 8, 2026

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 7636be5.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Jun 8, 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
.claude/skills/screenshot-dev-app/SKILL.md:37
**Mismatched table column count**

The "Task detail" row has a third column (`Needs a real task id`) but the table only declares two headers (`View` and `Hash route`). Most Markdown renderers (including GitHub) will render this third cell without a header, resulting in a stray column. The note would be cleaner as a parenthetical inside the hash-route cell, e.g. `` `#/code/tasks/<taskId>` (needs a real task id) ``.

Reviews (1): Last reviewed commit: "Add screenshot-dev-app skill for web pre..." | Re-trigger Greptile

Comment thread .claude/skills/screenshot-dev-app/SKILL.md
Co-authored-by: Michael Matloka <dev@twixes.com>
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