From 1e788de9a099b796f29bf1a86c08c68a59954847 Mon Sep 17 00:00:00 2001
From: miteruzo <miteruzo@naver.com>
Date: Mon, 9 Mar 2026 21:06:58 +0900
Subject: [PATCH 1/4] =?UTF-8?q?#281=20=E3=83=86=E3=82=B9=E3=83=88=E3=81=BE?=
 =?UTF-8?q?=E3=81=A0=E9=80=9A=E3=81=A3=E3=81=A6=E3=81=AA=E3=81=84=E3=81=AE?=
 =?UTF-8?q?=E3=81=A7=E8=A6=81=E7=A2=BA=E8=AA=8D?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 backend/app/models/tag.rb                     |  8 ++-
 backend/app/models/tag_name.rb                |  6 ++
 .../app/models/tag_name_sanitisation_rule.rb  | 55 ++++++++++++++++
 ...3200_create_tag_name_sanitisation_rules.rb | 31 +++++++++
 backend/db/schema.rb                          | 12 +++-
 backend/lib/tasks/sync_nico.rake              |  2 +-
 .../models/tag_name_sanitisation_rule_spec.rb | 63 +++++++++++++++++++
 backend/spec/models/tag_name_spec.rb          | 28 +++++++++
 backend/spec/models/tag_spec.rb               | 39 +++++++++---
 9 files changed, 232 insertions(+), 12 deletions(-)
 create mode 100644 backend/app/models/tag_name_sanitisation_rule.rb
 create mode 100644 backend/db/migrate/20260309123200_create_tag_name_sanitisation_rules.rb
 create mode 100644 backend/spec/models/tag_name_sanitisation_rule_spec.rb
 create mode 100644 backend/spec/models/tag_name_spec.rb

diff --git a/backend/app/models/tag.rb b/backend/app/models/tag.rb
index eaf51da..1fd85e6 100644
--- a/backend/app/models/tag.rb
+++ b/backend/app/models/tag.rb
@@ -160,9 +160,15 @@ class Tag < ApplicationRecord
         end
 
         tag_name = st.tag_name
+        nico_flg = st.nico?
         st.destroy!
         tag_name.reload
-        tag_name.update!(canonical: target_tag.tag_name)
+
+        if nico_flg
+          tag_name.destroy!
+        else
+          tag_name.update!(canonical: target_tag.tag_name)
+        end
       end
 
       # 投稿件数を再集計
diff --git a/backend/app/models/tag_name.rb b/backend/app/models/tag_name.rb
index 225343d..5b39a06 100644
--- a/backend/app/models/tag_name.rb
+++ b/backend/app/models/tag_name.rb
@@ -1,4 +1,6 @@
 class TagName < ApplicationRecord
+  before_validation :sanitise_name
+
   has_one :tag
   has_one :wiki_page
 
@@ -22,6 +24,10 @@ class TagName < ApplicationRecord
 
   private
 
+  def sanitise_name
+    self.name = TagNameSanitisationRule.sanitise(name)
+  end
+
   def canonical_must_be_canonical
     if canonical&.canonical_id?
       errors.add :canonical, 'canonical は実体を示す必要があります．'
diff --git a/backend/app/models/tag_name_sanitisation_rule.rb b/backend/app/models/tag_name_sanitisation_rule.rb
new file mode 100644
index 0000000..d0c8ea1
--- /dev/null
+++ b/backend/app/models/tag_name_sanitisation_rule.rb
@@ -0,0 +1,55 @@
+class TagNameSanitisationRule < ApplicationRecord
+  include Discard::Model
+
+  self.primary_key = :priority
+
+  default_scope -> { kept }
+
+  validates :source_pattern, presence: true, uniqueness: true
+
+  validate :source_pattern_must_be_regexp
+
+  class << self
+    def sanitise(name) =
+        rules.reduce(name.dup) { |name, (pattern, replacement)| name.gsub(pattern, replacement) }
+
+    def apply!
+      TagName.find_each do |tn|
+        name = sanitise(tn.name)
+        next if name == tn.name
+
+        TagName.transaction do
+          existing_tn = TagName.find_by(name:)
+          if existing_tn
+            existing_tn = existing_tn.canonical || existing_tn
+            next if existing_tn.id == tn.id
+
+            if existing_tn.tag
+              Tag.merge_tags!(existing_tn.tag, tn.tag) if tn.tag
+            elsif tn.tag
+              tn.tag.update_columns(tag_name_id: existing_tn.id, updated_at: Time.current)
+            end
+            tn.destroy!
+
+            next
+          end
+
+          # TagName 側の自動サニタイズを回避
+          tn.update_columns(name:, updated_at: Time.current)
+        end
+      end
+    end
+
+    private
+
+    def rules = kept.order(:priority).map { |r| [Regexp.new(r.source_pattern), r.replacement] }
+  end
+
+  private
+
+  def source_pattern_must_be_regexp
+    Regexp.new(source_pattern)
+  rescue RegexpError
+    errors.add :source_pattern, '変な正規表現だね〜（笑）'
+  end
+end
diff --git a/backend/db/migrate/20260309123200_create_tag_name_sanitisation_rules.rb b/backend/db/migrate/20260309123200_create_tag_name_sanitisation_rules.rb
new file mode 100644
index 0000000..e187ce9
--- /dev/null
+++ b/backend/db/migrate/20260309123200_create_tag_name_sanitisation_rules.rb
@@ -0,0 +1,31 @@
+class CreateTagNameSanitisationRules < ActiveRecord::Migration[8.0]
+  def up
+    create_table :tag_name_sanitisation_rules, id: :integer, primary_key: :priority do |t|
+      t.string :source_pattern, null: false
+      t.string :replacement, null: false
+      t.timestamps
+      t.datetime :discarded_at
+      t.index :source_pattern, unique: true
+      t.index :discarded_at
+    end
+
+    now = ActiveRecord::Base.connection.quote(Time.current)
+    execute <<~SQL
+      INSERT INTO
+        tag_name_sanitisation_rules(priority, source_pattern, replacement, created_at, updated_at)
+      VALUES
+        (10, '\\\\*',           '_',         #{ now }, #{ now })
+      , (20, '\\\\?',           '_',         #{ now }, #{ now })
+      , (25, '\\\\/',           '_',         #{ now }, #{ now })
+      , (30, '_+',              '_',         #{ now }, #{ now })
+      , (40, '_$',              '',          #{ now }, #{ now })
+      , (45, '^([^:]+\\\\:)?_', '\\\\1',     #{ now }, #{ now })
+      , (50, '^([^:]+\\\\:)?$', '\\\\1null', #{ now }, #{ now })
+      ;
+    SQL
+  end
+
+  def down
+    drop_table :tag_name_sanitisation_rules
+  end
+end
diff --git a/backend/db/schema.rb b/backend/db/schema.rb
index 8077fac..58a7158 100644
--- a/backend/db/schema.rb
+++ b/backend/db/schema.rb
@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[8.0].define(version: 2026_03_03_122700) do
+ActiveRecord::Schema[8.0].define(version: 2026_03_09_123200) do
   create_table "active_storage_attachments", charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
     t.string "name", null: false
     t.string "record_type", null: false
@@ -127,6 +127,16 @@ ActiveRecord::Schema[8.0].define(version: 2026_03_03_122700) do
     t.index ["tag_id"], name: "index_tag_implications_on_tag_id"
   end
 
+  create_table "tag_name_sanitisation_rules", primary_key: "priority", id: :integer, charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
+    t.string "source_pattern", null: false
+    t.string "replacement", null: false
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.datetime "discarded_at"
+    t.index ["discarded_at"], name: "index_tag_name_sanitisation_rules_on_discarded_at"
+    t.index ["source_pattern"], name: "index_tag_name_sanitisation_rules_on_source_pattern", unique: true
+  end
+
   create_table "tag_names", charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
     t.string "name", null: false
     t.bigint "canonical_id"
diff --git a/backend/lib/tasks/sync_nico.rake b/backend/lib/tasks/sync_nico.rake
index 6ee9c26..09be474 100644
--- a/backend/lib/tasks/sync_nico.rake
+++ b/backend/lib/tasks/sync_nico.rake
@@ -108,7 +108,7 @@ namespace :nico do
       desired_non_nico_tag_ids = []
 
       datum['tags'].each do |raw|
-        name = "nico:#{ raw }"
+        name = TagNameSanitisationRule.sanitise("nico:#{ raw }")
         tag = Tag.find_or_create_by_tag_name!(name, category: :nico)
         desired_nico_tag_based_ids << tag.id
 
diff --git a/backend/spec/models/tag_name_sanitisation_rule_spec.rb b/backend/spec/models/tag_name_sanitisation_rule_spec.rb
new file mode 100644
index 0000000..9a95c3b
--- /dev/null
+++ b/backend/spec/models/tag_name_sanitisation_rule_spec.rb
@@ -0,0 +1,63 @@
+require 'rails_helper'
+
+RSpec.describe TagNameSanitisationRule, type: :model do
+  describe '.sanitise' do
+    before do
+      described_class.create!(priority: 10, source_pattern: '_', replacement: '')
+      described_class.create!(priority: 20, source_pattern: 'ABC', replacement: 'abc')
+    end
+
+    it 'applies rules in priority order' do
+      expect(described_class.sanitise('A_B_C')).to eq('abc')
+    end
+
+    it 'does not fail when a rule does not match' do
+      expect { described_class.sanitise('xyz') }.not_to raise_error
+      expect(described_class.sanitise('xyz')).to eq('xyz')
+    end
+  end
+
+  describe 'validations' do
+    it 'is invalid when source_pattern is not a valid regexp' do
+      rule = described_class.new(priority: 10, source_pattern: '[', replacement: '')
+      expect(rule).to be_invalid
+      expect(rule.errors[:source_pattern]).to be_present
+    end
+  end
+
+  describe '.apply!' do
+    before do
+      described_class.create!(priority: 10, source_pattern: '_', replacement: '')
+    end
+
+    context 'when no conflicting tag_name exists' do
+      let!(:tag_name) { TagName.create!(name: 'foo_bar') }
+
+      it 'renames the tag_name' do
+        described_class.apply!
+        expect(tag_name.reload.name).to eq('foobar')
+      end
+    end
+
+    context 'when a conflicting canonical tag_name exists' do
+      let!(:existing) { TagName.create!(name: 'foobar') }
+      let!(:source)   { TagName.create!(name: 'foo_bar') }
+
+      it 'deletes the source tag_name' do
+        described_class.apply!
+        expect(TagName.exists?(source.id)).to be(false)
+        expect(existing.reload.name).to eq('foobar')
+      end
+    end
+
+    context 'when the source tag_name has a tag and the existing one has no tag' do
+      let!(:existing) { TagName.create!(name: 'foobar') }
+      let!(:source_tag) { create(:tag, name: 'foo_bar', category: :general) }
+
+      it 'moves the tag to the existing tag_name' do
+        described_class.apply!
+        expect(source_tag.reload.tag_name_id).to eq(existing.id)
+      end
+    end
+  end
+end
diff --git a/backend/spec/models/tag_name_spec.rb b/backend/spec/models/tag_name_spec.rb
new file mode 100644
index 0000000..b89583c
--- /dev/null
+++ b/backend/spec/models/tag_name_spec.rb
@@ -0,0 +1,28 @@
+require 'rails_helper'
+
+RSpec.describe TagName, type: :model do
+  describe 'before_validation :sanitise_name' do
+    before do
+      TagNameSanitisationRule.create!(priority: 10, source_pattern: '_', replacement: '')
+    end
+
+    it 'sanitises name on create' do
+      tag_name = TagName.create!(name: 'foo_bar')
+      expect(tag_name.reload.name).to eq('foobar')
+    end
+
+    it 'sanitises name on update' do
+      tag_name = TagName.create!(name: 'foobar')
+      tag_name.update!(name: 'foo_bar')
+      expect(tag_name.reload.name).to eq('foobar')
+    end
+
+    it 'becomes invalid when sanitised name conflicts with another tag_name' do
+      TagName.create!(name: 'foobar')
+      tag_name = TagName.new(name: 'foo_bar')
+
+      expect(tag_name).to be_invalid
+      expect(tag_name.errors[:name]).to be_present
+    end
+  end
+end
diff --git a/backend/spec/models/tag_spec.rb b/backend/spec/models/tag_spec.rb
index 389f1a1..b63710a 100644
--- a/backend/spec/models/tag_spec.rb
+++ b/backend/spec/models/tag_spec.rb
@@ -2,10 +2,13 @@ require 'rails_helper'
 
 RSpec.describe Tag, type: :model do
   describe '.merge_tags!' do
-    let!(:target_tag) { create(:tag) }
-    let!(:source_tag) { create(:tag) }
+    let!(:target_tag) { create(:tag, category: :general) }
+    let!(:source_tag) { create(:tag, category: :general) }
+    let!(:source_tag_name) { source_tag.tag_name }
 
-    let!(:post_record) { Post.create!(url: 'https://example.com/posts/1', title: 'test post') }
+    let!(:post_record) do
+      Post.create!(url: 'https://example.com/posts/1', title: 'test post')
+    end
 
     context 'when merging a simple source tag' do
       let!(:source_post_tag) { PostTag.create!(post: post_record, tag: source_tag) }
@@ -15,7 +18,8 @@ RSpec.describe Tag, type: :model do
 
         expect(source_post_tag.reload.tag_id).to eq(target_tag.id)
         expect(Tag.exists?(source_tag.id)).to be(false)
-        expect(source_tag.tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
+        expect(source_tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
+        expect(target_tag.reload.post_count).to eq(1)
       end
     end
 
@@ -31,8 +35,10 @@ RSpec.describe Tag, type: :model do
 
         expect(active.count).to eq(1)
         expect(discarded_source.discarded_at).to be_present
+        expect(discarded_source.tag_id).to eq(source_tag.id)
         expect(Tag.exists?(source_tag.id)).to be(false)
-        expect(source_tag.tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
+        expect(source_tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
+        expect(target_tag.reload.post_count).to eq(1)
       end
     end
 
@@ -45,6 +51,7 @@ RSpec.describe Tag, type: :model do
         expect(Tag.exists?(target_tag.id)).to be(true)
         expect(Tag.exists?(source_tag.id)).to be(false)
         expect(source_post_tag.reload.tag_id).to eq(target_tag.id)
+        expect(target_tag.reload.post_count).to eq(1)
       end
     end
 
@@ -52,9 +59,10 @@ RSpec.describe Tag, type: :model do
       let!(:source_post_tag) { PostTag.create!(post: post_record, tag: source_tag) }
       let!(:wiki_page) do
         WikiPage.create!(
-            tag_name: source_tag.tag_name,
-            created_user: create_admin_user!,
-            updated_user: create_admin_user!)
+          tag_name: source_tag_name,
+          created_user: create_admin_user!,
+          updated_user: create_admin_user!
+        )
       end
 
       it 'rolls back the transaction' do
@@ -64,7 +72,20 @@ RSpec.describe Tag, type: :model do
 
         expect(Tag.exists?(source_tag.id)).to be(true)
         expect(source_post_tag.reload.tag_id).to eq(source_tag.id)
-        expect(source_tag.tag_name.reload.canonical_id).to be_nil
+        expect(source_tag_name.reload.canonical_id).to be_nil
+      end
+    end
+
+    context 'when merging a nico source tag' do
+      let!(:target_tag) { create(:tag, category: :nico, name: 'nico:foo') }
+      let!(:source_tag) { create(:tag, category: :nico, name: 'nico:bar') }
+      let!(:source_tag_name_id) { source_tag.tag_name_id }
+
+      it 'deletes the source tag_name instead of aliasing it' do
+        described_class.merge_tags!(target_tag, [source_tag])
+
+        expect(Tag.exists?(source_tag.id)).to be(false)
+        expect(TagName.exists?(source_tag_name_id)).to be(false)
       end
     end
   end
-- 
2.34.1


From 474e0695babd3f25cde42eb2e6aa2d41b9f2d48e Mon Sep 17 00:00:00 2001
From: miteruzo <miteruzo@naver.com>
Date: Tue, 10 Mar 2026 22:35:08 +0900
Subject: [PATCH 2/4] #281

---
 backend/app/models/tag.rb                     |  9 ++--
 backend/app/models/tag_name.rb                |  9 +++-
 backend/spec/factories/tags.rb                | 12 ++++-
 .../models/tag_name_sanitisation_rule_spec.rb | 45 +++++++++++++++++--
 backend/spec/models/tag_name_spec.rb          | 28 ------------
 backend/spec/models/tag_spec.rb               | 18 ++++----
 6 files changed, 72 insertions(+), 49 deletions(-)
 delete mode 100644 backend/spec/models/tag_name_spec.rb

diff --git a/backend/app/models/tag.rb b/backend/app/models/tag.rb
index 1fd85e6..e8aee64 100644
--- a/backend/app/models/tag.rb
+++ b/backend/app/models/tag.rb
@@ -152,11 +152,9 @@ class Tag < ApplicationRecord
         st.post_tags.find_each do |pt|
           if PostTag.kept.exists?(post_id: pt.post_id, tag_id: target_tag.id)
             pt.discard_by!(nil)
-            # discard 後の update! は禁止なので DB を直に更新
-            pt.update_columns(tag_id: target_tag.id, updated_at: Time.current)
-          else
-            pt.update!(tag: target_tag)
           end
+          # discard 後の update! は禁止なので DB を直に更新
+          pt.update_columns(tag_id: target_tag.id, updated_at: Time.current)
         end
 
         tag_name = st.tag_name
@@ -167,7 +165,8 @@ class Tag < ApplicationRecord
         if nico_flg
           tag_name.destroy!
         else
-          tag_name.update!(canonical: target_tag.tag_name)
+          tag_name.update_columns(canonical_id: target_tag.tag_name&.id,
+                                  updated_at: Time.current)
         end
       end
 
diff --git a/backend/app/models/tag_name.rb b/backend/app/models/tag_name.rb
index 5b39a06..239f932 100644
--- a/backend/app/models/tag_name.rb
+++ b/backend/app/models/tag_name.rb
@@ -1,6 +1,4 @@
 class TagName < ApplicationRecord
-  before_validation :sanitise_name
-
   has_one :tag
   has_one :wiki_page
 
@@ -12,6 +10,7 @@ class TagName < ApplicationRecord
   validate :canonical_must_be_canonical
   validate :alias_name_must_not_have_prefix
   validate :canonical_must_not_be_present_with_tag_or_wiki_page
+  validate :name_must_be_sanitised
 
   def self.canonicalise names
     names = Array(names).map { |n| n.to_s.strip }.reject(&:blank?)
@@ -45,4 +44,10 @@ class TagName < ApplicationRecord
       errors.add :canonical, 'タグもしくは Wiki の参照がある名前はエーリアスになれません．'
     end
   end
+
+  def name_must_be_sanitised
+    if name != TagNameSanitisationRule.sanitise(name)
+      errors.add :name, '名前に使用できない文字が含まれてゐます．'
+    end
+  end
 end
diff --git a/backend/spec/factories/tags.rb b/backend/spec/factories/tags.rb
index e2d1120..9da4b9d 100644
--- a/backend/spec/factories/tags.rb
+++ b/backend/spec/factories/tags.rb
@@ -1,12 +1,22 @@
 FactoryBot.define do
   factory :tag do
+    transient do
+      name { nil }
+    end
+
     category { :general }
     post_count { 0 }
     association :tag_name
 
+    after(:build) do |tag, evaluator|
+      tag.name = evaluator.name if evaluator.name.present?
+    end
+
     trait :nico do
       category { :nico }
-      tag_name { association(:tag_name, name: "nico:#{ SecureRandom.hex(4) }") }
+      transient do
+        name { "nico:#{ SecureRandom.hex(4) }" }
+      end
     end
   end
 end
diff --git a/backend/spec/models/tag_name_sanitisation_rule_spec.rb b/backend/spec/models/tag_name_sanitisation_rule_spec.rb
index 9a95c3b..bf90acb 100644
--- a/backend/spec/models/tag_name_sanitisation_rule_spec.rb
+++ b/backend/spec/models/tag_name_sanitisation_rule_spec.rb
@@ -7,7 +7,7 @@ RSpec.describe TagNameSanitisationRule, type: :model do
       described_class.create!(priority: 20, source_pattern: 'ABC', replacement: 'abc')
     end
 
-    it 'applies rules in priority order' do
+    it 'applies sanitisation rules sequentially in priority order' do
       expect(described_class.sanitise('A_B_C')).to eq('abc')
     end
 
@@ -31,7 +31,11 @@ RSpec.describe TagNameSanitisationRule, type: :model do
     end
 
     context 'when no conflicting tag_name exists' do
-      let!(:tag_name) { TagName.create!(name: 'foo_bar') }
+      let!(:tag_name) do
+        TagName.create!(name: 'tmp').tap do |tn|
+          tn.update_columns(name: 'foo_bar', updated_at: Time.current)
+        end
+      end
 
       it 'renames the tag_name' do
         described_class.apply!
@@ -41,7 +45,11 @@ RSpec.describe TagNameSanitisationRule, type: :model do
 
     context 'when a conflicting canonical tag_name exists' do
       let!(:existing) { TagName.create!(name: 'foobar') }
-      let!(:source)   { TagName.create!(name: 'foo_bar') }
+      let!(:source) do
+        TagName.create!(name: 'tmp').tap do |tn|
+          tn.update_columns(name: 'foo_bar', updated_at: Time.current)
+        end
+      end
 
       it 'deletes the source tag_name' do
         described_class.apply!
@@ -52,11 +60,40 @@ RSpec.describe TagNameSanitisationRule, type: :model do
 
     context 'when the source tag_name has a tag and the existing one has no tag' do
       let!(:existing) { TagName.create!(name: 'foobar') }
-      let!(:source_tag) { create(:tag, name: 'foo_bar', category: :general) }
+      let!(:source_tag) { create(:tag, name: 'tmp', category: :general) }
+      let!(:source_tag_name_id) { source_tag.tag_name_id }
+
+      before do
+        source_tag.tag_name.update_columns(name: 'foo_bar', updated_at: Time.current)
+      end
 
       it 'moves the tag to the existing tag_name' do
         described_class.apply!
         expect(source_tag.reload.tag_name_id).to eq(existing.id)
+        expect(TagName.exists?(source_tag_name_id)).to be(false)
+      end
+    end
+
+    context 'when both source and existing tag_names have tags' do
+      let!(:existing_tn) { TagName.create!(name: 'foobar') }
+      let!(:existing_tag) { Tag.create!(tag_name: existing_tn, category: :general) }
+
+      let!(:source_tn) { TagName.create!(name: 'tmp') }
+      let!(:source_tag) { Tag.create!(tag_name: source_tn, category: :general) }
+      let!(:source_tag_name_id) { source_tn.id }
+
+      before do
+        source_tn.update_columns(name: 'foo_bar', updated_at: Time.current)
+      end
+
+      it 'merges the source tag into the existing tag and deletes the source tag_name' do
+        expect(TagName.find_by(name: 'foobar')&.tag&.id).to eq(existing_tag.id)
+        expect(TagName.find_by(name: 'foo_bar')&.tag&.id).to eq(source_tag.id)
+
+        described_class.apply!
+
+        expect(Tag.exists?(source_tag.id)).to be(false)
+        expect(TagName.exists?(source_tag.tag_name_id)).to be(false)
       end
     end
   end
diff --git a/backend/spec/models/tag_name_spec.rb b/backend/spec/models/tag_name_spec.rb
deleted file mode 100644
index b89583c..0000000
--- a/backend/spec/models/tag_name_spec.rb
+++ /dev/null
@@ -1,28 +0,0 @@
-require 'rails_helper'
-
-RSpec.describe TagName, type: :model do
-  describe 'before_validation :sanitise_name' do
-    before do
-      TagNameSanitisationRule.create!(priority: 10, source_pattern: '_', replacement: '')
-    end
-
-    it 'sanitises name on create' do
-      tag_name = TagName.create!(name: 'foo_bar')
-      expect(tag_name.reload.name).to eq('foobar')
-    end
-
-    it 'sanitises name on update' do
-      tag_name = TagName.create!(name: 'foobar')
-      tag_name.update!(name: 'foo_bar')
-      expect(tag_name.reload.name).to eq('foobar')
-    end
-
-    it 'becomes invalid when sanitised name conflicts with another tag_name' do
-      TagName.create!(name: 'foobar')
-      tag_name = TagName.new(name: 'foo_bar')
-
-      expect(tag_name).to be_invalid
-      expect(tag_name.errors[:name]).to be_present
-    end
-  end
-end
diff --git a/backend/spec/models/tag_spec.rb b/backend/spec/models/tag_spec.rb
index b63710a..fdec509 100644
--- a/backend/spec/models/tag_spec.rb
+++ b/backend/spec/models/tag_spec.rb
@@ -35,7 +35,7 @@ RSpec.describe Tag, type: :model do
 
         expect(active.count).to eq(1)
         expect(discarded_source.discarded_at).to be_present
-        expect(discarded_source.tag_id).to eq(source_tag.id)
+        expect(discarded_source.tag_id).to eq(target_tag.id)
         expect(Tag.exists?(source_tag.id)).to be(false)
         expect(source_tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
         expect(target_tag.reload.post_count).to eq(1)
@@ -55,7 +55,7 @@ RSpec.describe Tag, type: :model do
       end
     end
 
-    context 'when aliasing the source tag_name is invalid' do
+    context 'when aliasing the source tag_name would be invalid under normal validation' do
       let!(:source_post_tag) { PostTag.create!(post: post_record, tag: source_tag) }
       let!(:wiki_page) do
         WikiPage.create!(
@@ -65,14 +65,13 @@ RSpec.describe Tag, type: :model do
         )
       end
 
-      it 'rolls back the transaction' do
-        expect {
-          described_class.merge_tags!(target_tag, [source_tag])
-        }.to raise_error(ActiveRecord::RecordInvalid)
+      it 'still merges by bypassing validations' do
+        described_class.merge_tags!(target_tag, [source_tag])
 
-        expect(Tag.exists?(source_tag.id)).to be(true)
-        expect(source_post_tag.reload.tag_id).to eq(source_tag.id)
-        expect(source_tag_name.reload.canonical_id).to be_nil
+        expect(Tag.exists?(source_tag.id)).to be(false)
+        expect(source_post_tag.reload.tag_id).to eq(target_tag.id)
+        expect(source_tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
+        expect(source_tag_name.reload).not_to be_valid
       end
     end
 
@@ -86,6 +85,7 @@ RSpec.describe Tag, type: :model do
 
         expect(Tag.exists?(source_tag.id)).to be(false)
         expect(TagName.exists?(source_tag_name_id)).to be(false)
+        expect(target_tag.reload.post_count).to eq(0)
       end
     end
   end
-- 
2.34.1


From 2e2b99f74b02240fafd720fcbb19849057af6254 Mon Sep 17 00:00:00 2001
From: miteruzo <miteruzo@naver.com>
Date: Thu, 12 Mar 2026 00:46:21 +0900
Subject: [PATCH 3/4] #281

---
 backend/app/models/tag.rb                     | 14 ++-
 backend/app/models/tag_name.rb                |  4 +
 .../app/models/tag_name_sanitisation_rule.rb  | 13 ++-
 backend/app/models/wiki_page.rb               |  4 +
 ...311232100_add_discarded_at_to_tag_names.rb |  6 ++
 ...11232300_add_discarded_at_to_wiki_pages.rb |  6 ++
 backend/db/schema.rb                          |  6 +-
 .../models/tag_name_sanitisation_rule_spec.rb |  3 +-
 backend/spec/models/tag_spec.rb               | 91 ++++++++++++-------
 9 files changed, 104 insertions(+), 43 deletions(-)
 create mode 100644 backend/db/migrate/20260311232100_add_discarded_at_to_tag_names.rb
 create mode 100644 backend/db/migrate/20260311232300_add_discarded_at_to_wiki_pages.rb

diff --git a/backend/app/models/tag.rb b/backend/app/models/tag.rb
index 3a216b6..9f480cd 100644
--- a/backend/app/models/tag.rb
+++ b/backend/app/models/tag.rb
@@ -162,14 +162,18 @@ class Tag < ApplicationRecord
         end
 
         source_tag_name = source_tag.tag_name
-        nico_flg = source_tag.nico?
+
+        if source_tag_name.wiki_page.present?
+          raise ActiveRecord::RecordInvalid.new(source_tag_name)
+        end
+
         source_tag.discard!
-        source_tag_name.reload
 
-        if nico_flg
-          source_tag_name.destroy!
+        if source_tag.nico?
+          source_tag_name.discard!
         else
-          source_tag_name.update!(canonical: target_tag.tag_name)
+          source_tag_name.update_columns(canonical_id: target_tag.tag_name_id,
+                                         updated_at: Time.current)
         end
       end
 
diff --git a/backend/app/models/tag_name.rb b/backend/app/models/tag_name.rb
index 239f932..c353d2f 100644
--- a/backend/app/models/tag_name.rb
+++ b/backend/app/models/tag_name.rb
@@ -1,4 +1,8 @@
 class TagName < ApplicationRecord
+  include Discard::Model
+
+  default_scope -> { kept }
+
   has_one :tag
   has_one :wiki_page
 
diff --git a/backend/app/models/tag_name_sanitisation_rule.rb b/backend/app/models/tag_name_sanitisation_rule.rb
index d0c8ea1..e37738a 100644
--- a/backend/app/models/tag_name_sanitisation_rule.rb
+++ b/backend/app/models/tag_name_sanitisation_rule.rb
@@ -24,12 +24,15 @@ class TagNameSanitisationRule < ApplicationRecord
             existing_tn = existing_tn.canonical || existing_tn
             next if existing_tn.id == tn.id
 
-            if existing_tn.tag
-              Tag.merge_tags!(existing_tn.tag, tn.tag) if tn.tag
-            elsif tn.tag
-              tn.tag.update_columns(tag_name_id: existing_tn.id, updated_at: Time.current)
+            existing_tag = Tag.find_by(tag_name_id: existing_tn.id)
+            source_tag = Tag.find_by(tag_name_id: tn.id)
+
+            if existing_tag
+              Tag.merge_tags!(existing_tag, source_tag) if tn.tag
+            elsif source_tag
+              source_tag.update_columns(tag_name_id: existing_tn.id, updated_at: Time.current)
             end
-            tn.destroy!
+            tn.discard!
 
             next
           end
diff --git a/backend/app/models/wiki_page.rb b/backend/app/models/wiki_page.rb
index d795a8e..b1d1b02 100644
--- a/backend/app/models/wiki_page.rb
+++ b/backend/app/models/wiki_page.rb
@@ -2,6 +2,10 @@ require 'set'
 
 
 class WikiPage < ApplicationRecord
+  include Discard::Model
+
+  default_scope -> { kept }
+
   has_many :wiki_revisions, dependent: :destroy
   belongs_to :created_user, class_name: 'User'
   belongs_to :updated_user, class_name: 'User'
diff --git a/backend/db/migrate/20260311232100_add_discarded_at_to_tag_names.rb b/backend/db/migrate/20260311232100_add_discarded_at_to_tag_names.rb
new file mode 100644
index 0000000..b161873
--- /dev/null
+++ b/backend/db/migrate/20260311232100_add_discarded_at_to_tag_names.rb
@@ -0,0 +1,6 @@
+class AddDiscardedAtToTagNames < ActiveRecord::Migration[8.0]
+  def change
+    add_column :tag_names, :discarded_at, :datetime
+    add_index :tag_names, :discarded_at
+  end
+end
diff --git a/backend/db/migrate/20260311232300_add_discarded_at_to_wiki_pages.rb b/backend/db/migrate/20260311232300_add_discarded_at_to_wiki_pages.rb
new file mode 100644
index 0000000..26435e0
--- /dev/null
+++ b/backend/db/migrate/20260311232300_add_discarded_at_to_wiki_pages.rb
@@ -0,0 +1,6 @@
+class AddDiscardedAtToWikiPages < ActiveRecord::Migration[8.0]
+  def change
+    add_column :wiki_pages, :discarded_at, :datetime
+    add_index :wiki_pages, :discarded_at
+  end
+end
diff --git a/backend/db/schema.rb b/backend/db/schema.rb
index b6ffbd3..b6eb1f3 100644
--- a/backend/db/schema.rb
+++ b/backend/db/schema.rb
@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[8.0].define(version: 2026_03_11_123100) do
+ActiveRecord::Schema[8.0].define(version: 2026_03_11_232300) do
   create_table "active_storage_attachments", charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
     t.string "name", null: false
     t.string "record_type", null: false
@@ -142,7 +142,9 @@ ActiveRecord::Schema[8.0].define(version: 2026_03_11_123100) do
     t.bigint "canonical_id"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
+    t.datetime "discarded_at"
     t.index ["canonical_id"], name: "index_tag_names_on_canonical_id"
+    t.index ["discarded_at"], name: "index_tag_names_on_discarded_at"
     t.index ["name"], name: "index_tag_names_on_name", unique: true
   end
 
@@ -204,7 +206,9 @@ ActiveRecord::Schema[8.0].define(version: 2026_03_11_123100) do
     t.bigint "updated_user_id", null: false
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
+    t.datetime "discarded_at"
     t.index ["created_user_id"], name: "index_wiki_pages_on_created_user_id"
+    t.index ["discarded_at"], name: "index_wiki_pages_on_discarded_at"
     t.index ["tag_name_id"], name: "index_wiki_pages_on_tag_name_id", unique: true
     t.index ["updated_user_id"], name: "index_wiki_pages_on_updated_user_id"
   end
diff --git a/backend/spec/models/tag_name_sanitisation_rule_spec.rb b/backend/spec/models/tag_name_sanitisation_rule_spec.rb
index bf90acb..654f85f 100644
--- a/backend/spec/models/tag_name_sanitisation_rule_spec.rb
+++ b/backend/spec/models/tag_name_sanitisation_rule_spec.rb
@@ -69,7 +69,8 @@ RSpec.describe TagNameSanitisationRule, type: :model do
 
       it 'moves the tag to the existing tag_name' do
         described_class.apply!
-        expect(source_tag.reload.tag_name_id).to eq(existing.id)
+        expected_tag_name_id = existing.canonical_id || existing.id
+        expect(source_tag.reload.tag_name_id).to eq(expected_tag_name_id)
         expect(TagName.exists?(source_tag_name_id)).to be(false)
       end
     end
diff --git a/backend/spec/models/tag_spec.rb b/backend/spec/models/tag_spec.rb
index 83583db..a9fc35e 100644
--- a/backend/spec/models/tag_spec.rb
+++ b/backend/spec/models/tag_spec.rb
@@ -13,14 +13,9 @@ RSpec.describe Tag, type: :model do
     context 'when merging a simple source tag' do
       let!(:source_post_tag) { PostTag.create!(post: post_record, tag: source_tag) }
 
-      it 'discards the source post_tag, creates an active target post_tag, ' +
-         'discards the source tag, and aliases the source tag_name' do
+      it 'discards the source post_tag, creates an active target post_tag, discards the source tag, and aliases the source tag_name' do
         described_class.merge_tags!(target_tag, [source_tag])
 
-        expect(Tag.exists?(source_tag.id)).to be(false)
-        expect(source_tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
-        expect(target_tag.reload.post_count).to eq(1)
-
         source_pt = PostTag.with_discarded.find(source_post_tag.id)
         active_target = PostTag.kept.find_by(post_id: post_record.id, tag_id: target_tag.id)
 
@@ -29,7 +24,9 @@ RSpec.describe Tag, type: :model do
         expect(active_target).to be_present
 
         expect(Tag.with_discarded.find(source_tag.id)).to be_discarded
-        expect(source_tag.tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
+        expect(TagName.with_discarded.find(source_tag_name.id)).not_to be_discarded
+        expect(source_tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
+        expect(target_tag.reload.post_count).to eq(1)
       end
     end
 
@@ -37,8 +34,7 @@ RSpec.describe Tag, type: :model do
       let!(:target_post_tag) { PostTag.create!(post: post_record, tag: target_tag) }
       let!(:source_post_tag) { PostTag.create!(post: post_record, tag: source_tag) }
 
-      it 'discards the source post_tag, keeps one active target post_tag, ' +
-         'and aliases the source tag_name' do
+      it 'discards the source post_tag, keeps one active target post_tag, discards the source tag, and aliases the source tag_name' do
         described_class.merge_tags!(target_tag, [source_tag])
 
         source_pt = PostTag.with_discarded.find(source_post_tag.id)
@@ -47,17 +43,12 @@ RSpec.describe Tag, type: :model do
         expect(source_pt.discarded_at).to be_present
         expect(source_pt.tag_id).to eq(source_tag.id)
         expect(active.count).to eq(1)
-
-        expect(source_pt.discarded_at).to be_present
-        expect(source_pt.tag_id).to eq(source_tag.id)
-        expect(Tag.exists?(source_tag.id)).to be(false)
-        expect(source_tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
-        expect(target_tag.reload.post_count).to eq(1)
-
         expect(active.first.id).to eq(target_post_tag.id)
 
         expect(Tag.with_discarded.find(source_tag.id)).to be_discarded
-        expect(source_tag.tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
+        expect(TagName.with_discarded.find(source_tag_name.id)).not_to be_discarded
+        expect(source_tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
+        expect(target_tag.reload.post_count).to eq(1)
       end
     end
 
@@ -67,10 +58,6 @@ RSpec.describe Tag, type: :model do
       it 'ignores the target in source_tags while still merging the source tag' do
         described_class.merge_tags!(target_tag, [source_tag, target_tag])
 
-        expect(Tag.exists?(target_tag.id)).to be(true)
-        expect(Tag.exists?(source_tag.id)).to be(false)
-        expect(target_tag.reload.post_count).to eq(1)
-
         source_pt = PostTag.with_discarded.find(source_post_tag.id)
         active_target = PostTag.kept.find_by(post_id: post_record.id, tag_id: target_tag.id)
 
@@ -79,23 +66,65 @@ RSpec.describe Tag, type: :model do
         expect(source_pt.discarded_at).to be_present
         expect(source_pt.tag_id).to eq(source_tag.id)
         expect(active_target).to be_present
+        expect(source_tag_name.reload.canonical_id).to eq(target_tag.tag_name_id)
+        expect(target_tag.reload.post_count).to eq(1)
+      end
+    end
+
+    context 'when the source tag_name is invalid under sanitisation rules' do
+      let!(:source_post_tag) { PostTag.create!(post: post_record, tag: source_tag) }
+      let!(:sanitisation_rule) do
+        TagNameSanitisationRule.create!(
+          priority: 99_999,
+          source_pattern: 'INVALIDTOKEN',
+          replacement: ''
+        )
+      end
+
+      before do
+        source_tag_name.update_columns(
+          name: "#{ source_tag_name.name }INVALIDTOKEN",
+          updated_at: Time.current
+        )
+      end
+
+      it 'still merges, but discards the source tag_name instead of aliasing it' do
+        described_class.merge_tags!(target_tag, [source_tag])
+
+        source_pt = PostTag.with_discarded.find(source_post_tag.id)
+        active_target = PostTag.kept.find_by(post_id: post_record.id, tag_id: target_tag.id)
+        discarded_source_tag_name = TagName.with_discarded.find(source_tag_name.id)
+
+        expect(source_pt.discarded_at).to be_present
+        expect(source_pt.tag_id).to eq(source_tag.id)
+        expect(active_target).to be_present
+
+        expect(Tag.with_discarded.find(source_tag.id)).to be_discarded
+        expect(target_tag.reload.post_count).to eq(1)
       end
     end
 
-    context 'when aliasing the source tag_name would be invalid under normal validation' do
+    context 'when the source tag_name has a wiki_page' do
       let!(:source_post_tag) { PostTag.create!(post: post_record, tag: source_tag) }
       let!(:wiki_page) do
         WikiPage.create!(
-          tag_name: source_tag.tag_name,
+          tag_name: source_tag_name,
           created_user: create_admin_user!,
           updated_user: create_admin_user!
         )
       end
 
-      it 'still merges by bypassing validations' do
+      it 'rolls back the transaction' do
         expect {
           described_class.merge_tags!(target_tag, [source_tag])
         }.to raise_error(ActiveRecord::RecordInvalid)
+
+        expect(Tag.with_discarded.find(source_tag.id)).not_to be_discarded
+        expect(TagName.with_discarded.find(source_tag_name.id)).not_to be_discarded
+        expect(PostTag.kept.find(source_post_tag.id).tag_id).to eq(source_tag.id)
+        expect(PostTag.kept.find_by(post_id: post_record.id, tag_id: target_tag.id)).to be_nil
+        expect(source_tag_name.reload.canonical_id).to be_nil
+        expect(target_tag.reload.post_count).to eq(0)
       end
     end
 
@@ -104,16 +133,16 @@ RSpec.describe Tag, type: :model do
       let!(:source_tag) { create(:tag, category: :nico, name: 'nico:bar') }
       let!(:source_tag_name_id) { source_tag.tag_name_id }
 
-      it 'deletes the source tag_name instead of aliasing it' do
+      it 'discards the source tag_name instead of aliasing it' do
         described_class.merge_tags!(target_tag, [source_tag])
 
-        expect(Tag.exists?(source_tag.id)).to be(false)
-        expect(TagName.exists?(source_tag_name_id)).to be(false)
-        expect(target_tag.reload.post_count).to eq(0)
+        discarded_source_tag = Tag.with_discarded.find(source_tag.id)
+        discarded_source_tag_name = TagName.with_discarded.find(source_tag_name_id)
 
-        expect(Tag.with_discarded.find(source_tag.id)).not_to be_discarded
-        expect(PostTag.kept.find(source_post_tag.id).tag_id).to eq(source_tag.id)
-        expect(source_tag.tag_name.reload.canonical_id).to be_nil
+        expect(discarded_source_tag).to be_discarded
+        expect(discarded_source_tag_name).to be_discarded
+        expect(discarded_source_tag_name.canonical_id).to be_nil
+        expect(target_tag.reload.post_count).to eq(0)
       end
     end
   end
-- 
2.34.1


From 60c8c63353bf702e9756f12bc59e6aef882277ea Mon Sep 17 00:00:00 2001
From: miteruzo <miteruzo@naver.com>
Date: Thu, 12 Mar 2026 21:41:28 +0900
Subject: [PATCH 4/4] #281

---
 .../app/controllers/wiki_pages_controller.rb  |  2 +-
 backend/app/models/my_discard.rb              | 20 +++++++++++++++++++
 backend/app/models/tag.rb                     |  6 +++---
 backend/app/models/tag_name.rb                |  8 ++------
 backend/app/models/wiki_page.rb               |  2 +-
 backend/spec/requests/posts_spec.rb           | 10 ++++++----
 backend/spec/tasks/nico_sync_spec.rb          |  4 ++--
 7 files changed, 35 insertions(+), 17 deletions(-)
 create mode 100644 backend/app/models/my_discard.rb

diff --git a/backend/app/controllers/wiki_pages_controller.rb b/backend/app/controllers/wiki_pages_controller.rb
index 749cf19..dc6c47f 100644
--- a/backend/app/controllers/wiki_pages_controller.rb
+++ b/backend/app/controllers/wiki_pages_controller.rb
@@ -90,7 +90,7 @@ class WikiPagesController < ApplicationController
 
     return head :unprocessable_entity if name.blank? || body.blank?
 
-    tag_name = TagName.find_or_create_by!(name:)
+    tag_name = TagName.find_undiscard_or_create_by!(name:)
     page = WikiPage.new(tag_name:, created_user: current_user, updated_user: current_user)
     if page.save
       message = params[:message].presence
diff --git a/backend/app/models/my_discard.rb b/backend/app/models/my_discard.rb
new file mode 100644
index 0000000..dc4a98d
--- /dev/null
+++ b/backend/app/models/my_discard.rb
@@ -0,0 +1,20 @@
+module MyDiscard
+  extend ActiveSupport::Concern
+
+  included { include Discard::Model }
+
+  class_methods do
+    def find_undiscard_or_create_by! attrs, &block
+      record = with_discarded.find_by(attrs)
+
+      if record&.discarded?
+        record.undiscard!
+        record.update_columns(created_at: record.reload.updated_at)
+      end
+
+      record or create!(attrs, &block)
+    rescue ActiveRecord::RecordNotUnique
+      retry
+    end
+  end
+end
diff --git a/backend/app/models/tag.rb b/backend/app/models/tag.rb
index 9f480cd..a8926c2 100644
--- a/backend/app/models/tag.rb
+++ b/backend/app/models/tag.rb
@@ -1,5 +1,5 @@
 class Tag < ApplicationRecord
-  include Discard::Model
+  include MyDiscard
 
   class NicoTagNormalisationError < ArgumentError
     ;
@@ -134,10 +134,10 @@ class Tag < ApplicationRecord
   end
 
   def self.find_or_create_by_tag_name! name, category:
-    tn = TagName.find_or_create_by!(name: name.to_s.strip)
+    tn = TagName.find_undiscard_or_create_by!(name: name.to_s.strip)
     tn = tn.canonical if tn.canonical_id?
 
-    Tag.find_or_create_by!(tag_name_id: tn.id) do |t|
+    Tag.find_undiscard_or_create_by!(tag_name_id: tn.id) do |t|
       t.category = category
     end
   rescue ActiveRecord::RecordNotUnique
diff --git a/backend/app/models/tag_name.rb b/backend/app/models/tag_name.rb
index c353d2f..b118300 100644
--- a/backend/app/models/tag_name.rb
+++ b/backend/app/models/tag_name.rb
@@ -1,5 +1,5 @@
 class TagName < ApplicationRecord
-  include Discard::Model
+  include MyDiscard
 
   default_scope -> { kept }
 
@@ -27,10 +27,6 @@ class TagName < ApplicationRecord
 
   private
 
-  def sanitise_name
-    self.name = TagNameSanitisationRule.sanitise(name)
-  end
-
   def canonical_must_be_canonical
     if canonical&.canonical_id?
       errors.add :canonical, 'canonical は実体を示す必要があります．'
@@ -50,7 +46,7 @@ class TagName < ApplicationRecord
   end
 
   def name_must_be_sanitised
-    if name != TagNameSanitisationRule.sanitise(name)
+    if name? && name != TagNameSanitisationRule.sanitise(name)
       errors.add :name, '名前に使用できない文字が含まれてゐます．'
     end
   end
diff --git a/backend/app/models/wiki_page.rb b/backend/app/models/wiki_page.rb
index b1d1b02..1573127 100644
--- a/backend/app/models/wiki_page.rb
+++ b/backend/app/models/wiki_page.rb
@@ -2,7 +2,7 @@ require 'set'
 
 
 class WikiPage < ApplicationRecord
-  include Discard::Model
+  include MyDiscard
 
   default_scope -> { kept }
 
diff --git a/backend/spec/requests/posts_spec.rb b/backend/spec/requests/posts_spec.rb
index b2f3e54..ee4fcc5 100644
--- a/backend/spec/requests/posts_spec.rb
+++ b/backend/spec/requests/posts_spec.rb
@@ -525,8 +525,9 @@ RSpec.describe 'Posts API', type: :request do
 
     context "when nico tag already exists in tags" do
       before do
-        Tag.find_or_create_by!(tag_name: TagName.find_or_create_by!(name: 'nico:nico_tag'),
-                               category: :nico)
+        Tag.find_undiscard_or_create_by!(
+            tag_name: TagName.find_undiscard_or_create_by!(name: 'nico:nico_tag'),
+            category: :nico)
       end
 
       it 'return 400' do
@@ -610,8 +611,9 @@ RSpec.describe 'Posts API', type: :request do
 
     context "when nico tag already exists in tags" do
       before do
-        Tag.find_or_create_by!(tag_name: TagName.find_or_create_by!(name: 'nico:nico_tag'),
-                               category: :nico)
+        Tag.find_undiscard_or_create_by!(
+            tag_name: TagName.find_undiscard_or_create_by!(name: 'nico:nico_tag'),
+            category: :nico)
       end
 
       it 'return 400' do
diff --git a/backend/spec/tasks/nico_sync_spec.rb b/backend/spec/tasks/nico_sync_spec.rb
index daa243e..d4f0e09 100644
--- a/backend/spec/tasks/nico_sync_spec.rb
+++ b/backend/spec/tasks/nico_sync_spec.rb
@@ -8,8 +8,8 @@ RSpec.describe "nico:sync" do
   end
 
   def create_tag!(name, category:)
-    tn = TagName.find_or_create_by!(name: name.to_s.strip)
-    Tag.find_or_create_by!(tag_name_id: tn.id) { |t| t.category = category }
+    tn = TagName.find_undiscard_or_create_by!(name: name.to_s.strip)
+    Tag.find_undiscard_or_create_by!(tag_name_id: tn.id) { |t| t.category = category }
   end
 
   def link_nico_to_tag!(nico_tag, tag)
-- 
2.34.1