Skip to content

Add screenshot-dev-app agent skill#2527

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

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

Conversation

@Twixes

@Twixes Twixes commented Jun 8, 2026

Copy link
Copy Markdown
Member

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

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit ce91cab.

@greptile-apps

greptile-apps Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor
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 Outdated
cursoragent and others added 3 commits June 8, 2026 13:09
Co-authored-by: Michael Matloka <dev@twixes.com>
…ill.

Ship screenshot:preview commands and a persistent-server capture script so
agents can verify the Vite preview without cursor-ide-browser.
Makes Electron renderer attachable on :9222 during normal dev, alongside
the Playwright preview screenshot tooling.
@Twixes Twixes mentioned this pull request Jun 9, 2026
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