Skip to content

Remove numTimesSent and maxTimesSent from communicator#775

Merged
chrxh merged 8 commits into
developfrom
worktree-claude+remove-num-times-sent
Jul 8, 2026
Merged

Remove numTimesSent and maxTimesSent from communicator#775
chrxh merged 8 commits into
developfrom
worktree-claude+remove-num-times-sent

Conversation

@chrxh

@chrxh chrxh commented Jul 1, 2026

Copy link
Copy Markdown
Owner

@
Removes the Signal::numTimesSent, Sender::maxTimesSent and SenderGenome::maxTimesSent fields together with their TO, GenomeTO, Desc and GenomeDesc equivalents, and all associated logic.

Removed

  • Structs/fields: Signal::numTimesSent, Sender::maxTimesSent, SenderGenome::maxTimesSent and the matching SignalTO/SenderTO/SenderGenomeTO/SignalDesc/SenderDesc/SenderGenomeDesc members
  • Logic: communicator send-count gating (processSender/tryTransmitSignal), neuron min-numTimesSent propagation, serializer members + IDs, GUI "Max times sent" inputs, validation clamps, mutation, genome hash, and the now-unused CommunicatorMaxTimesSent_* constants
  • Tests: dropped the two obsolete tests (sender_maxTimesSentExceeded_noSignalTransmitted, sender_signalPriority_lowerNumTimesSentWins), stripped field references from the rest, and bumped the mutation coverage-guard field count 2 -> 1

Testing

  • Release build: OK
  • EngineInterfaceTests (159), PersisterTests (64), EngineTests (3057): all pass

🤖 Generated with Claude Code
@

chrxh and others added 2 commits July 8, 2026 18:25
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@chrxh chrxh force-pushed the worktree-claude+remove-num-times-sent branch from 9bba163 to 70685a7 Compare July 8, 2026 16:48
@chrxh chrxh merged commit 8ae753d into develop Jul 8, 2026
1 check passed
@chrxh chrxh deleted the worktree-claude+remove-num-times-sent branch July 8, 2026 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant