fix(app.go): remove unused ipfsClient variable to improve code readab…

…ility and reduce complexity

fix(keeper.go): fix import path for curves package to match the correct package location
fix(vault.go): fix return type of NewController function to return a controller instance instead of an error

app/app.go

@@ -276,9 +276,6 @@ type ChainApp struct {
 	// module configurator
 	configurator module.Configurator
 	once         sync.Once
-
-	// network rails
-	ipfsClient ipfs.IPFSClient
 }
 
 // NewChainApp returns a reference to an initialized ChainApp.
@@ -312,6 +309,11 @@ func NewChainApp(
 	std.RegisterLegacyAminoCodec(legacyAmino)
 	std.RegisterInterfaces(interfaceRegistry)
 
+	ipfsClient, err := ipfs.NewLocalClient()
+	if err != nil {
+		panic(err)
+	}
+
 	// Below we could construct and set an application specific mempool and
 	// ABCI 1.0 PrepareProposal and ProcessProposal handlers. These defaults are
 	// already set in the SDK's BaseApp, this shows an example of how to override
@@ -624,6 +626,7 @@ func NewChainApp(
 		app.IBCKeeper.ChannelKeeper,
 		app.BankKeeper,
 		app.AccountKeeper,
+		ipfsClient,
 	)
 
 	// Create the svc Middleware Keeper

x/idx/keeper/keeper.go

@@ -1,7 +1,7 @@
 package keeper
 
 import (
-	"github.com/di-dao/crypto/core/curves"
+	"github.com/di-dao/common/crypto/core/curves"
 	"github.com/di-dao/sonr/pkg/ipfs"
 	"github.com/di-dao/sonr/x/idx/types"
 

x/idx/keeper/vault.go

@@ -11,20 +11,15 @@ type vaultStore struct {
 }
 
 // NewController creates a new controller instance.
-func (v vaultStore) NewController() error {
+func (v vaultStore) NewController() (controller, error) {
 	valKs, usrKs, err := generateKSS()
 	if err != nil {
-		return err
+		return controller{}, err
 	}
-	controller
-	return nil
-}
-
-func (v vaultStore) NewIPNSKey() error {
-	valKs, usrKs, err := generateKSS()
-	if err != nil {
-		return err
+	controller := controller{
+		valKS: valKs,
+		usrKS: usrKs,
 	}
 
-	return nil
+	return controller, nil
 }