fix(server): stop surfacing Claude 'task_updated' messages as work-log errors - #3741

💭 I'm looking at a PR that adds a test case and a new handler for task_updated messages in the ClaudeAdapter switch statement. Let me examine the surrounding code to understand how this fits with the Effect service conventions. I need to restructure the repo parameter as an object containing the URL instead. I'm adding a case for "task_updated" messages to prevent them from triggering runtime warnings, which changes the behavior from emitting warnings to silently dropping these messages. A test was added to cover this change, which aligns with the requirement for focused tests when backend behavior shifts. Now I need to check whether this violates any Effect service conventions. The imports weren't modified in the diff, so they're outside the scope of this review. The convention only applies to touched Effect service code, and since the import statements themselves weren't changed, there's no violation to flag. This looks good overall—no findings to report. The imports follow proper conventions and the test file is already in place, so the new test code integrates cleanly. This is a straightforward bug fix with appropriate test coverage and no violations of the Effect service patterns, so I'll mark it as success with an "All clear" response.