refactor: disposition alter functions to return strings

[OH296]

---

Summary by cubic

Refactored disposition helpers to optionally return formatted messages and centralized bounds so dispositions stay within 0–100.

• Refactors

  • alter_disposition(faction, value, return_string=false) and alter_dispositions(alterations, return_strings) can return strings ("Faction Name : +/-N"); faction_disp_mods now includes strings: [].
  • Bounds are enforced in calc_final_disp_value to keep results in 0–100; alter_disposition only applies the adjusted delta.

• Migration

  • Update any code that expected dispositions in -100..100 or relied on alter_disposition clamping. Use the return_string(s) flags if you need messages.

^{Written for commit 60558ab. Summary will update on new commits.}

[cubic-dev-ai]

3 issues found across 2 files

Confidence score: 2/5

• In scripts/scr_ChapterTraits/scr_ChapterTraits.gml, array_create(14, struct) makes all faction_disp_mods entries point to the same struct, so merge_mods can overwrite every faction with identical int_mod/mult values and cause broad gameplay regressions — initialize each array slot with a distinct struct before merging.
• In scripts/scr_diplomacy_helpers/scr_diplomacy_helpers.gml, a string template is missing the $ prefix, so {} placeholders will render literally and show broken text to players — add the $ so interpolation occurs as intended.
• In scripts/scr_ChapterTraits/scr_ChapterTraits.gml, the new strings field is currently unused and not merged, which adds confusing dead state (and currently inherits the same shared-reference risk) — either wire it into merge/use paths or remove it in this PR to reduce follow-on bugs.

<sub>Shadow auto-approve: would not auto-approve because issues were found.

Re-trigger cubic</sub>

[cubic-dev-ai]

0 issues found across 1 file (changes from recent commits).

<sub>Shadow auto-approve: would require human review. Refactors core disposition logic, altering bounds from -100..100 to 0..100 and adding string return capabilities. Even though it's a single file, this changes how disposition values are calculated and stored, which could break any code relying on the old range or behavior.

Re-trigger cubic</sub>

[github-actions]

This PR is marked as stale, because it has been open for 7 days with no activity.

[cubic-dev-ai]

No issues found across 2 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Re-trigger cubic}

[EttyKitty]

I've fixed an error on main with alter_disposition calls that were trying to use nonexistent return.
When this PR is merged, the alter_disposition call sites should be updated to use the new return logic and put the strings to use wherever needed.