Allow customisation of hs.chooser font sizes. Closes #226

[cmsj]

Closes #226

[greptile-apps]

Confidence Score: 5/5

[Medium risk] Adds customizable font sizes to the chooser UI.

The PR appears safe to merge; the remaining finding concerns regression-test coverage rather than demonstrated runtime behavior.

Findings

1.  Assignment exceptions go undetected ▶

Fix with agent prompt

### Issue 1
Hammerspoon 2Tests/IntegrationTests/HSChooserIntegrationTests.swift:487-488
`expectTrue` starts a new JavaScript evaluation, which clears the exception recorded by the assignment block before `hasException` checks it. If an invalid assignment throws but leaves the size unchanged, this test still passes, so it does not verify that invalid values are ignored without an exception. Check `hasException` immediately after the assignment block. The subtext and query tests have the same pattern.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

The PR adds configurable chooser title, subtitle, and query font sizes, adjusts row and panel sizing to match, and adds integration coverage. The latest changes reject invalid sizes and document that behavior.

_{Reviews (2) · Last reviewed commit: "Guard text sizes against being negative,..."}

[cmsj]

@greptile

[greptile-apps]

Assignment exceptions go undetected

expectTrue starts a new JavaScript evaluation, which clears the exception recorded by the assignment block before hasException checks it. If an invalid assignment throws but leaves the size unchanged, this test still passes, so it does not verify that invalid values are ignored without an exception. Check hasException immediately after the assignment block. The subtext and query tests have the same pattern.

Prompt To Fix With AI

This is a comment left during a code review.
Path: Hammerspoon 2Tests/IntegrationTests/HSChooserIntegrationTests.swift
Line: 487-488

Comment:
**Assignment exceptions go undetected**

`expectTrue` starts a new JavaScript evaluation, which clears the exception recorded by the assignment block before `hasException` checks it. If an invalid assignment throws but leaves the size unchanged, this test still passes, so it does not verify that invalid values are ignored without an exception. Check `hasException` immediately after the assignment block. The subtext and query tests have the same pattern.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.