Skip to content

T3 Marty/Spearhead Cost adjustments#7188

Open
vjt4 wants to merge 2 commits into
FAForever:developfrom
vjt4:t3Arty
Open

T3 Marty/Spearhead Cost adjustments#7188
vjt4 wants to merge 2 commits into
FAForever:developfrom
vjt4:t3Arty

Conversation

@vjt4

@vjt4 vjt4 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

#7187

7187.mp4

Description of the proposed changes

TLDR: T3 MArty goes from being 200% mass cost and 179% build time of a Spearhead. To 150% mass cost and build time of a Spearhead.

Reduce T3 MArty cost by 10%.
Reduce T3 MArty BT by 16.3%
Increase Spearhead Cost by 20%.

Numeric Changes

T3 mobile arty:

  • 800->720M
  • 8000->7200E
  • 4300->3600BT

Spearhead:

  • 400->480M
  • 3000->3600E

Both units now have a 18m/s drain from a T3 factory.
T3 MArty completes construction in 40 seconds instead of 47.78 seconds.

Testing done on the proposed changes

Build all the units in a development environment following this tutorial changes applied as expected.

Additional context

https://discord.com/channels/197033481883222026/1525410032924229672
https://discord.com/channels/197033481883222026/1519910763186094150

Checklist

vjt4 added 2 commits July 22, 2026 08:13
800->720m
8000->7200E
4300->3600BT

Spearhead
400->480M
3000->3600E

added snippet
@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

T3 mobile artillery construction costs and build times are reduced, while the T3 mobile missile platform’s mass and energy costs are increased. Blueprint formatting is normalized, and the balance changes are documented in a new changelog snippet.

Changes

T3 mobile unit balance

Layer / File(s) Summary
Unit blueprint economy updates
units/UAL0304/UAL0304_unit.bp, units/UEL0304/UEL0304_unit.bp, units/URL0304/URL0304_unit.bp, units/XEL0306/XEL0306_unit.bp, units/XSL0304/XSL0304_unit.bp
T3 mobile artillery build costs and times are lowered, while the T3 mobile missile platform’s mass and energy costs increase. Audio, rate-of-fire, and closing-brace changes are formatting-only.
Balance changelog entry
changelog/snippets/balance.7187.md
Documents the artillery reductions and missile platform cost increases.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related issues

  • FAForever/fa#7187 — Directly covers the T3 mobile artillery and Spearhead balance changes implemented here.

Suggested labels: area: balance idea, area: unit-blueprint

Suggested reviewers: rhaelya

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly summarizes the main balance changes to T3 Marty and Spearhead costs.
Description check ✅ Passed The description includes the required sections, testing notes, context, and a checklist with the changelog snippet noted.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@changelog/snippets/balance.7187.md`:
- Line 4: Update the changelog snippet’s PR identifier consistently: rename the
`balance.7187.md` snippet to use `7188` and change the referenced number in its
entry from `#7187` to `#7188`, unless the older identifier is intentionally
required.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0d31ccc7-8bba-49df-bd10-e541e3dd29a3

📥 Commits

Reviewing files that changed from the base of the PR and between 46a324c and 0214dfd.

📒 Files selected for processing (6)
  • changelog/snippets/balance.7187.md
  • units/UAL0304/UAL0304_unit.bp
  • units/UEL0304/UEL0304_unit.bp
  • units/URL0304/URL0304_unit.bp
  • units/XEL0306/XEL0306_unit.bp
  • units/XSL0304/XSL0304_unit.bp

{% unit uel0304 %}
T3 Mobile Arty
{% endunit %}
Reduce the cost and build time of all T3 Mobile Arty. (#7187)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Tie the changelog entry to PR #7188.

The reviewed PR is #7188, but this file is named balance.7187.md and line 4 references #7187. Rename the snippet and update the reference, or confirm that the older number is intentional.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@changelog/snippets/balance.7187.md` at line 4, Update the changelog snippet’s
PR identifier consistently: rename the `balance.7187.md` snippet to use `7188`
and change the referenced number in its entry from `#7187` to `#7188`, unless
the older identifier is intentionally required.

@lL1l1 lL1l1 added the area: balance idea related to suggestions for unit balance label Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: balance idea related to suggestions for unit balance

Projects

Status: To Discuss

Development

Successfully merging this pull request may close these issues.

2 participants