Merge pull request #10385 from pchote/d2k-worm-trail

Finish D2K worm implementation

OpenRA.Game/Effects/SpriteEffect.cs

@@ -20,13 +20,13 @@ namespace OpenRA.Effects
 		readonly WPos pos;
 		readonly bool scaleSizeWithZoom;
 
-		public SpriteEffect(WPos pos, World world, string image, string palette, bool scaleSizeWithZoom = false)
+		public SpriteEffect(WPos pos, World world, string image, string sequence, string palette, bool scaleSizeWithZoom = false)
 		{
 			this.pos = pos;
 			this.palette = palette;
 			this.scaleSizeWithZoom = scaleSizeWithZoom;
 			anim = new Animation(world, image);
-			anim.PlayThen("idle", () => world.AddFrameEndTask(w => w.Remove(this)));
+			anim.PlayThen(sequence, () => world.AddFrameEndTask(w => w.Remove(this)));
 		}
 
 		public void Tick(World world)

OpenRA.Game/Widgets/WorldInteractionControllerWidget.cs

@@ -214,7 +214,7 @@ namespace OpenRA.Widgets
 					else if (o.TargetLocation != CPos.Zero)
 					{
 						var pos = world.Map.CenterOfCell(cell);
-						world.AddFrameEndTask(w => w.Add(new SpriteEffect(pos, world, "moveflsh", "moveflash", true)));
+						world.AddFrameEndTask(w => w.Add(new SpriteEffect(pos, world, "moveflsh", "idle", "moveflash", true)));
 						flashed = true;
 					}
 				}

OpenRA.Mods.Common/OpenRA.Mods.Common.csproj

@@ -421,7 +421,6 @@
     <Compile Include="Traits\Render\TimedUpgradeBar.cs" />
     <Compile Include="Traits\Render\WithSpriteBarrel.cs" />
     <Compile Include="Traits\Render\WithBuildingExplosion.cs" />
-    <Compile Include="Traits\Render\WithActiveAnimation.cs" />
     <Compile Include="Traits\Render\WithAttackAnimation.cs" />
     <Compile Include="Traits\Render\WithMoveAnimation.cs" />
     <Compile Include="Traits\Render\WithSiloAnimation.cs" />
@@ -718,6 +717,7 @@
     <Compile Include="EditorBrushes\EditorCopyPasteBrush.cs" />
     <Compile Include="Traits\World\EditorSelectionLayer.cs" />
     <Compile Include="Graphics\DetectionCircleRenderable.cs" />
+    <Compile Include="Traits\Render\WithIdleAnimation.cs" />
   </ItemGroup>
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
   <PropertyGroup>

OpenRA.Mods.Common/Traits/Render/LeavesTrails.cs

@@ -18,9 +18,13 @@ namespace OpenRA.Mods.Common.Traits
 	public enum TrailType { Cell, CenterPosition }
 
 	[Desc("Renders a sprite effect when leaving a cell.")]
-	public class LeavesTrailsInfo : ITraitInfo
+	public class LeavesTrailsInfo : UpgradableTraitInfo
 	{
 		public readonly string Image = null;
+
+		[SequenceReference("Image")]
+		public readonly string[] Sequences = { "idle" };
+
 		[PaletteReference] public readonly string Palette = "effect";
 
 		[Desc("Only do so when the terrain types match with the previous cell.")]
@@ -42,43 +46,47 @@ namespace OpenRA.Mods.Common.Traits
 		[Desc("Delay between trail updates when moving.")]
 		public readonly int MovingInterval = 0;
 
-		public object Create(ActorInitializer init) { return new LeavesTrails(this, init.Self); }
+		public override object Create(ActorInitializer init) { return new LeavesTrails(init.Self, this); }
 	}
 
-	public class LeavesTrails : ITick
+	public class LeavesTrails : UpgradableTrait<LeavesTrailsInfo>, ITick
 	{
-		readonly LeavesTrailsInfo info;
+		public LeavesTrails(Actor self, LeavesTrailsInfo info)
-
+			: base(info) { }
-		public LeavesTrails(LeavesTrailsInfo info, Actor self)
-		{
-			this.info = info;
-		}
 
 		WPos cachedPosition;
 		int ticks;
 
 		public void Tick(Actor self)
 		{
-			var isMoving = self.CenterPosition != cachedPosition;
+			if (IsTraitDisabled)
-			if ((isMoving && !info.TrailWhileMoving) || (!isMoving && !info.TrailWhileStationary))
 				return;
 
-			var interval = isMoving ? info.MovingInterval :
+			var isMoving = self.CenterPosition != cachedPosition;
-				info.StationaryInterval;
+			if ((isMoving && !Info.TrailWhileMoving) || (!isMoving && !Info.TrailWhileStationary))
+				return;
+
+			var interval = isMoving ? Info.MovingInterval : Info.StationaryInterval;
 			if (++ticks >= interval)
 			{
 				var cachedCell = self.World.Map.CellContaining(cachedPosition);
 				var type = self.World.Map.GetTerrainInfo(cachedCell).Type;
 
-				var pos = info.Type == TrailType.CenterPosition ? cachedPosition :
+				var pos = Info.Type == TrailType.CenterPosition ? cachedPosition :
 					self.World.Map.CenterOfCell(cachedCell);
 
-				if (info.TerrainTypes.Contains(type) && !string.IsNullOrEmpty(info.Image))
+				if (Info.TerrainTypes.Contains(type) && !string.IsNullOrEmpty(Info.Image))
-					self.World.AddFrameEndTask(w => w.Add(new SpriteEffect(pos, self.World, info.Image, info.Palette)));
+					self.World.AddFrameEndTask(w => w.Add(new SpriteEffect(pos, self.World, Info.Image,
+						Info.Sequences.Random(Game.CosmeticRandom), Info.Palette)));
 
 				cachedPosition = self.CenterPosition;
 				ticks = 0;
 			}
 		}
+
+		protected override void UpgradeEnabled(Actor self)
+		{
+			cachedPosition = self.CenterPosition;
+		}
 	}
 }

OpenRA.Mods.Common/Traits/Render/WithActiveAnimation.cs

@@ -1,70 +0,0 @@
-#region Copyright & License Information
-/*
- * Copyright 2007-2015 The OpenRA Developers (see AUTHORS)
- * This file is part of OpenRA, which is free software. It is made
- * available to you under the terms of the GNU General Public License
- * as published by the Free Software Foundation. For more information,
- * see COPYING.
- */
-#endregion
-
-using System.Collections.Generic;
-using System.Linq;
-using OpenRA.Graphics;
-using OpenRA.Traits;
-
-namespace OpenRA.Mods.Common.Traits
-{
-	[Desc("Replaces the idle animation of a building.")]
-	public class WithActiveAnimationInfo : ITraitInfo, Requires<WithSpriteBodyInfo>
-	{
-		[Desc("Sequence name to use")]
-		[SequenceReference] public readonly string Sequence = "active";
-
-		public readonly int Interval = 750;
-
-		public readonly bool PauseOnLowPower = false;
-
-		public object Create(ActorInitializer init) { return new WithActiveAnimation(init.Self, this); }
-	}
-
-	public class WithActiveAnimation : ITick, INotifyBuildComplete, INotifySold
-	{
-		readonly WithActiveAnimationInfo info;
-		readonly WithSpriteBody wsb;
-
-		public WithActiveAnimation(Actor self, WithActiveAnimationInfo info)
-		{
-			wsb = self.Trait<WithSpriteBody>();
-			this.info = info;
-		}
-
-		int ticks;
-		public void Tick(Actor self)
-		{
-			if (!buildComplete)
-				return;
-
-			if (--ticks <= 0)
-			{
-				if (!(info.PauseOnLowPower && self.IsDisabled()))
-					wsb.PlayCustomAnimation(self, info.Sequence, () => wsb.CancelCustomAnimation(self));
-				ticks = info.Interval;
-			}
-		}
-
-		bool buildComplete = false;
-
-		public void BuildingComplete(Actor self)
-		{
-			buildComplete = true;
-		}
-
-		public void Selling(Actor self)
-		{
-			buildComplete = false;
-		}
-
-		public void Sold(Actor self) { }
-	}
-}

OpenRA.Mods.Common/Traits/Render/WithIdleAnimation.cs

@@ -0,0 +1,71 @@
+#region Copyright & License Information
+/*
+ * Copyright 2007-2015 The OpenRA Developers (see AUTHORS)
+ * This file is part of OpenRA, which is free software. It is made
+ * available to you under the terms of the GNU General Public License
+ * as published by the Free Software Foundation. For more information,
+ * see COPYING.
+ */
+#endregion
+
+using System.Collections.Generic;
+using System.Linq;
+using OpenRA.Graphics;
+using OpenRA.Traits;
+
+namespace OpenRA.Mods.Common.Traits
+{
+	[Desc("Periodically plays an idle animation, replacing the default body animation.")]
+	public class WithIdleAnimationInfo : UpgradableTraitInfo, Requires<WithSpriteBodyInfo>
+	{
+		[SequenceReference, Desc("Sequence names to use.")]
+		public readonly string[] Sequences = { "active" };
+
+		public readonly int Interval = 750;
+
+		[Desc("Pause when the actor is disabled.  Deprecated.  Use upgrades instead.")]
+		public readonly bool PauseOnLowPower = false;
+
+		public override object Create(ActorInitializer init) { return new WithIdleAnimation(init.Self, this); }
+	}
+
+	public class WithIdleAnimation : UpgradableTrait<WithIdleAnimationInfo>, ITick, INotifyBuildComplete, INotifySold
+	{
+		readonly WithSpriteBody wsb;
+		bool buildComplete;
+		int ticks;
+
+		public WithIdleAnimation(Actor self, WithIdleAnimationInfo info)
+			: base(info)
+		{
+			wsb = self.Trait<WithSpriteBody>();
+			buildComplete = !self.Info.HasTraitInfo<BuildingInfo>(); // always render instantly for units
+			ticks = info.Interval;
+		}
+
+		public void Tick(Actor self)
+		{
+			if (!buildComplete || IsTraitDisabled)
+				return;
+
+			if (--ticks <= 0)
+			{
+				if (!(Info.PauseOnLowPower && self.IsDisabled()))
+					wsb.PlayCustomAnimation(self, Info.Sequences.Random(Game.CosmeticRandom), () => wsb.CancelCustomAnimation(self));
+				ticks = Info.Interval;
+			}
+		}
+
+		public void BuildingComplete(Actor self)
+		{
+			buildComplete = true;
+		}
+
+		public void Selling(Actor self)
+		{
+			buildComplete = false;
+		}
+
+		public void Sold(Actor self) { }
+	}
+}

OpenRA.Mods.Common/Traits/SupportPowers/SpawnActorPower.cs

@@ -52,7 +52,7 @@ namespace OpenRA.Mods.Common.Traits
 					Game.Sound.Play(info.DeploySound, location);
 
 					if (!string.IsNullOrEmpty(info.EffectSequence) && !string.IsNullOrEmpty(info.EffectPalette))
-						w.Add(new SpriteEffect(location, w, info.EffectSequence, info.EffectPalette));
+						w.Add(new SpriteEffect(location, w, info.EffectSequence, "idle", info.EffectPalette));
 
 					var actor = w.CreateActor(info.Actor, new TypeDictionary
 					{

OpenRA.Mods.Common/UtilityCommands/UpgradeRules.cs

@@ -2818,6 +2818,18 @@ namespace OpenRA.Mods.Common.UtilityCommands
 					}
 				}
 
+				if (engineVersion < 20160103)
+				{
+					// Overhauled WithActiveAnimation -> WithIdleAnimation
+					if (node.Key == "WithActiveAnimation")
+					{
+						node.Key = "WithIdleAnimation";
+						foreach (var n in node.Value.Nodes)
+							if (n.Key == "Sequence")
+								n.Key = "Sequences";
+					}
+				}
+
 				UpgradeActorRules(engineVersion, ref node.Value.Nodes, node, depth + 1);
 			}
 		}

OpenRA.Mods.D2k/Activities/SwallowActor.cs

@@ -19,22 +19,22 @@ using OpenRA.Traits;
 
 namespace OpenRA.Mods.D2k.Activities
 {
-	enum AttackState { Burrowed, EmergingAboveGround, ReturningUnderground }
+	enum AttackState { Uninitialized, Burrowed, Attacking }
 
 	class SwallowActor : Activity
 	{
 		const int NearEnough = 1;
 
-		readonly CPos location;
 		readonly Target target;
 		readonly Sandworm sandworm;
+		readonly UpgradeManager manager;
 		readonly WeaponInfo weapon;
-		readonly WithSpriteBody withSpriteBody;
 		readonly RadarPings radarPings;
 		readonly AttackSwallow swallow;
 		readonly IPositionable positionable;
 
 		int countdown;
+		CPos burrowLocation;
 		AttackState stance;
 
 		public SwallowActor(Actor self, Target target, WeaponInfo weapon)
@@ -44,128 +44,133 @@ namespace OpenRA.Mods.D2k.Activities
 			sandworm = self.Trait<Sandworm>();
 			positionable = self.Trait<Mobile>();
 			swallow = self.Trait<AttackSwallow>();
-			withSpriteBody = self.Trait<WithSpriteBody>();
+			manager = self.Trait<UpgradeManager>();
 			radarPings = self.World.WorldActor.TraitOrDefault<RadarPings>();
-			countdown = swallow.Info.AttackTime;
-
-			withSpriteBody.DefaultAnimation.ReplaceAnim(sandworm.Info.BurrowedSequence);
-			stance = AttackState.Burrowed;
-			location = target.Actor.Location;
 		}
 
-		bool WormAttack(Actor worm)
+		bool AttackTargets(Actor self, IEnumerable<Actor> targets)
 		{
 			var targetLocation = target.Actor.Location;
-
+			foreach (var t in targets)
-			// The target has moved too far away
-			if ((location - targetLocation).Length > NearEnough)
-				return false;
-
-			var lunch = worm.World.ActorMap.GetActorsAt(targetLocation)
-				.Where(t => !t.Equals(worm) && weapon.IsValidAgainst(t, worm));
-
-			if (!lunch.Any())
-				return false;
-
-			stance = AttackState.EmergingAboveGround;
-			sandworm.IsAttacking = true;
-
-			foreach (var actor in lunch)
 			{
-				var actor1 = actor;	// loop variable in closure hazard
+				var target = t; // loop variable in closure hazard
 
-				actor.World.AddFrameEndTask(_ =>
+				self.World.AddFrameEndTask(_ =>
+				{
+					// Don't use Kill() because we don't want any of its side-effects (husks, etc)
+					target.Dispose();
+
+					// Harvester insurance
+					if (target.Info.HasTraitInfo<HarvesterInfo>())
 					{
-						actor1.Dispose();
+						var insurance = target.Owner.PlayerActor.TraitOrDefault<HarvesterInsurance>();
-
-						// Harvester insurance
-						if (!actor1.Info.HasTraitInfo<HarvesterInfo>())
-							return;
-
-						var insurance = actor1.Owner.PlayerActor.TraitOrDefault<HarvesterInsurance>();
-
 						if (insurance != null)
-							actor1.World.AddFrameEndTask(__ => insurance.TryActivate());
+							self.World.AddFrameEndTask(__ => insurance.TryActivate());
-					});
+					}
+				});
 			}
 
-			positionable.SetPosition(worm, targetLocation);
+			positionable.SetPosition(self, targetLocation);
 
-			var attackPosition = worm.CenterPosition;
+			var attackPosition = self.CenterPosition;
-			var affectedPlayers = lunch.Select(x => x.Owner).Distinct().ToList();
+			var affectedPlayers = targets.Select(x => x.Owner).Distinct().ToList();
-
-			PlayAttack(worm, attackPosition, affectedPlayers);
-			foreach (var notify in worm.TraitsImplementing<INotifyAttack>())
-				notify.Attacking(worm, target, null, null);
-
-			return true;
-		}
-
-		// List because IEnumerable gets evaluated too late.
-		void PlayAttack(Actor self, WPos attackPosition, List<Player> affectedPlayers)
-		{
-			withSpriteBody.PlayCustomAnimation(self, sandworm.Info.MouthSequence);
 			Game.Sound.Play(swallow.Info.WormAttackSound, self.CenterPosition);
 
 			Game.RunAfterDelay(1000, () =>
 			{
-				if (Game.IsCurrentWorld(self.World))
+				if (!Game.IsCurrentWorld(self.World))
-					foreach (var affectedPlayer in affectedPlayers)
+					return;
-						NotifyPlayer(affectedPlayer, attackPosition);
+
+				foreach (var player in affectedPlayers)
+				{
+					Game.Sound.PlayNotification(player.World.Map.Rules, player, "Speech", swallow.Info.WormAttackNotification, player.Faction.InternalName);
+
+					if (player == player.World.RenderPlayer)
+						radarPings.Add(() => true, attackPosition, Color.Red, 50);
+				}
 			});
-		}
 
-		void NotifyPlayer(Player player, WPos location)
+			foreach (var notify in self.TraitsImplementing<INotifyAttack>())
-		{
+				notify.Attacking(self, target, null, null);
-			Game.Sound.PlayNotification(player.World.Map.Rules, player, "Speech", swallow.Info.WormAttackNotification, player.Faction.InternalName);
 
-			if (player == player.World.RenderPlayer)
+			return true;
-				radarPings.Add(() => true, location, Color.Red, 50);
 		}
 
 		public override Activity Tick(Actor self)
 		{
-			if (countdown > 0)
+			switch (stance)
 			{
-				countdown--;
+				case AttackState.Uninitialized:
-				return this;
+					GrantUpgrades(self);
-			}
+					stance = AttackState.Burrowed;
+					countdown = swallow.Info.AttackDelay;
+					burrowLocation = self.Location;
+					break;
+				case AttackState.Burrowed:
+					if (--countdown > 0)
+						return this;
 
-			// Wait for the worm to get back underground
+					var targetLocation = target.Actor.Location;
-			if (stance == AttackState.ReturningUnderground)
-			{
-				sandworm.IsAttacking = false;
 
-				// There is a chance that the worm would just go away after attacking
+					// The target has moved too far away
-				if (self.World.SharedRandom.Next() % 100 <= sandworm.Info.ChanceToDisappear)
+					if ((burrowLocation - targetLocation).Length > NearEnough)
-				{
+					{
-					self.CancelActivity();
+						RevokeUpgrades(self);
-					self.World.AddFrameEndTask(w => self.Kill(self));
+						return NextActivity;
-				}
+					}
-				else
-					withSpriteBody.DefaultAnimation.ReplaceAnim(sandworm.Info.IdleSequence);
 
-				return NextActivity;
+					// The target reached solid ground
-			}
+					if (!positionable.CanEnterCell(targetLocation, null, false))
+					{
+						RevokeUpgrades(self);
+						return NextActivity;
+					}
 
-			// Wait for the worm to get in position
+					var targets = self.World.ActorMap.GetActorsAt(targetLocation)
-			if (stance == AttackState.Burrowed)
+						.Where(t => !t.Equals(self) && weapon.IsValidAgainst(t, self));
-			{
+
-				// This is so that the worm cancels an attack against a target that has reached solid rock
+					if (!targets.Any())
-				if (!positionable.CanEnterCell(target.Actor.Location, null, false))
+					{
+						RevokeUpgrades(self);
+						return NextActivity;
+					}
+
+					stance = AttackState.Attacking;
+					countdown = swallow.Info.ReturnDelay;
+					sandworm.IsAttacking = true;
+					AttackTargets(self, targets);
+
+					break;
+				case AttackState.Attacking:
+					if (--countdown > 0)
+						return this;
+
+					sandworm.IsAttacking = false;
+
+					// There is a chance that the worm would just go away after attacking
+					if (self.World.SharedRandom.Next(100) <= sandworm.Info.ChanceToDisappear)
+					{
+						self.CancelActivity();
+						self.World.AddFrameEndTask(w => self.Dispose());
+					}
+
+					RevokeUpgrades(self);
 					return NextActivity;
-
-				if (!WormAttack(self))
-				{
-					withSpriteBody.DefaultAnimation.ReplaceAnim(sandworm.Info.IdleSequence);
-					return NextActivity;
-				}
-
-				countdown = swallow.Info.ReturnTime;
-				stance = AttackState.ReturningUnderground;
 			}
 
 			return this;
 		}
+
+		void GrantUpgrades(Actor self)
+		{
+			foreach (var up in swallow.Info.AttackingUpgrades)
+				manager.GrantUpgrade(self, up, this);
+		}
+
+		void RevokeUpgrades(Actor self)
+		{
+			foreach (var up in swallow.Info.AttackingUpgrades)
+				manager.RevokeUpgrade(self, up, this);
+		}
 	}
 }

OpenRA.Mods.D2k/Traits/AttackSwallow.cs

@@ -20,10 +20,14 @@ namespace OpenRA.Mods.D2k.Traits
 	class AttackSwallowInfo : AttackFrontalInfo
 	{
 		[Desc("The number of ticks it takes to return underground.")]
-		public readonly int ReturnTime = 60;
+		public readonly int ReturnDelay = 60;
 
 		[Desc("The number of ticks it takes to get in place under the target to attack.")]
-		public readonly int AttackTime = 30;
+		public readonly int AttackDelay = 30;
+
+		[UpgradeGrantedReference]
+		[Desc("The upgrades to grant while attacking.")]
+		public readonly string[] AttackingUpgrades = { "attacking" };
 
 		public readonly string WormAttackSound = "Worm.wav";
 

OpenRA.Mods.D2k/Traits/Sandworm.cs

@@ -10,12 +10,13 @@
 
 using System;
 using System.Linq;
+using OpenRA.Graphics;
 using OpenRA.Mods.Common.Traits;
 using OpenRA.Traits;
 
 namespace OpenRA.Mods.D2k.Traits
 {
-	class SandwormInfo : WandersInfo, Requires<MobileInfo>, Requires<WithSpriteBodyInfo>, Requires<AttackBaseInfo>
+	class SandwormInfo : WandersInfo, Requires<MobileInfo>, Requires<AttackBaseInfo>
 	{
 		[Desc("Time between rescanning for targets (in ticks).")]
 		public readonly int TargetRescanInterval = 125;
@@ -29,15 +30,6 @@ namespace OpenRA.Mods.D2k.Traits
 		[Desc("The chance this actor has of disappearing after it attacks (in %).")]
 		public readonly int ChanceToDisappear = 100;
 
-		[Desc("Name of the sequence that is used when the actor is idle or moving (not attacking).")]
-		[SequenceReference] public readonly string IdleSequence = "idle";
-
-		[Desc("Name of the sequence that is used when the actor is attacking.")]
-		[SequenceReference] public readonly string MouthSequence = "mouth";
-
-		[Desc("Name of the sequence that is used when the actor is burrowed.")]
-		[SequenceReference] public readonly string BurrowedSequence = "burrowed";
-
 		public override object Create(ActorInitializer init) { return new Sandworm(init.Self, this); }
 	}
 
@@ -47,7 +39,6 @@ namespace OpenRA.Mods.D2k.Traits
 
 		readonly WormManager manager;
 		readonly Mobile mobile;
-		readonly WithSpriteBody withSpriteBody;
 		readonly AttackBase attackTrait;
 
 		public bool IsMovingTowardTarget { get; private set; }
@@ -61,19 +52,10 @@ namespace OpenRA.Mods.D2k.Traits
 		{
 			Info = info;
 			mobile = self.Trait<Mobile>();
-			withSpriteBody = self.Trait<WithSpriteBody>();
 			attackTrait = self.Trait<AttackBase>();
 			manager = self.World.WorldActor.Trait<WormManager>();
 		}
 
-		public override void OnBecomingIdle(Actor self)
-		{
-			if (withSpriteBody.DefaultAnimation.CurrentSequence.Name != Info.IdleSequence)
-				withSpriteBody.DefaultAnimation.PlayRepeating(Info.IdleSequence);
-
-			base.OnBecomingIdle(self);
-		}
-
 		public override void DoAction(Actor self, CPos targetCell)
 		{
 			IsMovingTowardTarget = false;

mods/cnc/rules/trees.yaml

@@ -3,7 +3,7 @@ SPLIT2:
 	SeedsResource:
 		ResourceType: Tiberium
 		Interval: 55
-	WithActiveAnimation:
+	WithIdleAnimation:
 
 SPLIT3:
 	Inherits: ^TibTree
@@ -12,7 +12,7 @@ SPLIT3:
 	SeedsResource:
 		ResourceType: Tiberium
 		Interval: 55
-	WithActiveAnimation:
+	WithIdleAnimation:
 
 SPLITBLUE:
 	Inherits: ^TibTree
@@ -21,7 +21,7 @@ SPLITBLUE:
 	SeedsResource:
 		ResourceType: BlueTiberium
 		Interval: 110
-	WithActiveAnimation:
+	WithIdleAnimation:
 	Tooltip:
 		Name: Blossom Tree (blue)
 	RadarColorFromTerrain:

mods/d2k/rules/arrakis.yaml

@@ -69,8 +69,20 @@ sandworm:
 			Spice: 100
 	Targetable:
 		TargetTypes: Ground
-	WithFacingSpriteBody:
+	WithSpriteBody:
-	WithAttackOverlay:
+	WithIdleAnimation:
+		Interval: 160
+		Sequences: lightninga, lightningb, lightningc, lightningd, lightninge, lightningf
+		UpgradeTypes: attacking
+		UpgradeMaxEnabledLevel: 0
+	AmbientSound:
+		SoundFile: WRMSIGN1.WAV
+		Interval: 160
+		UpgradeTypes: attacking
+		UpgradeMaxEnabledLevel: 0
+	WithAttackOverlay@mouth:
+		Sequence: mouth
+	WithAttackOverlay@sand:
 		Sequence: sand
 	HiddenUnderFog:
 	AppearsOnRadar:
@@ -88,6 +100,16 @@ sandworm:
 		PingRadar: True
 	RevealsShroud:
 		Range: 5c0
+	LeavesTrails:
+		Image: sandtrail
+		Sequences: traila, trailb, trailc
+		Palette: effect
+		Type: CenterPosition
+		TerrainTypes: Sand, Dune, SpiceSand, Spice
+		MovingInterval: 3
+		UpgradeTypes: attacking
+		UpgradeMaxEnabledLevel: 0
+	UpgradeManager:
 
 sietch:
 	Inherits: ^Building

mods/d2k/sequences/infantry.yaml

@@ -481,10 +481,35 @@ sandworm:
 		Length: 20
 		Tick: 100
 	idle: DATA.R8
-		Start: 3586
-		Length: 35
-		Tick: 180
-		BlendMode: Additive
-	burrowed: DATA.R8
 		Start: 39
+	lightninga: DATA.R8
+		Start: 3591
+		Length: 5
+		Tick: 80
+		BlendMode: Additive
+	lightningb: DATA.R8
+		Start: 3596
+		Length: 5
+		Tick: 80
+		BlendMode: Additive
+	lightningc: DATA.R8
+		Start: 3601
+		Length: 5
+		Tick: 80
+		BlendMode: Additive
+	lightningd: DATA.R8
+		Start: 3606
+		Length: 5
+		Tick: 80
+		BlendMode: Additive
+	lightninge: DATA.R8
+		Start: 3611
+		Length: 5
+		Tick: 80
+		BlendMode: Additive
+	lightningf: DATA.R8
+		Start: 3616
+		Length: 5
+		Tick: 80
+		BlendMode: Additive
 	icon: wormicon.shp

mods/d2k/sequences/misc.yaml

@@ -109,6 +109,17 @@ laserfire:
 		BlendMode: Additive
 		ZOffset: 511
 
+sandtrail:
+	Defaults:
+		Length: 8
+		Tick: 200
+		ZOffset: -512
+	traila: sandtrail.shp
+	trailb: sandtrail.shp
+		Frames: 2, 6, 4, 5, 0, 1, 3, 7
+	trailc: sandtrail.shp
+		Frames: 7, 4, 6, 5, 2, 0, 3, 1
+
 pips:
 	groups: DATA.R8
 		Start: 17

mods/ts/rules/defaults.yaml

@@ -571,7 +571,7 @@
 	SeedsResource:
 		ResourceType: Tiberium
 		Interval: 55
-	WithActiveAnimation:
+	WithIdleAnimation:
 
 ^Tree:
 	Inherits@1: ^SpriteActor