Skip to content

fix: [ENG-2120] [PRODUCT-541] handle breaking changes to alpha nodes API#170

Merged
sigmachirality merged 5 commits into
mainfrom
dt/update-nodes-sdk-usage
Aug 5, 2025
Merged

fix: [ENG-2120] [PRODUCT-541] handle breaking changes to alpha nodes API#170
sigmachirality merged 5 commits into
mainfrom
dt/update-nodes-sdk-usage

Conversation

@sigmachirality
Copy link
Copy Markdown
Member

@sigmachirality sigmachirality commented Aug 5, 2025

We've made some breaking changes to v1/nodes (as we are allowed to, as v1/nodes is in Alpha testing and not yet GA, so we are not yet committed to non-breaking semantic changes yet).

Updates CLI copy and API/SDK calls to handle this.

@semanticdiff-com
Copy link
Copy Markdown

semanticdiff-com Bot commented Aug 5, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  src/lib/nodes/utils.ts  14% smaller
  src/lib/nodes/list.tsx  3% smaller
  deno.lock Unsupported file format
  package.json  0% smaller
  src/lib/nodes/create.ts  0% smaller
  src/lib/nodes/index.ts  0% smaller
  src/lib/nodes/set.ts  0% smaller

@sigmachirality sigmachirality changed the title fix: handle breaking changes to alpha nodes API fix: [PRODUCT-541] handle breaking changes to alpha nodes API Aug 5, 2025
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Greptile Summary

This PR updates the CLI to handle breaking changes in the alpha nodes API by systematically replacing 'on-demand' node terminology with 'spot' nodes throughout the codebase. The changes include:

Dependency Update: The @sfcompute/nodes-sdk-alpha package is updated from version 0.1.0-alpha.5 to 0.1.0-alpha.7 in package.json, indicating compatibility with the new API version.

Terminology Standardization: All references to 'on-demand' nodes have been updated to 'spot' nodes across multiple files:

  • src/lib/nodes/utils.ts: Updated the printNodeType function to map 'spot' instead of 'on_demand'
  • src/lib/nodes/list.tsx: Updated display logic, conditional checks, and user-facing text to use 'spot' terminology
  • src/lib/nodes/create.ts: Updated error messages, comments, and API parameters to use 'spot' node types
  • src/lib/nodes/index.ts: Updated help text to describe 'spot pricing' instead of 'on-demand pricing'
  • src/lib/nodes/set.ts: Updated comments and error messages to reference 'spot nodes'

The terminology change aligns with common cloud computing patterns where 'spot' instances are preemptible/interruptible compute resources. All functional logic remains unchanged - spot nodes (formerly on-demand) still behave the same way with procurement-based pricing, no fixed duration, and the ability to be reclaimed by the provider. This is a coordinated update across the entire nodes module to maintain consistency with the updated API schema.

Confidence score: 5/5

  • This PR is safe to merge with minimal risk as it consists entirely of terminology updates with no functional logic changes
  • Score reflects straightforward string replacements and consistent application across all relevant files
  • No files require special attention as all changes are cosmetic terminology updates

6 files reviewed, no comments

Edit Code Review Bot Settings | Greptile

@sigmachirality sigmachirality changed the title fix: [PRODUCT-541] handle breaking changes to alpha nodes API fix: [ENG-2120] [PRODUCT-541] handle breaking changes to alpha nodes API Aug 5, 2025
@sigmachirality sigmachirality self-assigned this Aug 5, 2025
Copy link
Copy Markdown
Contributor

@joshi4 joshi4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@joshi4 joshi4 requested a review from romiphadte August 5, 2025 21:03
@sigmachirality sigmachirality merged commit 9f798cf into main Aug 5, 2025
1 check passed
@sigmachirality sigmachirality deleted the dt/update-nodes-sdk-usage branch August 5, 2025 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants