Pull request review #2

Merged

dikkadev merged 2 commits from cursor/pull-request-review-9add into claude/file-uri-context-menu-GcQln

2026-02-27 17:37:05 +00:00

dikkadev commented

2026-02-27 17:34:41 +00:00

(Migrated from github.com)

Fix URI escaping for file paths and update CI workflow to match all branch names.

The original URI construction did not percent-encode reserved characters, leading to invalid file URIs. The CI workflow's branches: ["*"] pattern also failed to match branches with slashes, preventing CI runs. A follow-up fix ensures UNC paths retain their authority.

Fix URI escaping for file paths and update CI workflow to match all branch names. The original URI construction did not percent-encode reserved characters, leading to invalid file URIs. The CI workflow's `branches: ["*"]` pattern also failed to match branches with slashes, preventing CI runs. A follow-up fix ensures UNC paths retain their authority. --- <p><a href="https://cursor.com/agents/bc-e5efabe4-2ba8-41c0-ba68-aee2e7914b85"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-web-light.png"><img alt="Open in Web" width="114" height="28" src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a> <a href="https://cursor.com/background-agent?bcId=bc-e5efabe4-2ba8-41c0-ba68-aee2e7914b85"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-cursor-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-cursor-light.png"><img alt="Open in Cursor" width="131" height="28" src="https://cursor.com/assets/images/open-in-cursor-dark.png"></picture></a> </p>

cursor[bot] commented

2026-02-27 17:34:42 +00:00

(Migrated from github.com)

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
_{Learn more about Cursor Agents}

Cursor Agent can help with this pull request. Just `@cursor` in comments and I'll start working on changes in this branch. <sub>[Learn more](https://cursor.com/docs/background-agent/web-and-mobile) about Cursor Agents</sub>