mirror/azerothcore-wotlk
1
0
Fork 1
mirror of https://gitcode.com/GitHub_Trending/az/azerothcore-wotlk.git synced 2026-10-10 07:06:38 +08:00
Code Issues Packages Projects Releases Wiki Activity

docs(Agents): add script, SQL and config conventions (#27045)

Browse Source
This commit is contained in:
Andrew
2026-08-08 16:28:19 -03:00
committed by GitHub
parent 2224d479e1
commit dd7da10af2
3 changed files with 10 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.agents/docs/cpp-scripts.md
+4
View File
@@ -14,5 +14,9 @@ Then declare and call `AddSC_<name>()` from the regional loader (`Spells/spells_
**Conventions:**
- Script ids (action/event/data/phase) get named enum entries — never raw literals, even when the file already uses them: add the entry and convert that literal's every call site and handler in the same change.
- A `SpellScript`/`AuraScript` without a matching `spell_script_names` row is inert — ship the binding SQL update in the same change as the C++ registration.
- Never add `UNIT_FLAG*` / `UNIT_FLAG2*` / `UNIT_DYNFLAG*` values without sniff or upstream evidence; the same flag used in another script is not evidence.
- Trigger NPCs (`creature_template.flags_extra` 0x80) have no threat list — `SelectTarget` / `AddThreat` / `UpdateVictim` chains on them silently do nothing. A never-evading helper NPC left on a boss's threatened-by list also stalls the boss's evade/reset forever; make such helpers `IMMUNE_TO_NPC`.
- A spell id missing from Wowhead is inconclusive — check the world DB's `spell_dbc` table (server-side spells) before concluding a sniffed id doesn't exist.
Custom (non-upstream) scripts go in `src/server/scripts/Custom/` (gitignored).