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

DEVX-40: Remove .taskid file: branch-name-only task ID resolution #64

Merged
emil merged 0 commits from DEVX-40-remove-taskid-file into master 2026-06-24 20:02:20 +00:00
Conversation 3 Commits - Files Changed -
emil commented 2026-06-24 19:52:40 +00:00
Owner
Copy Link
Copy Source

Summary

Removes the .taskid file fallback from auto_merge.py, making the branch name the sole source of truth for task ID resolution.

Problem

The .taskid file caused recurring issues:

  • Stale entries when branches were reused
  • Mismatches between branch name and .taskid causing auto-merge failures
  • Required manual updates when creating new PRs from existing branches

Solution

  • Branch name is the only source of task ID (e.g., DEVX-31-fix-bug → DEVX-31)
  • .taskid file is gitignored in all repos
  • A deprecation warning is printed if a stale .taskid file exists
  • Removed .taskid from classify_changes.py workflow-only list
  • Updated tests and documentation

Files changed

  • src/devx/ci/auto_merge.py — removed .taskid fallback in read_taskid()
  • src/devx/ci/classify_changes.py — removed .taskid from workflow-only list
  • .gitignore — added .taskid
  • AGENTS.md — updated task ID resolution docs
  • tests/unit/test_auto_merge.py — updated tests

Closes DEVX-40

## Summary Removes the `.taskid` file fallback from `auto_merge.py`, making the branch name the sole source of truth for task ID resolution. ### Problem The `.taskid` file caused recurring issues: - Stale entries when branches were reused - Mismatches between branch name and `.taskid` causing auto-merge failures - Required manual updates when creating new PRs from existing branches ### Solution - Branch name is the only source of task ID (e.g., `DEVX-31-fix-bug` → `DEVX-31`) - `.taskid` file is gitignored in all repos - A deprecation warning is printed if a stale `.taskid` file exists - Removed `.taskid` from `classify_changes.py` workflow-only list - Updated tests and documentation ### Files changed - `src/devx/ci/auto_merge.py` — removed `.taskid` fallback in `read_taskid()` - `src/devx/ci/classify_changes.py` — removed `.taskid` from workflow-only list - `.gitignore` — added `.taskid` - `AGENTS.md` — updated task ID resolution docs - `tests/unit/test_auto_merge.py` — updated tests Closes DEVX-40
emil reviewed 2026-06-24 19:53:10 +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: 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.

## 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 reviewed 2026-06-24 19:57:17 +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: 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.

## 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 changed title from DEVX-40: Remove .taskid file fallback, use branch name only to DEVX-40: Remove .taskid file: branch-name-only task ID resolution 2026-06-24 19:58:35 +00:00
emil reviewed 2026-06-24 20:01:51 +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: 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.

## 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 cf85964877 into master 2026-06-24 20:02:20 +00:00
emil deleted branch DEVX-40-remove-taskid-file 2026-06-24 20:02:21 +00:00
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
refactoring
Large refactoring PR (bypasses PR size check)
 tooling
Tool-related feedback or improvements
 workflow-improvement
Workflow alignment or pattern improvements
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
emil (emil User) emo kireto
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#64