Improve code comments and add logging for UI server startup

Co-authored-by: tikazyq <[email protected]>

Author: Copilot

packages/desktop/src-tauri/src/commands.rs

@@ -177,10 +177,16 @@ fn build_payload(app: &AppHandle, state: &DesktopState) -> Result<DesktopBootstr
     // Native SPA mode (spec 169 Phase 5) - no Node.js server needed
     // UI server only used if explicitly enabled via environment variable
     let ui_url = std::env::var("LEANSPEC_ENABLE_UI_SERVER").ok().and_then(|_| {
-        state
-            .ui_server
-            .ensure_running(app, active_project.as_ref())
-            .ok()
+        match state.ui_server.ensure_running(app, active_project.as_ref()) {
+            Ok(url) => {
+                eprintln!("Legacy UI server started at: {}", url);
+                Some(url)
+            }
+            Err(e) => {
+                eprintln!("Failed to start legacy UI server (this is normal in native SPA mode): {}", e);
+                None
+            }
+        }
     });
 
     Ok(DesktopBootstrapPayload {

packages/desktop/src-tauri/src/specs/reader.rs

@@ -190,8 +190,10 @@ impl SpecReader {
             format!("specs/{}/README.md", spec_name)
         };
 
-        // TODO: Add sub-specs tracking in the future if needed
-        // (currently we just load individual spec files)
+        // TODO: Implement sub-specs tracking feature
+        // Sub-specs are additional .md files in a spec directory beyond README.md
+        // This would allow specs to be broken into multiple documents
+        // Track in a future spec once the use case is validated
 
         Some(Spec {
             id,

packages/desktop/src-tauri/src/specs/validation.rs

@@ -111,7 +111,9 @@ pub fn validate_spec(spec: &Spec) -> ValidationResult {
 
     // Check for required sections
     let has_overview = body.contains("## Overview") || body.contains("## overview");
-    // TODO: Add validation for Design, Plan, Test sections in the future if needed
+    // Note: Validation for Design, Plan, Test sections was removed as these are
+    // optional sections per LeanSpec philosophy (minimal required structure).
+    // Only Overview section check remains as it's the core requirement.
 
     if !has_overview {
         issues.push(ValidationIssue {