feat: set default models to Sonnet 4.6, add prettier and golangci-lint installer support

Author: maxritter

installer/steps/dependencies.py

@@ -467,6 +467,30 @@ def install_typescript_lsp() -> bool:
     return _run_bash_with_retry(npm_global_cmd("npm install -g @vtsls/language-server typescript"))
 
 
+def install_prettier() -> bool:
+    """Install prettier code formatter globally for TypeScript/JavaScript files."""
+    if command_exists("prettier"):
+        return True
+    return _run_bash_with_retry(npm_global_cmd("npm install -g prettier"))
+
+
+def install_golangci_lint() -> bool:
+    """Install golangci-lint for comprehensive Go code linting.
+
+    Skips if Go is not installed, since golangci-lint requires it.
+    Uses the official install script to place the binary in $(go env GOPATH)/bin.
+    """
+    if command_exists("golangci-lint"):
+        return True
+    if not command_exists("go"):
+        return False
+    install_cmd = (
+        "curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh"
+        " | sh -s -- -b $(go env GOPATH)/bin"
+    )
+    return _run_bash_with_retry(install_cmd, timeout=120)
+
+
 def _is_ccusage_installed() -> bool:
     """Check if ccusage is installed globally."""
     try:
@@ -818,6 +842,12 @@ def run(self, ctx: InstallContext) -> None:
         if _install_with_spinner(ui, "vtsls (TypeScript LSP server)", install_typescript_lsp):
             installed.append("typescript_lsp")
 
+        if _install_with_spinner(ui, "prettier (TypeScript formatter)", install_prettier):
+            installed.append("prettier")
+
+        if _install_with_spinner(ui, "golangci-lint (Go linter)", install_golangci_lint):
+            installed.append("golangci_lint")
+
         if _install_with_spinner(ui, "ccusage (usage tracking)", install_ccusage):
             installed.append("ccusage")
 

installer/tests/unit/steps/test_dependencies.py

@@ -31,6 +31,8 @@ def test_dependencies_check_returns_false(self):
             )
             assert step.check(ctx) is False
 
+    @patch("installer.steps.dependencies.install_golangci_lint", return_value=True)
+    @patch("installer.steps.dependencies.install_prettier", return_value=True)
     @patch("installer.steps.dependencies._precache_npx_mcp_servers", return_value=True)
     @patch("installer.steps.dependencies.install_vexor")
     @patch("installer.steps.dependencies._install_plugin_dependencies")
@@ -49,6 +51,8 @@ def test_dependencies_run_installs_core(
         mock_plugin_deps,
         mock_vexor,
         _mock_precache,
+        _mock_prettier,
+        _mock_golangci_lint,
     ):
         """DependenciesStep installs all dependencies including Python tools."""
         from installer.context import InstallContext
@@ -769,3 +773,110 @@ def test_install_vexor_routes_to_mlx_on_macos_arm64(self, _mock_platform, mock_m
 
         assert result is True
         mock_mlx.assert_called_once()
+
+
+class TestInstallPrettier:
+    """Test prettier global installation."""
+
+    def test_install_prettier_exists(self):
+        """install_prettier function exists."""
+        from installer.steps.dependencies import install_prettier
+
+        assert callable(install_prettier)
+
+    @patch("installer.steps.dependencies.command_exists", return_value=True)
+    def test_install_prettier_skips_if_already_installed(self, _mock_cmd):
+        """install_prettier returns True without installing when prettier is in PATH."""
+        from installer.steps.dependencies import install_prettier
+
+        with patch("installer.steps.dependencies._run_bash_with_retry") as mock_run:
+            result = install_prettier()
+
+        assert result is True
+        mock_run.assert_not_called()
+
+    @patch("installer.steps.dependencies._run_bash_with_retry", return_value=True)
+    @patch("installer.steps.dependencies.command_exists", return_value=False)
+    def test_install_prettier_installs_via_npm(self, _mock_cmd, mock_run):
+        """install_prettier uses npm install -g prettier when not in PATH."""
+        from installer.steps.dependencies import install_prettier
+
+        result = install_prettier()
+
+        assert result is True
+        mock_run.assert_called_once()
+        assert "prettier" in mock_run.call_args[0][0]
+        assert "npm install -g" in mock_run.call_args[0][0]
+
+    @patch("installer.steps.dependencies._run_bash_with_retry", return_value=False)
+    @patch("installer.steps.dependencies.command_exists", return_value=False)
+    def test_install_prettier_returns_false_on_failure(self, _mock_cmd, mock_run):
+        """install_prettier returns False when npm install fails."""
+        from installer.steps.dependencies import install_prettier
+
+        result = install_prettier()
+
+        assert result is False
+
+
+class TestInstallGolangciLint:
+    """Test golangci-lint installation."""
+
+    def test_install_golangci_lint_exists(self):
+        """install_golangci_lint function exists."""
+        from installer.steps.dependencies import install_golangci_lint
+
+        assert callable(install_golangci_lint)
+
+    @patch("installer.steps.dependencies.command_exists", return_value=True)
+    def test_install_golangci_lint_skips_if_already_installed(self, mock_cmd):
+        """install_golangci_lint returns True without installing when already in PATH."""
+        from installer.steps.dependencies import install_golangci_lint
+
+        with patch("installer.steps.dependencies._run_bash_with_retry") as mock_run:
+            result = install_golangci_lint()
+
+        assert result is True
+        mock_run.assert_not_called()
+
+    @patch("installer.steps.dependencies.command_exists")
+    def test_install_golangci_lint_skips_without_go(self, mock_cmd):
+        """install_golangci_lint returns False when go is not installed."""
+        from installer.steps.dependencies import install_golangci_lint
+
+        mock_cmd.side_effect = lambda cmd: cmd != "golangci-lint" and False
+
+        with patch("installer.steps.dependencies._run_bash_with_retry") as mock_run:
+            result = install_golangci_lint()
+
+        assert result is False
+        mock_run.assert_not_called()
+
+    @patch("installer.steps.dependencies._run_bash_with_retry", return_value=True)
+    @patch("installer.steps.dependencies.command_exists")
+    def test_install_golangci_lint_uses_official_script(self, mock_cmd, mock_run):
+        """install_golangci_lint uses the official install.sh script."""
+        from installer.steps.dependencies import install_golangci_lint
+
+        mock_cmd.side_effect = lambda cmd: cmd == "go"
+
+        result = install_golangci_lint()
+
+        assert result is True
+        mock_run.assert_called_once()
+        call_args = mock_run.call_args[0][0]
+        assert "golangci-lint" in call_args
+        assert "install.sh" in call_args
+        assert "go env GOPATH" in call_args
+
+    @patch("installer.steps.dependencies._run_bash_with_retry", return_value=False)
+    @patch("installer.steps.dependencies.command_exists")
+    def test_install_golangci_lint_returns_false_on_failure(self, mock_cmd, mock_run):
+        """install_golangci_lint returns False when install script fails."""
+        from installer.steps.dependencies import install_golangci_lint
+
+        mock_cmd.side_effect = lambda cmd: cmd == "go"
+
+        result = install_golangci_lint()
+
+        assert result is False

pilot/commands/learn.md

@@ -1,6 +1,6 @@
 ---
 description: Extract reusable knowledge into skills - online learning system
-model: sonnet[1m]
+model: sonnet
 ---
 
 # /learn - Online Learning System

pilot/commands/spec-implement.md

@@ -2,7 +2,7 @@
 description: "Spec implementation phase - TDD loop for each task in the plan"
 argument-hint: "<path/to/plan.md>"
 user-invocable: false
-model: sonnet[1m]
+model: sonnet
 ---
 
 # /spec-implement - Implementation Phase

pilot/commands/spec-plan.md

@@ -2,7 +2,7 @@
 description: "Spec planning phase - explore codebase, design plan, get approval"
 argument-hint: "<task description> or <path/to/plan.md>"
 user-invocable: false
-model: opus[1m]
+model: opus
 hooks:
   Stop:
     - command: uv run python "${CLAUDE_PLUGIN_ROOT}/hooks/spec_plan_validator.py"

pilot/commands/spec.md

@@ -2,7 +2,7 @@
 description: Spec-driven development - plan, implement, verify workflow
 argument-hint: "<task description>" or "<path/to/plan.md>"
 user-invocable: true
-model: sonnet[1m]
+model: sonnet
 ---
 
 # /spec - Unified Spec-Driven Development

pilot/commands/sync.md

@@ -1,7 +1,7 @@
 ---
 description: Sync project rules and skills with codebase - reads existing rules/skills, explores code, updates documentation, creates new skills
 user-invocable: true
-model: sonnet[1m]
+model: sonnet
 ---
 
 # /sync - Sync Project Rules & Skills

pilot/commands/vault.md

@@ -1,7 +1,7 @@
 ---
 description: Manage Team Vault - share and install rules, commands, skills across your team via sx
 user-invocable: true
-model: sonnet[1m]
+model: sonnet
 ---
 
 # /vault - Team Vault Management

pilot/settings.json

@@ -1,5 +1,5 @@
 {
-  "model": "sonnet[1m]",
+  "model": "sonnet",
   "env": {
     "CLAUDE_CODE_ENABLE_TASKS": "true",
     "CLAUDE_CODE_HIDE_ACCOUNT_INFO": "true",