Skip to content

Replace execSync with async alternative in device-name-resolver #333

@cameroncooke

Description

@cameroncooke

Context

src/utils/device-name-resolver.ts uses execSync to call xcrun devicectl list devices, which blocks the Node.js event loop. While mitigated by a 10s timeout and 30s TTL cache, it would be cleaner to use an async alternative.

Flagged by Sentry bot on PR #325.

Recommendation

Replace execSync/readFileSync/unlinkSync with async equivalents (execFile from child_process/promises, fs.promises.readFile, fs.promises.unlink). Low priority given the MCP server's single-request-at-a-time model and the caching layer.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions