Install
$ agentstack add skill-caiaffa-claude-code-ultimate-engineering-system-node-runtime-reliability ✓ scanned · ✓ verified — works with Claude Code, Cursor, and more.
Security review
✓ PassedNo issues found. Passed automated security review. · v0.1.0 How review works →
- ✓ Prompt-injection patterns
- ✓ Secret / credential exfiltration
- ✓ Dangerous shell & filesystem operations
- ✓ Untrusted network calls
- ✓ Known-malicious package signatures
What it can access
- ✓ Network access No
- ✓ Filesystem access No
- ✓ Shell / process execution No
- ✓ Environment & secrets No
- ✓ Dynamic code execution No
From automated source analysis of v0.1.0. “Used” means the capability is present in the source — more access means more to trust, not that it’s unsafe.
About
Mission
Make Node.js services trustworthy under real production load, failure, and operational stress.
When to use
- Reviewing Node.js service reliability.
- Investigating latency, crashes, or memory issues.
- Designing async workflows.
- Validating shutdown, retries, and concurrency controls.
Handoff
- Receives from: backend-platform-engineer (implementation) or systematic-debugging (investigation).
- Hands off to: performance-analysis (if perf issue), otel-observability-architect (monitoring), kubernetes-operability (runtime config).
Node.js runtime risks
| Risk | Symptom | Detection | Fix | |---|---|---|---| | Event loop blocking | All requests slow simultaneously | Event loop lag metric > 100ms | Move CPU work to worker threads or separate process | | Unhandled promise rejection | Silent failure, missing data | unhandledRejection handler + logging | Always await or catch; add process handler as safety net | | Memory leak | RSS grows over hours/days | RSS metric trending up over 24h | Heap snapshot analysis; check caches, listeners, closures | | Unbounded concurrency | Dependency overwhelmed | Connection pool exhaustion, timeout spikes | Semaphore/queue limiting concurrent requests | | Missing cancellation | Work continues after request abandoned | CPU/memory usage without matching traffic | AbortController for HTTP calls; cancel tokens for async chains | | Bad shutdown | Jobs lost, connections leaked | Restart produces duplicate work or errors | SIGTERM handler: stop accepting, drain in-flight, close pools |
Shutdown implementation pattern
process.on('SIGTERM', async () => {
server.close(); // Stop accepting new connections
await drainInFlightRequests(); // Wait for current requests (with timeout)
await closeConnectionPools(); // DB, Redis, etc.
await flushTelemetry(); // Send remaining spans/metrics
process.exit(0);
});
Concurrency control pattern
// BAD: unbounded concurrent external calls
const results = await Promise.all(items.map(i => callExternalAPI(i)));
// GOOD: bounded concurrency
import pLimit from 'p-limit';
const limit = pLimit(10); // max 10 concurrent
const results = await Promise.all(items.map(i => limit(() => callExternalAPI(i))));
Red flags
- No
unhandledRejectionhandler registered. Promise.allon unbounded arrays calling external services.- No timeout on any HTTP client call.
setIntervalwithout cleanup in graceful shutdown.- Global mutable state shared across requests.
JSON.parseon user input without size limit.
Output format
- Runtime risk summary (critical / moderate / low)
- High-risk patterns found (with file/location if possible)
- Concurrency assessment (bounded? backpressure present?)
- Shutdown assessment (graceful? tested?)
- Memory risk (leaks, unbounded caches, growing listeners?)
- Resilience improvements (prioritized, concrete)
Source & license
This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.
- Author: caiaffa
- Source: caiaffa/claude-code-ultimate-engineering-system
- License: MIT
Install and usage instructions live in the source repository linked above.
Reviews
No reviews yet — be the first.
Write a review
Versions
- v0.1.0 Imported from the upstream source.