Skip to content

refactor(DB): Consolidate small SQL files by table#468

Merged
Nyeriah merged 1 commit intomainfrom
improvements
Mar 23, 2026
Merged

refactor(DB): Consolidate small SQL files by table#468
Nyeriah merged 1 commit intomainfrom
improvements

Conversation

@Nyeriah
Copy link
Member

@Nyeriah Nyeriah commented Mar 23, 2026

Summary

  • Consolidate small SQL files (1-5 lines) that target the same database table and share the same bracket prefix into single files per table
  • Reduces total SQL file count by 22 (from 230 to 208) while preserving all SQL content and comments
  • Document SQL file naming conventions and organization rules in README

Brackets affected

Bracket What was merged Files saved
Bracket_0 6 creature files → progression_0_creature.sql 5
Bracket_61_64 2 disables + 3 item_template files 3
Bracket_70_2_1 2 61_64 down files 1
Bracket_70_2_2 2 prefix-0 creature_loot_template files 1
Bracket_70_3_1 5 61_64 down files 4
Bracket_70_4_1 4 61_64 down files + 2 loot files 5
Bracket_70_6_1 3 61_64 item_template down files 2
Bracket_80_1_1 2 creature files 1

Rules followed

  1. Only merged files with the same bracket prefix (never mixed different brackets)
  2. Only merged files targeting the same database table
  3. Kept thematic event files (brewfest, hallows_end, etc.) separate
  4. Kept files over ~50 lines as standalone
  5. Preserved all original comments as section headers

Test plan

  • Verify SQL files load correctly via DBUpdater on server start
  • Confirm no SQL content was lost (all original statements preserved)
  • Check that merged files execute without errors on a clean world database

Note

AI tool (Claude) was used to assist with this PR.

🤖 Generated with Claude Code

Merge small SQL files that target the same database table and share the
same bracket prefix into consolidated files. This reduces file count by
22 (from 230 to 208) while preserving all SQL content and comments.
Document SQL file naming conventions and organization rules in README.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Nyeriah Nyeriah merged commit fe93b0b into main Mar 23, 2026
1 check failed
@Nyeriah Nyeriah deleted the improvements branch March 23, 2026 04:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant