Skip to content

fix(cursor): carry Windows bridge shell guidance onto dev2-go (#627) - #675

Merged
Wibias merged 1 commit into
lidge-jun:dev2-gofrom
Wibias:sync/627-to-dev2-go
Jul 29, 2026
Merged

fix(cursor): carry Windows bridge shell guidance onto dev2-go (#627)#675
Wibias merged 1 commit into
lidge-jun:dev2-gofrom
Wibias:sync/627-to-dev2-go

Conversation

@Wibias

@Wibias Wibias commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

Go port

Go has Cursor tool guidance / shell-bridge notes under go/internal/adapter/cursor/ (BuildToolGuidance, CursorShellAliasSystemNote) that do not yet include the #604 host-shell-neutral PowerShell 5.1 + one-retry-stop steering.

Deferred: #674 (needs-go-port)

Test plan

  • CI green on dev2-go
  • Cherry-pick of dcaede6e applied cleanly

@Wibias
fix(cursor): steer Windows bridge shell away from PS 5.1 syntax loops (…
d44e567 
…lidge-jun#627)

* fix(cursor): steer Windows bridge shell away from PS 5.1 syntax loops

Native-shell rejection told the model to replay the same bash/CMD command through the Codex bridge, which on Windows PowerShell 5.1 fails and loops (lidge-jun#604). Adapt for the host shell and inject PS-safe guidance instead.

* fix(cursor): address lidge-jun#604 review — host-shell-neutral retry-stop guidance

Drop proxy-OS platform branching and `;`-as-`&&` advice; use if ($?) and one corrected bridge attempt for Codex/CodeRabbit feedback on lidge-jun#627.

* fix(cursor): state PowerShell 5.1 has no &&/|| in lidge-jun#604 guidance

CodeRabbit follow-up: make unsupported-parser-error wording explicit and assert it in rejection/guidance tests.
@coderabbitai

coderabbitai Bot commented Jul 29, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (2)
  • ^dev$
  • ^preview$

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 6d446f06-3513-47a4-b95b-1ca909436c21

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the bug Something isn't working label Jul 29, 2026
@Wibias Wibias mentioned this pull request Jul 29, 2026
Merged
4 tasks
@Wibias

Wibias commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator Author

CI failures on this PR are pre-existing on dev2-go (same matrix red on recent post-merge heads: prepare-release-assets, durable-runtime / EXDEV, npm-pack mode). This carry only touches the four Cursor TS/test files from dcaede6e; not fixing those base failures here.

@Wibias
Wibias merged commit f708d2b into lidge-jun:dev2-go Jul 29, 2026
7 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lidge-jun lidge-jun Awaiting requested review from lidge-jun lidge-jun is a code owner
@Ingwannu Ingwannu Awaiting requested review from Ingwannu Ingwannu is a code owner

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Wibias