cal/claude-plugins
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
main
claude-plugins/plugins/save-doc/examples/troubleshooting.md
Cal Corum 51fe634ff5 refactor: convert 5 more skills to commands, update transcriber defaults 
Convert backlog, project-plan, save-doc, youtube-transcriber, and
z-image from skills/ to commands/ so they appear as user-invocable
slash commands with plugin name prefixes.

Update youtube-transcriber: switch default model from gpt-4o-transcribe
to gpt-4o-mini-transcribe (OpenAI's current recommendation, half cost)
and fix cost estimates that were 4-7x too high.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-19 14:41:37 -05:00

1.3 KiB
Raw Permalink Blame History

title description type domain tags
Fix: Scout Token Purchase Not Deducting Currency Scout token buy flow silently failed to deduct 200₼ due to using db_patch instead of the dedicated money endpoint. troubleshooting development
paper-dynasty
discord
api
bug-fix

Fix: Scout Token Purchase Not Deducting Currency

Date: 2026-03-15 PR: #90 Severity: High — players getting free tokens

Problem

The /buy scout-token command completed successfully but didn't deduct the 200₼ cost. Players could buy unlimited tokens.

Root Cause

The buy handler used db_patch('/players/{id}', {'scout_tokens': new_count}) to increment tokens, but this endpoint doesn't trigger the money deduction side-effect. The dedicated /players/{id}/money endpoint handles balance validation and atomic deduction.

Fix

Replaced the db_patch call with a two-step flow:

  1. POST /players/{id}/money with {"amount": -200, "reason": "scout_token_purchase"}
  2. Only increment scout_tokens if the money call succeeds

Lessons

  • Always use dedicated money endpoints for currency operations — never raw patches
  • The db_patch helper bypasses business logic by design (it's for admin corrections)
  • Added integration test covering the full buy→deduct→verify flow