Skip to content

Commit 95320d7

Browse files
authored
Merge pull request #1239 from manaflow-ai/cmux/update-gh-comment-bp8xb
Post one GitHub comment and update it
2 parents a815fe2 + b27dfa0 commit 95320d7

7 files changed

Lines changed: 715 additions & 79 deletions

File tree

apps/worker/src/screenshotCollector/claudeScreenshotCollector.ts

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -212,9 +212,14 @@ If no UI changes exist: Set hasUiChanges=false, take ZERO screenshots, and expla
212212
<PHASE_2_CAPTURE>
213213
If UI changes exist, capture screenshots:
214214
215-
1. Read CLAUDE.md or AGENTS.md (may be one level deeper) and install dependencies if needed
216-
2. Run tmux ls to check if the dev server is running. Dev server should be running on default. Otherwise, start the dev server. Look for instructions in README.md, CLAUDE.md, or framework-specific files (package.json, Makefile, Gemfile, composer.json, requirements.txt, etc.). Use dev_command above if provided.
217-
3. Look for a list of ports you need to check in the repo. Wait for the server to be ready (curl -s -o /dev/null -w "%{http_code}" http://localhost:PORT should return 200)
215+
1. FIRST, check if the dev server is ALREADY RUNNING:
216+
- Run \`tmux list-windows\` and \`tmux capture-pane -p -t <window>\` to see running processes and their logs
217+
- Check if there's a dev server process starting up or already running in any tmux window
218+
- The dev server is typically started automatically in this environment - BE PATIENT and monitor the logs
219+
- If you see the server is starting/compiling, WAIT for it to finish - do NOT kill it or restart it
220+
- Use \`ss -tlnp | grep LISTEN\` to see what ports have servers listening
221+
2. ONLY if no server is running anywhere: Read CLAUDE.md, README.md, or package.json for setup instructions. Install dependencies if needed, then start the dev server.
222+
3. BE PATIENT - servers can take time to compile. Monitor tmux logs to see progress. A response from curl (even 404) means the server is up. Do NOT restart the server if it's still compiling.
218223
4. Navigate to the pages/components modified in the PR
219224
5. Capture screenshots of the changes, including:
220225
- The default/resting state of changed components
@@ -224,7 +229,7 @@ If UI changes exist, capture screenshots:
224229
- Responsive layouts if the PR includes responsive changes
225230
6. Save screenshots to ${outputDir} with descriptive names like "component-state-${branch}.png"
226231
7. After taking a screenshot, always open the image to verify that the capture is expected
227-
8. If screenshot seems outdated, refresh the page and taking the screenshot again.
232+
8. If screenshot seems outdated, refresh the page and take the screenshot again.
228233
9. Delete any screenshot files from the filesystem that you do not want included
229234
</PHASE_2_CAPTURE>
230235

0 commit comments

Comments
 (0)