intake(lote-B): import upstream PR #1859#10
Open
nsalvacao wants to merge 7 commits intobaseline/develop-sync-2026-02-17from
Open
intake(lote-B): import upstream PR #1859#10nsalvacao wants to merge 7 commits intobaseline/develop-sync-2026-02-17from
nsalvacao wants to merge 7 commits intobaseline/develop-sync-2026-02-17from
Conversation
When task recovery fails (e.g., stuck subtasks with wrong file paths), the UI now provides clear feedback and an option to unstick subtasks. Changes: - Add error toast with stuck subtask count when recovery fails - Add "Unstick Subtasks" button in TaskWarnings component - Display stuck subtask reasons (file validation failures, etc.) - Add CLI --unstick command for manual recovery - Add get_stuck_subtasks() method to RecoveryManager Backend: - apps/backend/cli/main.py: Add --unstick flag - apps/backend/services/recovery.py: Add get_stuck_subtasks() Frontend: - Add IPC handlers TASK_GET_STUCK_INFO and TASK_UNSTICK_SUBTASKS - Add preload APIs getStuckInfo and unstickSubtasks - Update TaskWarnings to show stuck subtasks with unstick button - Update TaskCard to show error toast on recovery failure Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Fix Ruff I001: combine duplicate imports in cli/main.py - Fix Ruff F811: remove duplicate get_stuck_subtasks method in recovery.py - Fix bug where unstickSubtasks toast never fired (return raw IPCResult without flattening) - Add race condition fix in TaskWarnings useEffect with ignore flag - Remove redundant existsSync checks (TOCTOU fix) - Use Object.keys() instead of for...in for prototype safety - Use path aliases instead of deep relative imports - Extract StuckSubtaskInfo into shared type Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Ensures --unstick is rejected when combined with other action flags like --merge, --review, --discard, or --create-pr. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Wraps the clear_stuck_subtasks call in try/except and exits with non-zero status on failure, providing clear error message to user. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Both get_stuck_subtasks and clear_stuck_subtasks are now protected by the same exception handler. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ake/lote-B-pr-1859
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Upstream intake PR for isolated review.
intake/lote-B-pr-1859baseline/develop-sync-2026-02-17Checklist: