🥔✨ Journal: extract Terms from Entrys

app/furniture/journal/entry.rb

@@ -30,6 +30,7 @@ class Entry < ApplicationRecord
     belongs_to :journal, inverse_of: :entries
     has_one :room, through: :journal
     has_one :space, through: :journal
+    after_save :extract_keywords, if: :saved_change_to_body?
 
     def published?
       published_at.present?
@@ -50,6 +51,10 @@ def self.renderer
       )
     end
 
+    def extract_keywords
+      journal.keywords.extract_and_create_from!(body)
+    end
+
     def to_param
       slug
     end

app/furniture/journal/journal.rb

@@ -3,4 +3,5 @@ class Journal::Journal < Furniture
 
   extend StripsNamespaceFromModelName
   has_many :entries, inverse_of: :journal, dependent: :destroy
+  has_many :keywords, inverse_of: :journal, dependent: :destroy
 end

app/furniture/journal/keyword.rb

@@ -0,0 +1,18 @@
+class Journal
+  class Keyword < ApplicationRecord
+    location(parent: :journal)
+
+    self.table_name = "journal_keywords"
+    validates :canonical_keyword, presence: true, uniqueness: {case_sensitive: false, scope: :journal_id}
+    belongs_to :journal, inverse_of: :keywords
+
+    def self.extract_and_create_from!(body)
+      body.scan(/#(\w+)/)&.flatten&.each do |keyword|
+        next if where(":aliases = ANY (aliases)", aliases: keyword)
+          .or(where(canonical_keyword: keyword)).exists?
+
+        find_or_create_by!(canonical_keyword: keyword)
+      end
+    end
+  end
+end

db/migrate/20230704064303_add_journal_keywords.rb

@@ -0,0 +1,10 @@
+class AddJournalKeywords < ActiveRecord::Migration[7.0]
+  def change
+    create_table :journal_keywords, id: :uuid do |t|
+      t.references :journal, type: :uuid, foreign_key: {to_table: :furnitures}
+      t.string :canonical_keyword
+      t.string :aliases, array: true
+      t.timestamps
+    end
+  end
+end

db/schema.rb

@@ -117,6 +117,15 @@
     t.index ["journal_id"], name: "index_journal_entries_on_journal_id"
   end
 
+  create_table "journal_keywords", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|
+    t.uuid "journal_id"
+    t.string "canonical_keyword"
+    t.string "aliases", array: true
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["journal_id"], name: "index_journal_keywords_on_journal_id"
+  end
+
   create_table "marketplace_cart_products", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|
     t.uuid "cart_id"
     t.uuid "product_id"
@@ -276,6 +285,7 @@
   add_foreign_key "active_storage_attachments", "active_storage_blobs", column: "blob_id"
   add_foreign_key "active_storage_variant_records", "active_storage_blobs", column: "blob_id"
   add_foreign_key "journal_entries", "furnitures", column: "journal_id"
+  add_foreign_key "journal_keywords", "furnitures", column: "journal_id"
   add_foreign_key "marketplace_cart_products", "marketplace_orders", column: "cart_id"
   add_foreign_key "marketplace_cart_products", "marketplace_products", column: "product_id"
   add_foreign_key "marketplace_delivery_areas", "furnitures", column: "marketplace_id"

spec/furniture/journal/entry_spec.rb

@@ -13,13 +13,29 @@
   describe "#to_html" do
     subject(:to_html) { entry.to_html }
 
-    let(:entry) { build(:journal_entry, body: body) }
-
     context "when #body is 'https://www.google.com @[email protected]'" do
-      let(:body) { "https://www.google.com @[email protected]" }
+      let(:entry) { build(:journal_entry, body: "https://www.google.com @[email protected]") }
 
-      it { is_expected.to include('<a href="https://weirder.earth/@zee">@[email protected]</a>') }
       it { is_expected.to include('<a href="https://www.google.com">https://www.google.com</a>') }
     end
   end
+
+  describe "#save" do
+    let(:entry) { create(:journal_entry, body: "#GoodTimes") }
+    let(:journal) { entry.journal }
+
+    context "when the body is changing" do
+      it "idempotently creates `Keywords` in the `Journal`" do
+        bad_apple = entry.journal.keywords.create!(canonical_keyword: "BadApple", aliases: ["BadApples"])
+        good_times = entry.journal.keywords.find_by!(canonical_keyword: "GoodTimes")
+        expect do
+          entry.update!(body: "#GoodTimes #HardCider #BadApples")
+        end.not_to change { "#{bad_apple.reload.updated_at} - #{good_times.reload.updated_at}" }
+
+        expect(journal.keywords.where(canonical_keyword: "GoodTimes")).to exist
+        expect(journal.keywords.where(canonical_keyword: "HardCider")).to exist
+        expect(journal.keywords.where(canonical_keyword: "BadApples")).not_to exist
+      end
+    end
+  end
 end

spec/furniture/journal/journal_spec.rb

@@ -4,4 +4,5 @@
 
 RSpec.describe Journal::Journal, type: :model do
   it { is_expected.to have_many(:entries).inverse_of(:journal).dependent(:destroy) }
+  it { is_expected.to have_many(:keywords).inverse_of(:journal).dependent(:destroy) }
 end

spec/furniture/journal/keyword_spec.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe Journal::Keyword, type: :model do
+  it { is_expected.to validate_presence_of(:canonical_keyword) }
+  it { is_expected.to validate_uniqueness_of(:canonical_keyword).case_insensitive.scoped_to(:journal_id) }
+  it { is_expected.to belong_to(:journal).inverse_of(:keywords) }
+end