Skip to content

Conversation

@kzndotsh
Copy link
Contributor

@kzndotsh kzndotsh commented Aug 10, 2025

@sourcery-ai summary

@kzndotsh kzndotsh changed the title V0.1.0 DRAFT: v0.1.0 Aug 10, 2025
@kzndotsh kzndotsh self-assigned this Aug 10, 2025
@kzndotsh kzndotsh marked this pull request as ready for review August 11, 2025 12:50
sourcery-ai[bot]

This comment was marked as outdated.

sourcery-ai[bot]

This comment was marked as outdated.

sourcery-ai[bot]

This comment was marked as resolved.

SourceryAI

This comment was marked as resolved.

@allthingslinux allthingslinux deleted a comment from cursor bot Aug 12, 2025
@kzndotsh kzndotsh requested a review from SourceryAI August 14, 2025 12:35
sourcery-ai[bot]

This comment was marked as resolved.

SourceryAI

This comment was marked as resolved.

@allthingslinux allthingslinux deleted a comment from cursor bot Aug 15, 2025
sourcery-ai[bot]

This comment was marked as resolved.

SourceryAI

This comment was marked as resolved.

@github-actions

This comment was marked as resolved.

@allthingslinux allthingslinux deleted a comment from github-actions bot Aug 23, 2025
@allthingslinux allthingslinux deleted a comment from github-actions bot Aug 23, 2025
@allthingslinux allthingslinux deleted a comment from github-actions bot Aug 23, 2025
github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

markdownlint

[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 8. Security"]


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Database credentials man..."]

- [ ] Database credentials management


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 9. Performance & Monitoring"]

### 9. Performance & Monitoring


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Query performance monito..."]

- [ ] Query performance monitoring


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 10. Production Readiness"]

### 10. Production Readiness


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Backup and recovery proc..."]

- [ ] Backup and recovery procedures


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 11. Error Handling & Resilience"]

### 11. Error Handling & Resilience


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Database connection fail..."]

- [ ] Database connection failure handling


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 12. Documentation & Maintenance"]

### 12. Documentation & Maintenance


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Database schema document..."]

- [ ] Database schema documentation


[markdownlint] reported by reviewdog 🐶
MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Overall Assessment: 🟢 GOOD FO..."]

**Overall Assessment: 🟢 GOOD FOUNDATION with some IMPORTANT improvements needed**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Database accessible from any..."]

- Database accessible from any network interface


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```conf"]


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Dictionary attacks possible"]

- Dictionary attacks possible


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Use strong, randomly generat..."]

- Use strong, randomly generated passwords (32+ characters)


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```python"]


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- May overwhelm database in hi..."]

- May overwhelm database in high-traffic scenarios


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Exponential backoff for conn..."]

- Exponential backoff for connection retries


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Slow query log analysis"]

- Slow query log analysis


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Automated backup scripts"]

- Automated backup scripts


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### ✅ Architecture: Clean Separation"]

### **Architecture: Clean Separation**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Unit tests (py-pglite) vs In..."]

- Unit tests (py-pglite) vs Integration tests (Docker) perfectly separated


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### ✅ Configuration: Smart Environment Handling"]

### **Configuration: Smart Environment Handling**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Automatic URL construction f..."]

- Automatic URL construction from individual variables


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### ✅ Performance: Good Connection Pooling"]

### **Performance: Good Connection Pooling**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Reasonable pool sizes for de..."]

- Reasonable pool sizes for development


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### ✅ Testing: Excellent Test Infrastructure"]

### **Testing: Excellent Test Infrastructure**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- py-pglite for fast unit test..."]

- py-pglite for fast unit tests (10-100x faster)


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### ✅ Migrations: Well-Configured Alembic"]

### **Migrations: Well-Configured Alembic**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Proper sync/async URL conver..."]

- Proper sync/async URL conversion


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Phase 1: Security (Week 1)"]

### **Phase 1: Security (Week 1)**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "1. 🔴 Fix PostgreSQL listen_ad..."]

1. 🔴 Fix PostgreSQL listen_addresses (docker/postgres/postgresql.conf)


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Phase 2: Reliability (Week 2)"]

### **Phase 2: Reliability (Week 2)**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "1. 🟡 Implement circuit breake..."]

1. 🟡 Implement circuit breaker pattern


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Phase 3: Operations (Week 3)"]

### **Phase 3: Operations (Week 3)**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "1. 🟡 Set up automated backups"]

1. 🟡 Set up automated backups


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Phase 4: Optimization (Ongoing)"]

### **Phase 4: Optimization (Ongoing)**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "1. ℹ️ Performance tuning based..."]

1. ℹ️ Performance tuning based on metrics


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Current State: 🟢 GOOD for Development"]

### **Current State: 🟢 GOOD for Development**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Excellent testing infrastruc..."]

- Excellent testing infrastructure


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Production Readiness: 🟡 NEEDS IMPORTANT FIXES"]

### **Production Readiness: 🟡 NEEDS IMPORTANT FIXES**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- **Security issues must be re..."]

- **Security issues must be resolved** before any public deployment


[markdownlint] reported by reviewdog 🐶
MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "1. Delete the migration file"]

1. Delete the migration file


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- The baseline migration estab..."]

- The baseline migration establishes Alembic's version tracking


[markdownlint] reported by reviewdog 🐶
MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```"]


[markdownlint] reported by reviewdog 🐶
MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]


[markdownlint] reported by reviewdog 🐶
MD013/line-length Line length [Expected: 100; Actual: 108]

**Symptoms**: `alembic revision --autogenerate -m "baseline"` creates a migration with `def upgrade(): pass`


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Package name: Install `p..."]

- **Package name**: Install `psycopg[binary]` (not `psycopg3`)


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Use options string instead..."]

- Use `options` string instead of `server_settings` dict


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```python"]


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```python"]


[markdownlint] reported by reviewdog 🐶
MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Contributors can safely ..."]

- **Contributors** can safely develop and test database changes


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- All commands should complete..."]

- All commands should complete without errors


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Current PostgreSQL settings"]

- Current PostgreSQL settings


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```"]


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```"]


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```"]


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Most settings are managed au..."]

- Most settings are managed automatically


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 📅 Daily Tasks:"]

### **📅 Daily Tasks:**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 📅 Weekly Tasks:"]

### **📅 Weekly Tasks:**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 📅 Monthly Tasks:"]

### **📅 Monthly Tasks:**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 📅 As Needed:"]

### **📅 As Needed:**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 📊 Health Monitoring:"]

### **📊 Health Monitoring:**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 🔧 Maintenance Operations:"]

### **🔧 Maintenance Operations:**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 📋 Information and Analysis:"]

### **📋 Information and Analysis:**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Problem: Slow Queries"]

### **Problem: Slow Queries**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Problem: High Memory Usage"]

### **Problem: High Memory Usage**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Problem: Tables Not Being Maintained"]

### **Problem: Tables Not Being Maintained**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Problem: Indexes Not Being Used"]

### **Problem: Indexes Not Being Used**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Connection Pooling:"]

### **Connection Pooling:**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```ini"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Partitioning:"]

### **Partitioning:**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```sql"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Parallel Query Processing:"]

### **Parallel Query Processing:**


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```ini"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### PostgreSQL Documentation:"]

### **PostgreSQL Documentation:**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [Performance Tuning](https:/..."]

- [Performance Tuning](https://www.postgresql.org/docs/current/runtime-config-query.html)


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Tux-Specific Commands:"]

### **Tux-Specific Commands:**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- make help-db - List all da..."]

- `make help-db` - List all database commands


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### External Tools:"]

### **External Tools:**


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- pgAdmin - GUI database a..."]

- **pgAdmin** - GUI database administration


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Prerequisites"]


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Fresh system/VM with no ..."]

- [ ] Fresh system/VM with no previous Tux installation


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 1. Environment Setup"]

### 1. Environment Setup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Install uv: `curl -LsS..."]

- [ ] Install `uv`: `curl -LsSf https://astral.sh/uv/install.sh | sh`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 2. Repository Setup"]

### 2. Repository Setup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] `git clone https://githu..."]

- [ ] `git clone https://github.com/allthingslinux/tux.git`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 3. Configuration Setup"]

### 3. Configuration Setup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] cp env.example .env"]

- [ ] `cp env.example .env`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 4. Dependencies & Environment"]

### 4. Dependencies & Environment


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] uv sync"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 5. Database Setup"]

### 5. Database Setup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Ensure PostgreSQL is run..."]

- [ ] Ensure PostgreSQL is running (or SQLite file is writable)


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 6. Bot Startup"]

### 6. Bot Startup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] make dev or `make star..."]

- [ ] `make dev` or `make start`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 7. Feature Testing"]

### 7. Feature Testing


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Test basic commands: `!h..."]

- [ ] Test basic commands: `!help`, `!ping`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 8. New Server Scenario Testing"]

### 8. New Server Scenario Testing


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Test bot joining new ser..."]

- [ ] Test bot joining new server (if possible)


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 1. Environment Setup"]

### 1. Environment Setup


[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### 1. Environment Setup"]

### 1. Environment Setup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Install Docker & Docker ..."]

- [ ] Install Docker & Docker Compose v2


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 2. Repository Setup"]

### 2. Repository Setup


[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### 2. Repository Setup"]

### 2. Repository Setup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] `git clone https://githu..."]

- [ ] `git clone https://github.com/allthingslinux/tux.git`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 3. Configuration Setup"]

### 3. Configuration Setup


[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### 3. Configuration Setup"]

### 3. Configuration Setup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] cp env.example .env"]

- [ ] `cp env.example .env`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 4. Docker Startup"]

### 4. Docker Startup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] make docker-dev or `ma..."]

- [ ] `make docker-dev` or `make prod`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 5. Testing"]


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Check logs: `docker comp..."]

- [ ] Check logs: `docker compose logs -f`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 1. Environment Setup"]

### 1. Environment Setup


[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### 1. Environment Setup"]

### 1. Environment Setup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Install Docker & Docker ..."]

- [ ] Install Docker & Docker Compose v2


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 2. Repository Setup"]

### 2. Repository Setup


[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### 2. Repository Setup"]

### 2. Repository Setup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] `git clone https://githu..."]

- [ ] `git clone https://github.com/allthingslinux/tux.git`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 3. Configuration Setup"]

### 3. Configuration Setup


[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### 3. Configuration Setup"]

### 3. Configuration Setup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] cp env.example .env"]

- [ ] `cp env.example .env`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 4. Docker Production Startup"]

### 4. Docker Production Startup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] make docker-prod or `m..."]

- [ ] `make docker-prod` or `make prod`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 5. Production Verification"]

### 5. Production Verification


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Check logs: `docker comp..."]

- [ ] Check logs: `docker compose logs -f`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Environment Variables"]

### Environment Variables


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] ENV variable works cor..."]

- [ ] `ENV` variable works correctly (dev/prod/test)


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Bot Configuration"]

### Bot Configuration


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Bot name and version dis..."]

- [ ] Bot name and version display correctly


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Database Configuration"]

### Database Configuration


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Connection established s..."]

- [ ] Connection established successfully


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Local Development"]

### Local Development


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Stop bot: Ctrl+C"]

- [ ] Stop bot: `Ctrl+C`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Docker Development"]

### Docker Development


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Stop containers: `docker..."]

- [ ] Stop containers: `docker compose down`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Production"]


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Stop containers: `docker..."]

- [ ] Stop containers: `docker compose down`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Python/UV Issues"]

### Python/UV Issues


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Problem: `uv: command no..."]

- **Problem**: `uv: command not found`


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Database Issues"]

### Database Issues


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Problem: Migration failu..."]

- **Problem**: Migration failures during startup


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Database Issues"]

### Database Issues


[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### Database Issues"]

### Database Issues


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Problem: Connection refu..."]

- **Problem**: Connection refused


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Docker Issues"]

### Docker Issues


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Problem: Port conflicts"]

- **Problem**: Port conflicts


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Bot Issues"]


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Problem: Bot doesn't con..."]

- **Problem**: Bot doesn't connect


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Developer Setup"]

### Developer Setup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Bot connects to Discord ..."]

- [ ] Bot connects to Discord successfully


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Production Setup"]

### Production Setup


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Bot runs in production m..."]

- [ ] Bot runs in production mode


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### All Setups"]


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Configuration loads with..."]

- [ ] Configuration loads without errors


[markdownlint] reported by reviewdog 🐶
MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### All Tests Pass (6/6)"]

### All Tests Pass (6/6)


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Configuration loads correctl..."]

- Configuration loads correctly


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Some Tests Fail (1-5/6)"]

### Some Tests Fail (1-5/6)


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Check the specific failing t..."]

- Check the specific failing tests


[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### All Tests Fail (0/6)"]

### All Tests Fail (0/6)


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Verify Python environment"]

- Verify Python environment


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Automatic migrations: Bo..."]

- **Automatic migrations**: Bot runs migrations automatically on startup in production


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

tux/README.md

Line 100 in 0cb9e17

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

tux/README.md

Line 106 in 0cb9e17

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

tux/README.md

Line 111 in 0cb9e17

```bash


[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]

tux/README.md

Line 117 in 0cb9e17

```bash

kzndotsh and others added 8 commits November 19, 2025 20:27
- Updated Codecov configuration to support unified coverage across all test types (unit, integration, e2e) with comprehensive reporting.
- Added new component-specific coverage requirements for better granularity.
- Introduced a new build script for Cloudflare Workers to automate test execution and documentation generation with fresh coverage reports.
- Updated GitHub Actions workflows to streamline test execution and coverage reporting, consolidating unit, integration, and e2e tests into a single job.
- Added coverage dependency to pyproject.toml and updated related documentation.
- Changed references from "audit log" to "mod log" in moderation-related code and tests for consistency.
- Added checks to ensure database availability in integration tests.
- Updated test cases to reflect changes in logging behavior and improve overall test reliability.
- Modified help command usage to reflect the dynamic prefix format.
- Updated error messages in eval and info modules to use the dynamic prefix.
- Adjusted AFK command to check for the dynamic prefix when processing messages.
- Enhanced error handling in the mock module to include dynamic prefix in usage instructions.
- Introduced new categories for error handling, including HTTPX Errors and Client Connection errors.
- Improved the dynamic test case generation for missing exceptions to ensure comprehensive coverage.
- Updated error configuration mapping to better reflect the hierarchy and specific error types.
- Enhanced documentation for error configuration to clarify the organization and fallback behavior.
@sentry
Copy link

sentry bot commented Nov 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.72%. Comparing base (d6bf84d) to head (5726dad).
⚠️ Report is 3 commits behind head on main.

❌ Your project check has failed because the head coverage (38.72%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##            main    #1020       +/-   ##
==========================================
+ Coverage   9.65%   38.72%   +29.06%     
==========================================
  Files        123      206       +83     
  Lines      10427    14788     +4361     
  Branches    1281     1762      +481     
==========================================
+ Hits        1007     5727     +4720     
+ Misses      9311     9061      -250     
+ Partials     109        0      -109     
Flag Coverage Δ
database ?
integration ?
unit ?

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

kzndotsh and others added 11 commits November 20, 2025 16:44
- Updated CI workflows (ci.yml, docker.yml, security.yml, tests.yml) to include 'permissions' for read access to contents in file detection jobs.
…nd configuration

- Modified assertions in end-to-end and unit tests to replace "Command Permissions" with "configure command permissions" for clarity.
- Enhanced test setup by mocking bot and guild objects to facilitate prefix manager access in error handling tests.
- Updated the fallback_format_message function to return a default error message when formatting fails, enhancing user experience.
- Modified the error formatter to prevent conflicts by excluding 'error' from kwargs when calling the detail extractor.
…ents

- Added a detailed roadmap for enhancing error handling, including documentation, user experience, performance, monitoring, configuration, and security.
- Introduced high-priority items such as smart command suggestions, error analytics dashboard, and per-guild customization to improve user experience and system reliability.
- Simplified the dependency installation process by removing the test-specific flag from the uv sync command.
- Updated the script to reflect a more general installation approach for dependencies.
…ndency

- Removed the coverage dependency from pyproject.toml.
- Updated versions for asgiref, cfgv, fastapi, redis, starlette, and tinycss2 in uv.lock to their latest releases.
- Introduced a new GitHub Actions workflow for building and deploying documentation.
- The workflow triggers on pushes and pull requests to the main and version branches, detecting changes in relevant files.
- Added steps for checking documentation and code changes, building MkDocs documentation, and deploying to Cloudflare Workers.
- Removed the outdated build script from the docs directory.
- Updated the names of various components in the documentation workflow for improved clarity, changing 'Documentation' to 'Docs' and 'Build Documentation' to 'Build'.
- Adjusted artifact names from 'documentation' to 'docs' for consistency.
- Enhanced deployment job naming by simplifying 'Deploy to Cloudflare Workers' to 'Deploy'.
- Modified test coverage action name from 'tux-coverage' to 'tux' for brevity.
- Replaced the cast of discord.User with ctx.author when creating the DM embed in the CommunicationService class.
- This change simplifies the code and ensures the correct user context is used for sending moderation DMs.
- Updated action versions in CI workflows to use the latest releases for improved performance and security.
- Refactored scripts for better organization and clarity, including the addition of new helper scripts for CI, cleanup, and maintenance tasks.
- Improved documentation and comments within workflow files to clarify the purpose and functionality of each step.
- Adjusted output handling in CI workflows to streamline change detection and reporting.
]


class EncodeDecode(BaseCog):
# -- DO NOT CHANGE ANYTHING BELOW THIS LINE --


class FlagRemover(BaseCog):
# -- DO NOT CHANGE ANYTHING BELOW THIS LINE --


class HarmfulCommands(BaseCog):
# TODO: Figure out how to make rolecount work without hard coded ids and icons


class RoleCount(BaseCog):
# -- DO NOT CHANGE ANYTHING BELOW THIS LINE --


class SupportNotifier(BaseCog):

class TtyRoles(BaseCog):

# Determine category based on exception hierarchy and functionality
if error_type.__module__.startswith("discord.app_commands"):
category = "Application Commands"
"NotOwner",
"NSFWChannelRequired",
]:
category = "Check Failures"
"MissingFlagArgument",
"TooManyFlags",
]:
category = "Flag Errors"
- Changed shfmt_flags in the CI workflow to include the -ln bash option, ensuring compatibility with bash scripts.
- Renamed regex variables for consistency, changing names to include the _RE suffix.
- Added new regex patterns for GitHub and GitLab URLs, enhancing the module's capabilities for parsing various links.
- Improved documentation within the regex module to clarify the purpose of each pattern.
- Added input sanitization for the language parameter in TuxUnsupportedLanguageError to prevent formatting issues and potential security risks.
- Extracted the first word of the language input and truncated it to a maximum of 30 characters for safer error message generation.
- Simplified the info command's documentation for clarity and conciseness.
- Adjusted the order of converter checks to prioritize member/user IDs over guild IDs.
- Enhanced handling of potential guild IDs, ensuring checks are performed only after other converters fail.
- Added conditional inclusion of InviteConverter based on entity length and format.
- Improved error messaging for cases where no information can be found about the provided entity.
Comment on lines +66 to +83
name: Wait for Tests
runs-on: ubuntu-latest
if: needs.changes.outputs.any == 'true' || github.event_name == 'workflow_dispatch'
needs: [changes]
steps:
# Wait for tests workflow to complete before building docs
# Note: This step requires GitHub API access and won't work with act (local testing)
# It will fail gracefully in act, allowing manual testing of docs build steps
- name: Wait for Tests Workflow
uses: lewagon/[email protected]
continue-on-error: true
with:
ref: ${{ github.event.pull_request.head.sha || github.sha }}
check-name: Run All Tests
repo-token: ${{ secrets.GITHUB_TOKEN }}
wait-interval: ${{ env.WAIT_INTERVAL_SECONDS }}
allowed-conclusions: success
build:

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

Copilot Autofix

AI 3 days ago

To fix the issue, you should add a permissions block to the wait job in .github/workflows/docs.yml. The token should have contents: read permission by default unless the job needs greater access. In this case, the lewagon/wait-on-check-action only requires permission to read repository metadata/status—contents: read is the minimal recommended setting. You should add the following block to the wait job, aligned and indented correctly:

permissions:
  contents: read

Insert this block immediately after the runs-on line (line 67) but before any other keys for the job (such as if, needs, or steps).

Suggested changeset 1
.github/workflows/docs.yml

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/.github/workflows/docs.yml b/.github/workflows/docs.yml
--- a/.github/workflows/docs.yml
+++ b/.github/workflows/docs.yml
@@ -65,6 +65,8 @@
   wait:
     name: Wait for Tests
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
     if: needs.changes.outputs.any == 'true' || github.event_name == 'workflow_dispatch'
     needs: [changes]
     steps:
EOF
@@ -65,6 +65,8 @@
wait:
name: Wait for Tests
runs-on: ubuntu-latest
permissions:
contents: read
if: needs.changes.outputs.any == 'true' || github.event_name == 'workflow_dispatch'
needs: [changes]
steps:
Copilot is powered by AI and may make mistakes. Always verify output.
)

URL_NO_PROTOCOL_RE = re.compile(
r"[-a-zA-Z0-9@:%._\+~#=]{1,256}\.[a-zA-Z0-9()]{1,6}\b([-a-zA-Z0-9()@:%_\+.~#?&//=]*)",
- Downgraded Pillow version to ensure compatibility, specifying a range of >=10.2.0,<12.0.0.
- Updated MkDocs Material to include optional imaging support in the dependencies.
- Adjusted related documentation and configuration files to reflect these changes.
- Updated shfmt_flags in the CI workflow by removing the -p option, streamlining the configuration for better clarity and functionality.
…n system

- Introduced a frozenset of restricted commands that cannot be assigned to permission ranks, ensuring only owners and sysadmins can use them.
- Updated the PermissionSystem to raise a ValueError when attempting to assign ranks to restricted commands.
- Enhanced HelpData and HelpNavigation to filter out restricted commands based on user permissions.
- Added tests to verify that restricted commands are correctly handled and not configurable via the permission system.
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.

7 participants