-
-
Notifications
You must be signed in to change notification settings - Fork 41
v0.1.0rc5 #1020
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
src/tux/database/migrations/versions/2025_08_19_0437-12574673e637_initial_baseline_migration.py
Outdated
Show resolved
Hide resolved
src/tux/database/migrations/versions/2025_08_19_0437-12574673e637_initial_baseline_migration.py
Outdated
Show resolved
Hide resolved
src/tux/database/migrations/versions/2025_08_19_0437-12574673e637_initial_baseline_migration.py
Outdated
Show resolved
Hide resolved
src/tux/database/migrations/versions/2025_08_19_0437-12574673e637_initial_baseline_migration.py
Outdated
Show resolved
Hide resolved
src/tux/database/migrations/versions/2025_08_19_0437-12574673e637_initial_baseline_migration.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this 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"]
| ### 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"]
| ```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"]
| ```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"]
tux/docs/content/db/database-lifecycle.md
Line 79 in 0cb9e17
| 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..."]
tux/docs/content/db/database-lifecycle.md
Line 156 in 0cb9e17
| - 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: "```"]
tux/docs/content/db/database-lifecycle.md
Line 162 in 0cb9e17
| ``` |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-lifecycle.md
Line 224 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-lifecycle.md
Line 244 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-lifecycle.md
Line 260 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```"]
tux/docs/content/db/database-lifecycle.md
Line 281 in 0cb9e17
| ``` |
[markdownlint] reported by reviewdog 🐶
MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
tux/docs/content/db/database-lifecycle.md
Line 281 in 0cb9e17
| ``` |
[markdownlint] reported by reviewdog 🐶
MD013/line-length Line length [Expected: 100; Actual: 108]
tux/docs/content/db/database-lifecycle.md
Line 291 in 0cb9e17
| **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"]
tux/docs/content/db/database-lifecycle.md
Line 311 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Package name: Install `p..."]
tux/docs/content/db/database-lifecycle.md
Line 380 in 0cb9e17
| - **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..."]
tux/docs/content/db/database-lifecycle.md
Line 427 in 0cb9e17
| - 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"]
tux/docs/content/db/database-lifecycle.md
Line 434 in 0cb9e17
| ```python |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-lifecycle.md
Line 446 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```python"]
tux/docs/content/db/database-lifecycle.md
Line 455 in 0cb9e17
| ```python |
[markdownlint] reported by reviewdog 🐶
MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
tux/docs/content/db/database-lifecycle.md
Line 515 in 0cb9e17
| ``` |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Contributors can safely ..."]
tux/docs/content/db/database-lifecycle.md
Line 524 in 0cb9e17
| - **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..."]
tux/docs/content/db/database-lifecycle.md
Line 569 in 0cb9e17
| - 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"]
| ```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"]
| ```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"]
| ```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..."]
tux/docs/content/db/database-optimization.md
Line 134 in 0cb9e17
| - 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:"]
tux/docs/content/db/database-optimization.md
Line 140 in 0cb9e17
| ### **📅 Daily Tasks:** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-optimization.md
Line 141 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 📅 Weekly Tasks:"]
tux/docs/content/db/database-optimization.md
Line 149 in 0cb9e17
| ### **📅 Weekly Tasks:** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-optimization.md
Line 150 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 📅 Monthly Tasks:"]
tux/docs/content/db/database-optimization.md
Line 158 in 0cb9e17
| ### **📅 Monthly Tasks:** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-optimization.md
Line 159 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 📅 As Needed:"]
tux/docs/content/db/database-optimization.md
Line 167 in 0cb9e17
| ### **📅 As Needed:** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-optimization.md
Line 168 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 📊 Health Monitoring:"]
tux/docs/content/db/database-optimization.md
Line 181 in 0cb9e17
| ### **📊 Health Monitoring:** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-optimization.md
Line 182 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### 🔧 Maintenance Operations:"]
tux/docs/content/db/database-optimization.md
Line 193 in 0cb9e17
| ### **🔧 Maintenance Operations:** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-optimization.md
Line 194 in 0cb9e17
| ```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:"]
tux/docs/content/db/database-optimization.md
Line 205 in 0cb9e17
| ### **📋 Information and Analysis:** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-optimization.md
Line 206 in 0cb9e17
| ```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"]
tux/docs/content/db/database-optimization.md
Line 245 in 0cb9e17
| ### **Problem: Slow Queries** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-optimization.md
Line 246 in 0cb9e17
| ```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"]
tux/docs/content/db/database-optimization.md
Line 257 in 0cb9e17
| ### **Problem: High Memory Usage** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-optimization.md
Line 258 in 0cb9e17
| ```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"]
tux/docs/content/db/database-optimization.md
Line 266 in 0cb9e17
| ### **Problem: Tables Not Being Maintained** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-optimization.md
Line 267 in 0cb9e17
| ```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"]
tux/docs/content/db/database-optimization.md
Line 275 in 0cb9e17
| ### **Problem: Indexes Not Being Used** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
tux/docs/content/db/database-optimization.md
Line 276 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Connection Pooling:"]
tux/docs/content/db/database-optimization.md
Line 286 in 0cb9e17
| ### **Connection Pooling:** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```ini"]
tux/docs/content/db/database-optimization.md
Line 288 in 0cb9e17
| ```ini |
[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### Partitioning:"]
tux/docs/content/db/database-optimization.md
Line 299 in 0cb9e17
| ### **Partitioning:** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```sql"]
tux/docs/content/db/database-optimization.md
Line 301 in 0cb9e17
| ```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:"]
tux/docs/content/db/database-optimization.md
Line 312 in 0cb9e17
| ### **Parallel Query Processing:** |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```ini"]
tux/docs/content/db/database-optimization.md
Line 314 in 0cb9e17
| ```ini |
[markdownlint] reported by reviewdog 🐶
MD022/blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "### PostgreSQL Documentation:"]
tux/docs/content/db/database-optimization.md
Line 324 in 0cb9e17
| ### **PostgreSQL Documentation:** |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [Performance Tuning](https:/..."]
tux/docs/content/db/database-optimization.md
Line 325 in 0cb9e17
| - [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/docs/content/db/database-optimization.md
Line 329 in 0cb9e17
| ### **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..."]
tux/docs/content/db/database-optimization.md
Line 330 in 0cb9e17
| - `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:"]
tux/docs/content/db/database-optimization.md
Line 334 in 0cb9e17
| ### **External Tools:** |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- pgAdmin - GUI database a..."]
tux/docs/content/db/database-optimization.md
Line 335 in 0cb9e17
| - **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"]
| ### 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 18 in 0cb9e17
| ### 1. Environment Setup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Install uv: `curl -LsS..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 19 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 23 in 0cb9e17
| ### 2. Repository Setup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] `git clone https://githu..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 24 in 0cb9e17
| - [ ] `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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 28 in 0cb9e17
| ### 3. Configuration Setup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] cp env.example .env"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 29 in 0cb9e17
| - [ ] `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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 37 in 0cb9e17
| ### 4. Dependencies & Environment |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] uv sync"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 38 in 0cb9e17
| - [ ] `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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 43 in 0cb9e17
| ### 5. Database Setup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Ensure PostgreSQL is run..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 44 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 50 in 0cb9e17
| ### 6. Bot Startup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] make dev or `make star..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 51 in 0cb9e17
| - [ ] `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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 56 in 0cb9e17
| ### 7. Feature Testing |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Test basic commands: `!h..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 57 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 64 in 0cb9e17
| ### 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..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 65 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 75 in 0cb9e17
| ### 1. Environment Setup |
[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### 1. Environment Setup"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 75 in 0cb9e17
| ### 1. Environment Setup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Install Docker & Docker ..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 76 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 80 in 0cb9e17
| ### 2. Repository Setup |
[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### 2. Repository Setup"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 80 in 0cb9e17
| ### 2. Repository Setup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] `git clone https://githu..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 81 in 0cb9e17
| - [ ] `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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 85 in 0cb9e17
| ### 3. Configuration Setup |
[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### 3. Configuration Setup"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 85 in 0cb9e17
| ### 3. Configuration Setup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] cp env.example .env"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 86 in 0cb9e17
| - [ ] `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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 92 in 0cb9e17
| ### 4. Docker Startup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] make docker-dev or `ma..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 93 in 0cb9e17
| - [ ] `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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 100 in 0cb9e17
| ### 5. Testing |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Check logs: `docker comp..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 101 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 109 in 0cb9e17
| ### 1. Environment Setup |
[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### 1. Environment Setup"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 109 in 0cb9e17
| ### 1. Environment Setup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Install Docker & Docker ..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 110 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 114 in 0cb9e17
| ### 2. Repository Setup |
[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### 2. Repository Setup"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 114 in 0cb9e17
| ### 2. Repository Setup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] `git clone https://githu..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 115 in 0cb9e17
| - [ ] `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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 120 in 0cb9e17
| ### 3. Configuration Setup |
[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### 3. Configuration Setup"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 120 in 0cb9e17
| ### 3. Configuration Setup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] cp env.example .env"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 121 in 0cb9e17
| - [ ] `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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 128 in 0cb9e17
| ### 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..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 129 in 0cb9e17
| - [ ] `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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 136 in 0cb9e17
| ### 5. Production Verification |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Check logs: `docker comp..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 137 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 147 in 0cb9e17
| ### Environment Variables |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] ENV variable works cor..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 148 in 0cb9e17
| - [ ] `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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 153 in 0cb9e17
| ### Bot Configuration |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Bot name and version dis..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 154 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 159 in 0cb9e17
| ### Database Configuration |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Connection established s..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 160 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 172 in 0cb9e17
| ### Local Development |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Stop bot: Ctrl+C"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 173 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 178 in 0cb9e17
| ### Docker Development |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Stop containers: `docker..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 179 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 184 in 0cb9e17
| ### Production |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Stop containers: `docker..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 185 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 194 in 0cb9e17
| ### Python/UV Issues |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Problem: `uv: command no..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 195 in 0cb9e17
| - **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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 201 in 0cb9e17
| ### Database Issues |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Problem: Migration failu..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 202 in 0cb9e17
| - **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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 211 in 0cb9e17
| ### Database Issues |
[markdownlint] reported by reviewdog 🐶
MD024/no-duplicate-heading Multiple headings with the same content [Context: "### Database Issues"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 211 in 0cb9e17
| ### Database Issues |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Problem: Connection refu..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 212 in 0cb9e17
| - **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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 218 in 0cb9e17
| ### Docker Issues |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Problem: Port conflicts"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 219 in 0cb9e17
| - **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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 225 in 0cb9e17
| ### Bot Issues |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Problem: Bot doesn't con..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 226 in 0cb9e17
| - **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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 236 in 0cb9e17
| ### Developer Setup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Bot connects to Discord ..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 237 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 243 in 0cb9e17
| ### Production Setup |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Bot runs in production m..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 244 in 0cb9e17
| - [ ] 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"]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 250 in 0cb9e17
| ### All Setups |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- [ ] Configuration loads with..."]
tux/docs/content/SETUP_TEST_CHECKLIST.md
Line 251 in 0cb9e17
| - [ ] Configuration loads without errors |
[markdownlint] reported by reviewdog 🐶
MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
tux/docs/content/SETUP_TESTING.md
Line 33 in 0cb9e17
| ``` |
[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)"]
tux/docs/content/SETUP_TESTING.md
Line 166 in 0cb9e17
| ### All Tests Pass (6/6) |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Configuration loads correctl..."]
tux/docs/content/SETUP_TESTING.md
Line 168 in 0cb9e17
| - 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)"]
tux/docs/content/SETUP_TESTING.md
Line 173 in 0cb9e17
| ### 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..."]
tux/docs/content/SETUP_TESTING.md
Line 175 in 0cb9e17
| - 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)"]
tux/docs/content/SETUP_TESTING.md
Line 180 in 0cb9e17
| ### All Tests Fail (0/6) |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Verify Python environment"]
tux/docs/content/SETUP_TESTING.md
Line 182 in 0cb9e17
| - Verify Python environment |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 10 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 17 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 23 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 34 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 40 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 46 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- Automatic migrations: Bo..."]
Line 163 in 0cb9e17
| - **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"]
Line 212 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 217 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 222 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 231 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 237 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 243 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 260 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 266 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 271 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 282 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 287 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 292 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 301 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 308 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 315 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 100 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 106 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 111 in 0cb9e17
| ```bash |
[markdownlint] reported by reviewdog 🐶
MD031/blanks-around-fences Fenced code blocks should be surrounded by blank lines [Context: "```bash"]
Line 117 in 0cb9e17
| ```bash |
- 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.
Codecov Report✅ All modified and coverable lines are covered by tests. ❌ 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
☔ View full report in Codecov by Sentry. |
- 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.
| 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
Show autofix suggestion
Hide autofix suggestion
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: readInsert this block immediately after the runs-on line (line 67) but before any other keys for the job (such as if, needs, or steps).
-
Copy modified lines R68-R69
| @@ -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: |
- 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.
@sourcery-ai summary