chore: add changeset for Claude Code provider feature

fix(models): add missing --claude-code flag to models command
The models command was missing the --claude-code provider flag, preventing users from setting Claude Code models via CLI. While the backend already supported claude-code as a provider hint, there was no command-line flag to trigger it. Changes: - Added --claude-code option to models command alongside existing provider flags - Updated provider flags validation to include claudeCode option - Added claude-code to providerHint logic for all three model roles (main, research, fallback) - Updated error message to include --claude-code in list of mutually exclusive flags - Added example usage in help text This allows users to properly set Claude Code models using commands like: task-master models --set-main sonnet --claude-code task-master models --set-main opus --claude-code Without this flag, users would get "Model ID not found" errors when trying to set claude-code models, as the system couldn't determine the correct provider for generic model names like "sonnet" or "opus".
2025-06-20 16:21:37 +03:00 · 2025-06-20 16:21:36 +03:00 · 2025-06-20 16:21:21 +03:00 · 2025-06-20 16:21:21 +03:00 · 2025-06-20 16:21:21 +03:00 · 2025-06-20 16:20:56 +03:00
18 changed files with 188 additions and 2139 deletions
--- a/.changeset/bright-llamas-enter.md
+++ b/.changeset/bright-llamas-enter.md
@@ -1,12 +0,0 @@
 ---
 "task-master-ai": patch
 ---
 Fix expand command preserving tagged task structure and preventing data corruption
 - Enhance E2E tests with comprehensive tag-aware expand testing to verify tag corruption fix
 - Add new test section for feature-expand tag creation and testing during expand operations
 - Verify tag preservation during expand, force expand, and expand --all operations
 - Test that master tag remains intact while feature-expand tag receives subtasks correctly
 - Fix file path references to use correct .taskmaster/config.json and .taskmaster/tasks/tasks.json locations
 - All tag corruption verification tests pass successfully, confirming the expand command tag corruption bug fix works as expected
--- a/.changeset/huge-moose-prove.md
+++ b/.changeset/huge-moose-prove.md
@@ -1,8 +0,0 @@
 ---
 "task-master-ai": minor
 ---
 Can now configure baseURL of provider with `<PROVIDER>_BASE_URL`
 - For example:
  - `OPENAI_BASE_URL`
--- a/.changeset/shy-groups-fly.md
+++ b/.changeset/shy-groups-fly.md
@@ -1,5 +0,0 @@
 ---
 "task-master-ai": minor
 ---
 Add better support for python projects by adding `pyproject.toml` as a projectRoot marker
--- a/.changeset/tiny-dogs-change.md
+++ b/.changeset/tiny-dogs-change.md
@@ -1,5 +0,0 @@
 ---
 "task-master-ai": patch
 ---
 Improve mcp keys check in cursor
--- a/README.md
+++ b/README.md
@@ -94,8 +94,6 @@ MCP (Model Control Protocol) lets you run Task Master directly from your editor.
 > 🔑 Replace `YOUR_…_KEY_HERE` with your real API keys. You can remove keys you don't use.
 > **Note**: If you see `0 tools enabled` in the MCP settings, try removing the `--package=task-master-ai` flag from `args`.
 ###### VS Code (`servers` + `type`)
 ```json
--- a/docs/configuration.md
+++ b/docs/configuration.md
@@ -72,7 +72,6 @@ Taskmaster uses two primary methods for configuration:
  - `XAI_API_KEY`: Your X-AI API key.
 - **Optional Endpoint Overrides:**
  - **Per-role `baseURL` in `.taskmasterconfig`:** You can add a `baseURL` property to any model role (`main`, `research`, `fallback`) to override the default API endpoint for that provider. If omitted, the provider's standard endpoint is used.
  - **Environment Variable Overrides (`<PROVIDER>_BASE_URL`):** For greater flexibility, especially with third-party services, you can set an environment variable like `OPENAI_BASE_URL` or `MISTRAL_BASE_URL`. This will override any `baseURL` set in the configuration file for that provider. This is the recommended way to connect to OpenAI-compatible APIs.
  - `AZURE_OPENAI_ENDPOINT`: Required if using Azure OpenAI key (can also be set as `baseURL` for the Azure model role).
  - `OLLAMA_BASE_URL`: Override the default Ollama API URL (Default: `http://localhost:11434/api`).
  - `VERTEX_PROJECT_ID`: Your Google Cloud project ID for Vertex AI. Required when using the 'vertex' provider.
@@ -132,14 +131,13 @@ PERPLEXITY_API_KEY=pplx-your-key-here
 # etc.
 # Optional Endpoint Overrides
 # Use a specific provider's base URL, e.g., for an OpenAI-compatible API
 # OPENAI_BASE_URL=https://api.third-party.com/v1
 #
 # AZURE_OPENAI_ENDPOINT=https://your-azure-endpoint.openai.azure.com/
 # OLLAMA_BASE_URL=http://custom-ollama-host:11434/api
 # Google Vertex AI Configuration (Required if using 'vertex' provider)
 # VERTEX_PROJECT_ID=your-gcp-project-id
 # VERTEX_LOCATION=us-central1
 # GOOGLE_APPLICATION_CREDENTIALS=/path/to/service-account-credentials.json
 ```
 ## Troubleshooting
--- a/docs/models.md
+++ b/docs/models.md
@@ -2,136 +2,130 @@
 ## Main Models
-| Provider    | Model Name                                     | SWE Score | Input Cost | Output Cost |
+| Provider   | Model Name                                     | SWE Score | Input Cost | Output Cost |
-| ----------- | ---------------------------------------------- | --------- | ---------- | ----------- |
+| ---------- | ---------------------------------------------- | --------- | ---------- | ----------- |
-| bedrock     | us.anthropic.claude-3-7-sonnet-20250219-v1:0   | 0.623     | 3          | 15          |
+| bedrock    | us.anthropic.claude-3-7-sonnet-20250219-v1:0   | 0.623     | 3          | 15          |
-| anthropic   | claude-sonnet-4-20250514                       | 0.727     | 3          | 15          |
+| anthropic  | claude-sonnet-4-20250514                       | 0.727     | 3          | 15          |
-| anthropic   | claude-opus-4-20250514                         | 0.725     | 15         | 75          |
+| anthropic  | claude-opus-4-20250514                         | 0.725     | 15         | 75          |
-| anthropic   | claude-3-7-sonnet-20250219                     | 0.623     | 3          | 15          |
+| anthropic  | claude-3-7-sonnet-20250219                     | 0.623     | 3          | 15          |
-| anthropic   | claude-3-5-sonnet-20241022                     | 0.49      | 3          | 15          |
+| anthropic  | claude-3-5-sonnet-20241022                     | 0.49      | 3          | 15          |
-| openai      | gpt-4o                                         | 0.332     | 2.5        | 10          |
+| openai     | gpt-4o                                         | 0.332     | 2.5        | 10          |
-| openai      | o1                                             | 0.489     | 15         | 60          |
+| openai     | o1                                             | 0.489     | 15         | 60          |
-| openai      | o3                                             | 0.5       | 2          | 8           |
+| openai     | o3                                             | 0.5       | 2          | 8           |
-| openai      | o3-mini                                        | 0.493     | 1.1        | 4.4         |
+| openai     | o3-mini                                        | 0.493     | 1.1        | 4.4         |
-| openai      | o4-mini                                        | 0.45      | 1.1        | 4.4         |
+| openai     | o4-mini                                        | 0.45      | 1.1        | 4.4         |
-| openai      | o1-mini                                        | 0.4       | 1.1        | 4.4         |
+| openai     | o1-mini                                        | 0.4       | 1.1        | 4.4         |
-| openai      | o1-pro                                         | —         | 150        | 600         |
+| openai     | o1-pro                                         | —         | 150        | 600         |
-| openai      | gpt-4-5-preview                                | 0.38      | 75         | 150         |
+| openai     | gpt-4-5-preview                                | 0.38      | 75         | 150         |
-| openai      | gpt-4-1-mini                                   | —         | 0.4        | 1.6         |
+| openai     | gpt-4-1-mini                                   | —         | 0.4        | 1.6         |
-| openai      | gpt-4-1-nano                                   | —         | 0.1        | 0.4         |
+| openai     | gpt-4-1-nano                                   | —         | 0.1        | 0.4         |
-| openai      | gpt-4o-mini                                    | 0.3       | 0.15       | 0.6         |
+| openai     | gpt-4o-mini                                    | 0.3       | 0.15       | 0.6         |
-| google      | gemini-2.5-pro-preview-05-06                   | 0.638     | —          | —           |
+| google     | gemini-2.5-pro-preview-05-06                   | 0.638     | —          | —           |
-| google      | gemini-2.5-pro-preview-03-25                   | 0.638     | —          | —           |
+| google     | gemini-2.5-pro-preview-03-25                   | 0.638     | —          | —           |
-| google      | gemini-2.5-flash-preview-04-17                 | 0.604     | —          | —           |
+| google     | gemini-2.5-flash-preview-04-17                 | 0.604     | —          | —           |
-| google      | gemini-2.0-flash                               | 0.518     | 0.15       | 0.6         |
+| google     | gemini-2.0-flash                               | 0.518     | 0.15       | 0.6         |
-| google      | gemini-2.0-flash-lite                          | —         | —          | —           |
+| google     | gemini-2.0-flash-lite                          | —         | —          | —           |
-| perplexity  | sonar-pro                                      | —         | 3          | 15          |
+| perplexity | sonar-pro                                      | —         | 3          | 15          |
-| perplexity  | sonar-reasoning-pro                            | 0.211     | 2          | 8           |
+| perplexity | sonar-reasoning-pro                            | 0.211     | 2          | 8           |
-| perplexity  | sonar-reasoning                                | 0.211     | 1          | 5           |
+| perplexity | sonar-reasoning                                | 0.211     | 1          | 5           |
-| xai         | grok-3                                         | —         | 3          | 15          |
+| xai        | grok-3                                         | —         | 3          | 15          |
-| xai         | grok-3-fast                                    | —         | 5          | 25          |
+| xai        | grok-3-fast                                    | —         | 5          | 25          |
-| ollama      | devstral:latest                                | —         | 0          | 0           |
+| ollama     | devstral:latest                                | —         | 0          | 0           |
-| ollama      | qwen3:latest                                   | —         | 0          | 0           |
+| ollama     | qwen3:latest                                   | —         | 0          | 0           |
-| ollama      | qwen3:14b                                      | —         | 0          | 0           |
+| ollama     | qwen3:14b                                      | —         | 0          | 0           |
-| ollama      | qwen3:32b                                      | —         | 0          | 0           |
+| ollama     | qwen3:32b                                      | —         | 0          | 0           |
-| ollama      | mistral-small3.1:latest                        | —         | 0          | 0           |
+| ollama     | mistral-small3.1:latest                        | —         | 0          | 0           |
-| ollama      | llama3.3:latest                                | —         | 0          | 0           |
+| ollama     | llama3.3:latest                                | —         | 0          | 0           |
-| ollama      | phi4:latest                                    | —         | 0          | 0           |
+| ollama     | phi4:latest                                    | —         | 0          | 0           |
-| openrouter  | google/gemini-2.5-flash-preview-05-20          | —         | 0.15       | 0.6         |
+| openrouter | google/gemini-2.5-flash-preview-05-20          | —         | 0.15       | 0.6         |
-| openrouter  | google/gemini-2.5-flash-preview-05-20:thinking | —         | 0.15       | 3.5         |
+| openrouter | google/gemini-2.5-flash-preview-05-20:thinking | —         | 0.15       | 3.5         |
-| openrouter  | google/gemini-2.5-pro-exp-03-25                | —         | 0          | 0           |
+| openrouter | google/gemini-2.5-pro-exp-03-25                | —         | 0          | 0           |
-| openrouter  | deepseek/deepseek-chat-v3-0324:free            | —         | 0          | 0           |
+| openrouter | deepseek/deepseek-chat-v3-0324:free            | —         | 0          | 0           |
-| openrouter  | deepseek/deepseek-chat-v3-0324                 | —         | 0.27       | 1.1         |
+| openrouter | deepseek/deepseek-chat-v3-0324                 | —         | 0.27       | 1.1         |
-| openrouter  | openai/gpt-4.1                                 | —         | 2          | 8           |
+| openrouter | openai/gpt-4.1                                 | —         | 2          | 8           |
-| openrouter  | openai/gpt-4.1-mini                            | —         | 0.4        | 1.6         |
+| openrouter | openai/gpt-4.1-mini                            | —         | 0.4        | 1.6         |
-| openrouter  | openai/gpt-4.1-nano                            | —         | 0.1        | 0.4         |
+| openrouter | openai/gpt-4.1-nano                            | —         | 0.1        | 0.4         |
-| openrouter  | openai/o3                                      | —         | 10         | 40          |
+| openrouter | openai/o3                                      | —         | 10         | 40          |
-| openrouter  | openai/codex-mini                              | —         | 1.5        | 6           |
+| openrouter | openai/codex-mini                              | —         | 1.5        | 6           |
-| openrouter  | openai/gpt-4o-mini                             | —         | 0.15       | 0.6         |
+| openrouter | openai/gpt-4o-mini                             | —         | 0.15       | 0.6         |
-| openrouter  | openai/o4-mini                                 | 0.45      | 1.1        | 4.4         |
+| openrouter | openai/o4-mini                                 | 0.45      | 1.1        | 4.4         |
-| openrouter  | openai/o4-mini-high                            | —         | 1.1        | 4.4         |
+| openrouter | openai/o4-mini-high                            | —         | 1.1        | 4.4         |
-| openrouter  | openai/o1-pro                                  | —         | 150        | 600         |
+| openrouter | openai/o1-pro                                  | —         | 150        | 600         |
-| openrouter  | meta-llama/llama-3.3-70b-instruct              | —         | 120        | 600         |
+| openrouter | meta-llama/llama-3.3-70b-instruct              | —         | 120        | 600         |
-| openrouter  | meta-llama/llama-4-maverick                    | —         | 0.18       | 0.6         |
+| openrouter | meta-llama/llama-4-maverick                    | —         | 0.18       | 0.6         |
-| openrouter  | meta-llama/llama-4-scout                       | —         | 0.08       | 0.3         |
+| openrouter | meta-llama/llama-4-scout                       | —         | 0.08       | 0.3         |
-| openrouter  | qwen/qwen-max                                  | —         | 1.6        | 6.4         |
+| openrouter | qwen/qwen-max                                  | —         | 1.6        | 6.4         |
-| openrouter  | qwen/qwen-turbo                                | —         | 0.05       | 0.2         |
+| openrouter | qwen/qwen-turbo                                | —         | 0.05       | 0.2         |
-| openrouter  | qwen/qwen3-235b-a22b                           | —         | 0.14       | 2           |
+| openrouter | qwen/qwen3-235b-a22b                           | —         | 0.14       | 2           |
-| openrouter  | mistralai/mistral-small-3.1-24b-instruct:free  | —         | 0          | 0           |
+| openrouter | mistralai/mistral-small-3.1-24b-instruct:free  | —         | 0          | 0           |
-| openrouter  | mistralai/mistral-small-3.1-24b-instruct       | —         | 0.1        | 0.3         |
+| openrouter | mistralai/mistral-small-3.1-24b-instruct       | —         | 0.1        | 0.3         |
-| openrouter  | mistralai/devstral-small                       | —         | 0.1        | 0.3         |
+| openrouter | mistralai/devstral-small                       | —         | 0.1        | 0.3         |
-| openrouter  | mistralai/mistral-nemo                         | —         | 0.03       | 0.07        |
+| openrouter | mistralai/mistral-nemo                         | —         | 0.03       | 0.07        |
-| openrouter  | thudm/glm-4-32b:free                           | —         | 0          | 0           |
+| openrouter | thudm/glm-4-32b:free                           | —         | 0          | 0           |
 | claude-code | opus                                           | 0.725     | 0          | 0           |
 | claude-code | sonnet                                         | 0.727     | 0          | 0           |
 ## Research Models
-| Provider    | Model Name                 | SWE Score | Input Cost | Output Cost |
+| Provider   | Model Name                 | SWE Score | Input Cost | Output Cost |
-| ----------- | -------------------------- | --------- | ---------- | ----------- |
+| ---------- | -------------------------- | --------- | ---------- | ----------- |
-| bedrock     | us.deepseek.r1-v1:0        | —         | 1.35       | 5.4         |
+| bedrock    | us.deepseek.r1-v1:0        | —         | 1.35       | 5.4         |
-| openai      | gpt-4o-search-preview      | 0.33      | 2.5        | 10          |
+| openai     | gpt-4o-search-preview      | 0.33      | 2.5        | 10          |
-| openai      | gpt-4o-mini-search-preview | 0.3       | 0.15       | 0.6         |
+| openai     | gpt-4o-mini-search-preview | 0.3       | 0.15       | 0.6         |
-| perplexity  | sonar-pro                  | —         | 3          | 15          |
+| perplexity | sonar-pro                  | —         | 3          | 15          |
-| perplexity  | sonar                      | —         | 1          | 1           |
+| perplexity | sonar                      | —         | 1          | 1           |
-| perplexity  | deep-research              | 0.211     | 2          | 8           |
+| perplexity | deep-research              | 0.211     | 2          | 8           |
-| perplexity  | sonar-reasoning-pro        | 0.211     | 2          | 8           |
+| perplexity | sonar-reasoning-pro        | 0.211     | 2          | 8           |
-| perplexity  | sonar-reasoning            | 0.211     | 1          | 5           |
+| perplexity | sonar-reasoning            | 0.211     | 1          | 5           |
-| xai         | grok-3                     | —         | 3          | 15          |
+| xai        | grok-3                     | —         | 3          | 15          |
-| xai         | grok-3-fast                | —         | 5          | 25          |
+| xai        | grok-3-fast                | —         | 5          | 25          |
 | claude-code | opus                       | 0.725     | 0          | 0           |
 | claude-code | sonnet                     | 0.727     | 0          | 0           |
 ## Fallback Models
-| Provider    | Model Name                                     | SWE Score | Input Cost | Output Cost |
+| Provider   | Model Name                                     | SWE Score | Input Cost | Output Cost |
-| ----------- | ---------------------------------------------- | --------- | ---------- | ----------- |
+| ---------- | ---------------------------------------------- | --------- | ---------- | ----------- |
-| bedrock     | us.anthropic.claude-3-7-sonnet-20250219-v1:0   | 0.623     | 3          | 15          |
+| bedrock    | us.anthropic.claude-3-7-sonnet-20250219-v1:0   | 0.623     | 3          | 15          |
-| anthropic   | claude-sonnet-4-20250514                       | 0.727     | 3          | 15          |
+| anthropic  | claude-sonnet-4-20250514                       | 0.727     | 3          | 15          |
-| anthropic   | claude-opus-4-20250514                         | 0.725     | 15         | 75          |
+| anthropic  | claude-opus-4-20250514                         | 0.725     | 15         | 75          |
-| anthropic   | claude-3-7-sonnet-20250219                     | 0.623     | 3          | 15          |
+| anthropic  | claude-3-7-sonnet-20250219                     | 0.623     | 3          | 15          |
-| anthropic   | claude-3-5-sonnet-20241022                     | 0.49      | 3          | 15          |
+| anthropic  | claude-3-5-sonnet-20241022                     | 0.49      | 3          | 15          |
-| openai      | gpt-4o                                         | 0.332     | 2.5        | 10          |
+| openai     | gpt-4o                                         | 0.332     | 2.5        | 10          |
-| openai      | o3                                             | 0.5       | 2          | 8           |
+| openai     | o3                                             | 0.5       | 2          | 8           |
-| openai      | o4-mini                                        | 0.45      | 1.1        | 4.4         |
+| openai     | o4-mini                                        | 0.45      | 1.1        | 4.4         |
-| google      | gemini-2.5-pro-preview-05-06                   | 0.638     | —          | —           |
+| google     | gemini-2.5-pro-preview-05-06                   | 0.638     | —          | —           |
-| google      | gemini-2.5-pro-preview-03-25                   | 0.638     | —          | —           |
+| google     | gemini-2.5-pro-preview-03-25                   | 0.638     | —          | —           |
-| google      | gemini-2.5-flash-preview-04-17                 | 0.604     | —          | —           |
+| google     | gemini-2.5-flash-preview-04-17                 | 0.604     | —          | —           |
-| google      | gemini-2.0-flash                               | 0.518     | 0.15       | 0.6         |
+| google     | gemini-2.0-flash                               | 0.518     | 0.15       | 0.6         |
-| google      | gemini-2.0-flash-lite                          | —         | —          | —           |
+| google     | gemini-2.0-flash-lite                          | —         | —          | —           |
-| perplexity  | sonar-reasoning-pro                            | 0.211     | 2          | 8           |
+| perplexity | sonar-reasoning-pro                            | 0.211     | 2          | 8           |
-| perplexity  | sonar-reasoning                                | 0.211     | 1          | 5           |
+| perplexity | sonar-reasoning                                | 0.211     | 1          | 5           |
-| xai         | grok-3                                         | —         | 3          | 15          |
+| xai        | grok-3                                         | —         | 3          | 15          |
-| xai         | grok-3-fast                                    | —         | 5          | 25          |
+| xai        | grok-3-fast                                    | —         | 5          | 25          |
-| ollama      | devstral:latest                                | —         | 0          | 0           |
+| ollama     | devstral:latest                                | —         | 0          | 0           |
-| ollama      | qwen3:latest                                   | —         | 0          | 0           |
+| ollama     | qwen3:latest                                   | —         | 0          | 0           |
-| ollama      | qwen3:14b                                      | —         | 0          | 0           |
+| ollama     | qwen3:14b                                      | —         | 0          | 0           |
-| ollama      | qwen3:32b                                      | —         | 0          | 0           |
+| ollama     | qwen3:32b                                      | —         | 0          | 0           |
-| ollama      | mistral-small3.1:latest                        | —         | 0          | 0           |
+| ollama     | mistral-small3.1:latest                        | —         | 0          | 0           |
-| ollama      | llama3.3:latest                                | —         | 0          | 0           |
+| ollama     | llama3.3:latest                                | —         | 0          | 0           |
-| ollama      | phi4:latest                                    | —         | 0          | 0           |
+| ollama     | phi4:latest                                    | —         | 0          | 0           |
-| openrouter  | google/gemini-2.5-flash-preview-05-20          | —         | 0.15       | 0.6         |
+| openrouter | google/gemini-2.5-flash-preview-05-20          | —         | 0.15       | 0.6         |
-| openrouter  | google/gemini-2.5-flash-preview-05-20:thinking | —         | 0.15       | 3.5         |
+| openrouter | google/gemini-2.5-flash-preview-05-20:thinking | —         | 0.15       | 3.5         |
-| openrouter  | google/gemini-2.5-pro-exp-03-25                | —         | 0          | 0           |
+| openrouter | google/gemini-2.5-pro-exp-03-25                | —         | 0          | 0           |
-| openrouter  | deepseek/deepseek-chat-v3-0324:free            | —         | 0          | 0           |
+| openrouter | deepseek/deepseek-chat-v3-0324:free            | —         | 0          | 0           |
-| openrouter  | openai/gpt-4.1                                 | —         | 2          | 8           |
+| openrouter | openai/gpt-4.1                                 | —         | 2          | 8           |
-| openrouter  | openai/gpt-4.1-mini                            | —         | 0.4        | 1.6         |
+| openrouter | openai/gpt-4.1-mini                            | —         | 0.4        | 1.6         |
-| openrouter  | openai/gpt-4.1-nano                            | —         | 0.1        | 0.4         |
+| openrouter | openai/gpt-4.1-nano                            | —         | 0.1        | 0.4         |
-| openrouter  | openai/o3                                      | —         | 10         | 40          |
+| openrouter | openai/o3                                      | —         | 10         | 40          |
-| openrouter  | openai/codex-mini                              | —         | 1.5        | 6           |
+| openrouter | openai/codex-mini                              | —         | 1.5        | 6           |
-| openrouter  | openai/gpt-4o-mini                             | —         | 0.15       | 0.6         |
+| openrouter | openai/gpt-4o-mini                             | —         | 0.15       | 0.6         |
-| openrouter  | openai/o4-mini                                 | 0.45      | 1.1        | 4.4         |
+| openrouter | openai/o4-mini                                 | 0.45      | 1.1        | 4.4         |
-| openrouter  | openai/o4-mini-high                            | —         | 1.1        | 4.4         |
+| openrouter | openai/o4-mini-high                            | —         | 1.1        | 4.4         |
-| openrouter  | openai/o1-pro                                  | —         | 150        | 600         |
+| openrouter | openai/o1-pro                                  | —         | 150        | 600         |
-| openrouter  | meta-llama/llama-3.3-70b-instruct              | —         | 120        | 600         |
+| openrouter | meta-llama/llama-3.3-70b-instruct              | —         | 120        | 600         |
-| openrouter  | meta-llama/llama-4-maverick                    | —         | 0.18       | 0.6         |
+| openrouter | meta-llama/llama-4-maverick                    | —         | 0.18       | 0.6         |
-| openrouter  | meta-llama/llama-4-scout                       | —         | 0.08       | 0.3         |
+| openrouter | meta-llama/llama-4-scout                       | —         | 0.08       | 0.3         |
-| openrouter  | qwen/qwen-max                                  | —         | 1.6        | 6.4         |
+| openrouter | qwen/qwen-max                                  | —         | 1.6        | 6.4         |
-| openrouter  | qwen/qwen-turbo                                | —         | 0.05       | 0.2         |
+| openrouter | qwen/qwen-turbo                                | —         | 0.05       | 0.2         |
-| openrouter  | qwen/qwen3-235b-a22b                           | —         | 0.14       | 2           |
+| openrouter | qwen/qwen3-235b-a22b                           | —         | 0.14       | 2           |
-| openrouter  | mistralai/mistral-small-3.1-24b-instruct:free  | —         | 0          | 0           |
+| openrouter | mistralai/mistral-small-3.1-24b-instruct:free  | —         | 0          | 0           |
-| openrouter  | mistralai/mistral-small-3.1-24b-instruct       | —         | 0.1        | 0.3         |
+| openrouter | mistralai/mistral-small-3.1-24b-instruct       | —         | 0.1        | 0.3         |
-| openrouter  | mistralai/mistral-nemo                         | —         | 0.03       | 0.07        |
+| openrouter | mistralai/mistral-nemo                         | —         | 0.03       | 0.07        |
-| openrouter  | thudm/glm-4-32b:free                           | —         | 0          | 0           |
+| openrouter | thudm/glm-4-32b:free                           | —         | 0          | 0           |
 | claude-code | opus                                           | 0.725     | 0          | 0           |
 | claude-code | sonnet                                         | 0.727     | 0          | 0           |
--- a/mcp-server/src/core/direct-functions/expand-task.js
+++ b/mcp-server/src/core/direct-functions/expand-task.js
@@ -26,7 +26,6 @@ import { createLogWrapper } from '../../tools/utils.js';
 * @param {string} [args.prompt] - Additional context to guide subtask generation.
 * @param {boolean} [args.force] - Force expansion even if subtasks exist.
 * @param {string} [args.projectRoot] - Project root directory.
 * @param {string} [args.tag] - Tag for the task
 * @param {Object} log - Logger object
 * @param {Object} context - Context object containing session
 * @param {Object} [context.session] - MCP Session object
@@ -35,8 +34,7 @@ import { createLogWrapper } from '../../tools/utils.js';
 export async function expandTaskDirect(args, log, context = {}) {
 	const { session } = context; // Extract session
 	// Destructure expected args, including projectRoot
-	const { tasksJsonPath, id, num, research, prompt, force, projectRoot, tag } =
+	const { tasksJsonPath, id, num, research, prompt, force, projectRoot } = args;
 		args;
 	// Log session root data for debugging
 	log.info(
@@ -196,8 +194,7 @@ export async function expandTaskDirect(args, log, context = {}) {
 					session,
 					projectRoot,
 					commandName: 'expand-task',
-					outputType: 'mcp',
+					outputType: 'mcp'
 					tag
 				},
 				forceFlag
 			);
--- a/mcp-server/src/tools/expand-task.js
+++ b/mcp-server/src/tools/expand-task.js
@@ -45,8 +45,7 @@ export function registerExpandTaskTool(server) {
 				.boolean()
 				.optional()
 				.default(false)
-				.describe('Force expansion even if subtasks exist'),
+				.describe('Force expansion even if subtasks exist')
 			tag: z.string().optional().describe('Tag context to operate on')
 		}),
 		execute: withNormalizedProjectRoot(async (args, { log, session }) => {
 			try {
@@ -74,8 +73,7 @@ export function registerExpandTaskTool(server) {
 						research: args.research,
 						prompt: args.prompt,
 						force: args.force,
-						projectRoot: args.projectRoot,
+						projectRoot: args.projectRoot
 						tag: args.tag || 'master'
 					},
 					log,
 					{ session }
--- a/scripts/modules/config-manager.js
+++ b/scripts/modules/config-manager.js
@@ -571,11 +571,10 @@ function getMcpApiKeyStatus(providerName, projectRoot = null) {
 		const mcpConfigRaw = fs.readFileSync(mcpConfigPath, 'utf-8');
 		const mcpConfig = JSON.parse(mcpConfigRaw);
-		const mcpEnv =
+		const mcpEnv = mcpConfig?.mcpServers?.['taskmaster-ai']?.env;
 			mcpConfig?.mcpServers?.['task-master-ai']?.env ||
 			mcpConfig?.mcpServers?.['taskmaster-ai']?.env;
 		if (!mcpEnv) {
-			return false;
+			// console.warn(chalk.yellow('Warning: Could not find taskmaster-ai env in mcp.json.'));
 			return false; // Structure missing
 		}
 		let apiKeyToCheck = null;
@@ -783,15 +782,9 @@ function getAllProviders() {
 function getBaseUrlForRole(role, explicitRoot = null) {
 	const roleConfig = getModelConfigForRole(role, explicitRoot);
-	if (roleConfig && typeof roleConfig.baseURL === 'string') {
+	return roleConfig && typeof roleConfig.baseURL === 'string'
-		return roleConfig.baseURL;
+		? roleConfig.baseURL
-	}
+		: undefined;
 	const provider = roleConfig?.provider;
 	if (provider) {
 		const envVarName = `${provider.toUpperCase()}_BASE_URL`;
 		return resolveEnvVariable(envVarName, null, explicitRoot);
 	}
 	return undefined;
 }
 export {
--- a/scripts/modules/task-manager/expand-all-tasks.js
+++ b/scripts/modules/task-manager/expand-all-tasks.js
@@ -32,12 +32,7 @@ async function expandAllTasks(
 	context = {},
 	outputFormat = 'text' // Assume text default for CLI
 ) {
-	const {
+	const { session, mcpLog, projectRoot: providedProjectRoot } = context;
 		session,
 		mcpLog,
 		projectRoot: providedProjectRoot,
 		tag: contextTag
 	} = context;
 	const isMCPCall = !!mcpLog; // Determine if called from MCP
 	const projectRoot = providedProjectRoot || findProjectRoot();
@@ -79,7 +74,7 @@ async function expandAllTasks(
 	try {
 		logger.info(`Reading tasks from ${tasksPath}`);
-		const data = readJSON(tasksPath, projectRoot, contextTag);
+		const data = readJSON(tasksPath, projectRoot);
 		if (!data || !data.tasks) {
 			throw new Error(`Invalid tasks data in ${tasksPath}`);
 		}
@@ -129,7 +124,7 @@ async function expandAllTasks(
 					numSubtasks,
 					useResearch,
 					additionalContext,
-					{ ...context, projectRoot, tag: data.tag || contextTag }, // Pass the whole context object with projectRoot and resolved tag
+					{ ...context, projectRoot }, // Pass the whole context object with projectRoot
 					force
 				);
 				expandedCount++;
--- a/scripts/modules/task-manager/expand-task.js
+++ b/scripts/modules/task-manager/expand-task.js
@@ -417,7 +417,7 @@ async function expandTask(
 	context = {},
 	force = false
 ) {
-	const { session, mcpLog, projectRoot: contextProjectRoot, tag } = context;
+	const { session, mcpLog, projectRoot: contextProjectRoot } = context;
 	const outputFormat = mcpLog ? 'json' : 'text';
 	// Determine projectRoot: Use from context if available, otherwise derive from tasksPath
@@ -439,7 +439,7 @@ async function expandTask(
 	try {
 		// --- Task Loading/Filtering (Unchanged) ---
 		logger.info(`Reading tasks from ${tasksPath}`);
-		const data = readJSON(tasksPath, projectRoot, tag);
+		const data = readJSON(tasksPath, projectRoot);
 		if (!data || !data.tasks)
 			throw new Error(`Invalid tasks data in ${tasksPath}`);
 		const taskIndex = data.tasks.findIndex(
@@ -668,7 +668,7 @@ async function expandTask(
 		// --- End Change: Append instead of replace ---
 		data.tasks[taskIndex] = task; // Assign the modified task back
-		writeJSON(tasksPath, data, projectRoot, tag);
+		writeJSON(tasksPath, data);
 		// await generateTaskFiles(tasksPath, path.dirname(tasksPath));
 		// Display AI Usage Summary for CLI
--- a/scripts/modules/utils.js
+++ b/scripts/modules/utils.js
@@ -73,7 +73,7 @@ function resolveEnvVariable(key, session = null, projectRoot = null) {
 */
 function findProjectRoot(
 	startDir = process.cwd(),
-	markers = ['package.json', 'pyproject.toml', '.git', LEGACY_CONFIG_FILE]
+	markers = ['package.json', '.git', LEGACY_CONFIG_FILE]
 ) {
 	let currentPath = path.resolve(startDir);
 	const rootPath = path.parse(currentPath).root;
--- a/tests/e2e/run_e2e.sh
+++ b/tests/e2e/run_e2e.sh
@@ -333,8 +333,8 @@ log_step() {
  log_step "Initializing Task Master project (non-interactive)"
  task-master init -y --name="E2E Test $TIMESTAMP" --description="Automated E2E test run"
-  if [ ! -f ".taskmaster/config.json" ]; then
+  if [ ! -f ".taskmasterconfig" ]; then
-    log_error "Initialization failed: .taskmaster/config.json not found."
+    log_error "Initialization failed: .taskmasterconfig not found."
    exit 1
  fi
  log_success "Project initialized."
@@ -344,8 +344,8 @@ log_step() {
  exit_status_prd=$?
  echo "$cmd_output_prd"
  extract_and_sum_cost "$cmd_output_prd"
-  if [ $exit_status_prd -ne 0 ] || [ ! -s ".taskmaster/tasks/tasks.json" ]; then
+  if [ $exit_status_prd -ne 0 ] || [ ! -s "tasks/tasks.json" ]; then
-    log_error "Parsing PRD failed: .taskmaster/tasks/tasks.json not found or is empty. Exit status: $exit_status_prd"
+    log_error "Parsing PRD failed: tasks/tasks.json not found or is empty. Exit status: $exit_status_prd"
    exit 1
  else
    log_success "PRD parsed successfully."
@@ -386,95 +386,6 @@ log_step() {
  task-master list --with-subtasks > task_list_after_changes.log
  log_success "Task list after changes saved to task_list_after_changes.log"
  # === Start New Test Section: Tag-Aware Expand Testing ===
  log_step "Creating additional tag for expand testing"
  task-master add-tag feature-expand --description="Tag for testing expand command with tag preservation"
  log_success "Created feature-expand tag."
  log_step "Adding task to feature-expand tag"
  task-master add-task --tag=feature-expand --prompt="Test task for tag-aware expansion" --priority=medium
  # Get the new task ID dynamically
  new_expand_task_id=$(jq -r '.["feature-expand"].tasks[-1].id' .taskmaster/tasks/tasks.json)
  log_success "Added task $new_expand_task_id to feature-expand tag."
  log_step "Verifying tags exist before expand test"
  task-master tags > tags_before_expand.log
  tag_count_before=$(jq 'keys | length' .taskmaster/tasks/tasks.json)
  log_success "Tag count before expand: $tag_count_before"
  log_step "Expanding task in feature-expand tag (testing tag corruption fix)"
  cmd_output_expand_tagged=$(task-master expand --tag=feature-expand --id="$new_expand_task_id" 2>&1)
  exit_status_expand_tagged=$?
  echo "$cmd_output_expand_tagged"
  extract_and_sum_cost "$cmd_output_expand_tagged"
  if [ $exit_status_expand_tagged -ne 0 ]; then
    log_error "Tagged expand failed. Exit status: $exit_status_expand_tagged"
  else
    log_success "Tagged expand completed."
  fi
  log_step "Verifying tag preservation after expand"
  task-master tags > tags_after_expand.log
  tag_count_after=$(jq 'keys | length' .taskmaster/tasks/tasks.json)
  if [ "$tag_count_before" -eq "$tag_count_after" ]; then
    log_success "Tag count preserved: $tag_count_after (no corruption detected)"
  else
    log_error "Tag corruption detected! Before: $tag_count_before, After: $tag_count_after"
  fi
  log_step "Verifying master tag still exists and has tasks"
  master_task_count=$(jq -r '.master.tasks | length' .taskmaster/tasks/tasks.json 2>/dev/null || echo "0")
  if [ "$master_task_count" -gt "0" ]; then
    log_success "Master tag preserved with $master_task_count tasks"
  else
    log_error "Master tag corrupted or empty after tagged expand"
  fi
  log_step "Verifying feature-expand tag has expanded subtasks"
  expanded_subtask_count=$(jq -r ".\"feature-expand\".tasks[] | select(.id == $new_expand_task_id) | .subtasks | length" .taskmaster/tasks/tasks.json 2>/dev/null || echo "0")
  if [ "$expanded_subtask_count" -gt "0" ]; then
    log_success "Expand successful: $expanded_subtask_count subtasks created in feature-expand tag"
  else
    log_error "Expand failed: No subtasks found in feature-expand tag"
  fi
  log_step "Testing force expand with tag preservation"
  cmd_output_force_expand=$(task-master expand --tag=feature-expand --id="$new_expand_task_id" --force 2>&1)
  exit_status_force_expand=$?
  echo "$cmd_output_force_expand"
  extract_and_sum_cost "$cmd_output_force_expand"
  # Verify tags still preserved after force expand
  tag_count_after_force=$(jq 'keys | length' .taskmaster/tasks/tasks.json)
  if [ "$tag_count_before" -eq "$tag_count_after_force" ]; then
    log_success "Force expand preserved all tags"
  else
    log_error "Force expand caused tag corruption"
  fi
  log_step "Testing expand --all with tag preservation"
  # Add another task to feature-expand for expand-all testing
  task-master add-task --tag=feature-expand --prompt="Second task for expand-all testing" --priority=low
  second_expand_task_id=$(jq -r '.["feature-expand"].tasks[-1].id' .taskmaster/tasks/tasks.json)
  cmd_output_expand_all=$(task-master expand --tag=feature-expand --all 2>&1)
  exit_status_expand_all=$?
  echo "$cmd_output_expand_all"
  extract_and_sum_cost "$cmd_output_expand_all"
  # Verify tags preserved after expand-all
  tag_count_after_all=$(jq 'keys | length' .taskmaster/tasks/tasks.json)
  if [ "$tag_count_before" -eq "$tag_count_after_all" ]; then
    log_success "Expand --all preserved all tags"
  else
    log_error "Expand --all caused tag corruption"
  fi
  log_success "Completed expand --all tag preservation test."
  # === End New Test Section: Tag-Aware Expand Testing ===
  # === Test Model Commands ===
  log_step "Checking initial model configuration"
  task-master models > models_initial_config.log
@@ -715,7 +626,7 @@ log_step() {
  # Find the next available task ID dynamically instead of hardcoding 11, 12
  # Assuming tasks are added sequentially and we didn't remove any core tasks yet
-  last_task_id=$(jq '[.master.tasks[].id] | max' .taskmaster/tasks/tasks.json)
+  last_task_id=$(jq '[.tasks[].id] | max' tasks/tasks.json)
  manual_task_id=$((last_task_id + 1))
  ai_task_id=$((manual_task_id + 1))
@@ -836,30 +747,30 @@ log_step() {
  task-master list --with-subtasks > task_list_after_clear_all.log
  log_success "Task list after clear-all saved. (Manual/LLM check recommended to verify subtasks removed)"
-  log_step "Expanding Task 3 again (to have subtasks for next test)"
+  log_step "Expanding Task 1 again (to have subtasks for next test)"
-  task-master expand --id=3
+  task-master expand --id=1
-  log_success "Attempted to expand Task 3."
+  log_success "Attempted to expand Task 1 again."
-  # Verify 3.1 exists 
+  # Verify 1.1 exists again
-  if ! jq -e '.master.tasks[] | select(.id == 3) | .subtasks[] | select(.id == 1)' .taskmaster/tasks/tasks.json > /dev/null; then
+  if ! jq -e '.tasks[] | select(.id == 1) | .subtasks[] | select(.id == 1)' tasks/tasks.json > /dev/null; then
-      log_error "Subtask 3.1 not found in tasks.json after expanding Task 3."
+      log_error "Subtask 1.1 not found in tasks.json after re-expanding Task 1."
      exit 1
  fi
-  log_step "Adding dependency: Task 4 depends on Subtask 3.1"
+  log_step "Adding dependency: Task 3 depends on Subtask 1.1"
-  task-master add-dependency --id=4 --depends-on=3.1
+  task-master add-dependency --id=3 --depends-on=1.1
-  log_success "Added dependency 4 -> 3.1."
+  log_success "Added dependency 3 -> 1.1."
-  log_step "Showing Task 4 details (after adding subtask dependency)"
+  log_step "Showing Task 3 details (after adding subtask dependency)"
-  task-master show 4 > task_4_details_after_dep_add.log
+  task-master show 3 > task_3_details_after_dep_add.log
-  log_success "Task 4 details saved. (Manual/LLM check recommended for dependency [3.1])"
+  log_success "Task 3 details saved. (Manual/LLM check recommended for dependency [1.1])"
-  log_step "Removing dependency: Task 4 depends on Subtask 3.1"
+  log_step "Removing dependency: Task 3 depends on Subtask 1.1"
-  task-master remove-dependency --id=4 --depends-on=3.1
+  task-master remove-dependency --id=3 --depends-on=1.1
-  log_success "Removed dependency 4 -> 3.1."
+  log_success "Removed dependency 3 -> 1.1."
-  log_step "Showing Task 4 details (after removing subtask dependency)"
+  log_step "Showing Task 3 details (after removing subtask dependency)"
-  task-master show 4 > task_4_details_after_dep_remove.log
+  task-master show 3 > task_3_details_after_dep_remove.log
-  log_success "Task 4 details saved. (Manual/LLM check recommended to verify dependency removed)"
+  log_success "Task 3 details saved. (Manual/LLM check recommended to verify dependency removed)"
  # === End New Test Section ===
--- a/tests/integration/mcp-server/direct-functions.test.js
+++ b/tests/integration/mcp-server/direct-functions.test.js
@@ -625,38 +625,19 @@ describe('MCP Server Direct Functions', () => {
 			// For successful cases, record that functions were called but don't make real calls
 			mockEnableSilentMode();
-			// Mock expandAllTasks - now returns a structured object instead of undefined
+			// Mock expandAllTasks
 			const mockExpandAll = jest.fn().mockImplementation(async () => {
-				// Return the new structured response that matches the actual implementation
+				// Just simulate success without any real operations
-				return {
+				return undefined; // expandAllTasks doesn't return anything
 					success: true,
 					expandedCount: 2,
 					failedCount: 0,
 					skippedCount: 1,
 					tasksToExpand: 3,
 					telemetryData: {
 						timestamp: new Date().toISOString(),
 						commandName: 'expand-all-tasks',
 						totalCost: 0.05,
 						totalTokens: 1000,
 						inputTokens: 600,
 						outputTokens: 400
 					}
 				};
 			});
-			// Call mock expandAllTasks with the correct signature
+			// Call mock expandAllTasks
-			const result = await mockExpandAll(
+			await mockExpandAll(
-				args.file, // tasksPath
+				args.num,
-				args.num, // numSubtasks
+				args.research || false,
-				args.research || false, // useResearch
+				args.prompt || '',
-				args.prompt || '', // additionalContext
+				args.force || false,
-				args.force || false, // force
+				{ mcpLog: mockLogger, session: options.session }
 				{
 					mcpLog: mockLogger,
 					session: options.session,
 					projectRoot: args.projectRoot
 				}
 			);
 			mockDisableSilentMode();
@@ -664,14 +645,13 @@ describe('MCP Server Direct Functions', () => {
 			return {
 				success: true,
 				data: {
-					message: `Expand all operation completed. Expanded: ${result.expandedCount}, Failed: ${result.failedCount}, Skipped: ${result.skippedCount}`,
+					message: 'Successfully expanded all pending tasks with subtasks',
 					details: {
-						expandedCount: result.expandedCount,
+						numSubtasks: args.num,
-						failedCount: result.failedCount,
+						research: args.research || false,
-						skippedCount: result.skippedCount,
+						prompt: args.prompt || '',
-						tasksToExpand: result.tasksToExpand
+						force: args.force || false
-					},
+					}
 					telemetryData: result.telemetryData
 				}
 			};
 		}
@@ -691,13 +671,10 @@ describe('MCP Server Direct Functions', () => {
 			// Assert
 			expect(result.success).toBe(true);
-			expect(result.data.message).toMatch(/Expand all operation completed/);
+			expect(result.data.message).toBe(
-			expect(result.data.details.expandedCount).toBe(2);
+				'Successfully expanded all pending tasks with subtasks'
-			expect(result.data.details.failedCount).toBe(0);
+			);
-			expect(result.data.details.skippedCount).toBe(1);
+			expect(result.data.details.numSubtasks).toBe(3);
 			expect(result.data.details.tasksToExpand).toBe(3);
 			expect(result.data.telemetryData).toBeDefined();
 			expect(result.data.telemetryData.commandName).toBe('expand-all-tasks');
 			expect(mockEnableSilentMode).toHaveBeenCalled();
 			expect(mockDisableSilentMode).toHaveBeenCalled();
 		});
@@ -718,8 +695,7 @@ describe('MCP Server Direct Functions', () => {
 			// Assert
 			expect(result.success).toBe(true);
-			expect(result.data.details.expandedCount).toBe(2);
+			expect(result.data.details.research).toBe(true);
 			expect(result.data.telemetryData).toBeDefined();
 			expect(mockEnableSilentMode).toHaveBeenCalled();
 			expect(mockDisableSilentMode).toHaveBeenCalled();
 		});
@@ -739,8 +715,7 @@ describe('MCP Server Direct Functions', () => {
 			// Assert
 			expect(result.success).toBe(true);
-			expect(result.data.details.expandedCount).toBe(2);
+			expect(result.data.details.force).toBe(true);
 			expect(result.data.telemetryData).toBeDefined();
 			expect(mockEnableSilentMode).toHaveBeenCalled();
 			expect(mockDisableSilentMode).toHaveBeenCalled();
 		});
@@ -760,77 +735,11 @@ describe('MCP Server Direct Functions', () => {
 			// Assert
 			expect(result.success).toBe(true);
-			expect(result.data.details.expandedCount).toBe(2);
+			expect(result.data.details.prompt).toBe(
-			expect(result.data.telemetryData).toBeDefined();
+				'Additional context for subtasks'
 			);
 			expect(mockEnableSilentMode).toHaveBeenCalled();
 			expect(mockDisableSilentMode).toHaveBeenCalled();
 		});
 		test('should handle case with no eligible tasks', async () => {
 			// Arrange
 			const args = {
 				projectRoot: testProjectRoot,
 				file: testTasksPath,
 				num: 3
 			};
 			// Act - Mock the scenario where no tasks are eligible for expansion
 			async function testNoEligibleTasks(args, mockLogger, options = {}) {
 				mockEnableSilentMode();
 				const mockExpandAll = jest.fn().mockImplementation(async () => {
 					return {
 						success: true,
 						expandedCount: 0,
 						failedCount: 0,
 						skippedCount: 0,
 						tasksToExpand: 0,
 						telemetryData: null,
 						message: 'No tasks eligible for expansion.'
 					};
 				});
 				const result = await mockExpandAll(
 					args.file,
 					args.num,
 					false,
 					'',
 					false,
 					{
 						mcpLog: mockLogger,
 						session: options.session,
 						projectRoot: args.projectRoot
 					},
 					'json'
 				);
 				mockDisableSilentMode();
 				return {
 					success: true,
 					data: {
 						message: result.message,
 						details: {
 							expandedCount: result.expandedCount,
 							failedCount: result.failedCount,
 							skippedCount: result.skippedCount,
 							tasksToExpand: result.tasksToExpand
 						},
 						telemetryData: result.telemetryData
 					}
 				};
 			}
 			const result = await testNoEligibleTasks(args, mockLogger, {
 				session: mockSession
 			});
 			// Assert
 			expect(result.success).toBe(true);
 			expect(result.data.message).toBe('No tasks eligible for expansion.');
 			expect(result.data.details.expandedCount).toBe(0);
 			expect(result.data.details.tasksToExpand).toBe(0);
 			expect(result.data.telemetryData).toBeNull();
 		});
 	});
 });
--- a/tests/unit/mcp/tools/expand-all.test.js
+++ b/tests/unit/mcp/tools/expand-all.test.js
@@ -1,324 +0,0 @@
 /**
 * Tests for the expand-all MCP tool
 *
 * Note: This test does NOT test the actual implementation. It tests that:
 * 1. The tool is registered correctly with the correct parameters
 * 2. Arguments are passed correctly to expandAllTasksDirect
 * 3. Error handling works as expected
 *
 * We do NOT import the real implementation - everything is mocked
 */
 import { jest } from '@jest/globals';
 // Mock EVERYTHING
 const mockExpandAllTasksDirect = jest.fn();
 jest.mock('../../../../mcp-server/src/core/task-master-core.js', () => ({
 	expandAllTasksDirect: mockExpandAllTasksDirect
 }));
 const mockHandleApiResult = jest.fn((result) => result);
 const mockGetProjectRootFromSession = jest.fn(() => '/mock/project/root');
 const mockCreateErrorResponse = jest.fn((msg) => ({
 	success: false,
 	error: { code: 'ERROR', message: msg }
 }));
 const mockWithNormalizedProjectRoot = jest.fn((fn) => fn);
 jest.mock('../../../../mcp-server/src/tools/utils.js', () => ({
 	getProjectRootFromSession: mockGetProjectRootFromSession,
 	handleApiResult: mockHandleApiResult,
 	createErrorResponse: mockCreateErrorResponse,
 	withNormalizedProjectRoot: mockWithNormalizedProjectRoot
 }));
 // Mock the z object from zod
 const mockZod = {
 	object: jest.fn(() => mockZod),
 	string: jest.fn(() => mockZod),
 	number: jest.fn(() => mockZod),
 	boolean: jest.fn(() => mockZod),
 	optional: jest.fn(() => mockZod),
 	describe: jest.fn(() => mockZod),
 	_def: {
 		shape: () => ({
 			num: {},
 			research: {},
 			prompt: {},
 			force: {},
 			tag: {},
 			projectRoot: {}
 		})
 	}
 };
 jest.mock('zod', () => ({
 	z: mockZod
 }));
 // DO NOT import the real module - create a fake implementation
 // This is the fake implementation of registerExpandAllTool
 const registerExpandAllTool = (server) => {
 	// Create simplified version of the tool config
 	const toolConfig = {
 		name: 'expand_all',
 		description: 'Use Taskmaster to expand all eligible pending tasks',
 		parameters: mockZod,
 		// Create a simplified mock of the execute function
 		execute: mockWithNormalizedProjectRoot(async (args, context) => {
 			const { log, session } = context;
 			try {
 				log.info &&
 					log.info(`Starting expand-all with args: ${JSON.stringify(args)}`);
 				// Call expandAllTasksDirect
 				const result = await mockExpandAllTasksDirect(args, log, { session });
 				// Handle result
 				return mockHandleApiResult(result, log);
 			} catch (error) {
 				log.error && log.error(`Error in expand-all tool: ${error.message}`);
 				return mockCreateErrorResponse(error.message);
 			}
 		})
 	};
 	// Register the tool with the server
 	server.addTool(toolConfig);
 };
 describe('MCP Tool: expand-all', () => {
 	// Create mock server
 	let mockServer;
 	let executeFunction;
 	// Create mock logger
 	const mockLogger = {
 		debug: jest.fn(),
 		info: jest.fn(),
 		warn: jest.fn(),
 		error: jest.fn()
 	};
 	// Test data
 	const validArgs = {
 		num: 3,
 		research: true,
 		prompt: 'additional context',
 		force: false,
 		tag: 'master',
 		projectRoot: '/test/project'
 	};
 	// Standard responses
 	const successResponse = {
 		success: true,
 		data: {
 			message:
 				'Expand all operation completed. Expanded: 2, Failed: 0, Skipped: 1',
 			details: {
 				expandedCount: 2,
 				failedCount: 0,
 				skippedCount: 1,
 				tasksToExpand: 3,
 				telemetryData: {
 					commandName: 'expand-all-tasks',
 					totalCost: 0.15,
 					totalTokens: 2500
 				}
 			}
 		}
 	};
 	const errorResponse = {
 		success: false,
 		error: {
 			code: 'EXPAND_ALL_ERROR',
 			message: 'Failed to expand tasks'
 		}
 	};
 	beforeEach(() => {
 		// Reset all mocks
 		jest.clearAllMocks();
 		// Create mock server
 		mockServer = {
 			addTool: jest.fn((config) => {
 				executeFunction = config.execute;
 			})
 		};
 		// Setup default successful response
 		mockExpandAllTasksDirect.mockResolvedValue(successResponse);
 		// Register the tool
 		registerExpandAllTool(mockServer);
 	});
 	test('should register the tool correctly', () => {
 		// Verify tool was registered
 		expect(mockServer.addTool).toHaveBeenCalledWith(
 			expect.objectContaining({
 				name: 'expand_all',
 				description: expect.stringContaining('expand all eligible pending'),
 				parameters: expect.any(Object),
 				execute: expect.any(Function)
 			})
 		);
 		// Verify the tool config was passed
 		const toolConfig = mockServer.addTool.mock.calls[0][0];
 		expect(toolConfig).toHaveProperty('parameters');
 		expect(toolConfig).toHaveProperty('execute');
 	});
 	test('should execute the tool with valid parameters', async () => {
 		// Setup context
 		const mockContext = {
 			log: mockLogger,
 			session: { workingDirectory: '/mock/dir' }
 		};
 		// Execute the function
 		const result = await executeFunction(validArgs, mockContext);
 		// Verify expandAllTasksDirect was called with correct arguments
 		expect(mockExpandAllTasksDirect).toHaveBeenCalledWith(
 			validArgs,
 			mockLogger,
 			{ session: mockContext.session }
 		);
 		// Verify handleApiResult was called
 		expect(mockHandleApiResult).toHaveBeenCalledWith(
 			successResponse,
 			mockLogger
 		);
 		expect(result).toEqual(successResponse);
 	});
 	test('should handle expand all with no eligible tasks', async () => {
 		// Arrange
 		const mockDirectResult = {
 			success: true,
 			data: {
 				message:
 					'Expand all operation completed. Expanded: 0, Failed: 0, Skipped: 0',
 				details: {
 					expandedCount: 0,
 					failedCount: 0,
 					skippedCount: 0,
 					tasksToExpand: 0,
 					telemetryData: null
 				}
 			}
 		};
 		mockExpandAllTasksDirect.mockResolvedValue(mockDirectResult);
 		mockHandleApiResult.mockReturnValue({
 			success: true,
 			data: mockDirectResult.data
 		});
 		// Act
 		const result = await executeFunction(validArgs, {
 			log: mockLogger,
 			session: { workingDirectory: '/test' }
 		});
 		// Assert
 		expect(result.success).toBe(true);
 		expect(result.data.details.expandedCount).toBe(0);
 		expect(result.data.details.tasksToExpand).toBe(0);
 	});
 	test('should handle expand all with mixed success/failure', async () => {
 		// Arrange
 		const mockDirectResult = {
 			success: true,
 			data: {
 				message:
 					'Expand all operation completed. Expanded: 2, Failed: 1, Skipped: 0',
 				details: {
 					expandedCount: 2,
 					failedCount: 1,
 					skippedCount: 0,
 					tasksToExpand: 3,
 					telemetryData: {
 						commandName: 'expand-all-tasks',
 						totalCost: 0.1,
 						totalTokens: 1500
 					}
 				}
 			}
 		};
 		mockExpandAllTasksDirect.mockResolvedValue(mockDirectResult);
 		mockHandleApiResult.mockReturnValue({
 			success: true,
 			data: mockDirectResult.data
 		});
 		// Act
 		const result = await executeFunction(validArgs, {
 			log: mockLogger,
 			session: { workingDirectory: '/test' }
 		});
 		// Assert
 		expect(result.success).toBe(true);
 		expect(result.data.details.expandedCount).toBe(2);
 		expect(result.data.details.failedCount).toBe(1);
 	});
 	test('should handle errors from expandAllTasksDirect', async () => {
 		// Arrange
 		mockExpandAllTasksDirect.mockRejectedValue(
 			new Error('Direct function error')
 		);
 		// Act
 		const result = await executeFunction(validArgs, {
 			log: mockLogger,
 			session: { workingDirectory: '/test' }
 		});
 		// Assert
 		expect(mockLogger.error).toHaveBeenCalledWith(
 			expect.stringContaining('Error in expand-all tool')
 		);
 		expect(mockCreateErrorResponse).toHaveBeenCalledWith(
 			'Direct function error'
 		);
 	});
 	test('should handle different argument combinations', async () => {
 		// Test with minimal args
 		const minimalArgs = {
 			projectRoot: '/test/project'
 		};
 		// Act
 		await executeFunction(minimalArgs, {
 			log: mockLogger,
 			session: { workingDirectory: '/test' }
 		});
 		// Assert
 		expect(mockExpandAllTasksDirect).toHaveBeenCalledWith(
 			minimalArgs,
 			mockLogger,
 			expect.any(Object)
 		);
 	});
 	test('should use withNormalizedProjectRoot wrapper correctly', () => {
 		// Verify that the execute function is wrapped with withNormalizedProjectRoot
 		expect(mockWithNormalizedProjectRoot).toHaveBeenCalledWith(
 			expect.any(Function)
 		);
 	});
 });
--- a/tests/unit/scripts/modules/task-manager/expand-all-tasks.test.js
+++ b/tests/unit/scripts/modules/task-manager/expand-all-tasks.test.js
@@ -1,502 +0,0 @@
 /**
 * Tests for the expand-all-tasks.js module
 */
 import { jest } from '@jest/globals';
 // Mock the dependencies before importing the module under test
 jest.unstable_mockModule(
 	'../../../../../scripts/modules/task-manager/expand-task.js',
 	() => ({
 		default: jest.fn()
 	})
 );
 jest.unstable_mockModule('../../../../../scripts/modules/utils.js', () => ({
 	readJSON: jest.fn(),
 	log: jest.fn(),
 	isSilentMode: jest.fn(() => false),
 	findProjectRoot: jest.fn(() => '/test/project'),
 	aggregateTelemetry: jest.fn()
 }));
 jest.unstable_mockModule(
 	'../../../../../scripts/modules/config-manager.js',
 	() => ({
 		getDebugFlag: jest.fn(() => false)
 	})
 );
 jest.unstable_mockModule('../../../../../scripts/modules/ui.js', () => ({
 	startLoadingIndicator: jest.fn(),
 	stopLoadingIndicator: jest.fn(),
 	displayAiUsageSummary: jest.fn()
 }));
 jest.unstable_mockModule('chalk', () => ({
 	default: {
 		white: { bold: jest.fn((text) => text) },
 		cyan: jest.fn((text) => text),
 		green: jest.fn((text) => text),
 		gray: jest.fn((text) => text),
 		red: jest.fn((text) => text),
 		bold: jest.fn((text) => text)
 	}
 }));
 jest.unstable_mockModule('boxen', () => ({
 	default: jest.fn((text) => text)
 }));
 // Import the mocked modules
 const { default: expandTask } = await import(
 	'../../../../../scripts/modules/task-manager/expand-task.js'
 );
 const { readJSON, aggregateTelemetry, findProjectRoot } = await import(
 	'../../../../../scripts/modules/utils.js'
 );
 // Import the module under test
 const { default: expandAllTasks } = await import(
 	'../../../../../scripts/modules/task-manager/expand-all-tasks.js'
 );
 const mockExpandTask = expandTask;
 const mockReadJSON = readJSON;
 const mockAggregateTelemetry = aggregateTelemetry;
 const mockFindProjectRoot = findProjectRoot;
 describe('expandAllTasks', () => {
 	const mockTasksPath = '/test/tasks.json';
 	const mockProjectRoot = '/test/project';
 	const mockSession = { userId: 'test-user' };
 	const mockMcpLog = {
 		info: jest.fn(),
 		warn: jest.fn(),
 		error: jest.fn(),
 		debug: jest.fn()
 	};
 	const sampleTasksData = {
 		tag: 'master',
 		tasks: [
 			{
 				id: 1,
 				title: 'Pending Task 1',
 				status: 'pending',
 				subtasks: []
 			},
 			{
 				id: 2,
 				title: 'In Progress Task',
 				status: 'in-progress',
 				subtasks: []
 			},
 			{
 				id: 3,
 				title: 'Done Task',
 				status: 'done',
 				subtasks: []
 			},
 			{
 				id: 4,
 				title: 'Task with Subtasks',
 				status: 'pending',
 				subtasks: [{ id: '4.1', title: 'Existing subtask' }]
 			}
 		]
 	};
 	beforeEach(() => {
 		jest.clearAllMocks();
 		mockReadJSON.mockReturnValue(sampleTasksData);
 		mockAggregateTelemetry.mockReturnValue({
 			timestamp: '2024-01-01T00:00:00.000Z',
 			commandName: 'expand-all-tasks',
 			totalCost: 0.1,
 			totalTokens: 2000,
 			inputTokens: 1200,
 			outputTokens: 800
 		});
 	});
 	describe('successful expansion', () => {
 		test('should expand all eligible pending tasks', async () => {
 			// Arrange
 			const mockTelemetryData = {
 				timestamp: '2024-01-01T00:00:00.000Z',
 				commandName: 'expand-task',
 				totalCost: 0.05,
 				totalTokens: 1000
 			};
 			mockExpandTask.mockResolvedValue({
 				telemetryData: mockTelemetryData
 			});
 			// Act
 			const result = await expandAllTasks(
 				mockTasksPath,
 				3, // numSubtasks
 				false, // useResearch
 				'test context', // additionalContext
 				false, // force
 				{
 					session: mockSession,
 					mcpLog: mockMcpLog,
 					projectRoot: mockProjectRoot,
 					tag: 'master'
 				},
 				'json' // outputFormat
 			);
 			// Assert
 			expect(result.success).toBe(true);
 			expect(result.expandedCount).toBe(2); // Tasks 1 and 2 (pending and in-progress)
 			expect(result.failedCount).toBe(0);
 			expect(result.skippedCount).toBe(0);
 			expect(result.tasksToExpand).toBe(2);
 			expect(result.telemetryData).toBeDefined();
 			// Verify readJSON was called correctly
 			expect(mockReadJSON).toHaveBeenCalledWith(
 				mockTasksPath,
 				mockProjectRoot,
 				'master'
 			);
 			// Verify expandTask was called for eligible tasks
 			expect(mockExpandTask).toHaveBeenCalledTimes(2);
 			expect(mockExpandTask).toHaveBeenCalledWith(
 				mockTasksPath,
 				1,
 				3,
 				false,
 				'test context',
 				expect.objectContaining({
 					session: mockSession,
 					mcpLog: mockMcpLog,
 					projectRoot: mockProjectRoot,
 					tag: 'master'
 				}),
 				false
 			);
 		});
 		test('should handle force flag to expand tasks with existing subtasks', async () => {
 			// Arrange
 			mockExpandTask.mockResolvedValue({
 				telemetryData: { commandName: 'expand-task', totalCost: 0.05 }
 			});
 			// Act
 			const result = await expandAllTasks(
 				mockTasksPath,
 				2,
 				false,
 				'',
 				true, // force = true
 				{
 					session: mockSession,
 					mcpLog: mockMcpLog,
 					projectRoot: mockProjectRoot
 				},
 				'json'
 			);
 			// Assert
 			expect(result.expandedCount).toBe(3); // Tasks 1, 2, and 4 (including task with existing subtasks)
 			expect(mockExpandTask).toHaveBeenCalledTimes(3);
 		});
 		test('should handle research flag', async () => {
 			// Arrange
 			mockExpandTask.mockResolvedValue({
 				telemetryData: { commandName: 'expand-task', totalCost: 0.08 }
 			});
 			// Act
 			const result = await expandAllTasks(
 				mockTasksPath,
 				undefined, // numSubtasks not specified
 				true, // useResearch = true
 				'research context',
 				false,
 				{
 					session: mockSession,
 					mcpLog: mockMcpLog,
 					projectRoot: mockProjectRoot
 				},
 				'json'
 			);
 			// Assert
 			expect(result.success).toBe(true);
 			expect(mockExpandTask).toHaveBeenCalledWith(
 				mockTasksPath,
 				expect.any(Number),
 				undefined,
 				true, // research flag passed correctly
 				'research context',
 				expect.any(Object),
 				false
 			);
 		});
 		test('should return success with message when no tasks are eligible', async () => {
 			// Arrange - Mock tasks data with no eligible tasks
 			const noEligibleTasksData = {
 				tag: 'master',
 				tasks: [
 					{ id: 1, status: 'done', subtasks: [] },
 					{
 						id: 2,
 						status: 'pending',
 						subtasks: [{ id: '2.1', title: 'existing' }]
 					}
 				]
 			};
 			mockReadJSON.mockReturnValue(noEligibleTasksData);
 			// Act
 			const result = await expandAllTasks(
 				mockTasksPath,
 				3,
 				false,
 				'',
 				false, // force = false, so task with subtasks won't be expanded
 				{
 					session: mockSession,
 					mcpLog: mockMcpLog,
 					projectRoot: mockProjectRoot
 				},
 				'json'
 			);
 			// Assert
 			expect(result.success).toBe(true);
 			expect(result.expandedCount).toBe(0);
 			expect(result.failedCount).toBe(0);
 			expect(result.skippedCount).toBe(0);
 			expect(result.tasksToExpand).toBe(0);
 			expect(result.message).toBe('No tasks eligible for expansion.');
 			expect(mockExpandTask).not.toHaveBeenCalled();
 		});
 	});
 	describe('error handling', () => {
 		test('should handle expandTask failures gracefully', async () => {
 			// Arrange
 			mockExpandTask
 				.mockResolvedValueOnce({ telemetryData: { totalCost: 0.05 } }) // First task succeeds
 				.mockRejectedValueOnce(new Error('AI service error')); // Second task fails
 			// Act
 			const result = await expandAllTasks(
 				mockTasksPath,
 				3,
 				false,
 				'',
 				false,
 				{
 					session: mockSession,
 					mcpLog: mockMcpLog,
 					projectRoot: mockProjectRoot
 				},
 				'json'
 			);
 			// Assert
 			expect(result.success).toBe(true);
 			expect(result.expandedCount).toBe(1);
 			expect(result.failedCount).toBe(1);
 		});
 		test('should throw error when tasks.json is invalid', async () => {
 			// Arrange
 			mockReadJSON.mockReturnValue(null);
 			// Act & Assert
 			await expect(
 				expandAllTasks(
 					mockTasksPath,
 					3,
 					false,
 					'',
 					false,
 					{
 						session: mockSession,
 						mcpLog: mockMcpLog,
 						projectRoot: mockProjectRoot
 					},
 					'json'
 				)
 			).rejects.toThrow('Invalid tasks data');
 		});
 		test('should throw error when project root cannot be determined', async () => {
 			// Arrange - Mock findProjectRoot to return null for this test
 			mockFindProjectRoot.mockReturnValueOnce(null);
 			// Act & Assert
 			await expect(
 				expandAllTasks(
 					mockTasksPath,
 					3,
 					false,
 					'',
 					false,
 					{
 						session: mockSession,
 						mcpLog: mockMcpLog
 						// No projectRoot provided, and findProjectRoot will return null
 					},
 					'json'
 				)
 			).rejects.toThrow('Could not determine project root directory');
 		});
 	});
 	describe('telemetry aggregation', () => {
 		test('should aggregate telemetry data from multiple expand operations', async () => {
 			// Arrange
 			const telemetryData1 = {
 				commandName: 'expand-task',
 				totalCost: 0.03,
 				totalTokens: 600
 			};
 			const telemetryData2 = {
 				commandName: 'expand-task',
 				totalCost: 0.04,
 				totalTokens: 800
 			};
 			mockExpandTask
 				.mockResolvedValueOnce({ telemetryData: telemetryData1 })
 				.mockResolvedValueOnce({ telemetryData: telemetryData2 });
 			// Act
 			const result = await expandAllTasks(
 				mockTasksPath,
 				3,
 				false,
 				'',
 				false,
 				{
 					session: mockSession,
 					mcpLog: mockMcpLog,
 					projectRoot: mockProjectRoot
 				},
 				'json'
 			);
 			// Assert
 			expect(mockAggregateTelemetry).toHaveBeenCalledWith(
 				[telemetryData1, telemetryData2],
 				'expand-all-tasks'
 			);
 			expect(result.telemetryData).toBeDefined();
 			expect(result.telemetryData.commandName).toBe('expand-all-tasks');
 		});
 		test('should handle missing telemetry data gracefully', async () => {
 			// Arrange
 			mockExpandTask.mockResolvedValue({}); // No telemetryData
 			// Act
 			const result = await expandAllTasks(
 				mockTasksPath,
 				3,
 				false,
 				'',
 				false,
 				{
 					session: mockSession,
 					mcpLog: mockMcpLog,
 					projectRoot: mockProjectRoot
 				},
 				'json'
 			);
 			// Assert
 			expect(result.success).toBe(true);
 			expect(mockAggregateTelemetry).toHaveBeenCalledWith(
 				[],
 				'expand-all-tasks'
 			);
 		});
 	});
 	describe('output format handling', () => {
 		test('should use text output format for CLI calls', async () => {
 			// Arrange
 			mockExpandTask.mockResolvedValue({
 				telemetryData: { commandName: 'expand-task', totalCost: 0.05 }
 			});
 			// Act
 			const result = await expandAllTasks(
 				mockTasksPath,
 				3,
 				false,
 				'',
 				false,
 				{
 					projectRoot: mockProjectRoot
 					// No mcpLog provided, should use CLI logger
 				},
 				'text' // CLI output format
 			);
 			// Assert
 			expect(result.success).toBe(true);
 			// In text mode, loading indicators and console output would be used
 			// This is harder to test directly but we can verify the result structure
 		});
 		test('should handle context tag properly', async () => {
 			// Arrange
 			const taggedTasksData = {
 				...sampleTasksData,
 				tag: 'feature-branch'
 			};
 			mockReadJSON.mockReturnValue(taggedTasksData);
 			mockExpandTask.mockResolvedValue({
 				telemetryData: { commandName: 'expand-task', totalCost: 0.05 }
 			});
 			// Act
 			const result = await expandAllTasks(
 				mockTasksPath,
 				3,
 				false,
 				'',
 				false,
 				{
 					session: mockSession,
 					mcpLog: mockMcpLog,
 					projectRoot: mockProjectRoot,
 					tag: 'feature-branch'
 				},
 				'json'
 			);
 			// Assert
 			expect(mockReadJSON).toHaveBeenCalledWith(
 				mockTasksPath,
 				mockProjectRoot,
 				'feature-branch'
 			);
 			expect(mockExpandTask).toHaveBeenCalledWith(
 				mockTasksPath,
 				expect.any(Number),
 				3,
 				false,
 				'',
 				expect.objectContaining({
 					tag: 'feature-branch'
 				}),
 				false
 			);
 		});
 	});
 });
--- a/tests/unit/scripts/modules/task-manager/expand-task.test.js
+++ b/tests/unit/scripts/modules/task-manager/expand-task.test.js
@@ -1,888 +0,0 @@
 /**
 * Tests for the expand-task.js module
 */
 import { jest } from '@jest/globals';
 import fs from 'fs';
 // Mock the dependencies before importing the module under test
 jest.unstable_mockModule('../../../../../scripts/modules/utils.js', () => ({
 	readJSON: jest.fn(),
 	writeJSON: jest.fn(),
 	log: jest.fn(),
 	CONFIG: {
 		model: 'mock-claude-model',
 		maxTokens: 4000,
 		temperature: 0.7,
 		debug: false
 	},
 	sanitizePrompt: jest.fn((prompt) => prompt),
 	truncate: jest.fn((text) => text),
 	isSilentMode: jest.fn(() => false),
 	findTaskById: jest.fn(),
 	findProjectRoot: jest.fn((tasksPath) => '/mock/project/root'),
 	getCurrentTag: jest.fn(() => 'master'),
 	ensureTagMetadata: jest.fn((tagObj) => tagObj),
 	flattenTasksWithSubtasks: jest.fn((tasks) => {
 		const allTasks = [];
 		const queue = [...(tasks || [])];
 		while (queue.length > 0) {
 			const task = queue.shift();
 			allTasks.push(task);
 			if (task.subtasks) {
 				for (const subtask of task.subtasks) {
 					queue.push({ ...subtask, id: `${task.id}.${subtask.id}` });
 				}
 			}
 		}
 		return allTasks;
 	}),
 	readComplexityReport: jest.fn(),
 	markMigrationForNotice: jest.fn(),
 	performCompleteTagMigration: jest.fn(),
 	setTasksForTag: jest.fn(),
 	getTasksForTag: jest.fn((data, tag) => data[tag]?.tasks || [])
 }));
 jest.unstable_mockModule('../../../../../scripts/modules/ui.js', () => ({
 	displayBanner: jest.fn(),
 	getStatusWithColor: jest.fn((status) => status),
 	startLoadingIndicator: jest.fn(),
 	stopLoadingIndicator: jest.fn(),
 	succeedLoadingIndicator: jest.fn(),
 	failLoadingIndicator: jest.fn(),
 	warnLoadingIndicator: jest.fn(),
 	infoLoadingIndicator: jest.fn(),
 	displayAiUsageSummary: jest.fn(),
 	displayContextAnalysis: jest.fn()
 }));
 jest.unstable_mockModule(
 	'../../../../../scripts/modules/ai-services-unified.js',
 	() => ({
 		generateTextService: jest.fn().mockResolvedValue({
 			mainResult: JSON.stringify({
 				subtasks: [
 					{
 						id: 1,
 						title: 'Set up project structure',
 						description:
 							'Create the basic project directory structure and configuration files',
 						dependencies: [],
 						details:
 							'Initialize package.json, create src/ and test/ directories, set up linting configuration',
 						status: 'pending',
 						testStrategy:
 							'Verify all expected files and directories are created'
 					},
 					{
 						id: 2,
 						title: 'Implement core functionality',
 						description: 'Develop the main application logic and core features',
 						dependencies: [1],
 						details:
 							'Create main classes, implement business logic, set up data models',
 						status: 'pending',
 						testStrategy: 'Unit tests for all core functions and classes'
 					},
 					{
 						id: 3,
 						title: 'Add user interface',
 						description: 'Create the user interface components and layouts',
 						dependencies: [2],
 						details:
 							'Design UI components, implement responsive layouts, add user interactions',
 						status: 'pending',
 						testStrategy: 'UI tests and visual regression testing'
 					}
 				]
 			}),
 			telemetryData: {
 				timestamp: new Date().toISOString(),
 				userId: '1234567890',
 				commandName: 'expand-task',
 				modelUsed: 'claude-3-5-sonnet',
 				providerName: 'anthropic',
 				inputTokens: 1000,
 				outputTokens: 500,
 				totalTokens: 1500,
 				totalCost: 0.012414,
 				currency: 'USD'
 			}
 		})
 	})
 );
 jest.unstable_mockModule(
 	'../../../../../scripts/modules/config-manager.js',
 	() => ({
 		getDefaultSubtasks: jest.fn(() => 3),
 		getDebugFlag: jest.fn(() => false)
 	})
 );
 jest.unstable_mockModule(
 	'../../../../../scripts/modules/utils/contextGatherer.js',
 	() => ({
 		ContextGatherer: jest.fn().mockImplementation(() => ({
 			gather: jest.fn().mockResolvedValue({
 				contextSummary: 'Mock context summary',
 				allRelatedTaskIds: [],
 				graphVisualization: 'Mock graph'
 			})
 		}))
 	})
 );
 jest.unstable_mockModule(
 	'../../../../../scripts/modules/task-manager/generate-task-files.js',
 	() => ({
 		default: jest.fn().mockResolvedValue()
 	})
 );
 // Mock external UI libraries
 jest.unstable_mockModule('chalk', () => ({
 	default: {
 		white: { bold: jest.fn((text) => text) },
 		cyan: Object.assign(
 			jest.fn((text) => text),
 			{
 				bold: jest.fn((text) => text)
 			}
 		),
 		green: jest.fn((text) => text),
 		yellow: jest.fn((text) => text),
 		bold: jest.fn((text) => text)
 	}
 }));
 jest.unstable_mockModule('boxen', () => ({
 	default: jest.fn((text) => text)
 }));
 jest.unstable_mockModule('cli-table3', () => ({
 	default: jest.fn().mockImplementation(() => ({
 		push: jest.fn(),
 		toString: jest.fn(() => 'mocked table')
 	}))
 }));
 // Mock process.exit to prevent Jest worker crashes
 const mockExit = jest.spyOn(process, 'exit').mockImplementation((code) => {
 	throw new Error(`process.exit called with "${code}"`);
 });
 // Import the mocked modules
 const {
 	readJSON,
 	writeJSON,
 	log,
 	findTaskById,
 	ensureTagMetadata,
 	readComplexityReport,
 	findProjectRoot
 } = await import('../../../../../scripts/modules/utils.js');
 const { generateTextService } = await import(
 	'../../../../../scripts/modules/ai-services-unified.js'
 );
 const generateTaskFiles = (
 	await import(
 		'../../../../../scripts/modules/task-manager/generate-task-files.js'
 	)
 ).default;
 // Import the module under test
 const { default: expandTask } = await import(
 	'../../../../../scripts/modules/task-manager/expand-task.js'
 );
 describe('expandTask', () => {
 	const sampleTasks = {
 		master: {
 			tasks: [
 				{
 					id: 1,
 					title: 'Task 1',
 					description: 'First task',
 					status: 'done',
 					dependencies: [],
 					details: 'Already completed task',
 					subtasks: []
 				},
 				{
 					id: 2,
 					title: 'Task 2',
 					description: 'Second task',
 					status: 'pending',
 					dependencies: [],
 					details: 'Task ready for expansion',
 					subtasks: []
 				},
 				{
 					id: 3,
 					title: 'Complex Task',
 					description: 'A complex task that needs breakdown',
 					status: 'pending',
 					dependencies: [1],
 					details: 'This task involves multiple steps',
 					subtasks: []
 				},
 				{
 					id: 4,
 					title: 'Task with existing subtasks',
 					description: 'Task that already has subtasks',
 					status: 'pending',
 					dependencies: [],
 					details: 'Has existing subtasks',
 					subtasks: [
 						{
 							id: 1,
 							title: 'Existing subtask',
 							description: 'Already exists',
 							status: 'pending',
 							dependencies: []
 						}
 					]
 				}
 			]
 		},
 		'feature-branch': {
 			tasks: [
 				{
 					id: 1,
 					title: 'Feature Task 1',
 					description: 'Task in feature branch',
 					status: 'pending',
 					dependencies: [],
 					details: 'Feature-specific task',
 					subtasks: []
 				}
 			]
 		}
 	};
 	// Create a helper function for consistent mcpLog mock
 	const createMcpLogMock = () => ({
 		info: jest.fn(),
 		warn: jest.fn(),
 		error: jest.fn(),
 		debug: jest.fn(),
 		success: jest.fn()
 	});
 	beforeEach(() => {
 		jest.clearAllMocks();
 		mockExit.mockClear();
 		// Default readJSON implementation - returns tagged structure
 		readJSON.mockImplementation((tasksPath, projectRoot, tag) => {
 			const sampleTasksCopy = JSON.parse(JSON.stringify(sampleTasks));
 			const selectedTag = tag || 'master';
 			return {
 				...sampleTasksCopy[selectedTag],
 				tag: selectedTag,
 				_rawTaggedData: sampleTasksCopy
 			};
 		});
 		// Default findTaskById implementation
 		findTaskById.mockImplementation((tasks, taskId) => {
 			const id = parseInt(taskId, 10);
 			return tasks.find((t) => t.id === id);
 		});
 		// Default complexity report (no report available)
 		readComplexityReport.mockReturnValue(null);
 		// Mock findProjectRoot to return consistent path for complexity report
 		findProjectRoot.mockReturnValue('/mock/project/root');
 		writeJSON.mockResolvedValue();
 		generateTaskFiles.mockResolvedValue();
 		log.mockImplementation(() => {});
 		// Mock console.log to avoid output during tests
 		jest.spyOn(console, 'log').mockImplementation(() => {});
 	});
 	afterEach(() => {
 		console.log.mockRestore();
 	});
 	describe('Basic Functionality', () => {
 		test('should expand a task with AI-generated subtasks', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const numSubtasks = 3;
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			// Act
 			const result = await expandTask(
 				tasksPath,
 				taskId,
 				numSubtasks,
 				false,
 				'',
 				context,
 				false
 			);
 			// Assert
 			expect(readJSON).toHaveBeenCalledWith(
 				tasksPath,
 				'/mock/project/root',
 				undefined
 			);
 			expect(generateTextService).toHaveBeenCalledWith(expect.any(Object));
 			expect(writeJSON).toHaveBeenCalledWith(
 				tasksPath,
 				expect.objectContaining({
 					tasks: expect.arrayContaining([
 						expect.objectContaining({
 							id: 2,
 							subtasks: expect.arrayContaining([
 								expect.objectContaining({
 									id: 1,
 									title: 'Set up project structure',
 									status: 'pending'
 								}),
 								expect.objectContaining({
 									id: 2,
 									title: 'Implement core functionality',
 									status: 'pending'
 								}),
 								expect.objectContaining({
 									id: 3,
 									title: 'Add user interface',
 									status: 'pending'
 								})
 							])
 						})
 					]),
 					tag: 'master',
 					_rawTaggedData: expect.objectContaining({
 						master: expect.objectContaining({
 							tasks: expect.any(Array)
 						})
 					})
 				}),
 				'/mock/project/root',
 				undefined
 			);
 			expect(result).toEqual(
 				expect.objectContaining({
 					task: expect.objectContaining({
 						id: 2,
 						subtasks: expect.arrayContaining([
 							expect.objectContaining({
 								id: 1,
 								title: 'Set up project structure',
 								status: 'pending'
 							}),
 							expect.objectContaining({
 								id: 2,
 								title: 'Implement core functionality',
 								status: 'pending'
 							}),
 							expect.objectContaining({
 								id: 3,
 								title: 'Add user interface',
 								status: 'pending'
 							})
 						])
 					}),
 					telemetryData: expect.any(Object)
 				})
 			);
 		});
 		test('should handle research flag correctly', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const numSubtasks = 3;
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			// Act
 			await expandTask(
 				tasksPath,
 				taskId,
 				numSubtasks,
 				true, // useResearch = true
 				'Additional context for research',
 				context,
 				false
 			);
 			// Assert
 			expect(generateTextService).toHaveBeenCalledWith(
 				expect.objectContaining({
 					role: 'research',
 					commandName: expect.any(String)
 				})
 			);
 		});
 		test('should handle complexity report integration without errors', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			// Act & Assert - Should complete without errors
 			const result = await expandTask(
 				tasksPath,
 				taskId,
 				undefined, // numSubtasks not specified
 				false,
 				'',
 				context,
 				false
 			);
 			// Assert - Should successfully expand and return expected structure
 			expect(result).toEqual(
 				expect.objectContaining({
 					task: expect.objectContaining({
 						id: 2,
 						subtasks: expect.any(Array)
 					}),
 					telemetryData: expect.any(Object)
 				})
 			);
 			expect(generateTextService).toHaveBeenCalled();
 		});
 	});
 	describe('Tag Handling (The Critical Bug Fix)', () => {
 		test('should preserve tagged structure when expanding with default tag', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root',
 				tag: 'master' // Explicit tag context
 			};
 			// Act
 			await expandTask(tasksPath, taskId, 3, false, '', context, false);
 			// Assert - CRITICAL: Check tag is passed to readJSON and writeJSON
 			expect(readJSON).toHaveBeenCalledWith(
 				tasksPath,
 				'/mock/project/root',
 				'master'
 			);
 			expect(writeJSON).toHaveBeenCalledWith(
 				tasksPath,
 				expect.objectContaining({
 					tag: 'master',
 					_rawTaggedData: expect.objectContaining({
 						master: expect.any(Object),
 						'feature-branch': expect.any(Object)
 					})
 				}),
 				'/mock/project/root',
 				'master' // CRITICAL: Tag must be passed to writeJSON
 			);
 		});
 		test('should preserve tagged structure when expanding with non-default tag', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '1'; // Task in feature-branch
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root',
 				tag: 'feature-branch' // Different tag context
 			};
 			// Configure readJSON to return feature-branch data
 			readJSON.mockImplementation((tasksPath, projectRoot, tag) => {
 				const sampleTasksCopy = JSON.parse(JSON.stringify(sampleTasks));
 				return {
 					...sampleTasksCopy['feature-branch'],
 					tag: 'feature-branch',
 					_rawTaggedData: sampleTasksCopy
 				};
 			});
 			// Act
 			await expandTask(tasksPath, taskId, 3, false, '', context, false);
 			// Assert - CRITICAL: Check tag preservation for non-default tag
 			expect(readJSON).toHaveBeenCalledWith(
 				tasksPath,
 				'/mock/project/root',
 				'feature-branch'
 			);
 			expect(writeJSON).toHaveBeenCalledWith(
 				tasksPath,
 				expect.objectContaining({
 					tag: 'feature-branch',
 					_rawTaggedData: expect.objectContaining({
 						master: expect.any(Object),
 						'feature-branch': expect.any(Object)
 					})
 				}),
 				'/mock/project/root',
 				'feature-branch' // CRITICAL: Correct tag passed to writeJSON
 			);
 		});
 		test('should NOT corrupt tagged structure when tag is undefined', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 				// No tag specified - should default gracefully
 			};
 			// Act
 			await expandTask(tasksPath, taskId, 3, false, '', context, false);
 			// Assert - Should still preserve structure with undefined tag
 			expect(readJSON).toHaveBeenCalledWith(
 				tasksPath,
 				'/mock/project/root',
 				undefined
 			);
 			expect(writeJSON).toHaveBeenCalledWith(
 				tasksPath,
 				expect.objectContaining({
 					_rawTaggedData: expect.objectContaining({
 						master: expect.any(Object)
 					})
 				}),
 				'/mock/project/root',
 				undefined
 			);
 			// CRITICAL: Verify structure is NOT flattened to old format
 			const writeCallArgs = writeJSON.mock.calls[0][1];
 			expect(writeCallArgs).toHaveProperty('tasks'); // Should have tasks property from readJSON mock
 			expect(writeCallArgs).toHaveProperty('_rawTaggedData'); // Should preserve tagged structure
 		});
 	});
 	describe('Force Flag Handling', () => {
 		test('should replace existing subtasks when force=true', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '4'; // Task with existing subtasks
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			// Act
 			await expandTask(tasksPath, taskId, 3, false, '', context, true);
 			// Assert - Should replace existing subtasks
 			expect(writeJSON).toHaveBeenCalledWith(
 				tasksPath,
 				expect.objectContaining({
 					tasks: expect.arrayContaining([
 						expect.objectContaining({
 							id: 4,
 							subtasks: expect.arrayContaining([
 								expect.objectContaining({
 									id: 1,
 									title: 'Set up project structure'
 								})
 							])
 						})
 					])
 				}),
 				'/mock/project/root',
 				undefined
 			);
 		});
 		test('should append to existing subtasks when force=false', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '4'; // Task with existing subtasks
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			// Act
 			await expandTask(tasksPath, taskId, 3, false, '', context, false);
 			// Assert - Should append to existing subtasks with proper ID increments
 			expect(writeJSON).toHaveBeenCalledWith(
 				tasksPath,
 				expect.objectContaining({
 					tasks: expect.arrayContaining([
 						expect.objectContaining({
 							id: 4,
 							subtasks: expect.arrayContaining([
 								// Should contain both existing and new subtasks
 								expect.any(Object),
 								expect.any(Object),
 								expect.any(Object),
 								expect.any(Object) // 1 existing + 3 new = 4 total
 							])
 						})
 					])
 				}),
 				'/mock/project/root',
 				undefined
 			);
 		});
 	});
 	describe('Error Handling', () => {
 		test('should handle non-existent task ID', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '999'; // Non-existent task
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			findTaskById.mockReturnValue(null);
 			// Act & Assert
 			await expect(
 				expandTask(tasksPath, taskId, 3, false, '', context, false)
 			).rejects.toThrow('Task 999 not found');
 			expect(writeJSON).not.toHaveBeenCalled();
 		});
 		test('should expand tasks regardless of status (including done tasks)', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '1'; // Task with 'done' status
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			// Act
 			const result = await expandTask(
 				tasksPath,
 				taskId,
 				3,
 				false,
 				'',
 				context,
 				false
 			);
 			// Assert - Should successfully expand even 'done' tasks
 			expect(writeJSON).toHaveBeenCalled();
 			expect(result).toEqual(
 				expect.objectContaining({
 					task: expect.objectContaining({
 						id: 1,
 						status: 'done', // Status unchanged
 						subtasks: expect.arrayContaining([
 							expect.objectContaining({
 								id: 1,
 								title: 'Set up project structure',
 								status: 'pending'
 							})
 						])
 					}),
 					telemetryData: expect.any(Object)
 				})
 			);
 		});
 		test('should handle AI service failures', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			generateTextService.mockRejectedValueOnce(new Error('AI service error'));
 			// Act & Assert
 			await expect(
 				expandTask(tasksPath, taskId, 3, false, '', context, false)
 			).rejects.toThrow('AI service error');
 			expect(writeJSON).not.toHaveBeenCalled();
 		});
 		test('should handle file read errors', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			readJSON.mockImplementation(() => {
 				throw new Error('File read failed');
 			});
 			// Act & Assert
 			await expect(
 				expandTask(tasksPath, taskId, 3, false, '', context, false)
 			).rejects.toThrow('File read failed');
 			expect(writeJSON).not.toHaveBeenCalled();
 		});
 		test('should handle invalid tasks data', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			readJSON.mockReturnValue(null);
 			// Act & Assert
 			await expect(
 				expandTask(tasksPath, taskId, 3, false, '', context, false)
 			).rejects.toThrow();
 		});
 	});
 	describe('Output Format Handling', () => {
 		test('should display telemetry for CLI output format', async () => {
 			// Arrange
 			const { displayAiUsageSummary } = await import(
 				'../../../../../scripts/modules/ui.js'
 			);
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const context = {
 				projectRoot: '/mock/project/root'
 				// No mcpLog - should trigger CLI mode
 			};
 			// Act
 			await expandTask(tasksPath, taskId, 3, false, '', context, false);
 			// Assert - Should display telemetry for CLI users
 			expect(displayAiUsageSummary).toHaveBeenCalledWith(
 				expect.objectContaining({
 					commandName: 'expand-task',
 					modelUsed: 'claude-3-5-sonnet',
 					totalCost: 0.012414
 				}),
 				'cli'
 			);
 		});
 		test('should not display telemetry for MCP output format', async () => {
 			// Arrange
 			const { displayAiUsageSummary } = await import(
 				'../../../../../scripts/modules/ui.js'
 			);
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			// Act
 			await expandTask(tasksPath, taskId, 3, false, '', context, false);
 			// Assert - Should NOT display telemetry for MCP (handled at higher level)
 			expect(displayAiUsageSummary).not.toHaveBeenCalled();
 		});
 	});
 	describe('Edge Cases', () => {
 		test('should handle empty additional context', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			// Act
 			await expandTask(tasksPath, taskId, 3, false, '', context, false);
 			// Assert - Should work with empty context (but may include project context)
 			expect(generateTextService).toHaveBeenCalledWith(
 				expect.objectContaining({
 					prompt: expect.stringMatching(/.*/) // Just ensure prompt exists
 				})
 			);
 		});
 		test('should handle additional context correctly', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const additionalContext = 'Use React hooks and TypeScript';
 			const context = {
 				mcpLog: createMcpLogMock(),
 				projectRoot: '/mock/project/root'
 			};
 			// Act
 			await expandTask(
 				tasksPath,
 				taskId,
 				3,
 				false,
 				additionalContext,
 				context,
 				false
 			);
 			// Assert - Should include additional context in prompt
 			expect(generateTextService).toHaveBeenCalledWith(
 				expect.objectContaining({
 					prompt: expect.stringContaining('Use React hooks and TypeScript')
 				})
 			);
 		});
 		test('should handle missing project root in context', async () => {
 			// Arrange
 			const tasksPath = 'tasks/tasks.json';
 			const taskId = '2';
 			const context = {
 				mcpLog: createMcpLogMock()
 				// No projectRoot in context
 			};
 			// Act
 			await expandTask(tasksPath, taskId, 3, false, '', context, false);
 			// Assert - Should derive project root from tasksPath
 			expect(findProjectRoot).toHaveBeenCalledWith(tasksPath);
 			expect(readJSON).toHaveBeenCalledWith(
 				tasksPath,
 				'/mock/project/root',
 				undefined
 			);
 		});
 	});
 });
Author	SHA1	Message	Date
Ben Vargas	8df2d50bac	chore: add changeset for Claude Code provider feature	2025-06-20 16:21:37 +03:00
Ben Vargas	d0a7deb46c	fix(models): add missing --claude-code flag to models command The models command was missing the --claude-code provider flag, preventing users from setting Claude Code models via CLI. While the backend already supported claude-code as a provider hint, there was no command-line flag to trigger it. Changes: - Added --claude-code option to models command alongside existing provider flags - Updated provider flags validation to include claudeCode option - Added claude-code to providerHint logic for all three model roles (main, research, fallback) - Updated error message to include --claude-code in list of mutually exclusive flags - Added example usage in help text This allows users to properly set Claude Code models using commands like: task-master models --set-main sonnet --claude-code task-master models --set-main opus --claude-code Without this flag, users would get "Model ID not found" errors when trying to set claude-code models, as the system couldn't determine the correct provider for generic model names like "sonnet" or "opus".	2025-06-20 16:21:36 +03:00
Ben Vargas	18a5f63d06	style: apply biome formatting to test files	2025-06-20 16:21:21 +03:00
Ben Vargas	5d82b69610	docs: add Claude Code support information to README - Added Claude Code to the list of supported providers in Requirements section - Noted that Claude Code requires no API key but needs Claude Code CLI - Added example of configuring claude-code/sonnet model - Created dedicated Claude Code Support section with key information - Added link to detailed Claude Code setup documentation This ensures users are aware of the Claude Code option as a no-API-key alternative for using Claude models.	2025-06-20 16:21:21 +03:00
Ben Vargas	de77826bcc	revert: remove maxTokens update functionality from init This functionality was out of scope for the Claude Code provider PR. The automatic updating of maxTokens values in config.json during initialization is a general improvement that should be in a separate PR. Additionally, Claude Code ignores maxTokens and temperature parameters anyway, making this change irrelevant for the Claude Code integration. Removed: - scripts/modules/update-config-tokens.js - Import and usage in scripts/init.js	2025-06-20 16:21:21 +03:00
Ben Vargas	4125025abd	test: add comprehensive tests for ClaudeCodeProvider Addresses code review feedback about missing automated tests for the ClaudeCodeProvider. ## Changes - Added unit tests for ClaudeCodeProvider class covering constructor, validateAuth, and getClient methods - Added unit tests for ClaudeCodeLanguageModel testing lazy loading behavior and error handling - Added integration tests verifying optional dependency behavior when @anthropic-ai/claude-code is not installed ## Test Coverage 1. Unit Tests: - ClaudeCodeProvider: Basic functionality, no API key requirement, client creation - ClaudeCodeLanguageModel: Model initialization, lazy loading, error messages, warning generation 2. Integration Tests: - Optional dependency behavior when package is not installed - Clear error messages for users about missing package - Provider instantiation works but usage fails gracefully All tests pass and provide comprehensive coverage for the claude-code provider implementation.	2025-06-20 16:20:56 +03:00
Ben Vargas	72a324075c	feat: make @anthropic-ai/claude-code an optional dependency This change makes the Claude Code SDK package optional, preventing installation failures for users who don't need Claude Code functionality. Changes: - Added @anthropic-ai/claude-code to optionalDependencies in package.json - Implemented lazy loading in language-model.js to only import the SDK when actually used - Updated documentation to explain the optional installation requirement - Applied formatting fixes to ensure code consistency Benefits: - Users without Claude Code subscriptions don't need to install the dependency - Reduces package size for users who don't use Claude Code - Prevents installation failures if the package is unavailable - Provides clear error messages when the package is needed but not installed The implementation uses dynamic imports to load the SDK only when doGenerate() or doStream() is called, ensuring the provider can be instantiated without the package present.	2025-06-20 16:20:56 +03:00
Ben Vargas	93271e0a2d	fix(docs): correct invalid commands in claude-code usage examples - Remove non-existent 'do', 'estimate', and 'analyze' commands - Replace with actual Task Master commands: next, show, set-status - Use correct syntax for parse-prd and analyze-complexity	2025-06-20 16:20:56 +03:00
Ben Vargas	df9ce457ff	feat: add Claude Code provider support Implements Claude Code as a new AI provider that uses the Claude Code CLI without requiring API keys. This enables users to leverage Claude models through their local Claude Code installation. Key changes: - Add complete AI SDK v1 implementation for Claude Code provider - Custom SDK with streaming/non-streaming support - Session management for conversation continuity - JSON extraction for object generation mode - Support for advanced settings (maxTurns, allowedTools, etc.) - Integrate Claude Code into Task Master's provider system - Update ai-services-unified.js to handle keyless authentication - Add provider to supported-models.json with opus/sonnet models - Ensure correct maxTokens values are applied (opus: 32000, sonnet: 64000) - Fix maxTokens configuration issue - Add max_tokens property to getAvailableModels() output - Update setModel() to properly handle claude-code models - Create update-config-tokens.js utility for init process - Add comprehensive documentation - User guide with configuration examples - Advanced settings explanation and future integration options The implementation maintains full backward compatibility with existing providers while adding seamless Claude Code support to all Task Master commands.	2025-06-20 16:20:56 +03:00