Skip to content

Comments

codexmonitor 0.7.52 (new cask)#249005

Merged
bevanjkay merged 2 commits intoHomebrew:mainfrom
lucasloisp:feature/add-codexmonitor
Feb 13, 2026
Merged

codexmonitor 0.7.52 (new cask)#249005
bevanjkay merged 2 commits intoHomebrew:mainfrom
lucasloisp:feature/add-codexmonitor

Conversation

@lucasloisp
Copy link
Contributor

Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.

In the following questions <cask> is the token of the cask you're submitting.

After making any changes to a cask, existing or new, verify:

Additionally, if adding a new cask:

  • Named the cask according to the token reference.
  • Checked the cask was not already refused (add your cask's name to the end of the search field).
  • brew audit --cask --new <cask> worked successfully.
  • HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask <cask> worked successfully.
  • brew uninstall --cask <cask> worked successfully.

If AI was used to generate or assist with generating the PR:

  • I used AI to generate or assist with generating this PR. Please specify below how you used AI to help you.
  • I have personally reviewed, tested and verified all changes/additions, including zap stanza paths.

@p-linnane
Copy link
Member

I'm concerned with how rapid the release cadence for this software is. @bevanjkay @samford I only see 2 examples of throttling in Homebrew/cask and neither is using DSL.

@bevanjkay
Copy link
Member

Personally, I think it really only becomes a concern if the PRs become stale due to the release cadence - there's not as big overhead in homebrew-cask as homebrew-core. I think for standard use in homebrew-cask we need time-based throttling rather than version based, for it to be effective.

@lucasloisp
Copy link
Contributor Author

Slightly agree with @p-linnane if I'm being honest. Release cadence for this software is ~daily, with some days having more than one release. Having a new PR opened daily to update the cask is potentially too much overheadw.

Potentially, could add throttling like in these other casks:

# Throttle updates to every 5th release.

# a release for every version increase. This manually throttles versions to

Is time-based doable now? I presume not given what livecheck has access to it seems? If not then I suggest version-based, while not ideal, is sufficient.

@lucasloisp
Copy link
Contributor Author

Just pushed a new change using the same throttling strategy as commander.rb! Should be sufficient for the use case I'd argue

@bevanjkay bevanjkay requested a review from samford February 13, 2026 11:36
Copy link
Member

@samford samford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I recently merged a brew commit to add basic support for throttle handling to brew bump-cask-pr but it has the same limitations as the existing formula approach. Namely, it only throttles on the patch version and only works with versions that are divisible by the throttle rate. The current throttle-handling implementation doesn't work for the two casks that are throttled using a strategy block because one needs to throttle on version.csv.second and the other isn't compatible with the divisibility requirement (upstream doesn't create a release for every version number).

throttle should technically work for codexmonitor but upstream doesn't seem to be consistent with the release cadence (e.g., 0-3 releases per day). As we've acknowledged for some time, throttling based on time (e.g., one release per day) is generally what we're aiming for but the current throttle setup doesn't support it. [There's some recent internal discussion around this but it's something that will only happen when someone implements it and that's a bit of a challenge.]

That said, I think we have two options here:

  • Preemptively use throttle 2 and accept that there may be days where the cadence is lower and throttling leads to no update PRs even though there's been a release. This uses auto_update true, so that means some users may see an update after installation and anyone who disables the in-app updater and uses brew upgrade --greedy will see slower updates through Homebrew. We could use a higher number but five may be too high (the cadence has been lower the past few days and seems to top out at three versions per day, whereas commander regularly exceeds five per day).
  • Don't preemptively throttle the cask and see how the release cadence translates to autobump PRs. If the releases are spread throughout the day (with several hours in between), then autobump will produce multiple PRs. If some releases are close together (i.e., within the same autobump interval), then autobump will simply use the newest version at the time (skipping versions in between).

I'm fine with either option (the release cadence isn't as aggressive as commander) but I've added a simple livecheck block suggestion for the throttle approach, in case that's what we go with.


Besides that, 0.7.52 was released a few hours ago, so it would be good to update this.

@lucasloisp lucasloisp force-pushed the feature/add-codexmonitor branch from 7bcaa1f to 651dc4c Compare February 13, 2026 22:52
@lucasloisp
Copy link
Contributor Author

@samford terrific! Just updated to use throttle 2 -- and bumped to 0.7.52

@lucasloisp lucasloisp changed the title codexmonitor 0.7.50 (new cask) codexmonitor 0.7.52 (new cask) Feb 13, 2026
@bevanjkay bevanjkay enabled auto-merge February 13, 2026 22:53
@bevanjkay
Copy link
Member

Thank you @lucasloisp 🚀

@bevanjkay bevanjkay added this pull request to the merge queue Feb 13, 2026
Merged via the queue into Homebrew:main with commit 3d7dfa8 Feb 13, 2026
10 checks passed
@lucasloisp lucasloisp deleted the feature/add-codexmonitor branch February 13, 2026 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants