domain-skills: Best Coast Pairings (bestcoastpairings.com) - #540
Open
chocobo1900-pixel wants to merge 1 commit into
Open
domain-skills: Best Coast Pairings (bestcoastpairings.com)#540chocobo1900-pixel wants to merge 1 commit into
chocobo1900-pixel wants to merge 1 commit into
Conversation
…coming-event trap) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
✅ Skill review passedReviewed 1 file(s) — no findings. |
Contributor
There was a problem hiding this comment.
1 issue found across 1 file
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="domain-skills/bestcoastpairings/scraping.md">
<violation number="1" location="domain-skills/bestcoastpairings/scraping.md:88">
P3: The Pairings API endpoint is documented under the `## UI notes` heading, but it's a logged-out REST API call — no browser needed. The UI notes section should cover only browser/interaction concerns (React, cookie consent, location autocomplete). Consider promoting this to a `## Pairings` subsection under `## Private API` (or a sibling section alongside it) so readers looking for REST endpoints don't miss it.</violation>
</file>
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
| ### Pairings / match results (per round) | ||
|
|
||
| ``` | ||
| GET /v1/events/{eventId}/pairings?eventId={eventId}&round=N&pairingType=Pairing&limit=500 |
Contributor
There was a problem hiding this comment.
P3: The Pairings API endpoint is documented under the ## UI notes heading, but it's a logged-out REST API call — no browser needed. The UI notes section should cover only browser/interaction concerns (React, cookie consent, location autocomplete). Consider promoting this to a ## Pairings subsection under ## Private API (or a sibling section alongside it) so readers looking for REST endpoints don't miss it.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At domain-skills/bestcoastpairings/scraping.md, line 88:
<comment>The Pairings API endpoint is documented under the `## UI notes` heading, but it's a logged-out REST API call — no browser needed. The UI notes section should cover only browser/interaction concerns (React, cookie consent, location autocomplete). Consider promoting this to a `## Pairings` subsection under `## Private API` (or a sibling section alongside it) so readers looking for REST endpoints don't miss it.</comment>
<file context>
@@ -0,0 +1,102 @@
+### Pairings / match results (per round)
+
+```
+GET /v1/events/{eventId}/pairings?eventId={eventId}&round=N&pairingType=Pairing&limit=500
+```
+
</file context>
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
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.
Adds a domain skill for Best Coast Pairings, the tournament pairings platform for tabletop wargames.
Covers the logged-out REST API (
newprod-api.bestcoastpairings.comwith theclient-id: web-appheader): event search, rosters/placings, team events, per-round pairings, and army lists (the one auth-gated read).Field-tested traps documented:
active/dropped, notdataplacings=truereturns an empty roster — drop the param, and notedeletedmirrorsactiveexactlyteamis club affiliation, the event team isteamPlayerId🤖 Generated with Claude Code
Summary by cubic
Adds a new domain skill for Best Coast Pairings, detailing how to read their REST API logged out with the
client-id: web-appheader. Enables fetching events, rosters, team data, and pairings without driving the UI.active/dropped, plus upcoming-eventplacingstrap/teamplayersandteamPlayerIdvs clubteamactive, with player/game fieldsWritten for commit a975838. Summary will update on new commits.