fix: Windows build — use logging.Logger in terminal manager, share testLogger helper

The Windows-specific terminal manager still referenced *zap.SugaredLogger
after the non-Windows variant was updated to the SDK logging interface.
Also extracted testLogger into a shared helpers_test.go so plugin tests
compile on all platforms.

Author: joshuapare

backend/pkg/plugin/helpers_test.go

@@ -0,0 +1,12 @@
+package plugin
+
+import (
+	"testing"
+
+	logging "github.com/omniviewdev/plugin-sdk/log"
+)
+
+func testLogger(t *testing.T) logging.Logger {
+	t.Helper()
+	return logging.NewNop()
+}

backend/pkg/plugin/pids_test.go

@@ -11,14 +11,8 @@ import (
 
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
-	logging "github.com/omniviewdev/plugin-sdk/log"
 )
 
-func testLogger(t *testing.T) logging.Logger {
-	t.Helper()
-	return logging.NewNop()
-}
-
 func TestPluginPIDTracker_RecordAndRemove(t *testing.T) {
 	tracker := NewPluginPIDTracker()
 

backend/pkg/terminal/manager_windows.go

@@ -7,8 +7,7 @@ import (
 	"os"
 	"sync"
 
-	"go.uber.org/zap"
-
+	logging "github.com/omniviewdev/plugin-sdk/log"
 	sdkexec "github.com/omniviewdev/plugin-sdk/pkg/v1/exec"
 	"github.com/omniviewdev/plugin-sdk/pkg/types"
 )
@@ -18,7 +17,7 @@ var errUnsupported = fmt.Errorf("terminal sessions are not supported on Windows"
 // Manager manages terminal sessions. On Windows, PTY-based sessions are not
 // supported, so all operations return errors.
 type Manager struct {
-	log      *zap.SugaredLogger
+	log      logging.Logger
 	sessions map[string]*sdkexec.Session
 	ptys     map[string]*os.File
 
@@ -29,7 +28,7 @@ type Manager struct {
 }
 
 func NewManager(
-	log *zap.SugaredLogger,
+	log logging.Logger,
 ) (*Manager, chan sdkexec.StreamInput, chan sdkexec.StreamOutput, chan sdkexec.StreamResize) {
 	inMux := make(chan sdkexec.StreamInput)
 	outMux := make(chan sdkexec.StreamOutput)