Skip to content

Test/warning info features#2

Open
decispher wants to merge 2 commits into
masterfrom
test/warning-info-features
Open

Test/warning info features#2
decispher wants to merge 2 commits into
masterfrom
test/warning-info-features

Conversation

@decispher
Copy link
Copy Markdown
Owner

No description provided.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 8, 2026

⚠️ Decision Context Alert

This PR modifies 3 file(s) that trigger 3 architectural decision(s).

🔴 Critical Decisions (1)

DECISION-DB-001: Database Connection Pool Configuration

File src/database/connection\.js
Pattern src/database/connection\.js
Type Rule-based
Date 2024-03-15
Source .decispher/decisions.md

We fixed the connection pool size at **20 connections** after extensive load testing. This decision was made because:

1. **Memory constraints**: Each...

📖 Read full context

We fixed the connection pool size at **20 connections** after extensive load testing. This decision was made because:

1. **Memory constraints**: Each connection uses ~5MB of memory
2. **Database limits**: Our PostgreSQL instance supports max 100 connections
3. **Performance testing**: Higher values caused connection leaks under sustained traffic (5K req/s)

**Alternatives rejected:**
- Dynamic pooling: Added latency spikes during scaling
- MongoDB: Billing requires ACID compliance

[!WARNING]
Do not modify pool size without load testing. Contact @senior-engineer before changes.

**Related:**
- [Load Test Results](https://internal\-docs/load\-tests\)
- [Architecture Decision Record](https://internal\-docs/adr\-001\)


🟡 Important Decisions (1)

DECISION-API-001: Public API v1 Protection

File src/api/users\.js
Pattern src/api/\*\*/\*\.js
Type Rule-based
Date 2024-02-20
Source .decispher/decisions.md

The v1 API is used by **50+ external partners**. Any changes to these endpoints require:

1. **Advance notice**: Notify partners 2 weeks before breaki...

📖 Read full context

The v1 API is used by **50+ external partners**. Any changes to these endpoints require:

1. **Advance notice**: Notify partners 2 weeks before breaking changes
2. **Versioning**: Use semantic versioning for the API
3. **Documentation**: Update OpenAPI spec before merging

**Before merging changes to API files:**
- [ ] Update API documentation
- [ ] Notify integration partners if breaking
- [ ] Version bump if needed


ℹ️ Informational (1)

DECISION-CONFIG-001: Configuration File Format

File config/test\-feature\.yml
Pattern config/\*\.yml
Type Rule-based
Date 2024-02-01
Source .decispher/decisions.md

We use YAML for all configuration files for consistency. JSON configs were migrated to YAML in Feb 2024.



🤖 Generated by Decision Guardian. Update decisions in your .decispher/ folder if needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants