tower-sell #12

Merged

matt merged 6 commits from tower-sell into main

2026-07-14 23:54:08 -07:00

matt commented

2026-07-14 23:23:22 -07:00

Owner

Adding selling towers as a feature, updating pathfinding logic to remove hitching, and adding new sound and vfx files

matt added 5 commits 2026-07-14 23:23:23 -07:00

Add tower Sell action with refund + coin VFX/SFX 9be31b1645

Owner-validated sell RPC refunds SellRefundPercent of invested gold (Wall refunds 100% while un-upgraded), despawns the tower, and broadcasts a coin-burst VFX + rustle SFX. Refunds do not count as per-wave income. Investment/upgrade tracking added as the seam for the future upgrade system.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Fix NGO RequireOwnership deprecation warnings; add VFX script metas 86bc916c49

Replace deprecated [Rpc(..., RequireOwnership = true)] with InvokePermission = RpcInvokePermission.Owner on the owner-submitted draft/buff RPCs (PlayerDraft, PlayerBuffManager), matching the pattern already used elsewhere. Also commits the .meta files Unity generated for the new sell-VFX scripts so their GUIDs are stable across machines.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Fix sell hitch: batch footprint un-stamp; warn on missing SellEffectSpawner 1794f65b19

TowerInstance.StampFootprint now writes walkability via SetWalkableBatch instead of per-tile SetWalkable, so despawning (selling) a 2x2 tower fires one OnWalkabilityChanged / enemy re-path instead of four — matching placement and removing the frame hitch on sell. PlaySellEffectRpc now logs a one-time warning when no SellEffectSpawner is in the scene, since a missing spawner silently suppressed the coin VFX and sound.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Adding the feature to sell towers with VFX and audio included 0b4cde4590

Docs: mark Tower Sell done; note budgeted pathfinding re-path scheduler 6c03a2f250

Updates Project_Context.md + Project_Roadmap.md for the tower-sell work (refund rules, coin VFX/SFX, refund excluded from income) and the PathfindingService budgeted re-path scheduler that removed the sell/placement frame hitch. Follows commit 0b4cde4 which landed the code + art.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

matt added 1 commit 2026-07-14 23:28:18 -07:00

reverting 9player changes 9f87c04b4c