feat: migrate terminal output to herald typography system (#256)

* refactor: replace console package with herald typography system

Introduce herald-backed printer with themed typography, replacing raw
ANSI escape codes in the console package. Add theme.go with SleyTheme
and ResolveTheme for named theme support. Initialize typography via
printer.Init(theme) in the CLI Before hook.

Co-Authored-By: claude <noreply@anthropic.com>

* refactor: migrate command output to herald typography

Use herald Compose, H2, H4, UL, KV, KVGroup, Table, Code, CodeBlock,
Tags, and Section across discover, doctor, init, changelog, extension,
hooks, workspace formatter, and extensionmgr output. Replace manual
string formatting with structured typography elements for consistent
visual hierarchy.

Co-Authored-By: claude <noreply@anthropic.com>

* style: use faint text with selective color highlights across commands

Replace blanket PrintSuccess/PrintInfo with PrintFaint and Info()
highlights on key values (versions, paths, names) for a cleaner
visual balance. Errors and warnings remain colored. Quiet-mode
summaries are unchanged.

Co-Authored-By: claude <noreply@anthropic.com>

* chore(deps): update herald to v0.13.0 and use Section/BR from upstream

Remove local printer.Section and printer.BR implementations in favor
of herald v0.13.0 Typography.Section(). Update all call sites to use
ty.Section() directly.

Co-Authored-By: claude <noreply@anthropic.com>

---------

Co-authored-by: claude <noreply@anthropic.com>

Author: indaco

go.mod

@@ -8,6 +8,7 @@ require (
 	charm.land/lipgloss/v2 v2.0.2
 	github.com/charmbracelet/colorprofile v0.4.3
 	github.com/goccy/go-yaml v1.19.2
+	github.com/indaco/herald v0.13.0
 	github.com/pelletier/go-toml/v2 v2.3.0
 	github.com/tidwall/sjson v1.2.5
 	github.com/urfave/cli/v3 v3.8.0

go.sum

@@ -50,6 +50,8 @@ github.com/dustin/go-humanize v1.0.1 h1:GzkhY7T5VNhEkwH0PVJgjz+fX1rhBrR7pRT3mDkp
 github.com/dustin/go-humanize v1.0.1/go.mod h1:Mu1zIs6XwVuF/gI1OepvI0qD18qycQx+mFykh5fBlto=
 github.com/goccy/go-yaml v1.19.2 h1:PmFC1S6h8ljIz6gMRBopkjP1TVT7xuwrButHID66PoM=
 github.com/goccy/go-yaml v1.19.2/go.mod h1:XBurs7gK8ATbW4ZPGKgcbrY1Br56PdM69F7LkFRi1kA=
+github.com/indaco/herald v0.13.0 h1:+xVG9Fx5NpuWhwku/9IlRL6I009NnX4VUGKvlZHTRxU=
+github.com/indaco/herald v0.13.0/go.mod h1:T5g1+XLYvpjouhzAGHnAHDCKizhESkoV6+QPZ3DhgWA=
 github.com/lucasb-eyer/go-colorful v1.4.0 h1:UtrWVfLdarDgc44HcS7pYloGHJUjHV/4FwW4TvVgFr4=
 github.com/lucasb-eyer/go-colorful v1.4.0/go.mod h1:R4dSotOR9KMtayYi1e77YzuveK+i7ruzyGqttikkLy0=
 github.com/mattn/go-runewidth v0.0.21 h1:jJKAZiQH+2mIinzCJIaIG9Be1+0NR+5sz/lYEEjdM8w=

internal/cli/cli.go

@@ -17,7 +17,6 @@ import (
 	"github.com/indaco/sley/internal/commands/show"
 	"github.com/indaco/sley/internal/commands/tag"
 	"github.com/indaco/sley/internal/config"
-	"github.com/indaco/sley/internal/console"
 	"github.com/indaco/sley/internal/plugins"
 	"github.com/indaco/sley/internal/printer"
 	"github.com/indaco/sley/internal/tui"
@@ -64,7 +63,6 @@ func New(cfg *config.Config, registry *plugins.PluginRegistry) *urfavecli.Comman
 			},
 		},
 		Before: func(ctx context.Context, cmd *urfavecli.Command) (context.Context, error) {
-			console.SetNoColor(noColorFlag)
 			printer.SetNoColor(noColorFlag)
 
 			// Theme priority: CLI flag > env var > config file > default
@@ -80,6 +78,7 @@ func New(cfg *config.Config, registry *plugins.PluginRegistry) *urfavecli.Comman
 			}
 
 			tui.SetTheme(theme)
+			printer.Init(theme)
 			return ctx, nil
 		},
 		Commands: []*urfavecli.Command{

internal/clix/clix.go

@@ -10,6 +10,7 @@ import (
 	"github.com/indaco/sley/internal/apperrors"
 	"github.com/indaco/sley/internal/config"
 	"github.com/indaco/sley/internal/core"
+	"github.com/indaco/sley/internal/printer"
 	"github.com/indaco/sley/internal/semver"
 	"github.com/indaco/sley/internal/tui"
 	"github.com/indaco/sley/internal/workspace"
@@ -45,7 +46,7 @@ func getOrInitVersionFileWith(path string, strict bool, mgr *semver.VersionManag
 		return false, err
 	}
 	if created {
-		fmt.Printf("Auto-initialized %s with default version\n", path)
+		printer.PrintFaint(fmt.Sprintf("Auto-initialized %s with default version", printer.Info(path)))
 	}
 	return created, nil
 }
@@ -322,7 +323,8 @@ func shouldShowTUIPrompt(cmd *cli.Command, options *executionOptions) bool {
 	if !tui.IsInteractive() {
 		return false
 	}
-	if cmd.Bool("yes") || cmd.Bool("non-interactive") || cmd.Bool("all") {
+	if cmd.Bool("yes") || cmd.Bool("non-interactive") || cmd.Bool("all") ||
+		cmd.IsSet("module") || len(cmd.StringSlice("modules")) > 0 || cmd.IsSet("pattern") {
 		return false
 	}
 	return !options.defaultToAll

internal/commands/bump/auto.go

@@ -150,7 +150,7 @@ func determineBumpType(deps *bumpDeps, registry *plugins.PluginRegistry, label s
 			}
 
 			if inferred != "" {
-				printer.PrintInfo(fmt.Sprintf("Inferred bump type: %s", inferred))
+				printer.PrintFaint(fmt.Sprintf("Inferred bump type: %s", printer.Info(inferred)))
 				switch inferred {
 				case "minor":
 					return operations.BumpMinor
@@ -231,7 +231,7 @@ func runSingleModuleAuto(ctx context.Context, cmd *cli.Command, cfg *config.Conf
 		return err
 	}
 
-	printer.PrintSuccess(fmt.Sprintf("Bumped version from %s to %s", current.String(), next.String()))
+	printer.PrintFaint(fmt.Sprintf("Bumped version from %s to %s", current.String(), printer.Info(next.String())))
 	return nil
 }
 
@@ -267,7 +267,7 @@ func getNextVersion(
 			}
 
 			if inferred != "" {
-				printer.PrintInfo(fmt.Sprintf("Inferred bump type: %s", inferred))
+				printer.PrintFaint(fmt.Sprintf("Inferred bump type: %s", printer.Info(inferred)))
 
 				if current.PreRelease != "" {
 					return promotePreRelease(current, preserveMeta), nil

internal/commands/bump/helpers.go

@@ -168,7 +168,7 @@ func commitAndTagAfterBump(registry *plugins.PluginRegistry, version semver.SemV
 	if err := tm.CommitChanges(version, extraFiles); err != nil {
 		return fmt.Errorf("failed to commit release changes: %w", err)
 	}
-	printer.PrintSuccess(fmt.Sprintf("Committed release changes for %s", version.String()))
+	printer.PrintFaint(fmt.Sprintf("Committed release changes for %s", printer.Info(version.String())))
 
 	// Create tag on the new commit
 	message := fmt.Sprintf("Release %s (%s bump)", version.String(), bumpType)
@@ -177,10 +177,10 @@ func commitAndTagAfterBump(registry *plugins.PluginRegistry, version semver.SemV
 	}
 
 	tagName := tm.FormatTagName(version)
-	printer.PrintSuccess(fmt.Sprintf("Created tag: %s", tagName))
+	printer.PrintFaint(fmt.Sprintf("Created tag: %s", printer.Info(tagName)))
 
 	if tm.GetConfig().Push {
-		printer.PrintSuccess(fmt.Sprintf("Pushed tag: %s", tagName))
+		printer.PrintFaint(fmt.Sprintf("Pushed tag: %s", printer.Info(tagName)))
 	}
 
 	return nil
@@ -280,12 +280,12 @@ func generateChangelogAfterBump(registry *plugins.PluginRegistry, version, _ sem
 	cfg := cg.GetConfig()
 	switch cfg.Mode {
 	case "versioned":
-		printer.PrintSuccess(fmt.Sprintf("Generated changelog: %s/%s.md", cfg.ChangesDir, versionStr))
+		printer.PrintFaint(fmt.Sprintf("Generated changelog: %s", printer.Info(fmt.Sprintf("%s/%s.md", cfg.ChangesDir, versionStr))))
 	case "unified":
-		printer.PrintSuccess(fmt.Sprintf("Updated changelog: %s", cfg.ChangelogPath))
+		printer.PrintFaint(fmt.Sprintf("Updated changelog: %s", printer.Info(cfg.ChangelogPath)))
 	case "both":
-		printer.PrintSuccess(fmt.Sprintf("Generated changelog: %s/%s.md and %s",
-			cfg.ChangesDir, versionStr, cfg.ChangelogPath))
+		printer.PrintFaint(fmt.Sprintf("Generated changelog: %s and %s",
+			printer.Info(fmt.Sprintf("%s/%s.md", cfg.ChangesDir, versionStr)), printer.Info(cfg.ChangelogPath)))
 	}
 
 	return nil

internal/commands/changelog/changelogcmd.go

@@ -62,25 +62,23 @@ Examples:
 // runMergeCmd executes the merge operation.
 func runMergeCmd(cmd *cli.Command, cfg *config.Config) error {
 	// Check if changelog-generator plugin is enabled
+	ty := printer.Typography()
 	if !isChangelogGeneratorEnabled(cfg) {
-		printer.PrintWarning("Warning: The changelog-generator plugin is not enabled.")
-		printer.PrintInfo("To enable it, add the following to your .sley.yaml:")
-		fmt.Println("")
-		fmt.Println("  plugins:")
-		fmt.Println("    changelog-generator:")
-		fmt.Println("      enabled: true")
-		fmt.Println("      mode: \"versioned\"")
-		fmt.Println("")
-		printer.PrintInfo("Proceeding with merge using default settings...")
-		fmt.Println("")
+		fmt.Println(ty.Compose(
+			printer.Warning("Warning: The changelog-generator plugin is not enabled."),
+			printer.Info("To enable it, add the following to your .sley.yaml:"),
+			ty.CodeBlock("plugins:\n  changelog-generator:\n    enabled: true\n    mode: \"versioned\"", "yaml"),
+			printer.Info("Proceeding with merge using default settings..."),
+		))
 	} else {
 		// Warn if merge-after is set to something other than manual
 		mergeAfter := cfg.Plugins.ChangelogGenerator.GetMergeAfter()
 		if mergeAfter != "manual" {
-			printer.PrintWarning(fmt.Sprintf("Warning: 'merge-after' is set to '%s' in your configuration.", mergeAfter))
-			printer.PrintInfo("Versioned changelog files are already being merged automatically.")
-			printer.PrintInfo("This manual merge command may result in duplicate entries or unexpected behavior.")
-			fmt.Println("")
+			fmt.Println(ty.Compose(
+				printer.Warning(fmt.Sprintf("Warning: 'merge-after' is set to '%s' in your configuration.", mergeAfter)),
+				printer.Info("Versioned changelog files are already being merged automatically."),
+				printer.Info("This manual merge command may result in duplicate entries or unexpected behavior."),
+			))
 		}
 	}
 
@@ -98,8 +96,8 @@ func runMergeCmd(cmd *cli.Command, cfg *config.Config) error {
 		return fmt.Errorf("failed to merge changelog files: %w", err)
 	}
 
-	printer.PrintSuccess(fmt.Sprintf("Successfully merged changelog files from %s into %s",
-		genCfg.ChangesDir, genCfg.ChangelogPath))
+	printer.PrintFaint(fmt.Sprintf("Merged changelog files from %s into %s",
+		printer.Info(genCfg.ChangesDir), printer.Info(genCfg.ChangelogPath)))
 
 	return nil
 }

internal/commands/changelog/changelogcmd_test.go

@@ -109,7 +109,7 @@ func TestChangelogMergeCmd_Success(t *testing.T) {
 	}
 
 	// Check success message
-	expectedMsg := "Successfully merged changelog files"
+	expectedMsg := "Merged changelog files"
 	if !strings.Contains(output, expectedMsg) {
 		t.Errorf("expected output to contain %q, got:\n%s", expectedMsg, output)
 	}
@@ -225,7 +225,7 @@ func TestChangelogMergeCmd_NoVersionedFiles(t *testing.T) {
 	}
 
 	// Should still report success (nothing to merge)
-	expectedMsg := "Successfully merged changelog files"
+	expectedMsg := "Merged changelog files"
 	if !strings.Contains(output, expectedMsg) {
 		t.Errorf("expected output to contain %q, got:\n%s", expectedMsg, output)
 	}
@@ -424,7 +424,7 @@ func TestChangelogMergeCmd_WarningWhenPluginDisabled(t *testing.T) {
 	}
 
 	// Should still succeed with merge
-	expectedMsg := "Successfully merged changelog files"
+	expectedMsg := "Merged changelog files"
 	if !strings.Contains(output, expectedMsg) {
 		t.Errorf("expected output to contain %q, got:\n%s", expectedMsg, output)
 	}
@@ -469,7 +469,7 @@ func TestChangelogMergeCmd_WarningWhenMergeAfterImmediate(t *testing.T) {
 	}
 
 	// Should still succeed with merge
-	expectedMsg := "Successfully merged changelog files"
+	expectedMsg := "Merged changelog files"
 	if !strings.Contains(output, expectedMsg) {
 		t.Errorf("expected output to contain %q, got:\n%s", expectedMsg, output)
 	}
@@ -520,7 +520,7 @@ func TestChangelogMergeCmd_NoWarningWhenMergeAfterManual(t *testing.T) {
 	}
 
 	// Should still succeed with merge
-	expectedMsg := "Successfully merged changelog files"
+	expectedMsg := "Merged changelog files"
 	if !strings.Contains(output, expectedMsg) {
 		t.Errorf("expected output to contain %q, got:\n%s", expectedMsg, output)
 	}

internal/commands/discover/discovercmd.go

@@ -8,6 +8,7 @@ import (
 	"github.com/indaco/sley/internal/config"
 	"github.com/indaco/sley/internal/core"
 	"github.com/indaco/sley/internal/discovery"
+	"github.com/indaco/sley/internal/printer"
 	"github.com/urfave/cli/v3"
 )
 
@@ -102,21 +103,26 @@ func runDiscoverCmd(ctx context.Context, cmd *cli.Command, cfg *config.Config) e
 
 // printQuietSummary prints a minimal summary of discovery results.
 func printQuietSummary(result *discovery.Result) {
+	ty := printer.Typography()
 	moduleCount := len(result.Modules)
 	manifestCount := len(result.Manifests)
 	mismatchCount := len(result.Mismatches)
 
-	fmt.Printf("Mode: %s | Modules: %d | Manifests: %d", result.Mode, moduleCount, manifestCount)
+	pairs := [][2]string{
+		{"Mode", result.Mode.String()},
+		{"Modules", fmt.Sprintf("%d", moduleCount)},
+		{"Manifests", fmt.Sprintf("%d", manifestCount)},
+	}
 
 	if mismatchCount > 0 {
-		fmt.Printf(" | Mismatches: %d", mismatchCount)
+		pairs = append(pairs, [2]string{"Mismatches", fmt.Sprintf("%d", mismatchCount)})
 	}
 
 	if result.PrimaryVersion() != "" {
-		fmt.Printf(" | Version: %s", result.PrimaryVersion())
+		pairs = append(pairs, [2]string{"Version", result.PrimaryVersion()})
 	}
 
-	fmt.Println()
+	fmt.Println(ty.KVGroup(pairs))
 }
 
 // DiscoverAndSuggest is a helper function that performs discovery and returns

internal/commands/discover/discovercmd_test.go

@@ -242,7 +242,7 @@ func TestCLI_DiscoverCommand_QuietMode(t *testing.T) {
 	}
 
 	// Verify quiet mode shows summary only
-	if !strings.Contains(output, "Mode:") {
+	if !strings.Contains(output, "Mode") {
 		t.Errorf("expected 'Mode:' in quiet output, got: %q", output)
 	}
 	// Quiet mode should be shorter than regular output
@@ -467,7 +467,7 @@ func TestPrintQuietSummary_WithPrimaryVersion(t *testing.T) {
 	if !strings.Contains(output, "5.0.0") {
 		t.Errorf("expected version '5.0.0' in quiet summary, got: %q", output)
 	}
-	if !strings.Contains(output, "Version:") {
+	if !strings.Contains(output, "Version") {
 		t.Errorf("expected 'Version:' in quiet summary, got: %q", output)
 	}
 }
@@ -492,8 +492,8 @@ func TestPrintQuietSummary_MultiModule(t *testing.T) {
 	if !strings.Contains(output, "MultiModule") {
 		t.Errorf("expected 'MultiModule' in quiet summary, got: %q", output)
 	}
-	if !strings.Contains(output, "Modules: 2") {
-		t.Errorf("expected 'Modules: 2' in quiet summary, got: %q", output)
+	if !strings.Contains(output, "Modules") {
+		t.Errorf("expected 'Modules' in quiet summary, got: %q", output)
 	}
 }
 
@@ -519,10 +519,10 @@ func TestPrintQuietSummary_WithManifestsAndMismatches(t *testing.T) {
 		t.Fatalf("Failed to capture stdout: %v", err)
 	}
 
-	if !strings.Contains(output, "Manifests: 1") {
+	if !strings.Contains(output, "Manifests") {
 		t.Errorf("expected 'Manifests: 1' in quiet summary, got: %q", output)
 	}
-	if !strings.Contains(output, "Mismatches: 1") {
+	if !strings.Contains(output, "Mismatches") {
 		t.Errorf("expected 'Mismatches: 1' in quiet summary, got: %q", output)
 	}
 }