When a branch commit accidentally includes a task ID prefix (e.g. OBL-INFRA-364: fix: ...), the conventional commit regex failed to match, causing the fallback to return the full message. The auto-merge then prepended the task ID again, producing a doubled prefix in the squash-merge commit (e.g. OBL-INFRA-364: OBL-INFRA-364: fix: ...).
Fix: Strip any leading task ID prefix before matching and in the fallback.
Test plan
make pytest-cov — 1578 passed, 100% coverage
make lint-all — clean
Closes DEVX-107
## Summary
When a branch commit accidentally includes a task ID prefix (e.g. `OBL-INFRA-364: fix: ...`), the conventional commit regex failed to match, causing the fallback to return the full message. The auto-merge then prepended the task ID again, producing a doubled prefix in the squash-merge commit (e.g. `OBL-INFRA-364: OBL-INFRA-364: fix: ...`).
**Fix**: Strip any leading task ID prefix before matching and in the fallback.
## Test plan
- [x] `make pytest-cov` — 1578 passed, 100% coverage
- [x] `make lint-all` — clean
Closes DEVX-107
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.
kireto
approved these changes 2026-07-01 09:28:18 +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
kireto
Summary
When a branch commit accidentally includes a task ID prefix (e.g.
OBL-INFRA-364: fix: ...), the conventional commit regex failed to match, causing the fallback to return the full message. The auto-merge then prepended the task ID again, producing a doubled prefix in the squash-merge commit (e.g.OBL-INFRA-364: OBL-INFRA-364: fix: ...).Fix: Strip any leading task ID prefix before matching and in the fallback.
Test plan
make pytest-cov— 1578 passed, 100% coveragemake lint-all— cleanCloses DEVX-107
Automated PR Review
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
Automated PR Review
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
Auto-approved: all CI checks passed (quality, pr-review, release-dry-run).