pg_multitenant_schemas 0.2.0 → 0.2.3

data/lib/pg_multitenant_schemas/migration_schema_operations.rb

@@ -34,21 +34,115 @@ module PgMultitenantSchemas
     end
 
     def run_migrations
-      ActiveRecord::Base.connection.migrate
+      # Use the migration context to run migrations
+      migration_context_obj = migration_context
+      return unless migration_context_obj
+
+      migration_context_obj.migrate
     end
 
     def pending_migrations
-      ActiveRecord::Base.connection.migration_context.migrations.reject do |migration|
-        ActiveRecord::Base.connection.migration_context.get_all_versions.include?(migration.version)
+      migration_context_obj = migration_context
+      return [] unless migration_context_obj
+
+      all_migrations = migration_context_obj.migrations
+      applied_version_list = applied_versions
+
+      all_migrations.reject do |migration|
+        applied_version_list.include?(migration.version)
       end
     end
 
     def applied_migrations
-      ActiveRecord::Base.connection.migration_context.get_all_versions
+      applied_versions
     end
 
     def migration_paths
-      ActiveRecord::Base.connection.migration_context.migrations_paths
+      migration_context_obj = migration_context
+      if migration_context_obj.respond_to?(:migrations_paths)
+        migration_context_obj.migrations_paths
+      elsif defined?(::Rails) && ::Rails.application
+        # Fallback to default Rails migration paths
+        ::Rails.application.paths["db/migrate"].expanded
+      else
+        ["db/migrate"]
+      end
+    end
+
+    def migration_context
+      # Return nil if ActiveRecord is not available (for tests)
+      return nil unless defined?(ActiveRecord::Base)
+
+      # Rails 8 compatibility: Try multiple approaches
+      find_migration_context
+    rescue StandardError => e
+      # Use explicit Rails logger to avoid namespace conflicts
+      ::Rails.logger&.warn("Failed to get migration context: #{e.message}") if defined?(::Rails)
+      nil
+    end
+
+    def find_migration_context
+      if ActiveRecord::Base.respond_to?(:migration_context)
+        # Rails 8+: Try base migration context first
+        ActiveRecord::Base.migration_context
+      elsif ActiveRecord::Base.connection.respond_to?(:migration_context)
+        # Rails 7: Use connection migration context
+        ActiveRecord::Base.connection.migration_context
+      elsif defined?(ActiveRecord::MigrationContext)
+        # Fallback: Create a new migration context with default paths
+        create_fallback_migration_context
+      end
+    end
+
+    def create_fallback_migration_context
+      paths = if defined?(::Rails) && ::Rails.application
+                ::Rails.application.paths["db/migrate"].expanded
+              else
+                ["db/migrate"]
+              end
+      ActiveRecord::MigrationContext.new(paths)
+    end
+
+    # Get applied migration versions with Rails 8 compatibility
+    def applied_versions
+      # Return empty array if ActiveRecord is not available (for tests)
+      return [] unless defined?(ActiveRecord::Base)
+
+      applied_versions_from_context || applied_versions_from_database
+    rescue StandardError => e
+      # If anything fails, return empty array
+      ::Rails.logger&.warn("Failed to get applied versions: #{e.message}") if defined?(::Rails)
+      []
+    end
+
+    def applied_versions_from_context
+      # Try using migration context first (for tests and Rails 7)
+      migration_context_obj = migration_context
+      return migration_context_obj.get_all_versions if migration_context_obj.respond_to?(:get_all_versions)
+
+      nil
+    end
+
+    def applied_versions_from_database
+      # Fallback to direct database query (Rails 8)
+      connection = ActiveRecord::Base.connection
+      table_name = "schema_migrations"
+
+      ensure_schema_migrations_table_exists(connection, table_name)
+
+      # Query the table directly for maximum compatibility
+      connection.select_values("SELECT version FROM #{table_name} ORDER BY version")
+    end
+
+    def ensure_schema_migrations_table_exists(connection, table_name)
+      # Ensure the schema_migrations table exists
+      return if connection.table_exists?(table_name)
+
+      # Create the table if it doesn't exist
+      connection.create_table(table_name, id: false) do |t|
+        t.string :version, null: false
+      end
+      connection.add_index(table_name, :version, unique: true, name: "unique_schema_migrations")
     end
   end
 end

data/lib/pg_multitenant_schemas/migrator.rb

@@ -73,15 +73,72 @@ module PgMultitenantSchemas
         puts "⏪ Rolling back #{steps} steps for #{schema_name}" if verbose
         original_schema = current_schema
 
-        begin
-          switch_to_schema(schema_name)
-          ActiveRecord::Base.connection.migration_context.rollback(migration_paths, steps)
-          puts "  ✅ Rollback completed" if verbose
-        rescue StandardError => e
-          puts "  ❌ Rollback failed: #{e.message}" if verbose
-          raise
-        ensure
-          switch_to_schema(original_schema) if original_schema
+        perform_rollback(schema_name, steps, verbose)
+      ensure
+        switch_to_schema(original_schema) if original_schema
+      end
+
+      private
+
+      def perform_rollback(schema_name, steps, verbose)
+        switch_to_schema(schema_name)
+        migration_context_obj = migration_context
+
+        unless migration_context_obj
+          puts "  ❌ Cannot rollback: migration context not available" if verbose
+          return
+        end
+
+        migration_context_obj.rollback(migration_paths, steps)
+        puts "  ✅ Rollback completed" if verbose
+      rescue StandardError => e
+        puts "  ❌ Rollback failed: #{e.message}" if verbose
+        raise
+      end
+
+      def migration_context
+        # Return nil if ActiveRecord is not available (for tests)
+        return nil unless defined?(ActiveRecord::Base)
+
+        # Rails 8 compatibility: Try multiple approaches
+        find_migration_context
+      rescue StandardError => e
+        # Use explicit Rails logger to avoid namespace conflicts
+        ::Rails.logger&.warn("Failed to get migration context: #{e.message}") if defined?(::Rails)
+        nil
+      end
+
+      def find_migration_context
+        if ActiveRecord::Base.respond_to?(:migration_context)
+          # Rails 8+: Try base migration context first
+          ActiveRecord::Base.migration_context
+        elsif ActiveRecord::Base.connection.respond_to?(:migration_context)
+          # Rails 7: Use connection migration context
+          ActiveRecord::Base.connection.migration_context
+        elsif defined?(ActiveRecord::MigrationContext)
+          # Fallback: Create a new migration context with default paths
+          create_fallback_migration_context
+        end
+      end
+
+      def create_fallback_migration_context
+        paths = if defined?(::Rails) && ::Rails.application
+                  ::Rails.application.paths["db/migrate"].expanded
+                else
+                  ["db/migrate"]
+                end
+        ActiveRecord::MigrationContext.new(paths)
+      end
+
+      def migration_paths
+        migration_context_obj = migration_context
+        if migration_context_obj.respond_to?(:migrations_paths)
+          migration_context_obj.migrations_paths
+        elsif defined?(::Rails) && ::Rails.application
+          # Fallback to default Rails migration paths
+          ::Rails.application.paths["db/migrate"].expanded
+        else
+          ["db/migrate"]
         end
       end
     end

data/lib/pg_multitenant_schemas/rails/railtie.rb

@@ -14,7 +14,10 @@ module PgMultitenantSchemas
 
       # Add rake tasks
       rake_tasks do
-        load File.expand_path("../tasks/pg_multitenant_schemas.rake", __dir__)
+        # Load all task files
+        Dir[File.expand_path("../tasks/*.rake", __dir__)].each do |task_file|
+          load task_file
+        end
       end
 
       # Add generators

data/lib/pg_multitenant_schemas/schema_switcher.rb

@@ -80,8 +80,42 @@ module PgMultitenantSchemas
         get_result_value(result, 0, 0)
       end
 
+      # List all schemas in the database
+      def list_schemas
+        conn = connection
+        result = execute_sql(conn, <<~SQL)
+          SELECT schema_name FROM information_schema.schemata#{" "}
+          ORDER BY schema_name
+        SQL
+
+        extract_schemas_from_result(result)
+      end
+
       private
 
+      def extract_schemas_from_result(result)
+        schemas = []
+
+        if result.respond_to?(:rows)
+          # Rails ActiveRecord::Result
+          result.rows.each { |row| schemas << row[0] }
+        elsif result.respond_to?(:each)
+          # Raw PG::Result
+          result.each { |row| schemas << row["schema_name"] }
+        else
+          # Fallback for other result types
+          extract_schemas_fallback(result, schemas)
+        end
+
+        schemas
+      end
+
+      def extract_schemas_fallback(result, schemas)
+        (0...result.ntuples).each do |i|
+          schemas << get_result_value(result, i, 0)
+        end
+      end
+
       # Execute SQL - handles both Rails connections and raw PG connections
       def execute_sql(conn, sql)
         if conn.respond_to?(:execute)

data/lib/pg_multitenant_schemas/tasks/tenant_tasks.rake

@@ -1,8 +1,5 @@
 # frozen_string_literal: true
 
-# Load individual task files
-Dir[File.join(File.dirname(__FILE__), "*.rake")].each { |file| load file unless file == __FILE__ }
-
 require_relative "../tenant_task_helpers"
 
 namespace :tenants do

data/lib/pg_multitenant_schemas/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PgMultitenantSchemas
-  VERSION = "0.2.0"
+  VERSION = "0.2.3"
 end

data/pg_multitenant_schemas.gemspec

@@ -8,9 +8,9 @@ Gem::Specification.new do |spec|
   spec.authors = ["Ruben Paz"]
   spec.email = ["rubenpazchuspe@outlook.com"]
 
-  spec.summary = "Modern PostgreSQL schema-based multitenancy for Rails 8+ applications"
+  spec.summary = "Modern PostgreSQL schema-based multitenancy for Rails 7+ applications"
   spec.description = "A modern Ruby gem that provides PostgreSQL schema-based multitenancy with automatic tenant " \
-                     "resolution and schema switching. Built for Rails 8+ and Ruby 3.3+, focusing on security, " \
+                     "resolution and schema switching. Compatible with Rails 7+ and Ruby 3.0+, focusing on security, " \
                      "performance, and developer experience. Perfect for modern SaaS applications requiring " \
                      "secure tenant isolation."
   spec.homepage = "https://github.com/rubenpazch/pg_multitenant_schemas"
@@ -34,8 +34,8 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r{\Aexe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  # Runtime dependencies - Modern Rails only
-  spec.add_dependency "activerecord", ">= 8.0", "< 9.0"
-  spec.add_dependency "activesupport", ">= 8.0", "< 9.0"
+  # Runtime dependencies - Rails 7+ support with Rails 8 optimizations
+  spec.add_dependency "activerecord", ">= 7.0", "< 9.0"
+  spec.add_dependency "activesupport", ">= 7.0", "< 9.0"
   spec.add_dependency "pg", "~> 1.5"
 end

data/pre-push-check.sh

@@ -0,0 +1,95 @@
+#!/bin/bash
+# Pre-push check script for pg_multitenant_schemas gem
+# Run this before pushing to GitHub to catch issues early
+
+set -e  # Exit on any error
+
+echo "🔍 Running pre-push checks for pg_multitenant_schemas..."
+echo ""
+
+# Color codes for output
+RED='\033[0;31m'
+GREEN='\033[0;32m'
+YELLOW='\033[1;33m'
+NC='\033[0m' # No Color
+
+# Function to print colored output
+print_status() {
+    echo -e "${2}${1}${NC}"
+}
+
+# Function to run a check
+run_check() {
+    local name="$1"
+    local command="$2"
+    local optional="$3"
+    
+    echo "📋 $name..."
+    if eval "$command"; then
+        print_status "✅ $name passed" "$GREEN"
+    else
+        if [ "$optional" = "true" ]; then
+            print_status "⚠️  $name failed (optional)" "$YELLOW"
+        else
+            print_status "❌ $name failed" "$RED"
+            exit 1
+        fi
+    fi
+    echo ""
+}
+
+# 1. Check if we're in the right directory
+if [ ! -f "pg_multitenant_schemas.gemspec" ]; then
+    print_status "❌ Not in pg_multitenant_schemas directory" "$RED"
+    exit 1
+fi
+
+# 2. Install dependencies
+run_check "Bundle install" "bundle install --quiet"
+
+# 3. RuboCop
+run_check "RuboCop (code style)" "bundle exec rubocop"
+
+# 4. RSpec unit tests
+run_check "RSpec unit tests" "bundle exec rspec --exclude-pattern '**/integration/**/*_spec.rb'"
+
+# 5. Security audit
+run_check "Security audit" "bundle audit" "true"
+
+# 6. Gem build test
+run_check "Gem build test" "gem build pg_multitenant_schemas.gemspec > /dev/null 2>&1 && rm -f pg_multitenant_schemas-*.gem"
+
+# 7. Check for PostgreSQL and run integration tests
+if command -v psql &> /dev/null && pg_isready &> /dev/null; then
+    # Check if test database exists, create if not
+    if ! psql -lqt | cut -d \| -f 1 | grep -qw pg_multitenant_test; then
+        echo "📋 Creating test database..."
+        createdb pg_multitenant_test || true
+    fi
+    
+    run_check "Integration tests" "PGDATABASE=pg_multitenant_test bundle exec rspec --tag integration" "true"
+else
+    print_status "⚠️  PostgreSQL not available, skipping integration tests" "$YELLOW"
+fi
+
+# 8. Check for uncommitted changes
+if [ -n "$(git status --porcelain)" ]; then
+    print_status "⚠️  You have uncommitted changes" "$YELLOW"
+    git status --short
+    echo ""
+fi
+
+# 9. Check current branch
+current_branch=$(git branch --show-current)
+if [ "$current_branch" = "main" ]; then
+    print_status "⚠️  You're on the main branch" "$YELLOW"
+fi
+
+echo "🎉 All checks completed successfully!"
+echo ""
+echo "🚀 Your code is ready to push to GitHub!"
+echo ""
+echo "💡 Pro tip: You can test GitHub Actions locally using 'act':"
+echo "   brew install act"
+echo "   act push  # Test CI workflow"
+echo ""

data/test-github-setup.sh

@@ -0,0 +1,85 @@
+#!/bin/bash
+# Test GitHub Actions permissions and setup locally
+
+echo "🔍 Testing GitHub Actions setup and permissions..."
+echo ""
+
+# Check if we're in a Git repository
+if ! git rev-parse --git-dir > /dev/null 2>&1; then
+    echo "❌ Not in a Git repository"
+    exit 1
+fi
+
+# Check if we're on the right branch
+current_branch=$(git branch --show-current)
+echo "📋 Current branch: $current_branch"
+
+# Check if we have the workflow files
+if [ -f ".github/workflows/main.yml" ]; then
+    echo "✅ CI workflow file exists"
+else
+    echo "❌ CI workflow file missing"
+fi
+
+if [ -f ".github/workflows/release.yml" ]; then
+    echo "✅ Release workflow file exists"
+else
+    echo "❌ Release workflow file missing"
+fi
+
+# Check version file
+if [ -f "lib/pg_multitenant_schemas/version.rb" ]; then
+    version=$(grep -E "VERSION = ['\"]" lib/pg_multitenant_schemas/version.rb | cut -d'"' -f2 | cut -d"'" -f2)
+    echo "✅ Version file exists: $version"
+else
+    echo "❌ Version file missing"
+fi
+
+# Check if gemspec exists
+if [ -f "pg_multitenant_schemas.gemspec" ]; then
+    echo "✅ Gemspec file exists"
+else
+    echo "❌ Gemspec file missing"
+fi
+
+# Check remote origin
+remote_url=$(git remote get-url origin 2>/dev/null || echo "No remote")
+echo "📋 Remote origin: $remote_url"
+
+# Test Git config (what GitHub Actions would use)
+echo ""
+echo "🧪 Testing Git configuration..."
+git config --local user.email "41898282+github-actions[bot]@users.noreply.github.com"
+git config --local user.name "github-actions[bot]"
+echo "✅ Git config set for GitHub Actions bot"
+
+# Test gem build
+echo ""
+echo "🧪 Testing gem build..."
+if gem build pg_multitenant_schemas.gemspec > /dev/null 2>&1; then
+    echo "✅ Gem builds successfully"
+    rm -f pg_multitenant_schemas-*.gem
+else
+    echo "❌ Gem build failed"
+fi
+
+# Check for uncommitted changes
+if [ -n "$(git status --porcelain)" ]; then
+    echo ""
+    echo "⚠️  You have uncommitted changes:"
+    git status --short
+fi
+
+echo ""
+echo "🎯 Next Steps:"
+echo "1. Ensure repository has proper permissions set in GitHub:"
+echo "   - Go to Settings > Actions > General"
+echo "   - Set 'Workflow permissions' to 'Read and write permissions'"
+echo "   - Enable 'Allow GitHub Actions to create and approve pull requests'"
+echo ""
+echo "2. To trigger a release:"
+echo "   - Update version in lib/pg_multitenant_schemas/version.rb"
+echo "   - Commit and push to main branch"
+echo "   - GitHub Actions will automatically create a release"
+echo ""
+echo "📚 For more details, see: docs/github_actions_setup.md"

data/validate-github-commands.sh

@@ -0,0 +1,47 @@
+#!/bin/bash
+# Validate GitHub workflow commands locally
+# This script tests the exact commands used in GitHub Actions
+
+set -e
+
+echo "🧪 Testing GitHub Actions commands locally..."
+echo ""
+
+# Test RuboCop (exact command from workflow)
+echo "📋 Testing: bundle exec rubocop"
+if bundle exec rubocop > /dev/null 2>&1; then
+    echo "✅ RuboCop passed"
+else
+    echo "❌ RuboCop failed"
+    exit 1
+fi
+
+# Test unit tests (exact command from workflow) 
+echo "📋 Testing: bundle exec rspec --exclude-pattern '**/integration/**/*_spec.rb'"
+if bundle exec rspec --exclude-pattern '**/integration/**/*_spec.rb' > /dev/null 2>&1; then
+    echo "✅ Unit tests passed"
+else
+    echo "❌ Unit tests failed"
+    exit 1
+fi
+
+# Test integration tests (exact command from workflow)
+echo "📋 Testing: bundle exec rspec --tag integration"
+if bundle exec rspec --tag integration > /dev/null 2>&1; then
+    echo "✅ Integration tests passed"
+else
+    echo "⚠️  Integration tests failed (may need PostgreSQL setup)"
+fi
+
+# Test security audit (exact command from workflow)
+echo "📋 Testing: bundle audit"
+if bundle audit > /dev/null 2>&1; then
+    echo "✅ Security audit passed"
+else
+    echo "⚠️  Security audit failed (may have vulnerabilities)"
+fi
+
+echo ""
+echo "🎉 GitHub Actions command validation complete!"
+echo ""
+echo "💡 These are the exact commands that run in GitHub Actions CI"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pg_multitenant_schemas
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.3
 platform: ruby
 authors:
 - Ruben Paz
@@ -15,7 +15,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '8.0'
+        version: '7.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '9.0'
@@ -25,7 +25,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '8.0'
+        version: '7.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '9.0'
@@ -35,7 +35,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '8.0'
+        version: '7.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '9.0'
@@ -45,7 +45,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '8.0'
+        version: '7.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '9.0'
@@ -64,15 +64,17 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.5'
 description: A modern Ruby gem that provides PostgreSQL schema-based multitenancy
-  with automatic tenant resolution and schema switching. Built for Rails 8+ and Ruby
-  3.3+, focusing on security, performance, and developer experience. Perfect for modern
-  SaaS applications requiring secure tenant isolation.
+  with automatic tenant resolution and schema switching. Compatible with Rails 7+
+  and Ruby 3.0+, focusing on security, performance, and developer experience. Perfect
+  for modern SaaS applications requiring secure tenant isolation.
 email:
 - rubenpazchuspe@outlook.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".actrc"
+- ".env.local.example"
 - ".rspec"
 - ".rubocop.yml"
 - ".rubocop_simple.yml"
@@ -80,18 +82,24 @@ files:
 - CHANGELOG.md
 - CODE_OF_CONDUCT.md
 - LICENSE.txt
+- LOCAL_TESTING_SUMMARY.md
 - README.md
 - Rakefile
+- TESTING_LOCALLY.md
 - docs/README.md
 - docs/configuration.md
 - docs/context.md
 - docs/errors.md
+- docs/github_actions_permissions_fix.md
+- docs/github_actions_setup.md
 - docs/integration_testing.md
+- docs/local_workflow_testing.md
 - docs/migrator.md
 - docs/rails_integration.md
 - docs/schema_switcher.md
 - docs/tenant_resolver.md
 - docs/testing.md
+- docs/testing_rails_tasks.md
 - examples/context_management.rb
 - examples/migration_workflow.rb
 - examples/rails_integration/controller_examples.rb
@@ -111,15 +119,17 @@ files:
 - lib/pg_multitenant_schemas/schema_switcher.rb
 - lib/pg_multitenant_schemas/tasks/advanced_tasks.rake
 - lib/pg_multitenant_schemas/tasks/basic_tasks.rake
-- lib/pg_multitenant_schemas/tasks/pg_multitenant_schemas.rake
 - lib/pg_multitenant_schemas/tasks/tenant_tasks.rake
 - lib/pg_multitenant_schemas/tenant_resolver.rb
 - lib/pg_multitenant_schemas/tenant_task_helpers.rb
 - lib/pg_multitenant_schemas/version.rb
 - pg_multitenant_schemas.gemspec
+- pre-push-check.sh
 - rails_integration/app/controllers/application_controller.rb
 - rails_integration/app/models/tenant.rb
 - sig/pg_multitenant_schemas.rbs
+- test-github-setup.sh
+- validate-github-commands.sh
 homepage: https://github.com/rubenpazch/pg_multitenant_schemas
 licenses:
 - MIT
@@ -144,5 +154,5 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 rubygems_version: 3.6.7
 specification_version: 4
-summary: Modern PostgreSQL schema-based multitenancy for Rails 8+ applications
+summary: Modern PostgreSQL schema-based multitenancy for Rails 7+ applications
 test_files: []