sway icon indicating copy to clipboard operation
sway copied to clipboard

Add a forc node command

Open kayagokalp opened this issue 1 year ago • 6 comments

Currently fuel-core cli expects number of args to spin up a node, we want to explore ways to make it easier to do so probably by forc node beta-5 etc.

Couple of brain dumps to start discussion:

forc node --local forc node --memory forc node --testnet [beta-4 or latest] forc node --mainnet

kayagokalp avatar Mar 04 '24 23:03 kayagokalp

It would be great to leverage the [network] section of the manifest file to house some of these args.

forc ndoe could be confused with Node.js. Some other options might be forc core, forc client or forc api

sdankel avatar Mar 05 '24 23:03 sdankel

Just leaving a link to the docs showing how we currently expect users to do this. https://docs.fuel.network/guides/running-a-node/running-a-local-node/

JoshuaBatty avatar Mar 21 '24 05:03 JoshuaBatty

Also linking to the internal slack discussion about this. https://fuellabs.slack.com/archives/C031TTYJM60/p1709573694943729

JoshuaBatty avatar Mar 21 '24 05:03 JoshuaBatty

We should also make sure that with forc node we can easily have some default accounts funded. This can be done via providing a snapshot to fuel-core. This custom snapshot should fund some accounts we use by default which can be found in the forc-client/util/tx.rs named as DEFAULT_SIGNING_KEY IIRC. Also it would be nice to have way to specify a forc-wallet account index etc to fund user's own wallet in the local test node they spin up.

kayagokalp avatar Apr 11 '24 23:04 kayagokalp

I still think node is the correct terminology for this plugin. It's pretty common in the block chain space to want to connect to a 'node' in order to do the kind of operations this plugin will facilitate. I think it might be more confusing for users if this was named something else to avoid being associated with Node.js.

JoshuaBatty avatar Jun 17 '24 23:06 JoshuaBatty

I agree. It would mainly be backend TS/JS devs who might be confused by forc node, but they'd quickly see that we didn't repackage Node.js into a forc plugin 🙃

sdankel avatar Jun 24 '24 18:06 sdankel