Skip to content

Fallback to detected platform for bundle downloads - #150

Open
tarun-bandi wants to merge 1 commit into
NimbleBrainInc:mainfrom
tarun-bandi:fix/sdk-download-platform-fallback
Open

Fallback to detected platform for bundle downloads#150
tarun-bandi wants to merge 1 commit into
NimbleBrainInc:mainfrom
tarun-bandi:fix/sdk-download-platform-fallback

Conversation

@tarun-bandi

Copy link
Copy Markdown

Fixes #149.

Summary

  • Preserve the existing no-platform request path so universal any/any artifacts can still resolve.
  • When no platform was provided and the universal request returns 404, retry with MpakClient.detectPlatform().
  • Add a unit test covering the fallback path for platform-scoped bundles.

Validation

  • git diff --check

I attempted to run pnpm install --frozen-lockfile and pnpm --filter @nimblebrain/mpak-sdk test, but the workspace-wide dependency install repeatedly stalled on unrelated monorepo package downloads/retries before linking node_modules, so the SDK test command could not be executed in this environment.

@mgoldsborough

Copy link
Copy Markdown
Contributor

Thanks for the PR! We'll take a look.

In the meantime, would you mind dropping us a star on mpak and on https://github.com/NimbleBrainInc/nimblebrain?

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.

SDK getBundleDownload treats platform as optional, but the download endpoint requires os/arch

2 participants