Skip to content

fix: cap postage stamp calculator depth at 34#810

Merged
EchinopsisM merged 1 commit into
masterfrom
fix/calculator-depth-cap
May 27, 2026
Merged

fix: cap postage stamp calculator depth at 34#810
EchinopsisM merged 1 commit into
masterfrom
fix/calculator-depth-cap

Conversation

@EchinopsisM
Copy link
Copy Markdown
Collaborator

Summary

  • Removes depths 35-41 from effective volume tables in AmountAndDepthCalc.js and VolumeAndDurationCalc.js
  • Updates depth search loops, validation, placeholder text, and error messages to reflect the 17-34 range
  • Depth 34 is the practical upper limit: it matches bee-js effectiveSizeBreakpoints / capacityBreakpoints, and depths above 34 exceed realistic network storage capacity

Test plan

  • Open the postage stamp calculators and confirm depth inputs above 34 are rejected with the updated error message
  • Confirm volume lookups for all erasure/encryption combinations still resolve correctly up to depth 34
  • Confirm no regressions in the amount/depth and volume/duration calculators

Generated with Claude Code

- Remove depths 35-41 from effective volume tables in AmountAndDepthCalc.js
  and VolumeAndDurationCalc.js (10 encryption×erasure blocks each)
- Update depth search loops from d<=41 to d<=34
- Update depth input validation from >41 to >34 in VolumeAndDurationCalc
- Update placeholder text and error messages to reflect 17-34 range
- Depth 34 is the practical upper limit consistent with bee-js
  effectiveSizeBreakpoints; values match bee-js capacityBreakpoints exactly

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 27, 2026

Deploy Preview for test-twitter-preview-testing-3 ready!

Name Link
🔨 Latest commit c3880be
🔍 Latest deploy log https://app.netlify.com/projects/test-twitter-preview-testing-3/deploys/6a173f6c2c93f8000865ff4b
😎 Deploy Preview https://deploy-preview-810--test-twitter-preview-testing-3.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@EchinopsisM EchinopsisM merged commit c12a619 into master May 27, 2026
9 checks passed
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.

1 participant