🤖 Fix browser API handling of failed Results without error property

[kylecarbs]

Generated with cmux

Problem

The web main-server's HttpIpcMainAdapter was incorrectly handling failed Results that don't have an error property.

The check required both success === false AND 'error' in result:

if (
  result &&
  typeof result === "object" &&
  "success" in result &&
  result.success === false &&
  "error" in result  // <-- This line was the bug
) {
  res.json(result);
  return;
}
res.json({ success: true, data: result });

When a handler returned { success: false } without an error property, the check failed and the result was incorrectly wrapped as:

{
  "success": true,
  "data": { "success": false }
}

This broke error handling in the frontend - errors from operations like force workspace deletion weren't being properly communicated.

Solution

Remove the 'error' in result requirement. Any Result with success: false should be passed through directly, regardless of whether it has an error property.

Testing

Added test case in api.test.ts to verify Results without error property are handled correctly.