#351

AGENTS.md

@@ -12,21 +12,16 @@ BTRC Hub / タグ広場 is a split Rails API and React frontend repository.
 ## Stack
 
 - Backend: Ruby `3.2.2` from `backend/.ruby-version`, Rails `~> 8.0.2`.
-- Backend dependencies include `mysql2`, `sqlite3`, `rspec-rails`,
+- Backend dependencies include `mysql2`, `sqlite3`, `rspec-rails`, `factory_bot_rails`, `rack-cors`, `jwt`, `discard`, `gollum`, `whenever`, `aws-sdk-s3`, `brakeman`, and `rubocop-rails-omakase`.
-  `factory_bot_rails`, `rack-cors`, `jwt`, `discard`, `gollum`, `whenever`,
-  `aws-sdk-s3`, `brakeman`, and `rubocop-rails-omakase`.
 - Frontend: React `^19.1.0`, TypeScript `~5.8.3`, Vite `^6.3.5`.
-- Frontend data/UI dependencies include Axios, TanStack Query, Tailwind CSS,
+- Frontend data/UI dependencies include Axios, TanStack Query, Tailwind CSS, Framer Motion, Radix UI components, lucide-react, MDX/Markdown tooling, and Zustand.
-  Framer Motion, Radix UI components, lucide-react, MDX/Markdown tooling, and
-  Zustand.
 
 ## Main directories
 
 - `backend/app/controllers`: Rails API controllers.
 - `backend/app/models`: Active Record models.
 - `backend/app/representations`: API response representation classes.
-- `backend/app/services`: domain services such as version recording,
+- `backend/app/services`: domain services such as version recording, wiki commit, YouTube sync, and similarity calculation.
-  wiki commit, YouTube sync, and similarity calculation.
 - `backend/config/routes.rb`: API routes.
 - `backend/db/migrate`: migrations.
 - `backend/db/schema.rb`: current schema snapshot.
@@ -89,132 +84,52 @@ cd frontend
 npm run dev
 npm run build
 npm run lint
-npm run test
-npm run test:run
 npm run preview
 ```
 
-`npm run build` runs `tsc -b && vite build`, then `postbuild` runs
+`npm run build` runs `tsc -b && vite build`, then `postbuild` runs `node scripts/generate-sitemap.js`.
-`node scripts/generate-sitemap.js`.
 
-`npm run test` runs Vitest in watch mode. Use `npm run test:run` for a non-watch frontend test run.
+Do not write or report `npm test` as a repository command unless a `test` script is added to `frontend/package.json`.
 
 ## Coding style
 
 - Prefer precise, minimal changes.
 - Do not flatter or over-explain.
 - Explain risks directly.
-- Prefer single quotes for strings unless interpolation or escaping makes
+- Prefer single quotes for strings unless interpolation or escaping makes double quotes better.
-  double quotes better.
 - Ruby: never put a space before method-call parentheses.
-- Ruby: never put a line break immediately before `)`.
 - Ruby: do not use `%w` or `%i`.
-- Ruby hashes are not blocks; keep `}` on the same line as the final pair.
+- TypeScript and Python: use GNU-style spacing before parentheses where syntactically valid.
-- Ruby hashes keep the first pair on the same line as `{` unless line length
-  requires a break.
-- Ruby blocks use separate `{ ... }` rules from hashes, with 2-space body
-  indentation.
-- For arrays, never put whitespace or a line break immediately before `]`.
-- Keep the first element on the same line as `[` by default.
-- If an array would exceed the line limit, break after `[` and indent
-  elements by 4 spaces.
-- TypeScript and Python: use GNU-style spacing before parentheses where
-  syntactically valid.
-- Never write Ruby, TypeScript, or TSX lines longer than 99 characters.
-- Aim to keep Ruby, TypeScript, and TSX lines within 79 characters where practical.
-- TypeScript and TSX use 4-space logical indentation.
-- In TypeScript and TSX only, replace every leading run of 8 spaces with a tab.
-- Tabs are only for leading indentation, never for spaces after non-space text.
 - Do not add production dependencies without explicit approval.
 
 ## Backend rules
 
-- Inspect existing routes, controllers, models, services, and specs before
+- Inspect existing routes, controllers, models, services, and specs before editing backend behavior.
-  editing backend behavior.
 - For API behavior changes, add or update request specs under `backend/spec/requests`.
-- Prefer RSpec for new backend tests; existing minitest files under
+- Prefer RSpec for new backend tests; existing minitest files under `backend/test` do not make minitest the default for new coverage.
-  `backend/test` do not make minitest the default for new coverage.
 - Do not weaken authentication, BAN user checks, or IP BAN checks.
-- Preserve the `X-Transfer-Code` user identification flow unless the task
+- Preserve the `X-Transfer-Code` user identification flow unless the task explicitly changes authentication.
-  explicitly changes authentication.
+- Be careful with version tables, `version_no`, optimistic concurrency, wiki revisions, and restore/diff behavior.
-- Be careful with version tables, `version_no`, optimistic concurrency,
-  wiki revisions, and restore/diff behavior.
 - Be careful with tag names, tag normalization, implications, similarities, and discard behavior.
-- Be sensitive to N+1 queries; avoid introducing them and proactively fix
-  existing N+1 issues in the code path being edited.
 - Keep migration files and `backend/db/schema.rb` consistent when changing schema.
 
 ## Frontend rules
 
 - Use `frontend/src/lib/api.ts` for API calls so headers and camelCase conversion stay consistent.
-- Add or reuse TanStack Query keys through `frontend/src/lib/queryKeys.ts`;
+- Add or reuse TanStack Query keys through `frontend/src/lib/queryKeys.ts`; avoid ad hoc query key arrays.
-  avoid ad hoc query key arrays.
 - Encode URL path-segment values with `encodeURIComponent`.
 - React hooks must be called unconditionally.
-- Keep page-level code under `frontend/src/pages` and shared UI/feature code
+- Keep page-level code under `frontend/src/pages` and shared UI/feature code under `frontend/src/components` unless existing patterns point elsewhere.
-  under `frontend/src/components` unless existing patterns point elsewhere.
 - Match existing Tailwind, component, and import alias conventions.
 
-### Frontend TSX style
-
-- Preserve the local TSX formatting style.
-- Do not normalize TSX to common Prettier-style React formatting unless
-  explicitly asked.
-- Prefer `const` arrow functions for TypeScript/TSX component and helper declarations.
-- Put two blank lines before and after top-level `const` function
-  declarations, unless imports, exports, or file boundaries make that awkward.
-- In TSX, indent with 4-space logical indentation.
-- A leading tab is exactly equivalent to 8 leading spaces.
-- Keep a tag's closing marker on the same line as the final prop when the tag
-  spans multiple lines.
-- Do not put `/>` or `>` on its own line unless the existing surrounding code
-  does so.
-- Keep JSX closing parentheses in the existing compact style, for example
-  `</div>)` rather than moving `)` onto a separate line.
-- Do not add braces around `if`, `else`, or `for` bodies when the body is a
-  single physical line.
-- Always add braces around `if`, `else`, or `for` bodies when the body spans
-  two or more physical lines, even if it is one statement.
-
-Preferred:
-
-```tsx
-const PostFormTagsArea: FC<Props> = ({ tags, setTags, errors, ...rest }) => {
-  return (
-      <TextArea
-	  {...rest}
-	  ref={ref}
-	  value={tags}
-	  invalid={errors && errors.length > 0}
-	  onChange={ev => setTags (ev.target.value)}/>)
-}
-```
-
-Avoid:
-
-```tsx
-function PostFormTagsArea ({ tags, setTags }: Props) {
-  return (
-    <TextArea
-      value={tags}
-      onChange={ev => setTags (ev.target.value)}
-    />
-  )
-}
-```
-
 ## Codex workflow
 
 - First inspect existing patterns; do not invent new architecture when a local convention exists.
 - Keep changes scoped to the requested issue.
-- Do not scan or summarize dependency/generated/runtime directories such as
+- Do not scan or summarize dependency/generated/runtime directories such as `node_modules`, `dist`, `tmp`, `log`, and `storage` unless explicitly needed.
-  `node_modules`, `dist`, `tmp`, `log`, and `storage` unless explicitly needed.
+- Before touching wiki, tag, versioning, BAN, IP BAN, or authentication behavior, inspect the related request specs and service objects.
-- Before touching wiki, tag, versioning, BAN, IP BAN, or authentication
+- If frontend code changes, run the existing frontend verification commands that apply: `npm run build` and `npm run lint`.
-  behavior, inspect the related request specs and service objects.
+- If backend code changes, run the relevant RSpec command; for broad backend changes, run `bundle exec rspec`.
-- If frontend code changes, run the existing frontend verification commands
-  that apply: `npm run build`, `npm run lint`, and `npm run test:run`.
-- If backend code changes, run the relevant RSpec command; for broad backend
-  changes, run `bundle exec rspec`.
 - If a verification command cannot be run or fails, report the exact command and failure.
 
 ## Completion criteria

backend/AGENTS.md

@@ -4,9 +4,7 @@
 
 These rules apply to work under `backend/`.
 
-This is a Rails API app using Active Record, RSpec, request specs,
+This is a Rails API app using Active Record, RSpec, request specs, service objects, representation classes, and version tables for post/tag/wiki history.
-service objects, representation classes, and version tables for post/tag/wiki
-history.
 
 ## Commands
 
@@ -52,57 +50,32 @@ If a command cannot be run or fails, report the exact command and failure.
 - `app/controllers`: API controllers.
 - `app/models`: Active Record models and concerns.
 - `app/representations`: JSON response shaping.
-- `app/services`: domain services such as version recorders, wiki commit,
+- `app/services`: domain services such as version recorders, wiki commit, YouTube sync, and similarity calculation.
-  YouTube sync, and similarity calculation.
 - `config/routes.rb`: public API routes.
 - `db/migrate`: migrations.
 - `db/schema.rb`: schema snapshot.
 - `lib/tasks`: custom Rake tasks.
 - `spec`: RSpec tests.
 
-Before changing behavior, inspect the matching route, controller, model,
+Before changing behavior, inspect the matching route, controller, model, service, representation, and spec.
-service, representation, and spec.
 
 ## Ruby style
 
 - Prefer precise, minimal changes.
 - Use single quotes unless interpolation or escaping makes double quotes better.
 - Do not put a space before Ruby method-call parentheses.
-- Never put a line break immediately before `)` in Ruby.
 - Do not use `%w` or `%i` in new Ruby code.
-- Never write a Ruby line longer than 99 characters.
-- Aim to keep Ruby lines within 79 characters where practical.
-- For small Ruby method definitions that take keyword arguments, match the
-  local no-parentheses style when nearby code uses it.
-- Treat Ruby hash `{ ... }` style and Ruby block `{ ... }` style as separate
-  rules.
-- Do not format Ruby hashes like Ruby blocks.
-- For Ruby hashes, keep the closing `}` on the same line as the final pair.
-- Keep the first pair on the same line as `{` by default.
-- If the hash would exceed the line limit, break after `{` and indent pairs
-  by 4 spaces.
-- Put one logical pair per line when the expression would otherwise become
-  dense.
-- For Ruby arrays, never put whitespace or a line break immediately before `]`.
-- Keep the first element on the same line as `[` by default.
-- If an array would exceed the line limit, break after `[` and indent
-  elements by 4 spaces.
-- For Ruby blocks, use 2-space indentation for the block body.
 - Keep comments short and useful; avoid narrating obvious code.
 - Do not add production dependencies without approval.
 
 ## Authentication and authorization
 
-- Authentication is handled through the `X-Transfer-Code` header in
+- Authentication is handled through the `X-Transfer-Code` header in `ApplicationController#authenticate_user`.
-  `ApplicationController#authenticate_user`.
 - `current_user` is set by looking up `User.inheritance_code`.
-- Do not bypass or weaken the `X-Transfer-Code` flow unless the task
+- Do not bypass or weaken the `X-Transfer-Code` flow unless the task explicitly changes authentication.
-  explicitly changes authentication.
+- Unauthenticated write actions should return `:unauthorized` consistently with existing controllers.
-- Unauthenticated write actions should return `:unauthorized` consistently
-  with existing controllers.
 - Role checks use `User` enum roles: `guest`, `member`, and `admin`.
-- Use `current_user.gte_member?` for member-or-admin write permissions where
+- Use `current_user.gte_member?` for member-or-admin write permissions where existing controllers do so.
-  existing controllers do so.
 - Use `current_user.admin?` only for admin-only paths, such as tag child relationship changes.
 - Do not replace role checks with looser presence checks.
 
@@ -115,8 +88,7 @@ service, representation, and spec.
 - User and IP bans use `banned_at`, not a boolean `banned` column.
 - `User#banned?` and `IpAddress#banned?` check `banned_at.present?`.
 - Do not weaken BAN or IP BAN behavior.
-- If changing request authentication or controller before actions, add or
+- If changing request authentication or controller before actions, add or update request specs covering banned users and banned IP addresses.
-  update request specs covering banned users and banned IP addresses.
 
 ## RSpec
 
@@ -127,86 +99,49 @@ service, representation, and spec.
 - Put Rake task coverage under `spec/tasks`.
 - `spec/rails_helper.rb` loads `spec/support/**/*.rb`.
 - Request specs include `AuthHelper` and `JsonHelper`.
-- `AuthHelper#sign_in_as(user)` stubs
+- `AuthHelper#sign_in_as(user)` stubs `ApplicationController#current_user`; use it when matching existing request spec style.
-  `ApplicationController#current_user`; use it when matching existing
+- Add or update request specs for API behavior changes, especially status codes, permissions, response shape, and version conflict behavior.
-  request spec style.
-- Add or update request specs for API behavior changes, especially status
-  codes, permissions, response shape, and version conflict behavior.
 
 ## Migrations
 
 - Keep migrations and `db/schema.rb` consistent.
 - Use reversible migrations where practical; otherwise define explicit `up` and `down`.
-- For data backfills inside migrations, follow the existing pattern of
+- For data backfills inside migrations, follow the existing pattern of defining migration-local `ActiveRecord::Base` classes with `self.table_name`.
-  defining migration-local `ActiveRecord::Base` classes with
-  `self.table_name`.
 - Preserve existing indexes, foreign keys, check constraints, and null constraints.
 - Be careful with MySQL-specific options already present in migrations, such as `after:`.
-- Do not edit old migrations just to change current behavior unless
+- Do not edit old migrations just to change current behavior unless explicitly requested; add a new migration.
-  explicitly requested; add a new migration.
 
 ## Version tables
 
 - Versioned records include posts, tags, nico tags, and wiki pages.
-- Current records have `version_no`; version tables have positive
+- Current records have `version_no`; version tables have positive `version_no` with unique indexes scoped to the parent record.
-  `version_no` with unique indexes scoped to the parent record.
 - Version event types are `create`, `update`, `discard`, and `restore`.
 - Version rows are readonly through the `VersionRecord` concern.
-- Use the existing recorder services instead of manually inserting version
+- Use the existing recorder services instead of manually inserting version rows in application code:
-  rows in application code:
   - `PostVersionRecorder`
   - `TagVersionRecorder`
   - `NicoTagVersionRecorder`
   - `WikiVersionRecorder`
   - `TagVersioning`
-- `VersionRecorder` locks the current record, validates sequence consistency,
+- `VersionRecorder` locks the current record, validates sequence consistency, skips unchanged update snapshots, creates the next version row, and updates the record `version_no`.
-  skips unchanged update snapshots, creates the next version row, and updates
-  the record `version_no`.
 - Do not update versioned records without considering whether a version snapshot must be created.
-- For optimistic concurrency paths, preserve `base_version_no`, `force`, and
+- For optimistic concurrency paths, preserve `base_version_no`, `force`, and `merge` semantics and cover conflicts in request specs.
-  `merge` semantics and cover conflicts in request specs.
 
 ## Domain cautions
 
-- Posts have tag snapshots, parent post implications, original-created ranges,
+- Posts have tag snapshots, parent post implications, original-created ranges, viewed state, and version conflict behavior.
-  viewed state, and version conflict behavior.
+- Tags have canonical names, aliases through `TagName`, categories, parent implications, discard behavior, and version snapshots.
-- Tags have canonical names, aliases through `TagName`, categories, parent
+- Nico tags have separate relation/version behavior; do not treat them like normal editable tags without checking existing code.
-  implications, discard behavior, and version snapshots.
+- Wiki pages involve page content, revisions/history, version rows, title/tag-name behavior, and diff/restore paths.
-- Nico tags have separate relation/version behavior; do not treat them like
+- Materials, theatres, and comments have user and permission checks; inspect the controller before changing them.
-  normal editable tags without checking existing code.
-- Wiki pages involve page content, revisions/history, version rows,
-  title/tag-name behavior, and diff/restore paths.
-- Materials, theatres, and comments have user and permission checks; inspect
-  the controller before changing them.
 
 ## API responses
 
 - Use representation classes under `app/representations` when existing endpoints do.
-- Keep response keys consistent with existing JSON contracts.
+- Keep response keys consistent with existing JSON contracts; frontend code expects camelCase conversion client-side, while Rails params and JSON keys are generally snake_case.
-- Frontend code expects camelCase conversion client-side, while Rails params
+- Preserve existing HTTP status conventions: `:unauthorized` for no user, `:forbidden` for insufficient role or banned user, `:not_found` for missing records, and `:unprocessable_entity` for validation failures.
-  and JSON keys are generally snake_case.
-- Preserve existing HTTP status conventions:
-  `:unauthorized` for no user, `:forbidden` for insufficient role or banned
-  user, `:not_found` for missing records, and `:unprocessable_entity` for
-  validation failures.
-- For diagnostic or internal helper JSON, prefer a deliberately light response
-  shape over full representation classes when callers only need identifiers,
-  labels, URLs, or weights.
-
-## Active Record performance
-
-- When a controller action serializes nested associations, preload the
-  associations it will touch instead of allowing N+1 queries.
-- Be sensitive to N+1 queries in all backend work.
-- Avoid introducing N+1 queries, and proactively fix existing N+1 issues when
-  you find them in the code path you are editing.
-- When an association may already be preloaded, prefer loaded-association
-  checks that reuse the preloaded data without losing the efficient database
-  path.
 
 ## Files to avoid in routine work
 
-- Do not inspect or edit `tmp/`, `log/`, `storage/`, `vendor/`, or dependency
+- Do not inspect or edit `tmp/`, `log/`, `storage/`, `vendor/`, or dependency directories unless explicitly needed.
-  directories unless explicitly needed.
+- Do not modify generated schema or migration output without the corresponding migration when schema changes are made.
-- Do not modify generated schema or migration output without the corresponding
-  migration when schema changes are made.

backend/Gemfile

@@ -69,5 +69,3 @@ gem 'discard'
 gem "rspec-rails", "~> 8.0", :groups => [:development, :test]
 
 gem 'aws-sdk-s3', require: false
-
-gem 'rails-i18n', '~> 8.0.0'

backend/Gemfile.lock

@@ -306,9 +306,6 @@ GEM
     rails-html-sanitizer (1.6.2)
       loofah (~> 2.21)
       nokogiri (>= 1.15.7, != 1.16.7, != 1.16.6, != 1.16.5, != 1.16.4, != 1.16.3, != 1.16.2, != 1.16.1, != 1.16.0.rc1, != 1.16.0)
-    rails-i18n (8.0.2)
-      i18n (>= 0.7, < 2)
-      railties (>= 8.0.0, < 9)
     railties (8.0.2)
       actionpack (= 8.0.2)
       activesupport (= 8.0.2)
@@ -480,7 +477,6 @@ DEPENDENCIES
   puma (>= 5.0)
   rack-cors
   rails (~> 8.0.2)
-  rails-i18n (~> 8.0.0)
   rspec-rails (~> 8.0)
   rubocop-rails-omakase
   sprockets-rails

backend/app/controllers/application_controller.rb

@@ -1,7 +1,4 @@
 class ApplicationController < ActionController::API
-  rescue_from ActiveRecord::RecordInvalid, with: :render_record_invalid
-  rescue_from ActiveRecord::RecordNotUnique, with: :render_record_not_unique
-
   before_action :reject_banned_ip_address!
   before_action :authenticate_user
   before_action :reject_banned_user!
@@ -28,27 +25,6 @@ class ApplicationController < ActionController::API
     end
   end
 
-  def render_bad_request message = 'リクエストが不正です．'
-    render json: { type: 'bad_request',
-                   message:,
-                   errors: { },
-                   base_errors: [message] },
-           status: :bad_request
-  end
-
-  def render_unprocessable_entity message = '入力を確認してください．', field: nil
-    render_validation_error(fields: field ? { field => [message] } : { },
-                            base: field ? [] : [message])
-  end
-
-  def render_record_invalid error
-    render_validation_error error.record
-  end
-
-  def render_record_not_unique _error = nil
-    render_validation_error base: ['すでに存在してゐます．']
-  end
-
   def reject_banned_ip_address!
     ip_address = IpAddress.find_by(ip_address: IPAddr.new(request.remote_ip).hton)
     return unless ip_address&.banned?
@@ -61,28 +37,4 @@ class ApplicationController < ActionController::API
 
     head :forbidden
   end
-
-  def render_validation_error record = nil, fields: { }, base: [], status: :unprocessable_entity
-    errors = { }
-
-    if record
-      record.errors.each do |error|
-        errors[error.attribute] ||= []
-        errors[error.attribute] << error.message
-      end
-    end
-
-    fields.each do |attr, messages|
-      errors[attr.to_sym] ||= []
-      errors[attr.to_sym].concat(Array(messages))
-    end
-
-    base_errors = Array(base) + Array(errors.delete(:base))
-
-    render json: { type: 'validation_error',
-                   message: '入力内容を確認してください．',
-                   errors:,
-                   base_errors: },
-           status:
-  end
 end

backend/app/controllers/deerjikists_controller.rb

@@ -2,8 +2,7 @@ class DeerjikistsController < ApplicationController
   def show
     platform = params[:platform].to_s.strip
     code = params[:code].to_s.strip
-    return render_bad_request('platform は必須です．') if platform.blank?
+    return head :bad_request if platform.blank? || code.blank?
-    return render_bad_request('code は必須です．') if code.blank?
 
     deerjikist = Deerjikist
                  .joins(:tag)
@@ -23,9 +22,7 @@ class DeerjikistsController < ApplicationController
     platform = params[:platform].to_s.strip
     code = params[:code].to_s.strip
     tag_id = params[:tag_id].to_i
-    return render_bad_request('platform は必須です．') if platform.blank?
+    return head :bad_request if platform.blank? || code.blank? || tag_id <= 0
-    return render_bad_request('code は必須です．') if code.blank?
-    return render_bad_request('tag_id が不正です．') if tag_id <= 0
 
     deerjikist = Deerjikist.find_or_initialize_by(platform:, code:).tap do |d|
       d.tag_id = tag_id
@@ -41,8 +38,7 @@ class DeerjikistsController < ApplicationController
 
     platform = params[:platform].to_s.strip
     code = params[:code].to_s.strip
-    return render_bad_request('platform は必須です．') if platform.blank?
+    return head :bad_request if platform.blank? || code.blank?
-    return render_bad_request('code は必須です．') if code.blank?
 
     Deerjikist.find([platform, code]).destroy!
 

backend/app/controllers/materials_controller.rb

@@ -40,11 +40,7 @@ class MaterialsController < ApplicationController
     tag_name_raw = params[:tag].to_s.strip
     file = params[:file]
     url = params[:url].to_s.strip.presence
-    return render_unprocessable_entity('タグは必須です．', field: :tag) if tag_name_raw.blank?
+    return head :bad_request if tag_name_raw.blank? || (file.blank? && url.blank?)
-    if file.blank? && url.blank?
-      return render_validation_error fields: { file: ['ファイルまたは URL は必須です．'],
-                                               url: ['ファイルまたは URL は必須です．'] }
-    end
 
     tag_name = TagName.find_undiscard_or_create_by!(name: tag_name_raw)
     tag = tag_name.tag
@@ -58,7 +54,7 @@ class MaterialsController < ApplicationController
     if material.save
       render json: MaterialRepr.base(material, host: request.base_url), status: :created
     else
-      render_validation_error material
+      render json: { errors: material.errors.full_messages }, status: :unprocessable_entity
     end
   end
 
@@ -72,11 +68,7 @@ class MaterialsController < ApplicationController
     tag_name_raw = params[:tag].to_s.strip
     file = params[:file]
     url = params[:url].to_s.strip.presence
-    return render_unprocessable_entity('タグは必須です．', field: :tag) if tag_name_raw.blank?
+    return head :bad_request if tag_name_raw.blank? || (file.blank? && url.blank?)
-    if file.blank? && url.blank?
-      return render_validation_error fields: { file: ['ファイルまたは URL は必須です．'],
-                                               url: ['ファイルまたは URL は必須です．'] }
-    end
 
     tag_name = TagName.find_undiscard_or_create_by!(name: tag_name_raw)
     tag = tag_name.tag
@@ -92,7 +84,7 @@ class MaterialsController < ApplicationController
     if material.save
       render json: MaterialRepr.base(material, host: request.base_url)
     else
-      render_validation_error material
+      render json: { errors: material.errors.full_messages }, status: :unprocessable_entity
     end
   end
 

backend/app/controllers/nico_tags_controller.rb

@@ -1,69 +1,26 @@
 class NicoTagsController < ApplicationController
   def index
-    name = params[:name].presence
+    limit = (params[:limit] || 20).to_i
-    linked_tag = params[:linked_tag].presence
+    cursor = params[:cursor].presence
-    link_status = params[:link_status].presence
-    order = params[:order].to_s.split(':', 2).map(&:strip)
-    order[0] = 'updated_at' unless order[0].in?(['name', 'created_at', 'updated_at'])
-    unless order[1].in?(['asc', 'desc'])
-      order[1] = order[0] == 'name' ? 'asc' : 'desc'
-    end
-    page = (params[:page].presence || 1).to_i
-    limit = (params[:limit].presence || 20).to_i
-
-    page = 1 if page < 1
-    limit = 1 if limit < 1
-
-    post_tag_max_sql =
-        PostTag
-        .select('tag_id, MAX(created_at) AS max_created_at')
-        .group('tag_id')
-        .to_sql
 
     q = Tag.nico_tags
-           .joins(:tag_name)
-           .joins("LEFT JOIN (#{ post_tag_max_sql }) post_tag_max " \
-                  'ON post_tag_max.tag_id = tags.id')
            .includes(:tag_name, tag_name: :wiki_page, linked_tags: { tag_name: :wiki_page })
-    q = q.where('tag_names.name LIKE ?', "%#{ name }%") if name
+           .order(updated_at: :desc)
-    if linked_tag
+    q = q.where('tags.updated_at < ?', Time.iso8601(cursor)) if cursor
-      linked_tag_ids =
-        Tag
-        .joins(:tag_name)
-        .where('tag_names.name LIKE ?', "%#{ linked_tag }%")
-        .pluck(:id)
-      linked_nico_tag_ids = NicoTagRelation.where(tag_id: linked_tag_ids).pluck(:nico_tag_id)
-      q = q.where(id: linked_nico_tag_ids)
-    end
-    if link_status.in?(['linked', 'unlinked'])
-      exists_sql =
-          'EXISTS (SELECT 1 FROM nico_tag_relations ' \
-          'WHERE nico_tag_relations.nico_tag_id = tags.id)'
-      q = link_status == 'linked' ? q.where(exists_sql) : q.where("NOT #{ exists_sql }")
-    end
 
-    count = q.count
+    tags = q.limit(limit + 1).to_a
-    sort_sql =
+
-        case order[0]
+    next_cursor = nil
-        when 'name'
+    if tags.size > limit
-          'tag_names.name'
+      next_cursor = tags.last.updated_at.iso8601(6)
-        when 'updated_at'
+      tags = tags.first(limit)
-          'post_tag_max.max_created_at'
+    end
-        else
-          "tags.#{ order[0] }"
-        end
-    tags = q.reselect('tags.*',
-                      Arel.sql('post_tag_max.max_created_at AS recent_post_tag_created_at'))
-            .order(Arel.sql("#{ sort_sql } #{ order[1] }, tags.id #{ order[1] }"))
-            .limit(limit)
-            .offset((page - 1) * limit)
-            .to_a
 
     render json: { tags: tags.map { |tag|
-      TagRepr.base(tag).merge(
+      TagRepr.base(tag).merge(linked_tags: tag.linked_tags.map { |lt|
-          recent_post_tag_created_at: tag.recent_post_tag_created_at,
+        TagRepr.base(lt)
-          linked_tags: tag.linked_tags.map { |lt| TagRepr.base(lt) })
+      })
-    }, count: }
+    }, next_cursor: }
   end
 
   def update
@@ -73,18 +30,14 @@ class NicoTagsController < ApplicationController
     id = params[:id].to_i
 
     tag = Tag.find(id)
-    return render_bad_request('ニコニコ・タグを指定してください．') unless tag.nico?
+    return head :bad_request unless tag.nico?
 
     linked_tag_names = params[:tags].to_s.split
-    linked_tags = nil
+    linked_tags = Tag.normalise_tags!(linked_tag_names, with_tagme: false,
+                                                        with_no_deerjikist: false)
+    return head :bad_request if linked_tags.any? { |t| t.nico? }
 
     ApplicationRecord.transaction do
-      linked_tags = Tag.normalise_tags!(linked_tag_names, with_tagme: false,
-                                                          with_no_deerjikist: false)
-      if linked_tags.any? { |t| t.nico? }
-        raise Tag::NicoTagNormalisationError
-      end
-
       TagVersioning.record_tag_snapshots!(linked_tags, created_by_user: current_user)
 
       tag.linked_tags = linked_tags
@@ -94,21 +47,5 @@ class NicoTagsController < ApplicationController
     end
 
     render json: tag.linked_tags.map { |t| TagRepr.base(t) }, status: :ok
-  rescue Tag::NicoTagNormalisationError
-    render_validation_error fields: { tags: ['ニコニコ・タグ同士は連携できません．'] }
-  rescue ActiveRecord::RecordInvalid => e
-    render_nico_tag_form_record_invalid e.record
-  end
-
-  private
-
-  def render_nico_tag_form_record_invalid record
-    if record.is_a?(TagName) || record.is_a?(Tag)
-      render_validation_error fields: { tags: record.errors.full_messages.map { |message|
-        "タグ名 “#{ record.name }”: #{ message }"
-      } }
-    else
-      render_validation_error record
-    end
   end
 end

backend/app/controllers/posts_controller.rb

@@ -44,8 +44,7 @@ class PostsController < ApplicationController
         filtered_posts
         .joins("LEFT JOIN (#{ pt_max_sql }) pt_max ON pt_max.post_id = posts.id")
         .reselect('posts.*', Arel.sql("#{ updated_at_all_sql } AS updated_at_all"))
-        .preload(:uploaded_user, :parents, :children,
+        .preload(tags: [:deerjikists, :materials, { tag_name: :wiki_page }])
-                 tags: [:deerjikists, :materials, { tag_name: :wiki_page }])
         .with_attached_thumbnail
 
     q = q.where('posts.url LIKE ?', "%#{ url }%") if url
@@ -96,36 +95,22 @@ class PostsController < ApplicationController
   end
 
   def random
-    post = filtered_posts.preload(:uploaded_user, :parents, :children,
+    post = filtered_posts
-                                  tags: [:deerjikists, :materials, { tag_name: :wiki_page }])
+           .preload(tags: [:deerjikists, :materials, { tag_name: :wiki_page }])
-                         .with_attached_thumbnail
+           .order('RAND()')
-                         .order('RAND()')
+           .first
-                         .first
     return head :not_found unless post
 
     render json: PostRepr.base(post, current_user)
   end
 
   def show
-    post =
+    post = Post.includes(tags: [:deerjikists, :materials, { tag_name: :wiki_page }]).find_by(id: params[:id])
-        Post
-        .includes(:uploaded_user, :parents, :children,
-                  tags: [:deerjikists, :materials, { tag_name: :wiki_page }])
-        .with_attached_thumbnail
-        .find_by(id: params[:id])
     return head :not_found unless post
 
-    parent_posts = post.parents.with_attached_thumbnail.order(:id).to_a
+    render json: PostRepr.base(post, current_user)
-    child_posts = post.children.with_attached_thumbnail.order(:id).to_a
+                         .merge(tags: build_tag_tree_for(post),
-    sibling_posts = sibling_posts_by_parent(parent_posts.map(&:id))
+                                related: PostRepr.many(post.related(limit: 20)))
-    related = post.related(limit: 20).to_a
-
-    render json: PostRepr.detail(post, current_user,
-                                 parent_posts:,
-                                 child_posts:,
-                                 sibling_posts:,
-                                 related:)
-                         .merge(tags: build_tag_tree_for(post.tags))
   end
 
   def create
@@ -148,11 +133,11 @@ class PostsController < ApplicationController
     ApplicationRecord.transaction do
       post.save!
 
-      tags = Tag.normalise_tags!(tag_names)
+      Tag.normalise_tags!(tag_names, with_sections: true) => { tags:, sections: }
       TagVersioning.record_tag_snapshots!(tags, created_by_user: current_user)
 
       tags = Tag.expand_parent_tags(tags)
-      sync_post_tags!(post, tags)
+      sync_post_tags!(post, tags, sections)
 
       sync_parent_posts!(post, parent_post_ids)
 
@@ -164,11 +149,11 @@ class PostsController < ApplicationController
     post.reload
     render json: PostRepr.base(post), status: :created
   rescue Tag::NicoTagNormalisationError
-    render_validation_error fields: { tags: 'ニコニコ・タグは直接指定できません．' }
+    head :bad_request
   rescue ArgumentError => e
-    render_validation_error fields: { parent_post_ids: [e.message] }
+    render json: { errors: [e.message] }, status: :unprocessable_entity
   rescue ActiveRecord::RecordInvalid => e
-    render_post_form_record_invalid e.record
+    render json: { errors: e.record.errors.full_messages }, status: :unprocessable_entity
   end
 
   def viewed
@@ -191,10 +176,10 @@ class PostsController < ApplicationController
 
     force = bool?(:force)
     merge = bool?(:merge)
-    return render_bad_request('force と merge は同時に指定できません．') if force && merge
+    return head :bad_request if force && merge
 
     base_version_no = parse_base_version_no
-    return render_bad_request('base_version_no は必須です．') if !(force) && !(base_version_no)
+    return head :bad_request if !(force) && !(base_version_no)
 
     title = params[:title].presence
     tag_names = params[:tags].to_s.split
@@ -251,14 +236,14 @@ class PostsController < ApplicationController
 
     post.reload
     json = PostRepr.base(post, current_user)
-    json['tags'] = build_tag_tree_for(post.tags)
+    json['tags'] = build_tag_tree_for(post)
     render json:, status: :ok
   rescue Tag::NicoTagNormalisationError
-    render_validation_error fields: { tags: ['ニコニコ・タグは直接指定できません．'] }
+    head :bad_request
   rescue ArgumentError => e
-    render_validation_error fields: { parent_post_ids: [e.message] }
+    render json: { errors: [e.message] }, status: :unprocessable_entity
   rescue ActiveRecord::RecordInvalid => e
-    render_post_form_record_invalid e.record
+    render json: { errors: e.record.errors.full_messages }, status: :unprocessable_entity
   end
 
   def changes
@@ -353,7 +338,7 @@ class PostsController < ApplicationController
   def tagged_post_ids_for(name) =
       Post.joins(tags: :tag_name).where(tag_names: { name: }).select(:id)
 
-  def sync_post_tags! post, desired_tags
+  def sync_post_tags! post, desired_tags, sections
     desired_tags.each do |t|
       t.save! if t.new_record?
     end
@@ -372,13 +357,20 @@ class PostsController < ApplicationController
       end
     end
 
+    PostTagSection.where(post_id: post.id).destroy_all
+    sections.each do |tag_id, ranges|
+      ranges.each do |begin_ms, end_ms|
+        PostTagSection.create!(post_id: post.id, tag_id:, begin_ms:, end_ms:)
+      end
+    end
+
     PostTag.where(post_id: post.id, tag_id: to_remove.to_a).kept.find_each do |pt|
       pt.discard_by!(current_user)
     end
   end
 
-  def build_tag_tree_for tags
+  def build_tag_tree_for post
-    tags = tags.to_a
+    tags = post.tags.to_a
     tag_ids = tags.map(&:id)
 
     implications = TagImplication.where(parent_tag_id: tag_ids, tag_id: tag_ids)
@@ -400,8 +392,11 @@ class PostsController < ApplicationController
       tag = tags_by_id[tag_id]
       return nil unless tag
 
+      sections = PostTagSection.where(post_id: post.id, tag_id:)
+                 .as_json(only: [:begin_ms, :end_ms])
+
       if path.include?(tag_id)
-        return TagRepr.inline(tag).merge(children: [])
+        return TagRepr.base(tag).merge(children: [], sections:)
       end
 
       if memo.key?(tag_id)
@@ -413,32 +408,18 @@ class PostsController < ApplicationController
 
       children = child_ids.filter_map { |cid| build_node.(cid, new_path) }
 
-      memo[tag_id] = TagRepr.inline(tag).merge(children:)
+      memo[tag_id] = TagRepr.base(tag).merge(children:, sections:)
     end
 
     root_ids.filter_map { |id| build_node.call(id, []) }
   end
 
-  def sibling_posts_by_parent parent_post_ids
-    return { } if parent_post_ids.blank?
-
-    implications =
-        PostImplication
-        .where(parent_post_id: parent_post_ids)
-        .includes(post: { thumbnail_attachment: :blob })
-        .order(:parent_post_id, :post_id)
-
-    implications.group_by(&:parent_post_id).transform_values { |items|
-      items.map(&:post)
-    }
-  end
-
   def parse_parent_post_ids
     raise ArgumentError, 'parent_post_ids は必須です．' unless params.key?(:parent_post_ids)
 
     params[:parent_post_ids].to_s.split.map { |token|
       id = Integer(token, exception: false)
-      raise ArgumentError, "親投稿 Id. が不正です: #{ token }" if id.nil? || id <= 0
+      raise ArgumentError, "親投稿 Id. が不正です: #{ token }" if !(id) || id <= 0
 
       id
     }.uniq
@@ -446,7 +427,7 @@ class PostsController < ApplicationController
 
   def sync_parent_posts! post, parent_post_ids
     if parent_post_ids.include?(post.id)
-      post.errors.add :parent_post_ids, '自分自身を親投稿にはできません．'
+      post.errors.add(:base, '自分自身を親投稿にはできません．')
       raise ActiveRecord::RecordInvalid, post
     end
 
@@ -454,8 +435,7 @@ class PostsController < ApplicationController
     missing_ids = parent_post_ids - existing_ids
 
     if missing_ids.present?
-      post.errors.add :parent_post_ids,
+      post.errors.add(:base, "存在しない親投稿 ID があります: #{ missing_ids.join(' ') }")
-                      "存在しない親投稿 Id. があります: #{ missing_ids.join(' ') }"
       raise ActiveRecord::RecordInvalid, post
     end
 
@@ -501,7 +481,21 @@ class PostsController < ApplicationController
   end
 
   def editable_tag_names_from_post post
-    post.tags.not_nico.joins(:tag_name).order('tag_names.name').pluck('tag_names.name')
+    post
+        .post_tags
+        .kept
+        .joins(tag: :tag_name)
+        .merge(Tag.not_nico)
+        .includes(:sections, tag: :tag_name)
+        .order('tag_names.name')
+        .map do |post_tag|
+      name = post_tag.tag.tag_name.name
+      sections = post_tag.sections.sort_by(&:begin_ms)
+
+      next name if sections.empty?
+
+      "#{ name }#{ sections.map { Post.section_literal(_1) }.join }"
+    end
   end
 
   def post_incoming_snapshot title:, original_created_from:, original_created_before:,
@@ -533,9 +527,16 @@ class PostsController < ApplicationController
   end
 
   def incoming_tag_names_for_snapshot raw_tag_names
-    tags = Tag.normalise_tags!(raw_tag_names, with_tagme: false)
+    Tag.normalise_tags!(raw_tag_names, with_tagme: false, with_sections: true) =>
+        { tags:, sections: }
 
-    Tag.expand_parent_tags(tags).map(&:name).uniq.sort
+    Tag.expand_parent_tags(tags).uniq(&:id).map { |tag|
+      "#{ tag.name }#{ sections[tag.id].to_a.map { section_literal(_1) }.join }"
+    }.sort
+  end
+
+  def section_literal section
+    "[#{ Post.ms_to_time(section[0]) }-#{ Post.ms_to_time(section[1]) }]"
   end
 
   def post_conflict_json post:, base_version_no:, base_snapshot:,
@@ -622,7 +623,8 @@ class PostsController < ApplicationController
                  original_created_from: snapshot[:original_created_from],
                  original_created_before: snapshot[:original_created_before])
 
-    editable_tags = Tag.normalise_tags!(snapshot[:tag_names], with_tagme: false)
+    Tag.normalise_tags!(snapshot[:tag_names], with_tagme: false, with_sections: true) =>
+        { tags: editable_tags, sections: }
     TagVersioning.record_tag_snapshots!(editable_tags, created_by_user: current_user)
 
     readonly_tags = post.tags.nico.to_a
@@ -630,7 +632,7 @@ class PostsController < ApplicationController
     tags = readonly_tags + editable_tags
     tags = Tag.expand_parent_tags(tags)
 
-    sync_post_tags!(post, tags)
+    sync_post_tags!(post, tags, sections)
     sync_parent_posts!(post, snapshot[:parent_post_ids])
 
     PostVersionRecorder.record!(post:, event_type: :update, created_by_user: current_user)
@@ -671,14 +673,4 @@ class PostsController < ApplicationController
 
     merged.uniq.sort
   end
-
-  def render_post_form_record_invalid record
-    if record.is_a?(TagName) || record.is_a?(Tag)
-      render_validation_error fields: { tags: record.errors.full_messages.map { |message|
-        "タグ名 “#{ record.name }”: #{ message }"
-      } }
-    else
-      render_validation_error record
-    end
-  end
 end

backend/app/controllers/preview_controller.rb

@@ -4,7 +4,7 @@ class PreviewController < ApplicationController
     return head :unauthorized unless current_user
 
     url = params[:url]
-    return render_bad_request('URL は必須です．') unless url.present?
+    return head :bad_request unless url.present?
 
     unless url.start_with?(/http(s)?:\/\//)
       url = 'http://' + url
@@ -16,7 +16,7 @@ class PreviewController < ApplicationController
 
     render json: { title: title }
   rescue => e
-    render_bad_request(e.message)
+    render json: { error: e.message }, status: :bad_request
   end
 
   def thumbnail
@@ -25,7 +25,7 @@ class PreviewController < ApplicationController
     return head :unauthorized unless current_user
 
     url = params[:url]
-    return render_bad_request('URL は必須です．') if url.blank?
+    return head :bad_request if url.blank?
 
     unless url.start_with?(/http(s)?:\/\//)
       url = 'http://' + url
@@ -40,11 +40,7 @@ class PreviewController < ApplicationController
       File.delete(path) rescue nil
       send_file image.path, type: 'image/png', disposition: 'inline'
     else
-      render json: { type: 'internal_server_error',
+      render json: { error: 'Failed to generate thumbnail' }, status: :internal_server_error
-                     message: 'サムネールを生成できませんでした．',
-                     errors: { },
-                     base_errors: ['サムネールを生成できませんでした．'] },
-             status: :internal_server_error
     end
   end
 end

backend/app/controllers/tag_children_controller.rb

@@ -5,12 +5,11 @@ class TagChildrenController < ApplicationController
 
     parent_id = params[:parent_id]
     child_id = params[:child_id]
-    return render_bad_request('parent_id は必須です．') if parent_id.blank?
+    return head :bad_request if parent_id.blank? || child_id.blank?
-    return render_bad_request('child_id は必須です．') if child_id.blank?
 
     parent = Tag.find(parent_id)
     child = Tag.find(child_id)
-    return render_bad_request('ニコニコ・タグの階層は変更できません．') if parent.nico? || child.nico?
+    return head :bad_request if parent.nico? || child.nico?
 
     ApplicationRecord.transaction do
       TagVersioning.ensure_snapshot!(child, created_by_user: current_user)
@@ -28,12 +27,11 @@ class TagChildrenController < ApplicationController
 
     parent_id = params[:parent_id]
     child_id = params[:child_id]
-    return render_bad_request('parent_id は必須です．') if parent_id.blank?
+    return head :bad_request if parent_id.blank? || child_id.blank?
-    return render_bad_request('child_id は必須です．') if child_id.blank?
 
     parent = Tag.find(parent_id)
     child = Tag.find(child_id)
-    return render_bad_request('ニコニコ・タグの階層は変更できません．') if parent.nico? || child.nico?
+    return head :bad_request if parent.nico? || child.nico?
 
     ApplicationRecord.transaction do
       TagVersioning.ensure_snapshot!(child, created_by_user: current_user)

backend/app/controllers/tags_controller.rb

@@ -168,7 +168,7 @@ class TagsController < ApplicationController
 
   def show_by_name
     name = params[:name].to_s.strip
-    return render_bad_request('name は必須です．') if name.blank?
+    return head :bad_request if name.blank?
 
     tag = Tag.joins(:tag_name)
              .includes(:tag_name, :materials, tag_name: :wiki_page)
@@ -192,7 +192,7 @@ class TagsController < ApplicationController
 
   def deerjikists_by_name
     name = params[:name].to_s.strip
-    return render_bad_request('name は必須です．') if name.blank?
+    return head :bad_request if name.blank?
 
     tag = Tag.joins(:tag_name)
              .includes(:tag_name, tag_name: :wiki_page)
@@ -214,24 +214,21 @@ class TagsController < ApplicationController
 
     ApplicationRecord.transaction do
       tag.deerjikists = []
-      params[:_json].each.with_index do |item, i|
+      params[:_json].each do
-        platform = item[:platform]
+        platform = _1[:platform]
-        code = normalise_deerjikist_code(platform, item[:code])
+        code = normalise_deerjikist_code(platform, _1[:code])
         deerjikist = Deerjikist.find_or_initialize_by(platform:, code:)
         deerjikist.tag = tag
-        render_deerjikist_form_record_invalid(deerjikist, i) unless deerjikist.save
+        deerjikist.save!
-        raise ActiveRecord::Rollback if performed?
       end
     end
 
-    return if performed?
-
     render json: DeerjikistRepr.many(tag.reload.deerjikists)
   end
 
   def materials_by_name
     name = params[:name].to_s.strip
-    return render_bad_request('name は必須です．') if name.blank?
+    return head :bad_request if name.blank?
 
     tag = Tag.joins(:tag_name)
              .includes(:tag_name, :materials, tag_name: :wiki_page)
@@ -250,16 +247,17 @@ class TagsController < ApplicationController
 
     name = params[:name].to_s.strip
     category = params[:category].to_s.strip
-    return render_unprocessable_entity('名前は必須です．', field: :name) if name.blank?
+    return head :unprocessable_entity if name.blank? || category.blank?
-    return render_unprocessable_entity('カテゴリは必須です．', field: :category) if category.blank?
 
     if name != tag.name &&
         tag.in?([Tag.tagme, Tag.bot, Tag.no_deerjikist, Tag.video, Tag.niconico])
-      return render_unprocessable_entity('システム・タグの名称は変更できません．', field: :name)
+      return render json: { error: 'システム・タグの名称は変更できません．' },
+                    status: :unprocessable_entity
     end
 
     if tag.nico? || category == 'nico'
-      return render_unprocessable_entity('ニコタグは変更できません．', field: :category)
+      return render json: { error: 'ニコタグは変更できません．' },
+                    status: :unprocessable_entity
     end
 
     alias_names = params[:aliases].to_s.split.uniq
@@ -304,7 +302,8 @@ class TagsController < ApplicationController
     tag = Tag.find(params[:id])
 
     if tag.nico? || (category.present? && category == 'nico')
-      return render_unprocessable_entity('ニコタグは変更できません．', field: :category)
+      return render json: { error: 'ニコタグは変更できません．' },
+                    status: :unprocessable_entity
     end
 
     ApplicationRecord.transaction do
@@ -438,23 +437,4 @@ class TagsController < ApplicationController
   rescue
     nil
   end
-
-  def render_deerjikist_form_record_invalid deerjikist, index
-    fields = { }
-
-    deerjikist.errors.each do |error|
-      field =
-          case error.attribute
-          when :platform, :code
-            "deerjikists.#{ index }.#{ error.attribute }"
-          else
-            :deerjikists
-          end
-
-      fields[field] ||= []
-      fields[field] << error.full_message
-    end
-
-    render_validation_error fields:
-  end
 end

backend/app/controllers/theatre_comments_controller.rb

@@ -1,28 +1,21 @@
 class TheatreCommentsController < ApplicationController
   def index
-    limit = params[:limit].to_i
-    limit = 20 if limit <= 0
-
     no_gt = params[:no_gt].to_i
-    no_gt = 0 if no_gt < 0
+    no_gt = 0 if no_gt.negative?
 
     comments = TheatreComment
                .where(theatre_id: params[:theatre_id])
                .where('no > ?', no_gt)
                .order(no: :desc)
-               .limit(limit)
 
-    render json: comments.map {
+    render json: comments.as_json(include: { user: { only: [:id, :name] } })
-      _1.as_json(include: { user: { only: [:id, :name] } })
-          .merge(content: _1.discarded? ? nil : _1.content, deleted: _1.discarded?)
-    }
   end
 
   def create
     return head :unauthorized unless current_user
 
     content = params[:content]
-    return render_unprocessable_entity('本文は必須です．', field: :content) if content.blank?
+    return head :unprocessable_entity if content.blank?
 
     theatre = Theatre.find_by(id: params[:theatre_id])
     return head :not_found unless theatre
@@ -36,19 +29,4 @@ class TheatreCommentsController < ApplicationController
 
     render json: comment, status: :created
   end
-
-  def destroy
-    return head :unauthorized unless current_user
-
-    theatre_id = params[:theatre_id].to_i
-    no = params[:id].to_i
-
-    comment = TheatreComment.find_by(theatre_id:, no:)
-    return head :not_found unless comment
-    return head :forbidden unless comment.user == current_user
-
-    comment.discard!
-
-    head :no_content
-  end
 end

backend/app/controllers/theatre_programmes_controller.rb

@@ -1,22 +0,0 @@
-class TheatreProgrammesController < ApplicationController
-  def index
-    limit = params[:limit].to_i
-    limit = 100 if limit <= 0
-
-    position_gt = params[:position_gt].to_i
-    position_gt = 0 if position_gt < 0
-
-    programmes = TheatreProgramme
-                 .where(theatre_id: params[:theatre_id])
-                 .where('position > ?', position_gt)
-                 .includes(:post)
-                 .order(position: :desc).limit(100)
-                 .limit(limit)
-
-    render json: programmes.map { |programme|
-      programme.as_json.merge(post: { id: programme.post.id,
-                                      title: programme.post.title,
-                                      url: programme.post.url })
-    }
-  end
-end

backend/app/controllers/theatre_skip_events_controller.rb

@@ -1,22 +0,0 @@
-class TheatreSkipEventsController < ApplicationController
-  def index
-    limit = params[:limit].to_i
-    limit = 50 if limit <= 0
-
-    events =
-        TheatreSkipEvent
-        .where(theatre_id: params[:theatre_id])
-        .includes(:post, tags: :tag_name)
-        .order(created_at: :desc)
-        .limit(limit)
-
-    render json: events.map { |event|
-      { id: event.id,
-        theatre_id: event.theatre_id,
-        post: { id: event.post.id, title: event.post.title, url: event.post.url },
-        tags: event.tags.map { |tag| { id: tag.id, name: tag.name } },
-        programme_position: event.programme_position,
-        created_at: event.created_at }
-    }
-  end
-end

backend/app/controllers/theatres_controller.rb

@@ -31,7 +31,9 @@ class TheatresController < ApplicationController
       post_started_at = theatre.current_post_started_at
     end
 
-    render json: theatre_info_json(theatre, host_flg:, post_id:, post_started_at:)
+    render json: {
+        host_flg:, post_id:, post_started_at:,
+        watching_users: theatre.watching_users.as_json(only: [:id, :name]) }
   end
 
   def next_post
@@ -41,119 +43,12 @@ class TheatresController < ApplicationController
     return head :not_found unless theatre
     return head :forbidden if theatre.host_user != current_user
 
-    ApplicationRecord.transaction do
+    post = Post.where("url LIKE '%nicovideo.jp%'")
-      theatre.lock!
+           .or(Post.where("url LIKE '%youtube.com%'"))
-      TheatrePostAdvancer.call(theatre:)
+           .order('RAND()')
-    end
+           .first
+    theatre.update!(current_post: post, current_post_started_at: Time.current)
 
     head :no_content
   end
-
-  def skip_vote
-    return head :unauthorized unless current_user
-
-    theatre = Theatre.find_by(id: params[:id])
-    return head :not_found unless theatre
-    requested_post_id = params[:post_id].to_i
-    return head :unprocessable_entity if requested_post_id <= 0
-
-    skipped = false
-    conflicted = false
-
-    ApplicationRecord.transaction do
-      theatre.lock!
-      if theatre.current_post
-        TheatreWatchingUser.find_or_initialize_by(theatre:, user: current_user).tap {
-          _1.expires_at = 30.seconds.from_now
-        }.save!
-
-        if theatre.current_post_id != requested_post_id
-          conflicted = true
-          next
-        end
-
-        TheatreSkipVote.find_or_create_by!(theatre:, post_id: requested_post_id, user: current_user)
-
-        vote_status = skip_vote_status(theatre)
-        if vote_status[:votes_count] >= vote_status[:required_count]
-          TheatreSkipFinalizer.call(theatre:, user: current_user)
-          TheatrePostAdvancer.call(theatre:)
-          skipped = true
-        end
-      end
-    end
-
-    theatre.reload
-    return render json: theatre_info_json(theatre, skipped: false), status: :conflict if conflicted
-
-    render json: theatre_info_json(theatre, skipped:)
-  end
-
-  def unskip_vote
-    return head :unauthorized unless current_user
-
-    theatre = Theatre.find_by(id: params[:id])
-    return head :not_found unless theatre
-    requested_post_id = params[:post_id].to_i
-    return head :unprocessable_entity if requested_post_id <= 0
-
-    conflicted = false
-
-    theatre.with_lock do
-      if theatre.current_post
-        if theatre.current_post_id != requested_post_id
-          conflicted = true
-        else
-          TheatreSkipVote.where(theatre:, post_id: requested_post_id, user: current_user).delete_all
-        end
-      end
-    end
-
-    theatre.reload
-    return render json: theatre_info_json(theatre, skipped: false), status: :conflict if conflicted
-
-    render json: theatre_info_json(theatre, skipped: false)
-  end
-
-  def post_selection_weights
-    theatre = Theatre.find_by(id: params[:id])
-    return head :not_found unless theatre
-
-    render json: TheatrePostSelector.new(theatre:).weight_json
-  end
-
-  private
-
-  def theatre_info_json(theatre, host_flg: nil, post_id: nil, post_started_at: nil, skipped: nil)
-    host_flg = theatre.host_user_id == current_user&.id if host_flg.nil?
-    post_id = theatre.current_post_id if post_id.nil?
-    post_started_at = theatre.current_post_started_at if post_started_at.nil?
-
-    json = { host_flg:,
-             post_id:,
-             post_started_at:,
-             post_elapsed_ms: post_started_at ? ((Time.current - post_started_at) * 1000).floor : nil,
-             watching_users: theatre.watching_users.as_json(only: [:id, :name]),
-             skip_vote: skip_vote_status(theatre) }
-    json[:skipped] = skipped unless skipped.nil?
-    json
-  end
-
-  def skip_vote_status(theatre)
-    watching_user_ids = theatre.watching_users.ids
-    watching_users_count = watching_user_ids.size
-    required_count = (watching_users_count / 2) + 1
-    post = theatre.current_post
-    votes =
-        if post
-          TheatreSkipVote.where(theatre:, post:, user_id: watching_user_ids)
-        else
-          TheatreSkipVote.none
-        end
-
-    { votes_count: post ? votes.count : 0,
-      required_count:,
-      watching_users_count:,
-      voted: post && current_user ? votes.exists?(user_id: current_user.id) : false }
-  end
 end

backend/app/controllers/users_controller.rb

@@ -42,12 +42,12 @@ class UsersController < ApplicationController
     return head :unauthorized if user&.id != params[:id].to_i
 
     name = params[:name]
-    return render_unprocessable_entity('名前は必須です．', field: :name) if name.blank?
+    return head :bad_request if name.blank?
 
     if user.update(name:)
       render json: user.slice(:id, :name, :inheritance_code, :role), status: :ok
     else
-      render_validation_error user
+      render json: user.errors, status: :unprocessable_entity
     end
   end
 

backend/app/controllers/wiki_pages_controller.rb

@@ -46,7 +46,7 @@ class WikiPagesController < ApplicationController
 
   def diff
     id = params[:id]
-    return render_bad_request('id は必須です．') if id.blank?
+    return head :bad_request if id.blank?
 
     from = params[:from].presence
     to = params[:to].presence
@@ -56,7 +56,7 @@ class WikiPagesController < ApplicationController
     from_rev = from && page.wiki_revisions.find(from)
     to_rev = to ? page.wiki_revisions.find(to) : page.current_revision
     if ((from_rev && !(from_rev.content?)) || !(to_rev&.content?))
-      return render_unprocessable_entity('差分を表示できない版です．')
+      return head :unprocessable_entity
     end
 
     diffs = Diff::LCS.sdiff(from_rev&.body&.lines || [], to_rev.body.lines)
@@ -89,8 +89,7 @@ class WikiPagesController < ApplicationController
     body = params[:body].to_s
     message = params[:message].presence
 
-    return render_unprocessable_entity('タイトルは必須です．', field: :title) if title.blank?
+    return head :unprocessable_entity if title.blank? || body.blank?
-    return render_unprocessable_entity('本文は必須です．', field: :body) if body.blank?
 
     tag_name = TagName.find_undiscard_or_create_by!(name: title)
 
@@ -102,10 +101,8 @@ class WikiPagesController < ApplicationController
             message:)
 
     render json: WikiPageRepr.base(page), status: :created
-  rescue ActiveRecord::RecordInvalid => e
+  rescue ActiveRecord::RecordInvalid, ActiveRecord::RecordNotUnique
-    render_validation_error e.record
+    head :unprocessable_entity
-  rescue ActiveRecord::RecordNotUnique
-    render_record_not_unique
   end
 
   def update
@@ -115,8 +112,7 @@ class WikiPagesController < ApplicationController
     title = params[:title]&.strip
     body = params[:body].to_s
 
-    return render_unprocessable_entity('タイトルは必須です．', field: :title) if title.blank?
+    return head :unprocessable_entity if title.blank? || body.blank?
-    return render_unprocessable_entity('本文は必須です．', field: :body) if body.blank?
 
     page = WikiPage.find(params[:id])
     base_revision_id = params[:base_revision_id].presence

backend/app/models/post.rb

@@ -56,7 +56,38 @@ class Post < ApplicationRecord
     super(options).merge(thumbnail: nil)
   end
 
-  def snapshot_tag_names = tags.joins(:tag_name).order('tag_names.name').pluck('tag_names.name')
+  def snapshot_tag_names
+    post_tags
+        .kept
+        .joins(tag: :tag_name)
+        .includes(:sections, tag: :tag_name)
+        .order('tag_names.name')
+        .map do |post_tag|
+      name = post_tag.tag.tag_name.name
+      sections = post_tag.sections.sort_by(&:begin_ms)
+
+      next name if sections.empty?
+
+      "#{ name }#{ sections.map { Post.section_literal(_1) }.join }"
+    end
+  end
+
+  def self.section_literal section
+    "[#{ Post.ms_to_time(section.begin_ms) }-#{ Post.ms_to_time(section.end_ms) }]"
+  end
+
+  def self.ms_to_time ms
+    total_s = ms / 1_000
+    s = total_s % 60
+    min = (total_s / 60) % 60
+    h = total_s / 3_600
+
+    if h.positive?
+      '%d:%02d:%02d' % [h, min, s]
+    else
+      '%d:%02d' % [min, s]
+    end
+  end
 
   def snapshot_parent_post_ids = parents.order(:id).pluck(:id)
 
@@ -94,7 +125,7 @@ class Post < ApplicationRecord
     return if !(f) || !(b)
 
     if f >= b
-      errors.add :original_created_at, 'オリジナルの作成日時の順番がをかしぃです．'
+      errors.add :original_created_before, 'オリジナルの作成日時の順番がをかしぃです．'
     end
   end
 

backend/app/models/post_tag.rb

@@ -10,6 +10,12 @@ class PostTag < ApplicationRecord
   belongs_to :created_user, class_name: 'User', optional: true
   belongs_to :deleted_user, class_name: 'User', optional: true
 
+  has_many :sections, -> { order(:begin_ms) }, class_name: 'PostTagSection',
+                                               foreign_key: [:post_id, :tag_id],
+                                               primary_key: [:post_id, :tag_id],
+                                               dependent: :delete_all,
+                                               inverse_of: :post_tag
+
   validates :post_id, presence: true
   validates :tag_id, presence: true
   validates :post_id, uniqueness: {

backend/app/models/post_tag_section.rb

@@ -0,0 +1,20 @@
+class PostTagSection < ApplicationRecord
+  self.primary_key = :post_id, :tag_id, :begin_ms
+
+  belongs_to :post
+  belongs_to :tag
+
+  belongs_to :post_tag, -> { kept }, foreign_key: [:post_id, :tag_id],
+                                     primary_key: [:post_id, :tag_id],
+                                     inverse_of: :sections,
+                                     optional: true
+
+  validates :post_id, presence: true
+  validates :tag_id, presence: true
+
+  validates :begin_ms, presence: true,
+                       numericality: { only_integer: true, greater_than_or_equal_to: 0 }
+
+  validates :end_ms, presence: true,
+                     numericality: { only_integer: true, greater_than: :begin_ms }
+end

backend/app/models/tag.rb

@@ -81,7 +81,7 @@ class Tag < ApplicationRecord
 
   def material_id = materials.first&.id
 
-  def has_deerjikists = deerjikists.loaded? ? deerjikists.any? : deerjikists.exists?
+  def has_deerjikists = deerjikists.present?
 
   def self.tagme = find_or_create_by_tag_name!('タグ希望', category: :meta)
   def self.bot = find_or_create_by_tag_name!('bot操作', category: :meta)
@@ -92,22 +92,50 @@ class Tag < ApplicationRecord
 
   def self.normalise_tags! tag_names, with_tagme: true,
                                       with_no_deerjikist: true,
-                                      deny_nico: true
+                                      deny_nico: true,
+                                      with_sections: false
     if deny_nico && tag_names.any? { |n| n.downcase.start_with?('nico:') }
       raise NicoTagNormalisationError
     end
 
+    sections = { }
     tags = tag_names.map do |name|
       pf, cat = CATEGORY_PREFIXES.find { |p, _| name.downcase.start_with?(p) } || ['', nil]
-      name = TagName.canonicalise(name.sub(/\A#{ pf }/i, '')).first
+
+      name = name.sub(/\A#{ pf }/i, '')
+
+      sections_by_tag = []
+      while n = name.sub!(/^(\S*?)\[([0-9:.]*?)-([0-9:.]*?)\](\S*?)$/, '\1\4 \2 \3')
+        name, *section_raw = n.split
+
+        begin_ms, end_ms = section_raw.map { time_to_ms(_1) }
+        next if begin_ms == end_ms
+
+        begin_ms, end_ms = end_ms, begin_ms if begin_ms > end_ms
+
+        sections_by_tag << [begin_ms, end_ms]
+      end
+
+      name = TagName.canonicalise(name).first
+
       find_or_create_by_tag_name!(name, category: (cat || :general)).tap do |tag|
         tag.update!(category: cat) if cat && tag.category != cat
+        next if sections_by_tag.empty?
+
+        sections[tag.id] ||= []
+        sections[tag.id].concat(sections_by_tag)
       end
     end
 
     tags << Tag.tagme if with_tagme && tags.size < 10 && tags.none?(Tag.tagme)
     tags << Tag.no_deerjikist if with_no_deerjikist && tags.all? { |t| !(t.deerjikist?) }
-    tags.uniq(&:id)
+    tags.uniq!(&:id)
+
+    if with_sections
+      { tags:, sections: }
+    else
+      tags
+    end
   end
 
   def self.expand_parent_tags tags
@@ -228,4 +256,23 @@ class Tag < ApplicationRecord
       errors.add :category, 'ニジラーと紐づいてゐるタグはニジラー・カテゴリである必要があります．'
     end
   end
+
+  def self.time_to_ms str
+    parts = str.split(':')
+
+    s_part = parts.pop
+    s, ms = s_part.split('.')
+
+    total_s = s.to_i
+
+    if parts.length >= 1
+      total_s += parts.pop.to_i * 60
+    end
+
+    if parts.length >= 1
+      total_s += parts.pop.to_i * 3_600
+    end
+
+    total_s * 1_000 + ms.to_s.ljust(3, '0')[0, 3].to_i
+  end
 end

backend/app/models/theatre.rb

@@ -7,10 +7,6 @@ class Theatre < ApplicationRecord
            class_name: 'TheatreWatchingUser', inverse_of: :theatre
   has_many :watching_users, through: :active_theatre_watching_users, source: :user
 
-  has_many :programmes, class_name: 'TheatreProgramme'
-  has_many :skip_votes, class_name: 'TheatreSkipVote', dependent: :delete_all
-  has_many :skip_events, class_name: 'TheatreSkipEvent', dependent: :delete_all
-
   belongs_to :host_user, class_name: 'User', optional: true
   belongs_to :current_post, class_name: 'Post', optional: true
   belongs_to :created_by_user, class_name: 'User'

backend/app/models/theatre_programme.rb

@@ -1,6 +0,0 @@
-class TheatreProgramme < ApplicationRecord
-  self.primary_key = :theatre_id, :position
-
-  belongs_to :theatre
-  belongs_to :post
-end

backend/app/models/theatre_skip_event.rb

@@ -1,10 +0,0 @@
-class TheatreSkipEvent < ApplicationRecord
-  belongs_to :theatre
-  belongs_to :post
-  belongs_to :skipped_by_user, class_name: 'User'
-
-  has_many :voters, class_name: 'TheatreSkipEventVoter', dependent: :delete_all
-  has_many :event_tags, class_name: 'TheatreSkipEventTag', dependent: :delete_all
-  has_many :users, through: :voters
-  has_many :tags, through: :event_tags
-end

backend/app/models/theatre_skip_event_tag.rb

@@ -1,6 +0,0 @@
-class TheatreSkipEventTag < ApplicationRecord
-  self.primary_key = :theatre_skip_event_id, :tag_id
-
-  belongs_to :theatre_skip_event
-  belongs_to :tag
-end

backend/app/models/theatre_skip_event_voter.rb

@@ -1,6 +0,0 @@
-class TheatreSkipEventVoter < ApplicationRecord
-  self.primary_key = :theatre_skip_event_id, :user_id
-
-  belongs_to :theatre_skip_event
-  belongs_to :user
-end

backend/app/models/theatre_skip_vote.rb

@@ -1,7 +0,0 @@
-class TheatreSkipVote < ApplicationRecord
-  self.primary_key = :theatre_id, :post_id, :user_id
-
-  belongs_to :theatre
-  belongs_to :post
-  belongs_to :user
-end

backend/app/representations/post_repr.rb

@@ -2,65 +2,20 @@
 
 
 module PostRepr
-  BASE_FIELDS = [
+  BASE = { include: { tags: TagRepr::BASE, uploaded_user: UserRepr::BASE },
-      :id,
+           methods: [:parent_posts, :child_posts, :sibling_posts] }.freeze
-      :version_no,
-      :url,
-      :title,
-      :thumbnail_base,
-      :original_created_from,
-      :original_created_before,
-      :created_at,
-      :updated_at
-  ].freeze
 
   module_function
 
   def base post, current_user = nil
-    json = common(post)
+    json = post.as_json(BASE)
-    json['tags'] = tag_json(post.tags)
+    return json.merge(viewed: false) unless current_user
-    json['uploaded_user'] = post.uploaded_user && UserRepr.base(post.uploaded_user)
-    json['viewed'] = current_user ? current_user.viewed?(post) : false
-    json
-  end
 
-  def detail post, current_user = nil, parent_posts: [], child_posts: [],
+    viewed = current_user.viewed?(post)
-             sibling_posts: { }, related: []
+    json.merge(viewed:)
-    base(post, current_user).merge(
-        'parent_posts' => cards(parent_posts),
-        'child_posts' => cards(child_posts),
-        'sibling_posts' => sibling_posts.transform_keys(&:to_s).transform_values { |posts|
-          cards(posts)
-        },
-        'related' => cards(related))
-  end
-
-  def card post
-    common(post).merge('parent_posts' => [], 'child_posts' => [])
-  end
-
-  def cards posts
-    posts.map { |post| card(post) }
   end
 
   def many posts, current_user = nil
     posts.map { |p| base(p, current_user) }
   end
-
-  def common post
-    BASE_FIELDS.to_h { |field| [field.to_s, post.public_send(field)] }
-               .merge('thumbnail' => thumbnail_url(post))
-  end
-
-  def tag_json tags
-    tags.map { |tag| TagRepr.inline(tag) }
-  end
-
-  def thumbnail_url post
-    return nil unless post.thumbnail.attached?
-
-    Rails.application.routes.url_helpers.rails_blob_url(post.thumbnail, only_path: false)
-  rescue
-    nil
-  end
 end

backend/app/representations/tag_repr.rb

@@ -12,9 +12,5 @@ module TagRepr
                             parents: tag.parents.map { _1.as_json(BASE) })
   end
 
-  def inline tag
-    tag.as_json(BASE).merge(aliases: [], parents: [])
-  end
-
   def many(tags) = tags.map { |t| base(t) }
 end

backend/app/services/theatre_post_advancer.rb

@@ -1,29 +0,0 @@
-class TheatrePostAdvancer
-  def self.call(theatre:)
-    new(theatre:).call
-  end
-
-  def initialize(theatre:)
-    @theatre = theatre
-  end
-
-  def call
-    previous_post = theatre.current_post
-    post = TheatrePostSelector.new(theatre:).select
-
-    TheatreSkipVote.where(theatre:, post: previous_post).delete_all if previous_post
-
-    theatre.update!(current_post: post, current_post_started_at: post ? Time.current : nil)
-
-    if post
-      position = (theatre.programmes.maximum(:position) || 0) + 1
-      theatre.programmes.create!(position:, post:)
-    end
-
-    post
-  end
-
-  private
-
-  attr_reader :theatre
-end

backend/app/services/theatre_post_selector.rb

@@ -1,119 +0,0 @@
-class TheatrePostSelector
-  Candidate = Struct.new(:post, :weight, :penalty, :tags, keyword_init: true)
-  ELIGIBLE_POST_URL_CONDITION =
-      ["url LIKE '%nicovideo.jp%'",
-       "url LIKE '%youtube.com/watch%'",
-       "url LIKE '%youtu.be/%'"]
-      .join(' OR ')
-
-  def initialize theatre:
-    @theatre = theatre
-  end
-
-  def select
-    candidates = weighted_candidates
-    return nil if candidates.empty?
-
-    total = candidates.sum(&:weight)
-    target = rand * total
-
-    candidates.each do |candidate|
-      target -= candidate.weight
-      return candidate.post if target <= 0
-    end
-
-    candidates.last.post
-  end
-
-  def weight_json limit: 20
-    candidates = weighted_candidates
-    sorted = candidates.sort_by { |candidate| [candidate.weight, candidate.post.id] }
-
-    { tag_penalties: tag_penalty_json,
-      lightest_posts: post_weight_json(sorted.first(limit)),
-      heaviest_posts: post_weight_json(sorted.reverse.first(limit)) }
-  end
-
-  private
-
-  attr_reader :theatre
-
-  def weighted_candidates
-    @weighted_candidates ||= begin
-      penalties = tag_penalties
-      posts = eligible_posts.includes(tags: :tag_name).to_a
-
-      posts.map do |post|
-        post_tags = post.tags.to_a
-        penalty = post_tags.sum { |tag| penalties[tag.id].to_i }
-
-        Candidate.new(
-            post:,
-            penalty:,
-            tags: post_tags,
-            weight: 1.0 / (1.0 + penalty))
-      end
-    end
-  end
-
-  def eligible_posts
-    posts = Post.where(ELIGIBLE_POST_URL_CONDITION)
-    posts = posts.where.not(id: theatre.current_post_id) if theatre.current_post_id
-    posts
-  end
-
-  def active_user_ids
-    @active_user_ids ||= theatre.watching_users.ids
-  end
-
-  def tag_penalties
-    @tag_penalties ||=
-      if active_user_ids.empty?
-        {}
-      else
-        TheatreSkipEventVoter
-          .joins(theatre_skip_event: :event_tags)
-          .where(user_id: active_user_ids)
-          .group('theatre_skip_event_tags.tag_id')
-          .count
-      end
-  end
-
-  def tag_penalty_json
-    return [] if tag_penalties.empty?
-
-    tags = Tag.where(id: tag_penalties.keys).includes(:tag_name).index_by(&:id)
-
-    tag_penalties
-      .map { |tag_id, penalty|
-        tag = tags[tag_id]
-        next unless tag
-
-        { tag: light_tag_json(tag),
-          penalty: }
-      }
-      .compact
-      .sort_by { |row| [-row[:penalty], row[:tag][:name].to_s] }
-  end
-
-  def post_weight_json candidates
-    candidates.map { |candidate|
-      { post: light_post_json(candidate.post),
-        weight: candidate.weight,
-        penalty: candidate.penalty,
-        tags: candidate.tags.map { |tag| light_tag_json(tag) } }
-    }
-  end
-
-  def light_post_json post
-    { id: post.id,
-      title: post.title,
-      url: post.url }
-  end
-
-  def light_tag_json tag
-    { id: tag.id,
-      name: tag.name,
-      category: tag.category }
-  end
-end

backend/app/services/theatre_skip_finalizer.rb

@@ -1,40 +0,0 @@
-class TheatreSkipFinalizer
-  def self.call(theatre:, user:)
-    new(theatre:, user:).call
-  end
-
-  def initialize(theatre:, user:)
-    @theatre = theatre
-    @user = user
-  end
-
-  def call
-    return unless theatre.current_post
-
-    post = theatre.current_post
-    voters = TheatreSkipVote.where(theatre:, post:).includes(:user).map(&:user)
-    return if voters.empty?
-
-    event = TheatreSkipEvent.create!(
-        theatre:,
-        post:,
-        skipped_by_user: user,
-        programme_position: theatre.programmes.maximum(:position))
-
-    voters.uniq(&:id).each do |voter|
-      TheatreSkipEventVoter.create!(theatre_skip_event: event, user: voter)
-    end
-
-    post.tags.find_each do |tag|
-      TheatreSkipEventTag.create!(theatre_skip_event: event, tag:)
-    end
-
-    TheatreSkipVote.where(theatre:, post:).delete_all
-
-    event
-  end
-
-  private
-
-  attr_reader :theatre, :user
-end

backend/config/routes.rb

@@ -85,14 +85,9 @@ Rails.application.routes.draw do
     member do
       put :watching
       patch :next_post
-      put :skip_vote
-      delete :skip_vote, action: :unskip_vote
-      get :post_selection_weights
     end
 
-    resources :comments, controller: :theatre_comments, only: [:index, :create, :destroy]
+    resources :comments, controller: :theatre_comments, only: [:index, :create]
-    resources :programmes, controller: :theatre_programmes, only: [:index]
-    resources :skip_events, controller: :theatre_skip_events, only: [:index]
   end
 
   resources :materials, only: [:index, :show, :create, :update, :destroy]

backend/db/migrate/20260514221900_create_theatre_programmes.rb

@@ -1,10 +0,0 @@
-class CreateTheatreProgrammes < ActiveRecord::Migration[8.0]
-  def change
-    create_table :theatre_programmes, primary_key: [:theatre_id, :position] do |t|
-      t.references :theatre, null: false, foreign_key: true
-      t.integer :position, null: false
-      t.references :post, null: false, foreign_key: true
-      t.datetime :created_at, null: false
-    end
-  end
-end

backend/db/migrate/20260518235300_create_post_tag_sections.rb

@@ -0,0 +1,19 @@
+class CreatePostTagSections < ActiveRecord::Migration[8.0]
+  def change
+    create_table :post_tag_sections, primary_key: [:post_id, :tag_id, :begin_ms] do |t|
+      t.references :post, null: false, foreign_key: true, index: false
+      t.references :tag, null: false, foreign_key: true, index: false
+      t.integer :begin_ms, null: false
+      t.integer :end_ms, null: false
+      t.timestamps
+
+      t.index [:post_id, :begin_ms], name: 'idx_post_tag_sections_post_id_begin_ms'
+
+      t.check_constraint 'begin_ms >= 0',
+                         name: 'chk_post_tag_sections_begin_ms_natural'
+
+      t.check_constraint 'begin_ms < end_ms',
+                         name: 'chk_post_tag_sections_end_ms_after_begin_ms'
+    end
+  end
+end

backend/db/migrate/20260519013100_add_video_ms_to_posts.rb

@@ -0,0 +1,9 @@
+class AddVideoMsToPosts < ActiveRecord::Migration[8.0]
+  def change
+    add_column :posts, :video_ms, :integer
+    add_index :posts, [:video_ms, :id], name: 'idx_posts_video_ms_id'
+
+    add_check_constraint :posts, 'video_ms IS NULL OR video_ms > 0',
+                         name: 'chk_posts_video_ms_positive'
+  end
+end

backend/db/migrate/20260606000000_create_theatre_skip_votes_and_events.rb

@@ -1,36 +0,0 @@
-class CreateTheatreSkipVotesAndEvents < ActiveRecord::Migration[8.0]
-  def change
-    create_table :theatre_skip_votes, primary_key: [:theatre_id, :post_id, :user_id] do |t|
-      t.references :theatre, null: false, foreign_key: true
-      t.references :post, null: false, foreign_key: true
-      t.references :user, null: false, foreign_key: true
-      t.timestamps
-    end
-
-    create_table :theatre_skip_events do |t|
-      t.references :theatre, null: false, foreign_key: true
-      t.references :post, null: false, foreign_key: true
-      t.references :skipped_by_user, null: false, foreign_key: { to_table: :users }
-      t.integer :programme_position
-      t.datetime :created_at, null: false
-    end
-
-    create_table :theatre_skip_event_voters, primary_key: [:theatre_skip_event_id, :user_id] do |t|
-      t.references :theatre_skip_event, null: false, foreign_key: true
-      t.references :user, null: false, foreign_key: true
-    end
-
-    create_table :theatre_skip_event_tags, primary_key: [:theatre_skip_event_id, :tag_id] do |t|
-      t.references :theatre_skip_event, null: false, foreign_key: true
-      t.references :tag, null: false, foreign_key: true
-    end
-
-    add_index :theatre_skip_events, [:theatre_id, :created_at]
-    add_index :theatre_skip_votes, [:theatre_id, :post_id, :created_at],
-              name: 'idx_theatre_skip_votes_theatre_post_created'
-    add_index :theatre_skip_event_voters, [:user_id, :theatre_skip_event_id],
-              name: 'idx_theatre_skip_event_voters_user_event'
-    add_index :theatre_skip_event_tags, [:tag_id, :theatre_skip_event_id],
-              name: 'idx_theatre_skip_event_tags_tag_event'
-  end
-end

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_06_06_000000) do
+ActiveRecord::Schema[8.0].define(version: 2026_05_19_013100) 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
@@ -137,6 +137,19 @@ ActiveRecord::Schema[8.0].define(version: 2026_06_06_000000) do
     t.index ["target_post_id"], name: "index_post_similarities_on_target_post_id"
   end
 
+  create_table "post_tag_sections", primary_key: ["post_id", "tag_id", "begin_ms"], charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
+    t.bigint "post_id", null: false
+    t.bigint "tag_id", null: false
+    t.integer "begin_ms", null: false
+    t.integer "end_ms", null: false
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["post_id", "begin_ms"], name: "idx_post_tag_sections_post_id_begin_ms"
+    t.index ["tag_id"], name: "fk_rails_8be3847903"
+    t.check_constraint "`begin_ms` < `end_ms`", name: "chk_post_tag_sections_end_ms_after_begin_ms"
+    t.check_constraint "`begin_ms` >= 0", name: "chk_post_tag_sections_begin_ms_natural"
+  end
+
   create_table "post_tags", charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
     t.bigint "post_id", null: false
     t.bigint "tag_id", null: false
@@ -187,8 +200,11 @@ ActiveRecord::Schema[8.0].define(version: 2026_06_06_000000) do
     t.datetime "original_created_before"
     t.datetime "updated_at", null: false
     t.integer "version_no", null: false
+    t.integer "video_ms"
     t.index ["uploaded_user_id"], name: "index_posts_on_uploaded_user_id"
     t.index ["url"], name: "index_posts_on_url", unique: true
+    t.index ["video_ms", "id"], name: "idx_posts_video_ms_id"
+    t.check_constraint "(`video_ms` is null) or (`video_ms` > 0)", name: "chk_posts_video_ms_positive"
     t.check_constraint "`version_no` > 0", name: "chk_posts_version_no_positive"
   end
 
@@ -283,55 +299,6 @@ ActiveRecord::Schema[8.0].define(version: 2026_06_06_000000) do
     t.index ["user_id"], name: "index_theatre_comments_on_user_id"
   end
 
-  create_table "theatre_programmes", primary_key: ["theatre_id", "position"], charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
-    t.bigint "theatre_id", null: false
-    t.integer "position", null: false
-    t.bigint "post_id", null: false
-    t.datetime "created_at", null: false
-    t.index ["post_id"], name: "index_theatre_programmes_on_post_id"
-    t.index ["theatre_id"], name: "index_theatre_programmes_on_theatre_id"
-  end
-
-  create_table "theatre_skip_event_tags", primary_key: ["theatre_skip_event_id", "tag_id"], charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
-    t.bigint "theatre_skip_event_id", null: false
-    t.bigint "tag_id", null: false
-    t.index ["tag_id", "theatre_skip_event_id"], name: "idx_theatre_skip_event_tags_tag_event"
-    t.index ["tag_id"], name: "index_theatre_skip_event_tags_on_tag_id"
-    t.index ["theatre_skip_event_id"], name: "index_theatre_skip_event_tags_on_theatre_skip_event_id"
-  end
-
-  create_table "theatre_skip_event_voters", primary_key: ["theatre_skip_event_id", "user_id"], charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
-    t.bigint "theatre_skip_event_id", null: false
-    t.bigint "user_id", null: false
-    t.index ["theatre_skip_event_id"], name: "index_theatre_skip_event_voters_on_theatre_skip_event_id"
-    t.index ["user_id", "theatre_skip_event_id"], name: "idx_theatre_skip_event_voters_user_event"
-    t.index ["user_id"], name: "index_theatre_skip_event_voters_on_user_id"
-  end
-
-  create_table "theatre_skip_events", charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
-    t.bigint "theatre_id", null: false
-    t.bigint "post_id", null: false
-    t.bigint "skipped_by_user_id", null: false
-    t.integer "programme_position"
-    t.datetime "created_at", null: false
-    t.index ["post_id"], name: "index_theatre_skip_events_on_post_id"
-    t.index ["skipped_by_user_id"], name: "index_theatre_skip_events_on_skipped_by_user_id"
-    t.index ["theatre_id", "created_at"], name: "index_theatre_skip_events_on_theatre_id_and_created_at"
-    t.index ["theatre_id"], name: "index_theatre_skip_events_on_theatre_id"
-  end
-
-  create_table "theatre_skip_votes", primary_key: ["theatre_id", "post_id", "user_id"], charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
-    t.bigint "theatre_id", null: false
-    t.bigint "post_id", null: false
-    t.bigint "user_id", null: false
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
-    t.index ["post_id"], name: "index_theatre_skip_votes_on_post_id"
-    t.index ["theatre_id", "post_id", "created_at"], name: "idx_theatre_skip_votes_theatre_post_created"
-    t.index ["theatre_id"], name: "index_theatre_skip_votes_on_theatre_id"
-    t.index ["user_id"], name: "index_theatre_skip_votes_on_user_id"
-  end
-
   create_table "theatre_watching_users", primary_key: ["theatre_id", "user_id"], charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
     t.bigint "theatre_id", null: false
     t.bigint "user_id", null: false
@@ -339,6 +306,7 @@ ActiveRecord::Schema[8.0].define(version: 2026_06_06_000000) do
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
     t.index ["expires_at"], name: "index_theatre_watching_users_on_expires_at"
+    t.index ["theatre_id", "expires_at"], name: "idx_on_theatre_id_skip_expires_at_4c8de1dd42"
     t.index ["theatre_id", "expires_at"], name: "index_theatre_watching_users_on_theatre_id_and_expires_at"
     t.index ["theatre_id"], name: "index_theatre_watching_users_on_theatre_id"
     t.index ["user_id"], name: "index_theatre_watching_users_on_user_id"
@@ -495,6 +463,8 @@ ActiveRecord::Schema[8.0].define(version: 2026_06_06_000000) do
   add_foreign_key "post_implications", "posts", column: "parent_post_id"
   add_foreign_key "post_similarities", "posts"
   add_foreign_key "post_similarities", "posts", column: "target_post_id"
+  add_foreign_key "post_tag_sections", "posts"
+  add_foreign_key "post_tag_sections", "tags"
   add_foreign_key "post_tags", "posts"
   add_foreign_key "post_tags", "tags"
   add_foreign_key "post_tags", "users", column: "created_user_id"
@@ -513,18 +483,6 @@ ActiveRecord::Schema[8.0].define(version: 2026_06_06_000000) do
   add_foreign_key "tags", "tag_names"
   add_foreign_key "theatre_comments", "theatres"
   add_foreign_key "theatre_comments", "users"
-  add_foreign_key "theatre_programmes", "posts"
-  add_foreign_key "theatre_programmes", "theatres"
-  add_foreign_key "theatre_skip_event_tags", "tags"
-  add_foreign_key "theatre_skip_event_tags", "theatre_skip_events"
-  add_foreign_key "theatre_skip_event_voters", "theatre_skip_events"
-  add_foreign_key "theatre_skip_event_voters", "users"
-  add_foreign_key "theatre_skip_events", "posts"
-  add_foreign_key "theatre_skip_events", "theatres"
-  add_foreign_key "theatre_skip_events", "users", column: "skipped_by_user_id"
-  add_foreign_key "theatre_skip_votes", "posts"
-  add_foreign_key "theatre_skip_votes", "theatres"
-  add_foreign_key "theatre_skip_votes", "users"
   add_foreign_key "theatre_watching_users", "theatres"
   add_foreign_key "theatre_watching_users", "users"
   add_foreign_key "theatres", "posts", column: "current_post_id"

backend/spec/factories/post_tag_sections.rb

@@ -0,0 +1,8 @@
+FactoryBot.define do
+  factory :post_tag_section do
+    association :post
+    association :tag
+    begin_ms { 1_000 }
+    end_ms { 2_000 }
+  end
+end

backend/spec/factories/post_tags.rb

@@ -0,0 +1,6 @@
+FactoryBot.define do
+  factory :post_tag do
+    association :post
+    association :tag
+  end
+end

backend/spec/factories/posts.rb

@@ -0,0 +1,8 @@
+FactoryBot.define do
+  factory :post do
+    sequence(:url) { |n| "https://example.com/factory-post-#{ n }" }
+    title { 'factory post' }
+    thumbnail_base { nil }
+    uploaded_user { nil }
+  end
+end

backend/spec/models/post_tag_spec.rb

@@ -0,0 +1,29 @@
+RSpec.describe PostTag, type: :model do
+  describe '#sections' do
+    it 'loads sections by post_id and tag_id' do
+      post_tag = create(:post_tag)
+      section = create(:post_tag_section,
+                       post: post_tag.post,
+                       tag: post_tag.tag,
+                       begin_ms: 1000,
+                       end_ms: 2000)
+
+      expect(post_tag.sections).to contain_exactly(section)
+    end
+
+    it 'does not load sections for another tag on the same post' do
+      post = create(:post)
+      tag = create(:tag)
+      other_tag = create(:tag)
+
+      post_tag = create(:post_tag, post:, tag:)
+      create(:post_tag_section,
+             post:,
+             tag: other_tag,
+             begin_ms: 1000,
+             end_ms: 2000)
+
+      expect(post_tag.sections).to be_empty
+    end
+  end
+end

backend/spec/requests/error_responses_spec.rb

@@ -1,47 +0,0 @@
-require 'rails_helper'
-
-RSpec.describe 'error responses', type: :request do
-  describe 'manual input errors' do
-    it 'returns a stable payload for bad requests' do
-      get '/tags/name/%20/deerjikists'
-
-      expect(response).to have_http_status(:bad_request)
-      expect(json).to include(
-          'type' => 'bad_request',
-          'message' => be_present,
-          'errors' => {},
-          'base_errors' => [be_present])
-    end
-
-    it 'returns a stable field-error payload for unprocessable requests' do
-      member = create(:user, :member)
-      tag = create(:tag, :general, name: 'error_response_tag')
-      sign_in_as(member)
-
-      patch "/tags/#{ tag.id }", params: { category: 'nico' }
-
-      expect(response).to have_http_status(:unprocessable_entity)
-      expect(json).to include(
-          'type' => 'validation_error',
-          'message' => '入力内容を確認してください．',
-          'base_errors' => [])
-      expect(json.fetch('errors')).to include(
-          'category' => ['ニコタグは変更できません．'])
-    end
-  end
-
-  describe 'model validation errors' do
-    it 'returns field messages for model errors' do
-      user = create(:user)
-      sign_in_as(user)
-
-      put "/users/#{ user.id }", params: { name: 'a' * 256 }
-
-      expect(response).to have_http_status(:unprocessable_entity)
-      expect(json).to include(
-          'type' => 'validation_error',
-          'message' => '入力内容を確認してください．')
-      expect(json.fetch('errors').fetch('name')).to include(be_present)
-    end
-  end
-end

backend/spec/requests/materials_spec.rb

@@ -141,21 +141,16 @@ RSpec.describe 'Materials API', type: :request do
     context 'when logged in' do
       before { sign_in_as(guest_user) }
 
-      it 'returns 422 when tag is blank' do
+      it 'returns 400 when tag is blank' do
         post '/materials', params: { tag: '   ', file: dummy_upload }
 
-        expect(response).to have_http_status(:unprocessable_entity)
+        expect(response).to have_http_status(:bad_request)
-        expect(json.fetch('errors')).to include(
-            'tag' => ['タグは必須です．'])
       end
 
-      it 'returns 422 when both file and url are blank' do
+      it 'returns 400 when both file and url are blank' do
         post '/materials', params: { tag: 'material_create_blank' }
 
-        expect(response).to have_http_status(:unprocessable_entity)
+        expect(response).to have_http_status(:bad_request)
-        expect(json.fetch('errors')).to include(
-            'file' => ['ファイルまたは URL は必須です．'],
-            'url' => ['ファイルまたは URL は必須です．'])
       end
 
       it 'creates a material with an attached file' do
@@ -266,26 +261,21 @@ RSpec.describe 'Materials API', type: :request do
         expect(response).to have_http_status(:not_found)
       end
 
-      it 'returns 422 when tag is blank' do
+      it 'returns 400 when tag is blank' do
         put "/materials/#{ material.id }", params: {
           tag: '   ',
           file: dummy_upload
         }
 
-        expect(response).to have_http_status(:unprocessable_entity)
+        expect(response).to have_http_status(:bad_request)
-        expect(json.fetch('errors')).to include(
-            'tag' => ['タグは必須です．'])
       end
 
-      it 'returns 422 when both file and url are blank' do
+      it 'returns 400 when both file and url are blank' do
         put "/materials/#{ material.id }", params: {
           tag: 'material_update_no_payload'
         }
 
-        expect(response).to have_http_status(:unprocessable_entity)
+        expect(response).to have_http_status(:bad_request)
-        expect(json.fetch('errors')).to include(
-            'file' => ['ファイルまたは URL は必須です．'],
-            'url' => ['ファイルまたは URL は必須です．'])
       end
 
       it 'updates tag, url, file, and updated_by_user' do

backend/spec/requests/nico_tags_spec.rb

@@ -3,68 +3,12 @@ require 'rails_helper'
 
 RSpec.describe 'NicoTags', type: :request do
   describe 'GET /tags/nico' do
-    it 'returns paginated tags and total count' do
+    it 'returns tags and next_cursor when overflowing limit' do
-      create_list(:tag, 3, :nico)
+      create_list(:tag, 21, :nico)
-
+      get '/tags/nico', params: { limit: 20 }
-      get '/tags/nico', params: { page: 2, limit: 2 }
-
       expect(response).to have_http_status(:ok)
-      expect(json['tags'].size).to eq(1)
+      expect(json['tags'].size).to eq(20)
-      expect(json['count']).to eq(3)
+      expect(json['next_cursor']).to be_present
-    end
-
-    it 'filters by nico tag name, linked tag name, and link status' do
-      linked = create(:tag, :nico)
-      linked.tag_name.update!(name: 'nico:search_linked')
-      unlinked = create(:tag, :nico)
-      unlinked.tag_name.update!(name: 'nico:search_unlinked')
-      other = create(:tag, :nico)
-      other.tag_name.update!(name: 'nico:other')
-      destination = create(:tag, :general)
-      destination.tag_name.update!(name: 'destination_search')
-      NicoTagRelation.create!(nico_tag: linked, tag: destination)
-      NicoTagRelation.create!(nico_tag: other, tag: create(:tag, :general))
-
-      get '/tags/nico', params: {
-        name: 'search_',
-        linked_tag: 'destination_',
-        link_status: 'linked'
-      }
-
-      expect(response).to have_http_status(:ok)
-      expect(json['count']).to eq(1)
-      expect(json.fetch('tags').map { |tag| tag['id'] }).to eq([linked.id])
-
-      get '/tags/nico', params: { name: 'search_', link_status: 'unlinked' }
-
-      expect(json['count']).to eq(1)
-      expect(json.fetch('tags').map { |tag| tag['id'] }).to eq([unlinked.id])
-    end
-
-    it 'sorts by name and timestamps' do
-      older = create(:tag, :nico)
-      older.tag_name.update!(name: 'nico:a')
-      older.update_columns(created_at: 2.days.ago)
-      newer = create(:tag, :nico)
-      newer.tag_name.update!(name: 'nico:b')
-      newer.update_columns(created_at: 1.day.ago)
-      older_post_tag =
-        PostTag.create!(post: Post.create!(url: 'https://example.com/nico-older'), tag: older)
-      older_post_tag.update_columns(created_at: 1.hour.ago)
-      newer_post_tag =
-        PostTag.create!(post: Post.create!(url: 'https://example.com/nico-newer'), tag: newer)
-      newer_post_tag.update_columns(created_at: 2.hours.ago)
-
-      get '/tags/nico', params: { order: 'name:desc' }
-      expect(json.fetch('tags').map { |tag| tag['id'] }).to eq([newer.id, older.id])
-
-      get '/tags/nico', params: { order: 'created_at:asc' }
-      expect(json.fetch('tags').map { |tag| tag['id'] }).to eq([older.id, newer.id])
-
-      get '/tags/nico', params: { order: 'updated_at:desc' }
-      expect(json.fetch('tags').map { |tag| tag['id'] }).to eq([older.id, newer.id])
-      expect(Time.zone.parse(json.fetch('tags').first.fetch('recent_post_tag_created_at')))
-        .to be_within(1.second).of(older_post_tag.created_at)
     end
   end
 
@@ -131,7 +75,7 @@ RSpec.describe 'NicoTags', type: :request do
       expect(versions.last.created_by_user_id).to eq(admin.id)
     end
 
-    it 'returns 422 when linked tag normalises to nico tag' do
+    it '400 when linked tag normalises to nico tag' do
       sign_in_as(member)
 
       other_nico = create(:tag, :nico, name: 'nico:linked_ng')
@@ -143,37 +87,7 @@ RSpec.describe 'NicoTags', type: :request do
         patch "/tags/nico/#{nico_tag.id}", params: { tags: 'linked_ng_alias' }
       }.not_to change(NicoTagVersion, :count)
 
-      expect(response).to have_http_status(:unprocessable_entity)
+      expect(response).to have_http_status(:bad_request)
-      expect(json.fetch('errors')).to include(
-          'tags' => ['ニコニコ・タグ同士は連携できません．'])
-    end
-
-    it 'returns the tags field error when a nico tag is specified directly' do
-      sign_in_as(member)
-
-      patch "/tags/nico/#{nico_tag.id}", params: { tags: 'nico:linked_ng' }
-
-      expect(response).to have_http_status(:unprocessable_entity)
-      expect(json.fetch('errors')).to include(
-          'tags' => ['ニコニコ・タグ同士は連携できません．'])
-    end
-
-    it 'returns tag name validation errors on the tags field and rolls back created tags' do
-      sign_in_as(member)
-      TagNameSanitisationRule.create!(
-        priority: 1,
-        source_pattern: 'invalid',
-        replacement: 'valid'
-      )
-      nico_tag
-
-      expect {
-        patch "/tags/nico/#{nico_tag.id}", params: { tags: 'created_first invalid' }
-      }.not_to change(TagName, :count)
-
-      expect(response).to have_http_status(:unprocessable_entity)
-      expect(json.fetch('errors').fetch('tags')).to include(
-          a_string_including('タグ名 “invalid”:', '名前に使用できない文字が含まれてゐます．'))
     end
   end
 end

backend/spec/requests/posts_spec.rb

@@ -57,23 +57,6 @@ RSpec.describe 'Posts API', type: :request do
     post_write_params({ base_version_no: base_version.version_no }.merge(params))
   end
 
-  def count_sql_queries
-    count = 0
-
-    callback = lambda do |_name, _started, _finished, _id, payload|
-      next if payload[:cached]
-      next if ['SCHEMA', 'TRANSACTION'].include?(payload[:name])
-
-      count += 1
-    end
-
-    ActiveSupport::Notifications.subscribed(callback, 'sql.active_record') do
-      yield
-    end
-
-    count
-  end
-
   let!(:tag_name) { TagName.create!(name: 'spec_tag') }
   let!(:tag)      { Tag.create!(tag_name: tag_name, category: :general) }
 
@@ -575,59 +558,6 @@ RSpec.describe 'Posts API', type: :request do
           expect(sibling_ids).to include(sibling_post.id)
         end
       end
-
-      it 'does not issue a query per tag or related post' do
-        user = create_member_user!
-
-        tags =
-            15.times.map do |i|
-              tag_name = TagName.create!(name: "show_query_tag_#{ i }")
-              tag = Tag.create!(tag_name:, category: :general)
-              TagName.create!(name: "show_query_alias_#{ i }", canonical: tag_name)
-              PostTag.create!(post: post_record, tag:)
-              tag
-            end
-
-        tags.each_cons(2) do |parent_tag, child_tag|
-          TagImplication.create!(parent_tag:, tag: child_tag)
-        end
-
-        parent_post = Post.create!(
-          title: 'query parent post',
-          url: 'https://example.com/query-parent-post'
-        )
-        sibling_post = Post.create!(
-          title: 'query sibling post',
-          url: 'https://example.com/query-sibling-post'
-        )
-        child_post = Post.create!(
-          title: 'query child post',
-          url: 'https://example.com/query-child-post'
-        )
-
-        PostImplication.create!(post: post_record, parent_post:)
-        PostImplication.create!(post: sibling_post, parent_post:)
-        PostImplication.create!(post: child_post, parent_post: post_record)
-
-        20.times do |i|
-          related_post = Post.create!(
-            title: "query related post #{ i }",
-            url: "https://example.com/query-related-post-#{ i }"
-          )
-          PostSimilarity.create!(post: post_record,
-                                 target_post: related_post,
-                                 cos: 1.0 - (i / 100.0))
-        end
-
-        query_count =
-            count_sql_queries do
-              get "/posts/#{ post_record.id }",
-                  headers: { 'X-Transfer-Code' => user.inheritance_code }
-            end
-
-        expect(response).to have_http_status(:ok)
-        expect(query_count).to be <= 45
-      end
     end
 
     context 'when post does not exist' do
@@ -704,7 +634,7 @@ RSpec.describe 'Posts API', type: :request do
             category: :nico)
       end
 
-      it 'returns 422 with tag field errors' do
+      it 'return 400' do
         sign_in_as(member)
 
         post '/posts', params: post_write_params(
@@ -714,13 +644,7 @@ RSpec.describe 'Posts API', type: :request do
           thumbnail: dummy_upload
         )
 
-        expect(response).to have_http_status(:unprocessable_entity), response.body
+        expect(response).to have_http_status(:bad_request), response.body
-        expect(json).to include(
-            'type' => 'validation_error',
-            'message' => '入力内容を確認してください．',
-            'base_errors' => [])
-        expect(json.fetch('errors')).to include(
-            'tags' => ['ニコニコ・タグは直接指定できません．'])
       end
     end
 
@@ -937,7 +861,7 @@ RSpec.describe 'Posts API', type: :request do
             category: :nico)
       end
 
-      it 'returns 422 with tag field errors' do
+      it 'return 400' do
         sign_in_as(member)
 
         put "/posts/#{post_record.id}", params: post_update_params(
@@ -945,13 +869,7 @@ RSpec.describe 'Posts API', type: :request do
             title: 'updated title',
             tags: 'nico:nico_tag')
 
-        expect(response).to have_http_status(:unprocessable_entity), response.body
+        expect(response).to have_http_status(:bad_request), response.body
-        expect(json).to include(
-            'type' => 'validation_error',
-            'message' => '入力内容を確認してください．',
-            'base_errors' => [])
-        expect(json.fetch('errors')).to include(
-            'tags' => ['ニコニコ・タグは直接指定できません．'])
       end
     end
 

backend/spec/requests/tags_deerjikists_spec.rb

@@ -275,30 +275,6 @@ RSpec.describe 'Tags deerjikists API', type: :request do
         end
       end
 
-      context 'when a row is invalid' do
-        let(:payload) do
-          [
-            { platform: '', code: code1 },
-          ]
-        end
-
-        it 'returns 422 with indexed field errors and does not replace existing deerjikists' do
-          Deerjikist.create!(platform: platform1, code: code1, tag: tag)
-
-          expect {
-            do_request
-          }.not_to change { Deerjikist.where(tag: tag).map { |d| [d.platform, d.code] } }
-
-          expect(response).to have_http_status(:unprocessable_entity)
-          expect(json).to include(
-              'type' => 'validation_error',
-              'message' => '入力内容を確認してください．',
-              'base_errors' => [])
-          expect(json.fetch('errors')).to include(
-              'deerjikists.0.platform' => [be_present])
-        end
-      end
-
       context 'when youtube code is handle' do
         let(:channel_id) { 'UCabcdefghijklmnopqrstuv' }
         let(:payload) do

backend/spec/requests/theatre_comments_spec.rb

@@ -80,26 +80,6 @@ RSpec.describe 'TheatreComments', type: :request do
       expect(response).to have_http_status(:ok)
       expect(response.parsed_body.map { |row| row['no'] }).to eq([3, 2, 1])
     end
-
-    it '削除済みコメントは deleted として返し、本文を隠す' do
-      comment_2.discard!
-
-      get "/theatres/#{theatre.id}/comments", params: { no_gt: 1 }
-
-      expect(response).to have_http_status(:ok)
-
-      deleted_comment = response.parsed_body.find { _1['no'] == 2 }
-      expect(deleted_comment).to include(
-        'deleted' => true,
-        'content' => nil
-      )
-
-      visible_comment = response.parsed_body.find { _1['no'] == 3 }
-      expect(visible_comment).to include(
-        'deleted' => false,
-        'content' => 'third comment'
-      )
-    end
   end
 
   describe 'POST /theatres/:theatre_id/comments' do
@@ -167,44 +147,4 @@ RSpec.describe 'TheatreComments', type: :request do
       })
     end
   end
-
-  describe 'DELETE /theatres/:theatre_id/comments/:id' do
-    let(:theatre) { create(:theatre) }
-    let(:alice) { create(:user, name: 'Alice') }
-    let(:bob) { create(:user, name: 'Bob') }
-    let!(:comment) do
-      create(
-        :theatre_comment,
-        theatre: theatre,
-        no: 1,
-        user: alice,
-        content: 'delete target'
-      )
-    end
-
-    it 'returns 401 when not logged in' do
-      delete "/theatres/#{theatre.id}/comments/#{comment.no}"
-
-      expect(response).to have_http_status(:unauthorized)
-      expect(comment.reload.discarded?).to eq(false)
-    end
-
-    it 'allows the comment owner to delete it' do
-      sign_in_as(alice)
-
-      delete "/theatres/#{theatre.id}/comments/#{comment.no}"
-
-      expect(response).to have_http_status(:no_content)
-      expect(comment.reload.discarded?).to eq(true)
-    end
-
-    it 'returns 403 when another user tries to delete it' do
-      sign_in_as(bob)
-
-      delete "/theatres/#{theatre.id}/comments/#{comment.no}"
-
-      expect(response).to have_http_status(:forbidden)
-      expect(comment.reload.discarded?).to eq(false)
-    end
-  end
 end

backend/spec/requests/theatre_programmes_spec.rb

@@ -1,38 +0,0 @@
-require 'rails_helper'
-
-RSpec.describe 'TheatreProgrammes', type: :request do
-  describe 'GET /theatres/:theatre_id/programmes' do
-    let(:theatre) { create(:theatre) }
-    let(:other_theatre) { create(:theatre) }
-    let(:post_1) { Post.create!(title: 'first', url: 'https://www.nicovideo.jp/watch/sm1') }
-    let(:post_2) { Post.create!(title: 'second', url: 'https://www.nicovideo.jp/watch/sm2') }
-    let(:other_post) { Post.create!(title: 'other', url: 'https://www.nicovideo.jp/watch/sm3') }
-
-    before do
-      TheatreProgramme.create!(theatre:, position: 1, post: post_1, created_at: 2.minutes.ago)
-      TheatreProgramme.create!(theatre:, position: 2, post: post_2, created_at: 1.minute.ago)
-      TheatreProgramme.create!(
-        theatre: other_theatre,
-        position: 1,
-        post: other_post,
-        created_at: 1.minute.ago
-      )
-    end
-
-    it 'returns programmes for the theatre in descending position with post json' do
-      get "/theatres/#{theatre.id}/programmes"
-
-      expect(response).to have_http_status(:ok)
-      expect(json.map { _1['position'] }).to eq([2, 1])
-      expect(json.map { _1.dig('post', 'title') }).to eq(['second', 'first'])
-      expect(json.first['post']).to include('id' => post_2.id, 'url' => post_2.url)
-    end
-
-    it 'filters programmes by position_gt' do
-      get "/theatres/#{theatre.id}/programmes", params: { position_gt: 1 }
-
-      expect(response).to have_http_status(:ok)
-      expect(json.map { _1['position'] }).to eq([2])
-    end
-  end
-end

backend/spec/requests/theatres_spec.rb

@@ -14,24 +14,10 @@ RSpec.describe 'Theatres API', type: :request do
   let(:member) { create(:user, :member, name: 'member user') }
   let(:other_user) { create(:user, :member, name: 'other user') }
 
-  let!(:niconico_post) do
-    Post.create!(
-      title: 'niconico post',
-      url: 'https://www.nicovideo.jp/watch/sm123'
-    )
-  end
-
-  let!(:second_niconico_post) do
-    Post.create!(
-      title: 'second niconico post',
-      url: 'https://www.nicovideo.jp/watch/sm456'
-    )
-  end
-
   let!(:youtube_post) do
     Post.create!(
       title: 'youtube post',
-      url: 'https://www.youtube.com/watch?v=yt123'
+      url: 'https://www.youtube.com/watch?v=spec123'
     )
   end
 
@@ -134,8 +120,7 @@ RSpec.describe 'Theatres API', type: :request do
         expect(json).to include(
           'host_flg' => true,
           'post_id' => nil,
-          'post_started_at' => nil,
+          'post_started_at' => nil
-          'post_elapsed_ms' => nil
         )
 
         expect(json.fetch('watching_users')).to contain_exactly(
@@ -192,8 +177,7 @@ RSpec.describe 'Theatres API', type: :request do
         expect(json).to include(
           'host_flg' => false,
           'post_id' => nil,
-          'post_started_at' => nil,
+          'post_started_at' => nil
-          'post_elapsed_ms' => nil
         )
 
         expect(json.fetch('watching_users')).to contain_exactly(
@@ -220,7 +204,7 @@ RSpec.describe 'Theatres API', type: :request do
         )
         theatre.update!(
           host_user: other_user,
-          current_post: niconico_post,
+          current_post: youtube_post,
           current_post_started_at: started_at
         )
         sign_in_as(member)
@@ -236,11 +220,9 @@ RSpec.describe 'Theatres API', type: :request do
         expect(theatre.host_user_id).to eq(member.id)
 
         expect(json['host_flg']).to eq(true)
-        expect(json['post_id']).to eq(niconico_post.id)
+        expect(json['post_id']).to eq(youtube_post.id)
         expect(Time.zone.parse(json['post_started_at']))
           .to be_within(1.second).of(started_at)
-        expect(json['post_elapsed_ms'])
-          .to be_within(1_000).of(120_000)
       end
     end
   end
@@ -291,36 +273,16 @@ RSpec.describe 'Theatres API', type: :request do
       it 'sets current_post to an eligible post and updates current_post_started_at' do
         expect { do_request }
           .to change { theatre.reload.current_post_id }
+          .from(nil).to(youtube_post.id)
 
         expect(response).to have_http_status(:no_content)
-        expect([niconico_post.id, second_niconico_post.id, youtube_post.id])
-          .to include(theatre.reload.current_post_id)
         expect(theatre.reload.current_post_started_at)
           .to be_within(1.second).of(Time.current)
-        expect(theatre.programmes.count).to eq(1)
-      end
-    end
-
-    context 'when only a YouTube post is eligible' do
-      before do
-        niconico_post.destroy!
-        second_niconico_post.destroy!
-        theatre.update!(host_user: member)
-        sign_in_as(member)
-      end
-
-      it 'sets current_post to the YouTube post' do
-        do_request
-
-        expect(response).to have_http_status(:no_content)
-        expect(theatre.reload.current_post_id).to eq(youtube_post.id)
       end
     end
 
     context 'when current user is host and no eligible post exists' do
       before do
-        niconico_post.destroy!
-        second_niconico_post.destroy!
         youtube_post.destroy!
         theatre.update!(
           host_user: member,
@@ -337,189 +299,9 @@ RSpec.describe 'Theatres API', type: :request do
 
         theatre.reload
         expect(theatre.current_post_id).to be_nil
-        expect(theatre.current_post_started_at).to be_nil
+        expect(theatre.current_post_started_at)
+          .to be_within(1.second).of(Time.current)
       end
     end
   end
-
-  describe 'PUT /theatres/:id/skip_vote' do
-    subject(:do_request) do
-      put "/theatres/#{theatre.id}/skip_vote", params: { post_id: requested_post_id }
-    end
-
-    let(:third_user) { create(:user, :member, name: 'third user') }
-    let(:requested_post_id) { niconico_post.id }
-
-    before do
-      theatre.update!(current_post: niconico_post, current_post_started_at: 10.seconds.ago)
-      [member, other_user, third_user].each do |user|
-        TheatreWatchingUser.create!(
-          theatre:,
-          user:,
-          expires_at: 10.seconds.from_now
-        )
-      end
-    end
-
-    it 'returns 401 when not logged in' do
-      sign_out
-
-      expect { do_request }.not_to change(TheatreSkipVote, :count)
-
-      expect(response).to have_http_status(:unauthorized)
-    end
-
-    it 'returns 422 when post_id is invalid' do
-      sign_in_as(member)
-
-      expect {
-        put "/theatres/#{theatre.id}/skip_vote", params: { post_id: 'invalid' }
-      }.not_to change(TheatreSkipVote, :count)
-
-      expect(response).to have_http_status(:unprocessable_entity)
-    end
-
-    it 'records a vote and returns the current vote status before majority' do
-      sign_in_as(member)
-
-      expect { do_request }.to change(TheatreSkipVote, :count).by(1)
-
-      expect(response).to have_http_status(:ok)
-      expect(json['skipped']).to eq(false)
-      expect(json['post_id']).to eq(niconico_post.id)
-      expect(json['skip_vote']).to include(
-        'votes_count' => 1,
-        'required_count' => 2,
-        'watching_users_count' => 3,
-        'voted' => true
-      )
-    end
-
-    it 'finalizes skip when votes reach majority and stores voters and tag snapshots' do
-      tag = create(:tag, name: 'skip-target')
-      PostTag.create!(post: niconico_post, tag:)
-
-      TheatreSkipVote.create!(theatre:, post: niconico_post, user: member)
-      sign_in_as(other_user)
-
-      expect { do_request }
-        .to change(TheatreSkipEvent, :count).by(1)
-        .and change(TheatreSkipEventVoter, :count).by(2)
-        .and change(TheatreSkipEventTag, :count).by(1)
-
-      expect(response).to have_http_status(:ok)
-      expect(json['skipped']).to eq(true)
-      expect([second_niconico_post.id, youtube_post.id]).to include(json['post_id'])
-
-      event = TheatreSkipEvent.last
-      expect(event.post).to eq(niconico_post)
-      expect(event.users).to contain_exactly(member, other_user)
-      expect(event.tags).to contain_exactly(tag)
-      expect(TheatreSkipVote.where(theatre:, post: niconico_post)).to be_empty
-    end
-
-    it 'does not record a vote when requested post is no longer current' do
-      theatre.update!(current_post: second_niconico_post)
-      sign_in_as(member)
-
-      expect { do_request }.not_to change(TheatreSkipVote, :count)
-
-      expect(response).to have_http_status(:conflict)
-      expect(json['post_id']).to eq(second_niconico_post.id)
-      expect(json['skip_vote']).to include(
-        'votes_count' => 0,
-        'voted' => false
-      )
-    end
-  end
-
-  describe 'DELETE /theatres/:id/skip_vote' do
-    let(:requested_post_id) { niconico_post.id }
-
-    before do
-      theatre.update!(current_post: niconico_post, current_post_started_at: 10.seconds.ago)
-      TheatreWatchingUser.create!(theatre:, user: member, expires_at: 10.seconds.from_now)
-      TheatreSkipVote.create!(theatre:, post: niconico_post, user: member)
-      sign_in_as(member)
-    end
-
-    it 'removes the current user vote' do
-      expect {
-        delete "/theatres/#{theatre.id}/skip_vote", params: { post_id: requested_post_id }
-      }.to change(TheatreSkipVote, :count).by(-1)
-
-      expect(response).to have_http_status(:ok)
-      expect(json['skip_vote']).to include(
-        'votes_count' => 0,
-        'required_count' => 1,
-        'watching_users_count' => 1,
-        'voted' => false
-      )
-    end
-
-    it 'does not remove a vote when requested post is no longer current' do
-      theatre.update!(current_post: second_niconico_post)
-
-      expect {
-        delete "/theatres/#{theatre.id}/skip_vote", params: { post_id: requested_post_id }
-      }.not_to change(TheatreSkipVote, :count)
-
-      expect(response).to have_http_status(:conflict)
-      expect(json['post_id']).to eq(second_niconico_post.id)
-    end
-  end
-
-  describe 'GET /theatres/:id/skip_events' do
-    before do
-      sign_in_as(member)
-    end
-
-    it 'does not expose skip voters' do
-      event = TheatreSkipEvent.create!(
-        theatre:,
-        post: niconico_post,
-        skipped_by_user: member,
-        created_at: Time.current
-      )
-      TheatreSkipEventVoter.create!(theatre_skip_event: event, user: member)
-
-      get "/theatres/#{theatre.id}/skip_events"
-
-      expect(response).to have_http_status(:ok)
-      expect(json.first).to include(
-        'id' => event.id,
-        'theatre_id' => theatre.id
-      )
-      expect(json.first).not_to have_key('voters')
-      expect(json.first).not_to have_key('skipped_by_user')
-    end
-  end
-
-  describe 'GET /theatres/:id/post_selection_weights' do
-    before do
-      theatre.update!(current_post: niconico_post)
-      TheatreWatchingUser.create!(theatre:, user: member, expires_at: 10.seconds.from_now)
-      sign_in_as(member)
-    end
-
-    it 'returns tag penalties and candidate weights for the current watchers' do
-      tag = create(:tag, name: 'heavy-tag')
-      PostTag.create!(post: second_niconico_post, tag:)
-      event = TheatreSkipEvent.create!(
-        theatre:,
-        post: niconico_post,
-        skipped_by_user: member,
-        created_at: Time.current
-      )
-      TheatreSkipEventVoter.create!(theatre_skip_event: event, user: member)
-      TheatreSkipEventTag.create!(theatre_skip_event: event, tag:)
-
-      get "/theatres/#{theatre.id}/post_selection_weights"
-
-      expect(response).to have_http_status(:ok)
-      expect(json['tag_penalties'].first['penalty']).to eq(1)
-      expect(json['lightest_posts'].first['post']['id']).to eq(second_niconico_post.id)
-      expect(json['lightest_posts'].first['penalty']).to eq(1)
-    end
-  end
 end

backend/spec/requests/users_spec.rb

@@ -90,14 +90,12 @@ RSpec.describe 'Users', type: :request do
       expect(response).to have_http_status(:unauthorized)
     end
 
-    it 'returns 422 when name is blank' do
+    it 'returns 400 when name is blank' do
       put "/users/#{user.id}",
           params: { name: '   ' },
           headers: auth_headers(user)
 
-      expect(response).to have_http_status(:unprocessable_entity)
+      expect(response).to have_http_status(:bad_request)
-      expect(json.fetch('errors')).to include(
-          'name' => ['名前は必須です．'])
     end
 
     it 'updates name and returns user slice' do

frontend/AGENTS.md

@@ -4,8 +4,7 @@
 
 These rules apply to work under `frontend/`.
 
-This is a Vite + React + TypeScript app using TanStack Query, Tailwind CSS,
+This is a Vite + React + TypeScript app using TanStack Query, Tailwind CSS, Framer Motion, Radix UI-style components, MDX, and Zustand.
-Framer Motion, Radix UI-style components, MDX, and Zustand.
 
 ## Commands
 
@@ -18,11 +17,9 @@ npm run lint
 npm run preview
 ```
 
-`npm run build` runs `tsc -b && vite build`, and `postbuild` runs
+`npm run build` runs `tsc -b && vite build`, and `postbuild` runs `node scripts/generate-sitemap.js`.
-`node scripts/generate-sitemap.js`.
 
-There is currently no `test` script in `package.json`. Do not run or report
+There is currently no `test` script in `package.json`. Do not run or report `npm test` unless a test script is added.
-`npm test` unless a test script is added.
 
 After frontend changes, run:
 
@@ -35,37 +32,18 @@ If either command cannot be run or fails, report the exact command and failure.
 
 ## TypeScript
 
-- TypeScript is strict. `tsconfig.app.json` enables `strict`,
+- TypeScript is strict. `tsconfig.app.json` enables `strict`, `noUnusedLocals`, `noUnusedParameters`, `erasableSyntaxOnly`, `noFallthroughCasesInSwitch`, and `noUncheckedSideEffectImports`.
-  `noUnusedLocals`, `noUnusedParameters`, `erasableSyntaxOnly`,
-  `noFallthroughCasesInSwitch`, and `noUncheckedSideEffectImports`.
 - Keep types explicit at module boundaries, API helpers, and exported utilities.
 - Use `import type` for type-only imports.
 - Prefer existing shared types from `src/types.ts` before adding local duplicate types.
-- Preserve the repository's existing spacing style in TypeScript, including
+- Preserve the repository's existing spacing style in TypeScript, including GNU-style spacing before call parentheses where it is already used.
-  GNU-style spacing before call parentheses where it is already used.
 - Prefer single quotes for strings unless interpolation or escaping makes double quotes better.
-- Never write a TypeScript or TSX line longer than 99 characters.
-- Aim to keep TypeScript and TSX lines within 79 characters where practical.
-- Use 4-space logical indentation in TypeScript and TSX.
-- For arrays, never put whitespace or a line break immediately before `]`.
-- Keep the first element on the same line as `[` by default.
-- If an array would exceed the line limit, break after `[` and indent
-  elements by 4 spaces.
-- In TypeScript and TSX only, replace every leading run of 8 spaces with a tab
-  to reduce bytes.
-- Treat one leading tab as exactly equivalent to 8 leading spaces.
-- Use tabs only for leading indentation. Never replace spaces that occur after
-  a non-space character on the same line.
 
 ## React
 
 - Use function components.
-- Existing page components commonly export an anonymous function satisfying
+- Existing page components commonly export an anonymous function satisfying `FC`; match nearby file style when editing.
-  `FC`; match nearby file style when editing.
 - React hooks must be called unconditionally and at the top level of components or custom hooks.
-- Gate editing and other privileged controls with shared permission helpers
-  such as `canEditContent`, instead of showing controls and relying only on a
-  later API failure.
 - Keep page-level components under `src/pages`.
 - Keep shared and feature components under `src/components`.
 - Use `react-router-dom` route params and navigation patterns already present in `src/App.tsx`.
@@ -74,23 +52,17 @@ If either command cannot be run or fails, report the exact command and failure.
 ## TanStack Query
 
 - Use `@tanstack/react-query` for server state.
-- Query keys should come from `src/lib/queryKeys.ts`; add key builders there
+- Query keys should come from `src/lib/queryKeys.ts`; add key builders there instead of using ad hoc arrays in components.
-  instead of using ad hoc arrays in components.
+- Fetch functions should live in domain helpers under `src/lib`, such as `posts.ts`, `tags.ts`, or `wiki.ts`.
-- Fetch functions should live in domain helpers under `src/lib`, such as
+- Use `useQueryClient().invalidateQueries` with the shared root keys when mutations affect cached lists or detail views.
-  `posts.ts`, `tags.ts`, or `wiki.ts`.
+- The app-wide `QueryClient` is configured in `src/main.tsx`; do not create additional clients in feature code.
-- Use `useQueryClient().invalidateQueries` with the shared root keys when
-  mutations affect cached lists or detail views.
-- The app-wide `QueryClient` is configured in `src/main.tsx`; do not create
-  additional clients in feature code.
 
 ## API calls
 
 - Use `src/lib/api.ts` for HTTP calls.
-- The API wrapper attaches `X-Transfer-Code` from `localStorage` and converts
+- The API wrapper attaches `X-Transfer-Code` from `localStorage` and converts non-blob responses to camelCase.
-  non-blob responses to camelCase.
 - Send Rails snake_case params and request body keys where the backend expects them.
-- Do not bypass the API wrapper unless there is a specific reason, such as a
+- Do not bypass the API wrapper unless there is a specific reason, such as a third-party request outside the Rails API.
-  third-party request outside the Rails API.
 - For blob responses, pass `responseType: 'blob'` so the wrapper does not camelCase the body.
 
 ## Imports and aliases
@@ -104,41 +76,17 @@ If either command cannot be run or fails, report the exact command and failure.
 
 - Tailwind scans `src/**/*.{html,js,ts,jsx,tsx,mdx}`.
 - Use `cn` from `src/lib/utils.ts` for conditional class names and class merging.
-- Reuse components from `src/components/common`, `src/components/layout`, and
+- Reuse components from `src/components/common`, `src/components/layout`, and `src/components/ui` before adding new primitives.
-  `src/components/ui` before adding new primitives.
 - Keep Tailwind classes consistent with nearby components.
-- Prefer restrained, content-first UI chrome: avoid adding card backgrounds,
+- When adding dynamic tag color classes, update `tailwind.config.js` safelist if the class cannot be statically detected.
-  heavy borders, or nested panel decoration unless the surrounding screen
-  already uses them.
-- Keep operational screens dense and direct; trim explanatory copy and use
-  short Japanese labels that fit the control.
-- Preserve existing Japanese tone and orthography in nearby UI text, including
-  old-kana wording where the file already uses it.
-- When adding dynamic tag color classes, update `tailwind.config.js` safelist
-  if the class cannot be statically detected.
 - Do not introduce new UI libraries or production dependencies without approval.
 
-## TSX formatting
-
-- Preserve compact TSX expression shapes such as inline ternary branches and
-  closing `</div>)` forms when nearby code uses them.
-- For long Tailwind `className` strings, wrap across lines only when needed.
-- Keep continuation indentation aligned with the 4-space logical indentation
-  rule, using tabs only as leading 8-space compression.
-- Do not add braces around `if`, `else`, or `for` bodies when the body is a
-  single physical line.
-- Always add braces around `if`, `else`, or `for` bodies when the body spans
-  two or more physical lines, even if it is one statement.
-- Avoid reformatting unrelated JSX.
-
 ## Lint and build constraints
 
-- ESLint uses `@eslint/js`, `typescript-eslint`, `eslint-plugin-react-hooks`,
+- ESLint uses `@eslint/js`, `typescript-eslint`, `eslint-plugin-react-hooks`, and `eslint-plugin-react-refresh`.
-  and `eslint-plugin-react-refresh`.
 - The hooks rules are enforced; fix hook ordering instead of disabling the rule.
 - `react-refresh/only-export-components` is enabled as a warning with `allowConstantExport`.
-- Build failures from unused locals or unused parameters are TypeScript
+- Build failures from unused locals or unused parameters are TypeScript errors, not lint-only issues.
-  errors, not lint-only issues.
 
 ## Files to avoid in routine work
 

frontend/src/App.tsx

@@ -62,9 +62,8 @@ const RouteTransitionWrapper = ({ user, setUser }: {
 	  <Route path="/tags/:id" element={<TagDetailPage/>}/>
 	  <Route path="/tags/:id/deerjikists" element={<DeerjikistDetailPage/>}/>
 	  <Route path="/tags/nico" element={<NicoTagListPage user={user}/>}/>
-	  <Route path="/nico/tags" element={<NicoTagListPage user={user}/>}/>
 	  <Route path="/tags/changes" element={<TagHistoryPage/>}/>
-	  <Route path="/theatres/:id" element={<TheatreDetailPage user={user}/>}/>
+	  <Route path="/theatres/:id" element={<TheatreDetailPage/>}/>
 	  <Route path="/materials" element={<MaterialBasePage/>}>
 	    <Route index element={<MaterialListPage/>}/>
 	    <Route path="new" element={<MaterialNewPage/>}/>
@@ -159,4 +158,4 @@ const App: FC = () => {
       </>)
 }
 
-export default App
+export default App

frontend/src/components/NicoViewer.test.tsx

@@ -1,83 +0,0 @@
-import { act, fireEvent, render } from '@testing-library/react'
-import { afterEach, describe, expect, it, vi } from 'vitest'
-import { createRef } from 'react'
-
-import NicoViewer from '@/components/NicoViewer'
-
-import type { NiconicoViewerHandle } from '@/types'
-
-
-describe ('NicoViewer', () => {
-  afterEach (() => {
-    vi.useRealTimers ()
-  })
-
-  it ('does not time out after metadata reports a playable duration', () => {
-    vi.useFakeTimers ()
-
-    const onError = vi.fn ()
-    const onMetadataChange = vi.fn ()
-    const { container } = render (
-        <NicoViewer
-            id="sm12345"
-            width={640}
-            height={360}
-            onMetadataChange={onMetadataChange}
-            onError={onError}/>,
-    )
-    const iframe = container.querySelector ('iframe')
-    expect (iframe).not.toBeNull ()
-
-    fireEvent.load (iframe!)
-    act (() => {
-      window.dispatchEvent (new MessageEvent ('message', {
-        origin: 'https://embed.nicovideo.jp',
-        source: iframe!.contentWindow,
-        data:   {
-          eventName: 'playerMetadataChange',
-          data:      {
-            currentTime:           7,
-            duration:              120,
-            isVideoMetaDataLoaded: true,
-            maximumBuffered:       30,
-            muted:                 false,
-            showComment:           true,
-            volume:                1,
-          },
-        },
-      }))
-    })
-
-    act (() => {
-      vi.advanceTimersByTime (8_000)
-    })
-
-    expect (onMetadataChange).toHaveBeenCalled ()
-    expect (onError).not.toHaveBeenCalled ()
-  })
-
-  it ('seeks with milliseconds', () => {
-    const ref = createRef<NiconicoViewerHandle> ()
-    const { container } = render (
-        <NicoViewer
-            ref={ref}
-            id="sm12345"
-            width={640}
-            height={360}/>,
-    )
-    const iframe = container.querySelector ('iframe')!
-    const postMessage = vi.spyOn (iframe.contentWindow!, 'postMessage')
-
-    act (() => {
-      ref.current!.seek (7_000)
-    })
-
-    expect (postMessage).toHaveBeenCalledWith (
-        expect.objectContaining ({
-          eventName: 'seek',
-          data:      { time: 7_000 },
-        }),
-        'https://embed.nicovideo.jp',
-    )
-  })
-})

frontend/src/components/NicoViewer.tsx

@@ -1,11 +1,11 @@
 import { forwardRef,
-	 useCallback,
+         useCallback,
-	 useEffect,
+         useEffect,
-	 useImperativeHandle,
+         useImperativeHandle,
-	 useLayoutEffect,
+         useLayoutEffect,
-	 useMemo,
+         useMemo,
-	 useRef,
+         useRef,
-	 useState } from 'react'
+         useState } from 'react'
 
 import type { CSSProperties, ForwardedRef } from 'react'
 
@@ -14,20 +14,10 @@ import type { NiconicoMetadata, NiconicoVideoInfo, NiconicoViewerHandle } from '
 type NiconicoPlayerMessage =
     | { eventName: 'enterProgrammaticFullScreen' }
     | { eventName: 'exitProgrammaticFullScreen' }
-    | { eventName: 'loadComplete'
+    | { eventName: 'loadComplete'; playerId?: string; data: { videoInfo: NiconicoVideoInfo } }
-	playerId?:  string
+    | { eventName: 'playerMetadataChange'; playerId?: string; data: NiconicoMetadata }
-	data:       { videoInfo: NiconicoVideoInfo } }
+    | { eventName: 'playerStatusChange' | 'statusChange'; playerId?: string; data?: unknown }
-    | { eventName: 'playerMetadataChange'
+    | { eventName: 'error'; playerId?: string; data?: unknown; code?: string; message?: string }
-	playerId?:  string
-	data:       NiconicoMetadata }
-    | { eventName: 'playerStatusChange' | 'statusChange'
-	playerId?:  string
-	data?:      unknown }
-    | { eventName: 'error'
-	playerId?:  string
-	data?:      unknown
-	code?:      string
-	message?:   string }
 
 type NiconicoCommand =
     | { eventName: 'play'; sourceConnectorType: 1; playerId: string }
@@ -40,7 +30,6 @@ type NiconicoCommand =
 	data: { commentVisibility: boolean } }
 
 const EMBED_ORIGIN = 'https://embed.nicovideo.jp'
-const LOAD_COMPLETE_TIMEOUT_MS = 8_000
 
 type Props = {
     id:                 string
@@ -48,18 +37,14 @@ type Props = {
     height:             number
     style?:             CSSProperties
     onLoadComplete?:    (info: NiconicoVideoInfo) => void
-    onMetadataChange?:  (meta: NiconicoMetadata) => void
+    onMetadataChange?:  (meta: NiconicoMetadata) => void }
-    onError?:           (data: unknown) => void }
 
 
 export default forwardRef ((props: Props, ref: ForwardedRef<NiconicoViewerHandle>) => {
-  const { id, width, height, style = { }, onLoadComplete, onMetadataChange, onError } = props
+  const { id, width, height, style = { }, onLoadComplete, onMetadataChange } = props
 
   const iframeRef = useRef<HTMLIFrameElement> (null)
-  const loadCompleteTimerRef = useRef<ReturnType<typeof setTimeout> | null> (null)
+  const playerId = useMemo (() => `nico-${ id }-${ Math.random ().toString (36).slice (2) }`, [id])
-  const playerId = useMemo (
-      () => `nico-${ id }-${ Math.random ().toString (36).slice (2) }`,
-      [id])
 
   const [screenWidth, setScreenWidth] = useState<CSSProperties['width']> ()
   const [screenHeight, setScreenHeight] = useState<CSSProperties['height']> ()
@@ -79,39 +64,21 @@ export default forwardRef ((props: Props, ref: ForwardedRef<NiconicoViewerHandle
   const styleFullScreen: CSSProperties =
       fullScreen
       ? { top: 0,
-	  left: landscape ? 0 : '100%',
+          left: landscape ? 0 : '100%',
-	  position: 'fixed',
+          position: 'fixed',
-	  width: screenWidth,
+          width: screenWidth,
-	  height: screenHeight,
+          height: screenHeight,
-	  zIndex: 2_147_483_647,
+          zIndex: 2_147_483_647,
-	  maxWidth: 'none',
+          maxWidth: 'none',
-	  transformOrigin: '0% 0%',
+          transformOrigin: '0% 0%',
-	  transform: landscape ? 'none' : 'rotate(90deg)',
+          transform: landscape ? 'none' : 'rotate(90deg)',
-	  WebkitTransformOrigin: '0% 0%',
+          WebkitTransformOrigin: '0% 0%',
-	  WebkitTransform: landscape ? 'none' : 'rotate(90deg)' }
+          WebkitTransform: landscape ? 'none' : 'rotate(90deg)' }
       : { }
 
   const margedStyle: CSSProperties =
       { border: 'none', maxWidth: '100%', ...style, ...styleFullScreen }
 
-  const clearLoadCompleteTimer = useCallback (() => {
-    if (!(loadCompleteTimerRef.current))
-      return
-
-    clearTimeout (loadCompleteTimerRef.current)
-    loadCompleteTimerRef.current = null
-  }, [])
-
-  const startLoadCompleteTimer = useCallback (() => {
-    clearLoadCompleteTimer ()
-    loadCompleteTimerRef.current = setTimeout (() => {
-      onError?.({
-	  eventName: 'loadCompleteTimeout',
-	  reason:    'niconico video length was not reported by embed',
-      })
-    }, LOAD_COMPLETE_TIMEOUT_MS)
-  }, [clearLoadCompleteTimer, onError])
-
   const postToPlayer = useCallback ((message: NiconicoCommand) => {
     const win = iframeRef.current?.contentWindow
     if (!(win))
@@ -129,9 +96,7 @@ export default forwardRef ((props: Props, ref: ForwardedRef<NiconicoViewerHandle
   }, [playerId, postToPlayer])
 
   const seek = useCallback ((time: number) => {
-    postToPlayer (
+    postToPlayer ({ eventName: 'seek', sourceConnectorType: 1, playerId, data: { time } })
-	{ eventName: 'seek', sourceConnectorType: 1, playerId,
-	  data: { time } })
   }, [playerId, postToPlayer])
 
   const mute = useCallback (() => {
@@ -167,21 +132,21 @@ export default forwardRef ((props: Props, ref: ForwardedRef<NiconicoViewerHandle
   useEffect (() => {
     const onMessage = (event: MessageEvent<NiconicoPlayerMessage>) => {
       if (!(iframeRef.current)
-	  || (event.source !== iframeRef.current.contentWindow)
+          || (event.source !== iframeRef.current.contentWindow)
-	  || (event.origin !== EMBED_ORIGIN))
+          || (event.origin !== EMBED_ORIGIN))
-	return
+        return
 
       const data = event.data
 
       if (!(data)
 	  || typeof data !== 'object'
 	  || !('eventName' in data))
-	return
+        return
 
       if (('playerId' in data)
 	  && data.playerId
 	  && data.playerId !== playerId)
-	return
+        return
 
       if (data.eventName === 'enterProgrammaticFullScreen')
 	{
@@ -197,34 +162,24 @@ export default forwardRef ((props: Props, ref: ForwardedRef<NiconicoViewerHandle
 
       if (data.eventName === 'loadComplete')
 	{
-	  clearLoadCompleteTimer ()
 	  onLoadComplete?.(data.data.videoInfo)
 	  return
 	}
 
       if (data.eventName === 'playerMetadataChange')
 	{
-	  if (Number.isFinite (data.data.duration) && data.data.duration > 0)
-	    clearLoadCompleteTimer ()
-
 	  onMetadataChange?.(data.data)
 	  return
 	}
 
       if (data.eventName === 'error')
-	{
+        console.error ('niconico player error:', data)
-	  clearLoadCompleteTimer ()
-	  console.error ('niconico player error:', data)
-	  onError?.(data)
-	}
     }
 
     addEventListener ('message', onMessage)
 
     return () => removeEventListener ('message', onMessage)
-  }, [clearLoadCompleteTimer, onError, onLoadComplete, onMetadataChange, playerId])
+  }, [onLoadComplete, onMetadataChange, playerId])
-
-  useEffect (() => clearLoadCompleteTimer, [clearLoadCompleteTimer])
 
   useLayoutEffect (() => {
     if (!(fullScreen))
@@ -237,7 +192,7 @@ export default forwardRef ((props: Props, ref: ForwardedRef<NiconicoViewerHandle
 
     const pollingResize = () => {
       if (ended)
-	return
+        return
 
       const isLandscape = innerWidth >= innerHeight
       const windowWidth = `${ isLandscape ? innerWidth : innerHeight }px`
@@ -251,9 +206,9 @@ export default forwardRef ((props: Props, ref: ForwardedRef<NiconicoViewerHandle
 
     const startPollingResize = () => {
       if (requestAnimationFrame)
-	requestAnimationFrame (pollingResize)
+        requestAnimationFrame (pollingResize)
       else
-	pollingResize ()
+        pollingResize ()
     }
 
     startPollingResize ()
@@ -276,10 +231,9 @@ export default forwardRef ((props: Props, ref: ForwardedRef<NiconicoViewerHandle
       <iframe
 	  ref={iframeRef}
 	  src={src}
-		  width={width}
+	  width={width}
-		  height={height}
+	  height={height}
-		  style={margedStyle}
+	  style={margedStyle}
-		  onLoad={startLoadCompleteTimer}
+	  allowFullScreen
-		  allowFullScreen
+	  allow="autoplay"/>)
-		  allow="autoplay"/>)
 })

frontend/src/components/PostEditForm.tsx

@@ -2,36 +2,34 @@ import { useEffect, useState } from 'react'
 
 import PostFormTagsArea from '@/components/PostFormTagsArea'
 import PostOriginalCreatedTimeField from '@/components/PostOriginalCreatedTimeField'
-import FieldError from '@/components/common/FieldError'
+import Label from '@/components/common/Label'
-import FormField from '@/components/common/FormField'
 import { useDialogue } from '@/components/dialogues/DialogueProvider'
 import { Button } from '@/components/ui/button'
 import { toast } from '@/components/ui/use-toast'
 import { isApiError } from '@/lib/api'
 import { updatePost } from '@/lib/posts'
-import { inputClass } from '@/lib/utils'
+import { msToTime } from '@/lib/utils'
-import { useValidationErrors } from '@/lib/useValidationErrors'
 
 import type { FC, FormEvent } from 'react'
 
-import type { Post, Tag } from '@/types'
+import type { Post, TagWithSections } from '@/types'
-
-type PostFormField =
-    'parentPostIds' | 'tags' | 'originalCreatedAt'
 
 
-const tagsToStr = (tags: Tag[]): string => {
+const tagsToStr = (tags: TagWithSections[]): string => {
-  const result: Tag[] = []
+  const result: Omit<TagWithSections, 'children'>[] = []
 
-  const walk = (tag: Tag) => {
+  const walk = (tag: TagWithSections) => {
     const { children, ...rest } = tag
     result.push (rest)
-    children?.forEach (walk)
+    children.forEach (walk)
   }
 
   tags.filter (t => t.category !== 'nico').forEach (walk)
 
-  return [...(new Set (result.map (t => t.name)))].join (' ')
+  return [...(new Set (result.map (t =>
+      `${ t.name }${ t.sections
+		     .map (s => `[${ msToTime (s.beginMs) }-${ msToTime (s.endMs) }]`)
+		     .join ('') }`)))].join (' ')
 }
 
 
@@ -41,8 +39,6 @@ type Props = { post:    Post
 
 const PostEditForm: FC<Props> = ({ post, onSave }) => {
   const [disabled, setDisabled] = useState (false)
-  const { baseErrors, fieldErrors, clearValidationErrors, applyValidationError } =
-      useValidationErrors<PostFormField> ()
   const [originalCreatedBefore, setOriginalCreatedBefore] =
       useState<string | null> (post.originalCreatedBefore)
   const [originalCreatedFrom, setOriginalCreatedFrom] =
@@ -55,8 +51,6 @@ const PostEditForm: FC<Props> = ({ post, onSave }) => {
   const dialogue = useDialogue ()
 
   const update = async (...args: Parameters<typeof updatePost>) => {
-    clearValidationErrors ()
-
     try
     {
       const data = await updatePost (...args)
@@ -77,14 +71,7 @@ const PostEditForm: FC<Props> = ({ post, onSave }) => {
 
       if (response?.status !== 409)
 	{
-	  if (applyValidationError (e))
+	  toast ({ description: '更新はできなかったよ……' })
-	    {
-	      toast ({ description: '更新はできなかったよ……' })
-	      return
-	    }
-
-	  toast ({ title: '失敗……', description: '入力を確認してください．' })
-
 	  return
 	}
 
@@ -134,43 +121,38 @@ const PostEditForm: FC<Props> = ({ post, onSave }) => {
   }
 
   useEffect (() => {
-    setTags(tagsToStr (post.tags))
+    setTags (tagsToStr (post.tags))
   }, [post])
 
   return (
       <form onSubmit={handleSubmit} className="max-w-xl pt-2 space-y-4">
-	<FieldError messages={baseErrors}/>
-
 	{/* タイトル */}
-	<FormField label="タイトル">
+	<div>
-	  {({ invalid }) => (
+	  <Label>タイトル</Label>
-	      <input
+	  <input
-		  type="text"
+	      type="text"
-		  disabled={disabled}
+	      disabled={disabled}
-		  className={inputClass (invalid)}
+	      className="w-full border rounded p-2"
-		  value={title ?? ''}
+	      value={title ?? ''}
-		  onChange={ev => setTitle (ev.target.value)}/>)}
+	      onChange={ev => setTitle (ev.target.value)}/>
-	</FormField>
+	</div>
 
 	{/* 親投稿 */}
-	<FormField label="親投稿" messages={fieldErrors.parentPostIds}>
+	<div>
-	  {({ describedBy, invalid }) => (
+	  <Label>親投稿</Label>
-	      <input
+	  <input
-		  type="text"
+	      type="text"
-		  disabled={disabled}
+	      disabled={disabled}
-		  value={parentPostIds}
+	      value={parentPostIds}
-		  onChange={e => setParentPostIds (e.target.value)}
+	      onChange={e => setParentPostIds (e.target.value)}
-		  aria-describedby={describedBy}
+	      className="w-full border p-2 rounded"/>
-		  aria-invalid={invalid}
+	</div>
-		  className={inputClass (invalid)}/>)}
-	</FormField>
 
 	{/* タグ */}
 	<PostFormTagsArea
 	    disabled={disabled}
 	    tags={tags}
-	    setTags={setTags}
+	    setTags={setTags}/>
-	    errors={fieldErrors.tags}/>
 
 	{/* オリジナルの作成日時 */}
 	<PostOriginalCreatedTimeField
@@ -178,11 +160,12 @@ const PostEditForm: FC<Props> = ({ post, onSave }) => {
 	    originalCreatedFrom={originalCreatedFrom}
 	    setOriginalCreatedFrom={setOriginalCreatedFrom}
 	    originalCreatedBefore={originalCreatedBefore}
-	    setOriginalCreatedBefore={setOriginalCreatedBefore}
+	    setOriginalCreatedBefore={setOriginalCreatedBefore}/>
-	    errors={fieldErrors.originalCreatedAt}/>
 
 	{/* 送信 */}
-	<Button type="submit" disabled={disabled}>
+	<Button
+	    type="submit"
+	    disabled={disabled}>
 	  更新
 	</Button>
       </form>)

frontend/src/components/PostEmbed.test.tsx

@@ -8,19 +8,12 @@ const dialogue = vi.hoisted (() => ({
   confirm: vi.fn (),
 }))
 
-const nicoViewer = vi.hoisted (() => ({
-  props: vi.fn (),
-}))
-
 vi.mock ('@/components/dialogues/DialogueProvider', () => ({
   useDialogue: () => dialogue,
 }))
 
 vi.mock ('@/components/NicoViewer', () => ({
-  default: (props: { id: string }) => {
+  default: ({ id }: { id: string }) => <div>Nico:{id}</div>,
-    nicoViewer.props (props)
-    return <div>Nico:{props.id}</div>
-  },
 }))
 
 vi.mock ('react-youtube', () => ({
@@ -38,64 +31,6 @@ describe ('PostEmbed', () => {
     expect (screen.getByText ('Nico:sm12345')).toBeInTheDocument ()
   })
 
-  it ('reports niconico metadata as milliseconds', () => {
-    const onVideoReady = vi.fn ()
-    const onPlaybackChange = vi.fn ()
-    render (
-        <PostEmbed
-            post={buildPost ({ url: 'https://www.nicovideo.jp/watch/sm12345' })}
-            onVideoReady={onVideoReady}
-            onPlaybackChange={onPlaybackChange}/>,
-    )
-
-    nicoViewer.props.mock.calls[0][0].onMetadataChange ({
-      currentTime:            7_000,
-      duration:               120_000,
-      isVideoMetaDataLoaded:  true,
-      maximumBuffered:        30,
-      muted:                  false,
-      showComment:            true,
-      volume:                 1,
-    })
-
-    expect (onVideoReady).toHaveBeenCalledWith (120_000)
-    expect (onPlaybackChange).toHaveBeenCalledWith (7_000)
-  })
-
-  it ('reports niconico video readiness only once', () => {
-    const onVideoReady = vi.fn ()
-    render (
-        <PostEmbed
-            post={buildPost ({ url: 'https://www.nicovideo.jp/watch/sm12345' })}
-            onVideoReady={onVideoReady}/>,
-    )
-
-    nicoViewer.props.mock.calls[0][0].onLoadComplete ({
-      title:           '動画',
-      videoId:         'sm12345',
-      lengthInSeconds: 120,
-      thumbnailUrl:    'https://example.com/thumb.jpg',
-      description:     '',
-      viewCount:       1,
-      commentCount:    2,
-      mylistCount:     3,
-      postedAt:        '2026-01-02T03:04:05.000Z',
-      watchId:         12345,
-    })
-    nicoViewer.props.mock.calls[0][0].onMetadataChange ({
-      currentTime:            7_000,
-      duration:               120_000,
-      isVideoMetaDataLoaded:  true,
-      maximumBuffered:        30,
-      muted:                  false,
-      showComment:            true,
-      volume:                 1,
-    })
-
-    expect (onVideoReady).toHaveBeenCalledTimes (1)
-    expect (onVideoReady).toHaveBeenCalledWith (120_000)
-  })
-
   it ('embeds x/twitter status URLs', () => {
     render (<PostEmbed post={buildPost ({ url: 'https://x.com/someone/status/12345' })}/>)
 

frontend/src/components/PostEmbed.tsx

@@ -1,4 +1,4 @@
-import { useCallback, useEffect, useRef, useState } from 'react'
+import { useState } from 'react'
 import YoutubeEmbed from 'react-youtube'
 
 import NicoViewer from '@/components/NicoViewer'
@@ -8,113 +8,17 @@ import { useDialogue } from '@/components/dialogues/DialogueProvider'
 import type { FC, RefObject } from 'react'
 
 import type { NiconicoMetadata, NiconicoVideoInfo, NiconicoViewerHandle, Post } from '@/types'
-import type { YouTubePlayer } from 'react-youtube'
-
-type YouTubeEvent<T = unknown> = {
-    data:    T
-    target:  YouTubePlayer }
 
 type Props = {
     ref?:               RefObject<NiconicoViewerHandle | null>
     post:               Post
     onLoadComplete?:    (info: NiconicoVideoInfo) => void
-    onMetadataChange?:  (meta: NiconicoMetadata) => void
+    onMetadataChange?:  (meta: NiconicoMetadata) => void }
-    onVideoReady?:      (durationMs: number) => void
-    onPlaybackChange?:  (currentTimeMs: number) => number | void
-    onError?:           (data: unknown) => void }
 
 
-const PostEmbed: FC<Props> = ({
+const PostEmbed: FC<Props> = ({ ref, post, onLoadComplete, onMetadataChange }) => {
-    ref,
-    post,
-    onLoadComplete,
-    onMetadataChange,
-    onVideoReady,
-    onPlaybackChange,
-    onError,
-}) => {
   const dialogue = useDialogue ()
   const [framed, setFramed] = useState (false)
-  const [youtubePlayer, setYoutubePlayer] = useState<YouTubePlayer | null> (null)
-  const niconicoVideoReadyRef = useRef (false)
-
-  const notifyNiconicoVideoReady = useCallback ((durationMs: number) => {
-    if (niconicoVideoReadyRef.current
-	|| !(Number.isFinite (durationMs))
-	|| durationMs <= 0)
-      return
-
-    niconicoVideoReadyRef.current = true
-    onVideoReady?.(durationMs)
-  }, [onVideoReady])
-
-  const reportYoutubePlayback = useCallback (async (player: YouTubePlayer) => {
-    const currentTime = await player.getCurrentTime ()
-    const currentTimeMs = currentTime * 1_000
-    const targetTimeMs = onPlaybackChange?.(currentTimeMs)
-
-    if (typeof targetTimeMs !== 'number')
-      return
-
-    if (Math.abs (currentTimeMs - targetTimeMs) > 5_000)
-      await player.seekTo (targetTimeMs / 1_000, true)
-  }, [onPlaybackChange])
-
-  const handleYoutubeReady = async (event: YouTubeEvent) => {
-    setYoutubePlayer (event.target)
-
-    try
-    {
-      await event.target.playVideo ()
-
-      const duration = await event.target.getDuration ()
-      const durationMs = duration * 1_000
-      onVideoReady?.(durationMs)
-
-      if (!(Number.isFinite (durationMs)) || durationMs <= 0)
-	return
-
-      await reportYoutubePlayback (event.target)
-    }
-    catch (error)
-    {
-      onError?.({ platform: 'youtube', error })
-    }
-  }
-
-  const handleYoutubeStateChange = (event: YouTubeEvent<number>) => {
-    void reportYoutubePlayback (event.target)
-  }
-
-  const handleYoutubeError = (event: YouTubeEvent<number>) => {
-    onError?.({ platform: 'youtube', code: event.data })
-  }
-
-  const handleNiconicoLoadComplete = (info: NiconicoVideoInfo) => {
-    notifyNiconicoVideoReady (info.lengthInSeconds * 1_000)
-    onLoadComplete?.(info)
-  }
-
-  const handleNiconicoMetadataChange = (meta: NiconicoMetadata) => {
-    notifyNiconicoVideoReady (meta.duration)
-    onPlaybackChange?.(meta.currentTime)
-    onMetadataChange?.(meta)
-  }
-
-  useEffect (() => {
-    niconicoVideoReadyRef.current = false
-  }, [post.url])
-
-  useEffect (() => {
-    if (!(youtubePlayer) || !(onPlaybackChange))
-      return
-
-    const timer = setInterval (
-	() => void reportYoutubePlayback (youtubePlayer),
-	1_000)
-
-    return () => clearInterval (timer)
-  }, [onPlaybackChange, reportYoutubePlayback, youtubePlayer])
 
   const url = new URL (post.url)
 
@@ -134,9 +38,8 @@ const PostEmbed: FC<Props> = ({
 	      id={videoId}
 	      width={640}
 	      height={360}
-	      onLoadComplete={handleNiconicoLoadComplete}
+	      onLoadComplete={onLoadComplete}
-	      onMetadataChange={handleNiconicoMetadataChange}
+	      onMetadataChange={onMetadataChange}/>)
-	      onError={onError}/>)
     }
 
   case 'twitter.com':
@@ -166,10 +69,7 @@ const PostEmbed: FC<Props> = ({
 	      mute:         0,
 	      loop:         1,
 	      width:        '640',
-	      height:       '360' } }}
+	      height:       '360' } }}/>)
-	      onReady={handleYoutubeReady}
-	      onStateChange={handleYoutubeStateChange}
-	      onError={handleYoutubeError}/>)
     }
   }
 

frontend/src/components/PostFormTagsArea.tsx

@@ -3,7 +3,7 @@
 import { useRef, useState } from 'react'
 
 import TagSearchBox from '@/components/TagSearchBox'
-import FormField from '@/components/common/FormField'
+import Label from '@/components/common/Label'
 import TextArea from '@/components/common/TextArea'
 import { apiGet } from '@/lib/api'
 
@@ -33,11 +33,10 @@ const replaceToken = (value: string, start: number, end: number, text: string) =
 
 type Props = Omit<ComponentPropsWithoutRef<'textarea'>, 'value' | 'onChange' | 'onBlur'> & {
     tags:     string
-    setTags:  (tags: string) => void
+    setTags:  (tags: string) => void }
-    errors?:  string[] }
 
 
-const PostFormTagsArea: FC<Props> = ({ tags, setTags, errors, ...rest }) => {
+const PostFormTagsArea: FC<Props> = ({ tags, setTags, ...rest }) => {
   const ref = useRef<HTMLTextAreaElement> (null)
 
   const [bounds, setBounds] = useState<{ start: number; end: number }> ({ start: 0, end: 0 })
@@ -74,34 +73,30 @@ const PostFormTagsArea: FC<Props> = ({ tags, setTags, errors, ...rest }) => {
   }
 
   return (
-      <FormField className="relative w-full" label="タグ" messages={errors}>
+      <div className="relative w-full">
-	{({ describedBy, invalid }) => (
+	<Label>タグ</Label>
-	    <>
+	<TextArea
-	      <TextArea
+	    {...rest}
-		  {...rest}
+	    ref={ref}
-		  ref={ref}
+	    value={tags}
-		  value={tags}
+	    onChange={ev => setTags (ev.target.value)}
-		  aria-describedby={describedBy}
+	    onSelect={async (ev: SyntheticEvent<HTMLTextAreaElement>) => {
-		  invalid={invalid}
+	      const pos = (ev.target as HTMLTextAreaElement).selectionStart
-		  onChange={ev => setTags (ev.target.value)}
+	      await recompute (pos)
-		  onSelect={async (ev: SyntheticEvent<HTMLTextAreaElement>) => {
+	    }}
-		    const pos = (ev.target as HTMLTextAreaElement).selectionStart
+	    onFocus={() => setFocused (true)}
-		    await recompute (pos)
+	    onBlur={() => {
-		  }}
+	      setFocused (false)
-		  onFocus={() => setFocused (true)}
+	      setSuggestionsVsbl (false)
-		  onBlur={() => {
+	    }}/>
-		    setFocused (false)
+	{focused && (
-		    setSuggestionsVsbl (false)
+	    <TagSearchBox
-		  }}/>
+		suggestions={suggestionsVsbl && suggestions.length > 0
-	      {focused && (
+			     ? suggestions
-		  <TagSearchBox
+			     : [] as Tag[]}
-		      suggestions={suggestionsVsbl && suggestions.length > 0
+		activeIndex={-1}
-				   ? suggestions
+		onSelect={handleTagSelect}/>)}
-				   : [] as Tag[]}
+      </div>)
-		      activeIndex={-1}
-		      onSelect={handleTagSelect}/>)}
-	    </>)}
-      </FormField>)
 }
 
-export default PostFormTagsArea
+export default PostFormTagsArea

frontend/src/components/PostOriginalCreatedTimeField.tsx

@@ -1,5 +1,5 @@
 import DateTimeField from '@/components/common/DateTimeField'
-import FormField from '@/components/common/FormField'
+import Label from '@/components/common/Label'
 import { Button } from '@/components/ui/button'
 
 import type { FC } from 'react'
@@ -9,81 +9,68 @@ type Props = {
     originalCreatedFrom:      string | null
     setOriginalCreatedFrom:   (x: string | null) => void
     originalCreatedBefore:    string | null
-    setOriginalCreatedBefore: (x: string | null) => void
+    setOriginalCreatedBefore: (x: string | null) => void }
-    errors?:                  string[] }
 
 
-const PostOriginalCreatedTimeField: FC<Props> = (
+const PostOriginalCreatedTimeField: FC<Props> = ({ disabled,
-    { disabled,
+		   originalCreatedFrom,
-      originalCreatedFrom,
+		   setOriginalCreatedFrom,
-      setOriginalCreatedFrom,
+		   originalCreatedBefore,
-      originalCreatedBefore,
+		   setOriginalCreatedBefore }) => (
-      setOriginalCreatedBefore,
+    <div>
-      errors }: Props,
+      <Label>オリジナルの作成日時</Label>
-) => (
+      <div className="my-1 flex">
-    <FormField label="オリジナルの作成日時" messages={errors}>
+	<div className="w-80">
-      {({ describedBy, invalid }) => (
+	  <DateTimeField
-	  <>
+	      className="mr-2"
-	    <div className="my-1 flex">
+	      disabled={disabled ?? false}
-	      <div className="w-80">
+	      value={originalCreatedFrom ?? undefined}
-		<DateTimeField
+	      onChange={setOriginalCreatedFrom}
-		    className="mr-2"
+	      onBlur={ev => {
-		    disabled={disabled ?? false}
+		const v = ev.target.value
-		    aria-describedby={describedBy}
+		if (!(v))
-		    aria-invalid={invalid}
+		  return
-		    invalid={invalid}
-		    value={originalCreatedFrom ?? undefined}
-		    onChange={setOriginalCreatedFrom}
-		    onBlur={ev => {
-		      const v = ev.target.value
-		      if (!(v))
-			return
 
-		      const d = new Date (v)
+		const d = new Date (v)
-		      if (d.getMinutes () === 0 && d.getHours () === 0)
+		if (d.getMinutes () === 0 && d.getHours () === 0)
-			d.setDate (d.getDate () + 1)
+		  d.setDate (d.getDate () + 1)
-		      else
+		else
-			d.setMinutes (d.getMinutes () + 1)
+		  d.setMinutes (d.getMinutes () + 1)
-		      setOriginalCreatedBefore (d.toISOString ())
+		setOriginalCreatedBefore (d.toISOString ())
-		    }}/>
+	      }}/>
-		以降
+	  以降
-	      </div>
+	</div>
-	      <div>
+	<div>
-		<Button
+	  <Button
-		    className="bg-gray-600 text-white rounded"
+	      className="bg-gray-600 text-white rounded"
-		    disabled={disabled}
+	      disabled={disabled}
-		    onClick={() => {
+	      onClick={() => {
-		      setOriginalCreatedFrom (null)
+		setOriginalCreatedFrom (null)
-		    }}>
+	      }}>
-		  リセット
+	    リセット
-		</Button>
+	  </Button>
-	      </div>
+	</div>
-	    </div>
+      </div>
+      <div className="my-1 flex">
+	<div className="w-80">
+	  <DateTimeField
+	      className="mr-2"
+	      disabled={disabled}
+	      value={originalCreatedBefore ?? undefined}
+	      onChange={setOriginalCreatedBefore}/>
+	  より前
+	</div>
+	<div>
+	  <Button
+	      className="bg-gray-600 text-white rounded"
+	      disabled={disabled}
+	      onClick={() => {
+		setOriginalCreatedBefore (null)
+	      }}>
+	    リセット
+	  </Button>
+	</div>
+      </div>
+    </div>)
 
-	    <div className="my-1 flex">
+export default PostOriginalCreatedTimeField
-	      <div className="w-80">
-		<DateTimeField
-		    className="mr-2"
-		    disabled={disabled}
-		    aria-describedby={describedBy}
-		    aria-invalid={invalid}
-		    invalid={invalid}
-		    value={originalCreatedBefore ?? undefined}
-		    onChange={setOriginalCreatedBefore}/>
-		より前
-	      </div>
-	      <div>
-		<Button
-		    className="bg-gray-600 text-white rounded"
-		    disabled={disabled}
-		    onClick={() => {
-		      setOriginalCreatedBefore (null)
-		    }}>
-		  リセット
-		</Button>
-	      </div>
-	    </div>
-	  </>)}
-    </FormField>)
-
-export default PostOriginalCreatedTimeField

frontend/src/components/TagDetailSidebar.tsx

@@ -26,13 +26,13 @@ import { dateString, originalCreatedAtString } from '@/lib/utils'
 import type { DragEndEvent } from '@dnd-kit/core'
 import type { FC, MutableRefObject, ReactNode } from 'react'
 
-import type { Category, Post, Tag } from '@/types'
+import type { Category, Post, TagWithSections } from '@/types'
 
-type TagByCategory = { [key in Category]: Tag[] }
+type TagByCategory = { [key in Category]: TagWithSections[] }
 
 
 const renderTagTree = (
-    tag:              Tag,
+    tag:              TagWithSections,
     nestLevel:        number,
     path:             string,
     suppressClickRef: MutableRefObject<boolean>,
@@ -63,7 +63,7 @@ const renderTagTree = (
 
 
 const isDescendant = (
-    root:     Tag,
+    root:     TagWithSections,
     targetId: number,
 ): boolean => {
   if (!(root.children))
@@ -84,8 +84,8 @@ const isDescendant = (
 const findTag = (
     byCat:  TagByCategory,
     id:     number,
-): Tag | undefined => {
+): TagWithSections | undefined => {
-  const walk = (nodes: Tag[]): Tag | undefined => {
+  const walk = (nodes: TagWithSections[]): TagWithSections | undefined => {
     for (const t of nodes)
       {
 	if (t.id === id)
@@ -167,7 +167,7 @@ const TagDetailSidebar: FC<Props> = ({ post, sp }) => {
       }
 
     for (const cat of Object.keys (tagsTmp) as (keyof typeof tagsTmp)[])
-      tagsTmp[cat].sort ((tagA: Tag, tagB: Tag) => tagA.name < tagB.name ? -1 : 1)
+      tagsTmp[cat].sort ((tagA: TagWithSections, tagB: TagWithSections) => tagA.name < tagB.name ? -1 : 1)
 
     return tagsTmp
   }, [post])
@@ -378,4 +378,4 @@ const TagDetailSidebar: FC<Props> = ({ post, sp }) => {
       </SidebarComponent>)
 }
 
-export default TagDetailSidebar
+export default TagDetailSidebar

frontend/src/components/TagSearch.tsx

@@ -4,7 +4,6 @@ import { useEffect, useState } from 'react'
 import { useNavigate, useLocation } from 'react-router-dom'
 
 import { apiGet } from '@/lib/api'
-import { inputClass } from '@/lib/utils'
 
 import TagSearchBox from './TagSearchBox'
 
@@ -111,12 +110,11 @@ const TagSearch: FC = () => {
 	       onFocus={() => setSuggestionsVsbl (true)}
 	       onBlur={() => setSuggestionsVsbl (false)}
 	       onKeyDown={handleKeyDown}
-	       className={inputClass (false,
+	       className="w-full px-3 py-2 border rounded dark:border-gray-600 dark:bg-gray-800 dark:text-white"/>
-				      'px-3 py-2 dark:border-gray-600 dark:bg-gray-800 dark:text-white')}/>
 	<TagSearchBox suggestions={suggestionsVsbl && suggestions.length ? suggestions : [] as Tag[]}
 		      activeIndex={activeIndex}
 		      onSelect={handleTagSelect}/>
       </div>)
 }
 
-export default TagSearch
+export default TagSearch

frontend/src/components/TopNav.tsx

@@ -55,6 +55,12 @@ export const menuOutline = ({ tag, wikiId, user, pathName }: {
 	  { name: '追加', to: '/materials/new' },
 	  { name: '全体履歴', to: '/materials/changes', visible: false },
 	  { name: 'ヘルプ', to: '/wiki/ヘルプ:素材集' }] },
+      { name: '上映会', to: '/theatres/1', base: '/theatres', subMenu: [
+	  { name: <>第&thinsp;1&thinsp;会場</>, to: '/theatres/1' },
+	  { name: 'CyTube', to: '//cytube.mm428.net/r/deernijika' },
+	  { name: <>ニジカ放送局第&thinsp;1&thinsp;チャンネル</>,
+	    to: '//www.youtube.com/watch?v=DCU3hL4Uu6A' },
+	  { name: 'ヘルプ', to: '/wiki/ヘルプ:上映会' }] },
       { name: 'Wiki', to: '/wiki/ヘルプ:ホーム', base: '/wiki', subMenu: [
 	  { name: '検索', to: '/wiki' },
 	  { name: '新規', to: '/wiki/new' },
@@ -65,8 +71,6 @@ export const menuOutline = ({ tag, wikiId, user, pathName }: {
 	    visible: wikiPageFlg },
 	  { name: '履歴', to: `/wiki/changes?id=${ wikiId }`, visible: wikiPageFlg },
 	  { name: '編輯', to: `/wiki/${ wikiId || wikiTitle }/edit`, visible: wikiPageFlg }] },
-      { name: 'おたのしみ', visible: false, subMenu: [
-	  { name: '上映会 (β)', to: '/theatres/1' }] },
       { name: 'ユーザ', to: '/users/settings', visible: false, subMenu: [
 	  { name: '一覧', to: '/users', visible: false },
 	  { name: 'お前', to: `/users/${ user?.id }`, visible: false },
@@ -128,12 +132,8 @@ const TopNav: FC<Props> = ({ user }) => {
 
   const menu = menuOutline ({ tag, wikiId, user, pathName: location.pathname })
   const visibleMenu = menu.filter ((item): item is MenuVisibleItem => item.visible ?? true)
-  const moreMenu = menu.filter (item =>
-      !(item.visible ?? true)
-      || item.subMenu.filter (subItem => subItem.visible ?? true).length > 0)
   const activeIdx =
       visibleMenu.findIndex (item => location.pathname.startsWith (item.base || item.to))
-  const submenuHeight = moreVsbl ? 40 * moreMenu.length : (activeIdx < 0 ? 0 : 40)
 
   const prevActiveIdxRef = useRef<number> (activeIdx)
 
@@ -244,9 +244,9 @@ const TopNav: FC<Props> = ({ user }) => {
 	  <motion.div
 	      key="submenu-shell"
 	      layout
-	      className="relative z-20 hidden md:block overflow-hidden
+	      className="relative hidden md:block overflow-hidden
 			 bg-yellow-200 dark:bg-red-950"
-	      animate={{ height: submenuHeight }}
+	      style={{ height: moreVsbl ? 40 * menu.length : (activeIdx < 0 ? 0 : 40) }}
 	      onMouseLeave={() => {
 		if (moreVsbl)
 		  setMoreVsbl (false)
@@ -257,7 +257,7 @@ const TopNav: FC<Props> = ({ user }) => {
 	      }}>
 	  {moreVsbl
 	   ? (
-	      moreMenu.map ((item, i) => (
+	      menu.map ((item, i) => (
 		  <div key={i} className="relative h-[40px]">
 		    <div className="absolute inset-0 flex items-center px-3">
 		      <motion.div
@@ -267,7 +267,7 @@ const TopNav: FC<Props> = ({ user }) => {
 			       : { initial: { x: 40, y: -40, opacity: 0 },
 				   animate: { x: 0, y: 0, opacity: 1 },
 				   exit: { x: 40, y: -40, opacity: 0 } })}
-			  className="z-10 h-full flex items-center px-3 font-bold w-28">
+			  className="z-10 h-full flex items-center px-3 font-bold w-24">
 			<h2>{item.name}</h2>
 		      </motion.div>
 		      {item.subMenu

frontend/src/components/common/DateTimeField.tsx

@@ -22,11 +22,10 @@ type Props = Omit<ComponentPropsWithoutRef<'input'>, 'onChange'> & {
     value?:     string
     onChange?:  (isoUTC: string | null) => void
     className?: string
-    onBlur?:    (ev: FocusEvent<HTMLInputElement>) => void
+    onBlur?:    (ev: FocusEvent<HTMLInputElement>) => void }
-    invalid?:   boolean }
 
 
-const DateTimeField: FC<Props> = ({ value, onChange, className, onBlur, invalid, ...rest }) => {
+const DateTimeField: FC<Props> = ({ value, onChange, className, onBlur, ...rest }) => {
   const [local, setLocal] = useState ('')
 
   useEffect (() => {
@@ -36,19 +35,9 @@ const DateTimeField: FC<Props> = ({ value, onChange, className, onBlur, invalid,
   return (
       <input
 	  {...rest}
-	  className={cn ('border rounded p-2',
+	  className={cn ('border rounded p-2', className)}
-			 (invalid
-			  ? ['border-red-500 bg-red-50 text-red-900',
-			     'focus:border-red-500 focus:outline-none',
-			     'focus:ring-2 focus:ring-red-200',
-			     'dark:border-red-500 dark:bg-red-950/30 dark:text-red-100']
-			  : ['border-gray-300',
-			     'focus:border-blue-500 focus:outline-none',
-			     'focus:ring-2 focus:ring-blue-200']),
-			 className)}
 	  type="datetime-local"
 	  value={local}
-	  aria-invalid={invalid}
 	  onChange={ev => {
 	    const v = ev.target.value
 	    setLocal (v)
@@ -57,4 +46,4 @@ const DateTimeField: FC<Props> = ({ value, onChange, className, onBlur, invalid,
 	  onBlur={onBlur}/>)
 }
 
-export default DateTimeField
+export default DateTimeField

frontend/src/components/common/FieldError.tsx

@@ -1,18 +0,0 @@
-import type { FC } from 'react'
-
-type Props = { id?:        string
-	       messages?:  string[] }
-
-
-export const FieldError: FC<Props> = ({ id, messages }: Props) => {
-  if (!(messages) || messages.length === 0)
-    return null
-
-  return (
-      <ul id={id} className="mt-1 space-y-1 text-red-700 dark:text-red-300">
-	{messages.map ((message, i) => <li key={i}>{message}</li>)}
-      </ul>)
-}
-
-
-export default FieldError

frontend/src/components/common/FormField.tsx

@@ -1,36 +0,0 @@
-import { useId } from 'react'
-
-import FieldError from '@/components/common/FieldError'
-import Label from '@/components/common/Label'
-import { cn } from '@/lib/utils'
-
-import type { FC, ReactNode } from 'react'
-
-type FieldState = { describedBy?: string
-		    invalid:      boolean }
-
-type Props = {
-    children:   (state: FieldState) => ReactNode
-    checkBox?:   { label:    string
-		   checked:  boolean
-		   onChange: (event: React.ChangeEvent<HTMLInputElement>) => void }
-    className?:  string
-    label:      ReactNode
-    messages?:  string[] }
-
-
-const FormField: FC<Props> = ({ children, checkBox, className, label, messages }: Props) => {
-  const id = useId ()
-  const invalid = messages != null && messages.length > 0
-  const errorId = invalid ? `${ id }-error` : undefined
-
-  return (
-      <div className={cn (className)}>
-	<Label checkBox={checkBox} invalid={invalid}>{label}</Label>
-	{children ({ describedBy: errorId, invalid })}
-	<FieldError id={errorId} messages={messages}/>
-      </div>)
-}
-
-
-export default FormField

frontend/src/components/common/Label.tsx

@@ -1,39 +1,32 @@
 import React from 'react'
 
-import { cn } from '@/lib/utils'
-
 import type { FC } from 'react'
 
 type Props = { children:  React.ReactNode
-	       checkBox?: { label:    string
+               checkBox?: { label:    string
-			    checked:  boolean
+                            checked:  boolean
-			    onChange: (event: React.ChangeEvent<HTMLInputElement>) => void }
+                            onChange: (event: React.ChangeEvent<HTMLInputElement>) => void } }
-	       invalid?:  boolean }
 
 
-const Label: FC<Props> = ({ children, checkBox, invalid }: Props) => {
+const Label: FC<Props> = ({ children, checkBox }) => {
-  const labelClassName = cn ('block font-semibold mb-1',
-			     invalid && 'text-red-700 dark:text-red-300')
-
   if (!(checkBox))
     {
       return (
-	  <label className={labelClassName}>
+          <label className="block font-semibold mb-1">
-	    {children}
+            {children}
-	  </label>)
+          </label>)
     }
 
   return (
       <div className="flex gap-2 mb-1">
-	<label className="flex-1 block font-semibold">{children}</label>
+        <label className="flex-1 block font-semibold">{children}</label>
-	<label className="flex items-center block gap-1">
+        <label className="flex items-center block gap-1">
-	  <input type="checkbox"
+          <input type="checkbox"
-		 checked={checkBox.checked}
+                 checked={checkBox.checked}
-		 onChange={checkBox.onChange}/>
+                 onChange={checkBox.onChange}/>
-	  {checkBox.label}
+          {checkBox.label}
-	</label>
+        </label>
       </div>)
 }
 
-
 export default Label

frontend/src/components/common/TagInput.tsx

@@ -2,7 +2,6 @@ import { useState } from 'react'
 
 import TagSearchBox from '@/components/TagSearchBox'
 import { apiGet } from '@/lib/api'
-import { inputClass } from '@/lib/utils'
 
 import type { FC, ChangeEvent, KeyboardEvent } from 'react'
 
@@ -10,13 +9,10 @@ import type { Tag } from '@/types'
 
 
 type Props = {
-    describedBy?:  string
+    value:      string
-    invalid?:      boolean
+    setValue:   (value: string) => void }
-    value:         string
-    setValue:      (value: string) => void }
 
-
+const TagInput: FC<Props> = ({ value, setValue }) => {
-const TagInput: FC<Props> = ({ describedBy, invalid, value, setValue }) => {
   const [activeIndex, setActiveIndex] = useState (-1)
   const [suggestions, setSuggestions] = useState<Tag[]> ([])
   const [suggestionsVsbl, setSuggestionsVsbl] = useState (false)
@@ -89,14 +85,12 @@ const TagInput: FC<Props> = ({ describedBy, invalid, value, setValue }) => {
       <div className="relative">
 	<input
 	    type="text"
-	    aria-describedby={describedBy}
-	    aria-invalid={invalid}
 	    value={value}
 	    onChange={whenChanged}
 	    onFocus={() => setSuggestionsVsbl (true)}
 	    onBlur={() => setSuggestionsVsbl (false)}
 	    onKeyDown={handleKeyDown}
-	    className={inputClass (invalid)}/>
+	    className="w-full border p-2 rounded"/>
 	<TagSearchBox
 	    suggestions={
 		suggestionsVsbl && suggestions.length > 0 ? suggestions : [] as Tag[]}
@@ -105,4 +99,4 @@ const TagInput: FC<Props> = ({ describedBy, invalid, value, setValue }) => {
       </div>)
 }
 
-export default TagInput
+export default TagInput

frontend/src/components/common/TextArea.tsx

@@ -1,25 +1,9 @@
 import { forwardRef } from 'react'
 
-import { cn } from '@/lib/utils'
-
 import type { TextareaHTMLAttributes } from 'react'
 
-type Props = TextareaHTMLAttributes<HTMLTextAreaElement> & { invalid?: boolean }
+type Props = TextareaHTMLAttributes<HTMLTextAreaElement>
 
 
-export default forwardRef<HTMLTextAreaElement, Props> (
+export default forwardRef<HTMLTextAreaElement, Props> (({ ...props }, ref) => (
-    ({ className, invalid = false, ...props }, ref) => (
+    <textarea ref={ref} className="rounded border w-full p-2 h-32" {...props}/>))
-	<textarea
-	    ref={ref}
-	    aria-invalid={invalid}
-	    className={cn ('rounded border w-full p-2 h-32',
-			   (invalid
-			    ? ['border-red-500 bg-red-50 text-red-900',
-			       'focus:border-red-500 focus:outline-none focus:ring-2',
-			       'focus:ring-red-200',
-			       'dark:border-red-500 dark:bg-red-950/30 dark:text-red-100']
-			    : ['border-gray-300',
-			       'focus:border-blue-500 focus:outline-none focus:ring-2',
-			       'focus:ring-blue-200']),
-			   className)}
-	    {...props}/>))

frontend/src/lib/api.ts

@@ -64,14 +64,11 @@ export const apiPatch = async <T> (
 ): Promise<T> => apiP ('patch', path, body, opt)
 
 
-export const apiDelete = async <T = void> (
+export const apiDelete = async (
     path: string,
     opt?: Opt,
-): Promise<T> => {
+): Promise<void> => {
-  const res = await client.delete (path, withUserCode (opt))
+  await client.delete (path, withUserCode (opt))
-  if (res.data == null || res.data === '')
-    return undefined as T
-  return toCamel (res.data as Record<string, unknown>, { deep: true }) as T
 }
 
 

frontend/src/lib/apiErrors.test.ts

@@ -1,79 +0,0 @@
-import { describe, expect, it, vi } from 'vitest'
-
-const api = vi.hoisted (() => ({
-  isApiError: vi.fn (),
-}))
-
-vi.mock ('@/lib/api', () => api)
-
-describe ('extractValidationError', () => {
-  it ('extracts field and base errors from 422 validation responses', async () => {
-    api.isApiError.mockReturnValueOnce (true)
-
-    const { extractValidationError } = await import ('@/lib/apiErrors')
-    const validationError = extractValidationError<'name'> ({
-      response: {
-        status: 422,
-        data:   {
-          type:        'validation_error',
-          message:     '入力内容を確認してください．',
-          errors:      { name: ['名前は必須です．'] },
-          base_errors: ['全体エラー'],
-        },
-      },
-    })
-
-    expect (validationError).toEqual ({
-      message:     '入力内容を確認してください．',
-      fieldErrors: { name: ['名前は必須です．'] },
-      baseErrors:  ['全体エラー'],
-    })
-  })
-
-  it ('preserves dotted field keys for indexed form rows', async () => {
-    api.isApiError.mockReturnValueOnce (true)
-
-    const { extractValidationError } = await import ('@/lib/apiErrors')
-    const validationError = extractValidationError<'deerjikists.0.platform'> ({
-      response: {
-        status: 422,
-        data:   {
-          type:        'validation_error',
-          errors:      { 'deerjikists.0.platform': ['プラットフォームを入力してください．'] },
-          base_errors: [],
-        },
-      },
-    })
-
-    expect (validationError?.fieldErrors).toEqual ({
-      'deerjikists0Platform': ['プラットフォームを入力してください．'],
-    })
-  })
-
-  it ('does not treat 400 bad requests as form validation errors', async () => {
-    api.isApiError.mockReturnValueOnce (true)
-
-    const { extractValidationError } = await import ('@/lib/apiErrors')
-    const validationError = extractValidationError ({
-      response: {
-        status: 400,
-        data:   {
-          type:        'bad_request',
-          message:     'リクエストが不正です．',
-          errors:      {},
-          base_errors: ['リクエストが不正です．'],
-        },
-      },
-    })
-
-    expect (validationError).toBeNull ()
-  })
-
-  it ('ignores non-api errors', async () => {
-    api.isApiError.mockReturnValueOnce (false)
-
-    const { extractValidationError } = await import ('@/lib/apiErrors')
-
-    expect (extractValidationError (new Error ('network'))).toBeNull ()
-  })
-})

frontend/src/lib/apiErrors.ts

@@ -1,36 +0,0 @@
-import toCamel from 'camelcase-keys'
-
-import { isApiError } from '@/lib/api'
-
-export type FieldErrors<T extends string = string> = Partial<Record<T, string[]>>
-
-export type ValidationError<T extends string = string> =
-    { message:      string
-      fieldErrors:  FieldErrors<T>
-      baseErrors:   string[] }
-
-type RawValidationError = { type?:        string
-			    message?:     string
-			    errors?:      Record<string, string[]>
-			    baseErrors?:  string[] }
-
-
-export const extractValidationError = <T extends string = string> (err: unknown) => {
-  if (!(isApiError (err)) || err.response?.status !== 422)
-    return null
-
-  const rawData = toCamel ((err.response.data ?? { }) as Record<string, unknown>,
-			   { deep: true }) as RawValidationError
-  const data: RawValidationError = {
-      type:       rawData.type as string | undefined,
-      message:    rawData.message as string | undefined,
-      errors:     rawData.errors as Record<string, string[]> | undefined,
-      baseErrors: rawData.baseErrors as string[] | undefined }
-
-  if (data.type !== 'validation_error' && !(data.errors))
-    return null
-
-  return { message: data.message ?? '入力内容を確認してください．',
-	   fieldErrors: (data.errors ?? { }) as FieldErrors<T>,
-	   baseErrors: data.baseErrors ?? [] }
-}

frontend/src/lib/prefetchers.test.ts

@@ -10,7 +10,6 @@ const postsApi = vi.hoisted (() => ({
 }))
 
 const tagsApi = vi.hoisted (() => ({
-  fetchNicoTags:   vi.fn (),
   fetchTag:        vi.fn (),
   fetchTagByName:  vi.fn (),
   fetchTagChanges: vi.fn (),
@@ -38,7 +37,6 @@ describe ('prefetchForURL', () => {
     postsApi.fetchPost.mockResolvedValue ({ id: 1 })
     postsApi.fetchPostChanges.mockResolvedValue ({ versions: [], count: 0 })
     tagsApi.fetchTags.mockResolvedValue ({ tags: [], count: 0 })
-    tagsApi.fetchNicoTags.mockResolvedValue ({ tags: [], count: 0 })
     tagsApi.fetchTag.mockResolvedValue ({ id: 1 })
     tagsApi.fetchTagByName.mockResolvedValue (null)
     tagsApi.fetchTagChanges.mockResolvedValue ({ versions: [], count: 0 })
@@ -87,32 +85,6 @@ describe ('prefetchForURL', () => {
     )
   })
 
-  it ('prefetches nico tag indexes and their alias from query parameters', async () => {
-    await prefetchForURL (
-        qc (),
-        'http://localhost/tags/nico?name=source&linked_tag=destination'
-	    + '&link_status=linked&page=3&limit=10',
-    )
-    await prefetchForURL (qc (), 'http://localhost/nico/tags?page=2')
-
-    expect (tagsApi.fetchNicoTags).toHaveBeenNthCalledWith (1, {
-      name:       'source',
-      linkedTag:  'destination',
-      linkStatus: 'linked',
-      page:       3,
-      limit:      10,
-      order:      'updated_at:desc',
-    })
-    expect (tagsApi.fetchNicoTags).toHaveBeenNthCalledWith (2, {
-      name:       '',
-      linkedTag:  '',
-      linkStatus: 'all',
-      page:       2,
-      limit:      20,
-      order:      'updated_at:desc',
-    })
-  })
-
   it ('prefetches wiki show pages and related tag/post data', async () => {
     wikiApi.fetchWikiPageByTitle.mockResolvedValueOnce ({
       id:    3,

frontend/src/lib/prefetchers.ts

@@ -3,7 +3,7 @@ import { match } from 'path-to-regexp'
 
 import { fetchPost, fetchPosts, fetchPostChanges } from '@/lib/posts'
 import { postsKeys, tagsKeys, wikiKeys } from '@/lib/queryKeys'
-import { fetchNicoTags, fetchTagByName, fetchTag, fetchTagChanges, fetchTags } from '@/lib/tags'
+import { fetchTagByName, fetchTag, fetchTagChanges, fetchTags } from '@/lib/tags'
 import { fetchWikiPage,
 	 fetchWikiPageByTitle,
 	 fetchWikiPages } from '@/lib/wiki'
@@ -170,24 +170,6 @@ const prefetchTagsIndex: Prefetcher = async (qc, url) => {
 }
 
 
-const prefetchNicoTagsIndex: Prefetcher = async (qc, url) => {
-  const keys = {
-      name:       url.searchParams.get ('name') ?? '',
-      linkedTag:  url.searchParams.get ('linked_tag') ?? '',
-      linkStatus: (url.searchParams.get ('link_status') || 'all') as
-		  'all' | 'linked' | 'unlinked',
-      page:       Number (url.searchParams.get ('page') || 1),
-      limit:      Number (url.searchParams.get ('limit') || 20),
-      order:      (url.searchParams.get ('order') || 'updated_at:desc') as
-		  'name:asc' | 'name:desc' | 'created_at:asc' | 'created_at:desc'
-		  | 'updated_at:asc' | 'updated_at:desc' }
-
-  await qc.prefetchQuery ({
-      queryKey: tagsKeys.nicoIndex (keys),
-      queryFn:  () => fetchNicoTags (keys) })
-}
-
-
 const prefetchTagShow: Prefetcher = async (qc, url) => {
   const m = mTag (url.pathname)
   if (!(m))
@@ -224,8 +206,6 @@ export const routePrefetchers: { test: (u: URL) => boolean; run: Prefetcher }[]
 		  && Boolean (mWiki (u.pathname))),
       run: prefetchWikiPageShow },
     { test: u => u.pathname === '/tags', run: prefetchTagsIndex },
-    { test: u => ['/tags/nico', '/nico/tags'].includes (u.pathname),
-      run: prefetchNicoTagsIndex },
     { test: u => (!(['/tags/nico', '/tags/changes'].includes (u.pathname))
 		  && Boolean (mTag (u.pathname))),
       run: prefetchTagShow },

frontend/src/lib/queryKeys.ts

@@ -1,4 +1,4 @@
-import type { FetchNicoTagsParams, FetchPostsParams, FetchTagsParams } from '@/types'
+import type { FetchPostsParams, FetchTagsParams } from '@/types'
 
 export const postsKeys = {
     root:     ['posts'] as const,
@@ -11,8 +11,6 @@ export const postsKeys = {
 export const tagsKeys = {
     root:         ['tags'] as const,
     index:        (p: FetchTagsParams) => ['tags', 'index', p] as const,
-    nicoRoot:     ['tags', 'nico'] as const,
-    nicoIndex:    (p: FetchNicoTagsParams) => ['tags', 'nico', 'index', p] as const,
     show:         (name: string) => ['tags', name] as const,
     changes:      (p: { id?: string; page: number; limit: number }) =>
 		      ['tags', 'changes', p] as const,

frontend/src/lib/tags.ts

@@ -1,11 +1,6 @@
 import { apiGet } from '@/lib/api'
 
-import type { Deerjikist,
+import type { Deerjikist, FetchTagsParams, Tag, TagVersion } from '@/types'
-	      FetchNicoTagsParams,
-	      FetchTagsParams,
-	      NicoTag,
-	      Tag,
-	      TagVersion } from '@/types'
 
 
 export const fetchTags = async (
@@ -28,19 +23,6 @@ export const fetchTags = async (
 	...(order && { order }) } })
 
 
-export const fetchNicoTags = async (
-    { name, linkedTag, linkStatus, page, limit, order }: FetchNicoTagsParams,
-): Promise<{ tags:  NicoTag[]
-	     count: number }> =>
-    await apiGet ('/tags/nico', { params: {
-	page,
-	limit,
-	name,
-	linked_tag:  linkedTag,
-	link_status: linkStatus === 'all' ? '' : linkStatus,
-	order } })
-
-
 export const fetchTag = async (id: string): Promise<Tag | null> => {
   try
   {

frontend/src/lib/useValidationErrors.ts

@@ -1,28 +0,0 @@
-import { useState } from 'react'
-
-import { extractValidationError } from '@/lib/apiErrors'
-
-import type { FieldErrors } from '@/lib/apiErrors'
-
-
-export const useValidationErrors = <T extends string> () => {
-  const [baseErrors, setBaseErrors] = useState<string[]> ([])
-  const [fieldErrors, setFieldErrors] = useState<FieldErrors<T>> ({ })
-
-  const clearValidationErrors = () => {
-    setBaseErrors ([])
-    setFieldErrors ({ })
-  }
-
-  const applyValidationError = (error: unknown): boolean => {
-    const validationError = extractValidationError<T> (error)
-    if (!(validationError))
-      return false
-
-    setBaseErrors (validationError.baseErrors)
-    setFieldErrors (validationError.fieldErrors)
-    return true
-  }
-
-  return { baseErrors, fieldErrors, clearValidationErrors, applyValidationError }
-}

frontend/src/lib/users.test.ts

@@ -1,20 +0,0 @@
-import { describe, expect, it } from 'vitest'
-
-import { canEditContent } from '@/lib/users'
-
-import type { UserRole } from '@/types'
-
-const userWithRole = (role: UserRole) => ({ role })
-
-describe ('user permission helpers', () => {
-  it ('allows admins and members to edit content', () => {
-    expect (canEditContent (userWithRole ('admin'))).toBe (true)
-    expect (canEditContent (userWithRole ('member'))).toBe (true)
-  })
-
-  it ('does not allow guests or missing users to edit content', () => {
-    expect (canEditContent (userWithRole ('guest'))).toBe (false)
-    expect (canEditContent (null)).toBe (false)
-    expect (canEditContent (undefined)).toBe (false)
-  })
-})

frontend/src/lib/users.ts

@@ -1,8 +0,0 @@
-import type { User, UserRole } from '@/types'
-
-const CONTENT_EDITOR_ROLES: readonly UserRole[] = ['admin', 'member']
-
-
-export const canEditContent = (
-    user: Pick<User, 'role'> | null | undefined,
-): boolean => user != null && CONTENT_EDITOR_ROLES.includes (user.role)

frontend/src/lib/utils.ts

@@ -73,13 +73,13 @@ export const originalCreatedAtString = (
 }
 
 
-export const inputClass = (invalid?: boolean, className?: string): string =>
+export const msToTime = (ms: number): string => {
-    cn ('w-full rounded border p-2',
+  const totalS = Math.trunc (ms / 1_000)
-	(invalid
+  const s = String (totalS % 60)
-	 ? ['border-red-500 bg-red-50 text-red-900',
+  const min = String (Math.trunc (totalS / 60) % 60)
-	    'placeholder:text-red-300',
+  const h = Math.trunc (totalS / 3_600)
-	    'focus:border-red-500 focus:outline-none focus:ring-2 focus:ring-red-200',
+
-	    'dark:border-red-500 dark:bg-red-950/30 dark:text-red-100']
+  return (h > 0
-	 : ['border-gray-300',
+	  ? `${ h }:${ min.padStart (2, '0') }:${ s.padStart (2, '0') }`
-	    'focus:border-blue-500 focus:outline-none focus:ring-2 focus:ring-blue-200']),
+	  : `${ min }:${ s.padStart (2, '0') }`)
-	className)
+}

frontend/src/pages/deerjikists/DeerjikistDetailPage.test.tsx

@@ -1,72 +0,0 @@
-import { fireEvent, screen, waitFor } from '@testing-library/react'
-import { Route, Routes } from 'react-router-dom'
-import { beforeEach, describe, expect, it, vi } from 'vitest'
-
-import DeerjikistDetailPage from '@/pages/deerjikists/DeerjikistDetailPage'
-import { buildTag } from '@/test/factories'
-import { renderWithProviders } from '@/test/render'
-
-const tagsApi = vi.hoisted (() => ({
-  fetchDeerjikistsByTag: vi.fn (),
-}))
-
-const api = vi.hoisted (() => ({
-  apiPut:     vi.fn (),
-  isApiError: vi.fn (),
-}))
-
-const toastApi = vi.hoisted (() => ({
-  toast: vi.fn (),
-}))
-
-vi.mock ('@/lib/tags', () => tagsApi)
-vi.mock ('@/lib/api', () => api)
-vi.mock ('@/components/ui/use-toast', () => toastApi)
-
-const renderPage = () =>
-  renderWithProviders (
-      <Routes>
-        <Route path="/tags/:id/deerjikists" element={<DeerjikistDetailPage/>}/>
-      </Routes>,
-      { route: '/tags/7/deerjikists' },
-  )
-
-describe ('DeerjikistDetailPage', () => {
-  beforeEach (() => {
-    vi.clearAllMocks ()
-    api.isApiError.mockReturnValue (false)
-  })
-
-  it ('shows indexed validation errors returned for deerjikist rows', async () => {
-    tagsApi.fetchDeerjikistsByTag.mockResolvedValueOnce ({
-      tag:         buildTag ({ id: 7, name: 'deerjika', category: 'deerjikist' }),
-      deerjikists: [{ platform: null, code: 'abc' }],
-    })
-    api.isApiError.mockReturnValue (true)
-    api.apiPut.mockRejectedValueOnce ({
-      response: {
-        status: 422,
-        data:   {
-          type:        'validation_error',
-          message:     '入力内容を確認してください．',
-          errors:      { 'deerjikists.0.platform': ['プラットフォームを入力してください．'] },
-          base_errors: [],
-        },
-      },
-    })
-
-    renderPage ()
-
-    await screen.findByDisplayValue ('abc')
-    fireEvent.submit (screen.getByRole ('button', { name: '更新' }).closest ('form')!)
-
-    await waitFor (() => {
-      expect (api.apiPut).toHaveBeenCalledWith (
-          '/tags/7/deerjikists',
-          [{ platform: null, code: 'abc' }],
-      )
-    })
-    expect (await screen.findByText ('プラットフォームを入力してください．')).toBeInTheDocument ()
-    expect (screen.getByRole ('combobox')).toHaveAttribute ('aria-invalid', 'true')
-  })
-})

frontend/src/pages/deerjikists/DeerjikistDetailPage.tsx

@@ -1,10 +1,10 @@
 import { useQuery, useQueryClient } from '@tanstack/react-query'
 import { useEffect, useMemo, useState } from 'react'
+import { Helmet } from 'react-helmet-async'
 import { useParams } from 'react-router-dom'
 
 import TagLink from '@/components/TagLink'
-import FieldError from '@/components/common/FieldError'
+import Label from '@/components/common/Label'
-import FormField from '@/components/common/FormField'
 import PageTitle from '@/components/common/PageTitle'
 import MainArea from '@/components/layout/MainArea'
 import { toast } from '@/components/ui/use-toast'
@@ -12,16 +12,12 @@ import { PLATFORM_NAMES, PLATFORMS } from '@/consts'
 import { apiPut } from '@/lib/api'
 import { tagsKeys } from '@/lib/queryKeys'
 import { fetchDeerjikistsByTag } from '@/lib/tags'
-import { cn, inputClass } from '@/lib/utils'
+import { cn } from '@/lib/utils'
-import { useValidationErrors } from '@/lib/useValidationErrors'
 
 import type { FC, FormEvent } from 'react'
 
 import type { Deerjikist, Platform } from '@/types'
 
-type DeerjikistFormField =
-    'deerjikists' | `deerjikists${ number }Platform` | `deerjikists${ number }Code`
-
 
 const DeerjikistDetailPage: FC = () => {
   const { id } = useParams ()
@@ -36,14 +32,11 @@ const DeerjikistDetailPage: FC = () => {
   const [data, setData] =
       useState<(Omit<Deerjikist, 'platform'> & { platform: Platform | null })[]> ([])
   const [disabled, setDisabled] = useState (true)
-  const { baseErrors, fieldErrors, clearValidationErrors, applyValidationError } =
-      useValidationErrors<DeerjikistFormField> ()
 
   const qc = useQueryClient ()
 
   const handleSubmit = async (e: FormEvent) => {
     e.preventDefault ()
-    clearValidationErrors ()
 
     try
     {
@@ -54,9 +47,8 @@ const DeerjikistDetailPage: FC = () => {
 
       toast ({ description: '更新しました．' })
     }
-    catch (e)
+    catch
     {
-      applyValidationError (e)
       toast ({ title: '更新失敗', description: '入力内容を確認してください．' })
     }
     finally
@@ -78,6 +70,11 @@ const DeerjikistDetailPage: FC = () => {
 
   return (
       <MainArea>
+	<Helmet>
+	  <meta name="robots" content="noindex"/>
+	  {tag && <title>{tag.name} | ニジラー情報</title>}
+	</Helmet>
+
 	{(loading || !(tag)) ? 'Loading...' : (
 	    <div className="max-w-xl">
 	      <PageTitle>
@@ -85,9 +82,6 @@ const DeerjikistDetailPage: FC = () => {
 	      </PageTitle>
 
 	      <form onSubmit={handleSubmit} className="my-4 space-y-2">
-		<FieldError messages={baseErrors}/>
-		<FieldError messages={fieldErrors.deerjikists}/>
-
 		{data.map ((datum, i) => (
 		    <fieldset key={i} className="min-w-0 rounded-lg border border-gray-300
 						 dark:border-gray-700 p-4">
@@ -103,48 +97,40 @@ const DeerjikistDetailPage: FC = () => {
 		      </legend>
 
 		      {/* プラットフォーム */}
-		      <FormField
+		      <div>
-			  label="プラットフォーム"
+			<Label>プラットフォーム</Label>
-			  messages={fieldErrors[`deerjikists${ i }Platform`]}>
+			<select
-			{({ describedBy, invalid }) => (
+			    className="w-full border p-2 rounded"
-			    <select
+			    disabled={disabled}
-				disabled={disabled}
+			    value={datum.platform ?? ''}
-				value={datum.platform ?? ''}
+			    onChange={e => setData (prev => {
-				aria-describedby={describedBy}
+			      const rtn = [...prev]
-				aria-invalid={invalid}
+			      rtn[i] = { ...rtn[i],
-				className={inputClass (invalid)}
+					 platform: (e.target.value || null) as Platform | null }
-				onChange={e => setData (prev => {
+			      return rtn
-				  const rtn = [...prev]
+			    })}>
-				  rtn[i] = { ...rtn[i],
+			  <option value="">&nbsp;</option>
-					     platform: (e.target.value || null) as Platform | null }
+			  {PLATFORMS.map (p => (
-				  return rtn
+			      <option key={p} value={p}>
-				})}>
+				{PLATFORM_NAMES[p]}
-			      <option value="">&nbsp;</option>
+			      </option>))}
-			      {PLATFORMS.map (p => (
+			</select>
-				  <option key={p} value={p}>
+		      </div>
-				    {PLATFORM_NAMES[p]}
-				  </option>))}
-			    </select>)}
-		      </FormField>
 
 		      {/* コード */}
-		      <FormField
+		      <div>
-			  label="コード"
+			<Label>コード</Label>
-			  messages={fieldErrors[`deerjikists${ i }Code`]}>
+			<input
-			{({ describedBy, invalid }) => (
+			    type="text"
-			    <input
+			    disabled={disabled}
-				type="text"
+			    className="w-full border p-2 rounded"
-				disabled={disabled}
+			    value={datum.code}
-				value={datum.code}
+			    onChange={e => setData (prev => {
-				aria-describedby={describedBy}
+			      const rtn = [...prev]
-				aria-invalid={invalid}
+			      rtn[i] = { ...rtn[i], code: e.target.value }
-				className={inputClass (invalid)}
+			      return rtn
-				onChange={e => setData (prev => {
+			    })}/>
-				  const rtn = [...prev]
+		      </div>
-				  rtn[i] = { ...rtn[i], code: e.target.value }
-				  return rtn
-				})}/>)}
-		      </FormField>
 		    </fieldset>
 		))}
 

frontend/src/pages/materials/MaterialDetailPage.tsx

@@ -4,8 +4,7 @@ import { useParams } from 'react-router-dom'
 
 import TagLink from '@/components/TagLink'
 import WikiBody from '@/components/WikiBody'
-import FieldError from '@/components/common/FieldError'
+import Label from '@/components/common/Label'
-import FormField from '@/components/common/FormField'
 import PageTitle from '@/components/common/PageTitle'
 import TabGroup, { Tab } from '@/components/common/TabGroup'
 import TagInput from '@/components/common/TagInput'
@@ -14,8 +13,6 @@ import { Button } from '@/components/ui/button'
 import { toast } from '@/components/ui/use-toast'
 import { SITE_TITLE } from '@/config'
 import { apiGet, apiPut } from '@/lib/api'
-import { inputClass } from '@/lib/utils'
-import { useValidationErrors } from '@/lib/useValidationErrors'
 
 import type { FC } from 'react'
 
@@ -23,8 +20,6 @@ import type { Material, Tag } from '@/types'
 
 type MaterialWithTag = Material & { tag: Tag }
 
-type MaterialFormField = 'tag' | 'file' | 'url'
-
 
 const MaterialDetailPage: FC = () => {
   const { id } = useParams ()
@@ -36,12 +31,8 @@ const MaterialDetailPage: FC = () => {
   const [sending, setSending] = useState (false)
   const [tag, setTag] = useState ('')
   const [url, setURL] = useState ('')
-  const { baseErrors, fieldErrors, clearValidationErrors, applyValidationError } =
-      useValidationErrors<MaterialFormField> ()
 
   const handleSubmit = async () => {
-    clearValidationErrors ()
-
     const formData = new FormData
     if (tag.trim ())
       formData.append ('tag', tag)
@@ -57,9 +48,8 @@ const MaterialDetailPage: FC = () => {
       setMaterial (data)
       toast ({ title: '更新成功!' })
     }
-    catch (e)
+    catch
     {
-      applyValidationError (e)
       toast ({ title: '更新失敗……', description: '入力を見直してください．' })
     }
     finally
@@ -128,66 +118,54 @@ const MaterialDetailPage: FC = () => {
 
 		<Tab name="編輯">
 		  <div className="max-w-wl pt-2 space-y-4">
-		    <FieldError messages={baseErrors}/>
-
 		    {/* タグ */}
-		    <FormField label="タグ" messages={fieldErrors.tag}>
+		    <div>
-		      {({ describedBy, invalid }) => (
+		      <Label>タグ</Label>
-			  <TagInput
+		      <TagInput value={tag} setValue={setTag}/>
-			      describedBy={describedBy}
+		    </div>
-			      invalid={invalid}
-			      value={tag}
-			      setValue={setTag}/>)}
-		    </FormField>
 
 		    {/* ファイル */}
-		    <FormField label="ファイル" messages={fieldErrors.file}>
+		    <div>
-		      {({ describedBy, invalid }) => (
+		      <Label>ファイル</Label>
-			  <>
+		      <input
-			    <input
+			  type="file"
-				type="file"
+			  accept="image/*,video/*,audio/*"
-				accept="image/*,video/*,audio/*"
+			  onChange={e => {
-				aria-describedby={describedBy}
+			    const f = e.target.files?.[0]
-				aria-invalid={invalid}
+			    setFile (f ?? null)
-				onChange={e => {
+			    setFilePreview (f ? URL.createObjectURL (f) : '')
-				  const f = e.target.files?.[0]
+			  }}/>
-				  setFile (f ?? null)
+		      {(file && filePreview) && (
-				  setFilePreview (f ? URL.createObjectURL (f) : '')
+			  (/image\/.*/.test (file.type) && (
-				}}/>
+			      <img
-			    {(file && filePreview) && (
+				  src={filePreview}
-				(/image\/.*/.test (file.type) && (
+				  alt="preview"
-				    <img
+				  className="mt-2 max-h-48 rounded border"/>))
-					src={filePreview}
+			  || (/video\/.*/.test (file.type) && (
-					alt="preview"
+			      <video
-					className="mt-2 max-h-48 rounded border"/>))
+				  src={filePreview}
-				|| (/video\/.*/.test (file.type) && (
+				  controls
-				    <video
+				  className="mt-2 max-h-48 rounded border"/>))
-					src={filePreview}
+			  || (/audio\/.*/.test (file.type) && (
-					controls
+			      <audio
-					className="mt-2 max-h-48 rounded border"/>))
+				  src={filePreview}
-				|| (/audio\/.*/.test (file.type) && (
+				  controls
-				    <audio
+				  className="mt-2 max-h-48"/>))
-					src={filePreview}
+			  || (
-					controls
+			      <p className="text-red-600 dark:text-red-400">
-					className="mt-2 max-h-48"/>))
+				その形式のファイルには対応していません．
-				|| (
+			      </p>))}
-				    <p className="text-red-600 dark:text-red-400">
+		    </div>
-				      その形式のファイルには対応していません．
-				    </p>))}
-			  </>)}
-		    </FormField>
 
 		    {/* 参考 URL */}
-		    <FormField label="参考 URL" messages={fieldErrors.url}>
+		    <div>
-		      {({ describedBy, invalid }) => (
+		      <Label>参考 URL</Label>
-			  <input
+		      <input
-			      type="url"
+			  type="url"
-			      value={url}
+			  value={url}
-			      onChange={e => setURL (e.target.value)}
+			  onChange={e => setURL (e.target.value)}
-			      aria-describedby={describedBy}
+			  className="w-full border p-2 rounded"/>
-			      aria-invalid={invalid}
+		    </div>
-			      className={inputClass (invalid)}/>)}
-		    </FormField>
 
 		    {/* 送信 */}
 		    <Button
@@ -203,4 +181,4 @@ const MaterialDetailPage: FC = () => {
       </MainArea>)
 }
 
-export default MaterialDetailPage
+export default MaterialDetailPage

frontend/src/pages/materials/MaterialNewPage.test.tsx

@@ -1,13 +1,11 @@
 import { fireEvent, screen, waitFor } from '@testing-library/react'
-import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import MaterialNewPage from '@/pages/materials/MaterialNewPage'
 import { renderWithProviders } from '@/test/render'
 
 const api = vi.hoisted (() => ({
-  apiGet:     vi.fn (),
+  apiPost: vi.fn (),
-  apiPost:    vi.fn (),
-  isApiError: vi.fn (),
 }))
 
 const toastApi = vi.hoisted (() => ({
@@ -18,12 +16,6 @@ vi.mock ('@/lib/api', () => api)
 vi.mock ('@/components/ui/use-toast', () => toastApi)
 
 describe ('MaterialNewPage', () => {
-  beforeEach (() => {
-    vi.clearAllMocks ()
-    api.apiGet.mockResolvedValue ([])
-    api.isApiError.mockReturnValue (false)
-  })
-
   it ('initializes tag from query and submits form data', async () => {
     api.apiPost.mockResolvedValueOnce ({})
 
@@ -43,30 +35,4 @@ describe ('MaterialNewPage', () => {
     expect (formData.get ('url')).toBe ('https://example.com/ref')
     expect (toastApi.toast).toHaveBeenCalledWith ({ title: '送信成功!' })
   })
-
-  it ('shows validation errors for file and url fields', async () => {
-    api.isApiError.mockReturnValue (true)
-    api.apiPost.mockRejectedValueOnce ({
-      response: {
-        status: 422,
-        data:   {
-          type:        'validation_error',
-          message:     '入力内容を確認してください．',
-          errors:      {
-            file: ['ファイルまたは URL は必須です．'],
-            url:  ['ファイルまたは URL は必須です．'],
-          },
-          base_errors: [],
-        },
-      },
-    })
-
-    renderWithProviders (<MaterialNewPage/>)
-
-    fireEvent.change (screen.getAllByRole ('textbox')[0], { target: { value: '虹夏' } })
-    fireEvent.click (screen.getByRole ('button', { name: '追加' }))
-
-    expect (await screen.findAllByText ('ファイルまたは URL は必須です．')).toHaveLength (2)
-    expect (screen.getAllByRole ('textbox')[1]).toHaveAttribute ('aria-invalid', 'true')
-  })
 })

frontend/src/pages/materials/MaterialNewPage.tsx

@@ -2,9 +2,8 @@ import { useState } from 'react'
 import { Helmet } from 'react-helmet-async'
 import { useLocation, useNavigate } from 'react-router-dom'
 
-import FieldError from '@/components/common/FieldError'
 import Form from '@/components/common/Form'
-import FormField from '@/components/common/FormField'
+import Label from '@/components/common/Label'
 import PageTitle from '@/components/common/PageTitle'
 import TagInput from '@/components/common/TagInput'
 import MainArea from '@/components/layout/MainArea'
@@ -12,13 +11,9 @@ import { Button } from '@/components/ui/button'
 import { toast } from '@/components/ui/use-toast'
 import { SITE_TITLE } from '@/config'
 import { apiPost } from '@/lib/api'
-import { inputClass } from '@/lib/utils'
-import { useValidationErrors } from '@/lib/useValidationErrors'
 
 import type { FC } from 'react'
 
-type MaterialFormField = 'tag' | 'file' | 'url'
-
 
 const MaterialNewPage: FC = () => {
   const location = useLocation ()
@@ -32,12 +27,8 @@ const MaterialNewPage: FC = () => {
   const [sending, setSending] = useState (false)
   const [tag, setTag] = useState (tagQuery)
   const [url, setURL] = useState ('')
-  const { baseErrors, fieldErrors, clearValidationErrors, applyValidationError } =
-      useValidationErrors<MaterialFormField> ()
 
   const handleSubmit = async () => {
-    clearValidationErrors ()
-
     const formData = new FormData
     if (tag)
       formData.append ('tag', tag)
@@ -53,9 +44,8 @@ const MaterialNewPage: FC = () => {
       toast ({ title: '送信成功!' })
       navigate (`/materials?tag=${ encodeURIComponent (tag) }`)
     }
-    catch (e)
+    catch
     {
-      applyValidationError (e)
       toast ({ title: '送信失敗……', description: '入力を見直してください．' })
     }
     finally
@@ -72,66 +62,55 @@ const MaterialNewPage: FC = () => {
 
 	<Form>
 	  <PageTitle>素材追加</PageTitle>
-	  <FieldError messages={baseErrors}/>
 
 	  {/* タグ */}
-	  <FormField label="タグ" messages={fieldErrors.tag}>
+	  <div>
-	    {({ describedBy, invalid }) => (
+	    <Label>タグ</Label>
-		<TagInput
+	    <TagInput value={tag} setValue={setTag}/>
-		    describedBy={describedBy}
+	  </div>
-		    invalid={invalid}
-		    value={tag}
-		    setValue={setTag}/>)}
-	  </FormField>
 
 	  {/* ファイル */}
-	  <FormField label="ファイル" messages={fieldErrors.file}>
+	  <div>
-	    {({ describedBy, invalid }) => (
+	    <Label>ファイル</Label>
-		<>
+	    <input
-		  <input
+		type="file"
-		      type="file"
+		accept="image/*,video/*,audio/*"
-		      accept="image/*,video/*,audio/*"
+		onChange={e => {
-		      aria-describedby={describedBy}
+		  const f = e.target.files?.[0]
-		      aria-invalid={invalid}
+		  setFile (f ?? null)
-		      onChange={e => {
+		  setFilePreview (f ? URL.createObjectURL (f) : '')
-			const f = e.target.files?.[0]
+		}}/>
-			setFile (f ?? null)
+	    {(file && filePreview) && (
-			setFilePreview (f ? URL.createObjectURL (f) : '')
+		(/image\/.*/.test (file.type) && (
-		      }}/>
+		    <img
-		  {(file && filePreview) && (
+			src={filePreview}
-		      (/image\/.*/.test (file.type) && (
+			alt="preview"
-			  <img
+			className="mt-2 max-h-48 rounded border"/>))
-			      src={filePreview}
+		|| (/video\/.*/.test (file.type) && (
-			      alt="preview"
+		    <video
-			      className="mt-2 max-h-48 rounded border"/>))
+			src={filePreview}
-		      || (/video\/.*/.test (file.type) && (
+			controls
-			  <video
+			className="mt-2 max-h-48 rounded border"/>))
-			      src={filePreview}
+		|| (/audio\/.*/.test (file.type) && (
-			      controls
+		    <audio
-			      className="mt-2 max-h-48 rounded border"/>))
+			src={filePreview}
-		      || (/audio\/.*/.test (file.type) && (
+			controls
-			  <audio
+			className="mt-2 max-h-48"/>))
-			      src={filePreview}
+		|| (
-			      controls
+		    <p className="text-red-600 dark:text-red-400">
-			      className="mt-2 max-h-48"/>))
+		      その形式のファイルには対応していません．
-		      || (
+		    </p>))}
-			  <p className="text-red-600 dark:text-red-400">
+	  </div>
-			    その形式のファイルには対応していません．
-			  </p>))}
-		</>)}
-	  </FormField>
 
 	  {/* 参考 URL */}
-	  <FormField label="参考 URL" messages={fieldErrors.url}>
+	  <div>
-	    {({ describedBy, invalid }) => (
+	    <Label>参考 URL</Label>
-		<input
+	    <input
-		    type="url"
+		type="url"
-		    value={url}
+		value={url}
-		    onChange={e => setURL (e.target.value)}
+		onChange={e => setURL (e.target.value)}
-		    aria-describedby={describedBy}
+		className="w-full border p-2 rounded"/>
-		    aria-invalid={invalid}
+	  </div>
-		    className={inputClass (invalid)}/>)}
-	  </FormField>
 
 	  {/* 送信 */}
 	  <Button
@@ -144,4 +123,4 @@ const MaterialNewPage: FC = () => {
       </MainArea>)
 }
 
-export default MaterialNewPage
+export default MaterialNewPage

frontend/src/pages/materials/MaterialSearchPage.tsx

@@ -1,7 +1,7 @@
 import { useState } from 'react'
 import { Helmet } from 'react-helmet-async'
 
-import FormField from '@/components/common/FormField'
+import Label from '@/components/common/Label'
 import PageTitle from '@/components/common/PageTitle'
 import TagInput from '@/components/common/TagInput'
 import MainArea from '@/components/layout/MainArea'
@@ -29,23 +29,23 @@ const MaterialSearchPage: FC = () => {
 
 	  <form onSubmit={handleSearch} className="space-y-2">
 	    {/* タグ */}
-	    <FormField label="タグ">
+	    <div>
-	      {() => (
+	      <Label>タグ</Label>
-		  <TagInput
+	      <TagInput
-		      value={tagName}
+		  value={tagName}
-		      setValue={setTagName}/>)}
+		  setValue={setTagName}/>
-	    </FormField>
+	    </div>
 
 	    {/* 親タグ */}
-	    <FormField label="親タグ">
+	    <div>
-	      {() => (
+	      <Label>親タグ</Label>
-		  <TagInput
+	      <TagInput
-		      value={parentTagName}
+		  value={parentTagName}
-		      setValue={setParentTagName}/>)}
+		  setValue={setParentTagName}/>
-	    </FormField>
+	    </div>
 	  </form>
 	</div>
       </MainArea>)
 }
 
-export default MaterialSearchPage
+export default MaterialSearchPage

frontend/src/pages/posts/PostDetailPage.tsx

@@ -3,6 +3,7 @@ import { motion } from 'framer-motion'
 import { useEffect, useRef, useState } from 'react'
 import { Helmet } from 'react-helmet-async'
 import { useParams } from 'react-router-dom'
+
 import PostEditForm from '@/components/PostEditForm'
 import PostEmbed from '@/components/PostEmbed'
 import PostList from '@/components/PostList'
@@ -15,7 +16,6 @@ import { SITE_TITLE } from '@/config'
 import { isApiError } from '@/lib/api'
 import { fetchPost, toggleViewedFlg } from '@/lib/posts'
 import { postsKeys, tagsKeys } from '@/lib/queryKeys'
-import { canEditContent } from '@/lib/users'
 import { cn } from '@/lib/utils'
 import NotFound from '@/pages/NotFound'
 import ServiceUnavailable from '@/pages/ServiceUnavailable'
@@ -28,7 +28,6 @@ type Props = { user: User | null }
 
 
 const PostDetailPage: FC<Props> = ({ user }) => {
-  const editable = canEditContent (user)
   const { id } = useParams ()
   const postId = String (id ?? '')
   const postKey = postsKeys.show (postId)
@@ -165,7 +164,7 @@ const PostDetailPage: FC<Props> = ({ user }) => {
 		     ? <PostList posts={post.related}/>
 		     : 'まだないよ（笑）'}
 		  </Tab>
-		  {editable && (
+		  {['admin', 'member'].some (r => user?.role === r) && (
 		      <Tab name="編輯">
 			<PostEditForm
 			    post={post}

frontend/src/pages/posts/PostNewPage.test.tsx

@@ -1,14 +1,13 @@
 import { fireEvent, screen, waitFor } from '@testing-library/react'
-import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import PostNewPage from '@/pages/posts/PostNewPage'
 import { buildUser } from '@/test/factories'
 import { renderWithProviders } from '@/test/render'
 
 const api = vi.hoisted (() => ({
-  apiGet:     vi.fn (),
+  apiGet:  vi.fn (),
-  apiPost:    vi.fn (),
+  apiPost: vi.fn (),
-  isApiError: vi.fn (),
 }))
 
 const toastApi = vi.hoisted (() => ({
@@ -19,11 +18,6 @@ vi.mock ('@/lib/api', () => api)
 vi.mock ('@/components/ui/use-toast', () => toastApi)
 
 describe ('PostNewPage', () => {
-  beforeEach (() => {
-    vi.clearAllMocks ()
-    api.isApiError.mockReturnValue (false)
-  })
-
   it ('blocks guests', () => {
     renderWithProviders (<PostNewPage user={buildUser ({ role: 'guest' })}/>)
 
@@ -61,36 +55,4 @@ describe ('PostNewPage', () => {
     expect (formData.get ('tags')).toBe ('tag1 tag2')
     expect (toastApi.toast).toHaveBeenCalledWith ({ title: '投稿成功!' })
   })
-
-  it ('shows 422 validation errors for post fields', async () => {
-    api.apiGet.mockResolvedValue ([])
-    api.isApiError.mockReturnValue (true)
-    api.apiPost.mockRejectedValueOnce ({
-      response: {
-        status: 422,
-        data:   {
-          type:        'validation_error',
-          message:     '入力内容を確認してください．',
-          errors:      { tags: ['ニコニコ・タグは直接指定できません．'] },
-          base_errors: ['投稿内容を確認してください．'],
-        },
-      },
-    })
-
-    renderWithProviders (<PostNewPage user={buildUser ({ role: 'member' })}/>)
-
-    const checkboxes = screen.getAllByRole ('checkbox', { name: '自動' })
-    fireEvent.click (checkboxes[0])
-    fireEvent.click (checkboxes[1])
-
-    const textboxes = screen.getAllByRole ('textbox')
-    fireEvent.change (textboxes[0], { target: { value: 'https://example.com/post' } })
-    fireEvent.change (textboxes[1], { target: { value: '投稿タイトル' } })
-    fireEvent.change (textboxes[3], { target: { value: 'nico:nico_tag' } })
-    fireEvent.click (screen.getByRole ('button', { name: '追加' }))
-
-    expect (await screen.findByText ('投稿内容を確認してください．')).toBeInTheDocument ()
-    expect (screen.getByText ('ニコニコ・タグは直接指定できません．')).toBeInTheDocument ()
-    expect (screen.getAllByRole ('textbox')[3]).toHaveAttribute ('aria-invalid', 'true')
-  })
 })

frontend/src/pages/posts/PostNewPage.tsx

@@ -4,18 +4,14 @@ import { useNavigate } from 'react-router-dom'
 
 import PostFormTagsArea from '@/components/PostFormTagsArea'
 import PostOriginalCreatedTimeField from '@/components/PostOriginalCreatedTimeField'
-import FieldError from '@/components/common/FieldError'
 import Form from '@/components/common/Form'
-import FormField from '@/components/common/FormField'
+import Label from '@/components/common/Label'
 import PageTitle from '@/components/common/PageTitle'
 import MainArea from '@/components/layout/MainArea'
 import { Button } from '@/components/ui/button'
 import { toast } from '@/components/ui/use-toast'
 import { SITE_TITLE } from '@/config'
 import { apiGet, apiPost } from '@/lib/api'
-import { canEditContent } from '@/lib/users'
-import { inputClass } from '@/lib/utils'
-import { useValidationErrors } from '@/lib/useValidationErrors'
 import Forbidden from '@/pages/Forbidden'
 
 import type { FC } from 'react'
@@ -24,18 +20,12 @@ import type { User } from '@/types'
 
 type Props = { user: User | null }
 
-type PostFormField =
-    'url' | 'title' | 'tags' | 'parentPostIds' | 'originalCreatedAt' | 'thumbnail'
-
 
 const PostNewPage: FC<Props> = ({ user }) => {
-  const editable = canEditContent (user)
+  const editable = ['admin', 'member'].some (r => user?.role === r)
 
   const navigate = useNavigate ()
 
-  const { baseErrors, fieldErrors, clearValidationErrors, applyValidationError } =
-      useValidationErrors<PostFormField> ()
-
   const [originalCreatedBefore, setOriginalCreatedBefore] = useState<string | null> (null)
   const [originalCreatedFrom, setOriginalCreatedFrom] = useState<string | null> (null)
   const [parentPostIds, setParentPostIds] = useState ('')
@@ -53,8 +43,6 @@ const PostNewPage: FC<Props> = ({ user }) => {
   const thumbnailPreviewRef = useRef ('')
 
   const handleSubmit = async () => {
-    clearValidationErrors ()
-
     const formData = new FormData
     formData.append ('title', title)
     formData.append ('url', url)
@@ -73,9 +61,8 @@ const PostNewPage: FC<Props> = ({ user }) => {
       toast ({ title: '投稿成功!' })
       navigate ('/posts')
     }
-    catch (e)
+    catch
     {
-      applyValidationError (e)
       toast ({ title: '投稿失敗', description: '入力を確認してください。' })
     }
   }
@@ -139,99 +126,85 @@ const PostNewPage: FC<Props> = ({ user }) => {
 	</Helmet>
 	<Form>
 	  <PageTitle>広場に投稿を追加する</PageTitle>
-	  <FieldError messages={baseErrors}/>
 
 	  {/* URL */}
-	  <FormField label="URL" messages={fieldErrors.url}>
+	  <div>
-	    {({ describedBy, invalid }) => (
+	    <Label>URL</Label>
-		<input type="url"
+	    <input type="url"
-		       placeholder="例：https://www.nicovideo.jp/watch/..."
+		   placeholder="例：https://www.nicovideo.jp/watch/..."
-		       value={url}
+		   value={url}
-		       onChange={e => setURL (e.target.value)}
+		   onChange={e => setURL (e.target.value)}
-		       aria-describedby={describedBy}
+		   className="w-full border p-2 rounded"
-		       aria-invalid={invalid}
+		   onBlur={handleURLBlur}/>
-		       className={inputClass (invalid)}
+	  </div>
-		       onBlur={handleURLBlur}/>)}
-	  </FormField>
 
 	  {/* タイトル */}
-	  <FormField
+	  <div>
-	      checkBox={{
+	    <Label checkBox={{
-		  label:    '自動',
+		label:    '自動',
-		  checked:  titleAutoFlg,
+		checked:  titleAutoFlg,
-		  onChange: ev => setTitleAutoFlg (ev.target.checked)}}
+		onChange: ev => setTitleAutoFlg (ev.target.checked)}}>
-	      label="タイトル"
+	      タイトル
-	      messages={fieldErrors.title}>
+	    </Label>
-	    {({ describedBy, invalid }) => (
+	    <input type="text"
-		<input type="text"
+		   className="w-full border rounded p-2"
-		       aria-describedby={describedBy}
+		   value={title}
-		       aria-invalid={invalid}
+		   placeholder={titleLoading ? 'Loading...' : ''}
-		       className={inputClass (invalid)}
+		   onChange={ev => setTitle (ev.target.value)}
-		       value={title}
+		   disabled={titleAutoFlg}/>
-		       placeholder={titleLoading ? 'Loading...' : ''}
+	  </div>
-		       onChange={ev => setTitle (ev.target.value)}
-		       disabled={titleAutoFlg}/>)}
-	  </FormField>
 
 	  {/* サムネール */}
-	  <FormField
+	  <div>
-	      checkBox={{
+	    <Label checkBox={{
-		  label:    '自動',
+		label:    '自動',
-		  checked:  thumbnailAutoFlg,
+		checked:  thumbnailAutoFlg,
-		  onChange: ev => setThumbnailAutoFlg (ev.target.checked)}}
+		onChange: ev => setThumbnailAutoFlg (ev.target.checked)}}>
-	      label="サムネール"
+	      サムネール
-	      messages={fieldErrors.thumbnail}>
+	    </Label>
-	    {({ describedBy, invalid }) => (
+	    {thumbnailAutoFlg
-		<>
+	     ? (thumbnailLoading
-		  {thumbnailAutoFlg
+		? <p className="text-gray-500 text-sm">Loading...</p>
-		   ? (thumbnailLoading
+		: !(thumbnailPreview) && (
-		      ? <p className="text-gray-500 text-sm">Loading...</p>
+		    <p className="text-gray-500 text-sm">
-		      : !(thumbnailPreview) && (
+		      URL から自動取得されます。
-			  <p className="text-gray-500 text-sm">
+		    </p>))
-			    URL から自動取得されます。
+	     : (
-			  </p>))
+		<input type="file"
-		   : (
+		       accept="image/*"
-		      <input type="file"
+		       onChange={e => {
-			     accept="image/*"
+			 const file = e.target.files?.[0]
-			     aria-describedby={describedBy}
+			 if (file)
-			     aria-invalid={invalid}
+			   {
-			     onChange={e => {
+			     setThumbnailFile (file)
-			       const file = e.target.files?.[0]
+			     setThumbnailPreview (URL.createObjectURL (file))
-			       if (file)
+			   }
-				 {
+		       }}/>)}
-				   setThumbnailFile (file)
+	    {thumbnailPreview && (
-				   setThumbnailPreview (URL.createObjectURL (file))
+		<img src={thumbnailPreview}
-				 }
+		     alt="preview"
-			     }}/>)}
+		     className="mt-2 max-h-48 rounded border"/>)}
-		  {thumbnailPreview && (
+	  </div>
-		      <img src={thumbnailPreview}
-			   alt="preview"
-			   className="mt-2 max-h-48 rounded border"/>)}
-		</>)}
-	  </FormField>
 
 	  {/* 親投稿 */}
-	  <FormField label="親投稿" messages={fieldErrors.parentPostIds}>
+	  <div>
-	    {({ describedBy, invalid }) => (
+	    <Label>親投稿</Label>
-		<input
+	    <input
-		    type="text"
+		type="text"
-		    value={parentPostIds}
+		value={parentPostIds}
-		    onChange={e => setParentPostIds (e.target.value)}
+		onChange={e => setParentPostIds (e.target.value)}
-		    aria-describedby={describedBy}
+		className="w-full border p-2 rounded"/>
-		    aria-invalid={invalid}
+	  </div>
-		    className={inputClass (invalid)}/>)}
-	  </FormField>
 
 	  {/* タグ */}
-	  <PostFormTagsArea tags={tags} setTags={setTags} errors={fieldErrors.tags}/>
+	  <PostFormTagsArea tags={tags} setTags={setTags}/>
 
 	  {/* オリジナルの作成日時 */}
 	  <PostOriginalCreatedTimeField
 	      originalCreatedFrom={originalCreatedFrom}
 	      setOriginalCreatedFrom={setOriginalCreatedFrom}
 	      originalCreatedBefore={originalCreatedBefore}
-	      setOriginalCreatedBefore={setOriginalCreatedBefore}
+	      setOriginalCreatedBefore={setOriginalCreatedBefore}/>
-	      errors={fieldErrors.originalCreatedAt}/>
 
 	  {/* 送信 */}
 	  <Button onClick={handleSubmit}

frontend/src/pages/posts/PostSearchPage.tsx

@@ -8,7 +8,7 @@ import PrefetchLink from '@/components/PrefetchLink'
 import SortHeader from '@/components/SortHeader'
 import TagLink from '@/components/TagLink'
 import DateTimeField from '@/components/common/DateTimeField'
-import FormField from '@/components/common/FormField'
+import Label from '@/components/common/Label'
 import PageTitle from '@/components/common/PageTitle'
 import Pagination from '@/components/common/Pagination'
 import TagInput from '@/components/common/TagInput'
@@ -16,7 +16,7 @@ import MainArea from '@/components/layout/MainArea'
 import { SITE_TITLE } from '@/config'
 import { fetchPosts } from '@/lib/posts'
 import { postsKeys } from '@/lib/queryKeys'
-import { dateString, inputClass, originalCreatedAtString } from '@/lib/utils'
+import { dateString, originalCreatedAtString } from '@/lib/utils'
 
 import type { FC, FormEvent } from 'react'
 
@@ -138,33 +138,31 @@ const PostSearchPage: FC = () => {
 
 	  <form onSubmit={handleSearch} className="space-y-2">
 	    {/* タイトル */}
-	    <FormField label="タイトル">
+	    <div>
-	      {({ invalid }) => (
+	      <Label>タイトル</Label>
-		  <input
+	      <input
-		      type="text"
+		  type="text"
-		      value={title}
+		  value={title}
-		      onChange={e => setTitle (e.target.value)}
+		  onChange={e => setTitle (e.target.value)}
-		      className={inputClass (invalid)}/>)}
+		  className="w-full border p-2 rounded"/>
-	    </FormField>
+	    </div>
 
 	    {/* URL */}
-	    <FormField label="URL">
+	    <div>
-	      {({ invalid }) => (
+	      <Label>URL</Label>
-		  <input
+	      <input
-		      type="text"
+		  type="text"
-		      value={url}
+		  value={url}
-		      onChange={e => setURL (e.target.value)}
+		  onChange={e => setURL (e.target.value)}
-		      className={inputClass (invalid)}/>)}
+		  className="w-full border p-2 rounded"/>
-	    </FormField>
+	    </div>
 
 	    {/* タグ */}
-	    <FormField label="タグ">
-	      {() => (
-		  <TagInput
-		      value={tagsStr}
-		      setValue={setTagsStr}/>)}
-	    </FormField>
 	    <div>
+	      <Label>タグ</Label>
+	      <TagInput
+		  value={tagsStr}
+		  setValue={setTagsStr}/>
 	      <fieldset className="w-full my-2">
 		<label>検索区分：</label>
 		<label className="mx-2">
@@ -187,46 +185,40 @@ const PostSearchPage: FC = () => {
 	    </div>
 
 	    {/* オリジナルの投稿日時 */}
-	    <FormField label="オリジナルの投稿日時">
+	    <div>
-	      {() => (
+	      <Label>オリジナルの投稿日時</Label>
-		  <>
+	      <DateTimeField
-		    <DateTimeField
+		  value={originalCreatedFrom ?? undefined}
-			value={originalCreatedFrom ?? undefined}
+		  onChange={setOriginalCreatedFrom}/>
-			onChange={setOriginalCreatedFrom}/>
+	      <span className="mx-1">〜</span>
-		    <span className="mx-1">〜</span>
+	      <DateTimeField
-		    <DateTimeField
+		  value={originalCreatedTo ?? undefined}
-			value={originalCreatedTo ?? undefined}
+		  onChange={setOriginalCreatedTo}/>
-			onChange={setOriginalCreatedTo}/>
+	    </div>
-		  </>)}
-	    </FormField>
 
 	    {/* 投稿日時 */}
-	    <FormField label="投稿日時">
+	    <div>
-	      {() => (
+	      <Label>投稿日時</Label>
-		  <>
+	      <DateTimeField
-		    <DateTimeField
+		  value={createdFrom ?? undefined}
-			value={createdFrom ?? undefined}
+		  onChange={setCreatedFrom}/>
-			onChange={setCreatedFrom}/>
+	      <span className="mx-1">〜</span>
-		    <span className="mx-1">〜</span>
+	      <DateTimeField
-		    <DateTimeField
+		  value={createdTo ?? undefined}
-			value={createdTo ?? undefined}
+		  onChange={setCreatedTo}/>
-			onChange={setCreatedTo}/>
+	    </div>
-		  </>)}
-	    </FormField>
 
 	    {/* 更新日時 */}
-	    <FormField label="更新日時">
+	    <div>
-	      {() => (
+	      <Label>更新日時</Label>
-		  <>
+	      <DateTimeField
-		    <DateTimeField
+		  value={updatedFrom ?? undefined}
-			value={updatedFrom ?? undefined}
+		  onChange={setUpdatedFrom}/>
-			onChange={setUpdatedFrom}/>
+	      <span className="mx-1">〜</span>
-		    <span className="mx-1">〜</span>
+	      <DateTimeField
-		    <DateTimeField
+		  value={updatedTo ?? undefined}
-			value={updatedTo ?? undefined}
+		  onChange={setUpdatedTo}/>
-			onChange={setUpdatedTo}/>
+	    </div>
-		  </>)}
-	    </FormField>
 
 	    {/* 検索 */}
 	    <div className="py-3">

frontend/src/pages/tags/NicoTagListPage.test.tsx

@@ -1,273 +0,0 @@
-import { fireEvent, screen, waitFor } from '@testing-library/react'
-import { beforeEach, describe, expect, it, vi } from 'vitest'
-
-import NicoTagListPage from '@/pages/tags/NicoTagListPage'
-import { dateString } from '@/lib/utils'
-import { buildTag, buildUser } from '@/test/factories'
-import { renderWithProviders } from '@/test/render'
-
-import type { NicoTag } from '@/types'
-
-const api = vi.hoisted (() => ({
-  apiGet:     vi.fn (),
-  apiPut:     vi.fn (),
-  isApiError: vi.fn (),
-}))
-
-const toastApi = vi.hoisted (() => ({
-  toast: vi.fn (),
-}))
-
-const dialogue = vi.hoisted (() => ({
-  confirm: vi.fn (),
-}))
-
-const scrollIntoView = vi.fn ()
-
-vi.mock ('@/lib/api', () => api)
-vi.mock ('@/components/ui/use-toast', () => toastApi)
-vi.mock ('@/components/dialogues/DialogueProvider', () => ({
-  useDialogue: () => dialogue,
-}))
-
-const buildNicoTag = (values: Partial<NicoTag> = {}): NicoTag => ({
-    ...buildTag (),
-    ...values,
-    category:               'nico',
-    linkedTags:             values.linkedTags ?? [],
-    recentPostTagCreatedAt: values.recentPostTagCreatedAt ?? null,
-})
-
-const renderPage = (route = '/tags/nico') =>
-  renderWithProviders (
-      <NicoTagListPage user={buildUser ({ role: 'member' })}/>,
-      { route },
-  )
-
-describe ('NicoTagListPage', () => {
-  beforeEach (() => {
-    vi.clearAllMocks ()
-    api.isApiError.mockReturnValue (false)
-    dialogue.confirm.mockResolvedValue (true)
-    Element.prototype.scrollIntoView = scrollIntoView
-    scrollIntoView.mockClear ()
-  })
-
-  it ('loads a filtered page from URL search parameters', async () => {
-    api.apiGet.mockResolvedValue ({
-      tags:  [buildNicoTag ({
-        id:                     1,
-        name:                   'nico:linked',
-        createdAt:              '2024-01-02T03:04:05Z',
-        recentPostTagCreatedAt: '2025-01-02T03:04:05Z',
-        updatedAt:              '2026-01-02T03:04:05Z',
-      })],
-      count: 21,
-    })
-
-    renderPage (
-        '/tags/nico?name=linked&linked_tag=destination&link_status=linked'
-	+ '&page=2&order=name:asc',
-    )
-
-    await waitFor (() => {
-      expect (api.apiGet).toHaveBeenCalledWith (
-          '/tags/nico',
-          { params: {
-            page:        2,
-            limit:       20,
-            name:        'linked',
-            linked_tag:  'destination',
-            link_status: 'linked',
-            order:       'name:asc',
-          } },
-      )
-    })
-    expect (await screen.findByText ('21 件')).toBeInTheDocument ()
-    expect (screen.getByLabelText ('前のページ')).toBeInTheDocument ()
-    expect (screen.queryByText ('なし')).not.toBeInTheDocument ()
-    expect (screen.getByText (dateString ('2025-01-02T03:04:05Z'))).toBeInTheDocument ()
-    expect (screen.queryByText (dateString ('2026-01-02T03:04:05Z'))).not.toBeInTheDocument ()
-    expect (screen.getByRole ('link', { name: 'ニコニコタグ ▲' })).toHaveAttribute (
-        'href',
-        expect.stringContaining ('order=name%3Adesc'),
-    )
-    expect (screen.getByRole ('link', { name: '最初に記載された日時' })).toHaveAttribute (
-        'href',
-        expect.stringContaining ('order=created_at%3Adesc'),
-    )
-    expect (screen.getByRole ('link', { name: '最近記載された日時' })).toHaveAttribute (
-        'href',
-        expect.stringContaining ('order=updated_at%3Adesc'),
-    )
-
-    fireEvent.mouseEnter (screen.getByLabelText ('前のページ'))
-    await waitFor (() => {
-      expect (api.apiGet).toHaveBeenLastCalledWith (
-          '/tags/nico',
-          { params: {
-            page:        1,
-            limit:       20,
-            name:        'linked',
-            linked_tag:  'destination',
-            link_status: 'linked',
-            order:       'name:asc',
-          } },
-      )
-    })
-  })
-
-  it ('scrolls to the table when moving between pages', async () => {
-    api.apiGet.mockResolvedValue ({
-      tags:  [buildNicoTag ({ id: 1, name: 'nico:linked' })],
-      count: 21,
-    })
-
-    renderPage ()
-
-    fireEvent.click (await screen.findByLabelText ('次のページ'))
-
-    await waitFor (() => {
-      expect (scrollIntoView).toHaveBeenCalledWith ({ behavior: 'smooth' })
-    })
-  })
-
-  it ('navigates with submitted search conditions', async () => {
-    api.apiGet.mockResolvedValue ({ tags: [], count: 0 })
-    renderPage ()
-
-    fireEvent.change (screen.getByLabelText ('ニコニコタグ'), {
-      target: { value: 'source' },
-    })
-    fireEvent.change (screen.getByLabelText ('連携タグ'), {
-      target: { value: 'destination' },
-    })
-    fireEvent.change (screen.getByLabelText ('連携状態'), {
-      target: { value: 'unlinked' },
-    })
-    fireEvent.submit (screen.getByRole ('button', { name: '検索' }).closest ('form')!)
-
-    await waitFor (() => {
-      expect (api.apiGet).toHaveBeenLastCalledWith (
-          '/tags/nico',
-          { params: expect.objectContaining ({
-            page:        1,
-            name:        'source',
-            linked_tag:  'destination',
-            link_status: 'unlinked',
-          }) },
-      )
-    })
-  })
-
-  it ('updates links from a tag card', async () => {
-    api.apiGet
-      .mockResolvedValueOnce ({
-        tags:  [buildNicoTag ({ id: 7, name: 'nico:source' })],
-        count: 1,
-      })
-      .mockResolvedValueOnce ({
-        tags: [
-          buildNicoTag ({
-            id:         7,
-            name:       'nico:source',
-            linkedTags: [buildTag ({ id: 8, name: '連携先' })],
-          }),
-        ],
-        count: 1,
-      })
-    api.apiPut.mockResolvedValueOnce ([buildTag ({ id: 8, name: '連携先' })])
-
-    renderPage ()
-
-    fireEvent.click (await screen.findByRole ('button', { name: '編集' }))
-    fireEvent.change (screen.getByLabelText ('連携する広場タグ'), {
-      target: { value: '連携先' },
-    })
-    fireEvent.click (screen.getByRole ('button', { name: '保存' }))
-
-    await waitFor (() => {
-      expect (api.apiPut).toHaveBeenCalledWith (
-          '/tags/nico/7',
-          expect.any (FormData),
-          { headers: { 'Content-Type': 'multipart/form-data' } },
-      )
-    })
-    expect (await screen.findByText ('1 件')).toBeInTheDocument ()
-  })
-
-  it ('asks before discarding changes when editing another tag', async () => {
-    api.apiGet.mockResolvedValueOnce ({
-      tags: [
-        buildNicoTag ({ id: 1, name: 'nico:first' }),
-        buildNicoTag ({ id: 2, name: 'nico:second' }),
-      ],
-      count: 2,
-    })
-
-    renderPage ()
-    dialogue.confirm.mockResolvedValueOnce (false)
-
-    const editButtons = await screen.findAllByRole ('button', { name: '編集' })
-    fireEvent.click (editButtons[0])
-    fireEvent.change (screen.getByLabelText ('連携する広場タグ'), {
-      target: { value: '入力中' },
-    })
-    fireEvent.click (screen.getAllByRole ('button', { name: '編集' })[0])
-
-    await waitFor (() => {
-      expect (dialogue.confirm).toHaveBeenCalledWith ({
-        title:       '編集中の内容を破棄しますか?',
-        confirmText: '破棄',
-        variant:     'danger',
-      })
-    })
-    expect (screen.getAllByLabelText ('連携する広場タグ')).toHaveLength (1)
-    expect (screen.getByLabelText ('連携する広場タグ')).toHaveValue ('入力中')
-  })
-
-  it ('switches editing rows without confirmation when unchanged', async () => {
-    api.apiGet.mockResolvedValueOnce ({
-      tags: [
-        buildNicoTag ({ id: 1, name: 'nico:first' }),
-        buildNicoTag ({ id: 2, name: 'nico:second' }),
-      ],
-      count: 2,
-    })
-
-    renderPage ()
-
-    const editButtons = await screen.findAllByRole ('button', { name: '編集' })
-    fireEvent.click (editButtons[0])
-    fireEvent.click (screen.getAllByRole ('button', { name: '編集' })[0])
-
-    expect (dialogue.confirm).not.toHaveBeenCalled ()
-    expect (screen.getAllByLabelText ('連携する広場タグ')).toHaveLength (1)
-  })
-
-  it ('shows tags field validation errors inside the edited card', async () => {
-    api.apiGet.mockResolvedValueOnce ({
-      tags:  [buildNicoTag ({ id: 7, name: 'nico:source' })],
-      count: 1,
-    })
-    api.isApiError.mockReturnValue (true)
-    api.apiPut.mockRejectedValueOnce ({
-      response: {
-        status: 422,
-        data:   {
-          type:        'validation_error',
-          errors:      { tags: ['タグ名を確認してください．'] },
-          base_errors: [],
-        },
-      },
-    })
-
-    renderPage ()
-
-    fireEvent.click (await screen.findByRole ('button', { name: '編集' }))
-    fireEvent.click (screen.getByRole ('button', { name: '保存' }))
-
-    expect (await screen.findByText ('タグ名を確認してください．')).toBeInTheDocument ()
-    expect (screen.getByLabelText ('連携する広場タグ')).toHaveAttribute ('aria-invalid', 'true')
-  })
-})

frontend/src/pages/tags/NicoTagListPage.tsx

@@ -1,179 +1,104 @@
-import { useQuery, useQueryClient } from '@tanstack/react-query'
+import type { FC } from 'react'
-import { Check, LoaderCircle, Pencil, X } from 'lucide-react'
+
-import { useEffect, useMemo, useState } from 'react'
+import { useCallback, useEffect, useRef, useState } from 'react'
 import { Helmet } from 'react-helmet-async'
-import { useLocation, useNavigate } from 'react-router-dom'
 
 import TagLink from '@/components/TagLink'
-import SortHeader from '@/components/SortHeader'
-import FieldError from '@/components/common/FieldError'
-import FormField from '@/components/common/FormField'
 import PageTitle from '@/components/common/PageTitle'
-import Pagination from '@/components/common/Pagination'
 import TextArea from '@/components/common/TextArea'
-import { useDialogue } from '@/components/dialogues/DialogueProvider'
 import MainArea from '@/components/layout/MainArea'
 import { toast } from '@/components/ui/use-toast'
 import { SITE_TITLE } from '@/config'
-import { apiPut } from '@/lib/api'
+import { apiGet, apiPut } from '@/lib/api'
-import { extractValidationError } from '@/lib/apiErrors'
-import { tagsKeys } from '@/lib/queryKeys'
-import { fetchNicoTags } from '@/lib/tags'
-import { cn, dateString, inputClass } from '@/lib/utils'
-import { canEditContent } from '@/lib/users'
 
-import type { FC, FormEvent } from 'react'
+import type { NicoTag, Tag, User } from '@/types'
 
-import type { FetchNicoTagsOrder, FetchNicoTagsOrderField, NicoTag, Tag, User } from '@/types'
-
-type LinkStatus = 'all' | 'linked' | 'unlinked'
 type Props = { user: User | null }
 
 
-const setIf = (qs: URLSearchParams, key: string, value: string) => {
-  const trimmed = value.trim ()
-  if (trimmed)
-    qs.set (key, trimmed)
-}
-
-
 const NicoTagListPage: FC<Props> = ({ user }) => {
-  const dialogue = useDialogue ()
+  const [cursor, setCursor] = useState ('')
-  const location = useLocation ()
+  const [editing, setEditing] = useState<{ [key: number]: boolean }> ({ })
-  const navigate = useNavigate ()
+  const [loading, setLoading] = useState (false)
-  const queryClient = useQueryClient ()
+  const [nicoTags, setNicoTags] = useState<NicoTag[]> ([])
-  const query = useMemo (() => new URLSearchParams (location.search), [location.search])
+  const [rawTags, setRawTags] = useState<{ [key: number]: string }> ({ })
 
-  const page = Number (query.get ('page') ?? 1)
+  const loaderRef = useRef<HTMLDivElement | null> (null)
-  const limit = Number (query.get ('limit') ?? 20)
-  const qName = query.get ('name') ?? ''
-  const qLinkedTag = query.get ('linked_tag') ?? ''
-  const qLinkStatus = (query.get ('link_status') || 'all') as LinkStatus
-  const order = (query.get ('order') || 'updated_at:desc') as FetchNicoTagsOrder
 
-  const [editingId, setEditingId] = useState<number | null> (null)
+  const memberFlg = ['admin', 'member'].some (r => user?.role === r)
-  const [errorsByTagId, setErrorsByTagId] = useState<Record<number, string[]>> ({ })
-  const [linkStatus, setLinkStatus] = useState<LinkStatus> ('all')
-  const [linkedTag, setLinkedTag] = useState ('')
-  const [name, setName] = useState ('')
-  const [rawTags, setRawTags] = useState<Record<number, string>> ({ })
-  const [savingId, setSavingId] = useState<number | null> (null)
 
-  const keys = {
+  const applyLoadedTags = useCallback ((data: { tags: NicoTag[]; nextCursor: string },
-      name: qName, linkedTag: qLinkedTag, linkStatus: qLinkStatus, page, limit, order }
+					withCursor: boolean) => {
-  const { data, isError, isLoading: loading } = useQuery ({
+    setNicoTags (tags => [...(withCursor ? tags : []), ...data.tags])
-      queryKey: tagsKeys.nicoIndex (keys),
+    setCursor (data.nextCursor)
-      queryFn:  () => fetchNicoTags (keys) })
-  const nicoTags = data?.tags ?? []
-  const count = data?.count ?? 0
 
-  const editable = canEditContent (user)
+    const newEditing = Object.fromEntries (data.tags.map (t => [t.id, false]))
-  const totalPages = Math.ceil (count / limit)
+    setEditing (editing => ({ ...editing, ...newEditing }))
 
-  const handleSearch = (e: FormEvent) => {
+    const newRawTags = Object.fromEntries (
-    e.preventDefault ()
+	data.tags.map (t => [t.id, t.linkedTags.map (lt => lt.name).join (' ')]))
+    setRawTags (rawTags => ({ ...rawTags, ...newRawTags }))
+  }, [])
 
-    const qs = new URLSearchParams ()
+  const loadInitial = useCallback (async () => {
-    setIf (qs, 'name', name)
+    setLoading (true)
-    setIf (qs, 'linked_tag', linkedTag)
+
-    if (linkStatus !== 'all')
+    const data = await apiGet<{ tags: NicoTag[]; nextCursor: string }> ('/tags/nico')
-      qs.set ('link_status', linkStatus)
+    applyLoadedTags (data, false)
-    qs.set ('page', '1')
+
-    qs.set ('limit', String (limit))
+    setLoading (false)
-    qs.set ('order', order)
+  }, [applyLoadedTags])
-    navigate (`${ location.pathname }?${ qs.toString () }`)
+
+  const loadMore = useCallback (async () => {
+    setLoading (true)
+
+    const data = await apiGet<{ tags: NicoTag[]; nextCursor: string }> (
+	'/tags/nico', { params: { cursor } })
+    applyLoadedTags (data, true)
+
+    setLoading (false)
+  }, [applyLoadedTags, cursor])
+
+  const handleEdit = async (id: number) => {
+    if (editing[id])
+      {
+	const formData = new FormData
+	formData.append ('tags', rawTags[id])
+
+	const data = await apiPut<Tag[]> (`/tags/nico/${ id }`, formData,
+				    { headers: { 'Content-Type': 'multipart/form-data' } })
+	setNicoTags (nicoTags => {
+	  nicoTags.find (t => t.id === id)!.linkedTags = data
+	  return [...nicoTags]
+	})
+	setRawTags (rawTags => ({ ...rawTags, [id]: data.map (t => t.name).join (' ') }))
+
+	toast ({ title: '更新しました．' })
+      }
+
+    setEditing (editing => ({ ...editing, [id]: !(editing[id]) }))
   }
 
-  const defaultDirection = {
+  useEffect(() => {
-      name:       'asc',
+    const observer = new IntersectionObserver (entries => {
-      created_at: 'desc',
+      if (entries[0].isIntersecting && !(loading) && cursor)
-      updated_at: 'desc',
+	loadMore ()
-  } as const
+    }, { threshold: 1 })
 
-  const beginEdit = async (tag: NicoTag) => {
+    const target = loaderRef.current
-    const editingTag = nicoTags.find (tag => tag.id === editingId)
+    if (target)
-    const editingValue = editingTag?.linkedTags.map (tag => tag.name).join (' ') ?? ''
+      observer.observe (target)
-    const editingChanged = editingId != null && rawTags[editingId] !== editingValue
 
-    if (editingId != null && editingId !== tag.id && editingChanged
+    return () => {
-	&& !(await dialogue.confirm ({
+      if (target)
-	  title:       '編集中の内容を破棄しますか?',
+	observer.unobserve (target)
-	  confirmText: '破棄',
-	  variant:     'danger',
-	})))
-      return
-
-    setEditingId (tag.id)
-    setRawTags (rawTags => ({
-	...rawTags,
-	[tag.id]: tag.linkedTags.map (linkedTag => linkedTag.name).join (' '),
-    }))
-    setErrorsByTagId (errors => ({ ...errors, [tag.id]: [] }))
-  }
-
-  const cancelEdit = (tag: NicoTag) => {
-    setEditingId (null)
-    setRawTags (rawTags => ({
-	...rawTags,
-	[tag.id]: tag.linkedTags.map (linkedTag => linkedTag.name).join (' '),
-    }))
-    setErrorsByTagId (errors => ({ ...errors, [tag.id]: [] }))
-  }
-
-  const saveLinks = async (id: number) => {
-    const formData = new FormData
-    formData.append ('tags', rawTags[id] ?? '')
-    setSavingId (id)
-
-    try
-    {
-      await apiPut<Tag[]> (`/tags/nico/${ id }`, formData,
-			   { headers: { 'Content-Type': 'multipart/form-data' } })
-      setErrorsByTagId (errors => ({ ...errors, [id]: [] }))
-      setEditingId (null)
-      await queryClient.invalidateQueries ({ queryKey: tagsKeys.nicoRoot })
-      toast ({ description: '連携を更新しました．' })
     }
-    catch (e)
+  }, [cursor, loadMore, loading])
-    {
-      const validationError = extractValidationError<'tags'> (e)
-      setErrorsByTagId (errors => ({
-	  ...errors,
-	  [id]: validationError?.fieldErrors.tags
-	      ?? validationError?.baseErrors
-	      ?? ['更新できませんでした．'],
-      }))
-      toast ({ title: '更新失敗', description: '入力内容を確認してください．' })
-    }
-    finally
-    {
-      setSavingId (null)
-    }
-  }
 
   useEffect (() => {
-    setName (qName)
+    setNicoTags ([])
-    setLinkedTag (qLinkedTag)
+    loadInitial ()
-    setLinkStatus (qLinkStatus)
+  }, [loadInitial])
-    setEditingId (null)
-
-    document.querySelector ('table')?.scrollIntoView ({ behavior: 'smooth' })
-  }, [location.search, qLinkedTag, qLinkStatus, qName])
-
-  useEffect (() => {
-    if (!(data))
-      return
-
-    setRawTags (Object.fromEntries (data.tags.map (tag => [
-      tag.id,
-      tag.linkedTags.map (linkedTag => linkedTag.name).join (' '),
-    ])))
-  }, [data])
-
-  useEffect (() => {
-    if (isError)
-      toast ({ title: '読込失敗', description: 'ニコニコ連携を読み込めませんでした．' })
-  }, [isError])
 
   return (
       <MainArea>
@@ -183,201 +108,58 @@ const NicoTagListPage: FC<Props> = ({ user }) => {
 
 	<div className="max-w-xl">
 	  <PageTitle>ニコニコ連携</PageTitle>
-	  <p className="mb-4 text-sm text-gray-600 dark:text-gray-300">
-	    ニコニコタグを広場のタグへ結び付けます．
-	  </p>
-
-	  <form onSubmit={handleSearch} className="space-y-2">
-	    <FormField label="ニコニコタグ">
-	      {({ invalid }) => (
-		  <input
-		      type="text"
-		      aria-label="ニコニコタグ"
-		      value={name}
-		      onChange={e => setName (e.target.value)}
-		      className={inputClass (invalid)}/>)}
-	    </FormField>
-
-	    <FormField label="連携タグ">
-	      {({ invalid }) => (
-		  <input
-		      type="text"
-		      aria-label="連携タグ"
-		      value={linkedTag}
-		      onChange={e => setLinkedTag (e.target.value)}
-		      className={inputClass (invalid)}/>)}
-	    </FormField>
-
-	    <FormField label="連携状態">
-	      {({ invalid }) => (
-		  <select
-		      aria-label="連携状態"
-		      value={linkStatus}
-		      onChange={e => setLinkStatus (e.target.value as LinkStatus)}
-		      className={inputClass (invalid)}>
-		    <option value="all">すべて</option>
-		    <option value="linked">連携済み</option>
-		    <option value="unlinked">未連携</option>
-		  </select>)}
-	    </FormField>
-
-	    <div className="py-3">
-	      <button
-		  type="submit"
-		  className="rounded bg-blue-500 px-4 py-2 text-white">
-		検索
-	      </button>
-	    </div>
-	  </form>
 	</div>
 
-	{loading
+	<div className="mt-4">
-	 ? 'Loading...'
+	  {nicoTags.length > 0 && (
-	 : (
+	      <table className="table-auto w-full border-collapse mb-4">
-	    <div className="mt-6">
+		<thead className="border-b-2 border-black dark:border-white">
-	      <div className="mb-3 flex items-baseline justify-between gap-4">
+		  <tr>
-		<h2 className="text-lg font-bold">検索結果</h2>
+		    <th className="p-2 text-left">ニコニコタグ</th>
-		<span className="text-sm text-gray-500 dark:text-gray-400">{count} 件</span>
+		    <th className="p-2 text-left">連携タグ</th>
-	      </div>
+		    {memberFlg && <th></th>}
-
+		  </tr>
-	      {nicoTags.length > 0
+		</thead>
-	       ? (
+		<tbody>
-		  <div className="overflow-x-auto">
+		  {nicoTags.map ((tag, i) => (
-		    <table className="w-full min-w-[800px] table-fixed border-collapse">
+		      <tr key={i} className="even:bg-gray-100 dark:even:bg-gray-700">
-		      <colgroup>
+			<td className="p-2">
-			<col className="w-64"/>
+			  <TagLink tag={tag} withWiki={false} withCount={false}/>
-			<col className="w-[48rem]"/>
+			</td>
-			<col className="w-56"/>
+			<td className="p-2">
-			<col className="w-56"/>
+			  {editing[tag.id]
-			{editable && <col className="w-24"/>}
+			   ? (
-		      </colgroup>
+			      <TextArea value={rawTags[tag.id]} onChange={ev => {
-		      <thead className="border-b-2 border-black dark:border-white">
+				setRawTags (rawTags => ({ ...rawTags, [tag.id]: ev.target.value }))
-			<tr>
+			      }}/>)
-			  <th className="p-2 text-left whitespace-nowrap">
+			   : tag.linkedTags.map((lt, j) => (
-			    <SortHeader<FetchNicoTagsOrderField>
+			      <span key={j} className="mr-2">
-				by="name"
+				<TagLink tag={lt}
-				label="ニコニコタグ"
+					 linkFlg={false}
-				currentOrder={order}
+					 withCount={false}/>
-				defaultDirection={defaultDirection}/>
+			      </span>))}
-			  </th>
+			</td>
-			  <th className="p-2 text-left">連携タグ</th>
+			{memberFlg && (
-			  <th className="p-2 text-left whitespace-nowrap">
+			    <td className="p-2">
-			    <SortHeader<FetchNicoTagsOrderField>
+			      <a href="#" onClick={ev => {
-				by="created_at"
+				ev.preventDefault ()
-				label="最初に記載された日時"
+				handleEdit (tag.id)
-				currentOrder={order}
+			      }}>
-				defaultDirection={defaultDirection}/>
+			        {editing[tag.id]
-			  </th>
+				 ? (
-			  <th className="p-2 text-left whitespace-nowrap">
+				    <span className="text-red-600 hover:text-red-400
-			    <SortHeader<FetchNicoTagsOrderField>
+						     dark:text-red-300 dark:hover:text-red-100">
-				by="updated_at"
+				      更新
-				label="最近記載された日時"
+				    </span>)
-				currentOrder={order}
+				 : <span>編輯</span>}
-				defaultDirection={defaultDirection}/>
+			      </a>
-			  </th>
+			    </td>)}
-			  {editable && <th className="p-2"></th>}
+		      </tr>))}
-			</tr>
+		</tbody>
-		      </thead>
+	      </table>)}
-		      <tbody>
+	  {loading && 'Loading...'}
-			{nicoTags.map (tag => {
+	  <div ref={loaderRef} className="h-12"></div>
-			  const isEditing = editingId === tag.id
+	</div>
-
-			  return [
-			    <tr
-				key={tag.id}
-				className={cn (
-				    'border-b border-gray-200 dark:border-gray-700',
-				    isEditing
-				    ? 'bg-rose-50 dark:bg-rose-950/30'
-				    : 'even:bg-gray-100 dark:even:bg-gray-800')}>
-			      <td className="p-2 align-top font-semibold">
-				<TagLink tag={tag} withWiki={false} withCount={false}/>
-			      </td>
-			      <td className="p-2 align-top">
-				{tag.linkedTags.map ((linkedTag, i) => (
-				    <span key={linkedTag.id}>
-				      {i > 0 && ' '}
-				      <TagLink
-					  tag={linkedTag}
-					  linkFlg={false}
-					  withCount={false}/>
-				    </span>))}
-			      </td>
-			      <td className="p-2 align-top whitespace-nowrap">
-				{dateString (tag.createdAt)}
-			      </td>
-			      <td className="p-2 align-top whitespace-nowrap">
-				{tag.recentPostTagCreatedAt && dateString (tag.recentPostTagCreatedAt)}
-			      </td>
-			      {editable && (
-				  <td className="p-2 text-right align-top">
-				    {!(isEditing) && (
-					<button
-					    type="button"
-					    onClick={() => beginEdit (tag)}
-					    className="inline-flex items-center gap-1 text-sm text-blue-700
-						       hover:underline dark:text-blue-300">
-					  <Pencil className="size-3.5"/>
-					  編集
-					</button>)}
-				  </td>)}
-			    </tr>,
-			    isEditing && (
-				<tr key={`${ tag.id }-edit`}
-				    className="border-b border-rose-200 bg-rose-50 dark:border-rose-900
-					       dark:bg-rose-950/30">
-				  <td colSpan={editable ? 5 : 4} className="p-3">
-				    <div className="space-y-2">
-				      <label htmlFor={`nico-links-${ tag.id }`}
-					     className="block text-sm font-semibold">
-					連携する広場タグ
-				      </label>
-				      <TextArea
-					  id={`nico-links-${ tag.id }`}
-					  value={rawTags[tag.id] ?? ''}
-					  invalid={(errorsByTagId[tag.id] ?? []).length > 0}
-					  className="min-h-24 resize-y"
-					  placeholder="タグ名を空白または改行で区切って入力"
-					  onChange={e => setRawTags (rawTags => ({
-					      ...rawTags,
-					      [tag.id]: e.target.value,
-					  }))}/>
-				      <FieldError messages={errorsByTagId[tag.id]}/>
-				      <div className="flex justify-end gap-2">
-					<button
-					    type="button"
-					    disabled={savingId === tag.id}
-					    onClick={() => cancelEdit (tag)}
-					    className="inline-flex items-center gap-1 rounded border
-						       border-gray-300 px-3 py-1.5 text-sm
-						       disabled:opacity-50 dark:border-gray-700">
-					  <X className="size-3.5"/>
-					  取消
-					</button>
-					<button
-					    type="button"
-					    disabled={savingId === tag.id}
-					    onClick={() => saveLinks (tag.id)}
-					    className="inline-flex items-center gap-1 rounded bg-rose-700
-						       px-3 py-1.5 text-sm text-white disabled:opacity-50">
-					  {savingId === tag.id
-					   ? <LoaderCircle className="size-3.5 animate-spin"/>
-					   : <Check className="size-3.5"/>}
-					  保存
-					</button>
-				      </div>
-				    </div>
-				  </td>
-				</tr>),
-			  ]
-			})}
-		      </tbody>
-		    </table>
-		  </div>)
-	       : <p>結果ないよ（笑）</p>}
-
-	      <Pagination page={page} totalPages={totalPages}/>
-	    </div>)}
       </MainArea>)
 }
 

frontend/src/pages/tags/TagDetailPage.test.tsx

@@ -1,6 +1,6 @@
 import { fireEvent, screen, waitFor } from '@testing-library/react'
 import { Route, Routes } from 'react-router-dom'
-import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import TagDetailPage from '@/pages/tags/TagDetailPage'
 import { buildTag } from '@/test/factories'
@@ -11,8 +11,7 @@ const tagsApi = vi.hoisted (() => ({
 }))
 
 const api = vi.hoisted (() => ({
-  apiPut:     vi.fn (),
+  apiPut: vi.fn (),
-  isApiError: vi.fn (),
 }))
 
 const toastApi = vi.hoisted (() => ({
@@ -29,14 +28,9 @@ const renderPage = () =>
         <Route path="/tags/:id" element={<TagDetailPage/>}/>
       </Routes>,
       { route: '/tags/7' },
-)
+  )
 
 describe ('TagDetailPage', () => {
-  beforeEach (() => {
-    vi.clearAllMocks ()
-    api.isApiError.mockReturnValue (false)
-  })
-
   it ('loads and displays an editable tag', async () => {
     tagsApi.fetchTag.mockResolvedValueOnce (
         buildTag ({ id: 7, name: '虹夏', category: 'character', aliases: ['drums'] }),
@@ -74,28 +68,4 @@ describe ('TagDetailPage', () => {
 
     expect (await screen.findByRole ('button', { name: '更新' })).toBeDisabled ()
   })
-
-  it ('shows validation errors returned for tag fields', async () => {
-    tagsApi.fetchTag.mockResolvedValueOnce (buildTag ({ id: 7, name: 'old' }))
-    api.isApiError.mockReturnValue (true)
-    api.apiPut.mockRejectedValueOnce ({
-      response: {
-        status: 422,
-        data:   {
-          type:        'validation_error',
-          message:     '入力内容を確認してください．',
-          errors:      { category: ['ニコタグは変更できません．'] },
-          base_errors: [],
-        },
-      },
-    })
-
-    renderPage ()
-
-    await screen.findByDisplayValue ('old')
-    fireEvent.submit (screen.getByRole ('button', { name: '更新' }).closest ('form')!)
-
-    expect (await screen.findByText ('ニコタグは変更できません．')).toBeInTheDocument ()
-    expect (screen.getByRole ('combobox')).toHaveAttribute ('aria-invalid', 'true')
-  })
 })

frontend/src/pages/tags/TagDetailPage.tsx

@@ -3,8 +3,7 @@ import { useEffect, useState } from 'react'
 import { useParams } from 'react-router-dom'
 
 import TagLink from '@/components/TagLink'
-import FieldError from '@/components/common/FieldError'
+import Label from '@/components/common/Label'
-import FormField from '@/components/common/FormField'
 import PageTitle from '@/components/common/PageTitle'
 import MainArea from '@/components/layout/MainArea'
 import { toast } from '@/components/ui/use-toast'
@@ -12,15 +11,12 @@ import { CATEGORIES, CATEGORY_NAMES } from '@/consts'
 import { apiPut } from '@/lib/api'
 import { postsKeys, tagsKeys } from '@/lib/queryKeys'
 import { fetchTag } from '@/lib/tags'
-import { cn, inputClass } from '@/lib/utils'
+import { cn } from '@/lib/utils'
-import { useValidationErrors } from '@/lib/useValidationErrors'
 
 import type { FC, FormEvent } from 'react'
 
 import type { Category, Tag } from '@/types'
 
-type TagFormField = 'name' | 'category' | 'aliases' | 'parentTags'
-
 
 const TagDetailPage: FC = () => {
   const { id } = useParams ()
@@ -36,14 +32,11 @@ const TagDetailPage: FC = () => {
   const [aliases, setAliases] = useState ('')
   const [parentTags, setParentTags] = useState ('')
   const [disabled, setDisabled] = useState (true)
-  const { baseErrors, fieldErrors, clearValidationErrors, applyValidationError } =
-      useValidationErrors<TagFormField> ()
 
   const qc = useQueryClient ()
 
   const handleSubmit = async (e: FormEvent) => {
     e.preventDefault ()
-    clearValidationErrors ()
 
     const formData = new FormData
     formData.append ('name', name)
@@ -64,9 +57,8 @@ const TagDetailPage: FC = () => {
       qc.invalidateQueries ({ queryKey: tagsKeys.root })
       toast ({ description: '更新しました．' })
     }
-    catch (e)
+    catch
     {
-      applyValidationError (e)
       toast ({ description: '更新に失敗しました．' })
     }
   }
@@ -97,73 +89,57 @@ const TagDetailPage: FC = () => {
 	      </PageTitle>
 
 	      <form onSubmit={handleSubmit} className="my-4 space-y-2">
-		<FieldError messages={baseErrors}/>
-
 		{/* 名称 */}
-		<FormField label="名称" messages={fieldErrors.name}>
+		<div>
-		  {({ describedBy, invalid }) => (
+		  <Label>名称</Label>
-		      <>
+		  {/* TODO: 補完に対応させる */}
-			{/* TODO: 補完に対応させる */}
+		  <input
-			<input
+		      type="text"
-			    type="text"
+		      disabled={disabled}
-			    disabled={disabled}
+		      value={name}
-			    value={name}
+		      onChange={e => setName (e.target.value)}
-			    onChange={e => setName (e.target.value)}
+		      className="w-full border p-2 rounded"/>
-			    aria-describedby={describedBy}
+		</div>
-			    aria-invalid={invalid}
-			    className={inputClass (invalid)}/>
-		      </>)}
-		</FormField>
 
 		{/* カテゴリ */}
-		<FormField label="カテゴリ" messages={fieldErrors.category}>
+		<div>
-		  {({ describedBy, invalid }) => (
+		  <Label>カテゴリ</Label>
 		  <select
 		      disabled={disabled}
 		      value={category ?? ''}
 		      onChange={e => setCategory(e.target.value as Category)}
-		      aria-describedby={describedBy}
+		      className="w-full border p-2 rounded">
-		      aria-invalid={invalid}
-		      className={inputClass (invalid)}>
 		    {CATEGORIES.filter (cat => tag.category === 'nico' || cat !== 'nico')
 			       .map (cat => (
 			<option key={cat} value={cat}>
 			  {CATEGORY_NAMES[cat]}
 			</option>))}
-		  </select>)}
+		  </select>
-		</FormField>
+		</div>
 
 		{/* 別名 */}
-		<FormField label="別名" messages={fieldErrors.aliases}>
+		<div>
-		  {({ describedBy, invalid }) => (
+		  <Label>別名</Label>
-		      <>
+		  {/* TODO: 補完に対応させる */}
-			{/* TODO: 補完に対応させる */}
+		  <input
-			<input
+		      type="text"
-			    type="text"
+		      disabled={disabled}
-			    disabled={disabled}
+		      value={aliases}
-			    value={aliases}
+		      onChange={e => setAliases (e.target.value)}
-			    onChange={e => setAliases (e.target.value)}
+		      className="w-full border p-2 rounded"/>
-			    aria-describedby={describedBy}
+		</div>
-			    aria-invalid={invalid}
-			    className={inputClass (invalid)}/>
-		      </>)}
-		</FormField>
 
 		{/* 上位タグ */}
-		<FormField label="上位タグ" messages={fieldErrors.parentTags}>
+		<div>
-		  {({ describedBy, invalid }) => (
+		  <Label>上位タグ</Label>
-		      <>
+		  {/* TODO: 補完に対応させる */}
-			{/* TODO: 補完に対応させる */}
+		  <input
-			<input
+		      type="text"
-			    type="text"
+		      disabled={disabled}
-			    disabled={disabled}
+		      value={parentTags}
-			    value={parentTags}
+		      onChange={e => setParentTags (e.target.value)}
-			    onChange={e => setParentTags (e.target.value)}
+		      className="w-full border p-2 rounded"/>
-			    aria-describedby={describedBy}
+		</div>
-			    aria-invalid={invalid}
-			    className={inputClass (invalid)}/>
-		      </>)}
-		</FormField>
 
 		<div className="py-3">
 		  <button

frontend/src/pages/tags/TagListPage.tsx

@@ -7,7 +7,7 @@ import PrefetchLink from '@/components/PrefetchLink'
 import SortHeader from '@/components/SortHeader'
 import TagLink from '@/components/TagLink'
 import DateTimeField from '@/components/common/DateTimeField'
-import FormField from '@/components/common/FormField'
+import Label from '@/components/common/Label'
 import PageTitle from '@/components/common/PageTitle'
 import Pagination from '@/components/common/Pagination'
 import MainArea from '@/components/layout/MainArea'
@@ -15,7 +15,7 @@ import { SITE_TITLE } from '@/config'
 import { CATEGORIES, CATEGORY_NAMES } from '@/consts'
 import { tagsKeys } from '@/lib/queryKeys'
 import { fetchTags } from '@/lib/tags'
-import { dateString, inputClass } from '@/lib/utils'
+import { dateString } from '@/lib/utils'
 
 import type { FC, FormEvent } from 'react'
 
@@ -127,79 +127,71 @@ const TagListPage: FC = () => {
 
 	  <form onSubmit={handleSearch} className="space-y-2">
 	    {/* 名前 */}
-	    <FormField label="名前">
+	    <div>
-	      {({ invalid }) => (
+	      <Label>名前</Label>
-		  <input
+	      <input
-		      type="text"
+		  type="text"
-		      value={name}
+		  value={name}
-		      onChange={e => setName (e.target.value)}
+		  onChange={e => setName (e.target.value)}
-		      className={inputClass (invalid)}/>)}
+		  className="w-full border p-2 rounded"/>
-	    </FormField>
+	    </div>
 
 	    {/* カテゴリ */}
-	    <FormField label="カテゴリ">
+	    <div>
-	      {({ invalid }) => (
+	      <Label>カテゴリ</Label>
-		  <select
+	      <select
-		      value={category ?? ''}
+		  value={category ?? ''}
-		      onChange={e => setCategory((e.target.value || null) as Category | null)}
+		  onChange={e => setCategory((e.target.value || null) as Category | null)}
-		      className={inputClass (invalid)}>
+		  className="w-full border p-2 rounded">
-		    <option value="">&nbsp;</option>
+		<option value="">&nbsp;</option>
-		    {CATEGORIES.map (cat => (
+		{CATEGORIES.map (cat => (
-			<option key={cat} value={cat}>
+		    <option key={cat} value={cat}>
-			  {CATEGORY_NAMES[cat]}
+		      {CATEGORY_NAMES[cat]}
-			</option>))}
+		    </option>))}
-		  </select>)}
+	      </select>
-	    </FormField>
+	    </div>
 
 	    {/* 広場の投稿数 */}
-	    <FormField label="広場の投稿数">
+	    <div>
-	      {({ invalid }) => (
+	      <Label>広場の投稿数</Label>
-		  <>
+	      <input
-		    <input
+		  type="number"
-			type="number"
+		  min="0"
-			min="0"
+		  value={postCountGTE < 0 ? 0 : String (postCountGTE)}
-			value={postCountGTE < 0 ? 0 : String (postCountGTE)}
+		  onChange={e => setPostCountGTE (Number (e.target.value || 0))}
-			onChange={e => setPostCountGTE (Number (e.target.value || 0))}
+		  className="border rounded p-2"/>
-			className={inputClass (invalid, 'w-auto')}/>
+	      <span className="mx-1">〜</span>
-		    <span className="mx-1">〜</span>
+	      <input
-		    <input
+		  type="number"
-			type="number"
+		  min="0"
-			min="0"
+		  value={postCountLTE == null ? '' : String (postCountLTE)}
-			value={postCountLTE == null ? '' : String (postCountLTE)}
+		  onChange={e => setPostCountLTE (e.target.value ? Number (e.target.value) : null)}
-			onChange={e => setPostCountLTE (e.target.value
+		  className="border rounded p-2"/>
-							? Number (e.target.value)
+	    </div>
-							: null)}
-			className={inputClass (invalid, 'w-auto')}/>
-		  </>)}
-	    </FormField>
 
 	    {/* はじめて記載された日時 */}
-	    <FormField label="はじめて記載された日時">
+	    <div>
-	      {() => (
+	      <Label>はじめて記載された日時</Label>
-		  <>
+	      <DateTimeField
-		    <DateTimeField
+		  value={createdFrom ?? undefined}
-			value={createdFrom ?? undefined}
+		  onChange={setCreatedFrom}/>
-			onChange={setCreatedFrom}/>
+	      <span className="mx-1">〜</span>
-		    <span className="mx-1">〜</span>
+	      <DateTimeField
-		    <DateTimeField
+		  value={createdTo ?? undefined}
-			value={createdTo ?? undefined}
+		  onChange={setCreatedTo}/>
-			onChange={setCreatedTo}/>
+	    </div>
-		  </>)}
-	    </FormField>
 
 	    {/* 定義の更新日時 */}
-	    <FormField label="定義の更新日時">
+	    <div>
-	      {() => (
+	      <Label>定義の更新日時</Label>
-		  <>
+	      <DateTimeField
-		    <DateTimeField
+		  value={updatedFrom ?? undefined}
-			value={updatedFrom ?? undefined}
+		  onChange={setUpdatedFrom}/>
-			onChange={setUpdatedFrom}/>
+	      <span className="mx-1">〜</span>
-		    <span className="mx-1">〜</span>
+	      <DateTimeField
-		    <DateTimeField
+		  value={updatedTo ?? undefined}
-			value={updatedTo ?? undefined}
+		  onChange={setUpdatedTo}/>
-			onChange={setUpdatedTo}/>
+	    </div>
-		  </>)}
-	    </FormField>
 
 	    <div className="py-3">
 	      <button

frontend/src/pages/theatres/TheatreDetailPage.test.tsx

@@ -1,298 +0,0 @@
-import { act, fireEvent, screen, waitFor } from '@testing-library/react'
-import { Route, Routes } from 'react-router-dom'
-import { beforeEach, describe, expect, it, vi } from 'vitest'
-
-import TheatreDetailPage from '@/pages/theatres/TheatreDetailPage'
-import { buildPost,
-	 buildTheatre,
-	 buildTheatreComment,
-	 buildTheatreInfo,
-	 buildTheatrePostSelectionWeights,
-	 buildTheatreProgramme,
-	 buildUser } from '@/test/factories'
-import { renderWithProviders } from '@/test/render'
-
-import type { ReactNode } from 'react'
-
-const api = vi.hoisted (() => ({
-  apiDelete:  vi.fn (),
-  apiGet:     vi.fn (),
-  apiPatch:   vi.fn (),
-  apiPost:    vi.fn (),
-  apiPut:     vi.fn (),
-  isApiError: vi.fn (() => false),
-}))
-
-const postsApi = vi.hoisted (() => ({
-  fetchPost: vi.fn (),
-}))
-
-const dialogue = vi.hoisted (() => ({
-  confirm: vi.fn (),
-}))
-
-const postEmbed = vi.hoisted (() => ({
-  props: vi.fn (),
-  play:  vi.fn (),
-  seek:  vi.fn (),
-}))
-
-vi.mock ('@/lib/api', () => api)
-vi.mock ('@/lib/posts', () => postsApi)
-vi.mock ('@/components/dialogues/DialogueProvider', () => ({
-  useDialogue: () => dialogue,
-}))
-vi.mock ('@/components/PostEmbed', () => ({
-  default: (props: {
-    ref?: { current: unknown }
-    post: { title: string | null; url: string }
-  }) => {
-    postEmbed.props (props)
-    if (props.ref)
-      props.ref.current = {
-        play: postEmbed.play,
-        seek: postEmbed.seek,
-      }
-
-    return <div>Embed:{props.post.title || props.post.url}</div>
-  },
-}))
-vi.mock ('@/components/PostEditForm', () => ({
-  default: () => <div>Post edit form</div>,
-}))
-vi.mock ('framer-motion', () => ({
-  motion: {
-    aside: ({ children }: { children?: ReactNode }) => <aside>{children}</aside>,
-    div:   ({ children }: { children?: ReactNode }) => <div>{children}</div>,
-    main:  ({ children }: { children?: ReactNode }) => <main>{children}</main>,
-  },
-}))
-
-const currentPost = buildPost ({
-  id:    10,
-  title: '上映中の投稿',
-  url:   'https://www.nicovideo.jp/watch/sm10',
-})
-const theatre = buildTheatre ({ id: 7, name: '上映室' })
-const programme = buildTheatreProgramme ({
-  theatreId: 7,
-  position:  3,
-  post:      currentPost,
-})
-const weights = buildTheatrePostSelectionWeights ({
-  lightestPosts: [{
-    post:    currentPost,
-    penalty: 2,
-    weight:  0.5,
-    tags:    [],
-  }],
-})
-
-const renderPage = (user = buildUser ({ id: 1, role: 'member' })) =>
-  renderWithProviders (
-      <Routes>
-        <Route path="/theatres/:id" element={<TheatreDetailPage user={user}/>}/>
-      </Routes>,
-      { route: '/theatres/7' },
-  )
-
-const mockDefaultApi = () => {
-  api.apiGet.mockImplementation ((path: string) => {
-    switch (path)
-    {
-    case '/theatres/7':
-      return Promise.resolve (theatre)
-
-    case '/theatres/7/comments':
-      return Promise.resolve ([
-        buildTheatreComment ({
-          theatreId: 7,
-          no:        2,
-          user:      { id: 1, name: 'tester' },
-          content:   '視聴コメント',
-        }),
-      ])
-
-    case '/theatres/7/programmes':
-      return Promise.resolve ([programme])
-
-    case '/theatres/7/post_selection_weights':
-      return Promise.resolve (weights)
-
-    default:
-      return Promise.reject (new Error (`Unexpected GET ${ path }`))
-    }
-  })
-
-  api.apiPut.mockImplementation ((path: string) => {
-    switch (path)
-    {
-    case '/theatres/7/watching':
-      return Promise.resolve (buildTheatreInfo ({
-        postId:        currentPost.id,
-        postStartedAt: '2026-01-02T03:04:05.000Z',
-        postElapsedMs: 1_000,
-        watchingUsers: [{ id: 1, name: 'tester' }],
-        skipVote:      {
-          votesCount:         0,
-          requiredCount:      2,
-          watchingUsersCount: 1,
-          voted:              false,
-        },
-      }))
-
-    case '/theatres/7/skip_vote':
-      return Promise.resolve (buildTheatreInfo ({
-        postId:        currentPost.id,
-        postStartedAt: '2026-01-02T03:04:05.000Z',
-        postElapsedMs: 2_000,
-        watchingUsers: [{ id: 1, name: 'tester' }],
-        skipVote:      {
-          votesCount:         1,
-          requiredCount:      2,
-          watchingUsersCount: 1,
-          voted:              true,
-        },
-      }))
-
-    default:
-      return Promise.reject (new Error (`Unexpected PUT ${ path }`))
-    }
-  })
-
-  api.apiDelete.mockResolvedValue (undefined)
-  api.apiPatch.mockResolvedValue (undefined)
-  api.apiPost.mockResolvedValue (undefined)
-  postsApi.fetchPost.mockResolvedValue (currentPost)
-  dialogue.confirm.mockResolvedValue (true)
-}
-
-describe ('TheatreDetailPage', () => {
-  beforeEach (() => {
-    vi.useRealTimers ()
-    vi.clearAllMocks ()
-    mockDefaultApi ()
-  })
-
-  it ('loads theatre state, comments, current post, programme history, and weights', async () => {
-    renderPage ()
-
-    expect (await screen.findByText ('上映会場『上映室』')).toBeInTheDocument ()
-    expect (await screen.findByText ('Embed:上映中の投稿')).toBeInTheDocument ()
-    expect (screen.getAllByText ('視聴コメント')[0]).toBeInTheDocument ()
-    expect (screen.getAllByText ('上映中の投稿')[0]).toBeInTheDocument ()
-    expect (screen.getByText ('penalty 2')).toBeInTheDocument ()
-
-    await waitFor (() => {
-      expect (postsApi.fetchPost).toHaveBeenCalledWith ('10')
-    })
-  })
-
-  it ('votes to skip the current post', async () => {
-    renderPage ()
-
-    await screen.findByText ('Embed:上映中の投稿')
-
-    fireEvent.click (screen.getByRole ('button', { name: 'スキップ 0 / 2' }))
-
-    await waitFor (() => {
-      expect (api.apiPut).toHaveBeenCalledWith (
-          '/theatres/7/skip_vote',
-          { post_id: 10 },
-      )
-    })
-    expect (await screen.findByRole ('button', { name: 'スキップ取消 1 / 2' }))
-      .toBeInTheDocument ()
-  })
-
-  it ('does not seek to zero while applying video length from the player', async () => {
-    api.apiPut.mockImplementation ((path: string) => {
-      switch (path)
-      {
-      case '/theatres/7/watching':
-        return Promise.resolve (buildTheatreInfo ({
-          hostFlg:       true,
-          postId:        currentPost.id,
-          postStartedAt: '2026-01-02T03:04:05.000Z',
-          postElapsedMs: 7_000,
-          watchingUsers: [{ id: 1, name: 'tester' }],
-          skipVote:      {
-            votesCount:         0,
-            requiredCount:      2,
-            watchingUsersCount: 1,
-            voted:              false,
-          },
-        }))
-
-      default:
-        return Promise.reject (new Error (`Unexpected PUT ${ path }`))
-      }
-    })
-
-    renderPage ()
-    await screen.findByText ('Embed:上映中の投稿')
-
-    const props = postEmbed.props.mock.calls.at (-1)![0]
-
-    act (() => {
-      props.onVideoReady (120_000)
-      props.onPlaybackChange (0)
-    })
-
-    const seekMs = postEmbed.seek.mock.calls[0][0]
-    expect (seekMs).toBeGreaterThanOrEqual (7_000)
-    expect (seekMs).toBeLessThan (10_000)
-    expect (postEmbed.seek).not.toHaveBeenCalledWith (0)
-  })
-
-  it ('does not advance host post while video length is unknown', async () => {
-    api.apiPut.mockImplementation ((path: string) => {
-      switch (path)
-      {
-      case '/theatres/7/watching':
-        return Promise.resolve (buildTheatreInfo ({
-          hostFlg:       true,
-          postId:        currentPost.id,
-          postStartedAt: '2026-01-02T03:04:05.000Z',
-          postElapsedMs: 4_000,
-          watchingUsers: [{ id: 1, name: 'tester' }],
-          skipVote:      {
-            votesCount:         0,
-            requiredCount:      2,
-            watchingUsersCount: 1,
-            voted:              false,
-          },
-        }))
-
-      default:
-        return Promise.reject (new Error (`Unexpected PUT ${ path }`))
-      }
-    })
-
-    renderPage ()
-
-    await screen.findByText ('Embed:上映中の投稿')
-    await waitFor (() => {
-      expect (api.apiPut).toHaveBeenCalledWith ('/theatres/7/watching')
-    })
-
-    await waitFor (() => {
-      expect (api.apiPut).toHaveBeenCalledTimes (2)
-    }, { timeout: 2_500 })
-    expect (api.apiPatch).not.toHaveBeenCalledWith ('/theatres/7/next_post')
-  })
-
-  it ('deletes an owned comment after confirmation', async () => {
-    renderPage ()
-
-    fireEvent.click ((await screen.findAllByLabelText ('コメントを削除'))[0])
-
-    await waitFor (() => {
-      expect (dialogue.confirm).toHaveBeenCalled ()
-    })
-    await waitFor (() => {
-      expect (api.apiDelete).toHaveBeenCalledWith ('/theatres/7/comments/2')
-    })
-    expect (await screen.findAllByText ('削除されました．')).toHaveLength (2)
-  })
-})

frontend/src/pages/users/SettingPage.test.tsx

@@ -1,13 +1,12 @@
 import { fireEvent, screen, waitFor } from '@testing-library/react'
-import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import SettingPage from '@/pages/users/SettingPage'
 import { buildUser } from '@/test/factories'
 import { renderWithProviders } from '@/test/render'
 
 const api = vi.hoisted (() => ({
-  apiPut:     vi.fn (),
+  apiPut: vi.fn (),
-  isApiError: vi.fn (),
 }))
 
 const toastApi = vi.hoisted (() => ({
@@ -24,11 +23,6 @@ vi.mock ('@/components/users/InheritDialogue', () => ({
 }))
 
 describe ('SettingPage', () => {
-  beforeEach (() => {
-    vi.clearAllMocks ()
-    api.isApiError.mockReturnValue (false)
-  })
-
   it ('shows loading when user is absent', () => {
     renderWithProviders (<SettingPage user={null} setUser={vi.fn ()}/>)
 
@@ -57,28 +51,4 @@ describe ('SettingPage', () => {
     expect (setUser).toHaveBeenCalled ()
     expect (toastApi.toast).toHaveBeenCalledWith ({ title: '設定を更新しました．' })
   })
-
-  it ('shows validation errors returned for the name field', async () => {
-    const user = buildUser ({ id: 11, name: 'old' })
-    api.isApiError.mockReturnValue (true)
-    api.apiPut.mockRejectedValueOnce ({
-      response: {
-        status: 422,
-        data:   {
-          type:        'validation_error',
-          message:     '入力内容を確認してください．',
-          errors:      { name: ['名前は必須です．'] },
-          base_errors: [],
-        },
-      },
-    })
-
-    renderWithProviders (<SettingPage user={user} setUser={vi.fn ()}/>)
-
-    fireEvent.change (screen.getByRole ('textbox'), { target: { value: '' } })
-    fireEvent.click (screen.getByRole ('button', { name: '更新' }))
-
-    expect (await screen.findByText ('名前は必須です．')).toBeInTheDocument ()
-    expect (screen.getByRole ('textbox')).toHaveAttribute ('aria-invalid', 'true')
-  })
 })

frontend/src/pages/users/SettingPage.tsx

@@ -3,9 +3,7 @@ import type { FC } from 'react'
 import { useEffect, useState } from 'react'
 import { Helmet } from 'react-helmet-async'
 
-import FieldError from '@/components/common/FieldError'
 import Form from '@/components/common/Form'
-import FormField from '@/components/common/FormField'
 import Label from '@/components/common/Label'
 import PageTitle from '@/components/common/PageTitle'
 import MainArea from '@/components/layout/MainArea'
@@ -15,30 +13,22 @@ import { Button } from '@/components/ui/button'
 import { toast } from '@/components/ui/use-toast'
 import { SITE_TITLE } from '@/config'
 import { apiPut } from '@/lib/api'
-import { inputClass } from '@/lib/utils'
-import { useValidationErrors } from '@/lib/useValidationErrors'
 
 import type { User } from '@/types'
 
 type Props = { user:    User | null
 	       setUser: React.Dispatch<React.SetStateAction<User | null>> }
 
-type UserFormField = 'name'
-
 
 const SettingPage: FC<Props> = ({ user, setUser }) => {
   const [name, setName] = useState ('')
   const [userCodeVsbl, setUserCodeVsbl] = useState (false)
   const [inheritVsbl, setInheritVsbl] = useState (false)
-  const { baseErrors, fieldErrors, clearValidationErrors, applyValidationError } =
-      useValidationErrors<UserFormField> ()
 
   const handleSubmit = async () => {
     if (!(user))
       return
 
-    clearValidationErrors ()
-
     const formData = new FormData
     formData.append ('name', name)
 
@@ -50,9 +40,8 @@ const SettingPage: FC<Props> = ({ user, setUser }) => {
       setUser (user => ({ ...user, ...data }))
       toast ({ title: '設定を更新しました．' })
     }
-    catch (e)
+    catch
     {
-      applyValidationError (e)
       toast ({ title: 'しっぱい……' })
     }
   }
@@ -76,16 +65,11 @@ const SettingPage: FC<Props> = ({ user, setUser }) => {
 
 	  {user ? (
 	      <>
-		<FieldError messages={baseErrors}/>
-
 		{/* 名前 */}
-		<FormField label="表示名" messages={fieldErrors.name}>
+		<div>
-		  {({ describedBy, invalid }) => (
+		  <Label>表示名</Label>
-		      <>
 		  <input type="text"
-			 aria-describedby={describedBy}
+			 className="w-full border rounded p-2"
-			 aria-invalid={invalid}
-			 className={inputClass (invalid)}
 			 value={name}
 			 placeholder="名もなきニジラー"
 			 onChange={ev => setName (ev.target.value)}/>
@@ -93,8 +77,7 @@ const SettingPage: FC<Props> = ({ user, setUser }) => {
 		      <p className="mt-1 text-sm text-red-500">
 			名前が未設定のアカウントは 30 日間アクセスしないと削除されます!!!!
 		      </p>)}
-		      </>)}
+		</div>
-		</FormField>
 
 		{/* 送信 */}
 		<Button onClick={handleSubmit}