Skip to content

Allow docking/building of mercies on carriers and air staging.#7135

Open
Nuggets75 wants to merge 8 commits into
FAForever:developfrom
Nuggets75:mercy-carrier-docking
Open

Allow docking/building of mercies on carriers and air staging.#7135
Nuggets75 wants to merge 8 commits into
FAForever:developfrom
Nuggets75:mercy-carrier-docking

Conversation

@Nuggets75

@Nuggets75 Nuggets75 commented Jun 5, 2026

Copy link
Copy Markdown

Intended Change
Remove the CANNOTUSEAIRSTAGING category from the Mercy (DAA0206) so it can use air staging facilities and aircraft carriers, like most other air units.

Reasoning
Since the mercy rework, the mercy is now an anti t1/t2 army / structure / buildpower unit and no longer a snipe-unit so I see no reason to lock it out of carriers. I believe the main reason it was locked out was to limit their attack range with low amount of fuel before the rework. Now the mercy, while good, has an incredibly small window of opportunity that can be expanded with this change.

Expected behavior
The Mercy can now land on air staging and dock on / be built by aircraft carriers. No change to its health, damage, cost, build time, or any other stat.

Summary by CodeRabbit

  • Gameplay Changes
    • Updated air-staging interaction rules: this unit can now use air-staging right-click interactions.
    • Expanded command capabilities to include transport/docking/guard/patrol.
    • Increased fuel recharge rate.
    • Improved primary weapon target evaluation with more frequent re-checks and a larger tracking radius.
  • Documentation
    • Added an explanatory note to engine category documentation regarding air-staging behavior.

@coderabbitai

coderabbitai Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e8eaa5ab-e747-432d-9e03-6aa498af5a80

📥 Commits

Reviewing files that changed from the base of the PR and between 7dfce09 and e154d7a.

📒 Files selected for processing (1)
  • units/DAA0206/DAA0206_unit.bp

📝 Walkthrough

Walkthrough

Updates DAA0206 category restrictions, command capabilities, fuel recharge, and primary-weapon target evaluation. Adds an engine category comment and adjusts blueprint termination formatting.

Changes

DAA0206 unit updates

Layer / File(s) Summary
Categories removal and formatting
units/DAA0206/DAA0206_unit.bp, engine/Core/Categories.lua
Removes "CANNOTUSEAIRSTAGING", documents the related category behavior, and adjusts closing brace formatting.
Command capability additions
units/DAA0206/DAA0206_unit.bp
Adds transport, dock, guard, and patrol command capabilities.
Physics fuel recharge
units/DAA0206/DAA0206_unit.bp
Sets Physics.FuelRechargeRate from 0 to 5.
Weapon target re-evaluation
units/DAA0206/DAA0206_unit.bp
Enables AlwaysRecheckTarget, changes TargetCheckInterval to 1, and increases TrackingRadius from 0.5 to 1.1.

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

Possibly related PRs

  • FAForever/fa#7175: Modifies the same DAA0206 blueprint’s weapon damage, radius, and damage-over-time parameters.

Suggested reviewers: rhaelya

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: enabling Mercy docking/building on carriers and air staging.
Description check ✅ Passed The description covers the change, rationale, and expected behavior, but it omits the template's testing and checklist sections.
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.
✨ 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.

@lL1l1 lL1l1 added the area: balance idea related to suggestions for unit balance label Jun 9, 2026
lL1l1 added 6 commits June 12, 2026 13:15
needed for carrier deploy command to work
Tracking radius 0.5 -> 1.25 (bomber default)
This doesn't work because of a bug in target priorities causing parentheses-using expressions to not be parsed properly, thus the unit does not have ALLUNITS in priorities so it will not target units that aren't T1-T3 structures. Set the priorities by hotkey to get them to work.
@lL1l1

lL1l1 commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

I tested it and found that nothing actually worked. I implemented the proposed changes of docking in air staging and building in carriers, but this necessarily comes with the side effect of mercy recharging fuel while on the ground. It's the standard 1s fuel per 2s landed time since that rate is multiplied by air staging when refueling, and the unit needs to refuel fully to leave air staging, so a low base refuel rate is unacceptable. I also needed to add the guard order for mercies to be able to leave carriers using the deploy command, which issues a guard (assist) command on the ground where you click the deploy order.

Since it can now refuel, I thought it made sense to add the patrol ability.

When I was testing patrol, I discovered that it wasn't working because of a long target check interval, low tracking radius, and missing AlwaysRecheckTarget = true, so I fixed all of that. This also fixes the attack behavior on guard and attack move commands, so it is quite necessary for those commands, not just patrol.

so it doesn't suicide so easily to random units passing by. Enough radius to outrange ilshavoh (26 vs 27.5).
@Garanas

Garanas commented Jul 14, 2026

Copy link
Copy Markdown
Member

When I was testing patrol, I discovered that it wasn't working because of a long target check interval, low tracking radius, and missing AlwaysRecheckTarget = true, so I fixed all of that. This also fixes the attack behavior on guard and attack move commands, so it is quite necessary for those commands, not just patrol.

This was not a bug as far as I am aware. These parameters were intentional so that the Mercy does not trigger on a tech 1 scout that happens to walk by. This was, as far as I am aware, by design 😃 .

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.

3 participants