WIP: feat(site-builder): add permanent flag in site-builder publish command

[VLegakis]

site-builder delete command needs to be added.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
walrus-sites-sp	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 13, 2024 7:22pm
walrus-sites-sp-testnet	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 13, 2024 7:22pm
walrus-sites-sw	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 13, 2024 7:22pm
walrus-sites-sw-testnet	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 13, 2024 7:22pm

[codspeed-hq]

CodSpeed Performance Report

Merging #320 will not alter performance

_{Comparing SEINT-228-deletable-blobs (ac951ea) with main (f83a9c4)}

Summary

✅ 3 untouched benchmarks

[Tzal3x]

Welcome to the project @VLegakis!
Despite this being still a WIP, here is an early comment:
Regarding deletable sites in walrus, based on this discussion, we might need the opposite of what is implemented currently.
i.e. instead of publishing sites as deletable with a --deletable flag, we should always publish them as deletable by default, unless specified otherwise with --permanent.

Though they are not exactly permanent, as they will still get deleted when their epoch expires.

@giac-mysten do you think --only-expirable instead of --permanent would better depict the actual site lifetime being set?

[Tzal3x]

Also, please check out CONTRIBUTING.md and follow how to set your .editorconfig in order to automatically resolve lint errors like it's happening now.

[giac-mysten]

@giac-mysten do you think --only-expirable instead of --permanent would better depict the actual site lifetime being set?

@Tzal3x good question. "Permanent" is official Walrus terminology. When you walrus store <FILE>, the output says "Success: Permanent blob stored successfully."

So i'd say --permanent is fine.