chore(cli): replace snaps-utils logError with logger

[grypez]

Replace the logError import from @metamask/snaps-utils with the standard internal logger, removing an unnecessary dependency on the snaps-utils package. This simplifies the error handling in app.ts and serve.ts by using the same logging infrastructure as the rest of the CLI.

---

Note

Low Risk
Small refactor limited to CLI logging and dev server error reporting, with no changes to core functionality or security-sensitive logic.

Overview
Standardizes CLI logging by switching app.ts to a configured @metamask/logger instance with a custom console transport (no tag output) and by passing that logger through to the serve/start commands.

Updates getServer in commands/serve.ts to accept an optional Logger and use it for request-handler error reporting, replacing @metamask/snaps-utils’s logError; the @metamask/snaps-utils dependency is removed from package.json/yarn.lock.

^{Written by Cursor Bugbot for commit 88f0f43. This will update automatically on new commits. Configure here.}

[github-actions]

Coverage Report

Status	Category	Percentage	Covered / Total
🔵	Lines	78.21% ⬇️ -0.03%	6152 / 7865
🔵	Statements	78.19% ⬇️ -0.03%	6251 / 7994
🔵	Functions	76.52% ⬇️ -0.04%	1568 / 2049
🔵	Branches	78.21% ⬇️ -0.11%	2240 / 2864

File Coverage

File	Stmts	Branches	Functions	Lines	Uncovered Lines
Changed Files
packages/cli/src/app.ts	0% 🟰 ±0%	0% 🟰 ±0%	0% 🟰 ±0%	0% 🟰 ±0%	22-178
packages/cli/src/commands/serve.ts	100% 🟰 ±0%	100% 🟰 ±0%	100% 🟰 ±0%	100% 🟰 ±0%

Generated in workflow #3610 for commit 88f0f43 by the Vitest Coverage Report Action

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.