fix: address Gemini PR review feedback on reimport handler and CLI

- Use r.Body != nil && r.Body != http.NoBody instead of r.ContentLength
  for chunked/HTTP2 compatibility in reimport handler
- Add nil check for harness config after GetHarnessConfig lookup
- Use cmd.Context() instead of context.Background() in update command
- Add nil checks for resp after Hub List calls in both update functions

Author: Scion Agent (harness-journey-inv)

cmd/harness_config_update.go

@@ -72,7 +72,7 @@ func runHarnessConfigUpdate(cmd *cobra.Command, args []string) error {
 
 	PrintUsingHub(hubCtx.Endpoint)
 
-	ctx, cancel := context.WithTimeout(context.Background(), 120*time.Second)
+	ctx, cancel := context.WithTimeout(cmd.Context(), 120*time.Second)
 	defer cancel()
 
 	if all {
@@ -91,6 +91,9 @@ func updateSingleHarnessConfig(ctx context.Context, hubCtx *HubContext, name, ur
 	if err != nil {
 		return fmt.Errorf("failed to search Hub: %w", err)
 	}
+	if resp == nil {
+		return fmt.Errorf("harness-config %q not found on Hub", name)
+	}
 
 	var match *hubclient.HarnessConfig
 	for i := range resp.HarnessConfigs {
@@ -144,6 +147,9 @@ func updateAllHarnessConfigs(ctx context.Context, hubCtx *HubContext) error {
 	if err != nil {
 		return fmt.Errorf("failed to list harness-configs: %w", err)
 	}
+	if resp == nil {
+		return fmt.Errorf("no harness-configs found")
+	}
 
 	var updated, skipped, failed int
 	jsonOut := isJSONOutput()

pkg/hub/harness_config_handlers.go

@@ -719,9 +719,13 @@ func (s *Server) handleHarnessConfigReimport(w http.ResponseWriter, r *http.Requ
 		writeErrorFromErr(w, err, "")
 		return
 	}
+	if hc == nil {
+		NotFound(w, "HarnessConfig")
+		return
+	}
 
 	var req ReimportHarnessConfigRequest
-	if r.ContentLength > 0 {
+	if r.Body != nil && r.Body != http.NoBody {
 		if err := readJSON(r, &req); err != nil {
 			BadRequest(w, "Invalid request body: "+err.Error())
 			return