refactor: Improve alias detection, typecheck handling, and test coverage roadmap

- Use regex for more robust `cr=cmdr` alias detection in `cmd/cmdr/main.go` to avoid false positives.
- Keep raw command string in `CommandRunner` constructor and normalize in `Run()` within `internal/cmdrunner.go`.
- Conditionally add the `typecheck` synthesized command based on `hasTypecheckCapability()` in `internal/interactive.go`.
- Use `os.DevNull` for typecheck commands, improving portability and correctness in `internal/sources_other.go`.
- Add a roadmap item to enable skipped typecheck tests in `ROADMAP.md`.

Author: osteele

ROADMAP.md

@@ -151,6 +151,7 @@ For more experimental and creative ideas, see [docs/ideas.txt](docs/ideas.txt).
 ### Test Coverage
 - ⬜ **Increase test coverage** - Add more unit tests, especially for command discovery and source detection
 - ⬜ **Integration test framework** - Create optional integration tests that run when tools are available (controlled by environment variable or build tag)
+- ⬜ **Enable skipped typecheck tests** - Replace skipped tests with runnable versions using mocks or temporary wrappers to regain coverage over typecheck orchestration code paths
 
 ### Lazy Loading
 - Load runners on-demand

cmd/cmdr/main.go

@@ -4,6 +4,7 @@ import (
 	"fmt"
 	"os"
 	"path/filepath"
+	"regexp"
 	"strings"
 
 	"github.com/osteele/cmd-runner/internal"
@@ -273,7 +274,10 @@ func installAlias(dryRun bool) error {
 			return fmt.Errorf("failed to read %s: %w", targetFile, err)
 		}
 
-		if strings.Contains(string(content), aliasLine) {
+		// Match "alias cr=cmdr" as a complete statement (not substring)
+		// This avoids false positives like "alias cr=cmdr-dev" or commented lines
+		aliasPattern := regexp.MustCompile(`(?m)^\s*alias\s+cr=cmdr\s*$`)
+		if aliasPattern.Match(content) {
 			if dryRun {
 				fmt.Printf("[DRY RUN] Alias 'cr' is already installed in %s\n", targetFile)
 			} else {

internal/cmdrunner.go

@@ -20,7 +20,7 @@ type CommandRunner struct {
 
 func New(command string, args []string) *CommandRunner {
 	return &CommandRunner{
-		Command: NormalizeCommand(command),
+		Command: command, // Keep raw command; normalization happens in Run()
 		Args:    args,
 	}
 }

internal/interactive.go

@@ -84,9 +84,13 @@ func (s *InteractiveSession) gatherCommands() {
 
 	// Add synthesized commands
 	synth := map[string]CommandInfo{
-		"check":     {Description: "Runs lint, typecheck, and test", Execution: "synthesized"},
-		"fix":       {Description: "Runs format and lint fix", Execution: "synthesized"},
-		"typecheck": {Description: "Runs type checking", Execution: "synthesized"},
+		"check": {Description: "Runs lint, typecheck, and test", Execution: "synthesized"},
+		"fix":   {Description: "Runs format and lint fix", Execution: "synthesized"},
+	}
+
+	// Only add typecheck if project has capability
+	if s.runner.hasTypecheckCapability() {
+		synth["typecheck"] = CommandInfo{Description: "Runs type checking", Execution: "synthesized"}
 	}
 
 	for cmd, info := range synth {

internal/sources_other.go

@@ -136,8 +136,8 @@ func (g *GoSource) FindCommand(command string, args []string) *exec.Cmd {
 		"setup":     {"mod", "download"},
 		"install":   {"install", "."},
 		"lint":      {"vet", "./..."},
-		"typecheck": {"build", "-o", "/dev/null", "./..."},
-		"tc":        {"build", "-o", "/dev/null", "./..."},
+		"typecheck": {"build", "-o", os.DevNull, "./..."},
+		"tc":        {"build", "-o", os.DevNull, "./..."},
 	}
 
 	for _, variant := range GetCommandVariants(command) {