fix: change according jf's review comment

Signed-off-by: Daniel Hu <[email protected]>

Author: daniel-hutao

internal/pkg/develop/plugin/template/NAME.go

@@ -5,8 +5,6 @@ var NameGoDirTpl = "internal/pkg/plugin/[[ .Name | dirFormat ]]/"
 var NameGoMustExistFlag = true
 var NameGoContentTpl = `package [[ .Name | format ]]
 
-const Name = "[[ .Name ]]"
-
 // TODO(dtm): Add your logic here.
 `
 

internal/pkg/develop/plugin/template/create.go

@@ -6,18 +6,12 @@ var createGoContentTpl = `package [[ .Name | format ]]
 
 import (
 	"github.com/devstream-io/devstream/internal/pkg/configmanager"
-	. "github.com/devstream-io/devstream/internal/pkg/plugin/common"
 	"github.com/devstream-io/devstream/internal/pkg/plugin/installer"
 	"github.com/devstream-io/devstream/internal/pkg/statemanager"
 	"github.com/devstream-io/devstream/pkg/util/log"
 )
 
 func Create(options configmanager.RawOptions) (statemanager.ResourceStatus, error) {
-	var err error
-	defer func() {
-		HandleErrLogsWithPlugin(err, Name)
-	}()
-
 	// Initialize Operator with Operations
 	operator := &installer.Operator{
 		PreExecuteOperations: installer.PreExecuteOperations{

internal/pkg/develop/plugin/template/delete.go

@@ -6,16 +6,10 @@ var deleteGoContentTpl = `package [[ .Name | format ]]
 
 import (
 	"github.com/devstream-io/devstream/internal/pkg/configmanager"
-	. "github.com/devstream-io/devstream/internal/pkg/plugin/common"
 	"github.com/devstream-io/devstream/internal/pkg/plugin/installer"
 )
 
 func Delete(options configmanager.RawOptions) (bool, error) {
-	var err error
-	defer func() {
-		HandleErrLogsWithPlugin(err, Name)
-	}()
-
 	// Initialize Operator with Operations
 	operator := &installer.Operator{
 		PreExecuteOperations: installer.PreExecuteOperations{
@@ -27,7 +21,7 @@ func Delete(options configmanager.RawOptions) (bool, error) {
 	}
 
 	// Execute all Operations in Operator
-	_, err = operator.Execute(options)
+	_, err := operator.Execute(options)
 	if err != nil {
 		return false, err
 	}

internal/pkg/develop/plugin/template/read.go

@@ -6,18 +6,12 @@ var readGoContentTpl = `package [[ .Name | format ]]
 
 import (
 	"github.com/devstream-io/devstream/internal/pkg/configmanager"
-	. "github.com/devstream-io/devstream/internal/pkg/plugin/common"
 	"github.com/devstream-io/devstream/internal/pkg/plugin/installer"
 	"github.com/devstream-io/devstream/internal/pkg/statemanager"
 	"github.com/devstream-io/devstream/pkg/util/log"
 )
 
 func Read(options configmanager.RawOptions) (statemanager.ResourceStatus, error) {
-	var err error
-	defer func() {
-		HandleErrLogsWithPlugin(err, Name)
-	}()
-
 	// Initialize Operator with Operations
 	operator := &installer.Operator{
 		PreExecuteOperations: installer.PreExecuteOperations{

internal/pkg/develop/plugin/template/update.go

@@ -6,18 +6,12 @@ var updateGoContentTpl = `package [[ .Name | format ]]
 
 import (
 	"github.com/devstream-io/devstream/internal/pkg/configmanager"
-	. "github.com/devstream-io/devstream/internal/pkg/plugin/common"
 	"github.com/devstream-io/devstream/internal/pkg/plugin/installer"
 	"github.com/devstream-io/devstream/internal/pkg/statemanager"
 	"github.com/devstream-io/devstream/pkg/util/log"
 )
 
 func Update(options configmanager.RawOptions) (statemanager.ResourceStatus, error) {
-	var err error
-	defer func() {
-		HandleErrLogsWithPlugin(err, Name)
-	}()
-
 	// Initialize Operator with Operations
 	operator := &installer.Operator{
 		PreExecuteOperations: installer.PreExecuteOperations{

internal/pkg/plugin/argocdapp/argocdapp.go

@@ -4,7 +4,5 @@ import (
 	_ "embed"
 )
 
-const Name = "argocdapp"
-
 //go:embed tpl/argocd.tpl.yaml
 var templateFileLoc string

internal/pkg/plugin/argocdapp/create.go

@@ -2,7 +2,6 @@ package argocdapp
 
 import (
 	"github.com/devstream-io/devstream/internal/pkg/configmanager"
-	. "github.com/devstream-io/devstream/internal/pkg/plugin/common"
 	"github.com/devstream-io/devstream/internal/pkg/plugin/installer"
 	"github.com/devstream-io/devstream/internal/pkg/plugin/installer/kubectl"
 	"github.com/devstream-io/devstream/internal/pkg/statemanager"
@@ -12,11 +11,6 @@ import (
 
 // Create creates an ArgoCD app YAML and applies it.
 func Create(options configmanager.RawOptions) (statemanager.ResourceStatus, error) {
-	var err error
-	defer func() {
-		HandleErrLogsWithPlugin(err, Name)
-	}()
-
 	// Initialize Operator with Operations
 	operator := &installer.Operator{
 		PreExecuteOperations: installer.PreExecuteOperations{

internal/pkg/plugin/argocdapp/delete.go

@@ -2,18 +2,12 @@ package argocdapp
 
 import (
 	"github.com/devstream-io/devstream/internal/pkg/configmanager"
-	. "github.com/devstream-io/devstream/internal/pkg/plugin/common"
 	"github.com/devstream-io/devstream/internal/pkg/plugin/installer"
 	"github.com/devstream-io/devstream/internal/pkg/plugin/installer/kubectl"
 	kubectlUtil "github.com/devstream-io/devstream/pkg/util/kubectl"
 )
 
 func Delete(options configmanager.RawOptions) (bool, error) {
-	var err error
-	defer func() {
-		HandleErrLogsWithPlugin(err, Name)
-	}()
-
 	// Initialize Operator with Operations
 	operator := &installer.Operator{
 		PreExecuteOperations: installer.PreExecuteOperations{
@@ -25,7 +19,7 @@ func Delete(options configmanager.RawOptions) (bool, error) {
 	}
 
 	// Execute all Operations in Operator
-	_, err = operator.Execute(options)
+	_, err := operator.Execute(options)
 	if err != nil {
 		return false, err
 	}

internal/pkg/plugin/argocdapp/read.go

@@ -2,17 +2,11 @@ package argocdapp
 
 import (
 	"github.com/devstream-io/devstream/internal/pkg/configmanager"
-	. "github.com/devstream-io/devstream/internal/pkg/plugin/common"
 	"github.com/devstream-io/devstream/internal/pkg/plugin/installer"
 	"github.com/devstream-io/devstream/internal/pkg/statemanager"
 )
 
 func Read(options configmanager.RawOptions) (statemanager.ResourceStatus, error) {
-	var err error
-	defer func() {
-		HandleErrLogsWithPlugin(err, Name)
-	}()
-
 	// Initialize Operator with Operations
 	operator := &installer.Operator{
 		PreExecuteOperations: installer.PreExecuteOperations{

internal/pkg/plugin/cigeneric/cigeneric.go

@@ -1,3 +1 @@
 package cigeneric
-
-const Name = "ci-generic"