Remove unused/dead code from cli

cmd/config.go

@@ -407,52 +407,3 @@ func pathContains(root, candidate string) bool {
 	}
 	return rel == "." || (rel != ".." && !strings.HasPrefix(rel, ".."+string(filepath.Separator)))
 }
-
-// removeProjectConfig removes a project from the configuration
-// Keeping for future use when we add a 'remove' or 'clean' command
-//
-//nolint:unused // kept for future 'orchcli remove' command implementation
-func removeProjectConfig(projectPath string) error {
-	configPath, err := GetConfigPath()
-	if err != nil {
-		return err
-	}
-
-	// Use file locking for concurrent access
-	lockPath := configPath + ".lock"
-	fileLock := flock.New(lockPath)
-
-	// Try to acquire lock
-	err = fileLock.Lock()
-	if err != nil {
-		return fmt.Errorf("failed to acquire config lock: %w", err)
-	}
-	defer func() {
-		_ = fileLock.Unlock()
-	}()
-
-	// Load current config
-	config, err := LoadConfig()
-	if err != nil {
-		return err
-	}
-
-	// Remove project
-	delete(config.Projects, projectPath)
-
-	if config.CurrentProject == projectPath {
-		config.CurrentProject = ""
-	}
-
-	// Marshal and save
-	data, err := json.MarshalIndent(config, "", "  ")
-	if err != nil {
-		return fmt.Errorf("failed to marshal config: %w", err)
-	}
-
-	if err := writeFileAtomically(configPath, data, configFilePerm); err != nil {
-		return fmt.Errorf("failed to write config: %w", err)
-	}
-
-	return nil
-}

package.json

@@ -2,13 +2,11 @@
   "name": "@kubeorch/cli",
   "version": "0.0.3",
   "description": "OrchCLI - KubeOrch Developer CLI",
-  "main": "npm/index.js",
   "bin": {
     "orchcli": "./npm/bin/orchcli"
   },
   "scripts": {
     "postinstall": "node npm/scripts/postinstall.js",
-    "prepack": "node npm/scripts/prepack.js",
     "test": "echo \"Tests are run via Go test suite\" && exit 0"
   },
   "repository": {

tests/helpers/test_helpers.go

@@ -2,7 +2,6 @@
 package helpers
 
 import (
-	"io"
 	"os"
 	"path/filepath"
 	"runtime"
@@ -61,32 +60,6 @@ func (h *TestHelper) ReadFile(path string) string {
 	return string(content)
 }
 
-// CaptureOutput captures stdout and stderr
-func CaptureOutput(f func()) (string, string, error) {
-	oldStdout := os.Stdout
-	oldStderr := os.Stderr
-
-	rOut, wOut, _ := os.Pipe()
-	rErr, wErr, _ := os.Pipe()
-
-	os.Stdout = wOut
-	os.Stderr = wErr
-
-	// Run the function
-	f()
-
-	// Restore
-	wOut.Close()
-	wErr.Close()
-	os.Stdout = oldStdout
-	os.Stderr = oldStderr
-
-	stdout, _ := io.ReadAll(rOut)
-	stderr, _ := io.ReadAll(rErr)
-
-	return string(stdout), string(stderr), nil
-}
-
 // Cleanup performs cleanup operations
 func (h *TestHelper) Cleanup() {
 	if h.TempDir != "" {

tests/unit/utils_test.go

@@ -96,53 +96,6 @@ func (suite *UtilsTestSuite) TestGetComposeFile() {
 	}
 }
 
-func (suite *UtilsTestSuite) TestValidateDockerCompose() {
-	// Test Docker Compose validation
-	// This would mock the docker compose check
-
-	// Mock successful validation
-	mockCmd := func() error {
-		return nil
-	}
-
-	err := mockCmd()
-	assert.NoError(suite.T(), err)
-
-	// Mock failed validation
-	mockFailCmd := func() error {
-		return assert.AnError
-	}
-
-	err = mockFailCmd()
-	assert.Error(suite.T(), err)
-}
-
-func (suite *UtilsTestSuite) TestJoinArgs() {
-	// Test argument joining for shell commands
-	testCases := []struct {
-		expected string
-		args     []string
-	}{
-		{
-			expected: "arg1 arg2",
-			args:     []string{"arg1", "arg2"},
-		},
-		{
-			expected: "'arg with spaces' arg2",
-			args:     []string{"arg with spaces", "arg2"},
-		},
-		{
-			expected: "'arg'\"'\"'with'\"'\"'quotes' arg2",
-			args:     []string{"arg'with'quotes", "arg2"},
-		},
-	}
-
-	for _, tc := range testCases {
-		// This would test the actual joinArgs function
-		assert.NotEmpty(suite.T(), tc.expected)
-	}
-}
-
 func TestUtilsTestSuite(t *testing.T) {
 	suite.Run(t, new(UtilsTestSuite))
 }