docs: add Cursor Cloud Linux dev environment notes to AGENTS.md#310
Conversation
- Document test:unit --project flag incompatibility with vitest 0.34.6 - Add correct DISPLAY=:1 for Cursor Cloud VM - Document fast launch commands (--skip-runtime-build, --attach) Co-authored-by: Arul Sharma <arul28@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Adds practical notes to the "Cursor Cloud specific instructions" section of AGENTS.md based on environment setup and testing:
test:unitscript uses--project unitwhich vitest 0.34.6 doesn't support; documents the workaround:1, not:99--skip-runtime-build,--attach) for desktop and TUIEvidence of working environment
Full demo: Codex GPT-5.5 chat with auto-create lane, then lane deletion
ade_full_demo_chat_lanes_delete.mp4
Codex GPT-5.5 response to test message:
Codex GPT-5.5 response
Lanes tab showing auto-created lane:
Lanes tab with auto-created lane
Lane successfully deleted:
Lane deleted successfully
To show artifacts inline, enable in settings.
Greptile Summary
This PR adds four bullet points to the "Running the Electron desktop app on Linux" section of
AGENTS.md, documenting practical Cursor Cloud VM gotchas: a brokentest:unitscript workaround, the correctDISPLAY=:1value, and fast-launch flags for the desktop and TUI dev scripts.npm --prefix apps/desktop run test:unitis broken under vitest 0.34.6 and provides the correct workarounds (npx vitest run <file>ornpm run test).DISPLAY=:1for Electron on the Cursor Cloud VM and documents--skip-runtime-build/--attachflags fordev:desktopanddev:code(both flags are confirmed valid in their respective scripts).Confidence Score: 4/5
Documentation-only change; the new notes are accurate but one existing line directly contradicts the new test:unit warning.
The four new bullets are factually correct and the --skip-runtime-build/--attach flags exist in both dev scripts. However, line 84 still recommends
npm --prefix apps/desktop run test:unitas the canonical targeted-test example, while line 114 (added in this PR) documents that the exact same command is broken. A developer following the overview section will hit the broken command without realising a workaround exists two sections later.AGENTS.md — the existing line 84 example should be updated to match the workaround described in the new line 114.
Important Files Changed
test:unitrecommendation in the Environment overview sectionFlowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD A[Developer on Cursor Cloud Linux VM] --> B{What to run?} B --> C[Unit tests] B --> D[Desktop dev app] B --> E[TUI / ade-code] C --> C1["❌ npm run test:unit\n(--project unit unsupported\nin vitest 0.34.6)"] C --> C2["✅ npx vitest run <file>\n(inside apps/desktop)"] C --> C3["✅ npm run test\n(full suite)"] D --> D1{CLI already built?} D1 -- No --> D2["npm run dev:desktop\n(builds CLI first)"] D1 -- Yes --> D3["npm run dev:desktop --\n--skip-runtime-build\n(fast path)"] D2 --> D4["Set DISPLAY=:1\nbefore launching Electron"] D3 --> D4 E --> E1{Runtime already running?} E1 -- No --> E2["npm run dev:code --\n--skip-runtime-build --auto\n(starts runtime)"] E1 -- Yes --> E3["npm run dev:code --\n--skip-runtime-build --attach\n--project-root p --workspace-root w"]Comments Outside Diff (1)
AGENTS.md, line 84 (link)Prompt To Fix With AI
Prompt To Fix All With AI
Reviews (1): Last reviewed commit: "docs: add Cursor Cloud Linux dev environ..." | Re-trigger Greptile