Skip to content

[From PR #919] catalog all_server_info/1 missing defensive try/catch around GenServer.call #922

@claude

Description

@claude

Context

From review of PR #919 (feat(mcp): catalog/ PTC-Lisp namespace with 4 builtins).

Problem

In lib/ptc_runner/lisp/eval/catalog_builtins.ex around line 323, all_server_info/1 calls GenServer.call(registry, :all_routings) directly without a try/catch :exit guard. The similar Catalog.configured_entries/1 wraps the same call pattern in try/catch :exit, _ -> %{}.

Fix

Add the same defensive pattern from Catalog.configured_entries to all_server_info/1 for consistency, or add a comment explaining why it's safe to skip it here.

Metadata

Metadata

Assignees

No one assigned

    Labels

    batch-pendingIssue included in an open batch-fix PRfrom-pr-reviewIssue created from PR review feedbackquick-fixTrivial fix, batched by batch-fix workflow

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions