From f47882778f0dcb35985273c88c149ee684b57ebd Mon Sep 17 00:00:00 2001 From: Aaron Stainback Date: Mon, 27 Apr 2026 12:25:46 -0400 Subject: [PATCH 1/2] ci(low-memory): close 0-diff drift on "What this workflow does" comment block LFG main has comment improvements in low-memory.yml that didn't make it back to AceHack via #47 (which only backported the concurrency + timeout fixes). Close the residual drift: - Add push-to-main bullet at the top of "What this workflow does" - Add backstop parenthetical to schedule bullet - Add ubuntu-slim-was-removed-from-gate.yml note explaining the no- duplicate-run guarantee Per #43's diff-minimization invariant: AceHack and LFG should match on main with any drift rigorously accounted for. After this lands, \`git diff acehack/main..origin/main -- .github/workflows/low-memory.yml\` should return zero. Co-Authored-By: Claude Opus 4.7 --- .github/workflows/low-memory.yml | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/.github/workflows/low-memory.yml b/.github/workflows/low-memory.yml index 0276ee820..7d70412ba 100644 --- a/.github/workflows/low-memory.yml +++ b/.github/workflows/low-memory.yml @@ -35,10 +35,15 @@ # so the per-merge run has no cost downside. # # What this workflow does: -# - Schedule: daily at 06:00 UTC. +# - push to main: runs on every merge (primary trigger). +# - Schedule: daily at 06:00 UTC (backstop for weekends + missed +# pushes). # - workflow_dispatch: manual trigger for ad-hoc verification. # - Single ubuntu-slim leg matching gate.yml's install / build / -# test sequence on the smaller runner. +# test sequence on the smaller runner. ubuntu-slim was REMOVED +# from gate.yml's matrix when the per-merge trigger landed (per +# Codex P2 review on LFG #644) so we don't double-run the slim +# leg on every push. # - Failure = drift on low-memory runners. File a BACKLOG row; # does not block PRs in flight. # From 88cc2d924e3438eca28e27c694b20e65e98713d6 Mon Sep 17 00:00:00 2001 From: Aaron Stainback Date: Mon, 27 Apr 2026 12:30:44 -0400 Subject: [PATCH 2/2] fix(low-memory): match precise push-to-main wording in "What this workflow does" bullet (Copilot P2) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Copilot flagged the bullet's "every merge (primary trigger)" as technically imprecise — the trigger is push.branches:[main] which includes direct/force pushes if ever allowed. Match the more precise wording used in the header (line 4-6) per the suggestion. Now the bullet reads: push to main: runs on every push to main (in practice every merge; primary trigger). Same imprecise wording is on LFG main (came in via LFG #644). Will forward-sync this fix to LFG via the next sibling sync to keep 0-diff. Co-Authored-By: Claude Opus 4.7 --- .github/workflows/low-memory.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/low-memory.yml b/.github/workflows/low-memory.yml index 7d70412ba..5702420dd 100644 --- a/.github/workflows/low-memory.yml +++ b/.github/workflows/low-memory.yml @@ -35,7 +35,8 @@ # so the per-merge run has no cost downside. # # What this workflow does: -# - push to main: runs on every merge (primary trigger). +# - push to main: runs on every push to main (in practice every +# merge; primary trigger). # - Schedule: daily at 06:00 UTC (backstop for weekends + missed # pushes). # - workflow_dispatch: manual trigger for ad-hoc verification.