feat: move ack job to separate workflow to eliminate skipped-check noise

The `ack` job was defined in `demo.yml`, which triggers on both
`pull_request` and `issue_comment`. On every PR push, GitHub would mark
it as "Skipped" — visual noise in the checks list, especially for
non-developer reviewers.

Fix: move `ack` to `glimpse-ack.yml`, which only listens to
`issue_comment`. It never triggers on `pull_request` events, so it
never appears as a skipped check on PRs.

The new file is clearly documented as OPTIONAL — the demo pipeline
works without it. It only adds the immediate 👀 reaction UX.

Also adds `issues: write` to the `demo` job so the hooray/confused
completion reactions via `gh api` work reliably.

https://claude.ai/code/session_01JHYRHYBPULqbjvS5eayG2e

Author: claude

.github/workflows/demo.yml

@@ -17,24 +17,6 @@ on:
     types: [created]
 
 jobs:
-  # Lightweight job: reacts with 👀 immediately — no checkout or install needed.
-  # Runs in parallel with the demo job so the acknowledgment appears as fast as possible.
-  ack:
-    if: >-
-      github.event_name == 'issue_comment' &&
-      github.event.issue.pull_request != null &&
-      contains(github.event.comment.body, '/glimpse')
-    runs-on: ubuntu-latest
-    permissions:
-      issues: write
-    steps:
-      - name: React with eyes
-        env:
-          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        run: |
-          gh api repos/$GITHUB_REPOSITORY/issues/comments/${{ github.event.comment.id }}/reactions \
-            --method POST --field content=eyes || true
-
   demo:
     runs-on: ubuntu-latest
     # Run on PR events, or on issue_comment only if it's a PR comment containing /glimpse
@@ -46,6 +28,7 @@ jobs:
     permissions:
       pull-requests: write
       contents: write
+      issues: write
 
     steps:
       - uses: actions/checkout@v4

.github/workflows/glimpse-ack.yml

@@ -0,0 +1,40 @@
+# GitGlimpse Acknowledge Workflow
+#
+# OPTIONAL — this workflow is not required for git-glimpse to work.
+# The demo pipeline runs and posts its comment regardless of whether
+# this file exists.
+#
+# What it does: reacts with 👀 on the /glimpse comment as soon as a
+# runner is available (~15–30s), giving the commenter immediate visual
+# confirmation that their request was received — before the heavy
+# pipeline steps begin.
+#
+# Why it's a separate file: keeping it separate from the main workflow
+# means it never shows up as a "skipped" check on pull_request events.
+# If it were in the same file as the demo job, GitHub would show an
+# "ack (skipped)" entry in the PR checks on every push — noise for
+# non-developer reviewers.
+#
+# ⚠️  Changes only take effect after merging to main (GitHub reads
+#     issue_comment workflows from the default branch).
+name: GitGlimpse Acknowledge
+
+on:
+  issue_comment:
+    types: [created]
+
+jobs:
+  ack:
+    if: >-
+      github.event.issue.pull_request != null &&
+      contains(github.event.comment.body, '/glimpse')
+    runs-on: ubuntu-latest
+    permissions:
+      issues: write
+    steps:
+      - name: React with eyes
+        env:
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        run: |
+          gh api repos/$GITHUB_REPOSITORY/issues/comments/${{ github.event.comment.id }}/reactions \
+            --method POST --field content=eyes || true