fix: address Copilot review comments on self-reflection PR

- Use scan_labels input in gh pr list query (was declared but unused)
- Add copilot CLI install step to workflow (prevents failure on fresh runner)
- Change simulation template from ESM import to CommonJS require (matches
  repo's module system)
- Replace ts-node recommendation with plain JS default and Node.js
  --experimental-strip-types for TS (no external tooling dependency)
- Replace non-existent fetch_webpage tool reference with runTerminal + curl

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>

Author: YunchuWang

.github/skills/js-fact-checking/SKILL.md

@@ -35,7 +35,7 @@ Clearly state the specific behavioral claim you need to verify. Examples:
 Search the following authoritative sources **in priority order**:
 
 1. **Node.js Official Docs** — https://nodejs.org/api/
-   - Use `fetch_webpage` tool to retrieve the relevant API page
+   - Use `runTerminal` with `curl` to retrieve the relevant API page
    - Look for the specific section about the API in question
    - Pay attention to "Stability" markers and version-specific behavior
 

.github/skills/simulation/SKILL.md

@@ -60,12 +60,12 @@ Write a **single TypeScript or JavaScript file** that:
 
 ### Script Template
 
-```typescript
+```javascript
 // Simulation: <one-line description of what we're verifying>
 // Question: <the specific behavioral question>
 // Expected: <what we expect to happen if our assumption is correct>
 
-import { EventEmitter } from "events";
+const { EventEmitter } = require("events");
 
 function main() {
   console.log("=== SIMULATION START ===");
@@ -114,13 +114,10 @@ main();
 ```bash
 cd /tmp/dt-simulation-<timestamp>
 
-# For TypeScript files:
-npx ts-node --swc simulation.ts
-
-# For plain JavaScript files:
+# Preferred: plain JavaScript (no external tooling needed)
 node simulation.js
 
-# If ts-node is not available, use Node.js directly with ESM:
+# For TypeScript files (requires Node.js >= 22.6.0):
 node --experimental-strip-types simulation.ts
 ```
 

.github/workflows/self-reflection.yaml

@@ -16,6 +16,12 @@ on:
           Leave empty to scan all merged PRs.
         required: false
         type: string
+      install_copilot:
+        description: >-
+          Whether to install copilot CLI (set to false if pre-installed on runner).
+        required: false
+        type: boolean
+        default: true
 
 permissions:
   contents: write
@@ -61,13 +67,23 @@ jobs:
 
           echo "Fetching merged PRs since ${SCAN_SINCE}..."
 
+          # Build label filter if provided
+          LABEL_FILTER=""
+          if [ -n "${{ inputs.scan_labels }}" ]; then
+            IFS=',' read -ra LABELS <<< "${{ inputs.scan_labels }}"
+            for label in "${LABELS[@]}"; do
+              LABEL_FILTER="${LABEL_FILTER} --label \"$(echo "$label" | xargs)\""
+            done
+          fi
+
           # Fetch merged PRs with reviews and comments
-          MERGED_PRS=$(gh pr list \
+          MERGED_PRS=$(eval gh pr list \
             --state merged \
             --limit 100 \
             --search "merged:>=${SCAN_SINCE}" \
+            ${LABEL_FILTER} \
             --json number,title,body,mergedAt,url,files,labels \
-            --jq 'sort_by(.mergedAt) | reverse' \
+            --jq "'sort_by(.mergedAt) | reverse'" \
             2>/dev/null || echo "[]")
 
           PR_COUNT=$(echo "$MERGED_PRS" | jq 'length')
@@ -107,6 +123,12 @@ jobs:
           GH_TOKEN: ${{ github.token }}
           GH_REPO: ${{ github.repository }}
 
+      - name: 📦 Install Copilot CLI
+        if: inputs.install_copilot != false
+        run: |
+          npm install -g @github/copilot
+          copilot --version
+
       - name: 🪞 Run Self-Reflection Agent
         run: |
           SCAN_SINCE="${{ steps.scan-window.outputs.scan_since }}"