Skip to content

fix: recompile optimizer lock files to match current .md sources#3073

Merged
lpcox merged 1 commit into
mainfrom
fix/recompile-optimizers
May 13, 2026
Merged

fix: recompile optimizer lock files to match current .md sources#3073
lpcox merged 1 commit into
mainfrom
fix/recompile-optimizers

Conversation

@lpcox
Copy link
Copy Markdown
Collaborator

@lpcox lpcox commented May 13, 2026

Problem

Both token optimizer workflows fail at the activation job with:

ERR_CONFIG: Lock file is outdated! The workflow file frontmatter has changed.

Root cause: PR #3057 removed skip-if-match from both .md files, then PR #3071 recompiled all workflows — but #3071's branch was based on main before #3057 merged. The lock files got compiled with the old frontmatter hash, so the hash check fails on the current .md sources.

Fix

Recompile both optimizer lock files against the current .md sources on main.

Copilot AI review requested due to automatic review settings May 13, 2026 00:07
@lpcox lpcox requested a review from Mossaka as a code owner May 13, 2026 00:07
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 addresses failing “token optimizer” workflows by recompiling their generated .lock.yml files so the embedded frontmatter_hash matches the current .md workflow sources on main, unblocking the activation job hash check.

Changes:

  • Recompiled .github/workflows/copilot-token-optimizer.lock.yml against the current .md source (updated frontmatter_hash and regenerated compiled content).
  • Recompiled .github/workflows/claude-token-optimizer.lock.yml against the current .md source (updated frontmatter_hash and regenerated compiled content).
Show a summary per file
File Description
.github/workflows/copilot-token-optimizer.lock.yml Updates generated lock content (notably frontmatter_hash) to match current copilot-token-optimizer.md.
.github/workflows/claude-token-optimizer.lock.yml Updates generated lock content (notably frontmatter_hash) to match current claude-token-optimizer.md.

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: 0

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results

  • ❌ GitHub MCP
  • ✅ Playwright
  • ✅ File Writing
  • ✅ Bash Tool

Status: 3/4 PASS (gh CLI unavailable)

💥 [THE END] — Illustrated by Smoke Claude

@github-actions
Copy link
Copy Markdown
Contributor

🤖 Smoke Test Results

Test Status
GitHub MCP connectivity ❌ (401 - credentials unavailable in sandbox)
GitHub.com HTTP connectivity ⚠️ (pre-step template vars not expanded)
File write/read ✅ (smoke-test-copilot-25769700901.txt verified)

Overall: PARTIAL — File I/O passed; GitHub MCP and HTTP tests could not be fully verified due to sandbox credential isolation.

Pre-step smoke file content: Smoke test passed for Copilot at Wed May 13 00:07:43 UTC 2026

📰 BREAKING: Report filed by Smoke Copilot

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test: Copilot BYOK (Offline) Mode

Test Status
GitHub MCP connectivity ❌ (401 Bad credentials)
GitHub.com HTTP connectivity ⚠️ Pre-step data not expanded
File write/read ⚠️ Pre-step data not expanded
BYOK inference (api-proxy → api.githubcopilot.com)

Note: Running in BYOK offline mode (COPILOT_OFFLINE=true) via api-proxy → api.githubcopilot.com.

Overall: FAIL — GitHub MCP auth failed; pre-step template variables were not expanded.

🔑 BYOK report filed by Smoke Copilot BYOK

Recompiled all workflows and found 6 lock files with hashes that
don't match their .md source frontmatter:

- claude-token-optimizer.lock.yml (skip-if-match removal)
- copilot-token-optimizer.lock.yml (skip-if-match removal)
- issue-monster.lock.yml (github-script v8→v9 action drop)
- secret-digger-codex.lock.yml (frontmatter change)
- smoke-chroot.lock.yml (setup-go SHA update)
- actions-lock.json (setup-go version update)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@lpcox lpcox force-pushed the fix/recompile-optimizers branch from 18d4be1 to f8cae56 Compare May 13, 2026 00:10
@lpcox lpcox merged commit 0258250 into main May 13, 2026
31 of 33 checks passed
@lpcox lpcox deleted the fix/recompile-optimizers branch May 13, 2026 00:10
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.

2 participants