Feat: Add Spark connector

[rolznz]

spark connector

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	hasown@​2.0.0 ⏵ 2.0.2	+1		-6
	is-core-module@​2.13.0 ⏵ 2.16.1	+1		+1
	function-bind@​1.1.1 ⏵ 1.1.2			-1
	@​protobufjs/​inquire@​1.1.0
	@​protobufjs/​pool@​1.1.0
	bare-zlib@​1.3.1
	@​protobufjs/​fetch@​1.1.0
	@​protobufjs/​aspromise@​1.1.2
	bare-dns@​2.1.3
	@​types/​zen-observable@​0.8.3
	@​protobufjs/​path@​1.1.2
	module-details-from-path@​1.0.4
	@​protobufjs/​utf8@​1.1.0
	@​protobufjs/​base64@​1.1.2
	ts-proto-descriptors@​2.0.0
	@​protobufjs/​eventemitter@​1.1.0
	bare-form-data@​1.1.6
	bare-net@​2.0.1
	undici-types@​7.10.0
	zen-observable-ts@​1.1.0
	bare-fetch@​2.5.0
	bare-pipe@​4.0.7
	acorn-import-attributes@​1.9.5
	bare-events@​2.6.1
	light-bolt11-decoder@​3.2.0
	ts-error@​1.0.6
	dprint-node@​1.0.8
	fast-fifo@​1.3.2
	nice-grpc-common@​2.0.2
	isomorphic-ws@​5.0.0
	abort-controller-x@​0.4.3
	zen-observable@​0.8.15
	async-mutex@​0.5.0
See 64 more rows in the dashboard

View full report

[rolznz]

@pavanjoshi914 I think we should fix this before merging. The link to setup the keys is also broken because it shows steps to create a new master key.

I think it would be simpler to derive the nostr key too when generating the master key and extract the code so the same things happen when we generate a master key for Spark as what happens if you generate a master key normally.

CC @reneaaron @bumi

[rolznz]

I'm just wondering if this is the best approach since we add more complexity to these functions to both handle accounts and non-accounts.

But I can't see any other way except actually creating the account before we validate it 🤔

[pavanjoshi914]

creating account before validating will go totally opposite to the existing flows that we have for other connectors. instead its better to handle this at script level

it just generates key in two way. either getting mnemonic from account id or if its passed explicitly. i guess this is more better to do somehow

[rolznz]

@bumi @reneaaron we introduce something new here with the mnemonic generation before the account creation, and it creates a bit of complexity in the code. Not sure if this is OK for now or there is a better way we could do it?