ragdoll 0.1.9 → 0.1.11

data/lib/tasks/db.rake

@@ -25,22 +25,17 @@ namespace :db do
       )
       
       # Run individual SQL commands to avoid transaction block issues
-      begin
-        ActiveRecord::Base.connection.execute("DROP DATABASE IF EXISTS ragdoll_development")
-      rescue => e
-        puts "Note: #{e.message}" if e.message.include?("does not exist")
-      end
-      
-      begin
-        ActiveRecord::Base.connection.execute("DROP ROLE IF EXISTS ragdoll")
-      rescue => e
-        puts "Note: #{e.message}" if e.message.include?("does not exist")
-      end
+      # Note: Removed the DROP DATABASE/ROLE here since that should be done via db:drop task
       
       begin
         ActiveRecord::Base.connection.execute("CREATE ROLE ragdoll WITH LOGIN CREATEDB")
+        puts "Role 'ragdoll' created successfully"
       rescue => e
-        puts "Note: Role already exists, continuing..." if e.message.include?("already exists")
+        if e.message.include?("already exists")
+          puts "Note: Role 'ragdoll' already exists, continuing..."
+        else
+          raise e
+        end
       end
       
       begin
@@ -50,8 +45,16 @@ namespace :db do
             ENCODING = 'UTF8'
             CONNECTION LIMIT = -1
         SQL
+        puts "Database 'ragdoll_development' created successfully"
       rescue => e
-        puts "Note: Database already exists, continuing..." if e.message.include?("already exists")
+        if e.message.include?("already exists")
+          puts "ERROR: Database 'ragdoll_development' already exists!"
+          puts "Please run 'rake db:drop' first to remove the existing database, then run 'rake db:create' again."
+          puts "Or use 'rake db:reset' to drop, create, and migrate in one step."
+          exit 1
+        else
+          raise e
+        end
       end
       
       ActiveRecord::Base.connection.execute("GRANT ALL PRIVILEGES ON DATABASE ragdoll_development TO ragdoll")
@@ -97,8 +100,53 @@ namespace :db do
     puts "Dropping database with config: #{config.database.inspect}"
 
     case config.database[:adapter]
-    when "postgresql", "mysql2"
-      puts "For #{config.database[:adapter]}, please drop the database manually on your server"
+    when "postgresql"
+      puts "PostgreSQL database drop - running as superuser to drop database and role..."
+      
+      # Connect as superuser to drop database and role
+      ActiveRecord::Base.establish_connection(
+        adapter: 'postgresql',
+        database: 'postgres', # Connect to postgres database initially
+        username: ENV.fetch('POSTGRES_SUPERUSER', 'postgres'),
+        password: ENV['POSTGRES_SUPERUSER_PASSWORD'],
+        host: config.database[:host] || 'localhost',
+        port: config.database[:port] || 5432
+      )
+      
+      # Drop the database if it exists
+      begin
+        ActiveRecord::Base.connection.execute("DROP DATABASE IF EXISTS ragdoll_development")
+        puts "Database 'ragdoll_development' dropped successfully"
+      rescue => e
+        puts "Error dropping database: #{e.message}"
+      end
+      
+      # Optionally drop the role (commented out by default to preserve user)
+      # begin
+      #   ActiveRecord::Base.connection.execute("DROP ROLE IF EXISTS ragdoll")
+      #   puts "Role 'ragdoll' dropped successfully"
+      # rescue => e
+      #   puts "Error dropping role: #{e.message}"
+      # end
+      
+    when "mysql2"
+      puts "MySQL database drop - connecting to drop database..."
+      
+      # Connect without specifying database
+      ActiveRecord::Base.establish_connection(
+        adapter: 'mysql2',
+        username: config.database[:username],
+        password: config.database[:password],
+        host: config.database[:host] || 'localhost',
+        port: config.database[:port] || 3306
+      )
+      
+      begin
+        ActiveRecord::Base.connection.execute("DROP DATABASE IF EXISTS #{config.database[:database]}")
+        puts "Database '#{config.database[:database]}' dropped successfully"
+      rescue => e
+        puts "Error dropping database: #{e.message}"
+      end
     end
 
     puts "Database drop completed"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ragdoll
 version: !ruby/object:Gem::Version
-  version: 0.1.9
+  version: 0.1.11
 platform: ruby
 authors:
 - Dewayne VanHoozer
@@ -406,12 +406,12 @@ files:
 - app/services/ragdoll/search_engine.rb
 - app/services/ragdoll/text_chunker.rb
 - app/services/ragdoll/text_generation_service.rb
-- db/migrate/001_enable_postgresql_extensions.rb
-- db/migrate/004_create_ragdoll_documents.rb
-- db/migrate/005_create_ragdoll_embeddings.rb
-- db/migrate/006_create_ragdoll_contents.rb
-- db/migrate/007_create_ragdoll_searches.rb
-- db/migrate/008_create_ragdoll_search_results.rb
+- db/migrate/20250815234901_enable_postgresql_extensions.rb
+- db/migrate/20250815234902_create_ragdoll_documents.rb
+- db/migrate/20250815234903_create_ragdoll_embeddings.rb
+- db/migrate/20250815234904_create_ragdoll_contents.rb
+- db/migrate/20250815234905_create_ragdoll_searches.rb
+- db/migrate/20250815234906_create_ragdoll_search_results.rb
 - lib/ragdoll-core.rb
 - lib/ragdoll.rb
 - lib/ragdoll/core.rb

data/db/migrate/004_create_ragdoll_documents.rb

@@ -1,70 +0,0 @@
-class CreateRagdollDocuments < ActiveRecord::Migration[7.0]
-  def change
-    create_table :ragdoll_documents,
-      comment: "Core documents table with LLM-generated structured metadata" do |t|
-
-      t.string :location, null: false,
-        comment: "Source location of document (file path, URL, or identifier)"
-
-      t.string :title, null: false,
-        comment: "Human-readable document title for display and search"
-
-      t.text :summary, null: false, default: "",
-        comment: "LLM-generated summary of document content"
-
-      t.text :keywords , null: false, default: "",
-        comment: "LLM-generated comma-separated keywords of document"
-
-      t.string :document_type, null: false, default: "text",
-        comment: "Document format type"
-
-      t.string :status, null: false, default: "pending",
-        comment: "Document processing status"
-
-      t.json :metadata, default: {},
-        comment: "LLM-generated structured metadata about the file"
-
-      t.timestamp :file_modified_at, null: false, default: -> { "CURRENT_TIMESTAMP" },
-        comment: "Timestamp when the source file was last modified"
-
-      t.timestamps null: false,
-        comment: "Standard creation and update timestamps"
-
-      ###########
-      # Indexes #
-      ###########
-
-      t.index :location, unique: true,
-        comment: "Unique index for document source lookup"
-
-      t.index :title,
-        comment: "Index for title-based search"
-
-      t.index :document_type,
-        comment: "Index for filtering by document type"
-
-      t.index :status,
-        comment: "Index for filtering by processing status"
-
-      t.index :created_at,
-        comment: "Index for chronological sorting"
-
-      t.index %i[document_type status],
-        comment: "Composite index for type+status filtering"
-
-      t.index "to_tsvector('english', COALESCE(title, '') ||
-        ' ' ||
-        COALESCE(metadata->>'summary', '') ||
-        ' ' || COALESCE(metadata->>'keywords', '') ||
-        ' ' || COALESCE(metadata->>'description', ''))",
-        using: :gin, name: "index_ragdoll_documents_on_fulltext_search",
-        comment: "Full-text search across title and metadata fields"
-
-      t.index "(metadata->>'document_type')", name: "index_ragdoll_documents_on_metadata_type",
-        comment: "Index for filtering by document type"
-
-      t.index "(metadata->>'classification')", name: "index_ragdoll_documents_on_metadata_classification",
-        comment: "Index for filtering by document classification"
-    end
-  end
-end