Skip to content

fix(scripts): check prerequisites before issue mutations - #406

Open
woahwhattheheck wants to merge 1 commit into
Stellar-Analysis:mainfrom
woahwhattheheck:submit/stellar-issue-script-preflight-20261006
Open

woahwhattheheck wants to merge 1 commit into
Stellar-Analysis:mainfrom
woahwhattheheck:submit/stellar-issue-script-preflight-20261006

Conversation

@woahwhattheheck

@woahwhattheheck woahwhattheheck commented Oct 6, 2026 •

Copy link
Copy Markdown

The legacy issue generator assumes backend, SDK and mobile source roots in one checkout, while this frontend tree lacks backend/src and mobile/src. The recreation wrapper also closes issues before checking for its named replacement script, create_all_70_detailed.py, which is absent from this tree.

This focused change checks the generator's source roots before its first issue-creation call, anchors its existing subprocess to the script's checkout, and stops the recreation wrapper before any gh call when its named replacement is missing or unreadable. The task payloads, closure loop and manual recreation command remain unchanged. This does not implement backend issue #323 or make issue closure/recreation transactional.

Validation: 3 offline regression tests pass (python -m unittest discover -s scripts/tests -v), including a wrapper invocation from an unrelated directory with a local gh sentinel proving no gh invocation occurs when the replacement is absent. bash -n and Python compilation pass. No real GitHub CLI issue listing, creation or closure was executed.

The scripts' executable modes are preserved; original repository authorship is retained.

@vercel

vercel Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

@woahwhattheheck is attempting to deploy a commit to the christabel888's projects Team on Vercel.

A member of the Team first needs to authorize it.

@woahwhattheheck

Copy link
Copy Markdown
Author

I am claiming consideration for any GrantFox reward applicable to my contribution in this PR for #323, payable to @woahwhattheheck. Please confirm that this PR is associated with the original contributor's campaign record and included in the reward evaluation, and confirm any allocation and payout date once approved. Existing contributors' attribution and any remaining acceptance requirements remain documented in this PR.

This branch has not been deployed

No deployments
Sign up for free to 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