Skip to content

Disable daily AI credits guardrail for Expert Code Review workflow - #14314

Merged
ViktorHofer merged 2 commits into
mainfrom
copilot/aw-raise-daily-limit
Jul 9, 2026
Merged

ViktorHofer merged 2 commits into
mainfrom
copilot/aw-raise-daily-limit

Conversation

Copilot AI commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

The Expert Code Review (command) agentic workflow hit its per-workflow daily AI Credits guardrail (5.7K used vs. the 5K default cap), blocking /review runs until the 24h window resets. Per the remediation steps in the failure issue, disable the guardrail for this workflow.

Context

  • The guardrail is a per-workflow 24h AIC spending cap defaulting to 5K (GH_AW_DEFAULT_MAX_DAILY_AI_CREDITS).
  • This workflow is slash-command triggered (/review), so runs are already opt-in and bounded by the shared PAT pool budget, making the fixed daily cap unnecessarily restrictive.

Changes Made

  • review.agent.md: added max-daily-ai-credits: -1 to the frontmatter to explicitly disable the guardrail.
  • review.agent.lock.yml: regenerated via gh aw compile (v0.79.8), removing the guardrail wiring:
    • activation-job Check daily workflow token guardrail step, its GH_AW_MAX_DAILY_AI_CREDITS env, and daily_ai_credits_* outputs
    • daily_ai_credits_exceeded gating on the agent and conclusion jobs, plus the safe-output-artifact-client condition and GH_AW_DAILY_AI_CREDITS_* conclusion env
    • updated frontmatter_hash metadata to match the new source
timeout-minutes: 60

max-daily-ai-credits: -1

Testing

  • Regenerated the lock file with the pinned compiler; confirmed the diff against the original is limited strictly to guardrail wiring plus the frontmatter hash, and the result is valid YAML.

Notes

  • Scoped to review.agent only; other agentic workflows retain the default cap.
  • The lock file was regenerated offline by building gh-aw@v0.79.8 from source (the gh aw extension cannot be installed in the sandbox); a maintainer re-running gh aw compile should produce an identical result.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Hello @copilot, I noticed that you’re changing an .swr file or any file under src/Package/MSBuild.VSSetup.. Please make sure to validate this change by an experimental VS insertion. This is accomplished by pushing to an exp/* branch, which requires write permissions to this repo.

Co-authored-by: JanProvaznik <25267098+JanProvaznik@users.noreply.github.com>
Copilot AI changed the title [WIP] Update daily AI credits budget for expert code review Disable daily AI credits guardrail for Expert Code Review workflow Jul 9, 2026
Copilot AI requested a review from JanProvaznik July 9, 2026 15:41
@JanProvaznik

JanProvaznik commented Jul 9, 2026

Copy link
Copy Markdown
Member

@ViktorHofer let's do either this PR or set the GH_AW_DEFAULT_MAX_DAILY_AI_CREDITS repo variable to something higher like 100000
we're hitting this every day

@ViktorHofer
ViktorHofer marked this pull request as ready for review July 9, 2026 19:39
Copilot AI review requested due to automatic review settings July 9, 2026 19:39
@ViktorHofer
ViktorHofer requested a review from a team as a code owner July 9, 2026 19:39
@ViktorHofer
ViktorHofer merged commit ad7e074 into main Jul 9, 2026
14 checks passed
@ViktorHofer
ViktorHofer deleted the copilot/aw-raise-daily-limit branch July 9, 2026 19:39
@ViktorHofer

Copy link
Copy Markdown
Member

Cool, I didn't know about this credits cap. Thanks

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Disables the per-workflow 24h “daily AI Credits” spending cap for the Expert Code Review (command) slash-command workflow (/review), addressing the guardrail exhaustion reported in #14312 by explicitly opting out via frontmatter and updating the compiled lock workflow accordingly.

Changes:

  • Add max-daily-ai-credits: -1 to review.agent.md frontmatter to disable the per-workflow daily AI Credits guardrail.
  • Regenerate review.agent.lock.yml to remove the guardrail job wiring (activation guardrail step, related outputs/env, and downstream gating).
Show a summary per file
File Description
.github/workflows/review.agent.md Adds frontmatter setting to disable the daily AI Credits guardrail for the /review workflow.
.github/workflows/review.agent.lock.yml Updates the compiled workflow to remove the daily AI Credits guardrail step/outputs and related gating logic.

Copilot's findings

  • Files reviewed: 2/2 changed files
  • Comments generated: 0

WarperSan pushed a commit to WarperSan/ThunderPipe that referenced this pull request Sep 17, 2026
Updated
[Microsoft.Build.Utilities.Core](https://github.com/dotnet/msbuild) from
18.9.6 to 18.10.1.

<details>
<summary>Release notes</summary>

_Sourced from [Microsoft.Build.Utilities.Core's
releases](https://github.com/dotnet/msbuild/releases)._

## 18.10.1

## What's Changed
* [vs16.11] Update dependencies from dotnet/arcade by
@​dotnet-maestro[bot] in dotnet/msbuild#13103
* [vs17.12] Update dependencies from dotnet/arcade by
@​dotnet-maestro[bot] in dotnet/msbuild#13796
* [vs17.8] Update dependencies from dotnet/arcade by
@​dotnet-maestro[bot] in dotnet/msbuild#13902
* [vs17.11] Update dependencies from dotnet/arcade by
@​dotnet-maestro[bot] in dotnet/msbuild#13903
* [vs17.12] Update dependencies from dotnet/arcade by
@​dotnet-maestro[bot] in dotnet/msbuild#13909
* [vs17.12] Update dependencies from dotnet/arcade by
@​dotnet-maestro[bot] in dotnet/msbuild#13986
* Add vs18.9 to merge-flow config; retire vs18.3 by @​JanProvaznik in
dotnet/msbuild#14214
* Bump labeler-cache-retention to use issue-labeler v2.1.0 by
@​jeffhandley in dotnet/msbuild#14171
* Bump main to 18.10.0 after vs18.9 snap by @​JanProvaznik in
dotnet/msbuild#14216
* Improve release skill: Phase 2 DARC rules, VMR backflow, deterministic
baseline by @​JanProvaznik in
dotnet/msbuild#14220
* Determinize release: hardcode OptProf baseline + Phase 3.2 baseline
resolver by @​JanProvaznik in
dotnet/msbuild#14222
* Serialize BuildRequestConfiguration.RequestedTargets to fix solution
metaproject MSB4057 in parallel builds by @​ViktorHofer in
dotnet/msbuild#14223
* [main] Update dependencies from nuget/nuget.client by
@​dotnet-maestro[bot] in dotnet/msbuild#14203
* Core support for AbsolutePath/FileInfo/DirectoryInfo and ITaskItem<T>
as task parameters by @​baronfel in
dotnet/msbuild#13971
* [main] Update dependencies from dotnet/roslyn by @​dotnet-maestro[bot]
in dotnet/msbuild#14206
* Fix existence cache kind poisoning by @​AlesProkop in
dotnet/msbuild#14249
* [main] Source code updates from dotnet/dotnet by @​dotnet-maestro[bot]
in dotnet/msbuild#14226
* Don't disable the MSBuild server for /mt builds when node reuse is off
by @​AR-May in dotnet/msbuild#14248
* Enhance expert reviewer guidelines with additional checks. by @​AR-May
in dotnet/msbuild#14255
* [main] Source code updates from dotnet/dotnet by @​dotnet-maestro[bot]
in dotnet/msbuild#14253
* [main] Update dependencies from dotnet/roslyn by @​dotnet-maestro[bot]
in dotnet/msbuild#14268
* [main] Update dependencies from nuget/nuget.client by
@​dotnet-maestro[bot] in dotnet/msbuild#14267
* Bump github/gh-aw-actions/setup from 0.81.6 to 0.82.2 by
@​dependabot[bot] in dotnet/msbuild#14266
* Avoid boxing the struct enumerator in
PropertyDictionary<T>.GetEnumerator() by @​nareshjo in
dotnet/msbuild#14272
* Refresh copy marker when implementation output changes by @​AlesProkop
in dotnet/msbuild#14231
* Send task-host build process environment as delta by @​OvesN in
dotnet/msbuild#14126
* Add regression coverage for metadata newline preservation by
@​VolPlita in dotnet/msbuild#14261
* Fix EmbedInBinlog items with relative paths from child projects by
@​huulinhnguyen-dev in dotnet/msbuild#13990
* Stop requiring VersionPrefix updates in servicing - insert prerelease
versions to VS by @​ViktorHofer in
dotnet/msbuild#14277
* Fix WriteLinesToFile rewriting unchanged file when custom encoding is
used by @​huulinhnguyen-dev in
dotnet/msbuild#14146
* Enable trim/AOT analyzers for Microsoft.Build and clean up annotations
by @​JeremyKuhne in dotnet/msbuild#14064
* [automated] Merge branch 'vs18.9' => 'main' by @​github-actions[bot]
in dotnet/msbuild#14291
* Fix MicroBuild plugin feed URL to use allowed pkgs.dev.azure.com
format by @​AlesProkop in dotnet/msbuild#14295
* Pass ExcludeRestorePackageImports during restore to avoid redundant
evaluations by @​ViktorHofer with @​Copilot in
dotnet/msbuild#14274
* [vs18.7] Update dependencies from dotnet/arcade by
@​dotnet-maestro[bot] in dotnet/msbuild#13988
* Adopt Clever Test Selection (CTS) as parallel, non-blocking PR
pipeline by @​jankratochvilcz in
dotnet/msbuild#14212
* Harden exceptions when connecting to server by @​JanProvaznik in
dotnet/msbuild#14292
* Update MicrosoftBuildVersion in analyzer template by
@​github-actions[bot] in dotnet/msbuild#13886
* Fix MSBuild Server client dropping build result under WaitAny race
(#​14172) by @​JanProvaznik in
dotnet/msbuild#14251
* Partially revert #​13660: remove NuGet RestoreTask transient TaskHost
workaround by @​JanProvaznik in
dotnet/msbuild#14297
* Disable daily AI credits guardrail for Expert Code Review workflow by
@​JanProvaznik with @​Copilot in
dotnet/msbuild#14314
* Localized file check-in by OneLocBuild Task: Build definition ID 9434:
Build ID 14614733 by @​dotnet-bot in
dotnet/msbuild#14246
* Add opt-in partial (stop-after-pass) project evaluation by
@​ViktorHofer in dotnet/msbuild#14290
* Use partial evaluation for -getProperty/-getItem without a target by
@​ViktorHofer in dotnet/msbuild#14296
* [main] Source code updates from dotnet/dotnet by @​dotnet-maestro[bot]
in dotnet/msbuild#14324
* [main] Update dependencies from dotnet/roslyn by @​dotnet-maestro[bot]
in dotnet/msbuild#14333
* [main] Update dependencies from nuget/nuget.client by
@​dotnet-maestro[bot] in dotnet/msbuild#14330
* Bump github/gh-aw-actions/setup from 0.82.2 to 0.82.8 by
@​dependabot[bot] in dotnet/msbuild#14328
* Restrict partial evaluation to ProjectInstance by @​ViktorHofer in
dotnet/msbuild#14340
 ... (truncated)

Commits viewable in [compare
view](dotnet/msbuild@v18.9.6...v18.10.1).
</details>

[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=Microsoft.Build.Utilities.Core&package-manager=nuget&previous-version=18.9.6&new-version=18.10.1)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

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-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will 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 version` will 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 dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

[aw] Expert Code Review (command) exceeded daily AI credits budget

4 participants