fix(test): remove avoidable host prerequisites - #2806
Conversation
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains. Reviews (2): Last reviewed commit: "fix(test): retain fake Cursor parent ide..." | Re-trigger Greptile |
|
Speaking as Kun's firstmate: VISION: aligns. Setup stays simple and field incidents stay coverage: drop the accidental C-compiler fixture and the Ruby-only YAML parser without weakening the contracts. Kernel-visible Class: corrective. Test-host prerequisites only. Security: none. Test fixtures and CONTRIBUTING prose. No product path, no workflow file, no credentials. Overlap: none with the hold pair, spawn-freshen, Herdr, or watch PRs. Files: CI: HEAD Workflows approved: yes (CI 32601008330, NM 32601008350). Land-eligible: NO. Waiting on CI; matching NM attestation still required from the author. Not waiting on the captain. Captain-flag NOW: no. |
|
CI failure diagnosis and scoped fix:
The separate no-mistakes policy check is unchanged and intentionally not acted on under this task's direct-PR instruction. |
Summary
Diagnosis
cursor-agentexecutable identity and-cbehavior.Validation
timeout 600s tests/fm-cursor-primary.test.shtimeout 600s tests/fm-test-run.test.shtimeout 600s tests/fm-lint-workflows.test.shtimeout 600s tests/fm-lint.test.shtimeout 600s bin/fm-lint.sh tests/fm-cursor-primary.test.sh tests/fm-test-run.test.shtimeout 600s bin/fm-lint-workflows.shtimeout 600s bin/fm-doc-audience-check.shbash -n tests/fm-cursor-primary.test.sh tests/fm-test-run.test.shtests/fm-calm-pi-extension.test.shreaches the required rendered-DOM assertion and reports the documented missing native Chrome/Chromium prerequisite on this WSL host. Earlier non-browser assertions pass; the test remains intentionally failing locally rather than silently passing.