Skip to content

docs: add async file I/O and auth rate limiting patterns to DEVELOPMENT.md#284

Merged
hoiekim merged 1 commit intohoiekim:mainfrom
moltboie:docs/async-io-and-auth-patterns
Mar 26, 2026
Merged

docs: add async file I/O and auth rate limiting patterns to DEVELOPMENT.md#284
hoiekim merged 1 commit intohoiekim:mainfrom
moltboie:docs/async-io-and-auth-patterns

Conversation

@moltboie
Copy link
Copy Markdown
Contributor

Changes

Adds two new patterns to DEVELOPMENT.md:

File I/O in Async Context

Documents the anti-pattern of wrapping writeFileSync in Promise constructors (gives false impression of async behavior while still blocking the event loop). Recommends fs.promises APIs and mkdir({ recursive: true }) over check-then-create.

IMAP/SMTP Authentication Rate Limiting

Documents the need for rate limiting across all auth protocols (HTTP, IMAP, SMTP) with shared counters to prevent protocol-switching brute force attacks.

Context

E2E Testing

Documentation-only change. Verified markdown renders correctly and no code changes.

@moltboie
Copy link
Copy Markdown
Contributor Author

Self-Review

Discussion thread status:

  • New PR. No prior feedback. Adds two patterns to DEVELOPMENT.md: async file I/O and auth rate limiting.

Checked:

Issues found:

  • None.

Confidence: High

@moltboie moltboie force-pushed the docs/async-io-and-auth-patterns branch from ff8374d to cb4b4c6 Compare March 21, 2026 16:40
@hoiekim hoiekim merged commit 135a91f into hoiekim:main Mar 26, 2026
2 checks passed
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