feat: change default strategy from pr to commit

Also rename remaining "cq" references to "issuebot" in source code.

Author: JonCanning

README.md

@@ -70,7 +70,7 @@ issuebot --local --command "./my-issue-handler.sh {prompt}"
 | `--org` | | GitHub org to watch (all non-archived source repos) |
 | `--repo` | | Single `owner/repo` to watch |
 | `--label` | | Only process issues with this label (default: all open issues) |
-| `--strategy` | `pr` | Git strategy: `pr` (branch + PR) or `commit` (push to default branch) |
+| `--strategy` | `commit` | Git strategy: `pr` (branch + PR) or `commit` (push to default branch) |
 | `--interval` | `30s` | Polling interval |
 | `--workers` | `5` | Max concurrent repo workers |
 | `--workspace` | `~/.issuebot/repos` | Directory for cloned repos |

cmd/watch.go

@@ -164,7 +164,7 @@ func runWatch(cmd *cobra.Command, args []string) error {
 		target = cfg.Repo
 	}
 
-	slog.Info("starting cq",
+	slog.Info("starting issuebot",
 		"target", target, "label", cfg.Label,
 		"strategy", cfg.Strategy, "interval", cfg.Interval,
 		"workers", cfg.Workers,

internal/config/config.go

@@ -42,7 +42,7 @@ var issueConfigRe = regexp.MustCompile(`(?s)<!--\s*issuebot\s*\n(.*?)\n-->`)
 func DefaultCLIConfig() CLIConfig {
 	return CLIConfig{
 		Label:      "",
-		Strategy:   "pr",
+		Strategy:   "commit",
 		Interval:   30 * time.Second,
 		Workers:    5,
 		MaxRetries: 3,
@@ -62,7 +62,7 @@ func ParseIssueConfig(body string) IssueConfig {
 	}
 
 	if err := yaml.Unmarshal([]byte(m[1]), &cfg); err != nil {
-		slog.Warn("malformed cq config in issue body", "error", err)
+		slog.Warn("malformed issuebot config in issue body", "error", err)
 	}
 
 	return cfg

internal/config/config_test.go

@@ -82,7 +82,7 @@ func TestDefaultConfig(t *testing.T) {
 		got  interface{}
 		want interface{}
 	}{
-		{"strategy", cfg.Strategy, "pr"},
+		{"strategy", cfg.Strategy, "commit"},
 		{"interval", cfg.Interval, 30 * time.Second},
 		{"workers", cfg.Workers, 5},
 		{"max-retries", cfg.MaxRetries, 3},
@@ -134,8 +134,8 @@ func TestResolveStrategy_FallbackToCLI(t *testing.T) {
 	cli := DefaultCLIConfig()
 
 	issue := IssueConfig{}
-	if got := ResolveStrategy(cli, issue); got != "pr" {
-		t.Errorf("got %q, want %q", got, "pr")
+	if got := ResolveStrategy(cli, issue); got != "commit" {
+		t.Errorf("got %q, want %q", got, "commit")
 	}
 }