chore(release): 0.39.43 - #99
Merged
Merged
Conversation
Ships #98 (issue #97): GitHub Copilot quota now reports the premium-request count — "158/300", or "142/300" in Remaining mode — instead of a percentage the reader has to convert. The numbers were already being fetched from api.github.com and discarded after one division. The independent Codex QA gate returned SHIP: NO on the first cut and all three findings were real: the bar and its caption could be drawn from different fields and disagree (70% beside 228/300, which is 76%); a null `remaining` coerced to zero and reported the whole allowance consumed; and the dashboard clamped only one end, so an unclamped payload could render "312/300" in either mode. Fixed, each with a regression test built from the exact reproduction. First release where the `version` lifecycle hook from #96 did its job: the bump synced TokenTrackerBar/project.yml and TokenTrackerWin.csproj, which is what made 0.39.39–0.39.42 unable to cut a desktop build. prepublishOnly re-vendored the LiteLLM seed: 2,522 models, no rate changes, only _meta.generated_at moved.
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Published to npm as
@ipv9/tokentracker-cli@0.39.43before opening this — the publish job self-skips a version that already exists, so CI stays green.Ships
#98 (issue #97) — GitHub Copilot quota now reports the premium-request count (
158/300, or142/300in Remaining mode) instead of a percentage the reader has to convert. The numbers were already being fetched fromapi.github.comand discarded after one division.The QA gate earned its keep
An independent Codex pass at
xhighreturned SHIP: NO on the first cut. All three findings were real and all three were mine:percent_remaining: 30withentitlement: 300, remaining: 72produced a 70%-wide bar captioned228/300— which is 76%. The percentage is now derived from the counts whenever both arrive.Number(null)is0, so a missingremainingreported the entire allowance consumed:300/300beside a percentage saying 10% used. Counts now require a real finite number.312/300in Used mode fromused: 312and in Remaining mode fromused: -12. Clamping into[0, limit]now happens before the mode flip — which is what my own discarded comment had claimed to handle.Each is closed with a regression test built from the exact reproduction. A narrow re-check returned SHIP: YES, plus two non-blocking stale comments, also fixed.
First real use of #96's lockstep hook
npm version 0.39.43syncedTokenTrackerBar/project.ymlandTokenTrackerWin.csprojautomatically. That drift is precisely what made 0.39.39–0.39.42 unable to cut a desktop build.This PR also proves the #96 branch protection
I attempted the direct push to
mainfirst, as this repo has always done for release commits. It was rejected:That closes the one verification gap #96 shipped with — it was config-read-back only, because I would not land an unreviewed commit on
mainto test it. Release commits now go through a PR like everything else.Test plan
npm run ci:local— exit 0, 841/841 root, 255 dashboardunpriced_models: []npm pack --dry-runcontainsdashboard/dist/index.html+ both service scriptsnpm viewconfirms 0.39.43 on the registryprepublishOnly: 2,522 models, 0 rate changes, only_meta.generated_atconfigured: false)