Skip to content

docs: log the invalid fake key caught while making the film - #29

Merged
makoydev merged 1 commit into
nextfrom
docs/log-film-key-mistake
Oct 4, 2026
Merged

makoydev merged 1 commit into
nextfrom
docs/log-film-key-mistake

Conversation

@makoydev

@makoydev makoydev commented Oct 4, 2026

Copy link
Copy Markdown
Owner

Adds one row to "What the AI got wrong, and how it was caught" in docs/HOW-THIS-WAS-BUILT.md.

While making the explainer film (makoydev.github.io/vetted/film/), the fake cloud key I wrote for the sample code contained the digit 9. Real AWS access key IDs only use A–Z and 2–7, so Vetted correctly ignored it. Running the sample lines through the real v0.1.0 engine before animating caught it, and the key was replaced.

No code changes.

If asked in an interview

Every AI mistake goes in the build log, including small ones outside the product code. This one shows why the film's labels were checked against the real engine instead of being typed in by hand: the check caught a fake key that real keys could never look like. The alternative, leaving it out because it never shipped, would make the log look cleaner than the process actually was.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@makoydev makoydev added ai-drafted Change drafted by Claude Code; reviewed and merged by a human ai-merged Merged into the next integration branch by Claude Code after CI passed; Michael reviews next -> main labels Oct 4, 2026
@makoydev
makoydev merged commit 68bb0fa into next Oct 4, 2026
9 checks passed
@makoydev
makoydev deleted the docs/log-film-key-mistake branch October 4, 2026 14:30
makoydev added a commit that referenced this pull request Oct 5, 2026
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-drafted Change drafted by Claude Code; reviewed and merged by a human ai-merged Merged into the next integration branch by Claude Code after CI passed; Michael reviews next -> main

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant