fix(server): stop OpenCode child sessions - #9005

💭 I'm reviewing the changes to OpenCodeAdapter.ts, focusing on the new descendant abort logic, semaphore usage, error filtering, and how errors get converted into request errors with detail fields. I want to check the imports and surrounding code to confirm this matches existing conventions.

1. browse_code — apps/server/src/provider/Layers/OpenCodeAdapter.ts

2. git_tools — isOpenCodeNotFound|toRequestError

💭 The code follows existing file conventions—namespace imports, structural predicate checks, and reused error-handling patterns. I'm confirming the new abortOpenCodeDescendants function using a semaphore with capacity 8 doesn't introduce new service definitions or require lint suppressions. The manual failure-collection pattern (returning an optional error then yielding it) isn't ideal Effect style but isn't a listed convention violation either. I'm also verifying the added tests are justified since this is a genuine behavior change, and double-checking there are no new imports or magic-number duplication issues worth flagging beyond minor style. With no actual convention violations found, I need to produce output in the required JSON schema rather than plain text, so I'll set the state to success with a clear "All clear" title and summary reflecting no findings.