Clean dist/ before build — build_package() now removes dist/ before building. This prevents 409 Conflict errors when stale packages from previous builds exist in dist/ (e.g. when the publish workflow is re-run after a partial failure).
Add workflow_dispatch trigger — The publish workflow now supports manual triggering with a tag input. This is needed because tag pushes from within the release workflow don't trigger the publish workflow in Gitea (workflows don't trigger other workflows). With workflow_dispatch, we can manually dispatch publish for existing tags.
Closes DEVX-38
## Summary
Two fixes for the publish workflow:
1. **Clean dist/ before build** — `build_package()` now removes `dist/` before building. This prevents 409 Conflict errors when stale packages from previous builds exist in `dist/` (e.g. when the publish workflow is re-run after a partial failure).
2. **Add `workflow_dispatch` trigger** — The publish workflow now supports manual triggering with a `tag` input. This is needed because tag pushes from within the release workflow don't trigger the publish workflow in Gitea (workflows don't trigger other workflows). With `workflow_dispatch`, we can manually dispatch publish for existing tags.
Closes DEVX-38
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 cb037aa69c into master2026-06-24 19:02:03 +00:00
emil
deleted branch DEVX-38-fix-publish-workflow2026-06-24 19:02:04 +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
Two fixes for the publish workflow:
Clean dist/ before build —
build_package()now removesdist/before building. This prevents 409 Conflict errors when stale packages from previous builds exist indist/(e.g. when the publish workflow is re-run after a partial failure).Add
workflow_dispatchtrigger — The publish workflow now supports manual triggering with ataginput. This is needed because tag pushes from within the release workflow don't trigger the publish workflow in Gitea (workflows don't trigger other workflows). Withworkflow_dispatch, we can manually dispatch publish for existing tags.Closes DEVX-38
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 dist/ before build is correct approach, workflow_dispatch with fallback expression is idiomatic. Code quality: minimal changes, well-commented. Security: no secrets exposed, token passed via env. i18n: no new user-facing strings. Testing: 26 tests pass with 100% coverage, new test for dist cleanup. Performance: negligible. UX: enables manual publish dispatch. Documentation: inline comments explain rationale. Workflow compliance: conventional commit, DEVX-38 prefix. Maintainability: straightforward. Resource management: properly cleans dist/. Backwards compatibility: fully compatible. Logging: N/A.