Retrospective fixes for issues that caused multiple CI cycles:
configure_repo: Parse owner/repo from DEVX_REPO_NAME when it contains a slash — prevents 404 when workflows set the full path
auto_merge: Warn when .taskid file disagrees with branch task ID — prevents silent mismatches when branching from an existing branch
config: Remove default oblachno-oss for REPO_OWNER — projects must set DEVX_REPO_OWNER explicitly (prevents silent 404s)
AGENTS.md: Document task ID resolution, always() requirement for auto-merge, and all DEVX_ env vars
Tests: Add coverage for slash parsing and stale .taskid warning
Closes DEVX-34
## Summary
Retrospective fixes for issues that caused multiple CI cycles:
- **configure_repo**: Parse `owner/repo` from `DEVX_REPO_NAME` when it contains a slash — prevents 404 when workflows set the full path
- **auto_merge**: Warn when `.taskid` file disagrees with branch task ID — prevents silent mismatches when branching from an existing branch
- **config**: Remove default `oblachno-oss` for `REPO_OWNER` — projects must set `DEVX_REPO_OWNER` explicitly (prevents silent 404s)
- **AGENTS.md**: Document task ID resolution, `always()` requirement for auto-merge, and all `DEVX_` env vars
- **Tests**: Add coverage for slash parsing and stale `.taskid` warning
Closes DEVX-34
Retrospective fixes addressing CI/CD friction. All 13 categories verified. Architecture: slash parsing is backward-compatible, warning-only for .taskid mismatch. Security: no hardcoded secrets, no shell=True. Tests: 1019 unit tests pass, new tests for slash parsing and stale .taskid warning. Docs: AGENTS.md updated with task ID resolution, always() requirement, and all DEVX_ env vars.
Retrospective fixes addressing CI/CD friction. All 13 categories verified. Architecture: slash parsing is backward-compatible, warning-only for .taskid mismatch. Security: no hardcoded secrets, no shell=True. Tests: 1019 unit tests pass, new tests for slash parsing and stale .taskid warning. Docs: AGENTS.md updated with task ID resolution, always() requirement, and all DEVX_ env vars.
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: OK
- Tests: OK
- Commit conventions: OK
No issues found by automated checks.
---
**Auto-merge:** If all CI checks pass, this PR will be merged automatically.
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: OK
- 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 3f2d19d7ac into master2026-06-24 16:41:23 +00:00
emil
deleted branch DEVX-34-retrospective-fixes2026-06-24 16:41:23 +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
Summary
Retrospective fixes for issues that caused multiple CI cycles:
owner/repofromDEVX_REPO_NAMEwhen it contains a slash — prevents 404 when workflows set the full path.taskidfile disagrees with branch task ID — prevents silent mismatches when branching from an existing branchoblachno-ossforREPO_OWNER— projects must setDEVX_REPO_OWNERexplicitly (prevents silent 404s)always()requirement for auto-merge, and allDEVX_env vars.taskidwarningCloses DEVX-34
Retrospective fixes addressing CI/CD friction. All 13 categories verified. Architecture: slash parsing is backward-compatible, warning-only for .taskid mismatch. Security: no hardcoded secrets, no shell=True. Tests: 1019 unit tests pass, new tests for slash parsing and stale .taskid warning. Docs: AGENTS.md updated with task ID resolution, always() requirement, and all DEVX_ env vars.
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.