Skip to content

Remove estimated cost from audit-workflows report#36356

Merged
pelikhan merged 2 commits into
mainfrom
copilot/remove-estimated-cost-audit-report
Jun 1, 2026
Merged

Remove estimated cost from audit-workflows report#36356
pelikhan merged 2 commits into
mainfrom
copilot/remove-estimated-cost-audit-report

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jun 1, 2026

The audit-workflows daily report was generating a "Token & Cost" chart with a cost line and referencing cost analysis in its performance guidelines. Estimated cost figures are noisy/misleading in this context.

Changes

  • audit-workflows.md (prompt body only, no frontmatter changes):
    • Renamed chart 2 from Token & CostToken Usage; dropped the cost line from its description
    • Updated chart save path: token_cost_trends.pngtoken_trends.png
    • Removed costs from the Performance analysis bullet in the Audit Process section

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove estimated cost from audit-workflows report Remove estimated cost from audit-workflows report Jun 1, 2026
Copilot AI requested a review from pelikhan June 1, 2026 23:02
@pelikhan pelikhan marked this pull request as ready for review June 1, 2026 23:15
Copilot AI review requested due to automatic review settings June 1, 2026 23:15
@pelikhan pelikhan merged commit 3bdc029 into main Jun 1, 2026
@pelikhan pelikhan deleted the copilot/remove-estimated-cost-audit-report branch June 1, 2026 23:16
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes estimated cost reporting from the audit-workflows daily report prompt by renaming the “Token & Cost” chart to “Token Usage”, dropping the cost line reference, and removing “costs” from the performance analysis guideline. It also updates the compiled workflow lock file, which results in additional workflow/runtime changes beyond the prompt text update.

Changes:

  • Update audit-workflows.md prompt to remove “estimated cost” from charting and performance analysis guidance.
  • Regenerate audit-workflows.lock.yml, introducing broader workflow/runtime differences (notably cron timing and MCP container strategy) alongside the prompt body hash update.
Show a summary per file
File Description
.github/workflows/audit-workflows.md Removes cost references from the audit report’s charting and performance guidance.
.github/workflows/audit-workflows.lock.yml Regenerated compiled workflow; includes additional runtime/scheduling/containerization changes that should be explicitly acknowledged and reviewed.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Comment on lines 70 to 74
name: "Agentic Workflow Audit Agent"
on:
schedule:
- cron: "6 21 * * *"
- cron: "51 4 * * *"
# Friendly format: daily (scattered)
Comment on lines +1 to +2
# gh-aw-metadata: {"schema_version":"v4","frontmatter_hash":"cd18c35dfadc12bd2ec5d9708151ceef4203a63458cdc2ba743844b3b9ca2171","body_hash":"975c6cd4bcdc7582a0cc51cf1dc2dbdf5cb819352a2b26d3b35785f4cc8daf46","compiler_version":"v0.77.5","strict":true,"agent_id":"claude"}
# gh-aw-manifest: {"version":1,"secrets":["ANTHROPIC_API_KEY","GH_AW_GITHUB_MCP_SERVER_TOKEN","GH_AW_GITHUB_TOKEN","GH_AW_OTEL_GRAFANA_AUTHORIZATION","GH_AW_OTEL_GRAFANA_ENDPOINT","GH_AW_OTEL_SENTRY_AUTHORIZATION","GH_AW_OTEL_SENTRY_ENDPOINT","GITHUB_TOKEN"],"actions":[{"repo":"actions/cache/restore","sha":"27d5ce7f107fe9357f9df03efb73ab90386fccae","version":"v5.0.5"},{"repo":"actions/cache/save","sha":"27d5ce7f107fe9357f9df03efb73ab90386fccae","version":"v5.0.5"},{"repo":"actions/checkout","sha":"de0fac2e4500dabe0009e67214ff5f5447ce83dd","version":"v6.0.2"},{"repo":"actions/download-artifact","sha":"3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c","version":"v8.0.1"},{"repo":"actions/github-script","sha":"3a2844b7e9c422d3c10d287c895573f7108da1b3","version":"v9.0.0"},{"repo":"actions/setup-node","sha":"48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e","version":"v6.4.0"},{"repo":"actions/setup-python","sha":"a309ff8b426b58ec0e2a45f0f869d46889d02405","version":"v6.2.0"},{"repo":"actions/upload-artifact","sha":"043fb46d1a93c77aae656e7c1c64a875d1fc6a0a","version":"v7.0.1"},{"repo":"github/gh-aw-actions/setup","sha":"v0.77.5","version":"v0.77.5"},{"repo":"github/gh-aw/actions/setup-cli","sha":"v0.77.5","version":"v0.77.5"}],"resolution_failures":[{"repo":"github/gh-aw/actions/setup-cli","ref":"v0.77.5","error_type":"dynamic_resolution_failed"}],"containers":[{"image":"alpine:latest","digest":"sha256:5b10f432ef3da1b8d4c7eb6c487f2f5a8f096bc91145e68878dd4a5019afde11","pinned_image":"alpine:latest@sha256:5b10f432ef3da1b8d4c7eb6c487f2f5a8f096bc91145e68878dd4a5019afde11"},{"image":"ghcr.io/github/gh-aw-firewall/agent:0.25.58"},{"image":"ghcr.io/github/gh-aw-firewall/api-proxy:0.25.58"},{"image":"ghcr.io/github/gh-aw-firewall/squid:0.25.58"},{"image":"ghcr.io/github/gh-aw-mcpg:v0.3.22"},{"image":"ghcr.io/github/github-mcp-server:v1.1.0"},{"image":"node:lts-alpine","digest":"sha256:2bdb65ed1dab192432bc31c95f94155ca5ad7fc1392fb7eb7526ab682fa5bf14","pinned_image":"node:lts-alpine@sha256:2bdb65ed1dab192432bc31c95f94155ca5ad7fc1392fb7eb7526ab682fa5bf14"}]}
"agenticworkflows": {
"container": "localhost/gh-aw:dev",
"mounts": ["\${GITHUB_WORKSPACE}:\${GITHUB_WORKSPACE}:rw", "/tmp/gh-aw:/tmp/gh-aw:rw"],
"container": "alpine:latest",
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.

[q] Remove estimated cost from audit-workflows report

3 participants