chore(supabase): drop dead SMP queries - #9
Open
jochemwhite wants to merge 1 commit into
Open
jochemwhite wants to merge 1 commit into
jochemwhite wants to merge 1 commit into
Conversation
The Minecraft SMP tables are being dropped from the database (streamwizard#222), so the queries that read them cannot work any more. queries/smp.ts held updateSmpPlayerOnlineStatus and getSmpActionsByTrigger. Nothing in this repo imported either, so removing them changes no behaviour — they were already dead, and would have started throwing the moment anything wired them up. The "./queries/smp" subpath export goes with the file, otherwise package.json would advertise an entry point that no longer resolves, and the vendored generated types lose the four SMP tables. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0177pZz5wqi3H3BJq4PLM1be
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Companion to streamwizard#222, which drops the Minecraft SMP tables from the database.
What goes
packages/supabase/src/queries/smp.ts—updateSmpPlayerOnlineStatus(writessmp_players.is_online) andgetSmpActionsByTrigger(readssmp_actions). Nothing in this repo imports either, so this changes no behaviour. They were already dead code; once #222 lands they'd throw the moment anything wired them up."./queries/smp"subpath export inpackages/supabase/package.json— it points at the deleted file, so leaving it would advertise an entry point that no longer resolves.Structure only, no runtime code paths touched.
Verification
bun x turbo run check-types— the twoCannot find module 'next/server'/'next/headers'failures in@repo/supabaseare pre-existing, confirmed identical on a stashed clean tree. This branch adds none.package.jsonstill parses as valid JSONsmpreferences anywhere in the repoOrdering
Independent of #222 — safe to merge either way round, since the code is unreferenced today and the tables are unused.
🤖 Generated with Claude Code
https://claude.ai/code/session_0177pZz5wqi3H3BJq4PLM1be