feat(history): add action field

Author: chillfox

migration/db/migrations/20251204120500000_add_history_table.sql

@@ -5,6 +5,7 @@ CREATE TABLE IF NOT EXISTS "history" (
   id TEXT NOT NULL PRIMARY KEY,
   type TEXT NOT NULL,
   resource_id TEXT NOT NULL,
+  action TEXT NOT NULL,
   changed_fields TEXT[] NOT NULL,
   created_at TIMESTAMPTZ NOT NULL,
   updated_at TIMESTAMPTZ NOT NULL

spec/generator.cr

@@ -745,11 +745,13 @@ module PlaceOS::Model
     def self.history(
       type : String = "zone",
       resource_id : String = "zone-#{RANDOM.hex(4)}",
+      action : String = "update",
       changed_fields : Array(String) = ["name", "description"],
     )
       History.new(
         type: type,
         resource_id: resource_id,
+        action: action,
         changed_fields: changed_fields
       )
     end

spec/history_spec.cr

@@ -19,13 +19,15 @@ module PlaceOS::Model
       found.id.should eq history.id
       found.type.should eq history.type
       found.resource_id.should eq history.resource_id
+      found.action.should eq history.action
       found.changed_fields.should eq history.changed_fields
     end
 
     it "requires type" do
       history = History.new(
         type: "",
-        resource_id: "zone-123"
+        resource_id: "zone-123",
+        action: "update"
       )
       history.valid?.should be_false
       history.errors.first.field.should eq :type
@@ -34,7 +36,8 @@ module PlaceOS::Model
     it "requires resource_id" do
       history = History.new(
         type: "zone",
-        resource_id: ""
+        resource_id: "",
+        action: "update"
       )
       history.valid?.should be_false
       history.errors.first.field.should eq :resource_id
@@ -43,13 +46,39 @@ module PlaceOS::Model
     it "defaults changed_fields to empty array" do
       history = History.new(
         type: "zone",
-        resource_id: "zone-123"
+        resource_id: "zone-123",
+        action: "create"
       )
       history.save!
 
       history.changed_fields.should eq [] of String
     end
 
+    it "saves action field" do
+      history = History.new(
+        type: "zone",
+        resource_id: "zone-123",
+        action: "update",
+        changed_fields: ["name"]
+      )
+      history.save!
+
+      found = History.find!(history.id.as(String))
+      found.action.should eq "update"
+    end
+
+    it "supports different action types" do
+      ["create", "update", "delete"].each do |action|
+        history = History.new(
+          type: "zone",
+          resource_id: "zone-123",
+          action: action
+        )
+        history.save!
+        history.action.should eq action
+      end
+    end
+
     it "sets timestamps on create" do
       history = Generator.history.save!
 

src/placeos-models/history.cr

@@ -8,12 +8,14 @@ module PlaceOS::Model
 
     attribute type : String, es_subfield: "keyword"
     attribute resource_id : String, es_subfield: "keyword"
+    attribute action : String, es_subfield: "keyword"
     attribute changed_fields : Array(String) = [] of String, es_type: "keyword"
 
     # Validation
     ###############################################################################################
 
     validates :type, presence: true
     validates :resource_id, presence: true
+    validates :action, presence: true
   end
 end