Modified install script to use venv setup

Author: ritwik-g

.flake8

@@ -1,5 +1,5 @@
 [flake8]
-max-line-length = 100
+max-line-length = 120
 exclude =
     .tox,
     .git,

.pre-commit-config.yaml

@@ -14,6 +14,7 @@ repos:
     hooks:
     -   id: black
         language_version: python3
+        args: ["--line-length", "120"]
 
 -   repo: https://github.com/pycqa/isort
     rev: 5.13.2

helm_values_manager/cli.py

@@ -5,7 +5,7 @@
 app = typer.Typer(
     name="values-manager",
     help="A Helm plugin to manage values and secrets across environments.",
-    add_completion=False,
+    add_completion=True,
 )
 
 
@@ -24,15 +24,16 @@ def main(ctx: typer.Context):
 
 @app.command()
 def init(
+    release_name: str = typer.Option(..., "--release", "-r", help="Name of the Helm release"),
     config_file: str = typer.Option(
         "values-manager.yaml",
         "--config",
         "-c",
         help="Path to the values manager configuration file",
-    )
+    ),
 ):
     """Initialize a new values manager configuration."""
-    typer.echo(f"Initializing values manager with config file: {config_file}")
+    typer.echo(f"Initializing values manager with config file: {config_file}, for the release: {release_name}.")
     # TODO: Implement initialization logic
 
 

plugin.yaml

@@ -1,23 +1,11 @@
 name: "values-manager"
-version: "0.1-rc1"
+version: "0.1.0rc1"
 usage: "Manage Helm values and secrets across environments"
 description: |-
   This plugin helps you manage Helm values and secrets across different environments
   while supporting multiple secret backends like AWS Secrets Manager, Azure Key Vault,
   and HashiCorp Vault.
-platformCommand:
-  - os: linux
-    arch: amd64
-    command: "$HELM_PLUGIN_DIR/bin/wrapper.sh"
-  - os: darwin
-    arch: amd64
-    command: "$HELM_PLUGIN_DIR/bin/wrapper.sh"
-  - os: darwin
-    arch: arm64
-    command: "$HELM_PLUGIN_DIR/bin/wrapper.sh"
-  - os: windows
-    arch: amd64
-    command: "$HELM_PLUGIN_DIR/bin/wrapper.bat"
+command: "$HELM_PLUGIN_DIR/venv/bin/python3 $HELM_PLUGIN_DIR/helm_values_manager/cli.py"
 hooks:
   install: "$HELM_PLUGIN_DIR/scripts/install.sh"
   update: "$HELM_PLUGIN_DIR/scripts/update.sh"

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "helm-values-manager"
-version = "0.1-rc1"
+version = "0.1.0rc1"
 description = "A Helm plugin to manage values and secrets across environments"
 readme = "README.md"
 requires-python = ">=3.8"
@@ -39,22 +39,22 @@ dev = [
 ]
 
 [tool.black]
-line-length = 100
+line-length = 120
 target-version = ['py38']
 include = '\.pyi?$'
 
 [tool.isort]
 profile = "black"
 multi_line_output = 3
-line_length = 100
+line_length = 120
 
 [tool.pytest.ini_options]
 testpaths = ["tests"]
 python_files = ["test_*.py"]
 addopts = "--cov=helm_values_manager --cov-report=xml --cov-report=term-missing"
 
 [tool.flake8]
-max-line-length = 100
+max-line-length = 120
 exclude = [
     ".tox",
     ".git",

scripts/install.sh

@@ -1,79 +1,30 @@
 #!/bin/bash
+set -e
 
-# Check if Python 3.8 or higher is available
-python_version=$(python3 -c 'import sys; print(".".join(map(str, sys.version_info[:2])))')
-required_version="3.8"
-
-if [ "$(printf '%s\n' "$required_version" "$python_version" | sort -V | head -n1)" = "$required_version" ]; then
-  # Python version is >= 3.8, proceed with installation
-  python3 -m pip install -e .
-else
-  echo "Error: Python 3.8 or higher is required (found $python_version)"
-  exit 1
-fi
-
-mkdir -p $HELM_PLUGIN_DIR/bin
-
-# Create Python script with error handling
-cat > $HELM_PLUGIN_DIR/bin/helm_values_manager.py << 'EOF'
-#!/usr/bin/env python3
-import sys
-import os
-
-try:
-    sys.path.insert(0, os.path.join(os.environ["HELM_PLUGIN_DIR"], "lib"))
-    from helm_values_manager import helm_values_manager
-except ImportError as e:
-    print(f"Error: Failed to import helm_values_manager: {e}", file=sys.stderr)
-    print("This might be due to missing dependencies or incorrect installation.", file=sys.stderr)
-    sys.exit(1)
-except Exception as e:
-    print(f"Error: {e}", file=sys.stderr)
-    sys.exit(1)
-
-if __name__ == '__main__':
-    sys.exit(helm_values_manager())
-EOF
-
-# Create wrapper script with error handling
-cat > $HELM_PLUGIN_DIR/bin/wrapper.sh << 'EOF'
-#!/bin/sh
-
-# Ensure HELM_PLUGIN_DIR is set
-if [ -z "$HELM_PLUGIN_DIR" ]; then
-    echo "Error: HELM_PLUGIN_DIR environment variable is not set"
+# Check if Python 3.7+ is available
+if ! command -v python3 &> /dev/null; then
+    echo "Error: Python 3 is required but not found"
     exit 1
 fi
 
-# Ensure the Python script exists
-if [ ! -f "$HELM_PLUGIN_DIR/bin/helm_values_manager.py" ]; then
-    echo "Error: helm_values_manager.py not found"
+PYTHON_VERSION=$(python3 -c 'import sys; print(sys.version_info[0] * 10 + sys.version_info[1])')
+if [ "$PYTHON_VERSION" -lt 37 ]; then
+    echo "Error: Python 3.7 or higher is required (found $PYTHON_VERSION)"
     exit 1
 fi
 
-# Run the Python script with proper error handling
-python3 "$HELM_PLUGIN_DIR/bin/helm_values_manager.py" "$@"
-EOF
-
-# Create Windows wrapper script
-cat > $HELM_PLUGIN_DIR/bin/wrapper.bat << 'EOF'
-@echo off
-setlocal
-
-if "%HELM_PLUGIN_DIR%"=="" (
-    echo Error: HELM_PLUGIN_DIR environment variable is not set
-    exit /b 1
-)
-
-if not exist "%HELM_PLUGIN_DIR%\bin\helm_values_manager.py" (
-    echo Error: helm_values_manager.py not found
-    exit /b 1
-)
-
-python "%HELM_PLUGIN_DIR%\bin\helm_values_manager.py" %*
-EOF
+# Create virtual environment if it doesn't exist
+if [ ! -d "$HELM_PLUGIN_DIR/venv" ]; then
+    echo "Creating virtual environment..."
+    python3 -m venv "$HELM_PLUGIN_DIR/venv"
+fi
 
-chmod +x $HELM_PLUGIN_DIR/bin/helm_values_manager.py
-chmod +x $HELM_PLUGIN_DIR/bin/wrapper.sh
+# Upgrade pip and install package
+echo "Installing dependencies..."
+"$HELM_PLUGIN_DIR/venv/bin/pip" install --upgrade pip
+"$HELM_PLUGIN_DIR/venv/bin/pip" install -e . || {
+    echo "Error: Failed to install package dependencies"
+    exit 1
+}
 
 echo "helm-values-manager plugin installed successfully"

scripts/update.sh

@@ -1,7 +1,5 @@
 #!/bin/bash
+set -e
 
-# Update the plugin using pip
-pip install --target $HELM_PLUGIN_DIR/lib --upgrade .
-
-# Re-run the install script to ensure all files are properly set up
-$HELM_PLUGIN_DIR/scripts/install.sh
+# Run the install script to update the binary
+"$HELM_PLUGIN_DIR/scripts/install.sh"