Fix pino logger.error argument order across codebase

Pino's API expects error object as first argument: logger.error(error, 'message')
or printf-style: logger.error('message: %s', value)

The incorrect pattern logger.error('message', error) silently discards the second
argument, making debugging impossible.

Fixed 27 files with broken logger.error calls.