Skip to content

fix(server): add error logging to MCP message processing - #191

Merged
gHashTag merged 1 commit into
mainfrom
feat/issue-189
Mar 11, 2026
Merged

gHashTag merged 1 commit into
mainfrom
feat/issue-189

Conversation

@gHashTag

Copy link
Copy Markdown
Owner

Summary

  • Added error logging for silent catch {} blocks in MCP message processing
  • Startup stderr writes documented as best-effort (stderr may not be available)
  • Critical processMessage errors now logged via std.log.err

Changes

  • Line 1165: Added comment clarifying stderr writes are best-effort
  • Line 1198: Added error logging for raw JSON message processing
  • Line 1246: Added error logging for Content-Length framed message processing

Test Plan

  • zig fmt --check passes
  • zig ast-check passes
  • Syntax verified for server.zig

Closes #189

🤖 Generated with Claude Code

Silent catch {} blocks in processMessage calls were dropping errors
with no indication. Now logs error name via std.log.err.

Startup stderr writes documented as best-effort since stderr may not
be available in all environments.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gHashTag
gHashTag merged commit 1fdbc9d into main Mar 11, 2026
4 of 6 checks passed
gHashTag added a commit that referenced this pull request Mar 18, 2026
fix(server): add error logging to MCP message processing
@gHashTag
gHashTag deleted the feat/issue-189 branch April 3, 2026 11:00
gHashTag added a commit that referenced this pull request Oct 3, 2026
A post on the board half of openXC7/prjxray-db#30: a BUFR counter
clocked through CLK_PERF2 <- CLKOUT3 and PERFCLK2 <- MUXED2 runs on
the AX7203 with the c030ed6 rows and stays dead with the master rows;
the two bitstreams differ in three bits, and clearing any one of them
stops the counter again. It also records the stale nextpnr-xilinx build
(#191 and #205 missing) and the edit-to-board loop going from about
104-111 s to about 19-28 s.

Two recordings published under t27.ai/term/ and replayed in the post:
perf2-knockouts (the three bits named, the knockout bitstreams rebuilt
byte-identical to the flashed ones) and perf2-loop (router1 to SRAM,
19.06 s in that run, at load average ~110).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
gHashTag added a commit that referenced this pull request Oct 4, 2026
A post on the board half of openXC7/prjxray-db#30: a BUFR counter
clocked through CLK_PERF2 <- CLKOUT3 and PERFCLK2 <- MUXED2 runs on
the AX7203 with the c030ed6 rows and stays dead with the master rows;
the two bitstreams differ in three bits, and clearing any one of them
stops the counter again. It also records the stale nextpnr-xilinx build
(#191 and #205 missing) and the edit-to-board loop going from about
104-111 s to about 19-28 s.

Two recordings published under t27.ai/term/ and replayed in the post:
perf2-knockouts (the three bits named, the knockout bitstreams rebuilt
byte-identical to the flashed ones) and perf2-loop (router1 to SRAM,
19.06 s in that run, at load average ~110).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
gHashTag added a commit that referenced this pull request Oct 4, 2026
)

A post on the board half of openXC7/prjxray-db#30: a BUFR counter
clocked through CLK_PERF2 <- CLKOUT3 and PERFCLK2 <- MUXED2 runs on
the AX7203 with the c030ed6 rows and stays dead with the master rows;
the two bitstreams differ in three bits, and clearing any one of them
stops the counter again. It also records the stale nextpnr-xilinx build
(#191 and #205 missing) and the edit-to-board loop going from about
104-111 s to about 19-28 s.

Two recordings published under t27.ai/term/ and replayed in the post:
perf2-knockouts (the three bits named, the knockout bitstreams rebuilt
byte-identical to the flashed ones) and perf2-loop (router1 to SRAM,
19.06 s in that run, at load average ~110).

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
github-actions Bot added a commit that referenced this pull request Oct 4, 2026
docs(blog): three bits, each one necessary - CLK_PERF2 on silicon (#1309)

A post on the board half of openXC7/prjxray-db#30: a BUFR counter
clocked through CLK_PERF2 <- CLKOUT3 and PERFCLK2 <- MUXED2 runs on
the AX7203 with the c030ed6 rows and stays dead with the master rows;
the two bitstreams differ in three bits, and clearing any one of them
stops the counter again. It also records the stale nextpnr-xilinx build
(#191 and #205 missing) and the edit-to-board loop going from about
104-111 s to about 19-28 s.

Two recordings published under t27.ai/term/ and replayed in the post:
perf2-knockouts (the three bits named, the knockout bitstreams rebuilt
byte-identical to the flashed ones) and perf2-loop (router1 to SRAM,
19.06 s in that run, at load average ~110).

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(server): add error logging to catch {} in MCP server.zig message processing

2 participants