chore: tidy PR for merge — resolve TODOs, fix misplaced import

- Remove push trigger (was for dev testing only)
- Restore concurrency group (was commented out for dev)
- Move misplaced import in qa-analyze-pr.ts to top of file

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

.github/workflows/pr-qa.yaml

@@ -10,9 +10,6 @@
 name: 'PR: QA'
 
 on:
-  # TODO: remove push trigger before merge
-  push:
-    branches: [sno-skills, sno-qa-*]
   pull_request:
     types: [labeled]
     branches: [main]
@@ -24,10 +21,9 @@ on:
         options: [focused, full]
         default: focused
 
-# TODO: restore concurrency group before merge (stuck-run issue on dev branches)
-# concurrency:
-#   group: ${{ github.workflow }}-${{ github.ref }}
-#   cancel-in-progress: true
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
 
 jobs:
   resolve-matrix:

scripts/qa-analyze-pr.ts

@@ -19,6 +19,7 @@
 import { execSync } from 'node:child_process'
 import { mkdirSync, readFileSync, writeFileSync } from 'node:fs'
 import { resolve } from 'node:path'
+import { fileURLToPath } from 'node:url'
 
 import { GoogleGenerativeAI } from '@google/generative-ai'
 
@@ -784,8 +785,6 @@ async function analyzePr(opts: Options) {
   console.warn(`  After:  ${afterPath}`)
 }
 
-import { fileURLToPath } from 'node:url'
-
 function isExecutedAsScript(metaUrl: string): boolean {
   const modulePath = fileURLToPath(metaUrl)
   const scriptPath = process.argv[1] ? resolve(process.argv[1]) : ''