fix: add multi-module flags to tag subcommands (#259)

* fix: add multi-module flags to tag subcommands

Move MultiModuleFlags to create, list, and push subcommands. Remove -m alias from --message on create to avoid conflict with --module alias.

Author: indaco

internal/commands/discover/workflow.go

@@ -96,7 +96,7 @@ func (w *Workflow) runInitWorkflow(ctx context.Context) (bool, error) {
 
 	// Check if we have useful suggestions
 	if len(w.result.SyncCandidates) == 0 && len(w.result.Modules) == 0 {
-		// No .version files found — check for monorepo workspace markers
+		// No .version files found - check for monorepo workspace markers
 		// (go.work, pnpm-workspace.yaml, package.json workspaces, Cargo.toml [workspace])
 		if monoInfo, err := initialize.DetectMonorepo(); err == nil && monoInfo != nil {
 			return w.runMonorepoInitWorkflow(ctx, monoInfo)

internal/commands/initialize/detection.go

@@ -300,7 +300,7 @@ func detectCargoWorkspace() (*MonorepoInfo, error) {
 			continue
 		}
 
-		// New section starts — stop if we were in [workspace]
+		// New section starts - stop if we were in [workspace]
 		if strings.HasPrefix(line, "[") && line != "[workspace]" {
 			if inWorkspace {
 				break

internal/commands/initialize/detection_test.go

@@ -566,7 +566,7 @@ func TestDetectMonorepo_Priority(t *testing.T) {
 	tmpDir := t.TempDir()
 	t.Chdir(tmpDir)
 
-	// Both go.work and pnpm-workspace.yaml present — go.work should win
+	// Both go.work and pnpm-workspace.yaml present - go.work should win
 	writeTestFile(t, "go.work", "go 1.21\nuse ./mod1\n")
 	mkdirTest(t, "mod1")
 	writeTestFile(t, "pnpm-workspace.yaml", "packages:\n  - \"packages/*\"\n")

internal/commands/tag/tagcmd.go

@@ -54,22 +54,24 @@ func Run(cfg *config.Config) *cli.Command {
 
 // createCmd returns the "tag create" subcommand.
 func (tc *TagCommand) createCmd(cfg *config.Config) *cli.Command {
+	flags := []cli.Flag{
+		&cli.BoolFlag{
+			Name:  "push",
+			Usage: "Push the tag to remote after creation",
+		},
+		&cli.StringFlag{
+			Name:  "message",
+			Usage: "Override the tag message (for annotated/signed tags)",
+		},
+	}
+	flags = append(flags, cliflags.MultiModuleFlags()...)
+
 	return &cli.Command{
 		Name:      "create",
 		Aliases:   []string{"c", "new"},
 		Usage:     "Create a git tag for the current version",
-		UsageText: "sley tag create [--push] [--message <msg>]",
-		Flags: []cli.Flag{
-			&cli.BoolFlag{
-				Name:  "push",
-				Usage: "Push the tag to remote after creation",
-			},
-			&cli.StringFlag{
-				Name:    "message",
-				Aliases: []string{"m"},
-				Usage:   "Override the tag message (for annotated/signed tags)",
-			},
-		},
+		UsageText: "sley tag create [--push] [--message <msg>] [--all] [--module name]",
+		Flags:     flags,
 		Action: func(ctx context.Context, cmd *cli.Command) error {
 			return tc.runCreateCmd(ctx, cmd, cfg)
 		},
@@ -78,19 +80,22 @@ func (tc *TagCommand) createCmd(cfg *config.Config) *cli.Command {
 
 // listCmd returns the "tag list" subcommand.
 func (tc *TagCommand) listCmd(cfg *config.Config) *cli.Command {
+	flags := []cli.Flag{
+		&cli.IntFlag{
+			Name:    "limit",
+			Aliases: []string{"n"},
+			Usage:   "Limit the number of tags shown",
+			Value:   0,
+		},
+	}
+	flags = append(flags, cliflags.MultiModuleFlags()...)
+
 	return &cli.Command{
 		Name:      "list",
 		Aliases:   []string{"l", "ls"},
 		Usage:     "List existing version tags",
-		UsageText: "sley tag list [--limit <n>]",
-		Flags: []cli.Flag{
-			&cli.IntFlag{
-				Name:    "limit",
-				Aliases: []string{"n"},
-				Usage:   "Limit the number of tags shown",
-				Value:   0,
-			},
-		},
+		UsageText: "sley tag list [--limit <n>] [--all] [--module name]",
+		Flags:     flags,
 		Action: func(ctx context.Context, cmd *cli.Command) error {
 			return tc.runListCmd(ctx, cmd, cfg)
 		},
@@ -103,7 +108,8 @@ func (tc *TagCommand) pushCmd(cfg *config.Config) *cli.Command {
 		Name:      "push",
 		Aliases:   []string{"p"},
 		Usage:     "Push a tag to remote",
-		UsageText: "sley tag push [tag-name]",
+		UsageText: "sley tag push [tag-name] [--all] [--module name]",
+		Flags:     cliflags.MultiModuleFlags(),
 		Action: func(ctx context.Context, cmd *cli.Command) error {
 			return tc.runPushCmd(ctx, cmd, cfg)
 		},

internal/git/git_error_recovery_test.go

@@ -129,7 +129,7 @@ func TestUpdateRepo_NonExistentRepo(t *testing.T) {
 // TestCloneOrUpdate_UpdateError tests error propagation when update fails.
 func TestCloneOrUpdate_UpdateError(t *testing.T) {
 	t.Parallel(
-	// Create a repo with no remote configured — git pull will fail
+	// Create a repo with no remote configured - git pull will fail
 	)
 
 	sourceRepo := setupTestRepo(t)

internal/git/ref.go

@@ -25,7 +25,7 @@ func SafeFallbackSince(execCommand func(string, ...string) *exec.Cmd, n int) str
 		return fmt.Sprintf("HEAD~%d", n)
 	}
 
-	// Fewer than n commits — use the root commit so the range covers everything.
+	// Fewer than n commits - use the root commit so the range covers everything.
 	root := execCommand("git", "rev-list", "--max-parents=0", "HEAD")
 	rootOut, err := root.Output()
 	if err != nil {

internal/pathutil/pathutil.go

@@ -28,7 +28,7 @@ func resolveSymlinks(absBase, absPath string) (string, string) {
 		return resolvedBase, filepath.Join(resolvedParent, filepath.Base(absPath))
 	}
 
-	// Cannot resolve path side — fall back to both unresolved to avoid mismatch
+	// Cannot resolve path side - fall back to both unresolved to avoid mismatch
 	return absBase, absPath
 }
 

internal/plugins/changeloggenerator/generator.go

@@ -519,15 +519,15 @@ func moduleNameFromPath(changesDir, filePath string) string {
 }
 
 // prefixVersionHeading injects a module name into the first ## heading of
-// changelog content. "## v0.1.0 - date" becomes "## <module> — v0.1.0 - date".
+// changelog content. "## v0.1.0 - date" becomes "## <module> - v0.1.0 - date".
 // If no ## heading is found, the content is returned unchanged.
 func prefixVersionHeading(content, moduleName string) string {
 	const h2 = "## "
 	before, after, ok := strings.Cut(content, h2)
 	if !ok {
 		return content
 	}
-	return before + h2 + moduleName + " — " + after
+	return before + h2 + moduleName + " - " + after
 }
 
 // MergeVersionedFiles merges all versioned changelog files into a unified CHANGELOG.md.

internal/plugins/changeloggenerator/git.go

@@ -101,7 +101,7 @@ func (g *GitOps) getCommitsWithMeta(since, until string) ([]CommitInfo, error) {
 	if since == "" {
 		lastTag, err := g.getLatestTag()
 		if err != nil {
-			// No tags found — fall back to a safe recent range.
+			// No tags found - fall back to a safe recent range.
 			// Use HEAD~10 if enough commits exist, otherwise use the repo root.
 			since = git.SafeFallbackSince(g.ExecCommandFn, 10)
 		} else {

internal/plugins/changeloggenerator/plugin.go

@@ -149,7 +149,7 @@ func (p *ChangelogGeneratorPlugin) writeChangelog(version, content string) error
 	mode := p.config.Mode
 
 	// For unified mode with module context, inject the module name into the
-	// version heading: "## v0.1.0 - date" becomes "## <module> — v0.1.0 - date"
+	// version heading: "## v0.1.0 - date" becomes "## <module> - v0.1.0 - date"
 	unifiedContent := content
 	if p.moduleName != "" {
 		unifiedContent = prefixVersionHeading(content, p.moduleName)