fix(ai-qa): stop dual delivery jobs; honor update-linked-issues on reopen

[Hicham Zinalabdin (hichamza)]

Fixes two Bugbot findings on the delivery-hygiene work that landed in #58.

Dual trigger (high)

The documented caller and the self-test registered both push to main and pull_request: types: [closed]. Every merge therefore ran two full delivery jobs (health, review, publish, cleanup) that could race on issues and labels.

Canonical caller (and self-test) is now:

on:
  pull_request:
    types: [closed]
  issues:
    types: [closed]

push remains a legacy merged-delivery-only alias. Do not combine it with pull_request: closed. The action still accepts push so existing callers that cannot use PR-closed keep working; we did not skip merged PR-closed in code, because that is now the canonical path.

update-linked-issues on issues.closed (medium)

The premature-reopen step ran whenever mode was issue-closed, ignoring update-linked-issues. Setting that input false still reopened issues and posted comments.

• shouldReopenPrematureClose returns { reopen: false, reason: 'update-linked-issues-off' } when the flag is false / 'false' (omit keeps the default on).
• The YAML step is gated with inputs.update-linked-issues == 'true'.
• The script also passes the input into the policy (belt and suspenders).

Unit tests: node --test "ai-qa/lib/**/*.test.js" — 29 pass.

  

✅ ai-qa: PASS

Deploy health: healthy · findings P0 0 · P1 0 · P2 0 · P3 0

Last updated by ai-qa · QA run

[mtm-bot]

✅ ai-qa: PASS

Merge commit 85e96cdd7cb7

Signal	Result
Deploy health (https://raw.githubusercontent.com/EdulyCom/github-actions/main/README.md)	✅ healthy
QA review	⚠️ did not run — reporting deploy-health signal only
Head branch	will delete hz/ai-qa-hygiene-bugbot-e361

Posted by mtm-bot[bot] · workflow run