fix install issue in version manager

[newbe36524]

Summary by CodeRabbit

• Documentation

  • Added technical proposal and implementation plan documentation for dependency installation process improvements.

• Refactor

  • Enhanced dependency installation by consolidating all operations into a single batch process, reducing total installation time and providing improved real-time progress monitoring and comprehensive error handling for installation failures.

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This PR optimizes dependency installation in OnboardingManager by replacing sequential per-dependency installs with a single batch call to installFromManifest. Changes include batch result aggregation, progress callback integration for real-time status updates, and enhanced error handling for batch failures. The implementation plan and proposal documentation outline the migration strategy and acceptance criteria.

Changes

Cohort / File(s)	Summary
Documentation & Specification openspec/changes/archive/2026-02-15-optimize-dependency-installation-single-script-call/original/proposal.md, openspec/changes/archive/2026-02-15-optimize-dependency-installation-single-script-call/original/tasks.md	Proposal document describing the optimization to collapse dependency installation into a single batch call; tasks document detailing implementation steps, testing scenarios, acceptance criteria, and code changes for the OnboardingManager refactor.
Core Implementation src/main/onboarding-manager.ts	Replaces loop-based per-dependency installation with batch installFromManifest call; introduces progress callback for real-time status updates; aggregates batch results with success/failure counts; expands error handling for batch failures and catch scenarios; updates logging for batch completion.

Sequence Diagram

sequenceDiagram
    participant Client
    participant OnboardingManager
    participant DependencyManager
    participant InstallScript

    Client->>OnboardingManager: installDependencies(versionId, onProgress)
    
    OnboardingManager->>OnboardingManager: Mark all missing as "installing"
    OnboardingManager->>Client: onProgress(updatedStatus)
    
    OnboardingManager->>DependencyManager: installFromManifest(dependencies, progressCallback)
    
    DependencyManager->>InstallScript: Execute batch install script
    
    InstallScript-->>DependencyManager: Progress: item X installed
    DependencyManager->>OnboardingManager: progressCallback(itemStatus)
    OnboardingManager->>Client: onProgress(updatedStatus)
    
    InstallScript-->>DependencyManager: Batch results (success/failure)
    DependencyManager-->>OnboardingManager: Install results
    
    OnboardingManager->>OnboardingManager: Aggregate & map failures
    OnboardingManager->>OnboardingManager: Log batch outcome
    
    OnboardingManager-->>Client: { success, error? }

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• fix dependency installation issue #17 — Modifies the same onboarding-manager.ts and dependency-manager code paths to integrate DependencyManager.installFromManifest with signature/entryPoint/progress handling.

Poem

🐰 A batch of dependencies, bundled with care,
No loops to delay what's already there,
One script to install them, swift and true,
Progress flows freely, so shiny and new! ✨

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/fix

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}