fix: move vscode lazy imports to module level (#1183)

src/copilot_usage/cli.py

@@ -39,6 +39,8 @@
     render_session_detail,
     render_summary,
 )
+from copilot_usage.vscode_parser import get_vscode_summary
+from copilot_usage.vscode_report import render_vscode_summary
 
 __all__: Final[list[str]] = [
     "main",
@@ -592,9 +594,6 @@ def live(ctx: click.Context, path: Path | None) -> None:
 )
 def vscode(vscode_logs: Path | None) -> None:
     """Show usage from VS Code Copilot Chat logs."""
-    from copilot_usage.vscode_parser import get_vscode_summary
-    from copilot_usage.vscode_report import render_vscode_summary
-
     c = Console()
     _print_version_header(target=c)
     summary = get_vscode_summary(vscode_logs)

tests/copilot_usage/test_cli.py

@@ -4064,3 +4064,21 @@ def test_vscode_header_and_body_same_console(
         assert "Copilot Usage" in output
         assert f"v{__version__}" in output
         assert "VS Code Copilot Chat" in output
+
+
+# ---------------------------------------------------------------------------
+# Regression: vscode imports are module-level (not deferred)
+# ---------------------------------------------------------------------------
+
+
+def test_vscode_imports_are_module_level() -> None:
+    """``get_vscode_summary`` and ``render_vscode_summary`` are module-level imports.
+
+    Regression test for GitHub issue #1183: the ``vscode`` command previously
+    used lazy in-function imports, which deferred ``ImportError`` discovery to
+    invocation time instead of failing fast at process start.
+    """
+    import copilot_usage.cli as cli_module
+
+    assert hasattr(cli_module, "get_vscode_summary")
+    assert hasattr(cli_module, "render_vscode_summary")