Flight Log / onlook-dev/onlook #1376
refactor: Move node-pty to separate process — a Devin (Cognition) bot flight
The Devin bot (Cognition) opened this pull request in onlook-dev/onlook on Feb 14, 2025; a burn — closed unmerged after human review: +114 −24, 2 files, 0 reviews.
Flown by
- Lines
- +114 −24
- Files
- 2
- Commits
- 1
- Reviews
- 0
- To merge
- —
- Task
- refactor
Line and file counts are summed per commit from the AIDev dataset; GitHub's net diff can be smaller.
Devin's refactor moves node-pty into a separate process launched with ELECTRON_RUN_AS_NODE, aiming to fix assertion errors during app closure. The change adds 114 lines and removes 24 across two files in one commit. The pull request drew zero formal reviews and was closed unmerged after a human reviewer commented "Not working". Logged as a burn.
“Not working”
— from the review (source)
From the pull request description
Move node-pty to run in a separate process using ELECTRON_RUN_AS_NODE to fix assertion errors during app closure. The changes: 1. Create a dedicated process for node-pty operations 2. Implement IPC communication between main process and PTY process 3. Clean up PTY processes properly on exit 4. Use …
Excerpt (first 300 characters) — the full description is on GitHub.
View the pull request on GitHub →
This is your flight? Claim it — sign in with GitHub. Ownership is proven by the sign-in: the account that opened the PR is the only one that can claim it.
Flight timeline
- —committedDevin AI
- Feb 14, 2025closedKitenite
Read next
Cite this flight: AgentRiders Flight Log, onlook-dev/onlook#1376, record updated Nov 4, 2026, https://agentriders.com/logs/onlook-dev/onlook/1376/
Data from public GitHub activity; this page is not affiliated with GitHub or the repository.