Skip to content

feat(allowedsymbols): add safety emoji legend to symbol comments and package README#126

Merged
AlexandreYang merged 2 commits intomainfrom
alex/allowed_symbols_emoji
Mar 18, 2026
Merged

feat(allowedsymbols): add safety emoji legend to symbol comments and package README#126
AlexandreYang merged 2 commits intomainfrom
alex/allowed_symbols_emoji

Conversation

@AlexandreYang
Copy link
Copy Markdown
Member

@AlexandreYang AlexandreYang commented Mar 18, 2026

  • Prefix every inline symbol comment in allowedsymbols/*.go with a traffic-light safety emoji: 🟢 pure/no side effects, 🟠 read-only OS/filesystem/syscall access or delegated writes, 🔴 network I/O or unsafe memory/native code loading
  • Add allowedsymbols/README.md with a short package overview: purpose, permanently banned packages, allowlist structure, two-layer builtin system, safety legend, enforcement mechanics, and instructions for adding new symbols

…package README

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@AlexandreYang AlexandreYang marked this pull request as ready for review March 18, 2026 19:15
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@AlexandreYang AlexandreYang added this pull request to the merge queue Mar 18, 2026
Merged via the queue into main with commit 160d1bd Mar 18, 2026
31 checks passed
@AlexandreYang AlexandreYang deleted the alex/allowed_symbols_emoji branch March 18, 2026 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants