Skip to content

feat: upgrade sdk version to 0.24.1#6

Merged
silent-cipher merged 4 commits intomainfrom
feat/bump-sdk-version
Sep 7, 2025
Merged

feat: upgrade sdk version to 0.24.1#6
silent-cipher merged 4 commits intomainfrom
feat/bump-sdk-version

Conversation

@silent-cipher
Copy link
Copy Markdown
Collaborator

No description provided.

@FilOzzy FilOzzy added this to FOC Sep 6, 2025
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FOC Sep 6, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Upgrades the Synapse SDK from version 0.24.0-dev1 to 0.24.1, introducing significant API changes including the use of the Synapse factory pattern and updated provider information structure.

  • Migrates to new SDK API with Synapse factory and async service initialization
  • Updates provider data structure from ApprovedProviderInfo to ProviderInfo with nested product information
  • Adds configuration option for dataset creation fee checking

Reviewed Changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
package.json Updates SDK version to 0.24.1
src/wallet-sdk/wallet-sdk.service.ts Refactors service initialization to use Synapse factory and new provider APIs
src/scheduler/scheduler.service.ts Adds provider count validation before operations
src/retrieval/retrieval.service.ts Updates provider service URL access to use nested product structure
src/infrastructure/database/entities/storage-provider.entity.ts Adds new provider metadata fields
src/domain/interfaces/external-services.interface.ts Updates type imports to use ProviderInfo
src/domain/entities/storage-provider.entity.ts Adds new provider metadata properties
src/deal/deal.service.ts Updates provider handling and service URL access patterns
src/config/app.config.ts Adds dataset creation fee configuration option
.env.example Documents new configuration variable
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)

src/retrieval/retrieval.service.ts:1

  • Using non-null assertion operator (!) without proper null checking is risky. Consider adding a null check before pushing to the array or handling the case where getProvider returns null.
import { Injectable, Inject, Logger } from "@nestjs/common";

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/config/app.config.ts Outdated
Comment thread src/deal/deal.service.ts
@silent-cipher silent-cipher merged commit a501c93 into main Sep 7, 2025
@github-project-automation github-project-automation Bot moved this from 📌 Triage to 🎉 Done in FOC Sep 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

3 participants