test: cover authored reward descriptor precedence

Author: MikelCalvo

internal/minimal/shared_world_test.go

@@ -9052,6 +9052,47 @@ func TestSharedWorldStaticActorDeathRewardFallsBackToRegisteredFormulaProfileDef
 	}
 }
 
+func TestSharedWorldStaticActorDeathRewardPrefersAuthoredDescriptorOverRegisteredProfileDefault(t *testing.T) {
+	const profile = "practice_authored_reward_override_mob"
+	if !worldruntime.RegisterStaticActorCombatProfile(profile, worldruntime.StaticActorCombatProfileDefaults{
+		MaxHP:                 8,
+		DamagePerNormalAttack: 2,
+		RespawnDelay:          worldruntime.PracticeMobBootstrapRespawnDelay,
+		DeathReward:           worldruntime.StaticActorDeathReward{Experience: 11, Gold: 7, DropVnums: []uint32{27002}},
+	}) {
+		t.Fatalf("expected %q profile registration to succeed", profile)
+	}
+	t.Cleanup(func() { worldruntime.UnregisterStaticActorCombatProfileForTest(profile) })
+
+	runtime := newSharedWorldRegistry()
+	actor := worldruntime.StaticEntity{
+		Entity:        worldruntime.Entity{ID: 0x01050245, Kind: worldruntime.EntityKindStaticActor, VID: 0x01050245, Name: "AuthoredRewardOverrideMob"},
+		Position:      worldruntime.NewPosition(bootstrapMapIndex, 1200, 2200),
+		RaceNum:       20350,
+		CombatProfile: profile,
+		CombatKind:    profile,
+		SpawnGroupRef: "practice.authored_reward_override_mob",
+	}
+	if _, ok := runtime.registerStaticActor(actor.Entity.ID, actor.Entity.Name, actor.Position.MapIndex, actor.Position.X, actor.Position.Y, actor.RaceNum, "", "", actor.CombatKind, actor.SpawnGroupRef, worldruntime.StaticActorDeathReward{Experience: 101, Gold: 55, DropVnums: []uint32{27004, 27003}}); !ok {
+		t.Fatal("expected authored reward override mob registration to succeed")
+	}
+
+	storedActor, ok := runtime.entities.StaticActor(actor.Entity.ID)
+	if !ok {
+		t.Fatal("expected authored reward override mob to remain registered")
+	}
+	reward := runtime.staticActorDeathRewardLocked(storedActor)
+	if reward.Experience != 101 || reward.Gold != 55 || len(reward.DropVnums) != 2 || reward.DropVnums[0] != 27003 || reward.DropVnums[1] != 27004 {
+		t.Fatalf("expected authored reward descriptor to override profile default, got %+v", reward)
+	}
+
+	reward.DropVnums[0] = 99999
+	reward = runtime.staticActorDeathRewardLocked(storedActor)
+	if len(reward.DropVnums) != 2 || reward.DropVnums[0] != 27003 || reward.DropVnums[1] != 27004 {
+		t.Fatalf("expected authored reward override lookup to return isolated clone, got %+v", reward)
+	}
+}
+
 func TestGameRuntimeCombinedScalarAndDropRewardEmitsAllRewards(t *testing.T) {
 	store := loginticket.NewFileStore(t.TempDir())
 	actor := worldruntime.StaticEntity{

spec/protocol/non-player-reward-bootstrap.md

@@ -48,6 +48,7 @@ That includes a drop-only profile default case where the killing hit emits the n
 That runtime coverage now also includes formula-only registered profiles whose `damage_per_normal_attack` is canonicalized from `attack_value - defense_value`, so the profile can both drive deterministic HP mutation and apply its reward descriptor on the same death edge.
 A formula-only registered profile may carry a combined descriptor as well; the killing hit still uses the formula-derived HP mutation first, then emits death/clear, scalar EXP/gold point changes, and deterministic owned drop frames in the normal reward ordering.
 If the authored spawn-group snapshot also carries an explicit reward descriptor, that authored descriptor wins over the profile-level default for the current actor life; profile defaults are only the fallback for spawn-backed actors whose live/authored snapshot has no explicit reward descriptor.
+Dedicated coverage freezes this precedence and clone isolation so a registered profile default cannot silently replace an authored EXP/gold/drop override.
 That profile-level default still does **not** make standalone runtime static actors reward-bearing: without a non-empty `spawn_group_ref` or explicit authored live snapshot descriptor, the shared-world death attempt returns a rewardless descriptor.
 
 ## Validation