Skip to content

feat(capabilities): add limit property to plan/get successful result - #645

Merged
volmedo merged 3 commits into
mainfrom
vic/feat/plan-get-limit
Jan 29, 2026
Merged

feat(capabilities): add limit property to plan/get successful result#645
volmedo merged 3 commits into
mainfrom
vic/feat/plan-get-limit

Conversation

@volmedo

@volmedo volmedo commented Jan 28, 2026

Copy link
Copy Markdown
Contributor

Ref. storacha/project-tracking#620

Add a limit property to plan/get successful result definition, as recently added to the spec (storacha/specs#150).

It is expressed in bytes, so it has been added as a string because we expect limits higher than 9PiBs in Forge.

@volmedo
volmedo requested a review from a team January 28, 2026 12:30
@volmedo volmedo self-assigned this Jan 28, 2026
@volmedo
volmedo requested a review from travis as a code owner January 28, 2026 12:30
@volmedo
volmedo merged commit d95199a into main Jan 29, 2026
1 check passed
@volmedo
volmedo deleted the vic/feat/plan-get-limit branch January 29, 2026 09:39
travis pushed a commit to storacha/w3infra that referenced this pull request Jan 30, 2026
Ref. storacha/project-tracking#620

Implements the new `limit` property for `plan/get` successful results,
as specified in storacha/specs#150 and defined
in storacha/upload-service#645.

`limit` is defined as a string to support limits over 9PiB. The result
of `planLimit` has been kept as a number because it is used in other
parts of the code for arithmetics. Those other call sites will be
refactored to use BigInt in a different PR. For now, the result of
`planLimit` is converted to string at the API boundary.
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