chore: quality of life improvements

.changeset/auto-update-changelog-highlights.md

@@ -0,0 +1,7 @@
+---
+"task-master-ai": minor
+---
+
+Add changelog highlights to auto-update notifications
+
+When the CLI auto-updates to a new version, it now displays a "What's New" section.

.changeset/dirty-hairs-know.md

@@ -1,5 +0,0 @@
----
-"task-master-ai": patch
----
-
-Improve auth token refresh flow

.changeset/fix-parent-directory-traversal.md

@@ -1,7 +0,0 @@
----
-"task-master-ai": patch
----
-
-Enable Task Master commands to traverse parent directories to find project root from nested paths
-
-Fixes #1301

.changeset/fix-warning-box-alignment.md

@@ -1,5 +0,0 @@
----
-"@tm/cli": patch
----
-
-Fix warning message box width to match dashboard box width for consistent UI alignment

.changeset/light-owls-stay.md

@@ -1,35 +0,0 @@
----
-"task-master-ai": minor
----
-
-Add configurable MCP tool loading to optimize LLM context usage
-
-You can now control which Task Master MCP tools are loaded by setting the `TASK_MASTER_TOOLS` environment variable in your MCP configuration. This helps reduce context usage for LLMs by only loading the tools you need.
-
-**Configuration Options:**
-
-- `all` (default): Load all 36 tools
-- `core` or `lean`: Load only 7 essential tools for daily development
-  - Includes: `get_tasks`, `next_task`, `get_task`, `set_task_status`, `update_subtask`, `parse_prd`, `expand_task`
-- `standard`: Load 15 commonly used tools (all core tools plus 8 more)
-  - Additional tools: `initialize_project`, `analyze_project_complexity`, `expand_all`, `add_subtask`, `remove_task`, `generate`, `add_task`, `complexity_report`
-- Custom list: Comma-separated tool names (e.g., `get_tasks,next_task,set_task_status`)
-
-**Example .mcp.json configuration:**
-
-```json
-{
-  "mcpServers": {
-    "task-master-ai": {
-      "command": "npx",
-      "args": ["-y", "task-master-ai"],
-      "env": {
-        "TASK_MASTER_TOOLS": "standard",
-        "ANTHROPIC_API_KEY": "your_key_here"
-      }
-    }
-  }
-}
-```
-
-For complete details on all available tools, configuration examples, and usage guidelines, see the [MCP Tools documentation](https://docs.task-master.dev/capabilities/mcp#configurable-tool-loading).

.changeset/mean-planes-wave.md

@@ -0,0 +1,47 @@
+---
+"task-master-ai": minor
+---
+
+Add Claude Code plugin with marketplace distribution
+
+This release introduces official Claude Code plugin support, marking the evolution from legacy `.claude` directory copying to a modern plugin-based architecture.
+
+## 🎉 New: Claude Code Plugin
+
+Task Master AI commands and agents are now distributed as a proper Claude Code plugin:
+
+- **49 slash commands** with clean naming (`/taskmaster:command-name`)
+- **3 specialized AI agents** (task-orchestrator, task-executor, task-checker)
+- **MCP server integration** for deep Claude Code integration
+
+**Installation:**
+
+```bash
+/plugin marketplace add eyaltoledano/claude-task-master
+/plugin install taskmaster@taskmaster
+```
+
+### The `rules add claude` command no longer copies commands and agents to `.claude/commands/` and `.claude/agents/`. Instead, it now
+
+- Shows plugin installation instructions
+- Only manages CLAUDE.md imports for agent instructions
+- Directs users to install the official plugin
+
+**Migration for Existing Users:**
+
+If you previously used `rules add claude`:
+
+1. The old commands in `.claude/commands/` will continue to work but won't receive updates
+2. Install the plugin for the latest features: `/plugin install taskmaster@taskmaster`
+3. remove old `.claude/commands/` and `.claude/agents/` directories
+
+**Why This Change?**
+
+Claude Code plugins provide:
+
+- ✅ Automatic updates when we release new features
+- ✅ Better command organization and naming
+- ✅ Seamless integration with Claude Code
+- ✅ No manual file copying or management
+
+The plugin system is the future of Task Master AI integration with Claude Code!

.changeset/nice-ways-hope.md

@@ -0,0 +1,17 @@
+---
+"task-master-ai": minor
+---
+
+Add RPG (Repository Planning Graph) method template for structured PRD creation. The new `example_prd_rpg.txt` template teaches AI agents and developers the RPG methodology through embedded instructions, inline good/bad examples, and XML-style tags for structure. This template enables creation of dependency-aware PRDs that automatically generate topologically-ordered task graphs when parsed with Task Master.
+
+Key features:
+- Method-as-template: teaches RPG principles (dual-semantics, explicit dependencies, topological order) while being used
+- Inline instructions at decision points guide AI through each section
+- Good/bad examples for immediate pattern matching
+- Flexible plain-text format with XML-style tags for parseability
+- Critical dependency-graph section ensures correct task ordering
+- Automatic inclusion during `task-master init`
+- Comprehensive documentation at [docs.task-master.dev/capabilities/rpg-method](https://docs.task-master.dev/capabilities/rpg-method)
+- Tool recommendations for code-context-aware PRD creation (Claude Code, Cursor, Gemini CLI, Codex/Grok)
+
+The RPG template complements the existing `example_prd.txt` and provides a more structured approach for complex projects requiring clear module boundaries and dependency chains.

.changeset/plain-falcons-serve.md

@@ -0,0 +1,7 @@
+---
+"task-master-ai": patch
+---
+
+Fix cross-level task dependencies not being saved
+
+Fixes an issue where adding dependencies between subtasks and top-level tasks (e.g., `task-master add-dependency --id=2.2 --depends-on=11`) would report success but fail to persist the changes. Dependencies can now be created in both directions between any task levels.

.changeset/pre.json

@@ -0,0 +1,21 @@
+{
+  "mode": "exit",
+  "tag": "rc",
+  "initialVersions": {
+    "task-master-ai": "0.28.0",
+    "@tm/cli": "",
+    "docs": "0.0.5",
+    "extension": "0.25.5",
+    "@tm/ai-sdk-provider-grok-cli": "",
+    "@tm/build-config": "",
+    "@tm/claude-code-plugin": "0.0.1",
+    "@tm/core": ""
+  },
+  "changesets": [
+    "auto-update-changelog-highlights",
+    "mean-planes-wave",
+    "nice-ways-hope",
+    "plain-falcons-serve",
+    "smart-owls-relax"
+  ]
+}

.changeset/silent-bushes-grow.md

@@ -0,0 +1,5 @@
+---
+"task-master-ai": patch
+---
+
+Improve refresh token when authenticating

.changeset/smart-owls-relax.md

@@ -0,0 +1,16 @@
+---
+"task-master-ai": minor
+---
+
+Enhance `expand_all` to intelligently use complexity analysis recommendations when expanding tasks. 
+
+The expand-all operation now automatically leverages recommendations from `analyze-complexity` to determine optimal subtask counts for each task, resulting in more accurate and context-aware task breakdowns.
+
+Key improvements:
+- Automatic integration with complexity analysis reports
+- Tag-aware complexity report path resolution
+- Intelligent subtask count determination based on task complexity
+- Falls back to defaults when complexity analysis is unavailable
+- Enhanced logging for better visibility into expansion decisions
+
+When you run `task-master expand --all` after `task-master analyze-complexity`, Task Master now uses the recommended subtask counts from the complexity analysis instead of applying uniform defaults, ensuring each task is broken down according to its actual complexity.

.env.example

@@ -14,4 +14,4 @@ OLLAMA_API_KEY=YOUR_OLLAMA_API_KEY_HERE
 VERTEX_PROJECT_ID=your-gcp-project-id
 VERTEX_LOCATION=us-central1
 # Optional: Path to service account credentials JSON file (alternative to API key)
-GOOGLE_APPLICATION_CREDENTIALS=/path/to/service-account-credentials.json
+GOOGLE_APPLICATION_CREDENTIALS=/path/to/service-account-credentials.json

CHANGELOG.md

@@ -1,94 +1,5 @@
 # task-master-ai
 
-## 0.29.0
-
-### Minor Changes
-
-- [#1286](https://github.com/eyaltoledano/claude-task-master/pull/1286) [`f12a16d`](https://github.com/eyaltoledano/claude-task-master/commit/f12a16d09649f62148515f11f616157c7d0bd2d5) Thanks [@Crunchyman-ralph](https://github.com/Crunchyman-ralph)! - Add changelog highlights to auto-update notifications
-
-  When the CLI auto-updates to a new version, it now displays a "What's New" section.
-
-- [#1293](https://github.com/eyaltoledano/claude-task-master/pull/1293) [`3010b90`](https://github.com/eyaltoledano/claude-task-master/commit/3010b90d98f3a7d8636caa92fc33d6ee69d4bed0) Thanks [@Crunchyman-ralph](https://github.com/Crunchyman-ralph)! - Add Claude Code plugin with marketplace distribution
-
-  This release introduces official Claude Code plugin support, marking the evolution from legacy `.claude` directory copying to a modern plugin-based architecture.
-
-  ## 🎉 New: Claude Code Plugin
-
-  Task Master AI commands and agents are now distributed as a proper Claude Code plugin:
-  - **49 slash commands** with clean naming (`/taskmaster:command-name`)
-  - **3 specialized AI agents** (task-orchestrator, task-executor, task-checker)
-  - **MCP server integration** for deep Claude Code integration
-
-  **Installation:**
-
-  ```bash
-  /plugin marketplace add eyaltoledano/claude-task-master
-  /plugin install taskmaster@taskmaster
-  ```
-
-  ### The `rules add claude` command no longer copies commands and agents to `.claude/commands/` and `.claude/agents/`. Instead, it now
-  - Shows plugin installation instructions
-  - Only manages CLAUDE.md imports for agent instructions
-  - Directs users to install the official plugin
-
-  **Migration for Existing Users:**
-
-  If you previously used `rules add claude`:
-  1. The old commands in `.claude/commands/` will continue to work but won't receive updates
-  2. Install the plugin for the latest features: `/plugin install taskmaster@taskmaster`
-  3. remove old `.claude/commands/` and `.claude/agents/` directories
-
-  **Why This Change?**
-
-  Claude Code plugins provide:
-  - ✅ Automatic updates when we release new features
-  - ✅ Better command organization and naming
-  - ✅ Seamless integration with Claude Code
-  - ✅ No manual file copying or management
-
-  The plugin system is the future of Task Master AI integration with Claude Code!
-
-- [#1285](https://github.com/eyaltoledano/claude-task-master/pull/1285) [`2a910a4`](https://github.com/eyaltoledano/claude-task-master/commit/2a910a40bac375f9f61d797bf55597303d556b48) Thanks [@Crunchyman-ralph](https://github.com/Crunchyman-ralph)! - Add RPG (Repository Planning Graph) method template for structured PRD creation. The new `example_prd_rpg.txt` template teaches AI agents and developers the RPG methodology through embedded instructions, inline good/bad examples, and XML-style tags for structure. This template enables creation of dependency-aware PRDs that automatically generate topologically-ordered task graphs when parsed with Task Master.
-
-  Key features:
-  - Method-as-template: teaches RPG principles (dual-semantics, explicit dependencies, topological order) while being used
-  - Inline instructions at decision points guide AI through each section
-  - Good/bad examples for immediate pattern matching
-  - Flexible plain-text format with XML-style tags for parseability
-  - Critical dependency-graph section ensures correct task ordering
-  - Automatic inclusion during `task-master init`
-  - Comprehensive documentation at [docs.task-master.dev/capabilities/rpg-method](https://docs.task-master.dev/capabilities/rpg-method)
-  - Tool recommendations for code-context-aware PRD creation (Claude Code, Cursor, Gemini CLI, Codex/Grok)
-
-  The RPG template complements the existing `example_prd.txt` and provides a more structured approach for complex projects requiring clear module boundaries and dependency chains.
-
-- [#1287](https://github.com/eyaltoledano/claude-task-master/pull/1287) [`90e6bdc`](https://github.com/eyaltoledano/claude-task-master/commit/90e6bdcf1c59f65ad27fcdfe3b13b9dca7e77654) Thanks [@Crunchyman-ralph](https://github.com/Crunchyman-ralph)! - Enhance `expand_all` to intelligently use complexity analysis recommendations when expanding tasks.
-
-  The expand-all operation now automatically leverages recommendations from `analyze-complexity` to determine optimal subtask counts for each task, resulting in more accurate and context-aware task breakdowns.
-
-  Key improvements:
-  - Automatic integration with complexity analysis reports
-  - Tag-aware complexity report path resolution
-  - Intelligent subtask count determination based on task complexity
-  - Falls back to defaults when complexity analysis is unavailable
-  - Enhanced logging for better visibility into expansion decisions
-
-  When you run `task-master expand --all` after `task-master analyze-complexity`, Task Master now uses the recommended subtask counts from the complexity analysis instead of applying uniform defaults, ensuring each task is broken down according to its actual complexity.
-
-### Patch Changes
-
-- [#1191](https://github.com/eyaltoledano/claude-task-master/pull/1191) [`aaf903f`](https://github.com/eyaltoledano/claude-task-master/commit/aaf903ff2f606c779a22e9a4b240ab57b3683815) Thanks [@Crunchyman-ralph](https://github.com/Crunchyman-ralph)! - Fix cross-level task dependencies not being saved
-
-  Fixes an issue where adding dependencies between subtasks and top-level tasks (e.g., `task-master add-dependency --id=2.2 --depends-on=11`) would report success but fail to persist the changes. Dependencies can now be created in both directions between any task levels.
-
-- [#1299](https://github.com/eyaltoledano/claude-task-master/pull/1299) [`4c1ef2c`](https://github.com/eyaltoledano/claude-task-master/commit/4c1ef2ca94411c53bcd2a78ec710b06c500236dd) Thanks [@Crunchyman-ralph](https://github.com/Crunchyman-ralph)! - Improve refresh token when authenticating
-
-## 0.29.0-rc.1
-
-### Patch Changes
-
-- [#1299](https://github.com/eyaltoledano/claude-task-master/pull/1299) [`a6c5152`](https://github.com/eyaltoledano/claude-task-master/commit/a6c5152f20edd8717cf1aea34e7c178b1261aa99) Thanks [@Crunchyman-ralph](https://github.com/Crunchyman-ralph)! - Improve refresh token when authenticating
-
 ## 0.29.0-rc.0
 
 ### Minor Changes

README.md

@@ -119,7 +119,6 @@ MCP (Model Control Protocol) lets you run Task Master directly from your editor.
       "command": "npx",
       "args": ["-y", "task-master-ai"],
       "env": {
-        // "TASK_MASTER_TOOLS": "all", // Options: "all", "standard", "core", or comma-separated list of tools
         "ANTHROPIC_API_KEY": "YOUR_ANTHROPIC_API_KEY_HERE",
         "PERPLEXITY_API_KEY": "YOUR_PERPLEXITY_API_KEY_HERE",
         "OPENAI_API_KEY": "YOUR_OPENAI_KEY_HERE",
@@ -149,7 +148,6 @@ MCP (Model Control Protocol) lets you run Task Master directly from your editor.
       "command": "npx",
       "args": ["-y", "task-master-ai"],
       "env": {
-        // "TASK_MASTER_TOOLS": "all", // Options: "all", "standard", "core", or comma-separated list of tools
         "ANTHROPIC_API_KEY": "YOUR_ANTHROPIC_API_KEY_HERE",
         "PERPLEXITY_API_KEY": "YOUR_PERPLEXITY_API_KEY_HERE",
         "OPENAI_API_KEY": "YOUR_OPENAI_KEY_HERE",
@@ -198,7 +196,7 @@ Initialize taskmaster-ai in my project
 
 #### 5. Make sure you have a PRD (Recommended)
 
-For **new projects**: Create your PRD at `.taskmaster/docs/prd.txt`.
+For **new projects**: Create your PRD at `.taskmaster/docs/prd.txt`  
 For **existing projects**: You can use `scripts/prd.txt` or migrate with `task-master migrate`
 
 An example PRD template is available after initialization in `.taskmaster/templates/example_prd.txt`.
@@ -284,76 +282,6 @@ task-master generate
 task-master rules add windsurf,roo,vscode
 ```
 
-## Tool Loading Configuration
-
-### Optimizing MCP Tool Loading
-
-Task Master's MCP server supports selective tool loading to reduce context window usage. By default, all 36 tools are loaded (~21,000 tokens) to maintain backward compatibility with existing installations.
-
-You can optimize performance by configuring the `TASK_MASTER_TOOLS` environment variable:
-
-### Available Modes
-
-| Mode | Tools | Context Usage | Use Case |
-|------|-------|--------------|----------|
-| `all` (default) | 36 | ~21,000 tokens | Complete feature set - all tools available |
-| `standard` | 15 | ~10,000 tokens | Common task management operations |
-| `core` (or `lean`) | 7 | ~5,000 tokens | Essential daily development workflow |
-| `custom` | Variable | Variable | Comma-separated list of specific tools |
-
-### Configuration Methods
-
-#### Method 1: Environment Variable in MCP Configuration
-
-Add `TASK_MASTER_TOOLS` to your MCP configuration file's `env` section:
-
-```jsonc
-{
-  "mcpServers": {  // or "servers" for VS Code
-    "task-master-ai": {
-      "command": "npx",
-      "args": ["-y", "--package=task-master-ai", "task-master-ai"],
-      "env": {
-        "TASK_MASTER_TOOLS": "standard",  // Options: "all", "standard", "core", "lean", or comma-separated list
-        "ANTHROPIC_API_KEY": "your-key-here",
-        // ... other API keys
-      }
-    }
-  }
-}
-```
-
-#### Method 2: Claude Code CLI (One-Time Setup)
-
-For Claude Code users, you can set the mode during installation:
-
-```bash
-# Core mode example (~70% token reduction)
-claude mcp add task-master-ai --scope user \
-  --env TASK_MASTER_TOOLS="core" \
-  -- npx -y task-master-ai@latest
-
-# Custom tools example
-claude mcp add task-master-ai --scope user \
-  --env TASK_MASTER_TOOLS="get_tasks,next_task,set_task_status" \
-  -- npx -y task-master-ai@latest
-```
-
-### Tool Sets Details
-
-**Core Tools (7):** `get_tasks`, `next_task`, `get_task`, `set_task_status`, `update_subtask`, `parse_prd`, `expand_task`
-
-**Standard Tools (15):** All core tools plus `initialize_project`, `analyze_project_complexity`, `expand_all`, `add_subtask`, `remove_task`, `generate`, `add_task`, `complexity_report`
-
-**All Tools (36):** Complete set including project setup, task management, analysis, dependencies, tags, research, and more
-
-### Recommendations
-
-- **New users**: Start with `"standard"` mode for a good balance
-- **Large projects**: Use `"core"` mode to minimize token usage
-- **Complex workflows**: Use `"all"` mode or custom selection
-- **Backward compatibility**: If not specified, defaults to `"all"` mode
-
 ## Claude Code Support
 
 Task Master now supports Claude models through the Claude Code CLI, which requires no API key:

apps/cli/CHANGELOG.md

@@ -11,13 +11,6 @@
 
 ### Patch Changes
 
-- Updated dependencies []:
-  - @tm/core@null
-
-## null
-
-### Patch Changes
-
 - Updated dependencies []:
   - @tm/core@null
 

apps/cli/src/commands/auth.command.ts

@@ -143,7 +143,7 @@ export class AuthCommand extends Command {
 	 */
 	private async executeStatus(): Promise<void> {
 		try {
-			const result = this.displayStatus();
+			const result = await this.displayStatus();
 			this.setLastResult(result);
 		} catch (error: any) {
 			this.handleError(error);
@@ -171,8 +171,8 @@ export class AuthCommand extends Command {
 	/**
 	 * Display authentication status
 	 */
-	private displayStatus(): AuthResult {
+	private async displayStatus(): Promise<AuthResult> {
-		const credentials = this.authManager.getCredentials();
+		const credentials = await this.authManager.getCredentials();
 
 		console.log(chalk.cyan('\n🔐 Authentication Status\n'));
 
@@ -325,7 +325,7 @@ export class AuthCommand extends Command {
 			]);
 
 			if (!continueAuth) {
-				const credentials = this.authManager.getCredentials();
+				const credentials = await this.authManager.getCredentials();
 				ui.displaySuccess('Using existing authentication');
 
 				if (credentials) {
@@ -490,7 +490,7 @@ export class AuthCommand extends Command {
 	/**
 	 * Get current credentials (for programmatic usage)
 	 */
-	getCredentials(): AuthCredentials | null {
+	getCredentials(): Promise<AuthCredentials | null> {
 		return this.authManager.getCredentials();
 	}
 

apps/cli/src/commands/context.command.ts

@@ -115,7 +115,7 @@ export class ContextCommand extends Command {
 	 */
 	private async executeShow(): Promise<void> {
 		try {
-			const result = this.displayContext();
+			const result = await this.displayContext();
 			this.setLastResult(result);
 		} catch (error: any) {
 			this.handleError(error);
@@ -126,7 +126,7 @@ export class ContextCommand extends Command {
 	/**
 	 * Display current context
 	 */
-	private displayContext(): ContextResult {
+	private async displayContext(): Promise<ContextResult> {
 		// Check authentication first
 		if (!this.authManager.isAuthenticated()) {
 			console.log(chalk.yellow('✗ Not authenticated'));
@@ -139,7 +139,7 @@ export class ContextCommand extends Command {
 			};
 		}
 
-		const context = this.authManager.getContext();
+		const context = await this.authManager.getContext();
 
 		console.log(chalk.cyan('\n🌍 Workspace Context\n'));
 
@@ -250,7 +250,7 @@ export class ContextCommand extends Command {
 			]);
 
 			// Update context
-			this.authManager.updateContext({
+			await this.authManager.updateContext({
 				orgId: selectedOrg.id,
 				orgName: selectedOrg.name,
 				// Clear brief when changing org
@@ -263,7 +263,7 @@ export class ContextCommand extends Command {
 			return {
 				success: true,
 				action: 'select-org',
-				context: this.authManager.getContext() || undefined,
+				context: (await this.authManager.getContext()) || undefined,
 				message: `Selected organization: ${selectedOrg.name}`
 			};
 		} catch (error) {
@@ -284,7 +284,7 @@ export class ContextCommand extends Command {
 			}
 
 			// Check if org is selected
-			const context = this.authManager.getContext();
+			const context = await this.authManager.getContext();
 			if (!context?.orgId) {
 				ui.displayError(
 					'No organization selected. Run "tm context org" first.'
@@ -343,7 +343,7 @@ export class ContextCommand extends Command {
 			if (selectedBrief) {
 				// Update context with brief
 				const briefName = `Brief ${selectedBrief.id.slice(0, 8)}`;
-				this.authManager.updateContext({
+				await this.authManager.updateContext({
 					briefId: selectedBrief.id,
 					briefName: briefName
 				});
@@ -353,12 +353,12 @@ export class ContextCommand extends Command {
 				return {
 					success: true,
 					action: 'select-brief',
-					context: this.authManager.getContext() || undefined,
+					context: (await this.authManager.getContext()) || undefined,
 					message: `Selected brief: ${selectedBrief.name}`
 				};
 			} else {
 				// Clear brief selection
-				this.authManager.updateContext({
+				await this.authManager.updateContext({
 					briefId: undefined,
 					briefName: undefined
 				});
@@ -368,7 +368,7 @@ export class ContextCommand extends Command {
 				return {
 					success: true,
 					action: 'select-brief',
-					context: this.authManager.getContext() || undefined,
+					context: (await this.authManager.getContext()) || undefined,
 					message: 'Cleared brief selection'
 				};
 			}
@@ -491,7 +491,7 @@ export class ContextCommand extends Command {
 
 			// Update context: set org and brief
 			const briefName = `Brief ${brief.id.slice(0, 8)}`;
-			this.authManager.updateContext({
+			await this.authManager.updateContext({
 				orgId: brief.accountId,
 				orgName,
 				briefId: brief.id,
@@ -508,7 +508,7 @@ export class ContextCommand extends Command {
 			this.setLastResult({
 				success: true,
 				action: 'set',
-				context: this.authManager.getContext() || undefined,
+				context: (await this.authManager.getContext()) || undefined,
 				message: 'Context set from brief'
 			});
 		} catch (error: any) {
@@ -613,7 +613,7 @@ export class ContextCommand extends Command {
 				};
 			}
 
-			this.authManager.updateContext(context);
+			await this.authManager.updateContext(context);
 			ui.displaySuccess('Context updated');
 
 			// Display what was set
@@ -631,7 +631,7 @@ export class ContextCommand extends Command {
 			return {
 				success: true,
 				action: 'set',
-				context: this.authManager.getContext() || undefined,
+				context: (await this.authManager.getContext()) || undefined,
 				message: 'Context updated'
 			};
 		} catch (error) {
@@ -682,7 +682,7 @@ export class ContextCommand extends Command {
 	/**
 	 * Get current context (for programmatic usage)
 	 */
-	getContext(): UserContext | null {
+	getContext(): Promise<UserContext | null> {
 		return this.authManager.getContext();
 	}
 

apps/cli/src/ui/components/next-task.component.ts

@@ -6,7 +6,7 @@
 import chalk from 'chalk';
 import boxen from 'boxen';
 import type { Task } from '@tm/core/types';
-import { getComplexityWithColor, getBoxWidth } from '../../utils/ui.js';
+import { getComplexityWithColor } from '../../utils/ui.js';
 
 /**
  * Next task display options
@@ -113,7 +113,7 @@ export function displayRecommendedNextTask(
 			borderColor: '#FFA500', // Orange color
 			title: chalk.hex('#FFA500')('⚡ RECOMMENDED NEXT TASK ⚡'),
 			titleAlignment: 'center',
-			width: getBoxWidth(0.97),
+			width: process.stdout.columns * 0.97,
 			fullscreen: false
 		})
 	);

apps/cli/src/ui/components/suggested-steps.component.ts

@@ -5,7 +5,6 @@
 
 import chalk from 'chalk';
 import boxen from 'boxen';
-import { getBoxWidth } from '../../utils/ui.js';
 
 /**
  * Display suggested next steps section
@@ -25,7 +24,7 @@ export function displaySuggestedNextSteps(): void {
 				margin: { top: 0, bottom: 1 },
 				borderStyle: 'round',
 				borderColor: 'gray',
-				width: getBoxWidth(0.97)
+				width: process.stdout.columns * 0.97
 			}
 		)
 	);

apps/cli/src/utils/ui.spec.ts

@@ -1,158 +0,0 @@
-/**
- * CLI UI utilities tests
- * Tests for apps/cli/src/utils/ui.ts
- */
-
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import type { MockInstance } from 'vitest';
-import { getBoxWidth } from './ui.js';
-
-describe('CLI UI Utilities', () => {
-	describe('getBoxWidth', () => {
-		let columnsSpy: MockInstance;
-		let originalDescriptor: PropertyDescriptor | undefined;
-
-		beforeEach(() => {
-			// Store original descriptor if it exists
-			originalDescriptor = Object.getOwnPropertyDescriptor(
-				process.stdout,
-				'columns'
-			);
-
-			// If columns doesn't exist or isn't a getter, define it as one
-			if (!originalDescriptor || !originalDescriptor.get) {
-				const currentValue = process.stdout.columns || 80;
-				Object.defineProperty(process.stdout, 'columns', {
-					get() {
-						return currentValue;
-					},
-					configurable: true
-				});
-			}
-
-			// Now spy on the getter
-			columnsSpy = vi.spyOn(process.stdout, 'columns', 'get');
-		});
-
-		afterEach(() => {
-			// Restore the spy
-			columnsSpy.mockRestore();
-
-			// Restore original descriptor or delete the property
-			if (originalDescriptor) {
-				Object.defineProperty(process.stdout, 'columns', originalDescriptor);
-			} else {
-				delete (process.stdout as any).columns;
-			}
-		});
-
-		it('should calculate width as percentage of terminal width', () => {
-			columnsSpy.mockReturnValue(100);
-			const width = getBoxWidth(0.9, 40);
-			expect(width).toBe(90);
-		});
-
-		it('should use default percentage of 0.9 when not specified', () => {
-			columnsSpy.mockReturnValue(100);
-			const width = getBoxWidth();
-			expect(width).toBe(90);
-		});
-
-		it('should use default minimum width of 40 when not specified', () => {
-			columnsSpy.mockReturnValue(30);
-			const width = getBoxWidth();
-			expect(width).toBe(40); // Should enforce minimum
-		});
-
-		it('should enforce minimum width when terminal is too narrow', () => {
-			columnsSpy.mockReturnValue(50);
-			const width = getBoxWidth(0.9, 60);
-			expect(width).toBe(60); // Should use minWidth instead of 45
-		});
-
-		it('should handle undefined process.stdout.columns', () => {
-			columnsSpy.mockReturnValue(undefined);
-			const width = getBoxWidth(0.9, 40);
-			// Should fall back to 80 columns: Math.floor(80 * 0.9) = 72
-			expect(width).toBe(72);
-		});
-
-		it('should handle custom percentage values', () => {
-			columnsSpy.mockReturnValue(100);
-			expect(getBoxWidth(0.95, 40)).toBe(95);
-			expect(getBoxWidth(0.8, 40)).toBe(80);
-			expect(getBoxWidth(0.5, 40)).toBe(50);
-		});
-
-		it('should handle custom minimum width values', () => {
-			columnsSpy.mockReturnValue(60);
-			expect(getBoxWidth(0.9, 70)).toBe(70); // 60 * 0.9 = 54, but min is 70
-			expect(getBoxWidth(0.9, 50)).toBe(54); // 60 * 0.9 = 54, min is 50
-		});
-
-		it('should floor the calculated width', () => {
-			columnsSpy.mockReturnValue(99);
-			const width = getBoxWidth(0.9, 40);
-			// 99 * 0.9 = 89.1, should floor to 89
-			expect(width).toBe(89);
-		});
-
-		it('should match warning box width calculation', () => {
-			// Test the specific case from displayWarning()
-			columnsSpy.mockReturnValue(80);
-			const width = getBoxWidth(0.9, 40);
-			expect(width).toBe(72);
-		});
-
-		it('should match table width calculation', () => {
-			// Test the specific case from createTaskTable()
-			columnsSpy.mockReturnValue(111);
-			const width = getBoxWidth(0.9, 100);
-			// 111 * 0.9 = 99.9, floor to 99, but max(99, 100) = 100
-			expect(width).toBe(100);
-		});
-
-		it('should match recommended task box width calculation', () => {
-			// Test the specific case from displayRecommendedNextTask()
-			columnsSpy.mockReturnValue(120);
-			const width = getBoxWidth(0.97, 40);
-			// 120 * 0.97 = 116.4, floor to 116
-			expect(width).toBe(116);
-		});
-
-		it('should handle edge case of zero terminal width', () => {
-			columnsSpy.mockReturnValue(0);
-			const width = getBoxWidth(0.9, 40);
-			// When columns is 0, it uses fallback of 80: Math.floor(80 * 0.9) = 72
-			expect(width).toBe(72);
-		});
-
-		it('should handle very large terminal widths', () => {
-			columnsSpy.mockReturnValue(1000);
-			const width = getBoxWidth(0.9, 40);
-			expect(width).toBe(900);
-		});
-
-		it('should handle very small percentages', () => {
-			columnsSpy.mockReturnValue(100);
-			const width = getBoxWidth(0.1, 5);
-			// 100 * 0.1 = 10, which is greater than min 5
-			expect(width).toBe(10);
-		});
-
-		it('should handle percentage of 1.0 (100%)', () => {
-			columnsSpy.mockReturnValue(80);
-			const width = getBoxWidth(1.0, 40);
-			expect(width).toBe(80);
-		});
-
-		it('should consistently return same value for same inputs', () => {
-			columnsSpy.mockReturnValue(100);
-			const width1 = getBoxWidth(0.9, 40);
-			const width2 = getBoxWidth(0.9, 40);
-			const width3 = getBoxWidth(0.9, 40);
-			expect(width1).toBe(width2);
-			expect(width2).toBe(width3);
-		});
-	});
-});

apps/cli/src/utils/ui.ts

@@ -126,20 +126,6 @@ export function getComplexityWithScore(complexity: number | undefined): string {
 	return color(`${complexity}/10 (${label})`);
 }
 
-/**
- * Calculate box width as percentage of terminal width
- * @param percentage - Percentage of terminal width to use (default: 0.9)
- * @param minWidth - Minimum width to enforce (default: 40)
- * @returns Calculated box width
- */
-export function getBoxWidth(
-	percentage: number = 0.9,
-	minWidth: number = 40
-): number {
-	const terminalWidth = process.stdout.columns || 80;
-	return Math.max(Math.floor(terminalWidth * percentage), minWidth);
-}
-
 /**
  * Truncate text to specified length
  */
@@ -190,8 +176,6 @@ export function displayBanner(title: string = 'Task Master'): void {
  * Display an error message (matches scripts/modules/ui.js style)
  */
 export function displayError(message: string, details?: string): void {
-	const boxWidth = getBoxWidth();
-
 	console.error(
 		boxen(
 			chalk.red.bold('X Error: ') +
@@ -200,8 +184,7 @@ export function displayError(message: string, details?: string): void {
 			{
 				padding: 1,
 				borderStyle: 'round',
-				borderColor: 'red',
+				borderColor: 'red'
-				width: boxWidth
 			}
 		)
 	);
@@ -211,16 +194,13 @@ export function displayError(message: string, details?: string): void {
  * Display a success message
  */
 export function displaySuccess(message: string): void {
-	const boxWidth = getBoxWidth();
-
 	console.log(
 		boxen(
 			chalk.green.bold(String.fromCharCode(8730) + ' ') + chalk.white(message),
 			{
 				padding: 1,
 				borderStyle: 'round',
-				borderColor: 'green',
+				borderColor: 'green'
-				width: boxWidth
 			}
 		)
 	);
@@ -230,14 +210,11 @@ export function displaySuccess(message: string): void {
  * Display a warning message
  */
 export function displayWarning(message: string): void {
-	const boxWidth = getBoxWidth();
-
 	console.log(
 		boxen(chalk.yellow.bold('⚠ ') + chalk.white(message), {
 			padding: 1,
 			borderStyle: 'round',
-			borderColor: 'yellow',
+			borderColor: 'yellow'
-			width: boxWidth
 		})
 	);
 }
@@ -246,14 +223,11 @@ export function displayWarning(message: string): void {
  * Display info message
  */
 export function displayInfo(message: string): void {
-	const boxWidth = getBoxWidth();
-
 	console.log(
 		boxen(chalk.blue.bold('i ') + chalk.white(message), {
 			padding: 1,
 			borderStyle: 'round',
-			borderColor: 'blue',
+			borderColor: 'blue'
-			width: boxWidth
 		})
 	);
 }
@@ -308,23 +282,23 @@ export function createTaskTable(
 	} = options || {};
 
 	// Calculate dynamic column widths based on terminal width
-	const tableWidth = getBoxWidth(0.9, 100);
+	const terminalWidth = process.stdout.columns * 0.9 || 100;
 	// Adjust column widths to better match the original layout
 	const baseColWidths = showComplexity
 		? [
-				Math.floor(tableWidth * 0.1),
+				Math.floor(terminalWidth * 0.1),
-				Math.floor(tableWidth * 0.4),
+				Math.floor(terminalWidth * 0.4),
-				Math.floor(tableWidth * 0.15),
+				Math.floor(terminalWidth * 0.15),
-				Math.floor(tableWidth * 0.1),
+				Math.floor(terminalWidth * 0.1),
-				Math.floor(tableWidth * 0.2),
+				Math.floor(terminalWidth * 0.2),
-				Math.floor(tableWidth * 0.1)
+				Math.floor(terminalWidth * 0.1)
 			] // ID, Title, Status, Priority, Dependencies, Complexity
 		: [
-				Math.floor(tableWidth * 0.08),
+				Math.floor(terminalWidth * 0.08),
-				Math.floor(tableWidth * 0.4),
+				Math.floor(terminalWidth * 0.4),
-				Math.floor(tableWidth * 0.18),
+				Math.floor(terminalWidth * 0.18),
-				Math.floor(tableWidth * 0.12),
+				Math.floor(terminalWidth * 0.12),
-				Math.floor(tableWidth * 0.2)
+				Math.floor(terminalWidth * 0.2)
 			]; // ID, Title, Status, Priority, Dependencies
 
 	const headers = [

apps/docs/CHANGELOG.md

@@ -1,7 +1,5 @@
 # docs
 
-## 0.0.6
-
 ## 0.0.5
 
 ## 0.0.4

apps/docs/capabilities/mcp.mdx

@@ -13,126 +13,6 @@ The MCP interface is built on top of the `fastmcp` library and registers a set o
 
 Each tool is defined with a name, a description, and a set of parameters that are validated using the `zod` library. The `execute` function of each tool calls the corresponding core logic function from `scripts/modules/task-manager.js`.
 
-## Configurable Tool Loading
-
-To optimize LLM context usage, you can control which Task Master MCP tools are loaded using the `TASK_MASTER_TOOLS` environment variable. This is particularly useful when working with LLMs that have context limits or when you only need a subset of tools.
-
-### Configuration Modes
-
-#### All Tools (Default)
-Loads all 36 available tools. Use when you need full Task Master functionality.
-
-```json
-{
-  "mcpServers": {
-    "task-master-ai": {
-      "command": "npx",
-      "args": ["-y", "task-master-ai"],
-      "env": {
-        "TASK_MASTER_TOOLS": "all",
-        "ANTHROPIC_API_KEY": "your_key_here"
-      }
-    }
-  }
-}
-```
-
-If `TASK_MASTER_TOOLS` is not set, all tools are loaded by default.
-
-#### Core Tools (Lean Mode)
-Loads only 7 essential tools for daily development. Ideal for minimal context usage.
-
-**Core tools included:**
-- `get_tasks` - List all tasks
-- `next_task` - Find the next task to work on
-- `get_task` - Get detailed task information
-- `set_task_status` - Update task status
-- `update_subtask` - Add implementation notes
-- `parse_prd` - Generate tasks from PRD
-- `expand_task` - Break down tasks into subtasks
-
-```json
-{
-  "mcpServers": {
-    "task-master-ai": {
-      "command": "npx",
-      "args": ["-y", "task-master-ai"],
-      "env": {
-        "TASK_MASTER_TOOLS": "core",
-        "ANTHROPIC_API_KEY": "your_key_here"
-      }
-    }
-  }
-}
-```
-
-You can also use `"lean"` as an alias for `"core"`.
-
-#### Standard Tools
-Loads 15 commonly used tools. Balances functionality with context efficiency.
-
-**Standard tools include all core tools plus:**
-- `initialize_project` - Set up new projects
-- `analyze_project_complexity` - Analyze task complexity
-- `expand_all` - Expand all eligible tasks
-- `add_subtask` - Add subtasks manually
-- `remove_task` - Remove tasks
-- `generate` - Generate task markdown files
-- `add_task` - Create new tasks
-- `complexity_report` - View complexity analysis
-
-```json
-{
-  "mcpServers": {
-    "task-master-ai": {
-      "command": "npx",
-      "args": ["-y", "task-master-ai"],
-      "env": {
-        "TASK_MASTER_TOOLS": "standard",
-        "ANTHROPIC_API_KEY": "your_key_here"
-      }
-    }
-  }
-}
-```
-
-#### Custom Tool Selection
-Specify exactly which tools to load using a comma-separated list. Tool names are case-insensitive and support both underscores and hyphens.
-
-```json
-{
-  "mcpServers": {
-    "task-master-ai": {
-      "command": "npx",
-      "args": ["-y", "task-master-ai"],
-      "env": {
-        "TASK_MASTER_TOOLS": "get_tasks,next_task,set_task_status,update_subtask",
-        "ANTHROPIC_API_KEY": "your_key_here"
-      }
-    }
-  }
-}
-```
-
-### Choosing the Right Configuration
-
-- **Use `core`/`lean`**: When working with basic task management workflows or when context limits are strict
-- **Use `standard`**: For most development workflows that include task creation and analysis
-- **Use `all`**: When you need full functionality including tag management, dependencies, and advanced features
-- **Use custom list**: When you have specific tool requirements or want to experiment with minimal sets
-
-### Verification
-
-When the MCP server starts, it logs which tools were loaded:
-
-```
-Task Master MCP Server starting...
-Tool mode configuration: standard
-Loading standard tools
-Registering 15 MCP tools (mode: standard)
-Successfully registered 15/15 tools
-```
-
 ## Tool Categories
 
 The MCP tools can be categorized in the same way as the core functionalities:

apps/docs/getting-started/quick-start/configuration-quick.mdx

@@ -37,25 +37,6 @@ For MCP/Cursor usage: Configure keys in the env section of your .cursor/mcp.json
 }
 ```
 
-<Tip>
-**Optimize Context Usage**: You can control which Task Master MCP tools are loaded using the `TASK_MASTER_TOOLS` environment variable. This helps reduce LLM context usage by only loading the tools you need.
-
-Options:
-- `all` (default) - All 36 tools
-- `standard` - 15 commonly used tools
-- `core` or `lean` - 7 essential tools
-
-Example:
-```json
-"env": {
-  "TASK_MASTER_TOOLS": "standard",
-  "ANTHROPIC_API_KEY": "your_key_here"
-}
-```
-
-See the [MCP Tools documentation](/capabilities/mcp#configurable-tool-loading) for details.
-</Tip>
-
 ### CLI Usage: `.env` File
 
 Create a `.env` file in your project root and include the keys for the providers you plan to use:

apps/docs/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "docs",
-	"version": "0.0.6",
+	"version": "0.0.5",
 	"private": true,
 	"description": "Task Master documentation powered by Mintlify",
 	"scripts": {

apps/extension/CHANGELOG.md

@@ -1,7 +1,5 @@
 # Change Log
 
-## 0.25.6
-
 ## 0.25.6-rc.0
 
 ### Patch Changes

apps/extension/package.json

@@ -3,7 +3,7 @@
 	"private": true,
 	"displayName": "TaskMaster",
 	"description": "A visual Kanban board interface for TaskMaster projects in VS Code",
-	"version": "0.25.6",
+	"version": "0.25.6-rc.0",
 	"publisher": "Hamster",
 	"icon": "assets/icon.png",
 	"engines": {

docs/configuration.md

@@ -59,76 +59,6 @@ Taskmaster uses two primary methods for configuration:
     - **Migration:** Use `task-master migrate` to move this to `.taskmaster/config.json`.
     - **Deprecation:** While still supported, you'll see warnings encouraging migration to the new structure.
 
-## MCP Tool Loading Configuration
-
-### TASK_MASTER_TOOLS Environment Variable
-
-The `TASK_MASTER_TOOLS` environment variable controls which tools are loaded by the Task Master MCP server. This allows you to optimize token usage based on your workflow needs.
-
-> Note
-> Prefer setting `TASK_MASTER_TOOLS` in your MCP client's `env` block (e.g., `.cursor/mcp.json`) or in CI/deployment env. The `.env` file is reserved for API keys/endpoints; avoid persisting non-secret settings there.
-
-#### Configuration Options
-
-- **`all`** (default): Loads all 36 available tools (~21,000 tokens)
-  - Best for: Users who need the complete feature set
-  - Use when: Working with complex projects requiring all Task Master features
-  - Backward compatibility: This is the default to maintain compatibility with existing installations
-
-- **`standard`**: Loads 15 commonly used tools (~10,000 tokens, 50% reduction)
-  - Best for: Regular task management workflows
-  - Tools included: All core tools plus project initialization, complexity analysis, task generation, and more
-  - Use when: You need a balanced set of features with reduced token usage
-
-- **`core`** (or `lean`): Loads 7 essential tools (~5,000 tokens, 70% reduction)
-  - Best for: Daily development with minimal token overhead
-  - Tools included: `get_tasks`, `next_task`, `get_task`, `set_task_status`, `update_subtask`, `parse_prd`, `expand_task`
-  - Use when: Working in large contexts where token usage is critical
-  - Note: "lean" is an alias for "core" (same tools, token estimate and recommended use). You can refer to it as either "core" or "lean" when configuring.
-
-- **Custom list**: Comma-separated list of specific tool names
-  - Best for: Specialized workflows requiring specific tools
-  - Example: `"get_tasks,next_task,set_task_status"`
-  - Use when: You know exactly which tools you need
-
-#### How to Configure
-
-1. **In MCP configuration files** (`.cursor/mcp.json`, `.vscode/mcp.json`, etc.) - **Recommended**:
-
-   ```jsonc
-   {
-     "mcpServers": {
-       "task-master-ai": {
-         "env": {
-           "TASK_MASTER_TOOLS": "standard",  // Set tool loading mode
-           // API keys can still use .env for security
-         }
-       }
-     }
-   }
-   ```
-
-2. **Via Claude Code CLI**:
-
-   ```bash
-   claude mcp add task-master-ai --scope user \
-     --env TASK_MASTER_TOOLS="core" \
-     -- npx -y task-master-ai@latest
-   ```
-
-3. **In CI/deployment environment variables**:
-   ```bash
-   export TASK_MASTER_TOOLS="standard"
-   node mcp-server/server.js
-   ```
-
-#### Tool Loading Behavior
-
-- When `TASK_MASTER_TOOLS` is unset or empty, the system defaults to `"all"`
-- Invalid tool names in a user-specified list are ignored (a warning is emitted for each)
-- If every tool name in a custom list is invalid, the system falls back to `"all"`
-- Tool names are case-insensitive (e.g., `"CORE"`, `"core"`, and `"Core"` are treated identically)
-
 ## Environment Variables (`.env` file or MCP `env` block - For API Keys Only)
 
 - Used **exclusively** for sensitive API keys and specific endpoint URLs.
@@ -293,10 +223,10 @@ node scripts/init.js
    ```bash
    # Set MCP provider for main role
    task-master models set-main --provider mcp --model claude-3-5-sonnet-20241022
-
+   
-   # Set MCP provider for research role
+   # Set MCP provider for research role  
    task-master models set-research --provider mcp --model claude-3-opus-20240229
-
+   
    # Verify configuration
    task-master models list
    ```
@@ -427,7 +357,7 @@ Azure OpenAI provides enterprise-grade OpenAI models through Microsoft's Azure c
          "temperature": 0.7
        },
        "fallback": {
-         "provider": "azure",
+         "provider": "azure", 
          "modelId": "gpt-4o-mini",
          "maxTokens": 10000,
          "temperature": 0.7
@@ -446,7 +376,7 @@ Azure OpenAI provides enterprise-grade OpenAI models through Microsoft's Azure c
      "models": {
        "main": {
          "provider": "azure",
-         "modelId": "gpt-4o",
+         "modelId": "gpt-4o", 
          "maxTokens": 16000,
          "temperature": 0.7,
          "baseURL": "https://your-resource-name.azure.com/openai/deployments"
@@ -460,7 +390,7 @@ Azure OpenAI provides enterprise-grade OpenAI models through Microsoft's Azure c
        "fallback": {
          "provider": "azure",
          "modelId": "gpt-4o-mini",
-         "maxTokens": 10000,
+         "maxTokens": 10000, 
          "temperature": 0.7,
          "baseURL": "https://your-resource-name.azure.com/openai/deployments"
        }
@@ -472,7 +402,7 @@ Azure OpenAI provides enterprise-grade OpenAI models through Microsoft's Azure c
    ```bash
    # In .env file
    AZURE_OPENAI_API_KEY=your-azure-openai-api-key-here
-
+   
    # Optional: Override endpoint for all Azure models
    AZURE_OPENAI_ENDPOINT=https://your-resource-name.azure.com/openai/deployments
    ```

mcp-server/src/index.js

@@ -4,14 +4,12 @@ import dotenv from 'dotenv';
 import { fileURLToPath } from 'url';
 import fs from 'fs';
 import logger from './logger.js';
-import {
+import { registerTaskMasterTools } from './tools/index.js';
-	registerTaskMasterTools,
-	getToolsConfiguration
-} from './tools/index.js';
 import ProviderRegistry from '../../src/provider-registry/index.js';
 import { MCPProvider } from './providers/mcp-provider.js';
 import packageJson from '../../package.json' with { type: 'json' };
 
+// Load environment variables
 dotenv.config();
 
 // Constants
@@ -31,10 +29,12 @@ class TaskMasterMCPServer {
 		this.server = new FastMCP(this.options);
 		this.initialized = false;
 
+		// Bind methods
 		this.init = this.init.bind(this);
 		this.start = this.start.bind(this);
 		this.stop = this.stop.bind(this);
 
+		// Setup logging
 		this.logger = logger;
 	}
 
@@ -44,34 +44,8 @@ class TaskMasterMCPServer {
 	async init() {
 		if (this.initialized) return;
 
-		const normalizedToolMode = getToolsConfiguration();
+		// Pass the manager instance to the tool registration function
-
+		registerTaskMasterTools(this.server, this.asyncManager);
-		this.logger.info('Task Master MCP Server starting...');
-		this.logger.info(`Tool mode configuration: ${normalizedToolMode}`);
-
-		const registrationResult = registerTaskMasterTools(
-			this.server,
-			normalizedToolMode
-		);
-
-		this.logger.info(
-			`Normalized tool mode: ${registrationResult.normalizedMode}`
-		);
-		this.logger.info(
-			`Registered ${registrationResult.registeredTools.length} tools successfully`
-		);
-
-		if (registrationResult.registeredTools.length > 0) {
-			this.logger.debug(
-				`Registered tools: ${registrationResult.registeredTools.join(', ')}`
-			);
-		}
-
-		if (registrationResult.failedTools.length > 0) {
-			this.logger.warn(
-				`Failed to register ${registrationResult.failedTools.length} tools: ${registrationResult.failedTools.join(', ')}`
-			);
-		}
 
 		this.initialized = true;
 

mcp-server/src/tools/index.js

@@ -3,238 +3,109 @@
  * Export all Task Master CLI tools for MCP server
  */
 
+import { registerListTasksTool } from './get-tasks.js';
 import logger from '../logger.js';
-import {
+import { registerSetTaskStatusTool } from './set-task-status.js';
-	toolRegistry,
+import { registerParsePRDTool } from './parse-prd.js';
-	coreTools,
+import { registerUpdateTool } from './update.js';
-	standardTools,
+import { registerUpdateTaskTool } from './update-task.js';
-	getAvailableTools,
+import { registerUpdateSubtaskTool } from './update-subtask.js';
-	getToolRegistration,
+import { registerGenerateTool } from './generate.js';
-	isValidTool
+import { registerShowTaskTool } from './get-task.js';
-} from './tool-registry.js';
+import { registerNextTaskTool } from './next-task.js';
+import { registerExpandTaskTool } from './expand-task.js';
+import { registerAddTaskTool } from './add-task.js';
+import { registerAddSubtaskTool } from './add-subtask.js';
+import { registerRemoveSubtaskTool } from './remove-subtask.js';
+import { registerAnalyzeProjectComplexityTool } from './analyze.js';
+import { registerClearSubtasksTool } from './clear-subtasks.js';
+import { registerExpandAllTool } from './expand-all.js';
+import { registerRemoveDependencyTool } from './remove-dependency.js';
+import { registerValidateDependenciesTool } from './validate-dependencies.js';
+import { registerFixDependenciesTool } from './fix-dependencies.js';
+import { registerComplexityReportTool } from './complexity-report.js';
+import { registerAddDependencyTool } from './add-dependency.js';
+import { registerRemoveTaskTool } from './remove-task.js';
+import { registerInitializeProjectTool } from './initialize-project.js';
+import { registerModelsTool } from './models.js';
+import { registerMoveTaskTool } from './move-task.js';
+import { registerResponseLanguageTool } from './response-language.js';
+import { registerAddTagTool } from './add-tag.js';
+import { registerDeleteTagTool } from './delete-tag.js';
+import { registerListTagsTool } from './list-tags.js';
+import { registerUseTagTool } from './use-tag.js';
+import { registerRenameTagTool } from './rename-tag.js';
+import { registerCopyTagTool } from './copy-tag.js';
+import { registerResearchTool } from './research.js';
+import { registerRulesTool } from './rules.js';
+import { registerScopeUpTool } from './scope-up.js';
+import { registerScopeDownTool } from './scope-down.js';
 
 /**
- * Helper function to safely read and normalize the TASK_MASTER_TOOLS environment variable
+ * Register all Task Master tools with the MCP server
- * @returns {string} The tools configuration string, defaults to 'all'
- */
-export function getToolsConfiguration() {
-	const rawValue = process.env.TASK_MASTER_TOOLS;
-
-	if (!rawValue || rawValue.trim() === '') {
-		logger.debug('No TASK_MASTER_TOOLS env var found, defaulting to "all"');
-		return 'all';
-	}
-
-	const normalizedValue = rawValue.trim();
-	logger.debug(`TASK_MASTER_TOOLS env var: "${normalizedValue}"`);
-	return normalizedValue;
-}
-
-/**
- * Register Task Master tools with the MCP server
- * Supports selective tool loading via TASK_MASTER_TOOLS environment variable
  * @param {Object} server - FastMCP server instance
- * @param {string} toolMode - The tool mode configuration (defaults to 'all')
- * @returns {Object} Object containing registered tools, failed tools, and normalized mode
  */
-export function registerTaskMasterTools(server, toolMode = 'all') {
+export function registerTaskMasterTools(server) {
-	const registeredTools = [];
-	const failedTools = [];
-
 	try {
-		const enabledTools = toolMode.trim();
+		// Register each tool in a logical workflow order
-		let toolsToRegister = [];
 
-		const lowerCaseConfig = enabledTools.toLowerCase();
+		// Group 1: Initialization & Setup
+		registerInitializeProjectTool(server);
+		registerModelsTool(server);
+		registerRulesTool(server);
+		registerParsePRDTool(server);
 
-		switch (lowerCaseConfig) {
+		// Group 2: Task Analysis & Expansion
-			case 'all':
+		registerAnalyzeProjectComplexityTool(server);
-				toolsToRegister = Object.keys(toolRegistry);
+		registerExpandTaskTool(server);
-				logger.info('Loading all available tools');
+		registerExpandAllTool(server);
-				break;
+		registerScopeUpTool(server);
-			case 'core':
+		registerScopeDownTool(server);
-			case 'lean':
-				toolsToRegister = coreTools;
-				logger.info('Loading core tools only');
-				break;
-			case 'standard':
-				toolsToRegister = standardTools;
-				logger.info('Loading standard tools');
-				break;
-			default:
-				const requestedTools = enabledTools
-					.split(',')
-					.map((t) => t.trim())
-					.filter((t) => t.length > 0);
 
-				const uniqueTools = new Set();
+		// Group 3: Task Listing & Viewing
-				const unknownTools = [];
+		registerListTasksTool(server);
+		registerShowTaskTool(server);
+		registerNextTaskTool(server);
+		registerComplexityReportTool(server);
 
-				const aliasMap = {
+		// Group 4: Task Status & Management
-					response_language: 'response-language'
+		registerSetTaskStatusTool(server);
-				};
+		registerGenerateTool(server);
 
-				for (const toolName of requestedTools) {
+		// Group 5: Task Creation & Modification
-					let resolvedName = null;
+		registerAddTaskTool(server);
-					const lowerToolName = toolName.toLowerCase();
+		registerAddSubtaskTool(server);
+		registerUpdateTool(server);
+		registerUpdateTaskTool(server);
+		registerUpdateSubtaskTool(server);
+		registerRemoveTaskTool(server);
+		registerRemoveSubtaskTool(server);
+		registerClearSubtasksTool(server);
+		registerMoveTaskTool(server);
 
-					if (aliasMap[lowerToolName]) {
+		// Group 6: Dependency Management
-						const aliasTarget = aliasMap[lowerToolName];
+		registerAddDependencyTool(server);
-						for (const registryKey of Object.keys(toolRegistry)) {
+		registerRemoveDependencyTool(server);
-							if (registryKey.toLowerCase() === aliasTarget.toLowerCase()) {
+		registerValidateDependenciesTool(server);
-								resolvedName = registryKey;
+		registerFixDependenciesTool(server);
-								break;
+		registerResponseLanguageTool(server);
-							}
-						}
-					}
 
-					if (!resolvedName) {
+		// Group 7: Tag Management
-						for (const registryKey of Object.keys(toolRegistry)) {
+		registerListTagsTool(server);
-							if (registryKey.toLowerCase() === lowerToolName) {
+		registerAddTagTool(server);
-								resolvedName = registryKey;
+		registerDeleteTagTool(server);
-								break;
+		registerUseTagTool(server);
-							}
+		registerRenameTagTool(server);
-						}
+		registerCopyTagTool(server);
-					}
 
-					if (!resolvedName) {
+		// Group 8: Research Features
-						const withHyphens = lowerToolName.replace(/_/g, '-');
+		registerResearchTool(server);
-						for (const registryKey of Object.keys(toolRegistry)) {
-							if (registryKey.toLowerCase() === withHyphens) {
-								resolvedName = registryKey;
-								break;
-							}
-						}
-					}
-
-					if (!resolvedName) {
-						const withUnderscores = lowerToolName.replace(/-/g, '_');
-						for (const registryKey of Object.keys(toolRegistry)) {
-							if (registryKey.toLowerCase() === withUnderscores) {
-								resolvedName = registryKey;
-								break;
-							}
-						}
-					}
-
-					if (resolvedName) {
-						uniqueTools.add(resolvedName);
-						logger.debug(`Resolved tool "${toolName}" to "${resolvedName}"`);
-					} else {
-						unknownTools.push(toolName);
-						logger.warn(`Unknown tool specified: "${toolName}"`);
-					}
-				}
-
-				toolsToRegister = Array.from(uniqueTools);
-
-				if (unknownTools.length > 0) {
-					logger.warn(`Unknown tools: ${unknownTools.join(', ')}`);
-				}
-
-				if (toolsToRegister.length === 0) {
-					logger.warn(
-						`No valid tools found in custom list. Loading all tools as fallback.`
-					);
-					toolsToRegister = Object.keys(toolRegistry);
-				} else {
-					logger.info(
-						`Loading ${toolsToRegister.length} custom tools from list (${uniqueTools.size} unique after normalization)`
-					);
-				}
-				break;
-		}
-
-		logger.info(
-			`Registering ${toolsToRegister.length} MCP tools (mode: ${enabledTools})`
-		);
-
-		toolsToRegister.forEach((toolName) => {
-			try {
-				const registerFunction = getToolRegistration(toolName);
-				if (registerFunction) {
-					registerFunction(server);
-					logger.debug(`Registered tool: ${toolName}`);
-					registeredTools.push(toolName);
-				} else {
-					logger.warn(`Tool ${toolName} not found in registry`);
-					failedTools.push(toolName);
-				}
-			} catch (error) {
-				if (error.message && error.message.includes('already registered')) {
-					logger.debug(`Tool ${toolName} already registered, skipping`);
-					registeredTools.push(toolName);
-				} else {
-					logger.error(`Failed to register tool ${toolName}: ${error.message}`);
-					failedTools.push(toolName);
-				}
-			}
-		});
-
-		logger.info(
-			`Successfully registered ${registeredTools.length}/${toolsToRegister.length} tools`
-		);
-		if (failedTools.length > 0) {
-			logger.warn(`Failed tools: ${failedTools.join(', ')}`);
-		}
-
-		return {
-			registeredTools,
-			failedTools,
-			normalizedMode: lowerCaseConfig
-		};
 	} catch (error) {
-		logger.error(
+		logger.error(`Error registering Task Master tools: ${error.message}`);
-			`Error parsing TASK_MASTER_TOOLS environment variable: ${error.message}`
+		throw error;
-		);
-		logger.info('Falling back to loading all tools');
-
-		const fallbackTools = Object.keys(toolRegistry);
-		for (const toolName of fallbackTools) {
-			const registerFunction = getToolRegistration(toolName);
-			if (registerFunction) {
-				try {
-					registerFunction(server);
-					registeredTools.push(toolName);
-				} catch (err) {
-					if (err.message && err.message.includes('already registered')) {
-						logger.debug(
-							`Fallback tool ${toolName} already registered, skipping`
-						);
-						registeredTools.push(toolName);
-					} else {
-						logger.warn(
-							`Failed to register fallback tool '${toolName}': ${err.message}`
-						);
-						failedTools.push(toolName);
-					}
-				}
-			} else {
-				logger.warn(`Tool '${toolName}' not found in registry`);
-				failedTools.push(toolName);
-			}
-		}
-		logger.info(
-			`Successfully registered ${registeredTools.length} fallback tools`
-		);
-
-		return {
-			registeredTools,
-			failedTools,
-			normalizedMode: 'all'
-		};
 	}
 }
 
-export {
-	toolRegistry,
-	coreTools,
-	standardTools,
-	getAvailableTools,
-	getToolRegistration,
-	isValidTool
-};
-
 export default {
 	registerTaskMasterTools
 };

mcp-server/src/tools/tool-registry.js

@@ -1,168 +0,0 @@
-/**
- * tool-registry.js
- * Tool Registry Object Structure - Maps all 36 tool names to registration functions
- */
-
-import { registerListTasksTool } from './get-tasks.js';
-import { registerSetTaskStatusTool } from './set-task-status.js';
-import { registerParsePRDTool } from './parse-prd.js';
-import { registerUpdateTool } from './update.js';
-import { registerUpdateTaskTool } from './update-task.js';
-import { registerUpdateSubtaskTool } from './update-subtask.js';
-import { registerGenerateTool } from './generate.js';
-import { registerShowTaskTool } from './get-task.js';
-import { registerNextTaskTool } from './next-task.js';
-import { registerExpandTaskTool } from './expand-task.js';
-import { registerAddTaskTool } from './add-task.js';
-import { registerAddSubtaskTool } from './add-subtask.js';
-import { registerRemoveSubtaskTool } from './remove-subtask.js';
-import { registerAnalyzeProjectComplexityTool } from './analyze.js';
-import { registerClearSubtasksTool } from './clear-subtasks.js';
-import { registerExpandAllTool } from './expand-all.js';
-import { registerRemoveDependencyTool } from './remove-dependency.js';
-import { registerValidateDependenciesTool } from './validate-dependencies.js';
-import { registerFixDependenciesTool } from './fix-dependencies.js';
-import { registerComplexityReportTool } from './complexity-report.js';
-import { registerAddDependencyTool } from './add-dependency.js';
-import { registerRemoveTaskTool } from './remove-task.js';
-import { registerInitializeProjectTool } from './initialize-project.js';
-import { registerModelsTool } from './models.js';
-import { registerMoveTaskTool } from './move-task.js';
-import { registerResponseLanguageTool } from './response-language.js';
-import { registerAddTagTool } from './add-tag.js';
-import { registerDeleteTagTool } from './delete-tag.js';
-import { registerListTagsTool } from './list-tags.js';
-import { registerUseTagTool } from './use-tag.js';
-import { registerRenameTagTool } from './rename-tag.js';
-import { registerCopyTagTool } from './copy-tag.js';
-import { registerResearchTool } from './research.js';
-import { registerRulesTool } from './rules.js';
-import { registerScopeUpTool } from './scope-up.js';
-import { registerScopeDownTool } from './scope-down.js';
-
-/**
- * Comprehensive tool registry mapping all 36 tool names to their registration functions
- * Used for dynamic tool registration and validation
- */
-export const toolRegistry = {
-	initialize_project: registerInitializeProjectTool,
-	models: registerModelsTool,
-	rules: registerRulesTool,
-	parse_prd: registerParsePRDTool,
-	'response-language': registerResponseLanguageTool,
-	analyze_project_complexity: registerAnalyzeProjectComplexityTool,
-	expand_task: registerExpandTaskTool,
-	expand_all: registerExpandAllTool,
-	scope_up_task: registerScopeUpTool,
-	scope_down_task: registerScopeDownTool,
-	get_tasks: registerListTasksTool,
-	get_task: registerShowTaskTool,
-	next_task: registerNextTaskTool,
-	complexity_report: registerComplexityReportTool,
-	set_task_status: registerSetTaskStatusTool,
-	generate: registerGenerateTool,
-	add_task: registerAddTaskTool,
-	add_subtask: registerAddSubtaskTool,
-	update: registerUpdateTool,
-	update_task: registerUpdateTaskTool,
-	update_subtask: registerUpdateSubtaskTool,
-	remove_task: registerRemoveTaskTool,
-	remove_subtask: registerRemoveSubtaskTool,
-	clear_subtasks: registerClearSubtasksTool,
-	move_task: registerMoveTaskTool,
-	add_dependency: registerAddDependencyTool,
-	remove_dependency: registerRemoveDependencyTool,
-	validate_dependencies: registerValidateDependenciesTool,
-	fix_dependencies: registerFixDependenciesTool,
-	list_tags: registerListTagsTool,
-	add_tag: registerAddTagTool,
-	delete_tag: registerDeleteTagTool,
-	use_tag: registerUseTagTool,
-	rename_tag: registerRenameTagTool,
-	copy_tag: registerCopyTagTool,
-	research: registerResearchTool
-};
-
-/**
- * Core tools array containing the 7 essential tools for daily development
- * These represent the minimal set needed for basic task management operations
- */
-export const coreTools = [
-	'get_tasks',
-	'next_task',
-	'get_task',
-	'set_task_status',
-	'update_subtask',
-	'parse_prd',
-	'expand_task'
-];
-
-/**
- * Standard tools array containing the 15 most commonly used tools
- * Includes all core tools plus frequently used additional tools
- */
-export const standardTools = [
-	...coreTools,
-	'initialize_project',
-	'analyze_project_complexity',
-	'expand_all',
-	'add_subtask',
-	'remove_task',
-	'generate',
-	'add_task',
-	'complexity_report'
-];
-
-/**
- * Get all available tool names
- * @returns {string[]} Array of tool names
- */
-export function getAvailableTools() {
-	return Object.keys(toolRegistry);
-}
-
-/**
- * Get tool counts for all categories
- * @returns {Object} Object with core, standard, and total counts
- */
-export function getToolCounts() {
-	return {
-		core: coreTools.length,
-		standard: standardTools.length,
-		total: Object.keys(toolRegistry).length
-	};
-}
-
-/**
- * Get tool arrays organized by category
- * @returns {Object} Object with arrays for each category
- */
-export function getToolCategories() {
-	const allTools = Object.keys(toolRegistry);
-	return {
-		core: [...coreTools],
-		standard: [...standardTools],
-		all: [...allTools],
-		extended: allTools.filter((t) => !standardTools.includes(t))
-	};
-}
-
-/**
- * Get registration function for a specific tool
- * @param {string} toolName - Name of the tool
- * @returns {Function|null} Registration function or null if not found
- */
-export function getToolRegistration(toolName) {
-	return toolRegistry[toolName] || null;
-}
-
-/**
- * Validate if a tool exists in the registry
- * @param {string} toolName - Name of the tool
- * @returns {boolean} True if tool exists
- */
-export function isValidTool(toolName) {
-	return toolName in toolRegistry;
-}
-
-export default toolRegistry;

package-lock.json

@@ -1,12 +1,12 @@
 {
 	"name": "task-master-ai",
-	"version": "0.29.0",
+	"version": "0.29.0-rc.0",
 	"lockfileVersion": 3,
 	"requires": true,
 	"packages": {
 		"": {
 			"name": "task-master-ai",
-			"version": "0.29.0",
+			"version": "0.29.0-rc.0",
 			"license": "MIT WITH Commons-Clause",
 			"workspaces": [
 				"apps/*",
@@ -125,13 +125,13 @@
 			}
 		},
 		"apps/docs": {
-			"version": "0.0.6",
+			"version": "0.0.5",
 			"devDependencies": {
 				"mintlify": "^4.2.111"
 			}
 		},
 		"apps/extension": {
-			"version": "0.25.6",
+			"version": "0.25.6-rc.0",
 			"devDependencies": {
 				"@dnd-kit/core": "^6.3.1",
 				"@dnd-kit/modifiers": "^9.0.0",
@@ -27136,7 +27136,7 @@
 		},
 		"packages/claude-code-plugin": {
 			"name": "@tm/claude-code-plugin",
-			"version": "0.0.2",
+			"version": "0.0.1",
 			"license": "MIT WITH Commons-Clause"
 		},
 		"packages/tm-core": {

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "task-master-ai",
-	"version": "0.29.0",
+	"version": "0.29.0-rc.0",
 	"description": "A task management system for ambitious AI-driven development that doesn't overwhelm and confuse Cursor.",
 	"main": "index.js",
 	"type": "module",

packages/ai-sdk-provider-grok-cli/CHANGELOG.md

@@ -1,5 +1,3 @@
 # @tm/ai-sdk-provider-grok-cli
 
 ## null
-
-## null

packages/build-config/CHANGELOG.md

@@ -4,6 +4,4 @@
 
 ## null
 
-## null
-
 ## 1.0.1

packages/claude-code-plugin/CHANGELOG.md

@@ -1,3 +0,0 @@
-# @tm/claude-code-plugin
-
-## 0.0.2

packages/claude-code-plugin/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@tm/claude-code-plugin",
-	"version": "0.0.2",
+	"version": "0.0.1",
 	"description": "Task Master AI plugin for Claude Code - AI-powered task management with commands, agents, and MCP integration",
 	"type": "module",
 	"private": true,

packages/tm-core/CHANGELOG.md

@@ -4,8 +4,6 @@
 
 ## null
 
-## null
-
 ## 0.26.1
 
 All notable changes to the @task-master/tm-core package will be documented in this file.

packages/tm-core/src/auth/auth-manager.test.ts

@@ -21,21 +21,16 @@ const CredentialStoreSpy = vi.fn();
 vi.mock('./credential-store.js', () => {
 	return {
 		CredentialStore: class {
-			static getInstance(config?: any) {
-				return new (this as any)(config);
-			}
-			static resetInstance() {
-				// Mock reset instance method
-			}
 			constructor(config: any) {
 				CredentialStoreSpy(config);
+				this.getCredentials = vi.fn(() => null);
 			}
-			getCredentials(_options?: any) {
+			getCredentials() {
 				return null;
 			}
 			saveCredentials() {}
 			clearCredentials() {}
-			hasCredentials() {
+			hasValidCredentials() {
 				return false;
 			}
 		}
@@ -90,7 +85,7 @@ describe('AuthManager Singleton', () => {
 		expect(instance1).toBe(instance2);
 	});
 
-	it('should use config on first call', async () => {
+	it('should use config on first call', () => {
 		const config = {
 			baseUrl: 'https://test.auth.com',
 			configDir: '/test/config',
@@ -106,7 +101,7 @@ describe('AuthManager Singleton', () => {
 
 		// Verify the config is passed to internal components through observable behavior
 		// getCredentials would look in the configured file path
-		const credentials = await instance.getCredentials();
+		const credentials = instance.getCredentials();
 		expect(credentials).toBeNull(); // File doesn't exist, but config was propagated correctly
 	});
 

packages/tm-core/src/auth/auth-manager.ts

@@ -29,6 +29,7 @@ export class AuthManager {
 	private oauthService: OAuthService;
 	private supabaseClient: SupabaseAuthClient;
 	private organizationService?: OrganizationService;
+	private logger = getLogger('AuthManager');
 
 	private constructor(config?: Partial<AuthConfig>) {
 		this.credentialStore = CredentialStore.getInstance(config);
@@ -36,10 +37,7 @@ export class AuthManager {
 		this.oauthService = new OAuthService(this.credentialStore, config);
 
 		// Initialize Supabase client with session restoration
-		// Fire-and-forget with catch handler to prevent unhandled rejections
+		this.initializeSupabaseSession();
-		this.initializeSupabaseSession().catch(() => {
-			// Errors are already logged in initializeSupabaseSession
-		});
 	}
 
 	/**
@@ -81,10 +79,49 @@ export class AuthManager {
 
 	/**
 	 * Get stored authentication credentials
-	 * Returns credentials as-is (even if expired). Refresh must be triggered explicitly
+	 * Automatically refreshes the token if expired
-	 * via refreshToken() or will occur automatically when using the Supabase client for API calls.
 	 */
-	getCredentials(): AuthCredentials | null {
+	async getCredentials(): Promise<AuthCredentials | null> {
+		const credentials = this.credentialStore.getCredentials();
+
+		// If credentials exist but are expired, try to refresh
+		if (!credentials) {
+			const expiredCredentials = this.credentialStore.getCredentials({
+				allowExpired: true
+			});
+
+			// Check if we have any credentials at all
+			if (!expiredCredentials) {
+				// No credentials found
+				return null;
+			}
+
+			// Check if refresh token is available
+			if (!expiredCredentials.refreshToken) {
+				this.logger.warn(
+					'Token expired but no refresh token available. Please re-authenticate.'
+				);
+				return null;
+			}
+
+			// Attempt refresh
+			try {
+				this.logger.info('Token expired, attempting automatic refresh...');
+				return await this.refreshToken();
+			} catch (error) {
+				this.logger.warn('Automatic token refresh failed:', error);
+				return null;
+			}
+		}
+
+		return credentials;
+	}
+
+	/**
+	 * Get stored authentication credentials (synchronous version)
+	 * Does not attempt automatic refresh
+	 */
+	getCredentialsSync(): AuthCredentials | null {
 		return this.credentialStore.getCredentials();
 	}
 
@@ -167,26 +204,25 @@ export class AuthManager {
 	}
 
 	/**
-	 * Check if authenticated (credentials exist, regardless of expiration)
+	 * Check if authenticated
-	 * @returns true if credentials are stored, including expired credentials
 	 */
 	isAuthenticated(): boolean {
-		return this.credentialStore.hasCredentials();
+		return this.credentialStore.hasValidCredentials();
 	}
 
 	/**
 	 * Get the current user context (org/brief selection)
 	 */
-	getContext(): UserContext | null {
+	async getContext(): Promise<UserContext | null> {
-		const credentials = this.getCredentials();
+		const credentials = await this.getCredentials();
 		return credentials?.selectedContext || null;
 	}
 
 	/**
 	 * Update the user context (org/brief selection)
 	 */
-	updateContext(context: Partial<UserContext>): void {
+	async updateContext(context: Partial<UserContext>): Promise<void> {
-		const credentials = this.getCredentials();
+		const credentials = await this.getCredentials();
 		if (!credentials) {
 			throw new AuthenticationError('Not authenticated', 'NOT_AUTHENTICATED');
 		}
@@ -211,8 +247,8 @@ export class AuthManager {
 	/**
 	 * Clear the user context
 	 */
-	clearContext(): void {
+	async clearContext(): Promise<void> {
-		const credentials = this.getCredentials();
+		const credentials = await this.getCredentials();
 		if (!credentials) {
 			throw new AuthenticationError('Not authenticated', 'NOT_AUTHENTICATED');
 		}
@@ -229,7 +265,7 @@ export class AuthManager {
 	private async getOrganizationService(): Promise<OrganizationService> {
 		if (!this.organizationService) {
 			// First check if we have credentials with a token
-			const credentials = this.getCredentials();
+			const credentials = await this.getCredentials();
 			if (!credentials || !credentials.token) {
 				throw new AuthenticationError('Not authenticated', 'NOT_AUTHENTICATED');
 			}

packages/tm-core/src/auth/credential-store.spec.ts

@@ -52,7 +52,7 @@ describe('CredentialStore - Token Expiration', () => {
 
 			credentialStore.saveCredentials(expiredCredentials);
 
-			const retrieved = credentialStore.getCredentials({ allowExpired: false });
+			const retrieved = credentialStore.getCredentials();
 
 			expect(retrieved).toBeNull();
 		});
@@ -69,7 +69,7 @@ describe('CredentialStore - Token Expiration', () => {
 
 			credentialStore.saveCredentials(validCredentials);
 
-			const retrieved = credentialStore.getCredentials({ allowExpired: false });
+			const retrieved = credentialStore.getCredentials();
 
 			expect(retrieved).not.toBeNull();
 			expect(retrieved?.token).toBe('valid-token');
@@ -92,25 +92,6 @@ describe('CredentialStore - Token Expiration', () => {
 			expect(retrieved).not.toBeNull();
 			expect(retrieved?.token).toBe('expired-token');
 		});
-
-		it('should return expired token by default (allowExpired defaults to true)', () => {
-			const expiredCredentials: AuthCredentials = {
-				token: 'expired-token-default',
-				refreshToken: 'refresh-token',
-				userId: 'test-user',
-				email: 'test@example.com',
-				expiresAt: new Date(Date.now() - 60000).toISOString(),
-				savedAt: new Date().toISOString()
-			};
-
-			credentialStore.saveCredentials(expiredCredentials);
-
-			// Call without options - should default to allowExpired: true
-			const retrieved = credentialStore.getCredentials();
-
-			expect(retrieved).not.toBeNull();
-			expect(retrieved?.token).toBe('expired-token-default');
-		});
 	});
 
 	describe('Clock Skew Tolerance', () => {
@@ -127,7 +108,7 @@ describe('CredentialStore - Token Expiration', () => {
 
 			credentialStore.saveCredentials(almostExpiredCredentials);
 
-			const retrieved = credentialStore.getCredentials({ allowExpired: false });
+			const retrieved = credentialStore.getCredentials();
 
 			expect(retrieved).toBeNull();
 		});
@@ -145,7 +126,7 @@ describe('CredentialStore - Token Expiration', () => {
 
 			credentialStore.saveCredentials(validCredentials);
 
-			const retrieved = credentialStore.getCredentials({ allowExpired: false });
+			const retrieved = credentialStore.getCredentials();
 
 			expect(retrieved).not.toBeNull();
 			expect(retrieved?.token).toBe('valid-token');
@@ -165,7 +146,7 @@ describe('CredentialStore - Token Expiration', () => {
 
 			credentialStore.saveCredentials(credentials);
 
-			const retrieved = credentialStore.getCredentials({ allowExpired: false });
+			const retrieved = credentialStore.getCredentials();
 
 			expect(retrieved).not.toBeNull();
 			expect(typeof retrieved?.expiresAt).toBe('number'); // Normalized to number
@@ -183,7 +164,7 @@ describe('CredentialStore - Token Expiration', () => {
 
 			credentialStore.saveCredentials(credentials);
 
-			const retrieved = credentialStore.getCredentials({ allowExpired: false });
+			const retrieved = credentialStore.getCredentials();
 
 			expect(retrieved).not.toBeNull();
 			expect(typeof retrieved?.expiresAt).toBe('number');
@@ -204,7 +185,7 @@ describe('CredentialStore - Token Expiration', () => {
 				mode: 0o600
 			});
 
-			const retrieved = credentialStore.getCredentials({ allowExpired: false });
+			const retrieved = credentialStore.getCredentials();
 
 			expect(retrieved).toBeNull();
 		});
@@ -222,7 +203,7 @@ describe('CredentialStore - Token Expiration', () => {
 				mode: 0o600
 			});
 
-			const retrieved = credentialStore.getCredentials({ allowExpired: false });
+			const retrieved = credentialStore.getCredentials();
 
 			expect(retrieved).toBeNull();
 		});
@@ -263,15 +244,15 @@ describe('CredentialStore - Token Expiration', () => {
 
 			credentialStore.saveCredentials(credentials);
 
-			const retrieved = credentialStore.getCredentials({ allowExpired: false });
+			const retrieved = credentialStore.getCredentials();
 
 			// Should be normalized to number for runtime use
 			expect(typeof retrieved?.expiresAt).toBe('number');
 		});
 	});
 
-	describe('hasCredentials', () => {
+	describe('hasValidCredentials', () => {
-		it('should return true for expired credentials', () => {
+		it('should return false for expired credentials', () => {
 			const expiredCredentials: AuthCredentials = {
 				token: 'expired-token',
 				refreshToken: 'refresh-token',
@@ -283,7 +264,7 @@ describe('CredentialStore - Token Expiration', () => {
 
 			credentialStore.saveCredentials(expiredCredentials);
 
-			expect(credentialStore.hasCredentials()).toBe(true);
+			expect(credentialStore.hasValidCredentials()).toBe(false);
 		});
 
 		it('should return true for valid credentials', () => {
@@ -298,11 +279,11 @@ describe('CredentialStore - Token Expiration', () => {
 
 			credentialStore.saveCredentials(validCredentials);
 
-			expect(credentialStore.hasCredentials()).toBe(true);
+			expect(credentialStore.hasValidCredentials()).toBe(true);
 		});
 
 		it('should return false when no credentials exist', () => {
-			expect(credentialStore.hasCredentials()).toBe(false);
+			expect(credentialStore.hasValidCredentials()).toBe(false);
 		});
 	});
 });

packages/tm-core/src/auth/credential-store.test.ts

@@ -197,7 +197,7 @@ describe('CredentialStore', () => {
 				JSON.stringify(mockCredentials)
 			);
 
-			const result = store.getCredentials({ allowExpired: false });
+			const result = store.getCredentials();
 
 			expect(result).toBeNull();
 			expect(mockLogger.warn).toHaveBeenCalledWith(
@@ -226,31 +226,6 @@ describe('CredentialStore', () => {
 			expect(result).not.toBeNull();
 			expect(result?.token).toBe('expired-token');
 		});
-
-		it('should return expired tokens by default (allowExpired defaults to true)', () => {
-			const expiredTimestamp = Date.now() - 3600000; // 1 hour ago
-			const mockCredentials = {
-				token: 'expired-token-default',
-				userId: 'user-expired',
-				expiresAt: expiredTimestamp,
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue(
-				JSON.stringify(mockCredentials)
-			);
-
-			// Call without options - should default to allowExpired: true
-			const result = store.getCredentials();
-
-			expect(result).not.toBeNull();
-			expect(result?.token).toBe('expired-token-default');
-			expect(mockLogger.warn).not.toHaveBeenCalledWith(
-				expect.stringContaining('Authentication token has expired')
-			);
-		});
 	});
 
 	describe('saveCredentials with timestamp normalization', () => {
@@ -476,7 +451,7 @@ describe('CredentialStore', () => {
 		});
 	});
 
-	describe('hasCredentials', () => {
+	describe('hasValidCredentials', () => {
 		it('should return true when valid unexpired credentials exist', () => {
 			const futureDate = new Date(Date.now() + 3600000); // 1 hour from now
 			const credentials = {
@@ -490,10 +465,10 @@ describe('CredentialStore', () => {
 			vi.mocked(fs.existsSync).mockReturnValue(true);
 			vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify(credentials));
 
-			expect(store.hasCredentials()).toBe(true);
+			expect(store.hasValidCredentials()).toBe(true);
 		});
 
-		it('should return true when credentials are expired', () => {
+		it('should return false when credentials are expired', () => {
 			const pastDate = new Date(Date.now() - 3600000); // 1 hour ago
 			const credentials = {
 				token: 'expired-token',
@@ -506,13 +481,13 @@ describe('CredentialStore', () => {
 			vi.mocked(fs.existsSync).mockReturnValue(true);
 			vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify(credentials));
 
-			expect(store.hasCredentials()).toBe(true);
+			expect(store.hasValidCredentials()).toBe(false);
 		});
 
 		it('should return false when no credentials exist', () => {
 			vi.mocked(fs.existsSync).mockReturnValue(false);
 
-			expect(store.hasCredentials()).toBe(false);
+			expect(store.hasValidCredentials()).toBe(false);
 		});
 
 		it('should return false when file contains invalid JSON', () => {
@@ -520,7 +495,7 @@ describe('CredentialStore', () => {
 			vi.mocked(fs.readFileSync).mockReturnValue('invalid json {');
 			vi.mocked(fs.renameSync).mockImplementation(() => undefined);
 
-			expect(store.hasCredentials()).toBe(false);
+			expect(store.hasValidCredentials()).toBe(false);
 		});
 
 		it('should return false for credentials without expiry', () => {
@@ -535,7 +510,7 @@ describe('CredentialStore', () => {
 			vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify(credentials));
 
 			// Credentials without expiry are considered invalid
-			expect(store.hasCredentials()).toBe(false);
+			expect(store.hasValidCredentials()).toBe(false);
 
 			// Should log warning about missing expiration
 			expect(mockLogger.warn).toHaveBeenCalledWith(
@@ -543,14 +518,14 @@ describe('CredentialStore', () => {
 			);
 		});
 
-		it('should use allowExpired=true', () => {
+		it('should use allowExpired=false by default', () => {
 			// Spy on getCredentials to verify it's called with correct params
 			const getCredentialsSpy = vi.spyOn(store, 'getCredentials');
 
 			vi.mocked(fs.existsSync).mockReturnValue(false);
-			store.hasCredentials();
+			store.hasValidCredentials();
 
-			expect(getCredentialsSpy).toHaveBeenCalledWith({ allowExpired: true });
+			expect(getCredentialsSpy).toHaveBeenCalledWith({ allowExpired: false });
 		});
 	});
 

packages/tm-core/src/auth/credential-store.ts

@@ -54,12 +54,9 @@ export class CredentialStore {
 
 	/**
 	 * Get stored authentication credentials
-	 * @param options.allowExpired - Whether to return expired credentials (default: true)
 	 * @returns AuthCredentials with expiresAt as number (milliseconds) for runtime use
 	 */
-	getCredentials({
+	getCredentials(options?: { allowExpired?: boolean }): AuthCredentials | null {
-		allowExpired = true
-	}: { allowExpired?: boolean } = {}): AuthCredentials | null {
 		try {
 			if (!fs.existsSync(this.config.configFile)) {
 				return null;
@@ -93,6 +90,7 @@ export class CredentialStore {
 
 			// Check if the token has expired (with clock skew tolerance)
 			const now = Date.now();
+			const allowExpired = options?.allowExpired ?? false;
 			if (now >= expiresAtMs - this.CLOCK_SKEW_MS && !allowExpired) {
 				this.logger.warn(
 					'Authentication token has expired or is about to expire',
@@ -105,7 +103,7 @@ export class CredentialStore {
 				return null;
 			}
 
-			// Return credentials (even if expired) to enable refresh flows
+			// Return valid token
 			return authData;
 		} catch (error) {
 			this.logger.error(
@@ -201,11 +199,10 @@ export class CredentialStore {
 	}
 
 	/**
-	 * Check if credentials exist (regardless of expiration status)
+	 * Check if credentials exist and are valid
-	 * @returns true if credentials are stored, including expired credentials
 	 */
-	hasCredentials(): boolean {
+	hasValidCredentials(): boolean {
-		const credentials = this.getCredentials({ allowExpired: true });
+		const credentials = this.getCredentials({ allowExpired: false });
 		return credentials !== null;
 	}
 

packages/tm-core/src/auth/oauth-service.ts

@@ -281,26 +281,15 @@ export class OAuthService {
 				// Exchange code for session using PKCE
 				const session = await this.supabaseClient.exchangeCodeForSession(code);
 
-				// Calculate expiration - can be overridden with TM_TOKEN_EXPIRY_MINUTES
-				let expiresAt: string | undefined;
-				const tokenExpiryMinutes = process.env.TM_TOKEN_EXPIRY_MINUTES;
-				if (tokenExpiryMinutes) {
-					const minutes = parseInt(tokenExpiryMinutes);
-					expiresAt = new Date(Date.now() + minutes * 60 * 1000).toISOString();
-					this.logger.warn(`Token expiry overridden to ${minutes} minute(s)`);
-				} else {
-					expiresAt = session.expires_at
-						? new Date(session.expires_at * 1000).toISOString()
-						: undefined;
-				}
-
 				// Save authentication data
 				const authData: AuthCredentials = {
 					token: session.access_token,
 					refreshToken: session.refresh_token,
 					userId: session.user.id,
 					email: session.user.email,
-					expiresAt,
+					expiresAt: session.expires_at
+						? new Date(session.expires_at * 1000).toISOString()
+						: undefined,
 					tokenType: 'standard',
 					savedAt: new Date().toISOString()
 				};
@@ -351,18 +340,10 @@ export class OAuthService {
 				// Get user info from the session
 				const user = await this.supabaseClient.getUser();
 
-				// Calculate expiration time - can be overridden with TM_TOKEN_EXPIRY_MINUTES
+				// Calculate expiration time
-				let expiresAt: string | undefined;
+				const expiresAt = expiresIn
-				const tokenExpiryMinutes = process.env.TM_TOKEN_EXPIRY_MINUTES;
+					? new Date(Date.now() + parseInt(expiresIn) * 1000).toISOString()
-				if (tokenExpiryMinutes) {
+					: undefined;
-					const minutes = parseInt(tokenExpiryMinutes);
-					expiresAt = new Date(Date.now() + minutes * 60 * 1000).toISOString();
-					this.logger.warn(`Token expiry overridden to ${minutes} minute(s)`);
-				} else {
-					expiresAt = expiresIn
-						? new Date(Date.now() + parseInt(expiresIn) * 1000).toISOString()
-						: undefined;
-				}
 
 				// Save authentication data
 				const authData: AuthCredentials = {
@@ -370,7 +351,7 @@ export class OAuthService {
 					refreshToken: refreshToken || undefined,
 					userId: user?.id || 'unknown',
 					email: user?.email,
-					expiresAt,
+					expiresAt: expiresAt,
 					tokenType: 'standard',
 					savedAt: new Date().toISOString()
 				};

packages/tm-core/src/auth/supabase-session-storage.ts

@@ -98,11 +98,11 @@ export class SupabaseSessionStorage implements SupportedStorage {
 		// Only handle Supabase session keys
 		if (key === STORAGE_KEY || key.includes('auth-token')) {
 			try {
-				this.logger.info('Supabase called setItem - storing refreshed session');
-
 				// Parse the session and update our credentials
 				const sessionUpdates = this.parseSessionToCredentials(value);
-				const existingCredentials = this.store.getCredentials();
+				const existingCredentials = this.store.getCredentials({
+					allowExpired: true
+				});
 
 				if (sessionUpdates.token) {
 					const updatedCredentials: AuthCredentials = {
@@ -113,9 +113,6 @@ export class SupabaseSessionStorage implements SupportedStorage {
 					} as AuthCredentials;
 
 					this.store.saveCredentials(updatedCredentials);
-					this.logger.info(
-						'Successfully saved refreshed credentials from Supabase'
-					);
 				}
 			} catch (error) {
 				this.logger.error('Error setting session:', error);

packages/tm-core/src/clients/supabase-client.ts

@@ -17,11 +17,10 @@ export class SupabaseAuthClient {
 	private client: SupabaseJSClient | null = null;
 	private sessionStorage: SupabaseSessionStorage;
 	private logger = getLogger('SupabaseAuthClient');
-	private credentialStore: CredentialStore;
 
 	constructor() {
-		this.credentialStore = CredentialStore.getInstance();
+		const credentialStore = CredentialStore.getInstance();
-		this.sessionStorage = new SupabaseSessionStorage(this.credentialStore);
+		this.sessionStorage = new SupabaseSessionStorage(credentialStore);
 	}
 
 	/**

packages/tm-core/src/services/export.service.ts

@@ -362,7 +362,7 @@ export class ExportService {
 
 		if (useAPIEndpoint) {
 			// Use the new bulk import API endpoint
-			const apiUrl = `${process.env.TM_PUBLIC_BASE_DOMAIN}/ai/api/v1/briefs/${briefId}/tasks`;
+			const apiUrl = `${process.env.TM_PUBLIC_BASE_DOMAIN}/ai/api/v1/briefs/${briefId}/tasks/bulk`;
 
 			// Transform tasks to flat structure for API
 			const flatTasks = this.transformTasksForBulkImport(tasks);
@@ -370,11 +370,11 @@ export class ExportService {
 			// Prepare request body
 			const requestBody = {
 				source: 'task-master-cli',
+				accountId: orgId,
 				options: {
 					dryRun: false,
 					stopOnError: false
 				},
-				accountId: orgId,
 				tasks: flatTasks
 			};
 

packages/tm-core/src/storage/storage-factory.ts

@@ -73,7 +73,7 @@ export class StorageFactory {
 						);
 					}
 					// Use auth token from AuthManager (synchronous - no auto-refresh here)
-					const credentials = authManager.getCredentials();
+					const credentials = authManager.getCredentialsSync();
 					if (credentials) {
 						// Merge with existing storage config, ensuring required fields
 						const nextStorage: StorageSettings = {
@@ -103,7 +103,7 @@ export class StorageFactory {
 
 				// Then check if authenticated via AuthManager
 				if (authManager.isAuthenticated()) {
-					const credentials = authManager.getCredentials();
+					const credentials = authManager.getCredentialsSync();
 					if (credentials) {
 						// Configure API storage with auth credentials
 						const nextStorage: StorageSettings = {

packages/tm-core/tests/auth/auth-refresh.test.ts

@@ -1,139 +0,0 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
-import fs from 'fs';
-import os from 'os';
-import path from 'path';
-import type { Session } from '@supabase/supabase-js';
-import { AuthManager } from '../../src/auth/auth-manager';
-import { CredentialStore } from '../../src/auth/credential-store';
-import type { AuthCredentials } from '../../src/auth/types';
-
-describe('AuthManager Token Refresh', () => {
-	let authManager: AuthManager;
-	let credentialStore: CredentialStore;
-	let tmpDir: string;
-	let authFile: string;
-
-	beforeEach(() => {
-		// Reset singletons
-		AuthManager.resetInstance();
-		CredentialStore.resetInstance();
-
-		// Create temporary directory for test isolation
-		tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'tm-auth-refresh-'));
-		authFile = path.join(tmpDir, 'auth.json');
-
-		// Initialize AuthManager with test config (this will create CredentialStore internally)
-		authManager = AuthManager.getInstance({
-			configDir: tmpDir,
-			configFile: authFile
-		});
-
-		// Get the CredentialStore instance that AuthManager created
-		credentialStore = CredentialStore.getInstance();
-		credentialStore.clearCredentials();
-	});
-
-	afterEach(() => {
-		// Clean up
-		try {
-			credentialStore.clearCredentials();
-		} catch {
-			// Ignore cleanup errors
-		}
-		AuthManager.resetInstance();
-		CredentialStore.resetInstance();
-		vi.restoreAllMocks();
-
-		// Remove temporary directory
-		if (tmpDir && fs.existsSync(tmpDir)) {
-			fs.rmSync(tmpDir, { recursive: true, force: true });
-		}
-	});
-
-	it('should return expired credentials to enable refresh flows', () => {
-		// Set up expired credentials with refresh token
-		const expiredCredentials: AuthCredentials = {
-			token: 'expired_access_token',
-			refreshToken: 'valid_refresh_token',
-			userId: 'test-user-id',
-			email: 'test@example.com',
-			expiresAt: new Date(Date.now() - 1000).toISOString(), // Expired 1 second ago
-			savedAt: new Date().toISOString()
-		};
-
-		credentialStore.saveCredentials(expiredCredentials);
-
-		// Get credentials should return them even if expired
-		// Refresh will be handled by explicit calls or client operations
-		const credentials = authManager.getCredentials();
-
-		expect(credentials).not.toBeNull();
-		expect(credentials?.token).toBe('expired_access_token');
-		expect(credentials?.refreshToken).toBe('valid_refresh_token');
-	});
-
-	it('should return valid credentials', () => {
-		// Set up valid (non-expired) credentials
-		const validCredentials: AuthCredentials = {
-			token: 'valid_access_token',
-			refreshToken: 'valid_refresh_token',
-			userId: 'test-user-id',
-			email: 'test@example.com',
-			expiresAt: new Date(Date.now() + 3600000).toISOString(), // Expires in 1 hour
-			savedAt: new Date().toISOString()
-		};
-
-		credentialStore.saveCredentials(validCredentials);
-
-		const credentials = authManager.getCredentials();
-
-		expect(credentials?.token).toBe('valid_access_token');
-	});
-
-	it('should return expired credentials even without refresh token', () => {
-		// Set up expired credentials WITHOUT refresh token
-		// We still return them - it's up to the caller to handle
-		const expiredCredentials: AuthCredentials = {
-			token: 'expired_access_token',
-			refreshToken: undefined,
-			userId: 'test-user-id',
-			email: 'test@example.com',
-			expiresAt: new Date(Date.now() - 1000).toISOString(), // Expired 1 second ago
-			savedAt: new Date().toISOString()
-		};
-
-		credentialStore.saveCredentials(expiredCredentials);
-
-		const credentials = authManager.getCredentials();
-
-		// Returns credentials even if expired
-		expect(credentials).not.toBeNull();
-		expect(credentials?.token).toBe('expired_access_token');
-	});
-
-	it('should return null if no credentials exist', () => {
-		const credentials = authManager.getCredentials();
-		expect(credentials).toBeNull();
-	});
-
-	it('should return credentials regardless of refresh token validity', () => {
-		// Set up expired credentials with refresh token
-		const expiredCredentials: AuthCredentials = {
-			token: 'expired_access_token',
-			refreshToken: 'invalid_refresh_token',
-			userId: 'test-user-id',
-			email: 'test@example.com',
-			expiresAt: new Date(Date.now() - 1000).toISOString(),
-			savedAt: new Date().toISOString()
-		};
-
-		credentialStore.saveCredentials(expiredCredentials);
-
-		const credentials = authManager.getCredentials();
-
-		// Returns credentials - refresh will be attempted by the client which will handle failure
-		expect(credentials).not.toBeNull();
-		expect(credentials?.token).toBe('expired_access_token');
-		expect(credentials?.refreshToken).toBe('invalid_refresh_token');
-	});
-});

packages/tm-core/tests/integration/auth-token-refresh.test.ts

@@ -6,9 +6,6 @@
  */
 
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
-import fs from 'fs';
-import os from 'os';
-import path from 'path';
 import type { Session } from '@supabase/supabase-js';
 import { AuthManager } from '../../src/auth/auth-manager';
 import { CredentialStore } from '../../src/auth/credential-store';
@@ -17,8 +14,6 @@ import type { AuthCredentials } from '../../src/auth/types';
 describe('AuthManager - Token Auto-Refresh Integration', () => {
 	let authManager: AuthManager;
 	let credentialStore: CredentialStore;
-	let tmpDir: string;
-	let authFile: string;
 
 	// Mock Supabase session that will be returned on refresh
 	const mockRefreshedSession: Session = {
@@ -39,21 +34,10 @@ describe('AuthManager - Token Auto-Refresh Integration', () => {
 	};
 
 	beforeEach(() => {
-		// Reset singletons
+		// Reset AuthManager singleton
 		AuthManager.resetInstance();
-		CredentialStore.resetInstance();
 
-		// Create temporary directory for test isolation
+		// Clear any existing credentials
-		tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'tm-auth-integration-'));
-		authFile = path.join(tmpDir, 'auth.json');
-
-		// Initialize AuthManager with test config (this will create CredentialStore internally)
-		authManager = AuthManager.getInstance({
-			configDir: tmpDir,
-			configFile: authFile
-		});
-
-		// Get the CredentialStore instance that AuthManager created
 		credentialStore = CredentialStore.getInstance();
 		credentialStore.clearCredentials();
 	});
@@ -66,17 +50,11 @@ describe('AuthManager - Token Auto-Refresh Integration', () => {
 			// Ignore cleanup errors
 		}
 		AuthManager.resetInstance();
-		CredentialStore.resetInstance();
 		vi.restoreAllMocks();
-
-		// Remove temporary directory
-		if (tmpDir && fs.existsSync(tmpDir)) {
-			fs.rmSync(tmpDir, { recursive: true, force: true });
-		}
 	});
 
 	describe('Expired Token Detection', () => {
-		it('should return expired token for Supabase to refresh', () => {
+		it('should detect expired token', async () => {
 			// Set up expired credentials
 			const expiredCredentials: AuthCredentials = {
 				token: 'expired-token',
@@ -91,15 +69,24 @@ describe('AuthManager - Token Auto-Refresh Integration', () => {
 
 			authManager = AuthManager.getInstance();
 
-			// Get credentials returns them even if expired
+			// Mock the Supabase refreshSession to return new tokens
-			const credentials = authManager.getCredentials();
+			const mockRefreshSession = vi
+				.fn()
+				.mockResolvedValue(mockRefreshedSession);
+			vi.spyOn(
+				authManager['supabaseClient'],
+				'refreshSession'
+			).mockImplementation(mockRefreshSession);
 
+			// Get credentials should trigger refresh
+			const credentials = await authManager.getCredentials();
+
+			expect(mockRefreshSession).toHaveBeenCalledTimes(1);
 			expect(credentials).not.toBeNull();
-			expect(credentials?.token).toBe('expired-token');
+			expect(credentials?.token).toBe('new-access-token-xyz');
-			expect(credentials?.refreshToken).toBe('valid-refresh-token');
 		});
 
-		it('should return valid token', () => {
+		it('should not refresh valid token', async () => {
 			// Set up valid credentials
 			const validCredentials: AuthCredentials = {
 				token: 'valid-token',
@@ -114,14 +101,22 @@ describe('AuthManager - Token Auto-Refresh Integration', () => {
 
 			authManager = AuthManager.getInstance();
 
-			const credentials = authManager.getCredentials();
+			// Mock refresh to ensure it's not called
+			const mockRefreshSession = vi.fn();
+			vi.spyOn(
+				authManager['supabaseClient'],
+				'refreshSession'
+			).mockImplementation(mockRefreshSession);
 
+			const credentials = await authManager.getCredentials();
+
+			expect(mockRefreshSession).not.toHaveBeenCalled();
 			expect(credentials?.token).toBe('valid-token');
 		});
 	});
 
 	describe('Token Refresh Flow', () => {
-		it('should manually refresh expired token and save new credentials', async () => {
+		it('should refresh expired token and save new credentials', async () => {
 			const expiredCredentials: AuthCredentials = {
 				token: 'old-token',
 				refreshToken: 'old-refresh-token',
@@ -145,24 +140,23 @@ describe('AuthManager - Token Auto-Refresh Integration', () => {
 				'refreshSession'
 			).mockResolvedValue(mockRefreshedSession);
 
-			// Explicitly call refreshToken() method
+			const refreshedCredentials = await authManager.getCredentials();
-			const refreshedCredentials = await authManager.refreshToken();
 
 			expect(refreshedCredentials).not.toBeNull();
-			expect(refreshedCredentials.token).toBe('new-access-token-xyz');
+			expect(refreshedCredentials?.token).toBe('new-access-token-xyz');
-			expect(refreshedCredentials.refreshToken).toBe('new-refresh-token-xyz');
+			expect(refreshedCredentials?.refreshToken).toBe('new-refresh-token-xyz');
 
 			// Verify context was preserved
-			expect(refreshedCredentials.selectedContext?.orgId).toBe('test-org');
+			expect(refreshedCredentials?.selectedContext?.orgId).toBe('test-org');
-			expect(refreshedCredentials.selectedContext?.briefId).toBe('test-brief');
+			expect(refreshedCredentials?.selectedContext?.briefId).toBe('test-brief');
 
 			// Verify new expiration is in the future
-			const newExpiry = new Date(refreshedCredentials.expiresAt!).getTime();
+			const newExpiry = new Date(refreshedCredentials!.expiresAt!).getTime();
 			const now = Date.now();
 			expect(newExpiry).toBeGreaterThan(now);
 		});
 
-		it('should throw error if manual refresh fails', async () => {
+		it('should return null if refresh fails', async () => {
 			const expiredCredentials: AuthCredentials = {
 				token: 'expired-token',
 				refreshToken: 'invalid-refresh-token',
@@ -182,11 +176,12 @@ describe('AuthManager - Token Auto-Refresh Integration', () => {
 				'refreshSession'
 			).mockRejectedValue(new Error('Refresh token expired'));
 
-			// Explicit refreshToken() call should throw
+			const credentials = await authManager.getCredentials();
-			await expect(authManager.refreshToken()).rejects.toThrow();
+
+			expect(credentials).toBeNull();
 		});
 
-		it('should return expired credentials even without refresh token', () => {
+		it('should return null if no refresh token available', async () => {
 			const expiredCredentials: AuthCredentials = {
 				token: 'expired-token',
 				// No refresh token
@@ -200,21 +195,18 @@ describe('AuthManager - Token Auto-Refresh Integration', () => {
 
 			authManager = AuthManager.getInstance();
 
-			const credentials = authManager.getCredentials();
+			const credentials = await authManager.getCredentials();
 
-			// Credentials are returned even without refresh token
+			expect(credentials).toBeNull();
-			expect(credentials).not.toBeNull();
-			expect(credentials?.token).toBe('expired-token');
-			expect(credentials?.refreshToken).toBeUndefined();
 		});
 
-		it('should return null if credentials missing expiresAt', () => {
+		it('should return null if credentials missing expiresAt', async () => {
 			const credentialsWithoutExpiry: AuthCredentials = {
 				token: 'test-token',
 				refreshToken: 'refresh-token',
 				userId: 'test-user-id',
 				email: 'test@example.com',
-				// Missing expiresAt - invalid token
+				// Missing expiresAt
 				savedAt: new Date().toISOString()
 			} as any;
 
@@ -222,17 +214,16 @@ describe('AuthManager - Token Auto-Refresh Integration', () => {
 
 			authManager = AuthManager.getInstance();
 
-			const credentials = authManager.getCredentials();
+			const credentials = await authManager.getCredentials();
 
-			// Tokens without valid expiration are considered invalid
+			// Should return null because no valid expiration
 			expect(credentials).toBeNull();
 		});
 	});
 
 	describe('Clock Skew Tolerance', () => {
-		it('should return credentials within 30-second expiry window', () => {
+		it('should refresh token within 30-second expiry window', async () => {
 			// Token expires in 15 seconds (within 30-second buffer)
-			// Supabase will handle refresh automatically
 			const almostExpiredCredentials: AuthCredentials = {
 				token: 'almost-expired-token',
 				refreshToken: 'valid-refresh-token',
@@ -246,16 +237,23 @@ describe('AuthManager - Token Auto-Refresh Integration', () => {
 
 			authManager = AuthManager.getInstance();
 
-			const credentials = authManager.getCredentials();
+			const mockRefreshSession = vi
+				.fn()
+				.mockResolvedValue(mockRefreshedSession);
+			vi.spyOn(
+				authManager['supabaseClient'],
+				'refreshSession'
+			).mockImplementation(mockRefreshSession);
 
-			// Credentials are returned (Supabase handles auto-refresh in background)
+			const credentials = await authManager.getCredentials();
-			expect(credentials).not.toBeNull();
+
-			expect(credentials?.token).toBe('almost-expired-token');
+			// Should trigger refresh due to 30-second buffer
-			expect(credentials?.refreshToken).toBe('valid-refresh-token');
+			expect(mockRefreshSession).toHaveBeenCalledTimes(1);
+			expect(credentials?.token).toBe('new-access-token-xyz');
 		});
 
-		it('should return valid token well before expiry', () => {
+		it('should not refresh token well before expiry', async () => {
-			// Token expires in 5 minutes
+			// Token expires in 5 minutes (well outside 30-second buffer)
 			const validCredentials: AuthCredentials = {
 				token: 'valid-token',
 				refreshToken: 'valid-refresh-token',
@@ -269,17 +267,21 @@ describe('AuthManager - Token Auto-Refresh Integration', () => {
 
 			authManager = AuthManager.getInstance();
 
-			const credentials = authManager.getCredentials();
+			const mockRefreshSession = vi.fn();
+			vi.spyOn(
+				authManager['supabaseClient'],
+				'refreshSession'
+			).mockImplementation(mockRefreshSession);
 
-			// Valid credentials are returned as-is
+			const credentials = await authManager.getCredentials();
-			expect(credentials).not.toBeNull();
+
+			expect(mockRefreshSession).not.toHaveBeenCalled();
 			expect(credentials?.token).toBe('valid-token');
-			expect(credentials?.refreshToken).toBe('valid-refresh-token');
 		});
 	});
 
 	describe('Synchronous vs Async Methods', () => {
-		it('getCredentials should return expired credentials', () => {
+		it('getCredentialsSync should not trigger refresh', () => {
 			const expiredCredentials: AuthCredentials = {
 				token: 'expired-token',
 				refreshToken: 'valid-refresh-token',
@@ -293,17 +295,40 @@ describe('AuthManager - Token Auto-Refresh Integration', () => {
 
 			authManager = AuthManager.getInstance();
 
-			// Returns credentials even if expired - Supabase will handle refresh
+			// Synchronous call should return null without refresh
-			const credentials = authManager.getCredentials();
+			const credentials = authManager.getCredentialsSync();
+
+			expect(credentials).toBeNull();
+		});
+
+		it('getCredentials async should trigger refresh', async () => {
+			const expiredCredentials: AuthCredentials = {
+				token: 'expired-token',
+				refreshToken: 'valid-refresh-token',
+				userId: 'test-user-id',
+				email: 'test@example.com',
+				expiresAt: new Date(Date.now() - 60000).toISOString(),
+				savedAt: new Date().toISOString()
+			};
+
+			credentialStore.saveCredentials(expiredCredentials);
+
+			authManager = AuthManager.getInstance();
+
+			vi.spyOn(
+				authManager['supabaseClient'],
+				'refreshSession'
+			).mockResolvedValue(mockRefreshedSession);
+
+			const credentials = await authManager.getCredentials();
 
 			expect(credentials).not.toBeNull();
-			expect(credentials?.token).toBe('expired-token');
+			expect(credentials?.token).toBe('new-access-token-xyz');
-			expect(credentials?.refreshToken).toBe('valid-refresh-token');
 		});
 	});
 
 	describe('Multiple Concurrent Calls', () => {
-		it('should handle concurrent getCredentials calls gracefully', () => {
+		it('should handle concurrent getCredentials calls gracefully', async () => {
 			const expiredCredentials: AuthCredentials = {
 				token: 'expired-token',
 				refreshToken: 'valid-refresh-token',
@@ -317,20 +342,29 @@ describe('AuthManager - Token Auto-Refresh Integration', () => {
 
 			authManager = AuthManager.getInstance();
 
-			// Make multiple concurrent calls (synchronous now)
+			const mockRefreshSession = vi
-			const creds1 = authManager.getCredentials();
+				.fn()
-			const creds2 = authManager.getCredentials();
+				.mockResolvedValue(mockRefreshedSession);
-			const creds3 = authManager.getCredentials();
+			vi.spyOn(
+				authManager['supabaseClient'],
+				'refreshSession'
+			).mockImplementation(mockRefreshSession);
 
-			// All should get the same credentials (even if expired)
+			// Make multiple concurrent calls
-			expect(creds1?.token).toBe('expired-token');
+			const [creds1, creds2, creds3] = await Promise.all([
-			expect(creds2?.token).toBe('expired-token');
+				authManager.getCredentials(),
-			expect(creds3?.token).toBe('expired-token');
+				authManager.getCredentials(),
+				authManager.getCredentials()
+			]);
 
-			// All include refresh token for Supabase to use
+			// All should get the refreshed token
-			expect(creds1?.refreshToken).toBe('valid-refresh-token');
+			expect(creds1?.token).toBe('new-access-token-xyz');
-			expect(creds2?.refreshToken).toBe('valid-refresh-token');
+			expect(creds2?.token).toBe('new-access-token-xyz');
-			expect(creds3?.refreshToken).toBe('valid-refresh-token');
+			expect(creds3?.token).toBe('new-access-token-xyz');
+
+			// Refresh might be called multiple times, but that's okay
+			// (ideally we'd debounce, but this is acceptable behavior)
+			expect(mockRefreshSession).toHaveBeenCalled();
 		});
 	});
 });

src/utils/path-utils.js

@@ -47,33 +47,21 @@ export function normalizeProjectRoot(projectRoot) {
 
 /**
  * Find the project root directory by looking for project markers
- * Traverses upwards from startDir until a project marker is found or filesystem root is reached
+ * @param {string} startDir - Directory to start searching from
- * Limited to 50 parent directory levels to prevent excessive traversal
+ * @returns {string|null} - Project root path or null if not found
- * @param {string} startDir - Directory to start searching from (defaults to process.cwd())
- * @returns {string} - Project root path (falls back to current directory if no markers found)
  */
 export function findProjectRoot(startDir = process.cwd()) {
-	// Define project markers that indicate a project root
-	// Prioritize Task Master specific markers first
 	const projectMarkers = [
-		'.taskmaster', // Task Master directory (highest priority)
+		'.taskmaster',
-		TASKMASTER_CONFIG_FILE, // .taskmaster/config.json
+		TASKMASTER_TASKS_FILE,
-		TASKMASTER_TASKS_FILE, // .taskmaster/tasks/tasks.json
+		'tasks.json',
-		LEGACY_CONFIG_FILE, // .taskmasterconfig (legacy)
+		LEGACY_TASKS_FILE,
-		LEGACY_TASKS_FILE, // tasks/tasks.json (legacy)
+		'.git',
-		'tasks.json', // Root tasks.json (legacy)
+		'.svn',
-		'.git', // Git repository
+		'package.json',
-		'.svn', // SVN repository
+		'yarn.lock',
-		'package.json', // Node.js project
+		'package-lock.json',
-		'yarn.lock', // Yarn project
+		'pnpm-lock.yaml'
-		'package-lock.json', // npm project
-		'pnpm-lock.yaml', // pnpm project
-		'Cargo.toml', // Rust project
-		'go.mod', // Go project
-		'pyproject.toml', // Python project
-		'requirements.txt', // Python project
-		'Gemfile', // Ruby project
-		'composer.json' // PHP project
 	];
 
 	let currentDir = path.resolve(startDir);
@@ -81,36 +69,19 @@ export function findProjectRoot(startDir = process.cwd()) {
 	const maxDepth = 50; // Reasonable limit to prevent infinite loops
 	let depth = 0;
 
-	// Traverse upwards looking for project markers
 	while (currentDir !== rootDir && depth < maxDepth) {
 		// Check if current directory contains any project markers
 		for (const marker of projectMarkers) {
 			const markerPath = path.join(currentDir, marker);
-			try {
+			if (fs.existsSync(markerPath)) {
-				if (fs.existsSync(markerPath)) {
+				return currentDir;
-					// Found a project marker - return this directory as project root
-					return currentDir;
-				}
-			} catch (error) {
-				// Ignore permission errors and continue searching
-				continue;
 			}
 		}
-
+		currentDir = path.dirname(currentDir);
-		// Move up one directory level
-		const parentDir = path.dirname(currentDir);
-
-		// Safety check: if dirname returns the same path, we've hit the root
-		if (parentDir === currentDir) {
-			break;
-		}
-
-		currentDir = parentDir;
 		depth++;
 	}
 
 	// Fallback to current working directory if no project root found
-	// This ensures the function always returns a valid path
 	return process.cwd();
 }
 

tests/helpers/tool-counts.js

@@ -1,123 +0,0 @@
-/**
- * tool-counts.js
- * Shared helper for validating tool counts across tests and validation scripts
- */
-
-import {
-	getToolCounts,
-	getToolCategories
-} from '../../mcp-server/src/tools/tool-registry.js';
-
-/**
- * Expected tool counts - update these when tools are added/removed
- * These serve as the canonical source of truth for expected counts
- */
-export const EXPECTED_TOOL_COUNTS = {
-	core: 7,
-	standard: 15,
-	total: 36
-};
-
-/**
- * Expected core tools list for validation
- */
-export const EXPECTED_CORE_TOOLS = [
-	'get_tasks',
-	'next_task',
-	'get_task',
-	'set_task_status',
-	'update_subtask',
-	'parse_prd',
-	'expand_task'
-];
-
-/**
- * Validate that actual tool counts match expected counts
- * @returns {Object} Validation result with isValid flag and details
- */
-export function validateToolCounts() {
-	const actual = getToolCounts();
-	const expected = EXPECTED_TOOL_COUNTS;
-
-	const isValid =
-		actual.core === expected.core &&
-		actual.standard === expected.standard &&
-		actual.total === expected.total;
-
-	return {
-		isValid,
-		actual,
-		expected,
-		differences: {
-			core: actual.core - expected.core,
-			standard: actual.standard - expected.standard,
-			total: actual.total - expected.total
-		}
-	};
-}
-
-/**
- * Validate that tool categories have correct structure and content
- * @returns {Object} Validation result
- */
-export function validateToolStructure() {
-	const categories = getToolCategories();
-	const counts = getToolCounts();
-
-	// Check that core tools are subset of standard tools
-	const coreInStandard = categories.core.every((tool) =>
-		categories.standard.includes(tool)
-	);
-
-	// Check that standard tools are subset of all tools
-	const standardInAll = categories.standard.every((tool) =>
-		categories.all.includes(tool)
-	);
-
-	// Check that expected core tools match actual
-	const expectedCoreMatch =
-		EXPECTED_CORE_TOOLS.every((tool) => categories.core.includes(tool)) &&
-		categories.core.every((tool) => EXPECTED_CORE_TOOLS.includes(tool));
-
-	// Check array lengths match counts
-	const lengthsMatch =
-		categories.core.length === counts.core &&
-		categories.standard.length === counts.standard &&
-		categories.all.length === counts.total;
-
-	return {
-		isValid:
-			coreInStandard && standardInAll && expectedCoreMatch && lengthsMatch,
-		details: {
-			coreInStandard,
-			standardInAll,
-			expectedCoreMatch,
-			lengthsMatch
-		},
-		categories,
-		counts
-	};
-}
-
-/**
- * Get a detailed report of all tool information
- * @returns {Object} Comprehensive tool information
- */
-export function getToolReport() {
-	const counts = getToolCounts();
-	const categories = getToolCategories();
-	const validation = validateToolCounts();
-	const structure = validateToolStructure();
-
-	return {
-		counts,
-		categories,
-		validation,
-		structure,
-		summary: {
-			totalValid: validation.isValid && structure.isValid,
-			countsValid: validation.isValid,
-			structureValid: structure.isValid
-		}
-	};
-}

tests/unit/mcp/tools/tool-registration.test.js

@@ -1,410 +0,0 @@
-/**
- * tool-registration.test.js
- * Comprehensive unit tests for the Task Master MCP tool registration system
- * Tests environment variable control system covering all configuration modes and edge cases
- */
-
-import {
-	describe,
-	it,
-	expect,
-	beforeEach,
-	afterEach,
-	jest
-} from '@jest/globals';
-
-import {
-	EXPECTED_TOOL_COUNTS,
-	EXPECTED_CORE_TOOLS,
-	validateToolCounts,
-	validateToolStructure
-} from '../../../helpers/tool-counts.js';
-
-import { registerTaskMasterTools } from '../../../../mcp-server/src/tools/index.js';
-import {
-	toolRegistry,
-	coreTools,
-	standardTools
-} from '../../../../mcp-server/src/tools/tool-registry.js';
-
-// Derive constants from imported registry to avoid brittle magic numbers
-const ALL_COUNT = Object.keys(toolRegistry).length;
-const CORE_COUNT = coreTools.length;
-const STANDARD_COUNT = standardTools.length;
-
-describe('Task Master Tool Registration System', () => {
-	let mockServer;
-	let originalEnv;
-
-	beforeEach(() => {
-		originalEnv = process.env.TASK_MASTER_TOOLS;
-
-		mockServer = {
-			tools: [],
-			addTool: jest.fn((tool) => {
-				mockServer.tools.push(tool);
-				return tool;
-			})
-		};
-
-		delete process.env.TASK_MASTER_TOOLS;
-	});
-
-	afterEach(() => {
-		if (originalEnv !== undefined) {
-			process.env.TASK_MASTER_TOOLS = originalEnv;
-		} else {
-			delete process.env.TASK_MASTER_TOOLS;
-		}
-
-		jest.clearAllMocks();
-	});
-
-	describe('Test Environment Setup', () => {
-		it('should have properly configured mock server', () => {
-			expect(mockServer).toBeDefined();
-			expect(typeof mockServer.addTool).toBe('function');
-			expect(Array.isArray(mockServer.tools)).toBe(true);
-			expect(mockServer.tools.length).toBe(0);
-		});
-
-		it('should have correct tool registry structure', () => {
-			const validation = validateToolCounts();
-			expect(validation.isValid).toBe(true);
-
-			if (!validation.isValid) {
-				console.error('Tool count validation failed:', validation);
-			}
-
-			expect(validation.actual.total).toBe(EXPECTED_TOOL_COUNTS.total);
-			expect(validation.actual.core).toBe(EXPECTED_TOOL_COUNTS.core);
-			expect(validation.actual.standard).toBe(EXPECTED_TOOL_COUNTS.standard);
-		});
-
-		it('should have correct core tools', () => {
-			const structure = validateToolStructure();
-			expect(structure.isValid).toBe(true);
-
-			if (!structure.isValid) {
-				console.error('Tool structure validation failed:', structure);
-			}
-
-			expect(coreTools).toEqual(expect.arrayContaining(EXPECTED_CORE_TOOLS));
-			expect(coreTools.length).toBe(EXPECTED_TOOL_COUNTS.core);
-		});
-
-		it('should have correct standard tools that include all core tools', () => {
-			const structure = validateToolStructure();
-			expect(structure.details.coreInStandard).toBe(true);
-			expect(standardTools.length).toBe(EXPECTED_TOOL_COUNTS.standard);
-
-			coreTools.forEach((tool) => {
-				expect(standardTools).toContain(tool);
-			});
-		});
-
-		it('should have all expected tools in registry', () => {
-			const expectedTools = [
-				'initialize_project',
-				'models',
-				'research',
-				'add_tag',
-				'delete_tag',
-				'get_tasks',
-				'next_task',
-				'get_task'
-			];
-			expectedTools.forEach((tool) => {
-				expect(toolRegistry).toHaveProperty(tool);
-			});
-		});
-	});
-
-	describe('Configuration Modes', () => {
-		it(`should register all tools (${ALL_COUNT}) when TASK_MASTER_TOOLS is not set (default behavior)`, () => {
-			delete process.env.TASK_MASTER_TOOLS;
-
-			registerTaskMasterTools(mockServer);
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(
-				EXPECTED_TOOL_COUNTS.total
-			);
-		});
-
-		it(`should register all tools (${ALL_COUNT}) when TASK_MASTER_TOOLS=all`, () => {
-			process.env.TASK_MASTER_TOOLS = 'all';
-
-			registerTaskMasterTools(mockServer);
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(ALL_COUNT);
-		});
-
-		it(`should register exactly ${CORE_COUNT} core tools when TASK_MASTER_TOOLS=core`, () => {
-			process.env.TASK_MASTER_TOOLS = 'core';
-
-			registerTaskMasterTools(mockServer, 'core');
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(
-				EXPECTED_TOOL_COUNTS.core
-			);
-		});
-
-		it(`should register exactly ${STANDARD_COUNT} standard tools when TASK_MASTER_TOOLS=standard`, () => {
-			process.env.TASK_MASTER_TOOLS = 'standard';
-
-			registerTaskMasterTools(mockServer, 'standard');
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(
-				EXPECTED_TOOL_COUNTS.standard
-			);
-		});
-
-		it(`should treat lean as alias for core mode (${CORE_COUNT} tools)`, () => {
-			process.env.TASK_MASTER_TOOLS = 'lean';
-
-			registerTaskMasterTools(mockServer, 'lean');
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(CORE_COUNT);
-		});
-
-		it('should handle case insensitive configuration values', () => {
-			process.env.TASK_MASTER_TOOLS = 'CORE';
-
-			registerTaskMasterTools(mockServer, 'CORE');
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(CORE_COUNT);
-		});
-	});
-
-	describe('Custom Tool Selection and Edge Cases', () => {
-		it('should register specific tools from comma-separated list', () => {
-			process.env.TASK_MASTER_TOOLS = 'get_tasks,next_task,get_task';
-
-			registerTaskMasterTools(mockServer, 'get_tasks,next_task,get_task');
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(3);
-		});
-
-		it('should handle mixed valid and invalid tool names gracefully', () => {
-			process.env.TASK_MASTER_TOOLS =
-				'invalid_tool,get_tasks,fake_tool,next_task';
-
-			registerTaskMasterTools(
-				mockServer,
-				'invalid_tool,get_tasks,fake_tool,next_task'
-			);
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(2);
-		});
-
-		it('should default to all tools with completely invalid input', () => {
-			process.env.TASK_MASTER_TOOLS = 'completely_invalid';
-
-			registerTaskMasterTools(mockServer);
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(ALL_COUNT);
-		});
-
-		it('should handle empty string environment variable', () => {
-			process.env.TASK_MASTER_TOOLS = '';
-
-			registerTaskMasterTools(mockServer);
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(ALL_COUNT);
-		});
-
-		it('should handle whitespace in comma-separated lists', () => {
-			process.env.TASK_MASTER_TOOLS = ' get_tasks , next_task , get_task ';
-
-			registerTaskMasterTools(mockServer, ' get_tasks , next_task , get_task ');
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(3);
-		});
-
-		it('should ignore duplicate tools in list', () => {
-			process.env.TASK_MASTER_TOOLS = 'get_tasks,get_tasks,next_task,get_tasks';
-
-			registerTaskMasterTools(
-				mockServer,
-				'get_tasks,get_tasks,next_task,get_tasks'
-			);
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(2);
-		});
-
-		it('should handle only commas and empty entries', () => {
-			process.env.TASK_MASTER_TOOLS = ',,,';
-
-			registerTaskMasterTools(mockServer);
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(ALL_COUNT);
-		});
-
-		it('should handle single tool selection', () => {
-			process.env.TASK_MASTER_TOOLS = 'get_tasks';
-
-			registerTaskMasterTools(mockServer, 'get_tasks');
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(1);
-		});
-	});
-
-	describe('Coverage Analysis and Integration Tests', () => {
-		it('should provide 100% code coverage for environment control logic', () => {
-			const testCases = [
-				{
-					env: undefined,
-					expectedCount: ALL_COUNT,
-					description: 'undefined env (all)'
-				},
-				{
-					env: '',
-					expectedCount: ALL_COUNT,
-					description: 'empty string (all)'
-				},
-				{ env: 'all', expectedCount: ALL_COUNT, description: 'all mode' },
-				{ env: 'core', expectedCount: CORE_COUNT, description: 'core mode' },
-				{
-					env: 'lean',
-					expectedCount: CORE_COUNT,
-					description: 'lean mode (alias)'
-				},
-				{
-					env: 'standard',
-					expectedCount: STANDARD_COUNT,
-					description: 'standard mode'
-				},
-				{
-					env: 'get_tasks,next_task',
-					expectedCount: 2,
-					description: 'custom list'
-				},
-				{
-					env: 'invalid_tool',
-					expectedCount: ALL_COUNT,
-					description: 'invalid fallback'
-				}
-			];
-
-			testCases.forEach((testCase) => {
-				delete process.env.TASK_MASTER_TOOLS;
-				if (testCase.env !== undefined) {
-					process.env.TASK_MASTER_TOOLS = testCase.env;
-				}
-
-				mockServer.tools = [];
-				mockServer.addTool.mockClear();
-
-				registerTaskMasterTools(mockServer, testCase.env || 'all');
-
-				expect(mockServer.addTool).toHaveBeenCalledTimes(
-					testCase.expectedCount
-				);
-			});
-		});
-
-		it('should have optimal performance characteristics', () => {
-			const startTime = Date.now();
-
-			process.env.TASK_MASTER_TOOLS = 'all';
-
-			registerTaskMasterTools(mockServer);
-
-			const endTime = Date.now();
-			const executionTime = endTime - startTime;
-
-			expect(executionTime).toBeLessThan(100);
-			expect(mockServer.addTool).toHaveBeenCalledTimes(ALL_COUNT);
-		});
-
-		it('should validate token reduction claims', () => {
-			expect(coreTools.length).toBeLessThan(standardTools.length);
-			expect(standardTools.length).toBeLessThan(
-				Object.keys(toolRegistry).length
-			);
-
-			expect(coreTools.length).toBe(CORE_COUNT);
-			expect(standardTools.length).toBe(STANDARD_COUNT);
-			expect(Object.keys(toolRegistry).length).toBe(ALL_COUNT);
-
-			const allToolsCount = Object.keys(toolRegistry).length;
-			const coreReduction =
-				((allToolsCount - coreTools.length) / allToolsCount) * 100;
-			const standardReduction =
-				((allToolsCount - standardTools.length) / allToolsCount) * 100;
-
-			expect(coreReduction).toBeGreaterThan(80);
-			expect(standardReduction).toBeGreaterThan(50);
-		});
-
-		it('should maintain referential integrity of tool registry', () => {
-			coreTools.forEach((tool) => {
-				expect(standardTools).toContain(tool);
-			});
-
-			standardTools.forEach((tool) => {
-				expect(toolRegistry).toHaveProperty(tool);
-			});
-
-			Object.keys(toolRegistry).forEach((tool) => {
-				expect(typeof toolRegistry[tool]).toBe('function');
-			});
-		});
-
-		it('should handle concurrent registration attempts', () => {
-			process.env.TASK_MASTER_TOOLS = 'core';
-
-			registerTaskMasterTools(mockServer, 'core');
-			registerTaskMasterTools(mockServer, 'core');
-			registerTaskMasterTools(mockServer, 'core');
-
-			expect(mockServer.addTool).toHaveBeenCalledTimes(CORE_COUNT * 3);
-		});
-
-		it('should validate all documented tool categories exist', () => {
-			const allTools = Object.keys(toolRegistry);
-
-			const projectSetupTools = allTools.filter((tool) =>
-				['initialize_project', 'models', 'rules', 'parse_prd'].includes(tool)
-			);
-			expect(projectSetupTools.length).toBeGreaterThan(0);
-
-			const taskManagementTools = allTools.filter((tool) =>
-				['get_tasks', 'get_task', 'next_task', 'set_task_status'].includes(tool)
-			);
-			expect(taskManagementTools.length).toBeGreaterThan(0);
-
-			const analysisTools = allTools.filter((tool) =>
-				['analyze_project_complexity', 'complexity_report'].includes(tool)
-			);
-			expect(analysisTools.length).toBeGreaterThan(0);
-
-			const tagManagementTools = allTools.filter((tool) =>
-				['add_tag', 'delete_tag', 'list_tags', 'use_tag'].includes(tool)
-			);
-			expect(tagManagementTools.length).toBeGreaterThan(0);
-		});
-
-		it('should handle error conditions gracefully', () => {
-			const problematicInputs = [
-				'null',
-				'undefined',
-				'   ',
-				'\n\t',
-				'special!@#$%^&*()characters',
-				'very,very,very,very,very,very,very,long,comma,separated,list,with,invalid,tools,that,should,fallback,to,all'
-			];
-
-			problematicInputs.forEach((input) => {
-				mockServer.tools = [];
-				mockServer.addTool.mockClear();
-
-				process.env.TASK_MASTER_TOOLS = input;
-
-				expect(() => registerTaskMasterTools(mockServer)).not.toThrow();
-
-				expect(mockServer.addTool).toHaveBeenCalledTimes(ALL_COUNT);
-			});
-		});
-	});
-});

tests/unit/path-utils-find-project-root.test.js

@@ -1,223 +0,0 @@
-/**
- * Unit tests for findProjectRoot() function
- * Tests the parent directory traversal functionality
- */
-
-import { jest } from '@jest/globals';
-import path from 'path';
-import fs from 'fs';
-
-// Import the function to test
-import { findProjectRoot } from '../../src/utils/path-utils.js';
-
-describe('findProjectRoot', () => {
-	describe('Parent Directory Traversal', () => {
-		test('should find .taskmaster in parent directory', () => {
-			const mockExistsSync = jest.spyOn(fs, 'existsSync');
-
-			mockExistsSync.mockImplementation((checkPath) => {
-				const normalized = path.normalize(checkPath);
-				// .taskmaster exists only at /project
-				return normalized === path.normalize('/project/.taskmaster');
-			});
-
-			const result = findProjectRoot('/project/subdir');
-
-			expect(result).toBe('/project');
-
-			mockExistsSync.mockRestore();
-		});
-
-		test('should find .git in parent directory', () => {
-			const mockExistsSync = jest.spyOn(fs, 'existsSync');
-
-			mockExistsSync.mockImplementation((checkPath) => {
-				const normalized = path.normalize(checkPath);
-				return normalized === path.normalize('/project/.git');
-			});
-
-			const result = findProjectRoot('/project/subdir');
-
-			expect(result).toBe('/project');
-
-			mockExistsSync.mockRestore();
-		});
-
-		test('should find package.json in parent directory', () => {
-			const mockExistsSync = jest.spyOn(fs, 'existsSync');
-
-			mockExistsSync.mockImplementation((checkPath) => {
-				const normalized = path.normalize(checkPath);
-				return normalized === path.normalize('/project/package.json');
-			});
-
-			const result = findProjectRoot('/project/subdir');
-
-			expect(result).toBe('/project');
-
-			mockExistsSync.mockRestore();
-		});
-
-		test('should traverse multiple levels to find project root', () => {
-			const mockExistsSync = jest.spyOn(fs, 'existsSync');
-
-			mockExistsSync.mockImplementation((checkPath) => {
-				const normalized = path.normalize(checkPath);
-				// Only exists at /project, not in any subdirectories
-				return normalized === path.normalize('/project/.taskmaster');
-			});
-
-			const result = findProjectRoot('/project/subdir/deep/nested');
-
-			expect(result).toBe('/project');
-
-			mockExistsSync.mockRestore();
-		});
-
-		test('should return current directory as fallback when no markers found', () => {
-			const mockExistsSync = jest.spyOn(fs, 'existsSync');
-
-			// No project markers exist anywhere
-			mockExistsSync.mockReturnValue(false);
-
-			const result = findProjectRoot('/some/random/path');
-
-			// Should fall back to process.cwd()
-			expect(result).toBe(process.cwd());
-
-			mockExistsSync.mockRestore();
-		});
-
-		test('should find markers at current directory before checking parent', () => {
-			const mockExistsSync = jest.spyOn(fs, 'existsSync');
-
-			mockExistsSync.mockImplementation((checkPath) => {
-				const normalized = path.normalize(checkPath);
-				// .git exists at /project/subdir, .taskmaster exists at /project
-				if (normalized.includes('/project/subdir/.git')) return true;
-				if (normalized.includes('/project/.taskmaster')) return true;
-				return false;
-			});
-
-			const result = findProjectRoot('/project/subdir');
-
-			// Should find /project/subdir first because .git exists there,
-			// even though .taskmaster is earlier in the marker array
-			expect(result).toBe('/project/subdir');
-
-			mockExistsSync.mockRestore();
-		});
-
-		test('should handle permission errors gracefully', () => {
-			const mockExistsSync = jest.spyOn(fs, 'existsSync');
-
-			mockExistsSync.mockImplementation((checkPath) => {
-				const normalized = path.normalize(checkPath);
-				// Throw permission error for checks in /project/subdir
-				if (normalized.startsWith('/project/subdir/')) {
-					throw new Error('EACCES: permission denied');
-				}
-				// Return true only for .taskmaster at /project
-				return normalized.includes('/project/.taskmaster');
-			});
-
-			const result = findProjectRoot('/project/subdir');
-
-			// Should handle permission errors in subdirectory and traverse to parent
-			expect(result).toBe('/project');
-
-			mockExistsSync.mockRestore();
-		});
-
-		test('should detect filesystem root correctly', () => {
-			const mockExistsSync = jest.spyOn(fs, 'existsSync');
-
-			// No markers exist
-			mockExistsSync.mockReturnValue(false);
-
-			const result = findProjectRoot('/');
-
-			// Should stop at root and fall back to process.cwd()
-			expect(result).toBe(process.cwd());
-
-			mockExistsSync.mockRestore();
-		});
-
-		test('should recognize various project markers', () => {
-			const projectMarkers = [
-				'.taskmaster',
-				'.git',
-				'package.json',
-				'Cargo.toml',
-				'go.mod',
-				'pyproject.toml',
-				'requirements.txt',
-				'Gemfile',
-				'composer.json'
-			];
-
-			projectMarkers.forEach((marker) => {
-				const mockExistsSync = jest.spyOn(fs, 'existsSync');
-
-				mockExistsSync.mockImplementation((checkPath) => {
-					const normalized = path.normalize(checkPath);
-					return normalized.includes(`/project/${marker}`);
-				});
-
-				const result = findProjectRoot('/project/subdir');
-
-				expect(result).toBe('/project');
-
-				mockExistsSync.mockRestore();
-			});
-		});
-	});
-
-	describe('Edge Cases', () => {
-		test('should handle empty string as startDir', () => {
-			const result = findProjectRoot('');
-
-			// Should use process.cwd() or fall back appropriately
-			expect(typeof result).toBe('string');
-			expect(result.length).toBeGreaterThan(0);
-		});
-
-		test('should handle relative paths', () => {
-			const mockExistsSync = jest.spyOn(fs, 'existsSync');
-
-			mockExistsSync.mockImplementation((checkPath) => {
-				// Simulate .git existing in the resolved path
-				return checkPath.includes('.git');
-			});
-
-			const result = findProjectRoot('./subdir');
-
-			expect(typeof result).toBe('string');
-
-			mockExistsSync.mockRestore();
-		});
-
-		test('should not exceed max depth limit', () => {
-			const mockExistsSync = jest.spyOn(fs, 'existsSync');
-
-			// Track how many times existsSync is called
-			let callCount = 0;
-			mockExistsSync.mockImplementation(() => {
-				callCount++;
-				return false; // Never find a marker
-			});
-
-			// Create a very deep path
-			const deepPath = '/a/'.repeat(100) + 'deep';
-			const result = findProjectRoot(deepPath);
-
-			// Should stop after max depth (50) and not check 100 levels
-			// Each level checks multiple markers, so callCount will be high but bounded
-			expect(callCount).toBeLessThan(1000); // Reasonable upper bound
-			// With 18 markers and max depth of 50, expect around 900 calls maximum
-			expect(callCount).toBeLessThanOrEqual(50 * 18);
-
-			mockExistsSync.mockRestore();
-		});
-	});
-});