chore: make the grumpy linter less grumpy

Author: simulot

app/cmd/upload/noui.go

@@ -19,13 +19,13 @@ func (upCmd *UpCmd) runNoUI(ctx context.Context, app *app.Application) error {
 	ctx, cancel := context.WithCancelCause(ctx)
 	lock := sync.RWMutex{}
 	needToResumeJobs := false
-	defer func() {
+	defer func(ctx context.Context) {
 		if needToResumeJobs {
 			// resume jobs if the UI was interrupted, the call context is already cancelled, so let's use a fresh one fpr this call
 			_ = upCmd.resumeJobs(context.Background(), app)
 			needToResumeJobs = false
 		}
-	}()
+	}(ctx)
 	defer cancel(nil)
 
 	var preparationDone atomic.Bool

app/cmd/upload/run.go

@@ -117,7 +117,6 @@ func (UpCmd *UpCmd) pauseJobs(ctx context.Context, app *app.Application) error {
 			if err != nil {
 				UpCmd.app.Jnl().Log().Error("Immich Job command sent", "pause", name, "err", err.Error())
 				return err
-
 			}
 			UpCmd.app.Jnl().Log().Info("Immich Job command sent", "pause", name)
 		}

app/cmd/upload/ui.go

@@ -55,13 +55,13 @@ func (upCmd *UpCmd) runUI(ctx context.Context, app *app.Application) error {
 	uiApp := tview.NewApplication()
 	ui := upCmd.newUI(ctx, app)
 
-	defer func() {
+	defer func(ctx context.Context) {
 		if needToResumeJobs {
 			// resume jobs if the UI was interrupted, the call context is already cancelled, so let's use a fresh one fpr this call
 			_ = upCmd.resumeJobs(context.Background(), app)
 			needToResumeJobs = false
 		}
-	}()
+	}(ctx)
 	defer cancel(nil)
 	pages := tview.NewPages()
 

immich/job.go

@@ -69,7 +69,7 @@ func (ic *ImmichClient) SendJobCommand(
 	command JobCommand,
 	force bool,
 ) (resp SendJobCommandResponse, err error) {
-	err = ic.newServerCall(ctx, EndPointSendJobCommand).do(putRequest("/jobs/"+string(jobID),
+	err = ic.newServerCall(ctx, EndPointSendJobCommand).do(putRequest("/jobs/"+jobID,
 		setJSONBody(struct {
 			Command JobCommand `json:"command"`
 			Force   bool       `json:"force"`