CHANGES:
- Add ensureOrgSelected to auth-guard.ts for all auth flows (existing & new sessions)
- Add ensureOrgSelected to parse-prd flow in commands.js (after authentication) -- NOT WORKING.
- Add ensureOrgSelected to init.js cloud storage flow (using shared utility)
- Remove duplicate 'Authentication successful!' boxen messages (auth.command.ts, auth-guard.ts)
- Export authenticateWithBrowserMFA and ensureOrgSelected from @tm/cli utils
- init.js now uses shared authenticateWithBrowserMFA instead of custom OAuth flow
- auth-guard.ts now checks context.orgId before making API calls (optimization)
FIXES:
- Org selection now happens after login in parse-prd, init, and export commands
- Single source of truth for browser auth with MFA support
- Removed redundant auth UI code from init.js
KNOWN ISSUES:
- tm init and tm export may hang after completion (Supabase auto-refresh timer)
- Root cause: AuthManager/Supabase client keeps event loop alive
- tm auth login works because it uses setupContextInteractive from ContextCommand
- Proper fix would be to add cleanup method to SupabaseAuthClient to stop auto-refresh
- Workaround (process.exit) was attempted but reverted as too dirty
The hanging issue requires further investigation into how auth login handles
cleanup vs how ensureAuthenticated/ensureOrgSelected interact with Supabase.
The sendTeamInvitations function was treating 200 OK responses as errors
because it expected a 'success' field that the API doesn't always return.
- Check for invitations array in response body as success indicator
- Handle various response structures (invitations, data.invitations, data)
- Only treat as error when response is not OK or has explicit error field
- Create shared authenticateWithBrowserMFA utility in auth-ui.ts
- Update auth.command.ts to use shared utility for tm auth login
- Update auth-guard.ts to use shared utility for parse-prd/export
- Fix oauth-service.ts to NOT call onError for MFA_REQUIRED
(MFA requirement is a continuation, not a failure)
All login paths now use the same MFA-aware browser auth flow:
- tm auth login
- tm parse-prd (Bring it to Hamster)
- tm export
* fix: treat cancelled tasks as complete in statistics and dependencies
Fixes#1392
## Changes
- Add TERMINAL_COMPLETE_STATUSES constant including 'done', 'completed', 'cancelled'
- Update calculateTaskStatistics() to include cancelled tasks in completion %
- Update calculateSubtaskStatistics() to include cancelled subtasks in completion %
- Update calculateDependencyStatistics() to treat cancelled tasks as satisfied dependencies
- Update findNextTask() to treat cancelled tasks as complete for dependency resolution
## Impact
- Dashboard now correctly shows 100% for projects with only done/cancelled tasks
- Tasks depending on cancelled tasks are no longer permanently blocked
- Dependency metrics accurately reflect project state
## Tests
- Added comprehensive test suite with RED-GREEN-REFACTOR approach
- 9 tests covering completion %, subtask %, and dependency resolution
- All tests pass after fixes applied
* style: apply biome formatting
* style: apply biome formatting to all files
* fix: display fraction should show all terminal complete tasks, not just done
Previously the completion percentage correctly counted cancelled tasks
as complete (e.g., 100% for 14 done + 1 cancelled), but the fraction
display used stats.done which only counted 'done' status, showing
"100% 14/15" instead of "100% 15/15".
Added completedCount field to TaskStatistics interface to track all
terminal complete tasks (done + completed + cancelled) and updated
display to use this field for accurate fraction representation.
Updates:
- TaskStatistics interface now includes completedCount field
- calculateTaskStatistics() sets completedCount for display
- calculateSubtaskStatistics() sets completedCount for display
- Display lines now use completedCount instead of done count
- Added completedCount assertions to all relevant tests
Related to #1392
* refactor: centralize terminal status logic in @tm/core
Following code review feedback, extracted TERMINAL_COMPLETE_STATUSES
and isTaskComplete() to @tm/core as single source of truth. This
prevents duplication and potential drift between CLI and MCP layers.
Changes:
- Added TERMINAL_COMPLETE_STATUSES constant to @tm/core constants
- Added isTaskComplete() helper function to @tm/core constants
- Updated dashboard.component.ts to import from tm-core via CLI util
- Updated list.command.ts to import from tm-core via CLI util
- Updated task-loader.service.ts to use shared isTaskComplete()
- Created CLI convenience re-export in utils/task-status.ts
Architecture:
- Business logic now lives in @tm/core (proper layer)
- CLI provides convenience re-exports for ergonomics
- MCP already delegates to @tm/core (no changes needed)
Addresses review comments from @Crunchyman-ralph on PR #1393
* fix: remove unused TaskStatus import in task-loader.service
TypeScript error: TaskStatus was imported but no longer used after
refactoring to use isTaskComplete() helper.
* fix: correct subtask display and progress bar for cancelled tasks
Fixed two display bugs identified by cursor bot:
1. Subtask completion display now shows completedCount instead of done
- Before: "Completed: 3/4" (only counted done tasks)
- After: "Completed: 4/4" (counts done + cancelled as terminal complete)
2. Progress bar no longer double-counts cancelled tasks
- Before: Green section (completionPercentage) + gray cancelled section
- After: Green section includes cancelled (as terminal complete status)
- Removed separate cancelled section from progress bar
- Updated section numbers in comments (2-6 instead of 2-7)
These changes ensure visual consistency with the fix that treats cancelled
tasks as terminal complete for percentage calculations and dependency resolution.
Fixes cursor bot comments on PR #1393
* style: run biome formatter
* fix: keep cancelled tasks visually distinct in progress bar
Per reviewer feedback from Crunchyman-ralph, cancelled tasks should remain
visible as a separate gray section in the progress bar for better visibility,
even though they count as complete for the percentage calculation.
Changes:
- Green section shows 'done' tasks only
- Gray section shows 'cancelled' tasks (visually distinct but terminal complete)
- completionPercentage still includes both done and cancelled (correct)
- This prevents double-counting while maintaining visual clarity
Addresses: https://github.com/eyaltoledano/claude-task-master/pull/1393#issuecomment-3517115852
* fix: add done property to StatusBreakdown interface
TypeScript error: StatusBreakdown interface was missing 'done' property
required by the progress bar visualization code.
* fix: Populate done field in calculateStatusBreakdown for progress bar
The calculateStatusBreakdown() function was missing the calculation for the
'done' field, causing statusBreakdown.done to always be undefined. This meant
the green section of the progress bar never rendered, making completed tasks
invisible in the visual representation.
Addresses cursor bot feedback on PR #1393.
---------
Co-authored-by: Ben Coombs <bjcoombs@users.noreply.github.com>
Removes `"version": ""` from internal workspace packages to fix
changesets validation errors. These private packages don't need
explicit version fields as they use workspace protocol.
Fixes validation errors:
- Package "task-master-ai" must depend on the current version of "@tm/ai-sdk-provider-grok-cli": "" vs "*"
- Package "task-master-ai" must depend on the current version of "@tm/cli": "" vs "*"
- Package "@tm/cli" must depend on the current version of "@tm/core": "" vs "*"
- Package "extension" must depend on the current version of "@tm/core": "" vs "*"
- Package "@tm/mcp" must depend on the current version of "@tm/core": "" vs "*"
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
