Skip to content

fix: recompile token-usage-analyzer lock file#1586

Merged
lpcox merged 1 commit intomainfrom
fix/recompile-token-usage-analyzer
Apr 1, 2026
Merged

fix: recompile token-usage-analyzer lock file#1586
lpcox merged 1 commit intomainfrom
fix/recompile-token-usage-analyzer

Conversation

@lpcox
Copy link
Copy Markdown
Collaborator

@lpcox lpcox commented Apr 1, 2026

Problem

The token-usage-analyzer.lock.yml has a frontmatter hash mismatch with its source .md file, causing the workflow to fail at the "Check workflow file timestamps" activation step.

Failed run: https://github.com/github/gh-aw-firewall/actions/runs/23873051548

The .md file was updated via GitHub web UI review suggestions in PR #1557, but the .lock.yml was not recompiled before merging.

Fix

Recompiled the lock file with gh aw compile to sync the frontmatter hash.

@lpcox
fix: recompile token-usage-analyzer lock file
7ba8270 
The .md source was updated via GitHub web UI review suggestions
but the .lock.yml was not recompiled before merging, causing a
frontmatter hash mismatch at activation time.

Fixes: https://github.com/github/gh-aw-firewall/actions/runs/23873051548

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@lpcox lpcox requested a review from Mossaka as a code owner April 1, 2026 22:38
Copilot AI review requested due to automatic review settings April 1, 2026 22:38
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 85.80% 85.90% 📈 +0.10%
Statements 85.68% 85.78% 📈 +0.10%
Functions 86.66% 86.66% ➡️ +0.00%
Branches 78.47% 78.52% 📈 +0.05%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/docker-manager.ts 85.8% → 86.2% (+0.41%) 85.3% → 85.8% (+0.40%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

Copilot AI approved these changes Apr 1, 2026
View reviewed changes
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

Recompiles the generated Agentic Workflows lock file for token-usage-analyzer to bring it back in sync with its source .md (resolving the frontmatter hash mismatch that was breaking workflow activation).

Changes:

  • Updated gh-aw-metadata.frontmatter_hash and regenerated compiled artifacts (prompt/safe-outputs/mcp heredoc identifiers).
  • Synced the compiled skip-if-match query to the source workflow definition (label:token-usage-report).

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

Smoke Test Results

Overall: PASS

💥 [THE END] — Illustrated by Smoke Claude for issue #1586

@github-actions github-actions bot mentioned this pull request Apr 1, 2026
Closed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

Smoke test results for run 23874305630:

✅ GitHub MCP — #1585 "Feat/token usage analyzer workflow", #1573 "[WIP] Add environment variables section to CLI reference"
✅ Playwright — github.com title contains "GitHub"
✅ File write — /tmp/gh-aw/agent/smoke-test-copilot-23874305630.txt created
✅ Bash verify — file content confirmed

Overall: PASS | PR by @lpcox, no assignees

📰 BREAKING: Report filed by Smoke Copilot for issue #1586

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

Smoke Test: GitHub Actions Services Connectivity

Check Status
Redis PING (host.docker.internal:6379) ✅ PONG
PostgreSQL ready (host.docker.internal:5432) ✅ accepting connections
PostgreSQL SELECT 1 (smoketest db) ✅ returned 1

All checks passed. (redis-cli unavailable; Redis verified via raw socket.)

🔌 Service connectivity validated by Smoke Services

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

🏗️ Build Test Suite Results

Ecosystem Project Build/Install Tests Status
Bun elysia ✅ 1/1 passed ✅ PASS
Bun hono ✅ 1/1 passed ✅ PASS
C++ fmt ✅ N/A ✅ PASS
C++ json ✅ N/A ✅ PASS
Deno oak N/A 1/1 passed ✅ PASS
Deno std N/A 1/1 passed ✅ PASS
.NET hello-world ✅ N/A ✅ PASS
.NET json-parse ✅ N/A ✅ PASS
Go color ✅ 1/1 passed ✅ PASS
Go env ✅ 1/1 passed ✅ PASS
Go uuid ✅ 1/1 passed ✅ PASS
Java gson ✅ 1/1 passed ✅ PASS
Java caffeine ✅ 1/1 passed ✅ PASS
Node.js clsx ✅ all passed ✅ PASS
Node.js execa ✅ all passed ✅ PASS
Node.js p-limit ✅ all passed ✅ PASS
Rust fd ✅ 1/1 passed ✅ PASS
Rust zoxide ✅ 1/1 passed ✅ PASS

Overall: 8/8 ecosystems passed — ✅ PASS

Generated by Build Test Suite for issue #1586 · ◷

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

Smoke test results:

  • feat: add daily token usage analyzer workflow ✅
  • [Test Coverage] Add 100% test coverage for logs-audit command ✅
  • Safe Inputs GH CLI (safeinputs-gh) ❌ (tool unavailable)
  • Playwright github.com title contains GitHub ❌ (MCP EACCES)
  • Tavily search ❌ (Tavily MCP unavailable)
  • File write/read + bash cat ✅
  • Discussion oracle comment ❌ (github-discussion-query unavailable)
  • npm ci && npm run build ✅
    Overall status: FAIL

🔮 The oracle has spoken through Smoke Codex

@lpcox lpcox merged commit 0199067 into main Apr 1, 2026
63 of 64 checks passed
@lpcox lpcox deleted the fix/recompile-token-usage-analyzer branch April 1, 2026 23:01
This was referenced Apr 1, 2026
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot approved these changes

@Mossaka Mossaka Awaiting requested review from Mossaka Mossaka is a code owner

Assignees

No one assigned

Labels

build-test smoke-claude smoke-copilot smoke-services

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lpcox