Corrected grammer, simplifed the makefile so it installs only the main binary, removed expermintal-install option from the make file

Author: nman98

CHANGELOG.md

@@ -11,14 +11,14 @@ Mostly it has some bug fixes.
 
 ### Fixed
 
-- The indexer would go into the blocks data and store the signers from the last commit, which is actually all of the block signers from the previous block. So it woult insert it like it was meant for that block height. From now on the indexer will fetch data from the /commit method and insert it properly. [de40740](https://github.com/Cogwheel-Validator/spectra-gnoland-indexer/commit/de407407e67ae588141b361307fae18215f53a18)
-- Gnoland can indeed execute multiple message types in the same transaction. The indexer wasn't able to hold this data properly and would cause and error because in the postgres primary key was attached making each message type unique. Now each message type has a message_counter which is a smallint(int16) which is used as a index for that transaction. [d647901](https://github.com/Cogwheel-Validator/spectra-gnoland-indexer/commit/d64790181c626e2ac0137dd9cce08d10ec2b6a7c)
+- The indexer would go into the blocks data and store the signers from the last commit, which is actually all of the block signers from the previous block. So it would insert it like it was meant for that block height. From now on the indexer will fetch data from the /commit method and insert it properly. [de40740](https://github.com/Cogwheel-Validator/spectra-gnoland-indexer/commit/de407407e67ae588141b361307fae18215f53a18)
+- Gnoland can indeed execute multiple message types in the same transaction. The indexer wasn't able to hold this data properly and would cause an error because in the postgres primary key was attached making each message type unique. Now each message type has a message_counter which is a smallint(int16) which is used as a index for that transaction. [d647901](https://github.com/Cogwheel-Validator/spectra-gnoland-indexer/commit/d64790181c626e2ac0137dd9cce08d10ec2b6a7c)
 
 ### Changes
 
 - The REST API now returns a map of int16 and message data for the `/transaction/{tx_hash}/message` route. [74a35b30](https://github.com/Cogwheel-Validator/spectra-gnoland-indexer/commit/74a35b301070bb1dab5cbadec6fe64b16ad7eb3b)
 - The indexer should stop using sync.Map and use sync.Mutex with regular map to store the addresses before handing the operation to the AddressSolver function. [dde82a4](https://github.com/Cogwheel-Validator/spectra-gnoland-indexer/commit/dde82a4ecc9bce50963fbff3f4e13a3a20b47f9d)
-- The Orchestrator needs to make a fetch to the commit RPC method. This operation is done side by side with fetching the blocks method. A side effect of this is that indexer at that moment will fecth 2X times more request to the RPC, and if the Gnoland node has a limit on the amount of RPC clients that can use it, it might cause the indexer to slow down and throw errors on this requests. In future releases this should be improved. [d647901](https://github.com/Cogwheel-Validator/spectra-gnoland-indexer/commit/d64790181c626e2ac0137dd9cce08d10ec2b6a7c)
+- The Orchestrator needs to make a fetch to the commit RPC method. This operation is done side by side with fetching the blocks method. A side effect of this is that indexer at that moment will fetch 2 times more request to the RPC, and if the Gnoland node has a limit on the amount of RPC clients that can use it, it might cause the indexer to slow down and throw errors on this requests. In future releases this should be improved. [d647901](https://github.com/Cogwheel-Validator/spectra-gnoland-indexer/commit/d64790181c626e2ac0137dd9cce08d10ec2b6a7c)
 - Updated the go version to 1.25.4
 
 ## [0.3.0] - 2025-11-10

Makefile

@@ -13,10 +13,10 @@ VERSION := $(if $(GIT_TAG),$(GIT_TAG),$(GIT_BRANCH)-$(GIT_COMMIT))
 
 build:
 	mkdir -p build
-	go build -ldflags="-X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Commit=$(GIT_COMMIT) -X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Version=$(VERSION)" -o build/indexer indexer/main.go
+	go build -ldflags="-X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Commit=$(GIT_COMMIT) -X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Version=$(VERSION)" -o build/indexer indexer/indexer.go
 
 install:
-	go install -ldflags="-X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Commit=$(GIT_COMMIT) -X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Version=$(VERSION)" indexer/main.go
+	go install -ldflags="-X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Commit=$(GIT_COMMIT) -X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Version=$(VERSION)" indexer/indexer.go
 
 build-api:
 	mkdir -p build
@@ -28,12 +28,7 @@ clean:
 # experimental build with greentea garbage collection
 # use at your own risk
 build-experimental:
-	GOEXPERIMENT=greenteagc go build -ldflags="-X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Commit=$(GIT_COMMIT) -X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Version=$(VERSION)" -o build/indexer-tea indexer/main.go
-
-# experimental install with greentea garbage collection
-# use at your own risk
-install-experimental:
-	cd indexer && GOEXPERIMENT=greenteagc go install ./... -ldflags="-X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Commit=$(GIT_COMMIT) -X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Version=$(VERSION)"
+	GOEXPERIMENT=greenteagc go build -ldflags="-X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Commit=$(GIT_COMMIT) -X github.com/Cogwheel-Validator/spectra-gnoland-indexer/indexer/cmd.Version=$(VERSION)" -o build/indexer-tea indexer/indexer.go
 
 ########################################################
 # Test the indexer

api/main.go

@@ -153,7 +153,9 @@ var rootCmd = &cobra.Command{
 			w.Header().Set("Content-Type", "image/x-icon")
 			_, err := w.Write(favicon)
 			if err != nil {
-				log.Fatalf("failed to write favicon: %v", err)
+				log.Printf("failed to write favicon: %v", err)
+				http.Error(w, "Internal Server Error", http.StatusInternalServerError)
+				return
 			}
 		})
 

indexer/data_processor/operator.go

@@ -449,7 +449,7 @@ func (d *DataProcessor) insertDbMessageGroups(groups *decoder.DbMessageGroups) e
 
 	// Insert DbMsgSend messages with address IDs
 	if msgSendCount > 0 {
-		timeout := 10*time.Second + time.Duration(msgSendCount)
+		timeout := 10*time.Second + (time.Duration(msgSendCount) * time.Second / 5)
 		ctx, cancel := context.WithTimeout(context.Background(), timeout)
 		err := d.dbPool.InsertMsgSend(ctx, groups.MsgSend)
 		cancel()
@@ -464,7 +464,7 @@ func (d *DataProcessor) insertDbMessageGroups(groups *decoder.DbMessageGroups) e
 
 	// Insert DbMsgCall messages with address IDs
 	if msgCallCount > 0 {
-		timeout := 10*time.Second + time.Duration(msgCallCount)
+		timeout := 10*time.Second + (time.Duration(msgCallCount) * time.Second / 5)
 		ctx, cancel := context.WithTimeout(context.Background(), timeout)
 		err := d.dbPool.InsertMsgCall(ctx, groups.MsgCall)
 		cancel()
@@ -479,7 +479,7 @@ func (d *DataProcessor) insertDbMessageGroups(groups *decoder.DbMessageGroups) e
 
 	// Insert DbMsgAddPackage messages with address IDs
 	if msgAddPkgCount > 0 {
-		timeout := 10*time.Second + time.Duration(msgAddPkgCount)
+		timeout := 10*time.Second + (time.Duration(msgAddPkgCount) * time.Second / 5)
 		ctx, cancel := context.WithTimeout(context.Background(), timeout)
 		err := d.dbPool.InsertMsgAddPackage(ctx, groups.MsgAddPkg)
 		cancel()
@@ -494,7 +494,7 @@ func (d *DataProcessor) insertDbMessageGroups(groups *decoder.DbMessageGroups) e
 
 	// Insert DbMsgRun messages with address IDs
 	if msgRunCount > 0 {
-		timeout := 10*time.Second + time.Duration(msgRunCount)
+		timeout := 10*time.Second + (time.Duration(msgRunCount) * time.Second / 5)
 		ctx, cancel := context.WithTimeout(context.Background(), timeout)
 		err := d.dbPool.InsertMsgRun(ctx, groups.MsgRun)
 		cancel()

indexer/main.go indexer/indexer.goindexer/main.go renamed to indexer/indexer.go

@@ -27,15 +27,19 @@ func TestAuthenticGeneration(t *testing.T) {
 
 	// Test key pair access
 	for range 500 {
-		generator.GetRandomKeyPair()
+		kp := generator.GetRandomKeyPair()
+		assert.NotNil(t, kp)
+		assert.NotEmpty(t, kp.Address)
+		assert.NotEmpty(t, kp.PublicKey)
 	}
 
 	// Test synthetic transaction generation 500 times
 	for range 500 {
 		events, tx := generator.GenerateTransaction()
 		assert.NotNil(t, tx)
-		assert.NotNil(t, events)
+		assert.NotNil(t, events.Events)
 	}
+
 }
 
 // RunAllTests runs all the test functions