NO-JIRA | chore: update planner-sdk to latest version that includes changes related with time estimation

[ammont82]

Update planner-sdk to latest version and update the related code

Summary by CodeRabbit

• Chores
  • Updated SDK dependency version
• Refactor
  • Updated migration estimation calculation handling in cluster sizing wizard to align with SDK API changes

[coderabbitai]

📝 Walkthrough

Walkthrough

The pull request updates the @openshift-migration-advisor/planner-sdk dependency to a build-tagged version and refactors migration estimation types across the codebase. The return type of calculateMigrationEstimation() is changed to MigrationEstimationResponse, with corresponding type adjustments in the view model and re-exports from the external SDK.

Changes

Cohort / File(s)	Summary
Dependency Update package.json	Updated @openshift-migration-advisor/planner-sdk from 0.10.0 to 0.10.0-783d8b3a2dc4 (build-tagged variant).
Interface Type Signature src/data/stores/interfaces/IAssessmentsStore.ts	Changed calculateMigrationEstimation() return type from Promise<Record<string, SchemaEstimationResult>> to Promise<MigrationEstimationResponse> and updated corresponding import.
View Model Type & Logic src/ui/report/view-models/useClusterSizingWizardViewModel.ts	Updated migrationEstimation property type from MigrationEstimationResponse | null to Record<string, SchemaEstimationResult> | null; modified processing logic to extract result?.estimation into schemas variable before storage.
Type Re-exports src/ui/report/views/cluster-sizer/types.ts	Removed local MigrationEstimationResponse type alias and now re-export it directly from @openshift-migration-advisor/planner-sdk; removed unused SchemaEstimationResult import.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• ECOPROJECT-4287 | feat: add multi-schema migration time estimation UI #523: Modifies the same interfaces for migration estimation (IAssessmentsStore, ClusterSizingWizardViewModel, cluster-sizer types) with inverse type direction changes.
• ECOPROJECT-4244 | feat: implement complexity by Disk&OS tab #566: Touches migration estimation surface and shared type re-exports from planner-sdk.

Suggested labels

lgtm

Suggested reviewers

• machacekondra
• bardielle
• nirarg

Poem

🐰 A hop, a skip, types realigned,
The planner-sdk's new home we find,
From local alias to import true,
MigrationEstimation shines anew! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly describes the main change: updating the planner-sdk dependency to a newer version that includes time estimation-related changes, which is reflected in all four modified files.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[ammont82]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ammont82

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [ammont82]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment