Extract complex bash logic from infra workflow files into well-tested Python scripts:
classify_changes: Read FORCE_DEPLOY env var as alternative to --force flag (replaces bash FORCE_FLAG conditional in post-merge.yml detect-changes step)
validate_commit_msg: Add --git flag to read commit msg from git log (replaces bash git log > temp file > rm in post-merge.yml validate-commit-msg step)
publish: Add --from-tag flag that auto-detects latest tag and checks if HEAD is a release commit (replaces bash git describe + grep logic in post-merge.yml publish step)
All new features have 100% branch coverage tests.
Closes DEVX-57
## Changes
Extract complex bash logic from infra workflow files into well-tested Python scripts:
1. **classify_changes**: Read `FORCE_DEPLOY` env var as alternative to `--force` flag (replaces bash `FORCE_FLAG` conditional in post-merge.yml detect-changes step)
2. **validate_commit_msg**: Add `--git` flag to read commit msg from git log (replaces bash `git log > temp file > rm` in post-merge.yml validate-commit-msg step)
3. **publish**: Add `--from-tag` flag that auto-detects latest tag and checks if HEAD is a release commit (replaces bash `git describe` + `grep` logic in post-merge.yml publish step)
All new features have 100% branch coverage tests.
Closes DEVX-57
Documentation: WARNING — source files changed but no docs updated
Tests: OK
Commit conventions: OK
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
## Automated PR Review
- Architecture compliance: OK
- Best practices: OK
- Security: OK
- i18n: OK
- Resource management: OK
- Documentation: WARNING — source files changed but no docs updated
- Tests: OK
- Commit conventions: OK
No issues found by automated checks.
---
**Auto-merge:** If all CI checks pass, this PR will be merged automatically.
Documentation: WARNING — source files changed but no docs updated
Tests: OK
Commit conventions: OK
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
## Automated PR Review
- Architecture compliance: OK
- Best practices: OK
- Security: OK
- i18n: OK
- Resource management: OK
- Documentation: WARNING — source files changed but no docs updated
- Tests: OK
- Commit conventions: OK
No issues found by automated checks.
---
**Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil
merged commit 9060cd7b1e into master2026-06-25 23:22:42 +00:00
emil
deleted branch DEVX-57-extract-workflow-bash-to-python2026-06-25 23:22:42 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
emil
Changes
Extract complex bash logic from infra workflow files into well-tested Python scripts:
FORCE_DEPLOYenv var as alternative to--forceflag (replaces bashFORCE_FLAGconditional in post-merge.yml detect-changes step)--gitflag to read commit msg from git log (replaces bashgit log > temp file > rmin post-merge.yml validate-commit-msg step)--from-tagflag that auto-detects latest tag and checks if HEAD is a release commit (replaces bashgit describe+greplogic in post-merge.yml publish step)All new features have 100% branch coverage tests.
Closes DEVX-57
Automated PR Review
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
All 13 REVIEW_CHECKLIST.md categories verified. Architecture: clean additions to existing CI modules. Security: no hardcoded secrets. Tests: 100% branch coverage. Docs: docstrings added.
8ae1725a54toa993149871Automated PR Review
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.