build(deps): Bump @primer/octicons-react from 19.26.0 to 19.28.1#147
Merged
asizikov merged 1 commit intoJun 14, 2026
Merged
Conversation
Bumps [@primer/octicons-react](https://github.com/primer/octicons) from 19.26.0 to 19.28.1. - [Release notes](https://github.com/primer/octicons/releases) - [Changelog](https://github.com/primer/octicons/blob/main/CHANGELOG.md) - [Commits](primer/octicons@v19.26.0...v19.28.1) --- updated-dependencies: - dependency-name: "@primer/octicons-react" dependency-version: 19.28.1 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
asizikov
approved these changes
Jun 14, 2026
xrvk
added a commit
to xrvk/billing-preview
that referenced
this pull request
Jun 17, 2026
…edit policy (#5) * build(deps): align with upstream dep bumps Match github/copilot-billing-preview dependency versions: - @primer/octicons-react 19.26.0 -> 19.28.1 (upstream github#147) - vitest 4.1.5 -> 4.1.8 (upstream github#128) - @vitejs/plugin-react 6.0.1 -> 6.0.2 (upstream github#127) - typescript-eslint 8.59.2 -> 8.60.1 (upstream github#126) - @types/node 25.7.0 -> 25.9.2 (upstream github#116) - actions/checkout v6.0.2 -> v6.0.3 (upstream github#130) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * refactor: add static chart legend items Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> (cherry picked from commit 8183710) * feat: show cumulative AIC usage on overview Port upstream 16f3574 from github/copilot-billing-preview, adapted for our fork: - Add the cumulative AIC gross cost chart (included vs additional) at the top of the OverviewView chart section as an additive UX win, rather than replacing the existing Daily Requests & AI Credits chart. - Preserve the hasPruUsage PRU-conditional rendering from 97109fd. - Skip the upstream "(Pooled) included credits are coming" banner re-introduction since d9ed68e removed it from this fork. - Expose the included AIC pool size from runPipeline (new capacity() accessor on PooledAic/IndividualAic allocators, new PipelineResult.includedAicPoolSize) and thread it through App.tsx state so OverviewView can compute the included value line. Co-authored-by: Anton Sizikov <asizikov@github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: reject reports without headers Port upstream 7412559 from github/copilot-billing-preview, adapted for our fork. Upstream rewrote validation around its new report-adapter layer; we keep the simpler validateFileHeader and just throw the existing InvalidReportError when streamLines yields no non-empty content. Same user-visible behavior: an empty CSV now surfaces "We couldn't read this report" instead of silently producing an empty preview. Co-authored-by: Anton Sizikov <asizikov@github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat: add included credit policy types and period-based resolution Port the user-visible effect of upstream 08a2b7d/6ba9623/8d70792/0e93440 from github/copilot-billing-preview, condensed for this fork. Adds src/pipeline/includedCreditsPolicy.ts with three named policies: - TRANSITION_PERIOD_INCLUDED_CREDITS_POLICY (Business 300 PRU/3000 AIC, Enterprise 1000 PRU/7000 AIC) for current PRU-era reports. - NATIVE_AI_CREDITS_SUMMER_PROMO_INCLUDED_CREDITS_POLICY (Business 1900/3000, Enterprise 3900/7000) for native AIC reports whose period starts before 2026-09-01. - NATIVE_AI_CREDITS_STANDARD_INCLUDED_CREDITS_POLICY (Business 1900/1900, Enterprise 3900/3900) for native AIC reports starting on or after 2026-09-01. resolveIncludedCreditsPolicy takes our existing ReportFormat-style quota unit ('pru' | 'aic') plus an optional report period and returns the right policy. Upstream resolves off their reportAdapters layer; we key off the fork's filename-derived report format instead. This sub- commit only adds the module + tests; allocator wiring lands next. Co-authored-by: Anton Sizikov <asizikov@github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat: resolve included credit policy from report period Thread reportFormat through runPipeline into the AIC included-credits allocator so calculateAicIncludedCreditsContext can pick the right policy from the report's quota unit and date range: - PRU reports keep the transition-period policy (Business 300/3000, Enterprise 1000/7000). - Native AIC reports whose period starts before 2026-09-01 use the summer promo policy (Business 1900/3000, Enterprise 3900/7000). - Native AIC reports on or after 2026-09-01 use the standard policy (Business 1900/1900, Enterprise 3900/3900). App.tsx now forwards detectReportFormatFromFileName(file.name) into PipelineOptions.reportFormat. Adapted from upstream 08a2b7d, 6ba9623, 8d70792, 0e93440. Co-authored-by: Anton Sizikov <asizikov@github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: apply resolved policy to license summary and seat editor The first pass at the policy plumbing only threaded the resolved policy into AIC allocation. License summary math (App.tsx and UsersView) still used the transition-period defaults, so post 2026-09-01 native AIC reports would have reported 3000/7000 per seat in the license table while the chart pool reflected the correct 1900/3900 values. Now createAicIncludedCreditsAllocator stores the policy, runPipeline exposes it on PipelineResult, App.tsx threads it into state and passes it to UsersView, and calculateLicenseSummary plus the seat editor in UsersView use the per-seat included-credit values from that policy. Also drops the QuotaUnitFormat alias and three restating test comments flagged by review, and extends coverage to a boundary-spanning AIC report, the runPipeline reportFormat option, and additional resolver edge cases (null, empty, malformed, 2026-09-02, 2025-12-31). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Anton Sizikov <asizikov@github.com>
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 free
to 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.
Bumps @primer/octicons-react from 19.26.0 to 19.28.1.
Release notes
Sourced from @primer/octicons-react's releases.
Changelog
Sourced from @primer/octicons-react's changelog.
Commits
6d9dcd9Version Packages (#1216)9d7b366Save stack-add-16.svg (#1221)4e8731cFix optimize pathspec to match icons in root directory (#1220)c6a8229Scope SVG optimize workflow to only changed icons (#1218)378d7afRemove hard-coded fill from StackRemove and StackCheck icons (#1215)f17b0beBump@tootallnate/oncefrom 2.0.0 to 2.0.1 in /lib/octicons_react (#1213)fef9dedVersion Packages (#1214)eddab3fFix VSCode icon and remove unnecessary size variants (#1208)067ee62Bump lodash from 4.17.23 to 4.18.1 in /lib/octicons_react (#1201)7ee4aafVersion Packages (#1210)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)