Skip to content

cursor rule testing#5596

Closed
jdx wants to merge 10 commits into
mainfrom
cursor-rule-testing
Closed

cursor rule testing#5596
jdx wants to merge 10 commits into
mainfrom
cursor-rule-testing

Conversation

@jdx

@jdx jdx commented Jul 12, 2025

Copy link
Copy Markdown
Owner

jdx and others added 10 commits July 12, 2025 05:59
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [Swatinem/rust-cache](https://github.com/Swatinem/rust-cache)
| action | pinDigest | -> `98c8021` |
| [actions/cache](https://github.com/actions/cache) | action |
pinDigest | -> `5a3ec84` |
| [actions/checkout](https://github.com/actions/checkout) |
action | pinDigest | -> `11bd719` |
|
[actions/download-artifact](https://github.com/actions/download-artifact)
| action | pinDigest | -> `d3f86a1` |
| [actions/setup-node](https://github.com/actions/setup-node) |
action | pinDigest | -> `49933ea` |
|
[actions/upload-artifact](https://github.com/actions/upload-artifact)
| action | pinDigest | -> `ea165f8` |
|
[amannn/action-semantic-pull-request](https://github.com/amannn/action-semantic-pull-request)
| action | pinDigest | -> `0723387` |
|
[apple-actions/import-codesign-certs](https://github.com/apple-actions/import-codesign-certs)
| action | pinDigest | -> `65cfbc3` |
|
[crazy-max/ghaction-import-gpg](https://github.com/crazy-max/ghaction-import-gpg)
| action | pinDigest | -> `e89d409` |
| [nick-fields/retry](https://github.com/nick-fields/retry) |
action | pinDigest | -> `ce71cc2` |
|
[softprops/action-gh-release](https://github.com/softprops/action-gh-release)
| action | pinDigest | -> `72f2c25` |
|
[taiki-e/install-action](https://github.com/taiki-e/install-action)
| action | pinDigest | -> `aa4fe33` |
|
[thollander/actions-comment-pull-request](https://github.com/thollander/actions-comment-pull-request)
| action | pinDigest | -> `24bffb9` |
|
[withfig/push-to-fig-autocomplete-action](https://github.com/withfig/push-to-fig-autocomplete-action)
| action | pinDigest | -> `fb320c2` |

---

### Configuration

📅 **Schedule**: Branch creation - "before 4am on Friday" in timezone
America/Chicago, Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get
[config
help](https://github.com/renovatebot/renovate/discussions) if
that's undesired.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/jdx/mise).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0MS4xNy4yIiwidXBkYXRlZEluVmVyIjoiNDEuMjMuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
This marks the release as a draft until the assets are successfully
added.

However, since `release-r2.sh` runs before it, `https://mise.run` will
still fail.
I think we need to move it to `post-release.sh` and run it after the
assets creation.
- Add backend hook modules for custom plugin backends
- Implement backend_list_versions, backend_install, backend_exec_env, backend_uninstall hooks
- Add example and test backend plugins
- Export backend hooks in public API
- Enable vfox plugins to act as custom backends for mise
- Add derive feature to serde dependency to enable Serialize/Deserialize macros
- Fix compilation errors in backend hook modules

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Markdown File Contains Duplicate Content

The conventional_commits.mdc file has its entire content duplicated from line 38 to 69. The duplicated section starts with malformed markdown on line 38, where the "Common Scopes" list is concatenated directly with "## Conventional Commits (REQUIRED)" due to a missing newline.

.cursor/rules/conventional_commits.mdc#L36-L69

https://github.com/jdx/mise/blob/b560ed99bebb149a01eb48e9bfe100577207226e/.cursor/rules/conventional_commits.mdc#L36-L69

Fix in CursorFix in Web


Bug: Command Execution Incompatibility and Injection Vulnerability

The exec function in cmd.rs directly executes user-provided commands via sh -c. This design introduces two critical issues:

  1. Cross-platform incompatibility: It fails on Windows systems, which require cmd /c or powershell -c for command execution.
  2. Command injection vulnerability: Lack of input validation allows malicious plugin code to execute arbitrary system commands.

crates/vfox/src/lua_mod/cmd.rs#L12-L34

fn exec(_lua: &Lua, (command,): (String,)) -> LuaResult<String> {
use std::process::Command;
let output = Command::new("sh")
.arg("-c")
.arg(&command)
.output()
.map_err(|e| mlua::Error::RuntimeError(format!("Failed to execute command: {}", e)))?;
let stdout = String::from_utf8_lossy(&output.stdout);
let stderr = String::from_utf8_lossy(&output.stderr);
if output.status.success() {
Ok(stdout.to_string())
} else {
Err(mlua::Error::RuntimeError(format!(
"Command failed with status {}: {}",
output.status,
stderr
)))
}
}

Fix in CursorFix in Web


BugBot free trial expires on July 22, 2025
You have used $0.00 of your $50.00 spend limit so far. Manage your spend limit in the Cursor dashboard.

Was this report helpful? Give feedback by reacting with 👍 or 👎

@jdx jdx closed this Jul 12, 2025
@jdx jdx deleted the cursor-rule-testing branch July 20, 2025 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants