fix: in case if plugin is not implementing the configure ignore the failure (#112)

Author: nnicora

pkg/catalog/catalog.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"io"
 	"log/slog"
+	"strings"
 
 	"github.com/openkcm/plugin-sdk/api"
 )
@@ -136,7 +137,8 @@ func New(ctx context.Context, config Config, repo api.Repository, builtIns ...Bu
 			return nil, fmt.Errorf("failed to bind plugin %q: %w", pluginConfig.Name, err)
 		}
 
-		if pluginConfig.DataSource == nil {
+		externalYamlConfiguration := strings.TrimSpace(pluginConfig.YamlConfiguration)
+		if pluginConfig.DataSource == nil && len(externalYamlConfiguration) > 0 {
 			pluginConfig.DataSource = FixedData(pluginConfig.YamlConfiguration)
 		}
 

pkg/catalog/configure.go

@@ -4,7 +4,6 @@ import (
 	"context"
 	"crypto/sha512"
 	"encoding/hex"
-	"errors"
 	"fmt"
 	"io"
 	"log/slog"
@@ -82,8 +81,8 @@ func configurePlugin(ctx context.Context, pluginLog *slog.Logger, configurer Con
 		// The plugin doesn't support configuration and no data source was configured. Nothing to do.
 		return nil, nil
 	case configurer == nil && dataSource != nil:
-		// The plugin does not support configuration but a data source was configured. This is a failure.
-		return nil, errors.New("no supported configuration interface found")
+		// The plugin does not support configuration but a data source was configured. Nothing to do.
+		return nil, nil
 	case configurer != nil && dataSource == nil:
 		// The plugin supports configuration but no data source was configured. Default to an empty, fixed configuration.
 		dataSource = FixedData("")