legionio 1.7.34 → 1.7.35

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4f7058dc8cbf92828614c9767e67004f572a6f2d076295020245f73f6d5dbf38
-  data.tar.gz: 64f195574809ad7f3f2962738a17748401a7153775bcc4afa492144a0b44294c
+  metadata.gz: 04e534c0bd277abe9dbf19d7087e1ed31a468061bd8c9e784fe94c46975d3b14
+  data.tar.gz: f5b7705b447d9df2f9e579f0f0fb7530c418293e0250d3e55b50238ff6989a86
 SHA512:
-  metadata.gz: 7170ca9aa9d93cbb366b499120d996fbd8dec86e13ef7951f0137eaf610c3e3f0604b7889c7ddbbd303f9a3bf53f0e7747331b46907101272ca1e2b1e2f68815
-  data.tar.gz: 0ae25c6ee83ce334fb1d481652facc04898b2d184ec1f02f274e9eec96d320342e4149eabd3f27adaf5118351ad72ea74c22eb4e14147f07809c12d8a58ec6f9
+  metadata.gz: 86290b73ddb0168fc3b012e4c4d598ae52de345e61abd29f64b24f446e1714bec2736442a8861c700cb482bf87350a16011e04b7eaa0c05fc3cd7e44562d4ea5
+  data.tar.gz: 0b4b02ff3603adc13602fca2e4d9da48ae38be3bb03a97b132f534fd0c74e8eb1969a6bd1c2abd3bf888a59d03a2b2a2fdff308f533aee83377a088e707ee18d

data/CHANGELOG.md

@@ -5,6 +5,18 @@
 ### Added
 - register_credential_providers step in boot sequence for Phase 8 credential-only identity module registration with Broker
 
+## [1.7.35] - 2026-04-09
+
+### Added
+- `Legion::Python` central module — single source of truth for venv paths, package list, and interpreter resolution
+- `legionio setup python` CLI command for creating/repairing Python venv with document/data packages
+- `PythonEnvCheck` doctor check for Python venv health
+- Homebrew packaging note: `LEGION_PYTHON` and `LEGION_PYTHON_VENV` are exported by Homebrew wrapper scripts in the companion tap, not by changes in this gem repository
+
+### Fixed
+- `notebook create` crash: removed `python:` kwarg that `Generator.generate` does not accept (`ArgumentError`)
+- `docs serve` now uses `Legion::Python.interpreter` instead of inline path resolution
+
 ## [1.7.33] - 2026-04-09
 
 ### Added

data/lib/legion/cli/docs_command.rb

@@ -2,6 +2,7 @@
 
 require 'thor'
 require 'legion/cli/output'
+require 'legion/python'
 
 module Legion
   module CLI
@@ -50,7 +51,7 @@ module Legion
         puts "  Open http://localhost:#{port}/ in your browser"
         puts "  Serving files from: #{File.expand_path(dir)}"
         puts ''
-        puts "  To start: python3 -m http.server #{port} --directory #{dir}"
+        puts "  To start: #{Legion::Python.interpreter} -m http.server #{port} --directory #{dir}"
         puts '  Press Ctrl+C to stop'
       end
 

data/lib/legion/cli/doctor/python_env_check.rb

@@ -0,0 +1,113 @@
+# frozen_string_literal: true
+
+require 'json'
+require 'open3'
+require 'legion/python'
+
+module Legion
+  module CLI
+    class Doctor
+      class PythonEnvCheck
+        def name
+          'Python env'
+        end
+
+        def run
+          return skip_result('python3 not found') unless Legion::Python.find_system_python3
+          unless Legion::Python.venv_exists?
+            return warn_result(
+              'Python venv missing',
+              'Run: legionio setup python',
+              auto_fixable: true
+            )
+          end
+
+          unless Legion::Python.venv_pip_exists?
+            return warn_result(
+              'pip not found in venv — venv may be corrupt',
+              'Run: legionio setup python --rebuild',
+              auto_fixable: true
+            )
+          end
+
+          unless Legion::Python.venv_python_exists?
+            return warn_result(
+              'python3 not found in venv — venv may be corrupt',
+              'Run: legionio setup python --rebuild',
+              auto_fixable: true
+            )
+          end
+
+          missing = missing_packages
+          if missing.any?
+            return warn_result(
+              "Missing packages: #{missing.join(', ')}",
+              'Run: legionio setup python',
+              auto_fixable: true
+            )
+          end
+
+          pass_result(venv_summary)
+        rescue StandardError => e
+          Legion::Logging.error("PythonEnvCheck#run: #{e.message}") if defined?(Legion::Logging)
+          Result.new(
+            name:         name,
+            status:       :fail,
+            message:      "Python env check error: #{e.message}",
+            prescription: 'Run: legionio setup python'
+          )
+        end
+
+        def fix
+          system('legionio', 'setup', 'python', '--rebuild')
+        end
+
+        private
+
+        def missing_packages
+          pip = Legion::Python.venv_pip
+          output, status = Open3.capture2e(pip, 'list', '--format=json')
+          return Legion::Python::PACKAGES.dup unless status.success?
+
+          installed_names = ::JSON.parse(output).map { |p| p['name'].downcase.tr('-', '_') }
+
+          Legion::Python::PACKAGES.reject do |pkg|
+            installed_names.include?(pkg.downcase.tr('-', '_'))
+          end
+        rescue StandardError
+          Legion::Python::PACKAGES.dup
+        end
+
+        def venv_summary
+          python_bin = Legion::Python.venv_python
+          if File.executable?(python_bin)
+            version = `"#{python_bin}" --version 2>&1`.strip
+            "#{version} at #{Legion::Python::VENV_DIR}"
+          else
+            Legion::Python::VENV_DIR
+          end
+        rescue StandardError
+          Legion::Python::VENV_DIR
+        end
+
+        def pass_result(message)
+          Result.new(name: name, status: :pass, message: message)
+        end
+
+        def warn_result(message, prescription, auto_fixable: false)
+          Result.new(
+            name:         name,
+            status:       :warn,
+            message:      message,
+            prescription: prescription,
+            auto_fixable: auto_fixable
+          )
+        end
+
+        def skip_result(message)
+          Result.new(name: name, status: :skip, message: message)
+        end
+      end
+    end
+  end
+end

data/lib/legion/cli/doctor_command.rb

@@ -19,6 +19,7 @@ module Legion
       autoload :TlsCheck,         'legion/cli/doctor/tls_check'
       autoload :ApiBindCheck,     'legion/cli/doctor/api_bind_check'
       autoload :ModeCheck,        'legion/cli/doctor/mode_check'
+      autoload :PythonEnvCheck,   'legion/cli/doctor/python_env_check'
 
       def self.exit_on_failure?
         true
@@ -41,6 +42,7 @@ module Legion
         TlsCheck
         ApiBindCheck
         ModeCheck
+        PythonEnvCheck
       ].freeze
 
       # Weights: security > connectivity > convenience
@@ -55,6 +57,7 @@ module Legion
         'Bundle'              => 1.5,
         'Config'              => 1.0,
         'Extensions'          => 1.0,
+        'Python env'          => 1.0,
         'PID files'           => 0.5
       }.freeze
 

data/lib/legion/cli/setup_command.rb

@@ -3,9 +3,11 @@
 require 'English'
 require 'json'
 require 'fileutils'
+require 'open3'
 require 'thor'
 require 'rbconfig'
 require 'legion/cli/output'
+require 'legion/python'
 
 module Legion
   module CLI
@@ -56,6 +58,10 @@ module Legion
         }
       }.freeze
 
+      PYTHON_PACKAGES = Legion::Python::PACKAGES
+      PYTHON_VENV_DIR = Legion::Python::VENV_DIR
+      PYTHON_MARKER   = Legion::Python::MARKER
+
       SKILL_CONTENT = <<~MARKDOWN
         ---
         name: legion
@@ -146,6 +152,74 @@ module Legion
         install_pack(:channels)
       end
 
+      desc 'python', 'Set up Legion Python environment (venv + document/data packages)'
+      option :packages, type: :array,   default: [],    banner: 'PKG [PKG...]', desc: 'Additional pip packages to install'
+      option :rebuild,  type: :boolean, default: false, desc: 'Destroy and recreate the venv from scratch'
+      def python # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
+        out = formatter
+        results = []
+
+        python3 = find_python3
+        unless python3
+          out.error('python3 not found. Install it with: brew install python')
+          exit 1
+        end
+
+        if options[:rebuild] && Dir.exist?(PYTHON_VENV_DIR)
+          out.header("Rebuilding Python venv at #{PYTHON_VENV_DIR}") unless options[:json]
+          FileUtils.rm_rf(PYTHON_VENV_DIR)
+        end
+
+        unless File.exist?("#{PYTHON_VENV_DIR}/pyvenv.cfg")
+          out.header("Creating Python venv at #{PYTHON_VENV_DIR}") unless options[:json]
+          FileUtils.mkdir_p(File.dirname(PYTHON_VENV_DIR))
+          unless system(python3, '-m', 'venv', PYTHON_VENV_DIR)
+            out.error('Failed to create Python venv')
+            exit 1
+          end
+          results << { action: 'created_venv', path: PYTHON_VENV_DIR }
+        end
+
+        pip = "#{PYTHON_VENV_DIR}/bin/pip"
+        unless File.executable?(pip)
+          out.error("pip not found at #{pip} — try: legionio setup python --rebuild")
+          exit 1
+        end
+
+        packages = PYTHON_PACKAGES + Array(options[:packages])
+        packages.uniq!
+
+        failed = false
+        packages.each do |pkg|
+          puts "  Installing #{pkg}..." unless options[:json]
+          output, status = Open3.capture2e(pip, 'install', '--quiet', '--upgrade', pkg)
+          if status.success?
+            out.success("  #{pkg}") unless options[:json]
+            results << { package: pkg, status: 'installed' }
+          else
+            failed = true
+            out.error("  #{pkg} failed") unless options[:json]
+            results << { package: pkg, status: 'failed', error: output.strip.lines.last&.strip }
+          end
+        end
+
+        write_python_marker(python3, packages)
+
+        if options[:json]
+          out.json(venv: PYTHON_VENV_DIR, python: python_version(python3), results: results)
+        else
+          out.spacer
+          out.success("Python environment ready: #{PYTHON_VENV_DIR}/bin/python3")
+          out.spacer
+          puts "  Interpreter:    #{PYTHON_VENV_DIR}/bin/python3"
+          puts '  Env var:        $LEGION_PYTHON'
+          puts '  Add packages:   legionio setup python --packages <name> [<name>...]'
+          puts '  Rebuild venv:   legionio setup python --rebuild'
+        end
+
+        exit 1 if failed
+      end
+
       desc 'packs', 'Show installed feature packs and available gems'
       def packs
         out = formatter
@@ -207,6 +281,36 @@ module Legion
 
         private
 
+        # -----------------------------------------------------------------------
+        # Python helpers
+        # -----------------------------------------------------------------------
+
+        def find_python3
+          Legion::Python.find_system_python3
+        end
+
+        def python_version(python3)
+          `"#{python3}" --version 2>&1`.strip
+        rescue StandardError
+          'unknown'
+        end
+
+        def write_python_marker(python3, packages)
+          FileUtils.mkdir_p(File.dirname(PYTHON_MARKER))
+          File.write(PYTHON_MARKER, ::JSON.pretty_generate(
+                                      venv:       PYTHON_VENV_DIR,
+                                      python:     python_version(python3),
+                                      packages:   packages,
+                                      updated_at: Time.now.utc.strftime('%Y-%m-%dT%H:%M:%SZ')
+                                    ))
+        rescue Errno::EPERM, Errno::EACCES, Errno::ENOENT => e
+          Legion::Logging.warn("SetupCommand#write_python_marker: #{e.message}") if defined?(Legion::Logging)
+        end
+
+        # -----------------------------------------------------------------------
+        # Pack helpers
+        # -----------------------------------------------------------------------
+
         def install_pack(pack_name)
           pack = PACKS[pack_name]
           installed, missing = partition_gems(pack[:gems])
@@ -345,6 +449,10 @@ module Legion
           end
         end
 
+        # -----------------------------------------------------------------------
+        # MCP / editor platform helpers
+        # -----------------------------------------------------------------------
+
         def install_claude_mcp(installed)
           settings_path = File.expand_path('~/.claude/settings.json')
           existing = load_json_file(settings_path)

data/lib/legion/python.rb

@@ -0,0 +1,68 @@
+# frozen_string_literal: true
+
+module Legion
+  module Python
+    VENV_DIR = File.expand_path('~/.legionio/python').freeze
+    MARKER   = File.expand_path('~/.legionio/.python-venv').freeze
+
+    PACKAGES = %w[
+      python-pptx
+      python-docx
+      openpyxl
+      pandas
+      pillow
+      requests
+      lxml
+      PyYAML
+      tabulate
+      markdown
+    ].freeze
+
+    SYSTEM_CANDIDATES = %w[
+      /opt/homebrew/bin/python3
+      /usr/local/bin/python3
+      /usr/bin/python3
+    ].freeze
+
+    module_function
+
+    def venv_exists?
+      File.exist?("#{VENV_DIR}/pyvenv.cfg")
+    end
+
+    def venv_python
+      "#{VENV_DIR}/bin/python3"
+    end
+
+    def venv_pip
+      "#{VENV_DIR}/bin/pip"
+    end
+
+    def venv_python_exists?
+      File.executable?(venv_python)
+    end
+
+    def venv_pip_exists?
+      File.executable?(venv_pip)
+    end
+
+    def interpreter
+      return venv_python if venv_python_exists?
+
+      find_system_python3 || 'python3'
+    end
+
+    def pip
+      return venv_pip if venv_pip_exists?
+
+      'pip3'
+    end
+
+    def find_system_python3
+      path_python = `command -v python3 2>/dev/null`.strip
+      candidates = SYSTEM_CANDIDATES.dup
+      candidates.unshift(path_python) unless path_python.empty?
+      candidates.uniq.find { |p| File.executable?(p) }
+    end
+  end
+end

data/lib/legion/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Legion
-  VERSION = '1.7.34'
+  VERSION = '1.7.35'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legionio
 version: !ruby/object:Gem::Version
-  version: 1.7.34
+  version: 1.7.35
 platform: ruby
 authors:
 - Esity
@@ -651,6 +651,7 @@ files:
 - lib/legion/cli/doctor/mode_check.rb
 - lib/legion/cli/doctor/permissions_check.rb
 - lib/legion/cli/doctor/pid_check.rb
+- lib/legion/cli/doctor/python_env_check.rb
 - lib/legion/cli/doctor/rabbitmq_check.rb
 - lib/legion/cli/doctor/result.rb
 - lib/legion/cli/doctor/ruby_version_check.rb
@@ -869,6 +870,7 @@ files:
 - lib/legion/process_role.rb
 - lib/legion/prompts.rb
 - lib/legion/provider.rb
+- lib/legion/python.rb
 - lib/legion/readiness.rb
 - lib/legion/region.rb
 - lib/legion/region/failover.rb