Skip to content

fix: resolve all 10 open playground issues#11

Merged
scottyg merged 1 commit into
mainfrom
fix/playground-issues-1-10
Apr 10, 2026
Merged

fix: resolve all 10 open playground issues#11
scottyg merged 1 commit into
mainfrom
fix/playground-issues-1-10

Conversation

@scottyg

@scottyg scottyg commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Closes #1, closes #2, closes #3, closes #4, closes #5, closes #6, closes #7, closes #8, closes #9, closes #10

Test plan

  • go build ./... and go vet ./... pass cleanly
  • Verify rate limiting triggers 429 after burst on /api/run
  • Verify snippet IDs with non-alphanumeric chars are rejected
  • Verify client disconnect cancels running Docker container
  • Verify LogCompile errors appear in server logs when DB is unavailable

🤖 Generated with Claude Code

- #1/#2: Add per-IP rate limiting (10 req/min) on /api/run and /api/share
- #3: Validate snippet IDs against base62 regex, not just length
- #4: Add snippet count cap (100k) with automatic pruning of oldest 10%
- #5: Check and log errors from LogCompile database writes
- #6: Propagate HTTP request context to runCode for client disconnect cancellation
- #7: Configure SQLite connection pool (MaxOpenConns=1) and verify with Ping
- #8: Simplify CompileSuccess to ExitCode == 0, removing fragile string matching
- #9: Check and log JSON encoding errors on all HTTP responses
- #10: Add 5s timeout to Docker container cleanup and log failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@scottyg scottyg merged commit c7cb212 into main Apr 10, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment