Repository navigation
Remove unused TWILIO_ACCOUNT_SID / TWILIO_AUTH_TOKEN from docs - #28
Conversation
No code in this repo has ever read either variable. They came from the original project plan's 'outbound functionality' placeholder, which was never defined or built; the app answers webhooks with TWiML and needs no Twilio credentials. Listing them invites operators to copy the account's auth token into every deployment for no benefit.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
Summary
TWILIO_ACCOUNT_SIDandTWILIO_AUTH_TOKENfrom.env.example, the README config table and example, andCLAUDE.md.git log -G'Getenv\("TWILIO'andgit grepacross every revision). They came fromproject-plan.mdin the first commit (b9663ac, 2025-06-13), which listed them under "Twilio (for outbound functionality)". That feature was never defined or built. The plan was deleted in93e6965, but the variables stayed in the docs.twilio-godependency is used only for itstwimlXML builder, not the REST client, so the app needs no Twilio credentials. Listing them encouraged operators (including OBACloud's template) to copy the account's auth token, which has full control of the account, into every deployment.Test plan
git grep TWILIO_outsidedocs/superpowers/**returns nothingmake testpasses (docs-only change)Review notes
Summary by CodeRabbit
CLAUDE.md, and.env.example. These credentials are no longer listed in the project’s setup documentation.