fix: small correctness and quality improvements

- fix(webrpc): discard unused history row id in ClusterTaskHistory scan
  The SQL query selects both 'id' and 'task_id', but both were scanned
  into TaskID — the first value was silently overwritten. Use a discard
  target for the unused column.

- fix(alertmanager): prevent duplicate alert output for sealing tasks
  Sealing tasks with >5 failures were printed twice: once for being a
  sealing task and once for exceeding the threshold. Use else-if so
  only one branch fires.

- fix(indexing): fix 'failed to to insert' double-word typos
  In both task_ipni.go and task_pdp_ipni.go error messages.

- fix(api): add missing Content-Type headers on JSON responses
  getSchema, getLayers (config API) and getSectors (sector API) were
  missing Content-Type: application/json — other handlers in the same
  files already set it.

- fix(cli): 'Command separated' -> 'Comma-separated' in --db-host help
  Typo in CLI flag description and translation catalog.

Author: Reiers

alertmanager/alerts.go

@@ -210,8 +210,7 @@ func taskFailureCheck(al *alerts) {
 	for name, count := range tmap {
 		if contains(sealingTasks, name) {
 			al.alertMap[Name].alertString += fmt.Sprintf("Task: %s, Failures: %d. ", name, count)
-		}
-		if count > 5 {
+		} else if count > 5 {
 			al.alertMap[Name].alertString += fmt.Sprintf("Task: %s, Failures: %d. ", name, count)
 		}
 	}

cmd/curio/internal/translations/catalog.go

@@ -128,13 +128,13 @@ func main() {
 			&cli.StringFlag{
 				Name:    "db-host",
 				EnvVars: []string{"CURIO_DB_HOST", "CURIO_HARMONYDB_HOSTS"},
-				Usage:   translations.T("Command separated list of hostnames for yugabyte cluster"),
+				Usage:   translations.T("Comma-separated list of hostnames for yugabyte cluster"),
 				Value:   "127.0.0.1",
 			},
 			&cli.StringFlag{
 				Name:        "db-host-cql",
 				EnvVars:     []string{"CURIO_DB_HOST_CQL"},
-				Usage:       translations.T("Command separated list of hostnames for yugabyte cluster"),
+				Usage:       translations.T("Comma-separated list of hostnames for yugabyte cluster"),
 				Value:       "",
 				DefaultText: "<--db-host>",
 			},

cmd/curio/main.go

@@ -562,7 +562,7 @@ func (I *IPNITask) schedule(ctx context.Context, taskFunc harmonytask.AddTaskFun
 
 				n, err := tx.Exec(`INSERT INTO ipni_peerid (sp_id, priv_key, peer_id) VALUES ($1, $2, $3) ON CONFLICT(sp_id) DO NOTHING `, p.SpID, privKey, pid.String())
 				if err != nil {
-					return false, xerrors.Errorf("failed to to insert the key into DB: %w", err)
+					return false, xerrors.Errorf("failed to insert the key into DB: %w", err)
 				}
 
 				if n == 0 {

tasks/indexing/task_ipni.go

@@ -542,7 +542,7 @@ func (P *PDPIPNITask) schedule(ctx context.Context, taskFunc harmonytask.AddTask
 
 				n, err := tx.Exec(`INSERT INTO ipni_peerid (sp_id, priv_key, peer_id) VALUES ($1, $2, $3) ON CONFLICT(sp_id) DO NOTHING `, -1, privKey, pid.String())
 				if err != nil {
-					return false, xerrors.Errorf("failed to to insert the key into DB: %w", err)
+					return false, xerrors.Errorf("failed to insert the key into DB: %w", err)
 				}
 
 				if n == 0 {

tasks/indexing/task_pdp_ipni.go

@@ -157,12 +157,14 @@ func getSch(w http.ResponseWriter, r *http.Request) {
 	}
 	allOpt(sch)
 
+	w.Header().Set("Content-Type", "application/json")
 	apihelper.OrHTTPFail(w, json.NewEncoder(w).Encode(sch))
 }
 
 func (c *cfg) getLayers(w http.ResponseWriter, r *http.Request) {
 	var layers []string
 	apihelper.OrHTTPFail(w, c.DB.Select(context.Background(), &layers, `SELECT title FROM harmony_config ORDER BY title`))
+	w.Header().Set("Content-Type", "application/json")
 	apihelper.OrHTTPFail(w, json.NewEncoder(w).Encode(layers))
 }
 

web/api/config/config.go

@@ -333,6 +333,7 @@ func (c *cfg) getSectors(w http.ResponseWriter, r *http.Request) {
 			sectors[i].Deals = fmt.Sprintf("Market: %d, DDO: %d", f05, ddo)
 		}
 	}
+	w.Header().Set("Content-Type", "application/json")
 	apihelper.OrHTTPFail(w, json.NewEncoder(w).Encode(map[string]any{"data": sectors}))
 }
 

web/api/sector/sector.go

@@ -118,7 +118,7 @@ func (a *WebRPC) ClusterTaskHistory(ctx context.Context, limit, offset int) ([]T
 		var t TaskHistorySummary
 		var posted, start, end time.Time
 
-		if err := rows.Scan(&t.TaskID, &t.Name, &t.TaskID, &posted, &start, &end, &t.Result, &t.Err, &t.CompletedBy); err != nil {
+		if err := rows.Scan(new(int64), &t.Name, &t.TaskID, &posted, &start, &end, &t.Result, &t.Err, &t.CompletedBy); err != nil {
 			return nil, err // Handle error
 		}