Skip to content

fanknob 1.5.1 - #10

Merged
SadriG91 merged 1 commit into
mainfrom
release/v1.5.1
Aug 11, 2026
Merged

SadriG91 merged 1 commit into
mainfrom
release/v1.5.1

Conversation

@SadriG91

Copy link
Copy Markdown
Owner

Version bump for the release. Once this is merged I'll tag v1.5.1 on main and the workflow takes over.

What ships

Three fixes, all of them corrections to things fanknob was reporting inaccurately (#9):

  • The fan-control light told the truth about the daemon. It answered via daemonReachable(), which only completes a connect() — the kernel accepts that into the listen backlog whether or not the daemon can still answer. The app could show a green "fan control available" light at the same moment it showed "daemon timed out", with the controls enabled and doing nothing. It's now derived from the state round-trip the same poll already performs.
  • A routine client disconnect is no longer logged as an error. EPIPE was mapped to .system, so a client hitting its own timeout produced a "Broken pipe" line every time — burying real incidents under hundreds of them.
  • The daemon log is timestamped. launchd appends to /var/log/fanknobd.log across reboots, so lines previously couldn't be lined up with a boot or a bug report.

Also on main since 1.5.0 but shipped in 1.5.0's wake: the thermal watchdog keeps maximum cooling active when it trips (#8).

No features and no breaking changes, hence a patch bump.

Deferred

#9 originally retried a daemon command whose reply timed out, so the app would stop reporting an error for a change that had in fact been applied. Review found two real defects — a retried timed set extends its own hold, and re-execution disturbs the watchdog's debounce state — neither fixable without request-ID deduplication or a richer state reply. Both are protocol changes, so it's deferred to 1.6.0.

Checked

  • make print-version1.5.1, and make app stamps CFBundleShortVersionString: 1.5.1 into the bundle. CFBundleVersion is the commit count, 98, up from 96 at v1.5.0 — it has to increase monotonically.
  • 83/83 tests pass; release build clean.
  • make ui-test has not been run for this release. It covers the fan-control light, which is the only user-visible change here.

🤖 Generated with Claude Code

@SadriG91
SadriG91 merged commit 0b8efe6 into main Aug 11, 2026
6 checks passed
@SadriG91
SadriG91 deleted the release/v1.5.1 branch August 11, 2026 22:56
@SadriG91

Copy link
Copy Markdown
Owner Author

Correction to the checked list above: CFBundleVersion ships as 99, not 98. I measured the commit count before committing the bump itself. The invariant that matters is unaffected — 96 at v1.5.0 → 99 here, still increasing.

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.

1 participant