Skip to content

chore: replace console.error in errors.ts and octto/session/server.ts with shared log utility #18

@dscv103

Description

@dscv103

Problem

Two files bypass the shared log utility from src/utils/logger.ts, violating the logging conventions in .mindmodel/patterns/logging.md:

src/utils/errors.ts:

// BAD console.error(`[${module}] ${extractErrorMessage(e)}`);

src/octto/session/server.ts:

// BAD console.error('[octto] Failed to parse WebSocket message:', error);

Fix

// errors.ts import { log } from './logger'; log.error(module, extractErrorMessage(e)); // server.ts import { log } from '../../utils/logger'; log.error('octto.server', 'Failed to parse WebSocket message', error);

Metadata

Metadata

Assignees

No one assigned

    Labels

    choreMaintenance, cleanup, tooling

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions