fix: Ensure ports are confirmed free before allocation in Cloudflare and Python runners#140
fix: Ensure ports are confirmed free before allocation in Cloudflare and Python runners#140constantinius wants to merge 1 commit intomainfrom
Conversation
…and Python runners
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit e2f2b03. Configure here.
| server.listen(port, "0.0.0.0", () => { | ||
| server.close(() => resolve(true)); | ||
| }); | ||
| }); |
There was a problem hiding this comment.
Comment claims IPv6 check but code only checks IPv4
Medium Severity
The JSDoc for isPortFree states it "Binds on both IPv4 and IPv6 loopback to match what most servers do," but the implementation only calls server.listen(port, "0.0.0.0", ...), which binds exclusively on IPv4 all-interfaces (not even loopback). No IPv6 check (e.g., on :: or ::1) is performed. On systems where localhost resolves to ::1 first (common on modern Linux/macOS), a port occupied on IPv6 would be reported as free, defeating the purpose of this PR.
Reviewed by Cursor Bugbot for commit e2f2b03. Configure here.
🟡 AI SDK Integration Test ResultsStatus: 2 tests fixed, 192 still failing Summary
✅ FixedThese tests were failing on main but are now passing:
Test MatrixAgent Tests
Embedding Tests
LLM Tests
MCP Tests
Legend: ✅ Pass | ❌ Fail | ✅🔧 Fixed | ❌📉 Regressed | ✅🆕 New (pass) | ❌🆕 New (fail) | 🗑️ Removed | str=streaming blk=blocking a=async s=sync io=stdio sse=sse hi=highlevel lo=lowlevel Generated by AI SDK Integration Tests |


No description provided.