mirror of
https://gh.wpcy.net/https://github.com/discourse/discourse.git
synced 2026-06-19 03:23:50 +08:00
`.annotaterb.yml` has carried `classified_sort: true` since the project
switched from `annotate` to `annotaterb` (commit 0eab7daea4, July 2025),
but annotaterb's default behaviour is to compare the existing schema
block against what it would generate and skip the rewrite when the
column list matches — even when the *ordering* of those columns differs.
The result is that models which haven't had a schema change since the
config landed never get reordered, and `classified_sort` drift
accumulates indefinitely.
`--force` makes annotaterb always rewrite, so a single `bin/rake
annotate:clean` run brings every model into the canonical format and
keeps them there. Every schema block is now grouped primary-key →
regular columns → timestamps → foreign keys (alphabetical within each
group). Pure annotation comment change — no code modifications.
Also cleans up the rake task to avoid string interpolation for `system`
calls.
56 lines
1.6 KiB
Ruby
Vendored
56 lines
1.6 KiB
Ruby
Vendored
# frozen_string_literal: true
|
|
|
|
class AiArtifactKeyValue < ActiveRecord::Base
|
|
belongs_to :ai_artifact
|
|
belongs_to :user
|
|
|
|
validates :key, presence: true, length: { maximum: 50 }
|
|
validates :value,
|
|
presence: true,
|
|
length: {
|
|
maximum: ->(_) { SiteSetting.ai_artifact_kv_value_max_length },
|
|
}
|
|
attribute :public, :boolean, default: false
|
|
validates :ai_artifact, presence: true
|
|
validates :user, presence: true
|
|
validates :key, uniqueness: { scope: %i[ai_artifact_id user_id] }
|
|
|
|
validate :validate_max_keys_per_user_per_artifact
|
|
|
|
private
|
|
|
|
def validate_max_keys_per_user_per_artifact
|
|
return unless ai_artifact_id && user_id
|
|
|
|
max_keys = SiteSetting.ai_artifact_max_keys_per_user_per_artifact
|
|
existing_count = self.class.where(ai_artifact_id: ai_artifact_id, user_id: user_id).count
|
|
|
|
# Don't count the current record if it's being updated
|
|
existing_count -= 1 if persisted?
|
|
|
|
if existing_count >= max_keys
|
|
errors.add(
|
|
:base,
|
|
I18n.t("discourse_ai.ai_artifact.errors.max_keys_exceeded", count: max_keys),
|
|
)
|
|
end
|
|
end
|
|
end
|
|
|
|
# == Schema Information
|
|
#
|
|
# Table name: ai_artifact_key_values
|
|
#
|
|
# id :bigint not null, primary key
|
|
# key :string(50) not null
|
|
# public :boolean default(FALSE), not null
|
|
# value :string(20000) not null
|
|
# created_at :datetime not null
|
|
# updated_at :datetime not null
|
|
# ai_artifact_id :bigint not null
|
|
# user_id :integer not null
|
|
#
|
|
# Indexes
|
|
#
|
|
# index_ai_artifact_kv_unique (ai_artifact_id,user_id,key) UNIQUE
|
|
#
|