chore: remove dead tryPoolFallback constant in antDepositFood

[LightAxe]

Location

src/sim/ant/ant-system.ts line 548

Description

antDepositFood declares:

const tryPoolFallback = true;
if (tryPoolFallback && remaining > 0) {
  // entrance-pool fallback deposit path

tryPoolFallback is always true — the condition simplifies to if (remaining > 0). The variable was introduced as a version-gate placeholder when the pool fallback path was made unconditional (the comment above it references "Issue #68 (v12+) — pre-v12 this was an else branch"), but the boolean flag was never removed.

Impact

No runtime impact. The name tryPoolFallback implies a togglable gate, misleading any reader into thinking there is a path where the pool fallback is skipped. The dead constant also creates noise in the logical structure of the deposit function.

Fix

Remove const tryPoolFallback = true and replace if (tryPoolFallback && remaining > 0) with if (remaining > 0).

Severity

Low — cleanup / dead code. No behaviour change.

[LightAxe]

Reviewed against current origin/main (16e4201) and this issue is correct.

antDepositFood still declares const tryPoolFallback = true at src/sim/ant/ant-system.ts:548, and the only use is if (tryPoolFallback && remaining > 0) at :549. Since the flag is a literal and there is no version gate left, this simplifies directly to if (remaining > 0).

Obvious fix path: delete the constant, simplify the condition, and keep the surrounding Issue #68 comments if they are still useful historical context. This should be no behavior change; the relevant safety net is the chamber-deposit / entrance-pool fallback / waiting-deposit test coverage.

[LightAxe]

Agree — confirmed, one-liner fix.

Verified against origin/main (16e4201). const tryPoolFallback = true at ant-system.ts:548 is the only occurrence of the symbol; the condition at :549 collapses to if (remaining > 0).

Fix: delete the constant declaration, simplify the condition. The Issue #68 comment above it (explaining why the pool fallback is unconditional) is useful historical context and should stay. No behavior change; existing deposit and waiting-state tests cover the path.

[LightAxe]

Resolved by PR #167 (squash-merged to main as 73d6d64).