docs(instructions): add KenPom CSS selector fragility to Common Pitfalls

- Add note about KenPom HTML structure changes (table#player-table, div.refline selectors) to both copilot-instructions.md and CLAUDE.md

Author: saiemgilani

.github/copilot-instructions.md

@@ -1,17 +1,18 @@
 <!-- START doctoc generated TOC please keep comment here to allow auto update -->
 <!-- DON'T EDIT THIS SECTION, INSTEAD RE-RUN doctoc TO UPDATE -->
 
-**Table of Contents** _generated with [DocToc](https://github.com/thlorenz/doctoc)_
 
 - [hoopR Copilot Instructions](#hoopr-copilot-instructions)
   - [Project Context](#project-context)
+  - [Repository Workflow](#repository-workflow)
   - [Code Style](#code-style)
-  - [Function Naming](#function-naming)
   - [HTTP Layer](#http-layer)
   - [Messaging Layer](#messaging-layer)
+  - [Function Naming](#function-naming)
   - [Roxygen Documentation](#roxygen-documentation)
   - [Testing](#testing)
     - [Environment Variables](#environment-variables)
+    - [CI Secrets](#ci-secrets)
   - [Conventional Commits](#conventional-commits)
   - [V3 API Notes](#v3-api-notes)
   - [Common Pitfalls](#common-pitfalls)
@@ -153,4 +154,5 @@ Types: `feat`, `fix`, `docs`, `test`, `refactor`, `chore`, `style`, `perf`, `ci`
 - IST Standings has dynamic game columns -- use `expect_true(all(core_cols %in% colnames()))`.
 - NBAGL legacy schemas are no longer stable references for tests. Prefer validating core columns from current API payloads and handle named-list returns explicitly in tests.
 - Local editor/worktree artifacts (e.g., `.vscode`, `.claude`, temp logs) can cause `R CMD check` notes/warnings if included in source checks.
+- KenPom HTML structure changes periodically -- CSS selectors for tables (`table#player-table`), referee links (`div.refline`), and navigation elements are fragile and may need updating.
 - Never edit `NAMESPACE` or `man/` files by hand; regenerate with `devtools::document()`.

CLAUDE.md

@@ -4,6 +4,7 @@
 
 - [CLAUDE.md -- hoopR Development Guide](#claudemd----hoopr-development-guide)
   - [Package Overview](#package-overview)
+  - [Branching & PR Workflow](#branching--pr-workflow)
   - [Build & Development Commands](#build--development-commands)
   - [Project Structure](#project-structure)
   - [Key Coding Conventions](#key-coding-conventions)
@@ -17,6 +18,7 @@
   - [Testing](#testing)
     - [Test Pattern](#test-pattern)
     - [Environment Variables for Tests](#environment-variables-for-tests)
+    - [CI Secrets](#ci-secrets)
     - [Rate Limiting](#rate-limiting)
   - [NAMESPACE](#namespace)
   - [Commit Convention](#commit-convention)
@@ -343,4 +345,5 @@ Split unrelated work into separate commits for reviewability.
 - `.v3_to_v2_format()` uses row-level loops for player resolution -- performance-sensitive for large PBP datasets. The `%||%` operator from rlang is used for null-safe named vector lookups in event type maps.
 - `.players_on_court_v3()` depends on `nba_gamerotation()` returning `IN_TIME_REAL`/`OUT_TIME_REAL` in tenths of a second -- ensure time unit consistency when modifying.
 - Local dev artifacts (for example `.vscode`, `.claude`, ad-hoc logs) can surface as `R CMD check` notes/warnings if not excluded from build inputs.
+- KenPom HTML structure changes periodically -- CSS selectors for tables (`table#player-table`), referee links (`div.refline`), and navigation elements are fragile and may need updating.
 - Never hand-edit `NAMESPACE` or files under `man/`; regenerate with `devtools::document()`.