Skip to content

chore: promote closure-router REPO_FULL fix to main - #43

Merged
LukasWodka merged 2 commits into
mainfrom
develop
May 15, 2026
Merged

chore: promote closure-router REPO_FULL fix to main#43
LukasWodka merged 2 commits into
mainfrom
develop

Conversation

@LukasWodka

Copy link
Copy Markdown
Contributor

Carries fix#42 onto main so the closure-router stops silently failing on every completed-issue close (and items stop ending up in Cancelled via the built-in fallback).

LukasWodkaand others added 2 commits May 15, 2026 11:21
Issue closed-as-completed events have been silently failing the closure-
router workflow since #38 landed: the Determine step references $REPO_FULL
(via REPO_NAME) and $ORG inside the closing-PR-base lookup, but neither
was declared in the step's env block. With `set -u`, the script aborted
on line 17 with "REPO_FULL: unbound variable" before writing any
status= output, which left the built-in "Item closed" project workflow
to set Status=Cancelled.
Net effect on the kanban: ~37 completed-as-COMPLETED issues sitting in
Cancelled instead of Prod since 2026-05-06.
Fix: declare ORG + REPO_FULL in the Determine step's env block, matching
the Update step. The script now correctly hits the case branch and
emits status=Prod (manual close) or mirror-PR-base.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
fix(closure-router): add ORG + REPO_FULL to Determine step env
@LukasWodka
LukasWodka merged commit 7cd9128 into mainMay 15, 2026
@LukasWodka
LukasWodka deleted the develop branch May 15, 2026 09:22
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@LukasWodka