fix(logger): restore wallclock timestamp on console output (macOS)

[chr1syy]

Summary

Follow-up to #48 — stacked on top of chore/logger-consolidation. Routes all console emitters through formatEntry(...).trimEnd() so log lines carry a wallclock ISO timestamp. Closes the macOS-only gap where maestro-relay-ctl logs (which tails the raw launchd output file) had no timestamps on debug/info/warn lines.

Note on the base branch: the base PR (#48) head lives in a fork, so to make this stack reviewable on RunMaestro/Maestro-Relay, the chore/logger-consolidation branch has been mirrored to this repo. Once #48 merges to main, this PR's base will be re-targeted to main (the diff stays the same) and the mirrored base branch can be deleted.

Why

templates/sh.maestro.relay.plist redirects stdout/stderr to a flat logs/maestro-relay.log file. launchd does not stamp lines. Before this PR, a macOS operator saw [INFO] [api/startup] listening on http://... with no time context.

On Linux, StandardOutput=journal in templates/maestro-relay.service means journalctl already adds a local-time prefix. The inner ISO timestamp this PR adds is mildly redundant on Linux but accepted as the simpler design — operators can filter on either timestamp. The alternative (TTY/env-gated formatting) is more code for marginal benefit.

Behavior change

Surface	Before	After
Linux journalctl	Jun 07 10:53:21 host relay[...]: [INFO] [ctx] msg	Jun 07 10:53:21 host relay[...]: [2026-06-07T08:53:21.123Z] INFO [ctx] msg
macOS maestro-relay-ctl logs	[INFO] [ctx] msg	[2026-06-07T08:53:21.123Z] INFO [ctx] msg
logs/errors.log	unchanged	unchanged

Implementation

• src/core/logger.ts: emit() helper and the error() console branch now use formatEntry(level, ctx, detail).trimEnd() (the file-append path still uses the un-trimmed formatEntry(...) so the trailing newline lands in errors.log).
• formatLine is left in place for now in case a future PR wants TTY-gated formatting; it builds cleanly without noUnusedLocals complaint.
• src/__tests__/logger.test.ts: updated the three regex assertions matching \[LEVEL\] \[ctx\] detail to match \[ISO-ts\] LEVEL \[ctx\] detail$ instead.

Test plan

• npm test — 219/219 pass with updated regex
• npm run build — clean
• Manual macOS smoke (optional): run dev server, confirm console emits ISO timestamps

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 6f4c5e73-2d3b-434c-a669-bda3f02529f3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/logger-macos-timestamp

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}