decision_agent 1.1.0 → 1.2.0

data/lib/decision_agent/web/public/dmn-editor.js

@@ -216,6 +216,11 @@ function renderModelEditor() {
 async function addDecision(e) {
     e.preventDefault();
 
+    if (!state.currentModel) {
+        showNotification('Please select or create a model first', 'error');
+        return;
+    }
+
     const decisionId = document.getElementById('decision-id-input').value;
     const name = document.getElementById('decision-name-input').value;
     const type = document.getElementById('decision-type-select').value;

data/lib/decision_agent/web/server.rb

@@ -46,7 +46,6 @@ require_relative "../versioning/version_manager"
 
 module DecisionAgent
   module Web
-    # rubocop:disable Metrics/ClassLength
     # Framework-agnostic Rack application - works with any Rack-compatible server
     class Server
       include RackHelpers
@@ -1210,6 +1209,14 @@ module DecisionAgent
             filename = file_param[:filename] || file_param["filename"] || "uploaded_file"
             file_extension = File.extname(filename).downcase
 
+            # Validate file type before processing
+            allowed_extensions = %w[.csv .xlsx .xls]
+            unless allowed_extensions.include?(file_extension)
+              ctx.status(422)
+              ctx.json({ error: "Unsupported file type '#{file_extension}'. Allowed types: .csv, .xlsx, .xls" })
+              next
+            end
+
             # Create temporary file
             temp_file = Tempfile.new(["batch_test", file_extension])
             temp_file.binmode
@@ -2493,6 +2500,5 @@ module DecisionAgent
         )
       end
     end
-    # rubocop:enable Metrics/ClassLength
   end
 end

data/lib/generators/decision_agent/install/templates/migration.rb

@@ -35,5 +35,19 @@ class CreateDecisionAgentTables < ActiveRecord::Migration[7.0]
     #           unique: true,
     #           where: "status = 'active'",
     #           name: 'index_rule_versions_one_active_per_rule'
+
+    # Version tags table — named pointers to specific versions (unique per model + name)
+    # NOTE: version_id is intentionally NOT a foreign key so tags survive version deletion
+    create_table :rule_version_tags do |t|
+      t.string :model_id, null: false
+      t.string :name, null: false
+      t.bigint :version_id, null: false
+      t.timestamps
+    end
+
+    # ✅ CRITICAL: Unique constraint — one tag name per model
+    add_index :rule_version_tags, %i[model_id name], unique: true
+    # Index for efficient tag listing per model
+    add_index :rule_version_tags, :model_id
   end
 end

data/lib/generators/decision_agent/install/templates/monitoring_migration.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 class CreateDecisionAgentMonitoringTables < ActiveRecord::Migration[7.0]
-  # rubocop:disable Metrics/AbcSize, Metrics/MethodLength
   def change
     # Decision logs table
     create_table :decision_logs do |t|
@@ -105,5 +104,4 @@ class CreateDecisionAgentMonitoringTables < ActiveRecord::Migration[7.0]
     #   PARTITION BY RANGE (created_at);
     # SQL
   end
-  # rubocop:enable Metrics/AbcSize, Metrics/MethodLength
 end

data/lib/generators/decision_agent/install/templates/rule_version_tag.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+# Stores named tags that point to a specific RuleVersion.
+# A tag is a mutable pointer (re-pointing updates the existing row) but
+# the content of the target version is immutable.
+#
+# version_id is stored as a plain integer without a foreign-key constraint
+# so that tags survive the deletion of the version they originally pointed to.
+class RuleVersionTag < ApplicationRecord
+  validates :model_id, presence: true
+  validates :name,     presence: true
+  validates :version_id, presence: true
+  validates :name, uniqueness: { scope: :model_id, message: "already exists for this model" }
+
+  scope :for_model, ->(model_id) { where(model_id: model_id) }
+  scope :sorted_by_name, -> { order(name: :asc) }
+
+  # Resolve the version this tag currently points to.
+  # Returns nil if the version has been deleted.
+  def version
+    RuleVersion.find_by(id: version_id)
+  end
+end

data/lib/generators/decision_agent/install/templates/versioning_migration.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+class CreateDecisionAgentVersioningTables < ActiveRecord::Migration[7.0]
+  def change
+    # Rule versions table — stores the full content of each version as JSON
+    create_table :rule_versions do |t|
+      t.string  :rule_id,        null: false, index: true
+      t.integer :version_number, null: false
+      t.text    :content,        null: false # JSON rule definition
+      t.string  :created_by,     null: false, default: "system"
+      t.text    :changelog
+      t.string  :status, null: false, default: "draft" # draft, active, archived
+      t.timestamps
+    end
+
+    # ✅ CRITICAL: Unique constraint prevents duplicate version numbers per rule.
+    # This protects against race conditions in concurrent version creation.
+    add_index :rule_versions, %i[rule_id version_number], unique: true
+
+    # Index for efficient queries by rule_id and status
+    add_index :rule_versions, %i[rule_id status]
+
+    # Optional: Partial unique index for PostgreSQL to enforce one active version per rule
+    # Uncomment if using PostgreSQL:
+    # add_index :rule_versions, [:rule_id, :status],
+    #           unique: true,
+    #           where: "status = 'active'",
+    #           name: 'index_rule_versions_one_active_per_rule'
+
+    # Version tags table — named pointers to specific versions (unique per model + name).
+    # version_id is intentionally NOT a foreign key so tags survive version deletion.
+    create_table :rule_version_tags do |t|
+      t.string :model_id,   null: false
+      t.string :name,       null: false
+      t.bigint :version_id, null: false
+      t.timestamps
+    end
+
+    # ✅ CRITICAL: Unique constraint — one tag name per model
+    add_index :rule_version_tags, %i[model_id name], unique: true
+    # Index for efficient tag listing per model
+    add_index :rule_version_tags, :model_id
+  end
+end

data/lib/generators/decision_agent/monitoring_migration/monitoring_migration_generator.rb

@@ -0,0 +1,67 @@
+# frozen_string_literal: true
+
+require "rails/generators"
+require "rails/generators/migration"
+
+module DecisionAgent
+  module Generators
+    # Standalone generator that installs only the monitoring tables and models.
+    #
+    # Usage:
+    #   rails generate decision_agent:monitoring_migration
+    #
+    # This is a focused alternative to the full install generator's --monitoring
+    # flag. It generates:
+    #   - db/migrate/create_decision_agent_monitoring_tables.rb
+    #   - app/models/decision_log.rb
+    #   - app/models/evaluation_metric.rb
+    #   - app/models/performance_metric.rb
+    #   - app/models/error_metric.rb
+    #   - lib/tasks/decision_agent.rake
+    class MonitoringMigrationGenerator < Rails::Generators::Base
+      include Rails::Generators::Migration
+
+      source_root File.expand_path("../../install/templates", __dir__)
+
+      desc "Generates the DecisionAgent monitoring migration and ActiveRecord models"
+
+      def self.next_migration_number(dirname)
+        next_migration_number = current_migration_number(dirname) + 1
+        ActiveRecord::Migration.next_migration_number(next_migration_number)
+      end
+
+      def copy_migration
+        migration_template "monitoring_migration.rb",
+                           "db/migrate/create_decision_agent_monitoring_tables.rb",
+                           migration_version: migration_version
+      end
+
+      def copy_models
+        copy_file "decision_log.rb",        "app/models/decision_log.rb"
+        copy_file "evaluation_metric.rb",   "app/models/evaluation_metric.rb"
+        copy_file "performance_metric.rb",  "app/models/performance_metric.rb"
+        copy_file "error_metric.rb",        "app/models/error_metric.rb"
+      end
+
+      def copy_rake_tasks
+        copy_file "decision_agent_tasks.rake", "lib/tasks/decision_agent.rake"
+      end
+
+      def show_readme
+        say "\n"
+        say "DecisionAgent monitoring tables generated successfully!", :green
+        say "\nNext steps:"
+        say "  1. Run migrations:    rails db:migrate"
+        say "  2. Verify models:     DecisionLog, EvaluationMetric, PerformanceMetric, ErrorMetric"
+        say "  3. Use the adapter:   DecisionAgent::Monitoring::Storage::ActiveRecordAdapter.new"
+        say "\nSee docs/PERSISTENT_MONITORING.md for configuration and query examples."
+      end
+
+      private
+
+      def migration_version
+        "[#{Rails::VERSION::MAJOR}.#{Rails::VERSION::MINOR}]"
+      end
+    end
+  end
+end

data/lib/generators/decision_agent/versioning_migration/versioning_migration_generator.rb

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+require "rails/generators"
+require "rails/generators/migration"
+
+module DecisionAgent
+  module Generators
+    # Standalone generator that installs only the versioning tables and models.
+    #
+    # Usage:
+    #   rails generate decision_agent:versioning_migration
+    #
+    # This is a focused alternative to the full install generator. It generates:
+    #   - db/migrate/create_decision_agent_versioning_tables.rb
+    #   - app/models/rule_version.rb
+    #   - app/models/rule_version_tag.rb
+    class VersioningMigrationGenerator < Rails::Generators::Base
+      include Rails::Generators::Migration
+
+      source_root File.expand_path("../../install/templates", __dir__)
+
+      desc "Generates the DecisionAgent versioning migration and ActiveRecord models"
+
+      def self.next_migration_number(dirname)
+        next_migration_number = current_migration_number(dirname) + 1
+        ActiveRecord::Migration.next_migration_number(next_migration_number)
+      end
+
+      def copy_migration
+        migration_template "versioning_migration.rb",
+                           "db/migrate/create_decision_agent_versioning_tables.rb",
+                           migration_version: migration_version
+      end
+
+      def copy_models
+        copy_file "rule_version.rb",     "app/models/rule_version.rb"
+        copy_file "rule_version_tag.rb", "app/models/rule_version_tag.rb"
+      end
+
+      def show_readme
+        say "\n"
+        say "DecisionAgent versioning tables generated successfully!", :green
+        say "\nNext steps:"
+        say "  1. Run migrations:      rails db:migrate"
+        say "  2. Verify models:       RuleVersion, RuleVersionTag"
+        say "  3. Use the adapter:     DecisionAgent::Versioning::ActiveRecordAdapter.new"
+        say "\nSee docs/VERSIONING.md for configuration and usage examples."
+      end
+
+      private
+
+      def migration_version
+        "[#{Rails::VERSION::MAJOR}.#{Rails::VERSION::MINOR}]"
+      end
+    end
+  end
+end

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: decision_agent
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Sam Aswin
+autorequire:
 bindir: bin
 cert_chain: []
-date: 1980-01-02 00:00:00.000000000 Z
+date: 2026-04-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bcrypt
@@ -372,6 +373,10 @@ files:
 - lib/generators/decision_agent/install/templates/performance_metric.rb
 - lib/generators/decision_agent/install/templates/rule.rb
 - lib/generators/decision_agent/install/templates/rule_version.rb
+- lib/generators/decision_agent/install/templates/rule_version_tag.rb
+- lib/generators/decision_agent/install/templates/versioning_migration.rb
+- lib/generators/decision_agent/monitoring_migration/monitoring_migration_generator.rb
+- lib/generators/decision_agent/versioning_migration/versioning_migration_generator.rb
 homepage: https://github.com/samaswin/decision_agent
 licenses:
 - MIT
@@ -381,6 +386,7 @@ metadata:
   changelog_uri: https://github.com/samaswin/decision_agent/blob/main/docs/CHANGELOG.md
   github_repo: https://github.com/samaswin/decision_agent
   rubygems_mfa_required: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -395,7 +401,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.3
+rubygems_version: 3.5.16
+signing_key:
 specification_version: 4
 summary: Deterministic, explainable, auditable decision engine for Ruby
 test_files: []