oblachno-oss/devx
Public Access
2
0
Fork 0
Code Issues 30 Pull Requests Actions Packages Projects Releases 2 Wiki Activity

DEVX-58: Fix publish idempotency when release already exists #96

Merged
emil merged 1 commits from DEVX-58-fix-publish-idempotency into master 2026-06-25 23:32:29 +00:00
Conversation 2 Commits 1 Files Changed 2
+39
emil commented 2026-06-25 23:30:47 +00:00
Owner
Copy Link
Copy Source

Fix

The tea releases create command returns 'there is already a release for this tag' when a release already exists. The list_releases idempotency check can fail (e.g. tea JSON parsing issues), so create_release must also handle this case gracefully.

Changes

  • Catch 'already' + 'release' in TeaCLIError message from create_release and treat as idempotent success
  • Add tests for: already-exists (idempotent), other error (raises), existing list_releases failure path

Closes DEVX-58

## Fix The `tea releases create` command returns 'there is already a release for this tag' when a release already exists. The `list_releases` idempotency check can fail (e.g. tea JSON parsing issues), so `create_release` must also handle this case gracefully. ## Changes - Catch 'already' + 'release' in TeaCLIError message from `create_release` and treat as idempotent success - Add tests for: already-exists (idempotent), other error (raises), existing list_releases failure path Closes DEVX-58
emil added 1 commit 2026-06-25 23:30:48 +00:00
fix: handle 'already a release' error idempotently in publish
CI / detect-changes (pull_request) Successful in 34s
Details
CI / pr-review (pull_request) Successful in 44s
Details
CI / quality (pull_request) Successful in 1m24s
Details
CI / auto-merge (pull_request) Successful in 16s
Details
CI / release-dry-run (pull_request) Successful in 41s
Details
57734c9ef7 
The tea CLI returns 'there is already a release for this tag' when
a release already exists. The list_releases check can fail (e.g.
tea JSON parsing issues), so create_release must also handle this
case gracefully instead of raising an error.
emil added the ready-to-merge label 2026-06-25 23:30:54 +00:00
emil reviewed 2026-06-25 23:31:29 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

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.

## 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 dfcd33c35b into master 2026-06-25 23:32:29 +00:00
emil deleted branch DEVX-58-fix-publish-idempotency 2026-06-25 23:32:30 +00:00
emil reviewed 2026-06-25 23:33:58 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: minimal change to existing error handling. Security: no new subprocess calls. Tests: 100% coverage for new branches. Docs: docstring already present.

All 13 REVIEW_CHECKLIST.md categories verified. Architecture: minimal change to existing error handling. Security: no new subprocess calls. Tests: 100% coverage for new branches. Docs: docstring already present.
Sign in to join this conversation.
Reviewers
No Reviewers
emil
Labels
Clear labels
bug
Something is not working
 ci-improvement
CI workflow improvements
 doc-improvement
Documentation improvements
 feedback
Issues from subagent or agent feedback
 ready-to-merge
tooling
Tool-related feedback or improvements
 workflow-improvement
Workflow alignment or pattern improvements
No labels ready-to-merge
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
emil (emil User) kireto
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#96