There is unpushed work implementing most of #49, parked in a worktree on this
machine. Recording it here so it is findable rather than living in a session
transcript.
Where
path ~/.local-llm-bench/work/monitor-jsonl
branch monitord-output-formats
HEAD 8a17cee (never pushed; `git ls-remote --heads origin` returns nothing)
tree clean
8 files, +204/-6: CSVLogFormat, Formatting, JSONSample, LogRetention,
Metric, RunSidecar (MonitorCore) and SMC, SMCSource (MonitorSources).
What it covers
Two things whoever picks it up must know
It is based on fe30192 (the #47 merge). main is now a500b78 / v1.6.2,
six commits and three releases later. PR #50 rewrote both CLIs onto
swift-argument-parser: Sources/monitord/main.swift is gone, replaced by
Sources/monitord/Monitord.swift, and there is a new CommandLineTests target.
The mitigating detail is that this WIP touches no file under
Sources/monitord, so the rebase should be mechanical rather than a rewrite.
RunSidecar and JSONSample in 8a17cee were reconstructed from session
notes, not from the original bytes. They should be read as a draft of the
intended design, not as reviewed code that merely needs rebasing. Anything
subtle in those two files deserves a fresh look rather than trust.
Options
Rebase onto main and open a PR, or discard the branch and build #49 fresh now
that the CLI has moved. The second is defensible given the reconstruction
caveat — the value here may be the design sketch rather than the diff.
Either way the branch should not be left as the only record of its own
existence: it is on one machine, unpushed, in a directory named for an
unrelated project's benchmark harness.
There is unpushed work implementing most of #49, parked in a worktree on this
machine. Recording it here so it is findable rather than living in a session
transcript.
Where
8 files, +204/-6:
CSVLogFormat,Formatting,JSONSample,LogRetention,Metric,RunSidecar(MonitorCore) andSMC,SMCSource(MonitorSources).What it covers
run.jsonsidecar, and stdout-only--format jsonl.sensor.fan.N.modecolumn from monitord: --help, --version, and unknown flags all start the daemon instead of printing usage #48's second comment, whichwas explicitly left out of scope when monitord: --help, --version, and unknown flags all start the daemon instead of printing usage #48 was closed via PR Give monitorctl and monitord a real front door #50 and is
therefore still unbuilt.
Two things whoever picks it up must know
It is based on
fe30192(the #47 merge).mainis nowa500b78/ v1.6.2,six commits and three releases later. PR #50 rewrote both CLIs onto
swift-argument-parser:
Sources/monitord/main.swiftis gone, replaced bySources/monitord/Monitord.swift, and there is a newCommandLineTeststarget.The mitigating detail is that this WIP touches no file under
Sources/monitord, so the rebase should be mechanical rather than a rewrite.RunSidecarandJSONSamplein8a17ceewere reconstructed from sessionnotes, not from the original bytes. They should be read as a draft of the
intended design, not as reviewed code that merely needs rebasing. Anything
subtle in those two files deserves a fresh look rather than trust.
Options
Rebase onto
mainand open a PR, or discard the branch and build #49 fresh nowthat the CLI has moved. The second is defensible given the reconstruction
caveat — the value here may be the design sketch rather than the diff.
Either way the branch should not be left as the only record of its own
existence: it is on one machine, unpushed, in a directory named for an
unrelated project's benchmark harness.