Skip to content

docs(claude): document bball_* Neon tables, flag Pente-scoped boundary rule - #45

Merged
brooksRoley merged 1 commit into
mainfrom
claude/youthful-davinci-nro6wx
Sep 15, 2026
Merged

brooksRoley merged 1 commit into
mainfrom
claude/youthful-davinci-nro6wx

Conversation

@brooksRoley

Copy link
Copy Markdown
Owner

Summary

  • Adds bball_runs, bball_board_states, bball_roster to CLAUDE.md's Neon Tables list — they existed in the DB but weren't documented.
  • Clarifies that the "Do NOT put game data in Neon" rule lives under the Pente Game Platform section and is scoped to Pente/Go's realtime-multiplayer + ELO needs; it doesn't cover bball, which predates the rule (commit 85ac30e, no rationale recorded) and has no realtime/multiplayer requirement.
  • Deliberately does not propose or perform a data migration — that tension is flagged for a human call, not resolved here.

Livelihood stream

Portfolio signal / engineering hygiene (CTO lens) — no direct revenue impact, but this is a documentation-accuracy fix, not a redesign.

Why

This item had been flagged twice in the cross-session Running Notes ledger ("CLAUDE.md Neon/Supabase boundary doc gap — CTO's beat, flag 2, unshipped") as of 2026-09-09. Per this repo's own anti-duplication protocol, a flag-2 agent-doable item should be shipped, not written up as a prompt a third time.

Verification

  • Doc-only change (CLAUDE.md), no code touched.
  • yarn test: 834/834 passing, 73 files (ran fresh this session on main before this change).
  • yarn lint: clean, 0 warnings.
  • yarn build: clean, all routes compile.

🤖 Generated with Claude Code

https://claude.ai/code/session_01DmLds4KzH4RbV5iSPZdX53


Generated by Claude Code

…y rule

The "Do NOT put game data in Neon" rule under Pente Game Platform is
scoped to Pente/Go's realtime-multiplayer needs, but bball_runs,
bball_board_states, and bball_roster (Basketball Data Platform) have
lived in Neon since before that decision was written and were missing
from the Tables list entirely. Documents the tables and flags the
tension rather than asserting a migration — that's a real project, not
a doc fix.

Flagged twice in the Running Notes ledger (CTO's beat); shipping this
session instead of re-recommending it a third time.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DmLds4KzH4RbV5iSPZdX53
@vercel

vercel Bot commented Sep 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
zero-next Ready Ready Preview Sep 10, 2026 3:59pm UTC

@brooksRoley
brooksRoley merged commit 5e7cf40 into main Sep 15, 2026
4 checks passed
@brooksRoley
brooksRoley deleted the claude/youthful-davinci-nro6wx branch September 15, 2026 02:56
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.

2 participants