From 4b57006a5c6565a2120f08c75c0e2f32ef3fcb9e Mon Sep 17 00:00:00 2001 From: Jeroen Schweitzer Date: Sat, 7 Mar 2026 08:48:20 +0100 Subject: [PATCH] chore(simulation): remove duplicate TODO and dead binding in generator spike Co-Authored-By: Claude Opus 4.6 --- server/src/bin/generator_spike.rs | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/server/src/bin/generator_spike.rs b/server/src/bin/generator_spike.rs index 5388223ce..9cdfc9954 100644 --- a/server/src/bin/generator_spike.rs +++ b/server/src/bin/generator_spike.rs @@ -295,9 +295,7 @@ const ALL_TRAITS: [PersonalityTrait; 10] = [ PersonalityTrait::Reclusive, ]; -// TODO(integration): Duplicates `traits_contradict` from npc/generate.rs. -// Extract to a shared helper in npc/mod.rs when wiring the spike into the main pipeline. -// TODO(#612 integration): duplicates the private `traits_contradict` in npc/generate.rs. +// TODO(integration): duplicates the private `traits_contradict` in npc/generate.rs. // When wiring the spike into the main pipeline, expose that function as pub(crate) // and remove this copy. Kept separate for now to avoid spike coupling to ECS internals. fn traits_contradict(a: PersonalityTrait, b: PersonalityTrait) -> bool { @@ -719,7 +717,6 @@ fn main() { // Pre-generate unique names without replacement (#3 fix). // build_name_pool caps at pool size — actual count may be less than requested. let name_pool = build_name_pool(&mut rng, &culture, npc_count); - let _npc_count = name_pool.len(); // use capped count (for documentation) // First pass: generate all NPCs (no relationships yet) let mut npcs: Vec = name_pool