Apply feedback from Ana's code review:

- Rename to `Keyword`
- Use an `array` field!
- validate!

app/furniture/journal/journal.rb

@@ -3,5 +3,5 @@ class Journal::Journal < Furniture
 
   extend StripsNamespaceFromModelName
   has_many :entries, inverse_of: :journal, dependent: :destroy
-  has_many :terms, 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("'?' = ANY (aliases)", keyword)
+          .or(where(canonical_keyword: keyword)).exists?
+
+        find_or_create_by!(canonical_keyword: keyword)
+      end
+    end
+  end
+end

app/furniture/journal/renderer.rb

@@ -8,7 +8,6 @@ def autolink(link, link_type)
     def postprocess(doc)
       doc
         .gsub(/@([a-zA-Z\d]*)@(.*\.[a-zA-Z]*)/, '<a href="https://\2/@\1">@\1@\2</a>')
-      # .gsub(/#(\w+)/, '<a href="../terms/\1">#\1</a>')
     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,13 +117,13 @@
     t.index ["journal_id"], name: "index_journal_entries_on_journal_id"
   end
 
-  create_table "journal_terms", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|
+  create_table "journal_keywords", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|
     t.uuid "journal_id"
-    t.string "canonical_term"
-    t.text "aliases"
+    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_terms_on_journal_id"
+    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|
@@ -285,7 +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_terms", "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/journal_spec.rb

@@ -4,5 +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(:terms).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