Skip to content

docs: add scheduled task concurrency and process lifecycle patterns#225

Merged
hoiekim merged 1 commit intohoiekim:mainfrom
moltboie:docs/scheduled-task-lifecycle-patterns
Mar 29, 2026
Merged

docs: add scheduled task concurrency and process lifecycle patterns#225
hoiekim merged 1 commit intohoiekim:mainfrom
moltboie:docs/scheduled-task-lifecycle-patterns

Conversation

@moltboie
Copy link
Copy Markdown
Contributor

Add two new design pattern sections to DEVELOPMENT.md:

Scheduled Task Concurrency

Documents the concurrency guard pattern for long-running scheduled tasks (like scheduledSync). Without a guard, slow API responses can cause overlapping executions that race on shared data.

Related: #222

Process Lifecycle Handlers

Documents that process-level signal handlers (SIGINT, SIGTERM, unhandledRejection, uncaughtException) should live in start.ts, not in library modules. Library imports shouldn't have global side effects.

Related: #223


E2E: Verified DEVELOPMENT.md renders correctly and sections are properly ordered within the Design Patterns section.

@moltboie
Copy link
Copy Markdown
Contributor Author

Self-Review

Discussion thread status:

Checked:

E2E Testing:

  • Documentation change only. Verified DEVELOPMENT.md renders correctly and the two new sections appear under Design Patterns in proper order.

Issues found:

  • None.

Confidence: High

@moltboie moltboie force-pushed the docs/scheduled-task-lifecycle-patterns branch from a173614 to 147b485 Compare March 21, 2026 16:49
@moltboie moltboie force-pushed the docs/scheduled-task-lifecycle-patterns branch 2 times, most recently from c2bf171 to 59b97c8 Compare March 28, 2026 17:26
@moltboie moltboie force-pushed the docs/scheduled-task-lifecycle-patterns branch from 59b97c8 to 44a9af0 Compare March 28, 2026 20:23
@moltboie
Copy link
Copy Markdown
Contributor Author

Rebased on main.

@hoiekim hoiekim merged commit d7b05b3 into hoiekim:main Mar 29, 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