fix: switch quality gate to orchestrator-dispatched workflow_dispatch

[microsasa]

Fixes #168

Problem

The quality gate workflow triggers on pull_request_review: submitted but filters out bot reviews in the pre-activation job. Since the autonomous pipeline has no human reviewers, the quality gate never fires — clean PRs sit open indefinitely.

Example: PR #167 — CI green, Copilot review clean (0 comments), quality gate never ran, PR stuck open for 30+ minutes.

Changes

Quality Gate (.lock.yml + .md)

• Trigger: pull_request_review: submitted → workflow_dispatch with pr_number input
• Removed if: label check on activation (orchestrator handles dispatch timing)
• All safe outputs: added target: "*" (no PR trigger context with workflow_dispatch)
• Updated concurrency group to use inputs.pr_number
• Updated prompt to use inputs.pr_number instead of github.event.pull_request.number
• Added workflow_dispatch to PR context prompt condition

Orchestrator (pipeline-orchestrator.yml)

• Added Step 4: after CI green + 0 open threads → dispatch quality gate
  • Skips if aw-quality-gate-approved label already present
  • Skips if quality gate already in flight
• Added "Quality Gate" to workflow_run trigger list (orchestrator fires after QG completes)

Flow (after this change)

Implementer creates PR
  → CI runs
  → Copilot reviews
  → Orchestrator sees: CI green + 0 threads
  → Orchestrator dispatches Quality Gate
  → Quality Gate approves (or flags for human)
  → Auto-merge triggers

[Copilot]

Pull request overview

This PR changes the “Quality Gate” workflow to be orchestrator-dispatched via workflow_dispatch (with a pr_number input) so clean PRs with zero Copilot comments still get evaluated and can auto-merge.

Changes:

• Converted Quality Gate trigger from pull_request_review: submitted to workflow_dispatch with pr_number.
• Updated Quality Gate safe-outputs config/prompting to work without PR-event context (target: "*", use inputs.pr_number).
• Extended Pipeline Orchestrator to dispatch Quality Gate on the “happy path” (CI green + no unresolved threads) and to react to Quality Gate completion.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File	Description
.github/workflows/quality-gate.md	Updates workflow frontmatter + instructions to accept pr_number and run when dispatched by orchestrator.
.github/workflows/quality-gate.lock.yml	Regenerated compiled workflow reflecting workflow_dispatch, updated concurrency/prompt context/safe-outputs targets.
.github/workflows/pipeline-orchestrator.yml	Adds Quality Gate to workflow_run triggers and adds Step 4 to dispatch Quality Gate when CI/threads are clear.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[microsasa]

Addressed review comments in ac22a2e:

Comment 1 (QG_RUNNING global check): Fixed — dropped the global check entirely. The quality gate's per-PR concurrency group handles dedup.

Comment 2 (activation if guard removed): By design — the orchestrator is the sole dispatcher and already verifies the aw label before calling.

Comment 3 (re-dispatch loop after QG completes): Fixed — added aw-quality-gate-evaluated label. Quality gate adds this for HIGH impact / poor quality decisions. Orchestrator skips dispatch when present.

Comment 4 (no persisted state for non-approvals): Fixed — same as #3. aw-quality-gate-evaluated persists the state.