Remove -d flag and set comprehensive defaults for -m and -t (#29)

* Initial plan

* Remove dirs flag and set suitable defaults for memories and tasks

Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>

Author: Copilot

README.md

@@ -82,11 +82,12 @@ coding-context [options] <task-name>
 
 Options:
   -b                Automatically run the bootstrap script after generating it
-  -d <directory>    Directory that may contain a memories/ or tasks/ subdirectory (can be used multiple times)
-                    Default: .prompts, ~/.config/prompts, /var/local/prompts
   -m <path>         Directory containing memories, or a single memory file (can be used multiple times)
-                    Default: AGENTS.md
+                    Defaults: AGENTS.md, .github/copilot-instructions.md, CLAUDE.md, .cursorrules, 
+                              .cursor/rules/, .instructions.md, .continuerules, .prompts/memories, 
+                              ~/.config/prompts/memories, /var/local/prompts/memories
   -t <path>         Directory containing tasks, or a single task file (can be used multiple times)
+                    Defaults: .prompts/tasks, ~/.config/prompts/tasks, /var/local/prompts/tasks
   -o <directory>    Output directory for generated files (default: .)
   -p <key=value>    Template parameter for prompt substitution (can be used multiple times)
   -s <key=value>    Include memories with matching frontmatter (can be used multiple times)
@@ -100,15 +101,6 @@ Options:
 coding-context -p feature="Authentication" -p language=Go add-feature
 ```
 
-**Example with directories:**
-```bash
-# Add a custom directory that contains memories/ and tasks/ subdirectories
-coding-context -d /path/to/custom/prompts my-task
-
-# The -d flag prepends directories to search paths, so custom directories take priority
-# Search order becomes: /path/to/custom/prompts/memories, .prompts/memories, ~/.config/prompts/memories, ...
-```
-
 **Example with custom memory and task paths:**
 ```bash
 # Specify explicit memory files or directories

integration_test.go

@@ -66,7 +66,7 @@ Please help with this task.
 	}
 
 	// Run the binary
-	cmd = exec.Command(binaryPath, "-d", contextDir, "-o", outputDir, "test-task")
+	cmd = exec.Command(binaryPath, "-m", memoriesDir, "-t", tasksDir, "-o", outputDir, "test-task")
 	cmd.Dir = tmpDir
 	if output, err := cmd.CombinedOutput(); err != nil {
 		t.Fatalf("failed to run binary: %v\n%s", err, output)
@@ -158,7 +158,7 @@ echo "Setting up Jira"
 	}
 
 	// Run the binary
-	cmd = exec.Command(binaryPath, "-d", contextDir, "-o", outputDir, "test-task")
+	cmd = exec.Command(binaryPath, "-m", memoriesDir, "-t", tasksDir, "-o", outputDir, "test-task")
 	cmd.Dir = tmpDir
 	if output, err := cmd.CombinedOutput(); err != nil {
 		t.Fatalf("failed to run binary: %v\n%s", err, output)
@@ -243,7 +243,7 @@ Please help with this task.
 	}
 
 	// Run the binary
-	cmd = exec.Command(binaryPath, "-d", contextDir, "-o", outputDir, "test-task")
+	cmd = exec.Command(binaryPath, "-m", memoriesDir, "-t", tasksDir, "-o", outputDir, "test-task")
 	cmd.Dir = tmpDir
 	if output, err := cmd.CombinedOutput(); err != nil {
 		t.Fatalf("failed to run binary: %v\n%s", err, output)
@@ -310,7 +310,7 @@ func TestMultipleBootstrapFiles(t *testing.T) {
 	}
 
 	// Run the binary
-	cmd = exec.Command(binaryPath, "-d", contextDir, "-o", outputDir, "test-task")
+	cmd = exec.Command(binaryPath, "-m", memoriesDir, "-t", tasksDir, "-o", outputDir, "test-task")
 	cmd.Dir = tmpDir
 	if output, err := cmd.CombinedOutput(); err != nil {
 		t.Fatalf("failed to run binary: %v\n%s", err, output)
@@ -370,7 +370,7 @@ func TestSelectorFiltering(t *testing.T) {
 	}
 
 	// Test 1: Include by env=production
-	cmd = exec.Command(binaryPath, "-d", contextDir, "-o", outputDir, "-s", "env=production", "test-task")
+	cmd = exec.Command(binaryPath, "-m", memoriesDir, "-t", tasksDir, "-o", outputDir, "-s", "env=production", "test-task")
 	cmd.Dir = tmpDir
 	if output, err := cmd.CombinedOutput(); err != nil {
 		t.Fatalf("failed to run binary: %v\n%s", err, output)
@@ -400,7 +400,7 @@ func TestSelectorFiltering(t *testing.T) {
 	os.RemoveAll(outputDir)
 
 	// Test 2: Include by language=go (should include prod and test, and nofm)
-	cmd = exec.Command(binaryPath, "-d", contextDir, "-o", outputDir, "-s", "language=go", "test-task")
+	cmd = exec.Command(binaryPath, "-m", memoriesDir, "-t", tasksDir, "-o", outputDir, "-s", "language=go", "test-task")
 	cmd.Dir = tmpDir
 	if output, err := cmd.CombinedOutput(); err != nil {
 		t.Fatalf("failed to run binary: %v\n%s", err, output)
@@ -428,7 +428,7 @@ func TestSelectorFiltering(t *testing.T) {
 	os.RemoveAll(outputDir)
 
 	// Test 3: Exclude by env=production (should include dev and test, and nofm)
-	cmd = exec.Command(binaryPath, "-d", contextDir, "-o", outputDir, "-S", "env=production", "test-task")
+	cmd = exec.Command(binaryPath, "-m", memoriesDir, "-t", tasksDir, "-o", outputDir, "-S", "env=production", "test-task")
 	cmd.Dir = tmpDir
 	if output, err := cmd.CombinedOutput(); err != nil {
 		t.Fatalf("failed to run binary: %v\n%s", err, output)
@@ -456,7 +456,7 @@ func TestSelectorFiltering(t *testing.T) {
 	os.RemoveAll(outputDir)
 
 	// Test 4: Multiple includes env=production language=go (should include only prod and nofm)
-	cmd = exec.Command(binaryPath, "-d", contextDir, "-o", outputDir, "-s", "env=production", "-s", "language=go", "test-task")
+	cmd = exec.Command(binaryPath, "-m", memoriesDir, "-t", tasksDir, "-o", outputDir, "-s", "env=production", "-s", "language=go", "test-task")
 	cmd.Dir = tmpDir
 	if output, err := cmd.CombinedOutput(); err != nil {
 		t.Fatalf("failed to run binary: %v\n%s", err, output)
@@ -484,7 +484,7 @@ func TestSelectorFiltering(t *testing.T) {
 	os.RemoveAll(outputDir)
 
 	// Test 5: Mix of include and exclude -s env=production -S language=python (should include only prod with go)
-	cmd = exec.Command(binaryPath, "-d", contextDir, "-o", outputDir, "-s", "env=production", "-S", "language=python", "test-task")
+	cmd = exec.Command(binaryPath, "-m", memoriesDir, "-t", tasksDir, "-o", outputDir, "-s", "env=production", "-S", "language=python", "test-task")
 	cmd.Dir = tmpDir
 	if output, err := cmd.CombinedOutput(); err != nil {
 		t.Fatalf("failed to run binary: %v\n%s", err, output)
@@ -543,7 +543,7 @@ The project is for $company.
 
 	// Run the binary with parameters
 	cmd = exec.Command(binaryPath, 
-		"-d", contextDir, 
+		"-t", tasksDir, 
 		"-o", outputDir,
 		"-p", "taskName=AddAuth",
 		"-p", "feature=Authentication",
@@ -608,7 +608,7 @@ Missing var: ${missingVar}
 
 	// Run the binary with only one parameter
 	cmd = exec.Command(binaryPath, 
-		"-d", contextDir, 
+		"-t", tasksDir, 
 		"-o", outputDir,
 		"-p", "providedVar=ProvidedValue",
 		"test-missing")
@@ -694,7 +694,7 @@ t.Fatalf("failed to write prompt file: %v", err)
 }
 
 // Run the binary WITH the -b flag
-cmd = exec.Command(binaryPath, "-d", contextDir, "-o", outputDir, "-b", "test-task")
+cmd = exec.Command(binaryPath, "-m", memoriesDir, "-t", tasksDir, "-o", outputDir, "-b", "test-task")
 cmd.Dir = tmpDir
 if output, err := cmd.CombinedOutput(); err != nil {
 t.Fatalf("failed to run binary: %v\n%s", err, output)
@@ -773,7 +773,7 @@ t.Fatalf("failed to write prompt file: %v", err)
 }
 
 // Run the binary WITHOUT the -b flag
-cmd = exec.Command(binaryPath, "-d", contextDir, "-o", outputDir, "test-task")
+cmd = exec.Command(binaryPath, "-m", memoriesDir, "-t", tasksDir, "-o", outputDir, "test-task")
 cmd.Dir = tmpDir
 if output, err := cmd.CombinedOutput(); err != nil {
 t.Fatalf("failed to run binary: %v\n%s", err, output)
@@ -842,7 +842,7 @@ t.Fatalf("failed to write prompt file: %v", err)
 }
 
 // Run the binary WITH the -b flag and send interrupt signal
-cmd = exec.Command(binaryPath, "-d", contextDir, "-o", outputDir, "-b", "test-task")
+cmd = exec.Command(binaryPath, "-m", memoriesDir, "-t", tasksDir, "-o", outputDir, "-b", "test-task")
 cmd.Dir = tmpDir
 
 // Start the command

main.go

@@ -18,7 +18,6 @@ import (
 var bootstrap string
 
 var (
-	dirs         stringSlice
 	memories     stringSlice
 	tasks        stringSlice
 	outputDir    = "."
@@ -38,17 +37,25 @@ func main() {
 		os.Exit(1)
 	}
 
-	dirs = []string{
-		".prompts",
-		filepath.Join(userConfigDir, "prompts"),
-		"/var/local/prompts",
-	}
-
 	memories = []string{
 		"AGENTS.md",
+		".github/copilot-instructions.md",
+		"CLAUDE.md",
+		".cursorrules",
+		".cursor/rules/",
+		".instructions.md",
+		".continuerules",
+		".prompts/memories",
+		filepath.Join(userConfigDir, "prompts", "memories"),
+		"/var/local/prompts/memories",
+	}
+
+	tasks = []string{
+		".prompts/tasks",
+		filepath.Join(userConfigDir, "prompts", "tasks"),
+		"/var/local/prompts/tasks",
 	}
 
-	flag.Var(&dirs, "d", "Directory that may contain a memories/ or tasks/ subdirectory. Can be specified multiple times.")
 	flag.Var(&memories, "m", "Directory containing memories, or a single memory file. Can be specified multiple times.")
 	flag.Var(&tasks, "t", "Directory containing tasks, or a single task file. Can be specified multiple times.")
 	flag.StringVar(&outputDir, "o", ".", "Directory to write the context files to.")
@@ -95,14 +102,6 @@ func run(ctx context.Context, args []string) error {
 	}
 	defer output.Close()
 
-	for _, dir := range dirs {
-		// insert the dirs into the start of the list
-		memory := filepath.Join(dir, "memories")
-		memories = append([]string{memory}, memories...)
-		task := filepath.Join(dir, "tasks")
-		tasks = append([]string{task}, tasks...)
-	}
-
 	for _, memory := range memories {
 
 		// Skip if the file doesn't exist