Skip to content

Legal checks as repo skills: /legals-update runs privacy, Steam and licence checks side by side - #215

Merged
ChefControl merged 5 commits into
mainfrom
claude/legals-update-skill-efe2ce
Oct 28, 2026
Merged

ChefControl merged 5 commits into
mainfrom
claude/legals-update-skill-efe2ce

Conversation

@ChefControl

Copy link
Copy Markdown
Owner

A repeatable check for each new feature, so the privacy notice (#150), the Steam content survey (#151) and the licences and credits (#147, #148, #149) stay right. Part of #142.

The skills

Each one also works when run on its own. They report and propose, and edit nothing unless asked. Nothing from inside Steamworks, and no NDA or SDA terms, goes in them. CLAUDE.md's agent skills section now points to them.

Dry run: /legals-update on PR #212 (merged)

The three subagents ran side by side: privacy 39 s, Steam 40 s, licences 65 s.

Legal check: PR #212 (casino slot machines recording)

Privacy: no change. src/roomTone.ts fetches slots.mp3 from the game's own host, as before. The new credit links (freesound.org, CC BY 4.0) are covered by Links to other sites. firestore.rules is untouched.

Steam: no change (#151 open, not submitted). The recording is still a real, credited one, so the AI text holds. The guests' machines now play a spin per pull, but the odds and the player's own $250 machine are unchanged, and the prep page's Slots row is still right.

Licences: 1 update.

  • OK: src/sounds/slots.mp3, "WOF_slots6" by fugacious, CC BY 4.0 (freesound.org/s/145539). It's credited in #creditsCard and docs/game.md's Sound credits, and the test checks it.
  • Needs a listen: src/sounds/slots.mp3, 0:10–0:14.2. Shazam matched "Feels Like Tonight" (Daughtry) somewhere in the source's 0:10–0:20, which overlaps the end of the cut. Its 10 s windows inside 0–15 s came back clean, but no owner listen of the final cut is posted on Steam: listen-check the casino room recordings for background music #147.

For the owner: listen to the last 4 s of slots.mp3 on headphones and post the result on #147 before closing it. (Steam also noticed the prep page's gambling table still shows 😭 for the corner of tears, which is 😢 since #210.)

The licence finding matches what PR #212's own description says.

🤖 Generated with Claude Code

…cence checks side by side

Four skills under .claude/skills/. legals-privacy compares a feature with the
privacy notice (public/privacy.html), its short version in Settings and the
report dialog's "What's sent" list. legals-steam compares it with the content
survey answers and store texts agreed on #151 and the prep page. legals-licenses
checks every new package and asset's licence and its credit under Credits &
Copyrights, and flags recordings for a listen. legals-update pins one target and
runs the three at once in subagents, then puts their reports together. Each
reports and proposes; none edits unless the owner asks. CLAUDE.md points to it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ChefControl
ChefControl added this pull request to stack #217 October 28, 2026 01:03
@ChefControl
ChefControl merged commit a0fd0e3 into main Oct 28, 2026
7 checks passed
@ChefControl
ChefControl deleted the claude/legals-update-skill-efe2ce branch October 28, 2026 01:12
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.

1 participant