Skip to content

Conversation

@bajtos
Copy link
Contributor

@bajtos bajtos commented Jan 26, 2026

Summary

  • Add filbeam.retrievalDomainName property to the Chain interface
  • Configure FilBeam domains for mainnet (filbeam.io), calibration (calibration.filbeam.io), and devnet (localhost:8080)
  • Update createPieceUrl to use chain configuration instead of hardcoded chainId check
  • Update all callers to pass Chain instead of chainId

Test plan

  • All 351 existing tests pass
  • New tests for createPieceUrl covering all three chains
  • Build passes
  • Linter passes

@github-project-automation github-project-automation bot moved this to 📌 Triage in FOC Jan 26, 2026
Replace hardcoded chainId check in createPieceUrl with chain configuration lookup. Add filbeam.retrievalDomainName to Chain interface and configure it for mainnet, calibration, and devnet.

Co-Authored-By: Claude Code <noreply@anthropic.com>

Signed-off-by: Miroslav Bajtoš <oss@bajtos.net>
@bajtos bajtos force-pushed the filbeam-chain-config branch from b95801b to 39f49e0 Compare January 26, 2026 14:13
@bajtos bajtos moved this from 📌 Triage to 🔎 Awaiting review in FOC Jan 26, 2026
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 26, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
synapse-dev 0551853 Jan 27 2026, 01:11 PM

add ENDORSEMENTS_ADDRESS env config to utils/example-storage-e2e.js
@rvagg
Copy link
Collaborator

rvagg commented Jan 27, 2026

seems good to me, although I hope that devnet address isn't important for the standard path because so many things want to listen to 8080 there's a good chance you'll hit something existing on dev machines

@bajtos
Copy link
Contributor Author

bajtos commented Jan 27, 2026

seems good to me, although I hope that devnet address isn't important for the standard path because so many things want to listen to 8080 there's a good chance you'll hit something existing on dev machines

Yeah, I share your concern.

Actually, FilBeam does not support local devnet. It would be nice to throw a helpful error when trying to retrieve a devnet piece via FilBeam, but it's not clear to me how to achieve that and I feel this is not worth our effort right now.

@rvagg
Copy link
Collaborator

rvagg commented Jan 27, 2026

there is the filbeam retriever in synapse-sdk, maybe it should be pulling the base address from the chain? in that case, a better port number might be advisable because I think we include it in the default retrieval path

@bajtos
Copy link
Contributor Author

bajtos commented Jan 27, 2026

How do you feel about using filbeam.localhost:8080 for devnet? It won't work out of the box, but it's possible to configure your machine to make that hostname work, plus the hostname is IMO descriptive enough to make it clear what's going on when somebody tries FilBeam retrievals for devnet pieces.

Copy link
Member

@hugomrdias hugomrdias left a comment

Choose a reason for hiding this comment

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

left a suggestion for the prop name

}
}
filbeam: {
retrievalDomainName: string
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
retrievalDomainName: string
retrievalDomain: string

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Renamed in daa9035

@github-project-automation github-project-automation bot moved this from 🔎 Awaiting review to ✔️ Approved by reviewer in FOC Jan 27, 2026
@bajtos bajtos merged commit 3f9f831 into hugomrdias/payments Jan 27, 2026
2 of 3 checks passed
@bajtos bajtos deleted the filbeam-chain-config branch January 27, 2026 13:09
@github-project-automation github-project-automation bot moved this from ✔️ Approved by reviewer to 🎉 Done in FOC Jan 27, 2026
@bajtos
Copy link
Contributor Author

bajtos commented Jan 27, 2026

there is the filbeam retriever in synapse-sdk, maybe it should be pulling the base address from the chain? in that case, a better port number might be advisable because I think we include it in the default retrieval path

I will follow-up with another PR

@bajtos
Copy link
Contributor Author

bajtos commented Jan 27, 2026

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.

6 participants