[WIP] Fix lint Go issues (#8705)

Author: Copilot

pkg/workflow/compiler_safe_outputs_core.go

@@ -959,7 +959,7 @@ func (c *Compiler) addHandlerManagerConfigEnvVar(steps *[]string, data *Workflow
 		if cfg.Target != "" {
 			handlerConfig["target"] = cfg.Target
 		}
-		if cfg.RequiredLabels != nil && len(cfg.RequiredLabels) > 0 {
+		if len(cfg.RequiredLabels) > 0 {
 			handlerConfig["required_labels"] = cfg.RequiredLabels
 		}
 		if cfg.RequiredTitlePrefix != "" {
@@ -974,7 +974,7 @@ func (c *Compiler) addHandlerManagerConfigEnvVar(steps *[]string, data *Workflow
 		if cfg.Max > 0 {
 			handlerConfig["max"] = cfg.Max
 		}
-		if cfg.AllowedReasons != nil && len(cfg.AllowedReasons) > 0 {
+		if len(cfg.AllowedReasons) > 0 {
 			handlerConfig["allowed_reasons"] = cfg.AllowedReasons
 		}
 		config["hide_comment"] = handlerConfig

pkg/workflow/compiler_safe_outputs_prs.go

@@ -1,11 +1,5 @@
 package workflow
 
-import (
-	"github.com/githubnext/gh-aw/pkg/logger"
-)
-
-var prSafeOutputsLog = logger.New("workflow:compiler_safe_outputs_prs")
-
 // buildMarkPullRequestAsReadyForReviewStepConfig builds the configuration for marking a PR as ready for review
 func (c *Compiler) buildMarkPullRequestAsReadyForReviewStepConfig(data *WorkflowData, mainJobName string, threatDetectionEnabled bool) SafeOutputStepConfig {
 	cfg := data.SafeOutputs.MarkPullRequestAsReadyForReview

pkg/workflow/js.go

@@ -27,8 +27,6 @@ func getAddReviewerScript() string                     { return "" }
 func getAssignMilestoneScript() string                 { return "" }
 func getAssignToAgentScript() string                   { return "" }
 func getAssignToUserScript() string                    { return "" }
-func getCloseDiscussionScript() string                 { return "" }
-func getCloseIssueScript() string                      { return "" }
 func getMarkPullRequestAsReadyForReviewScript() string { return "" }
 func getCreateCodeScanningAlertScript() string         { return "" }
 func getCreateDiscussionScript() string                { return "" }