Skip to content

fix(engine): skip write-only GC-retention sets, gate oh-my-pi - #184

Merged
yoominho91 merged 1 commit into
mainfrom
fix/write-only-gc-retention
Oct 10, 2026
Merged

yoominho91 merged 1 commit into
mainfrom
fix/write-only-gc-retention

Conversation

@yoominho91

Copy link
Copy Markdown
Contributor

AI ์—์ด์ „ํŠธ ์ €์žฅ์†Œ ์‚ฌ๋ƒฅ์—์„œ ๋‚˜์˜จ ์˜คํƒ(์ˆ˜๊ฑฐ ๋ฐฉ์ง€ ๋ณด๊ด€ Set) ๊ฐ€๋“œ ยท oh-my-pi ๊ฒŒ์ดํŠธ ๊ธฐ๋ก(๋ณธ์ธ ์ž‘์„ฑ ๋ฌธ์žฅ ํ•„์ˆ˜).

https://claude.ai/code/session_01W9Mk18mBNfkPfgg3hYweKq

- writeOnlyCollection: a Set that only holds references so objects are
  not collected is meant to be write-only (Electron Notification loses its
  click handler once collected). Skip retain/alive/live*/hold/pinned
  names [FP:write-only-gc-retention]: cindy liveNotifications, oh-my-pi
  retainedShells.
- repo-gates: oh-my-pi requires a sentence written by the submitter and
  a personally exercised repro in every PR body.

Claude-Session: https://claude.ai/code/session_01W9Mk18mBNfkPfgg3hYweKq
@vercel

vercel Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
fixearly Ready Ready Preview Oct 10, 2026 10:25am UTC

@yoominho91
yoominho91 merged commit a5960dd into main Oct 10, 2026
3 checks passed
@yoominho91
yoominho91 deleted the fix/write-only-gc-retention branch October 10, 2026 10:26

This branch was successfully deployed

1 active deployment
Preview โ€” e115eaa8 Deployed Oct 10, 2026 by vercel[bot]
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