refactor(server): Break up ServeHTTP for readability

[kalbasit]

TL;DR

Extracted HTTP handlers for /receive/pop and /receive/flush endpoints into separate methods.

What changed?

• Moved /receive/pop handler logic into receivePop method
• Moved /receive/flush handler logic into receiveFlush method
• Removed //nolint:nestif comment as it's no longer needed
• Main ServeHTTP method now delegates to these new methods

How to test?

1. Send HTTP requests to /receive/pop endpoint
   • Verify messages are received correctly
   • Check repeat-last functionality works as expected
2. Send HTTP requests to /receive/flush endpoint
   • Verify all messages are flushed properly
   • Confirm last message is stored when repeat-last is enabled

Why make this change?

Improves code organization and readability by breaking down the large ServeHTTP method into smaller, focused methods. This makes the code easier to maintain and test while reducing cognitive complexity.

[kalbasit]

• refactor(server): Break up ServeHTTP for readability #23 👈 (View in Graphite)
• feat(server): Repeat last message on /receive/pop #22
• fix(receiver): Ignore non-data messages #21
• fix(receiver): Reconnect websocket if receiving errors out #20
• refactor: server should start the receive loop #19
• feat(receiver): Expose all information from a Signal message #18
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[kalbasit]

Merge activity

• Dec 21, 4:05 PM EST: A user merged this pull request with Graphite.