Skip to content

/purge feature spec#6

Open
its-everdred wants to merge 4 commits into
its-everdred:mainfrom
its-applekid:purge-feature-spec
Open

/purge feature spec#6
its-everdred wants to merge 4 commits into
its-everdred:mainfrom
its-applekid:purge-feature-spec

Conversation

@its-everdred
Copy link
Copy Markdown
Owner

  • Comprehensive spec for automated message cleanup feature
  • Progressive 3-phase rollout: notify → manual → auto-delete
  • Per-channel retention policies (configurable days)
  • Scheduled checks with mod notifications
  • Commands: /mod purge view/set/execute/disable
  • Safety features: confirmation prompts, audit log, rate limiting
  • Database schema for config and history
  • Discord API considerations (bulk delete, rate limits)
  • Testing plan and future enhancements

- Comprehensive spec for automated message cleanup feature
- Progressive 3-phase rollout: notify → manual → auto-delete
- Per-channel retention policies (configurable days)
- Scheduled checks with mod notifications
- Commands: /mod purge view/set/execute/disable
- Safety features: confirmation prompts, audit log, rate limiting
- Database schema for config and history
- Discord API considerations (bulk delete, rate limits)
- Testing plan and future enhancements
- Removed PURGE_PHASE environment variable
- Phases are now a recommended build order, not deployment gates
- Updated implementation notes to focus on incremental development
- Clarified Phase 2/3 features available when implementation complete
- Updated Quick Start and configuration examples
- Added Privacy & Security section to spec
- Clarified READ_MESSAGE_HISTORY grants access to content, but implementation must not use it
- Bot only accesses message ID and timestamp (metadata)
- Message content, author, attachments never read or stored
- Updated deletion logic and message fetching with privacy notes
- Added implementation pattern examples (correct vs incorrect)
- Audit log only stores counts and timestamp ranges, never message data
- Updated summary with privacy section for quick reference
- Reduced PurgeFeatureSpec.md from ~10KB to ~6KB
- Reduced PurgeFeatureSummary.md from ~4.4KB to ~2.9KB
- Removed redundant examples and verbose explanations
- Kept all essential info: commands, schema, privacy, implementation
- Merged related sections for clarity
- More direct, concise writing style
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.

2 participants