fix: replace panic in gitextractor with error handling (#8386)

* replace panic in gitextractor with error handling.

* Fix subTaskCtx.GetLogger().Error arguments

---------

Co-authored-by: Klesh Wong <[email protected]>

Author: galachad

backend/plugins/gitextractor/tasks/repo_cloner.go

@@ -39,7 +39,8 @@ var CloneGitRepoMeta = plugin.SubTaskMeta{
 func CloneGitRepo(subTaskCtx plugin.SubTaskContext) errors.Error {
 	taskData, ok := subTaskCtx.GetData().(*parser.GitExtractorTaskData)
 	if !ok {
-		panic("git repo reference not found on context")
+		subTaskCtx.GetLogger().Error(nil, "git repo reference not found on context")
+		return errors.Default.New("git repo reference not found on context")
 	}
 	op := taskData.Options
 	storage := store.NewDatabase(subTaskCtx, op.RepoId)

backend/plugins/gitextractor/tasks/repo_collector.go

@@ -84,10 +84,12 @@ func CollectGitDiffLines(subTaskCtx plugin.SubTaskContext) errors.Error {
 func getGitRepo(subTaskCtx plugin.SubTaskContext) parser.RepoCollector {
 	taskData, ok := subTaskCtx.GetData().(*parser.GitExtractorTaskData)
 	if !ok {
-		panic("git repo reference not found on context")
+		subTaskCtx.GetLogger().Error(nil, "git repo reference not found on context")
+		return nil
 	}
 	if taskData.GitRepo == nil {
-		panic("git repo is empty, please check subtask: clone repo")
+		subTaskCtx.GetLogger().Error(nil, "git repo is empty, skipping Collect Commits subtask")
+		return nil
 	}
 	return taskData.GitRepo
 }