fix: rename discover_cursor_rules to enable_cursor_rules

Author: cristianmessel-cs

README.md

@@ -115,7 +115,7 @@ review:
 | `context.reserved_output_tokens` | Tokens reserved for the model's response (default: `4096`) |
 | `context.tokenizer_model` | Tiktoken encoding used for token counting (default: `o200k_base`) |
 | `context.model_limits` | Per-model context window and max output token settings |
-| `context.discover_cursor_rules` | When `true`, also read `AGENTS.md` and `.cursor/rules/*.mdc` (rules with `alwaysApply: true`, or no `globs`/`alwaysApply` frontmatter at all) from the reviewed repo as additional project context. Default: `false` |
+| `context.enable_cursor_rules` | When `true`, also read `AGENTS.md` and `.cursor/rules/*.mdc` (rules with `alwaysApply: true`, or no `globs`/`alwaysApply` frontmatter at all) from the reviewed repo as additional project context. Default: `false` |
 
 ### 4. Build
 
@@ -194,7 +194,7 @@ go install ./cmd/codestrike
 ln -s "$(pwd)" ~/.cursor/plugins/local/codestrike
 ```
 
-Reload Cursor (`Developer: Reload Window`) and confirm `pr-review` appears under **Customize → Skills**. codestrike's own review pipeline can also read Cursor-native project instructions (`AGENTS.md`, `.cursor/rules/*.mdc`) from the reviewed repo — see `context.discover_cursor_rules` above. See [`docs/cursor-integration.md`](docs/cursor-integration.md) for details and planned follow-up work (an MCP server).
+Reload Cursor (`Developer: Reload Window`) and confirm `pr-review` appears under **Customize → Skills**. codestrike's own review pipeline can also read Cursor-native project instructions (`AGENTS.md`, `.cursor/rules/*.mdc`) from the reviewed repo — see `context.enable_cursor_rules` above. See [`docs/cursor-integration.md`](docs/cursor-integration.md) for details and planned follow-up work (an MCP server).
 
 ## Development
 

internal/config/assets/default.yaml

@@ -24,7 +24,7 @@ review:
     # Set to true to also read AGENTS.md and .cursor/rules/*.mdc (rules with
     # alwaysApply: true, or no globs/alwaysApply frontmatter at all) from the
     # reviewed repo as additional project context. Off by default.
-    # discover_cursor_rules: true
+    # enable_cursor_rules: true
     model_limits:
       gpt-4o:
         context_window: 128000

internal/config/config.go

@@ -32,7 +32,7 @@ type ContextConfig struct {
 	ReservedOutputTokens int                   `yaml:"reserved_output_tokens"`
 	TokenizerModel       string                `yaml:"tokenizer_model"`
 	ModelLimits          map[string]ModelLimit `yaml:"model_limits"`
-	DiscoverCursorRules  bool                  `yaml:"discover_cursor_rules"`
+	EnableCursorRules    bool                  `yaml:"enable_cursor_rules"`
 }
 
 type ModelLimit struct {

internal/context/cursorrules.go

@@ -59,11 +59,11 @@ func isUnconditionallyApplicable(fm cursorRuleFrontmatter) bool {
 	return strings.TrimSpace(fm.Globs) == ""
 }
 
-// DiscoverCursorContext reads AGENTS.md and applicable .cursor/rules/*.mdc
+// LoadCursorContext reads AGENTS.md and applicable .cursor/rules/*.mdc
 // files from repoRoot (root level only, not recursive) and returns their
 // concatenated content formatted as "### <name>" sections, consistent with
 // how project context_files are rendered.
-func DiscoverCursorContext(repoRoot string) string {
+func LoadCursorContext(repoRoot string) string {
 	var sb strings.Builder
 
 	if data, err := os.ReadFile(filepath.Join(repoRoot, "AGENTS.md")); err == nil {

internal/context/cursorrules_test.go

@@ -9,7 +9,7 @@ import (
 	appcontext "github.com/CrowdStrike/codestrike/internal/context"
 )
 
-func TestDiscoverCursorContext_ReadsAgentsMdAndAlwaysApplyRules(t *testing.T) {
+func TestLoadCursorContext_ReadsAgentsMdAndAlwaysApplyRules(t *testing.T) {
 	dir := t.TempDir()
 	writeFile(t, dir, "AGENTS.md", "# Project Instructions\nUse snake_case.\n")
 
@@ -20,7 +20,7 @@ func TestDiscoverCursorContext_ReadsAgentsMdAndAlwaysApplyRules(t *testing.T) {
 	writeFile(t, rulesDir, "always.mdc", "---\nalwaysApply: true\n---\n\nAlways follow this rule.\n")
 	writeFile(t, rulesDir, "no-frontmatter-fields.mdc", "---\ndescription: agent decides\n---\n\nAgent-decided rule body.\n")
 
-	got := appcontext.DiscoverCursorContext(dir)
+	got := appcontext.LoadCursorContext(dir)
 
 	if !strings.Contains(got, "### AGENTS.md") || !strings.Contains(got, "Use snake_case.") {
 		t.Errorf("expected AGENTS.md content in output, got: %q", got)
@@ -33,7 +33,7 @@ func TestDiscoverCursorContext_ReadsAgentsMdAndAlwaysApplyRules(t *testing.T) {
 	}
 }
 
-func TestDiscoverCursorContext_SkipsGlobScopedAndDisabledRules(t *testing.T) {
+func TestLoadCursorContext_SkipsGlobScopedAndDisabledRules(t *testing.T) {
 	dir := t.TempDir()
 	rulesDir := filepath.Join(dir, ".cursor", "rules")
 	if err := os.MkdirAll(rulesDir, 0755); err != nil {
@@ -43,7 +43,7 @@ func TestDiscoverCursorContext_SkipsGlobScopedAndDisabledRules(t *testing.T) {
 	writeFile(t, rulesDir, "disabled.mdc", "---\nalwaysApply: false\n---\n\nManual only.\n")
 	writeFile(t, rulesDir, "not-a-rule.md", "not an mdc file\n")
 
-	got := appcontext.DiscoverCursorContext(dir)
+	got := appcontext.LoadCursorContext(dir)
 
 	if strings.Contains(got, "scoped.mdc") || strings.Contains(got, "Only for tsx files.") {
 		t.Errorf("expected glob-scoped rule to be excluded, got: %q", got)
@@ -56,25 +56,25 @@ func TestDiscoverCursorContext_SkipsGlobScopedAndDisabledRules(t *testing.T) {
 	}
 }
 
-func TestDiscoverCursorContext_MissingDirReturnsEmptyNoError(t *testing.T) {
+func TestLoadCursorContext_MissingDirReturnsEmptyNoError(t *testing.T) {
 	dir := t.TempDir()
 
-	got := appcontext.DiscoverCursorContext(dir)
+	got := appcontext.LoadCursorContext(dir)
 
 	if got != "" {
 		t.Errorf("expected empty result for repo with no AGENTS.md/.cursor/rules, got: %q", got)
 	}
 }
 
-func TestDiscoverCursorContext_SkipsFileWithoutFrontmatter(t *testing.T) {
+func TestLoadCursorContext_SkipsFileWithoutFrontmatter(t *testing.T) {
 	dir := t.TempDir()
 	rulesDir := filepath.Join(dir, ".cursor", "rules")
 	if err := os.MkdirAll(rulesDir, 0755); err != nil {
 		t.Fatal(err)
 	}
 	writeFile(t, rulesDir, "plain.mdc", "Just plain text, no frontmatter delimiters.\n")
 
-	got := appcontext.DiscoverCursorContext(dir)
+	got := appcontext.LoadCursorContext(dir)
 
 	if got != "" {
 		t.Errorf("expected file without frontmatter to be skipped, got: %q", got)

internal/review/pipeline.go

@@ -366,13 +366,13 @@ func (p *Pipeline) loadProjectContext() string {
 	var sb strings.Builder
 	sb.WriteString(p.loadContextFiles())
 
-	if p.config.Review.Context.DiscoverCursorRules {
+	if p.config.Review.Context.EnableCursorRules {
 		cwd, err := os.Getwd()
 		if err != nil {
 			p.logger.Debug().Err(err).Msg("could not determine working directory for cursor rule discovery")
 			return sb.String()
 		}
-		sb.WriteString(appcontext.DiscoverCursorContext(cwd))
+		sb.WriteString(appcontext.LoadCursorContext(cwd))
 	}
 
 	return sb.String()