Module: Assistant::CommentManifest

Defined in:
app/services/assistant/comment_manifest.rb

Overview

YAML-backed source of truth for database object/column comments and
first-pass AI safety metadata (e.g. restricted columns).

Constant Summary collapse

MANIFEST_DIR = 
Rails.root.join('db/comments').freeze
SUPPORTED_EXTENSIONS = 
%w[.yml .yaml].freeze

Class Method Summary collapse

Class Method Details

.apply(object_name: nil, dry_run: false, connection: ActiveRecord::Base.connection) ⇒ Object 



128
129
130
131
132
133
134
135
136
 
# File 'app/services/assistant/comment_manifest.rb', line 128

def apply(object_name: nil, dry_run: false, connection: ActiveRecord::Base.connection)
  targets = resolve_targets(object_name)
  statements = targets.flat_map { |target| sql_statements_for(object_name: target, connection: connection) }

  return statements if dry_run

  statements.each { |sql| connection.execute(sql) }
  statements
end

.build_domain_map ⇒ Object

Scan all manifests and build domain_name => Set[object_name] map.



454
455
456
457
458
459
460
461
462
463
 
# File 'app/services/assistant/comment_manifest.rb', line 454

def build_domain_map
  map = Hash.new { |by_domain, k| by_domain[k] = Set.new }

  object_names.each do |name|
    domains = domain_for(name)
    domains.each { |domain| map[domain] << name }
  end

  map
end

.build_relkind_map(connection) ⇒ Object 



427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
 
# File 'app/services/assistant/comment_manifest.rb', line 427

def build_relkind_map(connection)
  sql = <<~SQL
    SELECT c.relname, c.relkind
    FROM pg_class c
    JOIN pg_namespace n ON n.oid = c.relnamespace
    WHERE n.nspname = 'public'
  SQL

  connection.select_all(sql).to_a.each_with_object({}) do |row, out|
    out[row['relname']] = row['relkind']
  end
rescue StandardError => e
  Rails.logger.warn("[Assistant::CommentManifest] Failed to load relkind map: #{e.message}")
  {}
end

.column_comments(relation:, schema_name:, connection:) ⇒ Object 



272
273
274
275
276
277
278
279
280
281
282
283
284
285
 
# File 'app/services/assistant/comment_manifest.rb', line 272

def column_comments(relation:, schema_name:, connection:)
  sql = <<~SQL
    SELECT a.attname AS column_name, d.description AS comment
    FROM pg_class c
    JOIN pg_namespace n ON n.oid = c.relnamespace
    JOIN pg_attribute a ON a.attrelid = c.oid AND a.attnum > 0 AND NOT a.attisdropped
    LEFT JOIN pg_description d ON d.objoid = c.oid AND d.objsubid = a.attnum
    WHERE n.nspname = #{connection.quote(schema_name)}
      AND c.relname = #{connection.quote(relation)}
    ORDER BY a.attnum
  SQL

  connection.select_rows(sql).to_h
end

.column_types(relation, schema_name: 'public', connection: ActiveRecord::Base.connection) ⇒ Hash{String => String}

Read column names and SQL types from pg_attribute + pg_type.

Returns:

  • (Hash{String => String}) —

    { "id" => "bigint", "gl_date" => "date", ... }



393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
 
# File 'app/services/assistant/comment_manifest.rb', line 393

def column_types(relation, schema_name: 'public', connection: ActiveRecord::Base.connection)
  sql = <<~SQL
    SELECT a.attname AS column_name,
           pg_catalog.format_type(a.atttypid, a.atttypmod) AS data_type
    FROM pg_class c
    JOIN pg_namespace n ON n.oid = c.relnamespace
    JOIN pg_attribute a ON a.attrelid = c.oid AND a.attnum > 0 AND NOT a.attisdropped
    WHERE n.nspname = #{connection.quote(schema_name)}
      AND c.relname = #{connection.quote(relation)}
    ORDER BY a.attnum
  SQL

  connection.select_all(sql).to_a.each_with_object({}) do |row, out|
    out[row['column_name']] = row['data_type']
  end
end

.details(object_name, connection: ActiveRecord::Base.connection) ⇒ Hash?

Detailed schema for a specific object, merging pg_attribute types
with YAML manifest comments.

Parameters:

  • object_name (String)
  • connection (Object) (defaults to: ActiveRecord::Base.connection)

Returns:

  • (Hash, nil) —

    { name:, description:, columns:, tips:, type: }



364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
 
# File 'app/services/assistant/comment_manifest.rb', line 364

def details(object_name, connection: ActiveRecord::Base.connection)
  manifest = load_manifest(object_name)
  return nil unless manifest

  schema_name = manifest.fetch('schema', 'public')
  relation = manifest.fetch('object_name', object_name)
  relkind = relation_kind(relation: relation, schema_name: schema_name, connection: connection)
  return nil unless relkind

  types = column_types(relation, schema_name: schema_name, connection: connection)
  manifest_columns = manifest.fetch('columns', {})

  columns = types.each_with_object({}) do |(col_name, col_type), out|
    manifest_cfg = manifest_columns[col_name]
    comment = manifest_cfg.is_a?(Hash) ? manifest_cfg['comment'] : manifest_cfg
    out[col_name] = comment.present? ? "#{col_type} — #{comment}" : col_type
  end

  {
    name: object_name,
    description: manifest['comment'],
    columns: columns,
    tips: manifest['tips'],
    type: relation_type_label(relkind)
  }.compact
end

.diff(object_name: nil, connection: ActiveRecord::Base.connection) ⇒ Object 



138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
 
# File 'app/services/assistant/comment_manifest.rb', line 138

def diff(object_name: nil, connection: ActiveRecord::Base.connection)
  resolve_targets(object_name).flat_map do |target|
    manifest = load_manifest(target)
    next [] unless manifest

    schema_name = manifest.fetch('schema', 'public')
    relation = manifest.fetch('object_name', target)
    relkind = relation_kind(relation: relation, schema_name: schema_name, connection: connection)
    next [{ object: target, issue: 'missing_relation' }] if relkind.nil?

    expected_relation_comment = manifest['comment']
    actual_relation_comment = relation_comment(relation: relation, schema_name: schema_name, connection: connection)

    diffs = []
    if expected_relation_comment != actual_relation_comment
      diffs << {
        object: target,
        type: 'relation_comment',
        expected: expected_relation_comment,
        actual: actual_relation_comment
      }
    end

    expected_columns = manifest.fetch('columns', {})
    actual_columns = column_comments(relation: relation, schema_name: schema_name, connection: connection)

    expected_columns.each do |column_name, config|
      expected_comment = normalize_column_config(config)['comment']
      actual_comment = actual_columns[column_name.to_s]
      next if expected_comment == actual_comment

      diffs << {
        object: target,
        type: 'column_comment',
        column: column_name.to_s,
        expected: expected_comment,
        actual: actual_comment
      }
    end
    diffs
  end
end

.domain_for(object_name) ⇒ Object

Returns the domain(s) declared for an object as an Array of strings.
Returns [] if no domain is declared (admin-only).



43
44
45
46
47
48
49
50
51
 
# File 'app/services/assistant/comment_manifest.rb', line 43

def domain_for(object_name)
  manifest = load_manifest(object_name)
  return [] unless manifest

  raw = manifest['domain']
  return [] if raw.nil?

  Array(raw).map(&:to_s)
end

.domain_map ⇒ Object

Build a cached mapping of domain_name => Set[object_names].
Scans all manifests once; cached in production, re-read in dev/test.



68
69
70
71
72
73
74
 
# File 'app/services/assistant/comment_manifest.rb', line 68

def domain_map
  if Rails.env.development? || Rails.env.test?
    build_domain_map
  else
    @domain_map ||= build_domain_map
  end
end

.dump_model_tables(output_dir: MANIFEST_DIR, connection: ActiveRecord::Base.connection, object_name: nil) ⇒ Object 



181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
 
# File 'app/services/assistant/comment_manifest.rb', line 181

def dump_model_tables(output_dir: MANIFEST_DIR, connection: ActiveRecord::Base.connection, object_name: nil)
  Rails.application.eager_load!

  table_names = if object_name.present?
                  [object_name.to_s]
                else
                  ApplicationRecord.descendants
                                   .reject(&:abstract_class?)
                                   .map(&:table_name)
                                   .compact
                                   .map(&:to_s)
                                   .uniq
                                   .sort
                end

  FileUtils.mkdir_p(output_dir)

  table_names.each do |table_name|
    next unless relation_kind(relation: table_name, schema_name: 'public', connection: connection)

    payload = {
      'schema' => 'public',
      'object_name' => table_name,
      'comment' => relation_comment(relation: table_name, schema_name: 'public', connection: connection),
      'columns' => column_comments(relation: table_name, schema_name: 'public', connection: connection).to_h do |column_name, comment|
        [column_name, { 'comment' => comment, 'restricted' => false }]
      end
    }

    File.write(output_dir.join("#{table_name}.yml"), payload.to_yaml)
  end
end

.load_manifest(object_name) ⇒ Object 



238
239
240
241
242
243
 
# File 'app/services/assistant/comment_manifest.rb', line 238

def load_manifest(object_name)
  path = manifest_path(object_name)
  return nil unless path

  YAML.safe_load_file(path.to_s) || {}
end

.manifest_exists?(object_name) ⇒ Boolean

Returns:

  • (Boolean) 


15
16
17
 
# File 'app/services/assistant/comment_manifest.rb', line 15

def manifest_exists?(object_name)
  manifest_path(object_name).present?
end

.manifest_path(object_name) ⇒ Object 



324
325
326
327
328
329
330
331
 
# File 'app/services/assistant/comment_manifest.rb', line 324

def manifest_path(object_name)
  stem = object_name.to_s.downcase
  SUPPORTED_EXTENSIONS.each do |ext|
    candidate = MANIFEST_DIR.join("#{stem}#{ext}")
    return candidate if File.exist?(candidate)
  end
  nil
end

.normalize_column_config(config) ⇒ Object 



305
306
307
308
309
310
311
312
 
# File 'app/services/assistant/comment_manifest.rb', line 305

def normalize_column_config(config)
  case config
  when Hash
    config.transform_keys(&:to_s)
  else
    { 'comment' => config, 'restricted' => false }
  end
end

.object_names ⇒ Object 



19
20
21
22
23
 
# File 'app/services/assistant/comment_manifest.rb', line 19

def object_names
  Dir.glob(MANIFEST_DIR.join('*.y{a,}ml').to_s).map do |path|
    File.basename(path, File.extname(path)).downcase
  end.sort
end

.objects_for_domain(domain_name) ⇒ Object

Returns a Set of object names that declare the given domain.



54
55
56
 
# File 'app/services/assistant/comment_manifest.rb', line 54

def objects_for_domain(domain_name)
  domain_map[domain_name.to_s] || Set.new
end

.objects_for_domains(domain_names) ⇒ Object

Returns a Set of object names that declare ANY of the given domains (union).



59
60
61
62
63
64
 
# File 'app/services/assistant/comment_manifest.rb', line 59

def objects_for_domains(domain_names)
  map = domain_map
  Array(domain_names).each_with_object(Set.new) do |name, set|
    set.merge(map[name.to_s] || [])
  end
end

.qualified_relation_identifier(schema_name:, relation:, connection:) ⇒ Object 



320
321
322
 
# File 'app/services/assistant/comment_manifest.rb', line 320

def qualified_relation_identifier(schema_name:, relation:, connection:)
  "#{connection.quote_table_name(schema_name)}.#{connection.quote_table_name(relation)}"
end

.relation_comment(relation:, schema_name:, connection:) ⇒ Object 



258
259
260
261
262
263
264
265
266
267
268
269
270
 
# File 'app/services/assistant/comment_manifest.rb', line 258

def relation_comment(relation:, schema_name:, connection:)
  sql = <<~SQL
    SELECT d.description
    FROM pg_class c
    JOIN pg_namespace n ON n.oid = c.relnamespace
    LEFT JOIN pg_description d ON d.objoid = c.oid AND d.objsubid = 0
    WHERE n.nspname = #{connection.quote(schema_name)}
      AND c.relname = #{connection.quote(relation)}
    LIMIT 1
  SQL

  connection.select_value(sql)
end

.relation_keyword_for(relkind) ⇒ Object 



296
297
298
299
300
301
302
303
 
# File 'app/services/assistant/comment_manifest.rb', line 296

def relation_keyword_for(relkind)
  case relkind
  when 'm' then 'MATERIALIZED VIEW'
  when 'v' then 'VIEW'
  when 'f' then 'FOREIGN TABLE'
  else 'TABLE'
  end
end

.relation_kind(relation:, schema_name:, connection:) ⇒ Object 



245
246
247
248
249
250
251
252
253
254
255
256
 
# File 'app/services/assistant/comment_manifest.rb', line 245

def relation_kind(relation:, schema_name:, connection:)
  sql = <<~SQL
    SELECT c.relkind
    FROM pg_class c
    JOIN pg_namespace n ON n.oid = c.relnamespace
    WHERE n.nspname = #{connection.quote(schema_name)}
      AND c.relname = #{connection.quote(relation)}
    LIMIT 1
  SQL

  connection.select_value(sql)
end

.relation_type_label(relkind) ⇒ Object

Convert pg_class.relkind to a human-readable type label.



444
445
446
447
448
449
450
451
 
# File 'app/services/assistant/comment_manifest.rb', line 444

def relation_type_label(relkind)
  case relkind
  when 'm' then 'materialized_view'
  when 'v' then 'view'
  when 'f' then 'foreign_table'
  else 'table'
  end
end

.relkind_map(connection: ActiveRecord::Base.connection) ⇒ Object

Batch-load relkind for all public relations in one query.
Cached in production, fresh in dev/test.



419
420
421
422
423
424
425
 
# File 'app/services/assistant/comment_manifest.rb', line 419

def relkind_map(connection: ActiveRecord::Base.connection)
  if Rails.env.development? || Rails.env.test?
    build_relkind_map(connection)
  else
    @relkind_map ||= build_relkind_map(connection)
  end
end

.reset_domain_map! ⇒ Object 



76
77
78
 
# File 'app/services/assistant/comment_manifest.rb', line 76

def reset_domain_map!
  @domain_map = nil
end

.reset_schema_cache! ⇒ Object

Reset cached schema introspection data (for tests/console).



411
412
413
 
# File 'app/services/assistant/comment_manifest.rb', line 411

def reset_schema_cache!
  @relkind_map = nil
end

.resolve_targets(object_name) ⇒ Object

Raises:

  • (ArgumentError) 


287
288
289
290
291
292
293
294
 
# File 'app/services/assistant/comment_manifest.rb', line 287

def resolve_targets(object_name)
  return object_names if object_name.blank?

  target = object_name.to_s.downcase
  raise ArgumentError, "No manifest found for #{target}" unless manifest_exists?(target)

  [target]
end

.restricted_columns_for_objects(object_names:) ⇒ Object 



25
26
27
28
29
30
31
32
33
34
35
 
# File 'app/services/assistant/comment_manifest.rb', line 25

def restricted_columns_for_objects(object_names:)
  Array(object_names).flat_map do |object_name|
    manifest = load_manifest(object_name)
    next [] unless manifest

    manifest.fetch('columns', {}).filter_map do |column_name, config|
      cfg = normalize_column_config(config)
      column_name.to_s.downcase if cfg['restricted'] == true
    end
  end.uniq
end

.sql_statements_for(object_name:, connection: ActiveRecord::Base.connection) ⇒ Object 



214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
 
# File 'app/services/assistant/comment_manifest.rb', line 214

def sql_statements_for(object_name:, connection: ActiveRecord::Base.connection)
  manifest = load_manifest(object_name)
  return [] unless manifest

  schema_name = manifest.fetch('schema', 'public')
  relation = manifest.fetch('object_name', object_name)
  relkind = relation_kind(relation: relation, schema_name: schema_name, connection: connection)
  return [] if relkind.nil?

  relation_identifier = qualified_relation_identifier(schema_name: schema_name, relation: relation, connection: connection)
  relation_keyword = relation_keyword_for(relkind)

  statements = []
  statements << "COMMENT ON #{relation_keyword} #{relation_identifier} IS #{to_sql_comment(manifest['comment'], connection)};"

  manifest.fetch('columns', {}).each do |column_name, config|
    comment = normalize_column_config(config)['comment']
    column_identifier = connection.quote_column_name(column_name.to_s)
    statements << "COMMENT ON COLUMN #{relation_identifier}.#{column_identifier} IS #{to_sql_comment(comment, connection)};"
  end

  statements
end

.summary(connection: ActiveRecord::Base.connection) ⇒ Array<Hash>

Compact summary of all manifest-backed objects.

Returns:

  • (Array<Hash>) —

    [{ name:, description:, column_count:, type:, domain: }]



339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
 
# File 'app/services/assistant/comment_manifest.rb', line 339

def summary(connection: ActiveRecord::Base.connection)
  relkinds = relkind_map(connection: connection)

  object_names.filter_map do |name|
    manifest = load_manifest(name)
    next unless manifest

    relkind = relkinds[manifest.fetch('object_name', name)]
    next unless relkind # object doesn't exist in the database

    {
      name: name,
      description: manifest['comment'],
      column_count: manifest.fetch('columns', {}).size,
      type: relation_type_label(relkind),
      domain: Array(manifest['domain'])
    }
  end
end

.sync(object_name: nil, connection: ActiveRecord::Base.connection) ⇒ Hash

Synchronize YAML manifests with actual database columns.
Adds new columns (with comment: nil) and removes columns no longer in the DB.
Re-orders YAML columns to match the actual database column order.

Parameters:

  • object_name (String, nil) (defaults to: nil) —

    Specific object to sync, or nil for all

  • connection (Object) (defaults to: ActiveRecord::Base.connection)

Returns:

  • (Hash) —

    { added: { name => [cols] }, removed: { name => [cols] }, skipped: [names] }



86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
 
# File 'app/services/assistant/comment_manifest.rb', line 86

def sync(object_name: nil, connection: ActiveRecord::Base.connection)
  targets = object_name ? [object_name.to_s.downcase] : object_names
  changes = { added: {}, removed: {}, skipped: [] }

  targets.each do |name|
    manifest = load_manifest(name)
    next unless manifest

    schema_name = manifest.fetch('schema', 'public')
    relation = manifest.fetch('object_name', name)
    relkind = relation_kind(relation: relation, schema_name: schema_name, connection: connection)

    unless relkind
      changes[:skipped] << name
      next
    end

    db_columns = column_types(relation, schema_name: schema_name, connection: connection)
    manifest_columns = manifest.fetch('columns', {})

    new_cols = db_columns.keys - manifest_columns.keys
    removed_cols = manifest_columns.keys - db_columns.keys

    next if new_cols.empty? && removed_cols.empty?

    # Rebuild columns hash in DB column order, preserving existing config
    ordered_columns = {}
    db_columns.each_key do |col|
      ordered_columns[col] = manifest_columns.fetch(col, { 'comment' => nil })
    end

    manifest['columns'] = ordered_columns
    path = manifest_path(name)
    File.write(path, manifest.to_yaml)

    changes[:added][name] = new_cols unless new_cols.empty?
    changes[:removed][name] = removed_cols unless removed_cols.empty?
  end

  changes
end

.to_sql_comment(value, connection) ⇒ Object 



314
315
316
317
318
 
# File 'app/services/assistant/comment_manifest.rb', line 314

def to_sql_comment(value, connection)
  return 'NULL' if value.nil?

  connection.quote(value.to_s)
end