Apply suggestions from code review

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Author: ChristopherHX

actionsdotnetactcompat/action_cache.go

@@ -40,9 +40,11 @@ func (cache *ActionCacheBase) GetTarArchive(ctx context.Context, cacheDir, sha,
 		defer func() {
 			if r := recover(); r != nil {
 				log.Printf("panic recovered: %v\n%s", r, debug.Stack())
+				_ = pw.CloseWithError(fmt.Errorf("panic recovered in GetTarArchive: %v", r))
+				return
 			}
+			_ = pw.Close()
 		}()
-		defer func() { _ = pw.Close() }()
 		writer := tar.NewWriter(pw)
 		defer func() { _ = writer.Close() }()
 		reader, err := os.Open(cache.mapping[cacheDir+"@"+sha])

protocol/logger/job_logger.go

@@ -338,8 +338,7 @@ func (logger *BufferedLiveLogger) SendLog(wrapper *protocol.TimelineRecordFeedLi
 		select {
 		case <-data.logdrain:
 			return errors.New("buffered live logger closing")
-		default:
-			data.logchan <- wrapper
+		case data.logchan <- wrapper:
 		}
 	} else {
 		logchan := make(chan *protocol.TimelineRecordFeedLinesWrapper, websocketPingSize)
@@ -350,7 +349,11 @@ func (logger *BufferedLiveLogger) SendLog(wrapper *protocol.TimelineRecordFeedLi
 			logfinished: logfinished,
 		}
 		if logger.data.CompareAndSwap(data, &ndata) {
-			ndata.logchan <- wrapper
+			select {
+			case <-ndata.logdrain:
+				return errors.New("buffered live logger closing")
+			case ndata.logchan <- wrapper:
+			}
 			go logger.sendLogs(logchan, ndata.logdrain, logfinished)
 		} else {
 			close(ndata.logchan)