fix: align Cursor rule extension handling

Author: avifenesh

.github/workflows/mcp-release-watch.yml

@@ -64,7 +64,7 @@ jobs:
           if [[ -z "$latest_release" ]]; then
             # No final release, select the newest final date tag instead of
             # blindly accepting an RC/final-suffixed tag at index zero.
-            if ! latest_release=$(gh api "repos/$MCP_REPO/tags" --jq '[.[].name | select(test("^[0-9]{4}-[0-9]{2}-[0-9]{2}$"))][0] // empty' 2>/dev/null); then
+            if ! latest_release=$(gh api "repos/$MCP_REPO/tags" --jq '[.[].name | select(test("^[0-9]{4}-[0-9]{2}-[0-9]{2}$"))] | max // empty' 2>/dev/null); then
               echo "ERROR: Failed to query MCP releases and tags"
               exit 1
             fi

crates/agnix-cli/src/watch.rs

@@ -61,6 +61,10 @@ where
 fn is_relevant_file(path: &Path) -> bool {
     let filename = path.file_name().and_then(|n| n.to_str()).unwrap_or("");
     let extension = path.extension().and_then(|e| e.to_str()).unwrap_or("");
+    let is_cursor_rule = matches!(
+        agnix_core::detect_file_type(path),
+        agnix_core::FileType::CursorRule
+    );
     let parent = path
         .parent()
         .and_then(|p| p.file_name())
@@ -91,7 +95,8 @@ fn is_relevant_file(path: &Path) -> bool {
     ) || extension == "mcp"
         || is_codex_config
         || filename.ends_with(".mcp.json")
-        || filename.ends_with(".mdc")
+        || extension.eq_ignore_ascii_case("mdc")
+        || is_cursor_rule
         || filename.ends_with(".instructions.md")
         // Also watch for agent files
         || (extension == "md"
@@ -126,4 +131,18 @@ mod tests {
         assert!(!is_relevant_file(Path::new("configs/config.yaml")));
         assert!(!is_relevant_file(Path::new("configs/config.json")));
     }
+
+    #[test]
+    fn cursor_rule_files_are_relevant() {
+        for path in [
+            ".cursor/rules/test.md",
+            ".cursor/rules/test.MD",
+            ".cursor/rules/test.mdc",
+            ".cursor/rules/test.MDC",
+        ] {
+            assert!(is_relevant_file(Path::new(path)));
+        }
+
+        assert!(!is_relevant_file(Path::new("docs/test.md")));
+    }
 }

crates/agnix-core/src/rules/claude_md.rs

@@ -44,7 +44,12 @@ fn is_cursor_rules_file(path: &Path) -> bool {
         return true;
     }
 
-    (filename.ends_with(".md") || filename.ends_with(".mdc")) && is_path_under_cursor_rules(path)
+    path.extension()
+        .and_then(|extension| extension.to_str())
+        .is_some_and(|extension| {
+            extension.eq_ignore_ascii_case("md") || extension.eq_ignore_ascii_case("mdc")
+        })
+        && is_path_under_cursor_rules(path)
 }
 
 impl Validator for ClaudeMdValidator {
@@ -404,28 +409,29 @@ mod tests {
     fn test_cursor_rules_mdc_gets_rules() {
         let content = "Be helpful and accurate when responding.";
         let validator = ClaudeMdValidator;
-        let diagnostics = validator.validate(
-            Path::new(".cursor/rules/typescript.mdc"),
-            content,
-            &LintConfig::default(),
-        );
 
-        assert!(!diagnostics.is_empty());
-        assert!(diagnostics.iter().any(|d| d.rule == "CC-MEM-005"));
+        for path in [
+            ".cursor/rules/typescript.mdc",
+            ".cursor/rules/typescript.MDC",
+        ] {
+            let diagnostics = validator.validate(Path::new(path), content, &LintConfig::default());
+
+            assert!(!diagnostics.is_empty());
+            assert!(diagnostics.iter().any(|d| d.rule == "CC-MEM-005"));
+        }
     }
 
     #[test]
     fn test_cursor_rules_md_gets_rules() {
         let content = "Be helpful and accurate when responding.";
         let validator = ClaudeMdValidator;
-        let diagnostics = validator.validate(
-            Path::new(".cursor/rules/typescript.md"),
-            content,
-            &LintConfig::default(),
-        );
 
-        assert!(!diagnostics.is_empty());
-        assert!(diagnostics.iter().any(|d| d.rule == "CC-MEM-005"));
+        for path in [".cursor/rules/typescript.md", ".cursor/rules/typescript.MD"] {
+            let diagnostics = validator.validate(Path::new(path), content, &LintConfig::default());
+
+            assert!(!diagnostics.is_empty());
+            assert!(diagnostics.iter().any(|d| d.rule == "CC-MEM-005"));
+        }
     }
 
     #[test]

crates/agnix-core/src/rules/cursor.rs

@@ -996,7 +996,7 @@ fn validate_cursor_environment_file(
         }
     }
 
-    for field in ["$schema", "name", "user", "snapshot"] {
+    for field in ["name", "user", "snapshot"] {
         if root.get(field).is_some_and(|value| !value.is_string()) {
             diagnostics.push(
                 cursor_environment_error(
@@ -2492,7 +2492,6 @@ is_background: false
 
         for (name, content) in [
             ("unknown root field", r#"{"unknown":true}"#),
-            ("invalid schema association", r#"{"$schema":42}"#),
             ("invalid name", r#"{"name":1}"#),
             ("invalid user", r#"{"user":false}"#),
             (

crates/agnix-lsp/src/backend/helpers.rs

@@ -78,6 +78,10 @@ impl Backend {
             || file_name.eq_ignore_ascii_case("copilot-instructions.md")
             || file_name.to_lowercase().ends_with(".instructions.md")
             || file_name.to_lowercase().ends_with(".mdc")
+            || matches!(
+                agnix_core::detect_file_type(path),
+                agnix_core::FileType::CursorRule
+            )
             || file_name.eq_ignore_ascii_case("opencode.json")
             || file_name.eq_ignore_ascii_case("opencode.jsonc")
     }

crates/agnix-lsp/src/backend/tests.rs

@@ -2063,6 +2063,12 @@ fn test_is_project_level_trigger() {
     assert!(Backend::is_project_level_trigger(Path::new(
         "/project/.cursor/rules/test.mdc"
     )));
+    assert!(Backend::is_project_level_trigger(Path::new(
+        "/project/.cursor/rules/test.MDC"
+    )));
+    assert!(Backend::is_project_level_trigger(Path::new(
+        "/project/.cursor/rules/test.MD"
+    )));
     assert!(Backend::is_project_level_trigger(Path::new(
         "/project/GEMINI.md"
     )));

tests/ci_workflow.rs

@@ -56,6 +56,10 @@ fn mcp_release_watch_filters_prerelease_tags() {
         workflow.contains(r#"select(test("^[0-9]{4}-[0-9]{2}-[0-9]{2}$"))"#),
         "MCP tag fallback must select only final date-based versions"
     );
+    assert!(
+        workflow.contains("] | max // empty"),
+        "MCP tag fallback must select the newest final date-based version"
+    );
     assert!(
         workflow.contains("Ignoring non-final MCP release/tag"),
         "MCP prerelease tags must be ignored instead of failing the workflow"