| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Warning
The focused changes preserve override precedence, cover the changed defaults with tests, and have no identified blocking issues.
0 open findings
What changed in this PRMakes disabling TRAP caching the default when CodeQL overlay analysis is active, while preserving explicit workflow overrides.
Changes:
| File | Description |
|---|---|
| src/feature-flags.ts | Removes the flag and its environment-variable configuration. |
| src/config-utils.ts | Makes default selection synchronous and updates its caller. |
| src/config-utils.test.ts | Covers both overlay modes, explicit overrides, and runner defaults. |
| init/action.yml | Clarifies the input documentation. |
🧠 Review effort: Max
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Sorry, something went wrong.
There was a problem hiding this comment.
The changes look good. I just have one question about the effects on overlay analysis if TRAP caching is explicitly enabled. Maybe I knew the answer previously, but forgot.
Sorry, something went wrong.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
| Back | FazBrowse Home | New Git URL |
This removes the fully shipped OverlayAnalysisDisableTrapCaching feature flag and its environment variable. When overlay analysis is active, the Action now disables TRAP caching by default because overlay analysis supersedes it.
An explicit trap-caching: true or trap-caching: false input remains the highest-priority override. The default now also applies to manually enabled overlay analysis on GitHub Enterprise Server.
This does not add a changelog entry because the v4.34.0 changelog already announced the rollout.
Risk assessment
For internal use only. Please select the risk level of this change:
Which use cases does this change impact?
Workflow types:
Products:
Environments:
How did/will you validate this change?
If something goes wrong after this change is released, what are the mitigation and rollback strategies?
How will you know if something goes wrong after this change is released?
Are there any special considerations for merging or releasing this change?
Merge / deployment checklist