-ized
Usually I don't mind whether people use American or British spelling in comments, but using both in the same sentence is ugly.

which

let?

Shouldn't it be 2 * Math.PI?

idem

Aren't comments typically on a new line? (Also below.)

hold

idem

What they meant here was probably something like: "If yes, this holds (...)". I'm not even sure these comments (and below) are needed, because it seems obvious from the code, but that's not up to me ;)

This calculation seems to get the radius so 0.5 is not really a magic value, no?
It would be the same by dividing by 2.

Transltable strings are in en-US, code had been changed from british english to US english too, for example in the colour -> color commits. And Amour had always been planned to be renamed to Armor.

(Objects storing positions should be use the Vector prototype, so that one can use these algebraic operations. If it's globally consistent then these operations are available by default and new code would also be introduced with that feature.)

(A bit noisy)

Why brackets for one line ?
see similar solution at L1427

I think this one can be nuked
same for L1019 L1023

This should be obvious from next lines

This looks not needed same L1427

this should end with ,

make it consistent on function level, if count of both occurrences is even, make it consistent on file level :)

2 * Pi ? (spaces)

Could merge the two statements, and ternary for cmpUnit ai like so:

ret.attack[type].elevationAdaptedRange = cmpRangeManager.GetElevationAdaptedRange(cmpPosition.GetPosition(), cmpPosition.GetRotation(), range.max, range.elevationBonus, cmpUnitAI ? 0 : 2 * Math.PI);

One might also wonder why not checking for buildingAI.

Is it still worth creating new objects? I still like having vectors though.

Wonder if we couldn't just pass pos to cpp

Three patches modifying that line (this one, sockets, restoring turrets)

Better keep the brackets for this type of expression, otherwise diff seems OK.