chore: fix tests

next up: connecting it to everything else and testing that tm list works well and loads what it need to load

.changeset/config.json

@@ -6,7 +6,7 @@
       "repo": "eyaltoledano/claude-task-master"
     }
   ],
-  "commit": true,
+  "commit": false,
   "fixed": [],
   "linked": [],
   "access": "public",

.changeset/pre.json

@@ -1,5 +1,5 @@
 {
-  "mode": "exit",
+  "mode": "pre",
   "tag": "rc",
   "initialVersions": {
     "task-master-ai": "0.25.1",

.changeset/wild-ears-look.md

@@ -1,5 +0,0 @@
----
-"extension": minor
----
-
-Added a Start Build button to the VSCODE Task Properties Right Panel

.github/workflows/ci.yml

@@ -9,109 +9,70 @@ on:
     branches:
       - main
       - next
-  workflow_dispatch:
 
 permissions:
   contents: read
 
-env:
-  DO_NOT_TRACK: 1
-  NODE_ENV: development
-
 jobs:
-  # Fast checks that can run in parallel
-  format-check:
-    name: Format Check
+  setup:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
         with:
-          fetch-depth: 2
+          fetch-depth: 0
 
       - uses: actions/setup-node@v4
         with:
           node-version: 20
-          cache: "npm"
+          cache: 'npm'
 
-      - name: Install dependencies
-        run: npm install --frozen-lockfile --prefer-offline
-        timeout-minutes: 5
+      - name: Install Dependencies
+        id: install
+        run: npm ci
+        timeout-minutes: 2
+
+      - name: Cache node_modules
+        uses: actions/cache@v4
+        with:
+          path: node_modules
+          key: ${{ runner.os }}-node-modules-${{ hashFiles('**/package-lock.json') }}
+
+  format-check:
+    needs: setup
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 20
+
+      - name: Restore node_modules
+        uses: actions/cache@v4
+        with:
+          path: node_modules
+          key: ${{ runner.os }}-node-modules-${{ hashFiles('**/package-lock.json') }}
 
       - name: Format Check
         run: npm run format-check
         env:
           FORCE_COLOR: 1
 
-  typecheck:
-    name: Typecheck
-    timeout-minutes: 10
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-        with:
-          fetch-depth: 2
-
-      - uses: actions/setup-node@v4
-        with:
-          node-version: 20
-          cache: "npm"
-
-      - name: Install dependencies
-        run: npm install --frozen-lockfile --prefer-offline
-        timeout-minutes: 5
-
-      - name: Typecheck
-        run: npm run typecheck
-        env:
-          FORCE_COLOR: 1
-
-  # Build job to ensure everything compiles
-  build:
-    name: Build
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-        with:
-          fetch-depth: 2
-
-      - uses: actions/setup-node@v4
-        with:
-          node-version: 20
-          cache: "npm"
-
-      - name: Install dependencies
-        run: npm install --frozen-lockfile --prefer-offline
-        timeout-minutes: 5
-
-      - name: Build
-        run: npm run build
-        env:
-          NODE_ENV: production
-          FORCE_COLOR: 1
-
   test:
-    name: Test
-    timeout-minutes: 15
+    needs: setup
     runs-on: ubuntu-latest
-    needs: [format-check, typecheck, build]
     steps:
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 2
 
       - uses: actions/setup-node@v4
         with:
           node-version: 20
-          cache: "npm"
 
-      - name: Install dependencies
-        run: npm install --frozen-lockfile --prefer-offline
-        timeout-minutes: 5
-
-      - name: Build packages (required for tests)
-        run: npm run build:packages
-        env:
-          NODE_ENV: production
+      - name: Restore node_modules
+        uses: actions/cache@v4
+        with:
+          path: node_modules
+          key: ${{ runner.os }}-node-modules-${{ hashFiles('**/package-lock.json') }}
 
       - name: Run Tests
         run: |
@@ -120,6 +81,7 @@ jobs:
           NODE_ENV: test
           CI: true
           FORCE_COLOR: 1
+        timeout-minutes: 10
 
       - name: Upload Test Results
         if: always()

.github/workflows/claude-docs-trigger.yml

@@ -1,57 +0,0 @@
-name: Trigger Claude Documentation Update
-
-on:
-  push:
-    branches:
-      - next
-    paths-ignore:
-      - "apps/docs/**"
-      - "*.md"
-      - ".github/workflows/**"
-
-jobs:
-  trigger-docs-update:
-    # Only run if changes were merged (not direct pushes from bots)
-    if: github.actor != 'github-actions[bot]' && github.actor != 'dependabot[bot]'
-    runs-on: ubuntu-latest
-    permissions:
-      contents: read
-      actions: write
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 2 # Need previous commit for comparison
-
-      - name: Get changed files
-        id: changed-files
-        run: |
-          echo "Changed files in this push:"
-          git diff --name-only HEAD^ HEAD | tee changed_files.txt
-
-          # Store changed files for Claude to analyze (escaped for JSON)
-          CHANGED_FILES=$(git diff --name-only HEAD^ HEAD | jq -Rs .)
-          echo "changed_files=$CHANGED_FILES" >> $GITHUB_OUTPUT
-
-          # Get the commit message (escaped for JSON)
-          COMMIT_MSG=$(git log -1 --pretty=%B | jq -Rs .)
-          echo "commit_message=$COMMIT_MSG" >> $GITHUB_OUTPUT
-
-          # Get diff for documentation context (escaped for JSON)
-          COMMIT_DIFF=$(git diff HEAD^ HEAD --stat | jq -Rs .)
-          echo "commit_diff=$COMMIT_DIFF" >> $GITHUB_OUTPUT
-
-          # Get commit SHA
-          echo "commit_sha=${{ github.sha }}" >> $GITHUB_OUTPUT
-
-      - name: Trigger Claude workflow
-        env:
-          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        run: |
-          # Trigger the Claude docs updater workflow with the change information
-          gh workflow run claude-docs-updater.yml \
-            --ref next \
-            -f commit_sha="${{ steps.changed-files.outputs.commit_sha }}" \
-            -f commit_message=${{ steps.changed-files.outputs.commit_message }} \
-            -f changed_files=${{ steps.changed-files.outputs.changed_files }} \
-            -f commit_diff=${{ steps.changed-files.outputs.commit_diff }}

.github/workflows/claude-docs-updater.yml

@@ -1,27 +1,18 @@
 name: Claude Documentation Updater
 
 on:
-  workflow_dispatch:
-    inputs:
-      commit_sha:
-        description: 'The commit SHA that triggered this update'
-        required: true
-        type: string
-      commit_message:
-        description: 'The commit message'
-        required: true
-        type: string
-      changed_files:
-        description: 'List of changed files'
-        required: true
-        type: string
-      commit_diff:
-        description: 'Diff summary of changes'
-        required: true
-        type: string
+  push:
+    branches:
+      - next
+    paths-ignore:
+      - "apps/docs/**"
+      - "*.md"
+      - ".github/workflows/**"
 
 jobs:
   update-docs:
+    # Only run if changes were merged (not direct pushes from bots)
+    if: github.actor != 'github-actions[bot]' && github.actor != 'dependabot[bot]'
     runs-on: ubuntu-latest
     permissions:
       contents: write
@@ -31,8 +22,28 @@ jobs:
       - name: Checkout repository
         uses: actions/checkout@v4
         with:
-          ref: next
-          fetch-depth: 0 # Need full history to checkout specific commit
+          fetch-depth: 2 # Need previous commit for comparison
+
+      - name: Get changed files
+        id: changed-files
+        run: |
+          echo "Changed files in this push:"
+          git diff --name-only HEAD^ HEAD | tee changed_files.txt
+
+          # Store changed files for Claude to analyze
+          echo "changed_files<<EOF" >> $GITHUB_OUTPUT
+          git diff --name-only HEAD^ HEAD >> $GITHUB_OUTPUT
+          echo "EOF" >> $GITHUB_OUTPUT
+
+          # Get the commit message and changes summary
+          echo "commit_message<<EOF" >> $GITHUB_OUTPUT
+          git log -1 --pretty=%B >> $GITHUB_OUTPUT
+          echo "EOF" >> $GITHUB_OUTPUT
+
+          # Get diff for documentation context
+          echo "commit_diff<<EOF" >> $GITHUB_OUTPUT
+          git diff HEAD^ HEAD --stat >> $GITHUB_OUTPUT
+          echo "EOF" >> $GITHUB_OUTPUT
 
       - name: Create docs update branch
         id: create-branch
@@ -60,12 +71,12 @@ jobs:
             You are a documentation specialist. Analyze the recent changes pushed to the 'next' branch and update the documentation accordingly.
 
             Recent changes:
-            - Commit: ${{ inputs.commit_message }}
+            - Commit: ${{ steps.changed-files.outputs.commit_message }}
             - Changed files:
-            ${{ inputs.changed_files }}
+            ${{ steps.changed-files.outputs.changed_files }}
 
             - Changes summary:
-            ${{ inputs.commit_diff }}
+            ${{ steps.changed-files.outputs.commit_diff }}
 
             Your task:
             1. Analyze the changes to understand what functionality was added, modified, or removed
@@ -102,7 +113,7 @@ jobs:
 
             This PR was automatically generated to update documentation based on recent changes.
             
-            Original commit: ${{ inputs.commit_message }}
+            Original commit: ${{ steps.changed-files.outputs.commit_message }}
             
             Co-authored-by: Claude <claude-assistant@anthropic.com>"
           fi
@@ -122,12 +133,12 @@ jobs:
             This PR automatically updates documentation based on recent changes merged to the \`next\` branch.
 
             ### Original Changes
-            **Commit:** ${{ inputs.commit_sha }}
-            **Message:** ${{ inputs.commit_message }}
+            **Commit:** ${{ github.sha }}
+            **Message:** ${{ steps.changed-files.outputs.commit_message }}
             
             ### Changed Files in Original Commit
             \`\`\`
-            ${{ inputs.changed_files }}
+            ${{ steps.changed-files.outputs.changed_files }}
             \`\`\`
 
             ### Documentation Updates

.gitignore

@@ -94,6 +94,3 @@ apps/extension/.vscode-test/
 
 # apps/extension
 apps/extension/vsix-build/
-
-# turbo
-.turbo

apps/cli/package.json

@@ -4,32 +4,28 @@
 	"description": "Task Master CLI - Command line interface for task management",
 	"type": "module",
 	"main": "./dist/index.js",
-	"types": "./src/index.ts",
+	"types": "./dist/index.d.ts",
 	"exports": {
 		".": {
 			"types": "./src/index.ts",
-			"import": "./dist/index.js"
+			"import": "./dist/index.js",
+			"require": "./dist/index.js"
 		}
 	},
 	"files": ["dist", "README.md"],
 	"scripts": {
-		"build": "tsc",
-		"dev": "tsc --watch",
+		"build": "tsup",
+		"dev": "tsup --watch",
 		"typecheck": "tsc --noEmit",
 		"lint": "biome check src",
 		"format": "biome format --write src",
 		"test": "vitest run",
-		"test:watch": "vitest",
-		"test:coverage": "vitest run --coverage",
-		"test:unit": "vitest run -t unit",
-		"test:integration": "vitest run -t integration",
-		"test:e2e": "vitest run --dir tests/e2e",
-		"test:ci": "vitest run --coverage --reporter=dot"
+		"test:watch": "vitest"
 	},
 	"dependencies": {
 		"@tm/core": "*",
 		"boxen": "^7.1.1",
-		"chalk": "5.6.2",
+		"chalk": "^5.3.0",
 		"cli-table3": "^0.6.5",
 		"commander": "^12.1.0",
 		"inquirer": "^9.2.10",
@@ -39,6 +35,7 @@
 		"@biomejs/biome": "^1.9.4",
 		"@types/inquirer": "^9.0.3",
 		"@types/node": "^22.10.5",
+		"tsup": "^8.3.0",
 		"tsx": "^4.20.4",
 		"typescript": "^5.7.3",
 		"vitest": "^2.1.8"

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

@@ -1,514 +0,0 @@
-/**
- * @fileoverview Auth command using Commander's native class pattern
- * Extends Commander.Command for better integration with the framework
- */
-
-import { Command } from 'commander';
-import chalk from 'chalk';
-import inquirer from 'inquirer';
-import ora, { type Ora } from 'ora';
-import open from 'open';
-import {
-	AuthManager,
-	AuthenticationError,
-	type AuthCredentials
-} from '@tm/core/auth';
-import * as ui from '../utils/ui.js';
-
-/**
- * Result type from auth command
- */
-export interface AuthResult {
-	success: boolean;
-	action: 'login' | 'logout' | 'status' | 'refresh';
-	credentials?: AuthCredentials;
-	message?: string;
-}
-
-/**
- * AuthCommand extending Commander's Command class
- * This is a thin presentation layer over @tm/core's AuthManager
- */
-export class AuthCommand extends Command {
-	private authManager: AuthManager;
-	private lastResult?: AuthResult;
-
-	constructor(name?: string) {
-		super(name || 'auth');
-
-		// Initialize auth manager
-		this.authManager = AuthManager.getInstance();
-
-		// Configure the command with subcommands
-		this.description('Manage authentication with tryhamster.com');
-
-		// Add subcommands
-		this.addLoginCommand();
-		this.addLogoutCommand();
-		this.addStatusCommand();
-		this.addRefreshCommand();
-
-		// Default action shows help
-		this.action(() => {
-			this.help();
-		});
-	}
-
-	/**
-	 * Add login subcommand
-	 */
-	private addLoginCommand(): void {
-		this.command('login')
-			.description('Authenticate with tryhamster.com')
-			.action(async () => {
-				await this.executeLogin();
-			});
-	}
-
-	/**
-	 * Add logout subcommand
-	 */
-	private addLogoutCommand(): void {
-		this.command('logout')
-			.description('Logout and clear credentials')
-			.action(async () => {
-				await this.executeLogout();
-			});
-	}
-
-	/**
-	 * Add status subcommand
-	 */
-	private addStatusCommand(): void {
-		this.command('status')
-			.description('Display authentication status')
-			.action(async () => {
-				await this.executeStatus();
-			});
-	}
-
-	/**
-	 * Add refresh subcommand
-	 */
-	private addRefreshCommand(): void {
-		this.command('refresh')
-			.description('Refresh authentication token')
-			.action(async () => {
-				await this.executeRefresh();
-			});
-	}
-
-	/**
-	 * Execute login command
-	 */
-	private async executeLogin(): Promise<void> {
-		try {
-			const result = await this.performInteractiveAuth();
-			this.setLastResult(result);
-
-			if (!result.success) {
-				process.exit(1);
-			}
-
-			// Exit cleanly after successful authentication
-			// Small delay to ensure all output is flushed
-			setTimeout(() => {
-				process.exit(0);
-			}, 100);
-		} catch (error: any) {
-			this.handleError(error);
-			process.exit(1);
-		}
-	}
-
-	/**
-	 * Execute logout command
-	 */
-	private async executeLogout(): Promise<void> {
-		try {
-			const result = await this.performLogout();
-			this.setLastResult(result);
-
-			if (!result.success) {
-				process.exit(1);
-			}
-		} catch (error: any) {
-			this.handleError(error);
-			process.exit(1);
-		}
-	}
-
-	/**
-	 * Execute status command
-	 */
-	private async executeStatus(): Promise<void> {
-		try {
-			const result = this.displayStatus();
-			this.setLastResult(result);
-		} catch (error: any) {
-			this.handleError(error);
-			process.exit(1);
-		}
-	}
-
-	/**
-	 * Execute refresh command
-	 */
-	private async executeRefresh(): Promise<void> {
-		try {
-			const result = await this.refreshToken();
-			this.setLastResult(result);
-
-			if (!result.success) {
-				process.exit(1);
-			}
-		} catch (error: any) {
-			this.handleError(error);
-			process.exit(1);
-		}
-	}
-
-	/**
-	 * Display authentication status
-	 */
-	private displayStatus(): AuthResult {
-		const credentials = this.authManager.getCredentials();
-
-		console.log(chalk.cyan('\n🔐 Authentication Status\n'));
-
-		if (credentials) {
-			console.log(chalk.green('✓ Authenticated'));
-			console.log(chalk.gray(`  Email: ${credentials.email || 'N/A'}`));
-			console.log(chalk.gray(`  User ID: ${credentials.userId}`));
-			console.log(
-				chalk.gray(`  Token Type: ${credentials.tokenType || 'standard'}`)
-			);
-
-			if (credentials.expiresAt) {
-				const expiresAt = new Date(credentials.expiresAt);
-				const now = new Date();
-				const hoursRemaining = Math.floor(
-					(expiresAt.getTime() - now.getTime()) / (1000 * 60 * 60)
-				);
-
-				if (hoursRemaining > 0) {
-					console.log(
-						chalk.gray(
-							`  Expires: ${expiresAt.toLocaleString()} (${hoursRemaining} hours remaining)`
-						)
-					);
-				} else {
-					console.log(
-						chalk.yellow(`  Token expired at: ${expiresAt.toLocaleString()}`)
-					);
-				}
-			} else {
-				console.log(chalk.gray('  Expires: Never (API key)'));
-			}
-
-			console.log(
-				chalk.gray(`  Saved: ${new Date(credentials.savedAt).toLocaleString()}`)
-			);
-
-			return {
-				success: true,
-				action: 'status',
-				credentials,
-				message: 'Authenticated'
-			};
-		} else {
-			console.log(chalk.yellow('✗ Not authenticated'));
-			console.log(
-				chalk.gray('\n  Run "task-master auth login" to authenticate')
-			);
-
-			return {
-				success: false,
-				action: 'status',
-				message: 'Not authenticated'
-			};
-		}
-	}
-
-	/**
-	 * Perform logout
-	 */
-	private async performLogout(): Promise<AuthResult> {
-		try {
-			await this.authManager.logout();
-			ui.displaySuccess('Successfully logged out');
-
-			return {
-				success: true,
-				action: 'logout',
-				message: 'Successfully logged out'
-			};
-		} catch (error) {
-			const message = `Failed to logout: ${(error as Error).message}`;
-			ui.displayError(message);
-
-			return {
-				success: false,
-				action: 'logout',
-				message
-			};
-		}
-	}
-
-	/**
-	 * Refresh authentication token
-	 */
-	private async refreshToken(): Promise<AuthResult> {
-		const spinner = ora('Refreshing authentication token...').start();
-
-		try {
-			const credentials = await this.authManager.refreshToken();
-			spinner.succeed('Token refreshed successfully');
-
-			console.log(
-				chalk.gray(
-					`  New expiration: ${credentials.expiresAt ? new Date(credentials.expiresAt).toLocaleString() : 'Never'}`
-				)
-			);
-
-			return {
-				success: true,
-				action: 'refresh',
-				credentials,
-				message: 'Token refreshed successfully'
-			};
-		} catch (error) {
-			spinner.fail('Failed to refresh token');
-
-			if ((error as AuthenticationError).code === 'NO_REFRESH_TOKEN') {
-				ui.displayWarning(
-					'No refresh token available. Please re-authenticate.'
-				);
-			} else {
-				ui.displayError(`Refresh failed: ${(error as Error).message}`);
-			}
-
-			return {
-				success: false,
-				action: 'refresh',
-				message: `Failed to refresh: ${(error as Error).message}`
-			};
-		}
-	}
-
-	/**
-	 * Perform interactive authentication
-	 */
-	private async performInteractiveAuth(): Promise<AuthResult> {
-		ui.displayBanner('Task Master Authentication');
-
-		// Check if already authenticated
-		if (this.authManager.isAuthenticated()) {
-			const { continueAuth } = await inquirer.prompt([
-				{
-					type: 'confirm',
-					name: 'continueAuth',
-					message:
-						'You are already authenticated. Do you want to re-authenticate?',
-					default: false
-				}
-			]);
-
-			if (!continueAuth) {
-				const credentials = this.authManager.getCredentials();
-				ui.displaySuccess('Using existing authentication');
-
-				if (credentials) {
-					console.log(chalk.gray(`  Email: ${credentials.email || 'N/A'}`));
-					console.log(chalk.gray(`  User ID: ${credentials.userId}`));
-				}
-
-				return {
-					success: true,
-					action: 'login',
-					credentials: credentials || undefined,
-					message: 'Using existing authentication'
-				};
-			}
-		}
-
-		try {
-			// Direct browser authentication - no menu needed
-			const credentials = await this.authenticateWithBrowser();
-
-			ui.displaySuccess('Authentication successful!');
-			console.log(
-				chalk.gray(`  Logged in as: ${credentials.email || credentials.userId}`)
-			);
-
-			return {
-				success: true,
-				action: 'login',
-				credentials,
-				message: 'Authentication successful'
-			};
-		} catch (error) {
-			this.handleAuthError(error as AuthenticationError);
-
-			return {
-				success: false,
-				action: 'login',
-				message: `Authentication failed: ${(error as Error).message}`
-			};
-		}
-	}
-
-	/**
-	 * Authenticate with browser using OAuth 2.0 with PKCE
-	 */
-	private async authenticateWithBrowser(): Promise<AuthCredentials> {
-		let authSpinner: Ora | null = null;
-
-		try {
-			// Use AuthManager's new unified OAuth flow method with callbacks
-			const credentials = await this.authManager.authenticateWithOAuth({
-				// Callback to handle browser opening
-				openBrowser: async (authUrl) => {
-					await open(authUrl);
-				},
-				timeout: 5 * 60 * 1000, // 5 minutes
-
-				// Callback when auth URL is ready
-				onAuthUrl: (authUrl) => {
-					// Display authentication instructions
-					console.log(chalk.blue.bold('\n🔐 Browser Authentication\n'));
-					console.log(chalk.white('  Opening your browser to authenticate...'));
-					console.log(chalk.gray("  If the browser doesn't open, visit:"));
-					console.log(chalk.cyan.underline(`  ${authUrl}\n`));
-				},
-
-				// Callback when waiting for authentication
-				onWaitingForAuth: () => {
-					authSpinner = ora({
-						text: 'Waiting for authentication...',
-						spinner: 'dots'
-					}).start();
-				},
-
-				// Callback on success
-				onSuccess: () => {
-					if (authSpinner) {
-						authSpinner.succeed('Authentication successful!');
-					}
-				},
-
-				// Callback on error
-				onError: () => {
-					if (authSpinner) {
-						authSpinner.fail('Authentication failed');
-					}
-				}
-			});
-
-			return credentials;
-		} catch (error) {
-			throw error;
-		}
-	}
-
-	/**
-	 * Handle authentication errors
-	 */
-	private handleAuthError(error: AuthenticationError): void {
-		console.error(chalk.red(`\n✗ ${error.message}`));
-
-		switch (error.code) {
-			case 'NETWORK_ERROR':
-				ui.displayWarning(
-					'Please check your internet connection and try again.'
-				);
-				break;
-			case 'INVALID_CREDENTIALS':
-				ui.displayWarning('Please check your credentials and try again.');
-				break;
-			case 'AUTH_EXPIRED':
-				ui.displayWarning(
-					'Your session has expired. Please authenticate again.'
-				);
-				break;
-			default:
-				if (process.env.DEBUG) {
-					console.error(chalk.gray(error.stack || ''));
-				}
-		}
-	}
-
-	/**
-	 * Handle general errors
-	 */
-	private handleError(error: any): void {
-		if (error instanceof AuthenticationError) {
-			this.handleAuthError(error);
-		} else {
-			const msg = error?.getSanitizedDetails?.() ?? {
-				message: error?.message ?? String(error)
-			};
-			console.error(chalk.red(`Error: ${msg.message || 'Unexpected error'}`));
-
-			if (error.stack && process.env.DEBUG) {
-				console.error(chalk.gray(error.stack));
-			}
-		}
-	}
-
-	/**
-	 * Set the last result for programmatic access
-	 */
-	private setLastResult(result: AuthResult): void {
-		this.lastResult = result;
-	}
-
-	/**
-	 * Get the last result (for programmatic usage)
-	 */
-	getLastResult(): AuthResult | undefined {
-		return this.lastResult;
-	}
-
-	/**
-	 * Get current authentication status (for programmatic usage)
-	 */
-	isAuthenticated(): boolean {
-		return this.authManager.isAuthenticated();
-	}
-
-	/**
-	 * Get current credentials (for programmatic usage)
-	 */
-	getCredentials(): AuthCredentials | null {
-		return this.authManager.getCredentials();
-	}
-
-	/**
-	 * Clean up resources
-	 */
-	async cleanup(): Promise<void> {
-		// No resources to clean up for auth command
-		// But keeping method for consistency with other commands
-	}
-
-	/**
-	 * Static method to register this command on an existing program
-	 * This is for gradual migration - allows commands.js to use this
-	 */
-	static registerOn(program: Command): Command {
-		const authCommand = new AuthCommand();
-		program.addCommand(authCommand);
-		return authCommand;
-	}
-
-	/**
-	 * Alternative registration that returns the command for chaining
-	 * Can also configure the command name if needed
-	 */
-	static register(program: Command, name?: string): AuthCommand {
-		const authCommand = new AuthCommand(name);
-		program.addCommand(authCommand);
-		return authCommand;
-	}
-}

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

@@ -1,570 +0,0 @@
-/**
- * @fileoverview Context command for managing org/brief selection
- * Provides a clean interface for workspace context management
- */
-
-import { Command } from 'commander';
-import chalk from 'chalk';
-import inquirer from 'inquirer';
-import ora from 'ora';
-import {
-	AuthManager,
-	AuthenticationError,
-	type UserContext
-} from '@tm/core/auth';
-import * as ui from '../utils/ui.js';
-
-/**
- * Result type from context command
- */
-export interface ContextResult {
-	success: boolean;
-	action: 'show' | 'select-org' | 'select-brief' | 'clear' | 'set';
-	context?: UserContext;
-	message?: string;
-}
-
-/**
- * ContextCommand extending Commander's Command class
- * Manages user's workspace context (org/brief selection)
- */
-export class ContextCommand extends Command {
-	private authManager: AuthManager;
-	private lastResult?: ContextResult;
-
-	constructor(name?: string) {
-		super(name || 'context');
-
-		// Initialize auth manager
-		this.authManager = AuthManager.getInstance();
-
-		// Configure the command
-		this.description(
-			'Manage workspace context (organization and brief selection)'
-		);
-
-		// Add subcommands
-		this.addOrgCommand();
-		this.addBriefCommand();
-		this.addClearCommand();
-		this.addSetCommand();
-
-		// Default action shows current context
-		this.action(async () => {
-			await this.executeShow();
-		});
-	}
-
-	/**
-	 * Add org selection subcommand
-	 */
-	private addOrgCommand(): void {
-		this.command('org')
-			.description('Select an organization')
-			.action(async () => {
-				await this.executeSelectOrg();
-			});
-	}
-
-	/**
-	 * Add brief selection subcommand
-	 */
-	private addBriefCommand(): void {
-		this.command('brief')
-			.description('Select a brief within the current organization')
-			.action(async () => {
-				await this.executeSelectBrief();
-			});
-	}
-
-	/**
-	 * Add clear subcommand
-	 */
-	private addClearCommand(): void {
-		this.command('clear')
-			.description('Clear all context selections')
-			.action(async () => {
-				await this.executeClear();
-			});
-	}
-
-	/**
-	 * Add set subcommand for direct context setting
-	 */
-	private addSetCommand(): void {
-		this.command('set')
-			.description('Set context directly')
-			.option('--org <id>', 'Organization ID')
-			.option('--org-name <name>', 'Organization name')
-			.option('--brief <id>', 'Brief ID')
-			.option('--brief-name <name>', 'Brief name')
-			.action(async (options) => {
-				await this.executeSet(options);
-			});
-	}
-
-	/**
-	 * Execute show current context
-	 */
-	private async executeShow(): Promise<void> {
-		try {
-			const result = this.displayContext();
-			this.setLastResult(result);
-		} catch (error: any) {
-			this.handleError(error);
-			process.exit(1);
-		}
-	}
-
-	/**
-	 * Display current context
-	 */
-	private displayContext(): ContextResult {
-		// Check authentication first
-		if (!this.authManager.isAuthenticated()) {
-			console.log(chalk.yellow('✗ Not authenticated'));
-			console.log(chalk.gray('\n  Run "tm auth login" to authenticate first'));
-
-			return {
-				success: false,
-				action: 'show',
-				message: 'Not authenticated'
-			};
-		}
-
-		const context = this.authManager.getContext();
-
-		console.log(chalk.cyan('\n🌍 Workspace Context\n'));
-
-		if (context && (context.orgId || context.briefId)) {
-			if (context.orgName || context.orgId) {
-				console.log(chalk.green('✓ Organization'));
-				if (context.orgName) {
-					console.log(chalk.white(`  ${context.orgName}`));
-				}
-				if (context.orgId) {
-					console.log(chalk.gray(`  ID: ${context.orgId}`));
-				}
-			}
-
-			if (context.briefName || context.briefId) {
-				console.log(chalk.green('\n✓ Brief'));
-				if (context.briefName) {
-					console.log(chalk.white(`  ${context.briefName}`));
-				}
-				if (context.briefId) {
-					console.log(chalk.gray(`  ID: ${context.briefId}`));
-				}
-			}
-
-			if (context.updatedAt) {
-				console.log(
-					chalk.gray(
-						`\n  Last updated: ${new Date(context.updatedAt).toLocaleString()}`
-					)
-				);
-			}
-
-			return {
-				success: true,
-				action: 'show',
-				context,
-				message: 'Context loaded'
-			};
-		} else {
-			console.log(chalk.yellow('✗ No context selected'));
-			console.log(
-				chalk.gray('\n  Run "tm context org" to select an organization')
-			);
-			console.log(chalk.gray('  Run "tm context brief" to select a brief'));
-
-			return {
-				success: true,
-				action: 'show',
-				message: 'No context selected'
-			};
-		}
-	}
-
-	/**
-	 * Execute org selection
-	 */
-	private async executeSelectOrg(): Promise<void> {
-		try {
-			// Check authentication
-			if (!this.authManager.isAuthenticated()) {
-				ui.displayError('Not authenticated. Run "tm auth login" first.');
-				process.exit(1);
-			}
-
-			const result = await this.selectOrganization();
-			this.setLastResult(result);
-
-			if (!result.success) {
-				process.exit(1);
-			}
-		} catch (error: any) {
-			this.handleError(error);
-			process.exit(1);
-		}
-	}
-
-	/**
-	 * Select an organization interactively
-	 */
-	private async selectOrganization(): Promise<ContextResult> {
-		const spinner = ora('Fetching organizations...').start();
-
-		try {
-			// Fetch organizations from API
-			const organizations = await this.authManager.getOrganizations();
-			spinner.stop();
-
-			if (organizations.length === 0) {
-				ui.displayWarning('No organizations available');
-				return {
-					success: false,
-					action: 'select-org',
-					message: 'No organizations available'
-				};
-			}
-
-			// Prompt for selection
-			const { selectedOrg } = await inquirer.prompt([
-				{
-					type: 'list',
-					name: 'selectedOrg',
-					message: 'Select an organization:',
-					choices: organizations.map((org) => ({
-						name: org.name,
-						value: org
-					}))
-				}
-			]);
-
-			// Update context
-			await this.authManager.updateContext({
-				orgId: selectedOrg.id,
-				orgName: selectedOrg.name,
-				// Clear brief when changing org
-				briefId: undefined,
-				briefName: undefined
-			});
-
-			ui.displaySuccess(`Selected organization: ${selectedOrg.name}`);
-
-			return {
-				success: true,
-				action: 'select-org',
-				context: this.authManager.getContext() || undefined,
-				message: `Selected organization: ${selectedOrg.name}`
-			};
-		} catch (error) {
-			spinner.fail('Failed to fetch organizations');
-			throw error;
-		}
-	}
-
-	/**
-	 * Execute brief selection
-	 */
-	private async executeSelectBrief(): Promise<void> {
-		try {
-			// Check authentication
-			if (!this.authManager.isAuthenticated()) {
-				ui.displayError('Not authenticated. Run "tm auth login" first.');
-				process.exit(1);
-			}
-
-			// Check if org is selected
-			const context = this.authManager.getContext();
-			if (!context?.orgId) {
-				ui.displayError(
-					'No organization selected. Run "tm context org" first.'
-				);
-				process.exit(1);
-			}
-
-			const result = await this.selectBrief(context.orgId);
-			this.setLastResult(result);
-
-			if (!result.success) {
-				process.exit(1);
-			}
-		} catch (error: any) {
-			this.handleError(error);
-			process.exit(1);
-		}
-	}
-
-	/**
-	 * Select a brief within the current organization
-	 */
-	private async selectBrief(orgId: string): Promise<ContextResult> {
-		const spinner = ora('Fetching briefs...').start();
-
-		try {
-			// Fetch briefs from API
-			const briefs = await this.authManager.getBriefs(orgId);
-			spinner.stop();
-
-			if (briefs.length === 0) {
-				ui.displayWarning('No briefs available in this organization');
-				return {
-					success: false,
-					action: 'select-brief',
-					message: 'No briefs available'
-				};
-			}
-
-			// Prompt for selection
-			const { selectedBrief } = await inquirer.prompt([
-				{
-					type: 'list',
-					name: 'selectedBrief',
-					message: 'Select a brief:',
-					choices: [
-						{ name: '(No brief - organization level)', value: null },
-						...briefs.map((brief) => ({
-							name: `Brief ${brief.id.slice(0, 8)} (${new Date(brief.createdAt).toLocaleDateString()})`,
-							value: brief
-						}))
-					]
-				}
-			]);
-
-			if (selectedBrief) {
-				// Update context with brief
-				const briefName = `Brief ${selectedBrief.id.slice(0, 8)}`;
-				await this.authManager.updateContext({
-					briefId: selectedBrief.id,
-					briefName: briefName
-				});
-
-				ui.displaySuccess(`Selected brief: ${briefName}`);
-
-				return {
-					success: true,
-					action: 'select-brief',
-					context: this.authManager.getContext() || undefined,
-					message: `Selected brief: ${selectedBrief.name}`
-				};
-			} else {
-				// Clear brief selection
-				await this.authManager.updateContext({
-					briefId: undefined,
-					briefName: undefined
-				});
-
-				ui.displaySuccess('Cleared brief selection (organization level)');
-
-				return {
-					success: true,
-					action: 'select-brief',
-					context: this.authManager.getContext() || undefined,
-					message: 'Cleared brief selection'
-				};
-			}
-		} catch (error) {
-			spinner.fail('Failed to fetch briefs');
-			throw error;
-		}
-	}
-
-	/**
-	 * Execute clear context
-	 */
-	private async executeClear(): Promise<void> {
-		try {
-			// Check authentication
-			if (!this.authManager.isAuthenticated()) {
-				ui.displayError('Not authenticated. Run "tm auth login" first.');
-				process.exit(1);
-			}
-
-			const result = await this.clearContext();
-			this.setLastResult(result);
-
-			if (!result.success) {
-				process.exit(1);
-			}
-		} catch (error: any) {
-			this.handleError(error);
-			process.exit(1);
-		}
-	}
-
-	/**
-	 * Clear all context selections
-	 */
-	private async clearContext(): Promise<ContextResult> {
-		try {
-			await this.authManager.clearContext();
-			ui.displaySuccess('Context cleared');
-
-			return {
-				success: true,
-				action: 'clear',
-				message: 'Context cleared'
-			};
-		} catch (error) {
-			ui.displayError(`Failed to clear context: ${(error as Error).message}`);
-
-			return {
-				success: false,
-				action: 'clear',
-				message: `Failed to clear context: ${(error as Error).message}`
-			};
-		}
-	}
-
-	/**
-	 * Execute set context with options
-	 */
-	private async executeSet(options: any): Promise<void> {
-		try {
-			// Check authentication
-			if (!this.authManager.isAuthenticated()) {
-				ui.displayError('Not authenticated. Run "tm auth login" first.');
-				process.exit(1);
-			}
-
-			const result = await this.setContext(options);
-			this.setLastResult(result);
-
-			if (!result.success) {
-				process.exit(1);
-			}
-		} catch (error: any) {
-			this.handleError(error);
-			process.exit(1);
-		}
-	}
-
-	/**
-	 * Set context directly from options
-	 */
-	private async setContext(options: any): Promise<ContextResult> {
-		try {
-			const context: Partial<UserContext> = {};
-
-			if (options.org) {
-				context.orgId = options.org;
-			}
-			if (options.orgName) {
-				context.orgName = options.orgName;
-			}
-			if (options.brief) {
-				context.briefId = options.brief;
-			}
-			if (options.briefName) {
-				context.briefName = options.briefName;
-			}
-
-			if (Object.keys(context).length === 0) {
-				ui.displayWarning('No context options provided');
-				return {
-					success: false,
-					action: 'set',
-					message: 'No context options provided'
-				};
-			}
-
-			await this.authManager.updateContext(context);
-			ui.displaySuccess('Context updated');
-
-			// Display what was set
-			if (context.orgName || context.orgId) {
-				console.log(
-					chalk.gray(`  Organization: ${context.orgName || context.orgId}`)
-				);
-			}
-			if (context.briefName || context.briefId) {
-				console.log(
-					chalk.gray(`  Brief: ${context.briefName || context.briefId}`)
-				);
-			}
-
-			return {
-				success: true,
-				action: 'set',
-				context: this.authManager.getContext() || undefined,
-				message: 'Context updated'
-			};
-		} catch (error) {
-			ui.displayError(`Failed to set context: ${(error as Error).message}`);
-
-			return {
-				success: false,
-				action: 'set',
-				message: `Failed to set context: ${(error as Error).message}`
-			};
-		}
-	}
-
-	/**
-	 * Handle errors
-	 */
-	private handleError(error: any): void {
-		if (error instanceof AuthenticationError) {
-			console.error(chalk.red(`\n✗ ${error.message}`));
-
-			if (error.code === 'NOT_AUTHENTICATED') {
-				ui.displayWarning('Please authenticate first: tm auth login');
-			}
-		} else {
-			const msg = error?.message ?? String(error);
-			console.error(chalk.red(`Error: ${msg}`));
-
-			if (error.stack && process.env.DEBUG) {
-				console.error(chalk.gray(error.stack));
-			}
-		}
-	}
-
-	/**
-	 * Set the last result for programmatic access
-	 */
-	private setLastResult(result: ContextResult): void {
-		this.lastResult = result;
-	}
-
-	/**
-	 * Get the last result (for programmatic usage)
-	 */
-	getLastResult(): ContextResult | undefined {
-		return this.lastResult;
-	}
-
-	/**
-	 * Get current context (for programmatic usage)
-	 */
-	getContext(): UserContext | null {
-		return this.authManager.getContext();
-	}
-
-	/**
-	 * Clean up resources
-	 */
-	async cleanup(): Promise<void> {
-		// No resources to clean up for context command
-	}
-
-	/**
-	 * Static method to register this command on an existing program
-	 */
-	static registerOn(program: Command): Command {
-		const contextCommand = new ContextCommand();
-		program.addCommand(contextCommand);
-		return contextCommand;
-	}
-
-	/**
-	 * Alternative registration that returns the command for chaining
-	 */
-	static register(program: Command, name?: string): ContextCommand {
-		const contextCommand = new ContextCommand(name);
-		program.addCommand(contextCommand);
-		return contextCommand;
-	}
-}

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

@@ -15,7 +15,6 @@ import {
 	STATUS_ICONS,
 	type OutputFormat
 } from '@tm/core';
-import type { StorageType } from '@tm/core/types';
 import * as ui from '../utils/ui.js';
 
 /**
@@ -38,7 +37,7 @@ export interface ListTasksResult {
 	total: number;
 	filtered: number;
 	tag?: string;
-	storageType: Exclude<StorageType, 'auto'>;
+	storageType: 'file' | 'api';
 }
 
 /**

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

@@ -1,406 +0,0 @@
-/**
- * @fileoverview ShowCommand using Commander's native class pattern
- * Extends Commander.Command for better integration with the framework
- */
-
-import { Command } from 'commander';
-import chalk from 'chalk';
-import boxen from 'boxen';
-import { createTaskMasterCore, type Task, type TaskMasterCore } from '@tm/core';
-import type { StorageType } from '@tm/core/types';
-import * as ui from '../utils/ui.js';
-
-/**
- * Options interface for the show command
- */
-export interface ShowCommandOptions {
-	id?: string;
-	status?: string;
-	format?: 'text' | 'json';
-	silent?: boolean;
-	project?: string;
-}
-
-/**
- * Result type from show command
- */
-export interface ShowTaskResult {
-	task: Task | null;
-	found: boolean;
-	storageType: Exclude<StorageType, 'auto'>;
-}
-
-/**
- * Result type for multiple tasks
- */
-export interface ShowMultipleTasksResult {
-	tasks: Task[];
-	notFound: string[];
-	storageType: Exclude<StorageType, 'auto'>;
-}
-
-/**
- * ShowCommand extending Commander's Command class
- * This is a thin presentation layer over @tm/core
- */
-export class ShowCommand extends Command {
-	private tmCore?: TaskMasterCore;
-	private lastResult?: ShowTaskResult | ShowMultipleTasksResult;
-
-	constructor(name?: string) {
-		super(name || 'show');
-
-		// Configure the command
-		this.description('Display detailed information about one or more tasks')
-			.argument('[id]', 'Task ID(s) to show (comma-separated for multiple)')
-			.option(
-				'-i, --id <id>',
-				'Task ID(s) to show (comma-separated for multiple)'
-			)
-			.option('-s, --status <status>', 'Filter subtasks by status')
-			.option('-f, --format <format>', 'Output format (text, json)', 'text')
-			.option('--silent', 'Suppress output (useful for programmatic usage)')
-			.option('-p, --project <path>', 'Project root directory', process.cwd())
-			.action(
-				async (taskId: string | undefined, options: ShowCommandOptions) => {
-					await this.executeCommand(taskId, options);
-				}
-			);
-	}
-
-	/**
-	 * Execute the show command
-	 */
-	private async executeCommand(
-		taskId: string | undefined,
-		options: ShowCommandOptions
-	): Promise<void> {
-		try {
-			// Validate options
-			if (!this.validateOptions(options)) {
-				process.exit(1);
-			}
-
-			// Initialize tm-core
-			await this.initializeCore(options.project || process.cwd());
-
-			// Get the task ID from argument or option
-			const idArg = taskId || options.id;
-			if (!idArg) {
-				console.error(chalk.red('Error: Please provide a task ID'));
-				process.exit(1);
-			}
-
-			// Check if multiple IDs are provided (comma-separated)
-			const taskIds = idArg
-				.split(',')
-				.map((id) => id.trim())
-				.filter((id) => id.length > 0);
-
-			// Get tasks from core
-			const result =
-				taskIds.length > 1
-					? await this.getMultipleTasks(taskIds, options)
-					: await this.getSingleTask(taskIds[0], options);
-
-			// Store result for programmatic access
-			this.setLastResult(result);
-
-			// Display results
-			if (!options.silent) {
-				this.displayResults(result, options);
-			}
-		} catch (error: any) {
-			const msg = error?.getSanitizedDetails?.() ?? {
-				message: error?.message ?? String(error)
-			};
-			console.error(chalk.red(`Error: ${msg.message || 'Unexpected error'}`));
-			if (error.stack && process.env.DEBUG) {
-				console.error(chalk.gray(error.stack));
-			}
-			process.exit(1);
-		}
-	}
-
-	/**
-	 * Validate command options
-	 */
-	private validateOptions(options: ShowCommandOptions): boolean {
-		// Validate format
-		if (options.format && !['text', 'json'].includes(options.format)) {
-			console.error(chalk.red(`Invalid format: ${options.format}`));
-			console.error(chalk.gray(`Valid formats: text, json`));
-			return false;
-		}
-
-		return true;
-	}
-
-	/**
-	 * Initialize TaskMasterCore
-	 */
-	private async initializeCore(projectRoot: string): Promise<void> {
-		if (!this.tmCore) {
-			this.tmCore = await createTaskMasterCore({ projectPath: projectRoot });
-		}
-	}
-
-	/**
-	 * Get a single task from tm-core
-	 */
-	private async getSingleTask(
-		taskId: string,
-		_options: ShowCommandOptions
-	): Promise<ShowTaskResult> {
-		if (!this.tmCore) {
-			throw new Error('TaskMasterCore not initialized');
-		}
-
-		// Get the task
-		const task = await this.tmCore.getTask(taskId);
-
-		// Get storage type
-		const storageType = this.tmCore.getStorageType();
-
-		return {
-			task,
-			found: task !== null,
-			storageType: storageType as Exclude<StorageType, 'auto'>
-		};
-	}
-
-	/**
-	 * Get multiple tasks from tm-core
-	 */
-	private async getMultipleTasks(
-		taskIds: string[],
-		_options: ShowCommandOptions
-	): Promise<ShowMultipleTasksResult> {
-		if (!this.tmCore) {
-			throw new Error('TaskMasterCore not initialized');
-		}
-
-		const tasks: Task[] = [];
-		const notFound: string[] = [];
-
-		// Get each task individually
-		for (const taskId of taskIds) {
-			const task = await this.tmCore.getTask(taskId);
-			if (task) {
-				tasks.push(task);
-			} else {
-				notFound.push(taskId);
-			}
-		}
-
-		// Get storage type
-		const storageType = this.tmCore.getStorageType();
-
-		return {
-			tasks,
-			notFound,
-			storageType: storageType as Exclude<StorageType, 'auto'>
-		};
-	}
-
-	/**
-	 * Display results based on format
-	 */
-	private displayResults(
-		result: ShowTaskResult | ShowMultipleTasksResult,
-		options: ShowCommandOptions
-	): void {
-		const format = options.format || 'text';
-
-		switch (format) {
-			case 'json':
-				this.displayJson(result);
-				break;
-
-			case 'text':
-			default:
-				if ('task' in result) {
-					// Single task result
-					this.displaySingleTask(result, options);
-				} else {
-					// Multiple tasks result
-					this.displayMultipleTasks(result, options);
-				}
-				break;
-		}
-	}
-
-	/**
-	 * Display in JSON format
-	 */
-	private displayJson(result: ShowTaskResult | ShowMultipleTasksResult): void {
-		console.log(JSON.stringify(result, null, 2));
-	}
-
-	/**
-	 * Display a single task in text format
-	 */
-	private displaySingleTask(
-		result: ShowTaskResult,
-		options: ShowCommandOptions
-	): void {
-		if (!result.found || !result.task) {
-			console.log(
-				boxen(chalk.yellow(`Task not found!`), {
-					padding: { top: 0, bottom: 0, left: 1, right: 1 },
-					borderColor: 'yellow',
-					borderStyle: 'round',
-					margin: { top: 1 }
-				})
-			);
-			return;
-		}
-
-		const task = result.task;
-
-		// Header
-		console.log(
-			boxen(chalk.white.bold(`Task #${task.id} - ${task.title}`), {
-				padding: { top: 0, bottom: 0, left: 1, right: 1 },
-				borderColor: 'blue',
-				borderStyle: 'round',
-				margin: { top: 1 }
-			})
-		);
-
-		// Task details
-		console.log(
-			`\n${chalk.blue.bold('Status:')} ${ui.getStatusWithColor(task.status)}`
-		);
-		console.log(
-			`${chalk.blue.bold('Priority:')} ${ui.getPriorityWithColor(task.priority)}`
-		);
-
-		if (task.description) {
-			console.log(`\n${chalk.blue.bold('Description:')}`);
-			console.log(task.description);
-		}
-
-		if (task.details) {
-			console.log(`\n${chalk.blue.bold('Details:')}`);
-			console.log(task.details);
-		}
-
-		// Dependencies
-		if (task.dependencies && task.dependencies.length > 0) {
-			console.log(`\n${chalk.blue.bold('Dependencies:')}`);
-			task.dependencies.forEach((dep) => {
-				console.log(`  - ${chalk.cyan(dep)}`);
-			});
-		}
-
-		// Subtasks
-		if (task.subtasks && task.subtasks.length > 0) {
-			console.log(`\n${chalk.blue.bold('Subtasks:')}`);
-
-			// Filter subtasks by status if provided
-			const filteredSubtasks = options.status
-				? task.subtasks.filter((sub) => sub.status === options.status)
-				: task.subtasks;
-
-			if (filteredSubtasks.length === 0 && options.status) {
-				console.log(
-					chalk.gray(`  No subtasks with status '${options.status}'`)
-				);
-			} else {
-				filteredSubtasks.forEach((subtask) => {
-					console.log(
-						`  ${chalk.cyan(`${task.id}.${subtask.id}`)} ${ui.getStatusWithColor(subtask.status)} ${subtask.title}`
-					);
-					if (subtask.description) {
-						console.log(`    ${chalk.gray(subtask.description)}`);
-					}
-				});
-			}
-		}
-
-		if (task.testStrategy) {
-			console.log(`\n${chalk.blue.bold('Test Strategy:')}`);
-			console.log(task.testStrategy);
-		}
-
-		console.log(`\n${chalk.gray('Storage: ' + result.storageType)}`);
-	}
-
-	/**
-	 * Display multiple tasks in text format
-	 */
-	private displayMultipleTasks(
-		result: ShowMultipleTasksResult,
-		_options: ShowCommandOptions
-	): void {
-		// Header
-		ui.displayBanner(`Tasks (${result.tasks.length} found)`);
-
-		if (result.notFound.length > 0) {
-			console.log(chalk.yellow(`\n⚠ Not found: ${result.notFound.join(', ')}`));
-		}
-
-		if (result.tasks.length === 0) {
-			ui.displayWarning('No tasks found matching the criteria.');
-			return;
-		}
-
-		// Task table
-		console.log(chalk.blue.bold(`\n📋 Tasks:\n`));
-		console.log(
-			ui.createTaskTable(result.tasks, {
-				showSubtasks: true,
-				showDependencies: true
-			})
-		);
-
-		console.log(`\n${chalk.gray('Storage: ' + result.storageType)}`);
-	}
-
-	/**
-	 * Set the last result for programmatic access
-	 */
-	private setLastResult(
-		result: ShowTaskResult | ShowMultipleTasksResult
-	): void {
-		this.lastResult = result;
-	}
-
-	/**
-	 * Get the last result (for programmatic usage)
-	 */
-	getLastResult(): ShowTaskResult | ShowMultipleTasksResult | undefined {
-		return this.lastResult;
-	}
-
-	/**
-	 * Clean up resources
-	 */
-	async cleanup(): Promise<void> {
-		if (this.tmCore) {
-			await this.tmCore.close();
-			this.tmCore = undefined;
-		}
-	}
-
-	/**
-	 * Static method to register this command on an existing program
-	 * This is for gradual migration - allows commands.js to use this
-	 */
-	static registerOn(program: Command): Command {
-		const showCommand = new ShowCommand();
-		program.addCommand(showCommand);
-		return showCommand;
-	}
-
-	/**
-	 * Alternative registration that returns the command for chaining
-	 * Can also configure the command name if needed
-	 */
-	static register(program: Command, name?: string): ShowCommand {
-		const showCommand = new ShowCommand(name);
-		program.addCommand(showCommand);
-		return showCommand;
-	}
-}

apps/cli/src/index.ts

@@ -5,9 +5,6 @@
 
 // Commands
 export { ListTasksCommand } from './commands/list.command.js';
-export { ShowCommand } from './commands/show.command.js';
-export { AuthCommand } from './commands/auth.command.js';
-export { ContextCommand } from './commands/context.command.js';
 
 // UI utilities (for other commands to use)
 export * as ui from './utils/ui.js';

apps/cli/src/utils/ui.ts

@@ -6,7 +6,7 @@
 import chalk from 'chalk';
 import boxen from 'boxen';
 import Table from 'cli-table3';
-import type { Task, TaskStatus, TaskPriority } from '@tm/core/types';
+import type { Task, TaskStatus, TaskPriority } from '@tm/core';
 
 /**
  * Get colored status display with ASCII icons (matches scripts/modules/ui.js style)

apps/cli/tsup.config.ts

@@ -0,0 +1,15 @@
+import { defineConfig } from 'tsup';
+
+export default defineConfig({
+	entry: ['src/index.ts'],
+	format: ['esm'],
+	target: 'node18',
+	splitting: false,
+	sourcemap: true,
+	clean: true,
+	dts: true,
+	shims: true,
+	esbuildOptions(options) {
+		options.platform = 'node';
+	}
+});

apps/docs/package.json

@@ -5,9 +5,10 @@
 	"description": "Task Master documentation powered by Mintlify",
 	"scripts": {
 		"dev": "mintlify dev",
+		"build": "mintlify build",
 		"preview": "mintlify preview"
 	},
 	"devDependencies": {
-		"mintlify": "^4.2.111"
+		"mintlify": "^4.0.0"
 	}
 }

apps/extension/README.md

@@ -83,7 +83,6 @@ The extension automatically handles the Taskmaster MCP server connection:
 | **View Kanban Board** | `Ctrl/Cmd + Shift + P` → "Taskmaster: Show Board" |
 | **Change Task Status** | Drag task card to different column |
 | **View Task Details** | Click on any task card |
-| **Start Working on Task** | Click task → Click "Start Task" button in properties panel |
 | **Edit Task Content** | Click task → Use edit buttons in details panel |
 | **Add Subtasks** | Click the + button on parent task cards |
 | **Use AI Features** | Open task details → Click AI action buttons |
@@ -96,14 +95,6 @@ The extension automatically handles the Taskmaster MCP server connection:
 - ✅ **Done** - Completed tasks
 - ⏸️ **Deferred** - Postponed for later
 
-### **Built-in Development Tools**
-
-**Start Task Button** - Quickly begin working on any task:
-- **One-Click Terminal** - Opens a new VS Code terminal named after your task
-- **Context Awareness** - Terminal automatically opens in your workspace root
-- **Smart State Management** - Button is disabled for completed tasks
-- **Seamless Workflow** - Go from task planning to implementation instantly
-
 ### **AI-Powered Task Management**
 
 The extension integrates seamlessly with Taskmaster AI via MCP to provide:

apps/extension/esbuild.js

@@ -103,8 +103,8 @@ async function main() {
 		// This prevents the multiple React instances issue
 		// Ensure React is resolved from the workspace root to avoid duplicates
 		alias: {
-			react: path.resolve(__dirname, '../../node_modules/react'),
-			'react-dom': path.resolve(__dirname, '../../node_modules/react-dom')
+			react: path.resolve(__dirname, 'node_modules/react'),
+			'react-dom': path.resolve(__dirname, 'node_modules/react-dom')
 		},
 		define: {
 			'process.env.NODE_ENV': production ? '"production"' : '"development"',
@@ -135,8 +135,8 @@ async function main() {
 		jsxImportSource: 'react',
 		external: ['*.css'],
 		alias: {
-			react: path.resolve(__dirname, '../../node_modules/react'),
-			'react-dom': path.resolve(__dirname, '../../node_modules/react-dom')
+			react: path.resolve(__dirname, 'node_modules/react'),
+			'react-dom': path.resolve(__dirname, 'node_modules/react-dom')
 		},
 		define: {
 			'process.env.NODE_ENV': production ? '"production"' : '"development"',

apps/extension/package.json

@@ -229,7 +229,6 @@
 		"build": "npm run build:js && npm run build:css",
 		"build:js": "node ./esbuild.js --production",
 		"build:css": "npx @tailwindcss/cli -i ./src/webview/index.css -o ./dist/index.css --minify",
-		"dev": "npm run watch",
 		"package": "npm exec node ./package.mjs",
 		"package:direct": "node ./package.mjs",
 		"debug:env": "node ./debug-env.mjs",

apps/extension/src/components/TaskDetails/TaskMetadataSidebar.tsx

@@ -1,10 +1,9 @@
 import type React from 'react';
 import { useState, useEffect } from 'react';
 import { Button } from '@/components/ui/button';
-import { Loader2, Play } from 'lucide-react';
+import { Loader2 } from 'lucide-react';
 import { PriorityBadge } from './PriorityBadge';
 import type { TaskMasterTask } from '../../webview/types';
-import { useVSCodeContext } from '../../webview/contexts/VSCodeContext';
 
 interface TaskMetadataSidebarProps {
 	currentTask: TaskMasterTask;
@@ -29,12 +28,10 @@ export const TaskMetadataSidebar: React.FC<TaskMetadataSidebarProps> = ({
 	isRegenerating = false,
 	isAppending = false
 }) => {
-	const { vscode } = useVSCodeContext();
 	const [isLoadingComplexity, setIsLoadingComplexity] = useState(false);
 	const [mcpComplexityScore, setMcpComplexityScore] = useState<
 		number | undefined
 	>(undefined);
-	const [isStartingTask, setIsStartingTask] = useState(false);
 
 	// Get complexity score from task
 	const currentComplexityScore = complexity?.score;
@@ -100,29 +97,6 @@ export const TaskMetadataSidebar: React.FC<TaskMetadataSidebarProps> = ({
 		}
 	};
 
-	// Handle starting a task
-	const handleStartTask = () => {
-		if (!currentTask || isStartingTask) {
-			return;
-		}
-		
-		setIsStartingTask(true);
-		
-		// Send message to extension to open terminal
-		if (vscode) {
-			vscode.postMessage({
-				type: 'openTerminal',
-				taskId: currentTask.id,
-				taskTitle: currentTask.title
-			});
-		}
-		
-		// Reset loading state after a short delay
-		setTimeout(() => {
-			setIsStartingTask(false);
-		}, 500);
-	};
-
 	// Effect to handle complexity on task change
 	useEffect(() => {
 		if (currentTask?.id) {
@@ -310,24 +284,6 @@ export const TaskMetadataSidebar: React.FC<TaskMetadataSidebarProps> = ({
 					{currentTask.dependencies && currentTask.dependencies.length > 0 && (
 						<div className="border-b border-textSeparator-foreground" />
 					)}
-
-					{/* Start Task Button */}
-					<div className="mt-4">
-						<Button
-							onClick={handleStartTask}
-							variant="default"
-							size="sm"
-							className="w-full text-xs"
-							disabled={isRegenerating || isAppending || isStartingTask || currentTask?.status === 'done'}
-						>
-							{isStartingTask ? (
-								<Loader2 className="w-4 h-4 mr-2 animate-spin" />
-							) : (
-								<Play className="w-4 h-4 mr-2" />
-							)}
-							{isStartingTask ? 'Starting...' : 'Start Task'}
-						</Button>
-					</div>
 				</div>
 			</div>
 		</div>

apps/extension/src/services/webview-manager.ts

@@ -361,25 +361,6 @@ export class WebviewManager {
 					}
 					return;
 
-				case 'openTerminal':
-					// Open VS Code terminal for task execution
-					this.logger.info(`Opening terminal for task ${data.taskId}: ${data.taskTitle}`);
-					
-					try {
-						const terminal = vscode.window.createTerminal({
-							name: `Task ${data.taskId}: ${data.taskTitle}`,
-							cwd: this.workspaceRoot
-						});
-						terminal.show();
-						
-						this.logger.info('Terminal created and shown successfully');
-						response = { success: true };
-					} catch (error) {
-						this.logger.error('Failed to create terminal:', error);
-						response = { success: false, error: error.message };
-					}
-					break;
-
 				default:
 					throw new Error(`Unknown message type: ${type}`);
 			}

package.json

@@ -11,36 +11,23 @@
 	},
 	"workspaces": ["apps/*", "packages/*", "."],
 	"scripts": {
-		"build": "npm run build:build-config && tsup",
-		"dev": "tsup --watch",
-		"turbo:dev": "turbo dev",
-		"turbo:build": "turbo build",
-		"dev:main": "tsup --watch --onSuccess 'echo \"📦 Main package built\" && npm link'",
-		"dev:legacy": "npm run build:build-config && concurrently -n \"core,cli,main\" -c \"blue,green,yellow\" \"npm run dev:core\" \"npm run dev:cli\" \"npm run dev:main\"",
-		"dev:core": "npm run dev -w @tm/core",
-		"dev:cli": "npm run dev -w @tm/cli",
-		"build:packages": "turbo build --filter='./packages/*' --filter='./apps/*'",
-		"build:packages:parallel": "turbo build --filter='./packages/*' --filter='./apps/*'",
-		"build:build-config": "npm run build -w @tm/build-config",
-		"build:core": "npm run build -w @tm/core",
-		"build:cli": "npm run build -w @tm/cli",
-		"typecheck": "turbo typecheck",
-		"typecheck:all": "turbo typecheck",
-		"typecheck:core": "npm run typecheck -w @tm/core",
-		"typecheck:cli": "npm run typecheck -w @tm/cli",
+		"build": "npm run build:packages && tsup",
+		"dev": "npm run build:packages && npm link && (npm run dev:packages & tsup --watch --onSuccess 'echo Build complete && npm link')",
+		"dev:packages": "(cd packages/tm-core && npm run dev) & (cd apps/cli && npm run dev) & wait",
+		"dev:core": "cd packages/tm-core && npm run dev",
+		"dev:cli": "cd apps/cli && npm run dev",
+		"build:packages": "npm run build:core && npm run build:cli",
+		"build:core": "cd packages/tm-core && npm run build",
+		"build:cli": "cd apps/cli && npm run build",
 		"test": "node --experimental-vm-modules node_modules/.bin/jest",
-		"test:unit": "node --experimental-vm-modules node_modules/.bin/jest --testPathPattern=unit",
-		"test:integration": "node --experimental-vm-modules node_modules/.bin/jest --testPathPattern=integration",
 		"test:fails": "node --experimental-vm-modules node_modules/.bin/jest --onlyFailures",
 		"test:watch": "node --experimental-vm-modules node_modules/.bin/jest --watch",
 		"test:coverage": "node --experimental-vm-modules node_modules/.bin/jest --coverage",
-		"test:ci": "node --experimental-vm-modules node_modules/.bin/jest --coverage --ci",
 		"test:e2e": "./tests/e2e/run_e2e.sh",
 		"test:e2e-report": "./tests/e2e/run_e2e.sh --analyze-log",
 		"postpack": "chmod +x dist/task-master.js dist/mcp-server.js",
 		"changeset": "changeset",
 		"release": "changeset publish",
-		"publish-packages": "turbo run build lint test && changeset version && changeset publish",
 		"inspector": "npx @modelcontextprotocol/inspector node dist/mcp-server.js",
 		"mcp-server": "node dist/mcp-server.js",
 		"format-check": "biome format .",
@@ -62,7 +49,6 @@
 	"license": "MIT WITH Commons-Clause",
 	"dependencies": {
 		"@ai-sdk/amazon-bedrock": "^2.2.9",
-		"@tm/cli": "*",
 		"@ai-sdk/anthropic": "^1.2.10",
 		"@ai-sdk/azure": "^1.3.17",
 		"@ai-sdk/google": "^1.2.13",
@@ -81,7 +67,7 @@
 		"ajv": "^8.17.1",
 		"ajv-formats": "^3.0.1",
 		"boxen": "^8.0.1",
-		"chalk": "5.6.2",
+		"chalk": "^5.4.1",
 		"cli-highlight": "^2.1.11",
 		"cli-progress": "^3.12.0",
 		"cli-table3": "^0.6.5",
@@ -115,7 +101,6 @@
 	"engines": {
 		"node": ">=18.0.0"
 	},
-	"packageManager": "npm@10.9.2",
 	"repository": {
 		"type": "git",
 		"url": "git+https://github.com/eyaltoledano/claude-task-master.git"
@@ -134,10 +119,8 @@
 		"@changesets/changelog-github": "^0.5.1",
 		"@changesets/cli": "^2.28.1",
 		"@types/jest": "^29.5.14",
-		"concurrently": "^9.2.1",
-		"cross-env": "^10.0.0",
-		"dotenv-mono": "^1.5.1",
 		"execa": "^8.0.1",
+		"ink": "^5.0.1",
 		"jest": "^29.7.0",
 		"jest-environment-node": "^29.7.0",
 		"mock-fs": "^5.5.0",
@@ -145,7 +128,6 @@
 		"supertest": "^7.1.0",
 		"tsup": "^8.5.0",
 		"tsx": "^4.16.2",
-		"turbo": "^2.5.6",
 		"typescript": "^5.9.2"
 	}
 }

packages/build-config/package.json

@@ -1,29 +0,0 @@
-{
-	"name": "@tm/build-config",
-	"version": "1.0.0",
-	"description": "Shared build configuration for Task Master monorepo",
-	"type": "module",
-	"main": "./dist/tsup.base.js",
-	"types": "./dist/tsup.base.d.ts",
-	"exports": {
-		".": {
-			"types": "./dist/tsup.base.d.ts",
-			"import": "./dist/tsup.base.js"
-		}
-	},
-	"files": ["dist", "src"],
-	"keywords": ["build-config", "tsup", "monorepo"],
-	"author": "",
-	"license": "MIT",
-	"scripts": {
-		"build": "tsc",
-		"typecheck": "tsc --noEmit"
-	},
-	"devDependencies": {
-		"dotenv-mono": "^1.5.1",
-		"typescript": "^5.7.3"
-	},
-	"dependencies": {
-		"tsup": "^8.5.0"
-	}
-}

packages/build-config/src/tsup.base.ts

@@ -1,123 +0,0 @@
-/**
- * Base tsup configuration for Task Master monorepo
- * Provides shared configuration that can be extended by individual packages
- */
-import type { Options } from 'tsup';
-import * as dotenv from 'dotenv-mono';
-
-dotenv.load();
-
-console.log(
-	'TM_PUBLIC_BASE_DOMAIN:',
-	process.env.TM_PUBLIC_BASE_DOMAIN,
-	'TM_PUBLIC_SUPABASE_URL:',
-	process.env.TM_PUBLIC_SUPABASE_URL,
-	'TM_PUBLIC_SUPABASE_ANON_KEY:',
-	process.env.TM_PUBLIC_SUPABASE_ANON_KEY
-);
-
-const isProduction = process.env.NODE_ENV === 'production';
-const isDevelopment = !isProduction;
-
-const envVariables = {
-	TM_PUBLIC_BASE_DOMAIN: process.env.TM_PUBLIC_BASE_DOMAIN ?? '',
-	TM_PUBLIC_SUPABASE_URL: process.env.TM_PUBLIC_SUPABASE_URL ?? '',
-	TM_PUBLIC_SUPABASE_ANON_KEY: process.env.TM_PUBLIC_SUPABASE_ANON_KEY ?? ''
-};
-
-console.log('envVariables:', envVariables);
-
-/**
- * Environment helpers
- */
-export const env = {
-	isProduction,
-	isDevelopment,
-	NODE_ENV: process.env.NODE_ENV || 'development',
-	...envVariables
-};
-
-/**
- * Base tsup configuration for all packages
- * Since everything gets bundled into root dist/ anyway, use consistent settings
- */
-export const baseConfig: Partial<Options> = {
-	format: ['esm'],
-	target: 'node18',
-	sourcemap: isDevelopment,
-	clean: true,
-	dts: false,
-	minify: isProduction,
-	treeshake: isProduction,
-	splitting: false,
-	// Don't bundle any other dependencies (auto-external all node_modules)
-	external: [/^[^./]/],
-	env: envVariables,
-	esbuildOptions(options) {
-		options.platform = 'node';
-		// Allow importing TypeScript from JavaScript
-		options.resolveExtensions = ['.ts', '.js', '.mjs', '.json'];
-		// Better source mapping in development only
-		options.sourcesContent = isDevelopment;
-		// Keep original names for better debugging in development
-		options.keepNames = isDevelopment;
-	},
-	// Watch mode configuration for development
-	watch: false
-};
-
-/**
- * Legacy external modules list - kept for backwards compatibility
- * Note: When using tsup-node, this is not needed as it automatically
- * excludes dependencies and peerDependencies from package.json
- */
-export const commonExternals = [
-	// Native Node.js modules (for cases where tsup is used instead of tsup-node)
-	'fs',
-	'path',
-	'child_process',
-	'crypto',
-	'os',
-	'url',
-	'util',
-	'stream',
-	'http',
-	'https',
-	'events',
-	'assert',
-	'buffer',
-	'querystring',
-	'readline',
-	'zlib',
-	'tty',
-	'net',
-	'dgram',
-	'dns',
-	'tls',
-	'cluster',
-	'process',
-	'module'
-];
-
-/**
- * Utility function to merge configurations
- * Simplified for tsup-node usage
- */
-export function mergeConfig(
-	baseConfig: Partial<Options>,
-	overrides: Partial<Options>
-): Options {
-	return {
-		...baseConfig,
-		...overrides,
-		// Merge esbuildOptions
-		esbuildOptions(options, context) {
-			if (baseConfig.esbuildOptions) {
-				baseConfig.esbuildOptions(options, context);
-			}
-			if (overrides.esbuildOptions) {
-				overrides.esbuildOptions(options, context);
-			}
-		}
-	} as Options;
-}

packages/build-config/tsconfig.json

@@ -1,21 +0,0 @@
-{
-	"compilerOptions": {
-		"target": "ES2022",
-		"lib": ["ES2022"],
-		"module": "ESNext",
-		"moduleResolution": "bundler",
-		"allowSyntheticDefaultImports": true,
-		"esModuleInterop": true,
-		"baseUrl": ".",
-		"outDir": "dist",
-		"allowJs": true,
-		"strict": true,
-		"resolveJsonModule": true,
-		"isolatedModules": true,
-		"declaration": true,
-		"skipLibCheck": true,
-		"forceConsistentCasingInFileNames": true
-	},
-	"include": ["src/**/*"],
-	"exclude": ["node_modules", "dist"]
-}

packages/tm-core/package.json

@@ -1,60 +1,20 @@
 {
 	"name": "@tm/core",
 	"version": "1.0.0",
-	"private": true,
 	"description": "Core library for Task Master - TypeScript task management system",
 	"type": "module",
-	"types": "./src/index.ts",
+	"types": "./dist/index.d.ts",
 	"main": "./dist/index.js",
 	"exports": {
 		".": {
 			"types": "./src/index.ts",
-			"import": "./dist/index.js"
-		},
-		"./auth": {
-			"types": "./src/auth/index.ts",
-			"import": "./dist/auth/index.js"
-		},
-		"./storage": {
-			"types": "./src/storage/index.ts",
-			"import": "./dist/storage/index.js"
-		},
-		"./config": {
-			"types": "./src/config/index.ts",
-			"import": "./dist/config/index.js"
-		},
-		"./providers": {
-			"types": "./src/providers/index.ts",
-			"import": "./dist/providers/index.js"
-		},
-		"./services": {
-			"types": "./src/services/index.ts",
-			"import": "./dist/services/index.js"
-		},
-		"./errors": {
-			"types": "./src/errors/index.ts",
-			"import": "./dist/errors/index.js"
-		},
-		"./logger": {
-			"types": "./src/logger/index.ts",
-			"import": "./dist/logger/index.js"
-		},
-		"./types": {
-			"types": "./src/types/index.ts",
-			"import": "./dist/types/index.js"
-		},
-		"./interfaces": {
-			"types": "./src/interfaces/index.ts",
-			"import": "./dist/interfaces/index.js"
-		},
-		"./utils": {
-			"types": "./src/utils/index.ts",
-			"import": "./dist/utils/index.js"
+			"import": "./dist/index.js",
+			"require": "./dist/index.js"
 		}
 	},
 	"scripts": {
-		"build": "tsc",
-		"dev": "tsc --watch",
+		"build": "tsup",
+		"dev": "tsup --watch",
 		"test": "vitest run",
 		"test:watch": "vitest",
 		"test:coverage": "vitest run --coverage",
@@ -66,7 +26,6 @@
 		"typecheck": "tsc --noEmit"
 	},
 	"dependencies": {
-		"@supabase/supabase-js": "^2.57.4",
 		"zod": "^3.22.4"
 	},
 	"devDependencies": {
@@ -74,13 +33,14 @@
 		"@types/node": "^20.11.30",
 		"@vitest/coverage-v8": "^2.0.5",
 		"ts-node": "^10.9.2",
+		"tsup": "^8.0.2",
 		"typescript": "^5.4.3",
 		"vitest": "^2.0.5"
 	},
 	"engines": {
 		"node": ">=18.0.0"
 	},
-	"files": ["src", "README.md", "CHANGELOG.md"],
+	"files": ["dist", "README.md", "CHANGELOG.md"],
 	"keywords": ["task-management", "typescript", "ai", "prd", "parser"],
 	"author": "Task Master AI",
 	"license": "MIT"

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

@@ -1,150 +0,0 @@
-/**
- * Tests for AuthManager singleton behavior
- */
-
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-
-// Mock the logger to verify warnings (must be hoisted before SUT import)
-const mockLogger = {
-	warn: vi.fn(),
-	info: vi.fn(),
-	debug: vi.fn(),
-	error: vi.fn()
-};
-
-vi.mock('../logger/index.js', () => ({
-	getLogger: () => mockLogger
-}));
-
-// Spy on CredentialStore constructor to verify config propagation
-const CredentialStoreSpy = vi.fn();
-vi.mock('./credential-store.js', () => {
-	return {
-		CredentialStore: class {
-			constructor(config: any) {
-				CredentialStoreSpy(config);
-				this.getCredentials = vi.fn(() => null);
-			}
-			getCredentials() {
-				return null;
-			}
-			saveCredentials() {}
-			clearCredentials() {}
-			hasValidCredentials() {
-				return false;
-			}
-		}
-	};
-});
-
-// Mock OAuthService to avoid side effects
-vi.mock('./oauth-service.js', () => {
-	return {
-		OAuthService: class {
-			constructor() {}
-			authenticate() {
-				return Promise.resolve({});
-			}
-			getAuthorizationUrl() {
-				return null;
-			}
-		}
-	};
-});
-
-// Mock SupabaseAuthClient to avoid side effects
-vi.mock('../clients/supabase-client.js', () => {
-	return {
-		SupabaseAuthClient: class {
-			constructor() {}
-			refreshSession() {
-				return Promise.resolve({});
-			}
-			signOut() {
-				return Promise.resolve();
-			}
-		}
-	};
-});
-
-// Import SUT after mocks
-import { AuthManager } from './auth-manager.js';
-
-describe('AuthManager Singleton', () => {
-	beforeEach(() => {
-		// Reset singleton before each test
-		AuthManager.resetInstance();
-		vi.clearAllMocks();
-		CredentialStoreSpy.mockClear();
-	});
-
-	it('should return the same instance on multiple calls', () => {
-		const instance1 = AuthManager.getInstance();
-		const instance2 = AuthManager.getInstance();
-
-		expect(instance1).toBe(instance2);
-	});
-
-	it('should use config on first call', () => {
-		const config = {
-			baseUrl: 'https://test.auth.com',
-			configDir: '/test/config',
-			configFile: '/test/config/auth.json'
-		};
-
-		const instance = AuthManager.getInstance(config);
-		expect(instance).toBeDefined();
-
-		// Assert that CredentialStore was constructed with the provided config
-		expect(CredentialStoreSpy).toHaveBeenCalledTimes(1);
-		expect(CredentialStoreSpy).toHaveBeenCalledWith(config);
-
-		// Verify the config is passed to internal components through observable behavior
-		// getCredentials would look in the configured file path
-		const credentials = instance.getCredentials();
-		expect(credentials).toBeNull(); // File doesn't exist, but config was propagated correctly
-	});
-
-	it('should warn when config is provided after initialization', () => {
-		// Clear previous calls
-		mockLogger.warn.mockClear();
-
-		// First call with config
-		AuthManager.getInstance({ baseUrl: 'https://first.auth.com' });
-
-		// Second call with different config
-		AuthManager.getInstance({ baseUrl: 'https://second.auth.com' });
-
-		// Verify warning was logged
-		expect(mockLogger.warn).toHaveBeenCalledWith(
-			expect.stringMatching(/config.*after initialization.*ignored/i)
-		);
-	});
-
-	it('should not warn when no config is provided after initialization', () => {
-		// Clear previous calls
-		mockLogger.warn.mockClear();
-
-		// First call with config
-		AuthManager.getInstance({ configDir: '/test/config' });
-
-		// Second call without config
-		AuthManager.getInstance();
-
-		// Verify no warning was logged
-		expect(mockLogger.warn).not.toHaveBeenCalled();
-	});
-
-	it('should allow resetting the instance', () => {
-		const instance1 = AuthManager.getInstance();
-
-		// Reset the instance
-		AuthManager.resetInstance();
-
-		// Get new instance
-		const instance2 = AuthManager.getInstance();
-
-		// They should be different instances
-		expect(instance1).not.toBe(instance2);
-	});
-});

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

@@ -1,280 +0,0 @@
-/**
- * Authentication manager for Task Master CLI
- */
-
-import {
-	AuthCredentials,
-	OAuthFlowOptions,
-	AuthenticationError,
-	AuthConfig,
-	UserContext
-} from './types.js';
-import { CredentialStore } from './credential-store.js';
-import { OAuthService } from './oauth-service.js';
-import { SupabaseAuthClient } from '../clients/supabase-client.js';
-import {
-	OrganizationService,
-	type Organization,
-	type Brief,
-	type RemoteTask
-} from '../services/organization.service.js';
-import { getLogger } from '../logger/index.js';
-
-/**
- * Authentication manager class
- */
-export class AuthManager {
-	private static instance: AuthManager | null = null;
-	private credentialStore: CredentialStore;
-	private oauthService: OAuthService;
-	private supabaseClient: SupabaseAuthClient;
-	private organizationService?: OrganizationService;
-
-	private constructor(config?: Partial<AuthConfig>) {
-		this.credentialStore = CredentialStore.getInstance(config);
-		this.supabaseClient = new SupabaseAuthClient();
-		this.oauthService = new OAuthService(this.credentialStore, config);
-
-		// Initialize Supabase client with session restoration
-		this.initializeSupabaseSession();
-	}
-
-	/**
-	 * Initialize Supabase session from stored credentials
-	 */
-	private async initializeSupabaseSession(): Promise<void> {
-		try {
-			await this.supabaseClient.initialize();
-		} catch (error) {
-			// Log but don't throw - session might not exist yet
-			const logger = getLogger('AuthManager');
-			logger.debug('No existing session to restore');
-		}
-	}
-
-	/**
-	 * Get singleton instance
-	 */
-	static getInstance(config?: Partial<AuthConfig>): AuthManager {
-		if (!AuthManager.instance) {
-			AuthManager.instance = new AuthManager(config);
-		} else if (config) {
-			// Warn if config is provided after initialization
-			const logger = getLogger('AuthManager');
-			logger.warn(
-				'getInstance called with config after initialization; config is ignored.'
-			);
-		}
-		return AuthManager.instance;
-	}
-
-	/**
-	 * Reset the singleton instance (useful for testing)
-	 */
-	static resetInstance(): void {
-		AuthManager.instance = null;
-		CredentialStore.resetInstance();
-	}
-
-	/**
-	 * Get stored authentication credentials
-	 */
-	getCredentials(): AuthCredentials | null {
-		return this.credentialStore.getCredentials();
-	}
-
-	/**
-	 * Start OAuth 2.0 Authorization Code Flow with browser handling
-	 */
-	async authenticateWithOAuth(
-		options: OAuthFlowOptions = {}
-	): Promise<AuthCredentials> {
-		return this.oauthService.authenticate(options);
-	}
-
-	/**
-	 * Get the authorization URL (for browser opening)
-	 */
-	getAuthorizationUrl(): string | null {
-		return this.oauthService.getAuthorizationUrl();
-	}
-
-	/**
-	 * Refresh authentication token using Supabase session
-	 */
-	async refreshToken(): Promise<AuthCredentials> {
-		try {
-			// Use Supabase's built-in session refresh
-			const session = await this.supabaseClient.refreshSession();
-
-			if (!session) {
-				throw new AuthenticationError(
-					'Failed to refresh session',
-					'REFRESH_FAILED'
-				);
-			}
-
-			// Get existing credentials to preserve context
-			const existingCredentials = this.credentialStore.getCredentials({
-				allowExpired: true
-			});
-
-			// Update authentication data from session
-			const newAuthData: AuthCredentials = {
-				token: session.access_token,
-				refreshToken: session.refresh_token,
-				userId: session.user.id,
-				email: session.user.email,
-				expiresAt: session.expires_at
-					? new Date(session.expires_at * 1000).toISOString()
-					: undefined,
-				savedAt: new Date().toISOString(),
-				selectedContext: existingCredentials?.selectedContext
-			};
-
-			this.credentialStore.saveCredentials(newAuthData);
-			return newAuthData;
-		} catch (error) {
-			if (error instanceof AuthenticationError) {
-				throw error;
-			}
-			throw new AuthenticationError(
-				`Token refresh failed: ${(error as Error).message}`,
-				'REFRESH_FAILED'
-			);
-		}
-	}
-
-	/**
-	 * Logout and clear credentials
-	 */
-	async logout(): Promise<void> {
-		try {
-			// First try to sign out from Supabase to revoke tokens
-			await this.supabaseClient.signOut();
-		} catch (error) {
-			// Log but don't throw - we still want to clear local credentials
-			getLogger('AuthManager').warn('Failed to sign out from Supabase:', error);
-		}
-
-		// Always clear local credentials (removes auth.json file)
-		this.credentialStore.clearCredentials();
-	}
-
-	/**
-	 * Check if authenticated
-	 */
-	isAuthenticated(): boolean {
-		return this.credentialStore.hasValidCredentials();
-	}
-
-	/**
-	 * Get the current user context (org/brief selection)
-	 */
-	getContext(): UserContext | null {
-		const credentials = this.getCredentials();
-		return credentials?.selectedContext || null;
-	}
-
-	/**
-	 * Update the user context (org/brief selection)
-	 */
-	async updateContext(context: Partial<UserContext>): Promise<void> {
-		const credentials = this.getCredentials();
-		if (!credentials) {
-			throw new AuthenticationError('Not authenticated', 'NOT_AUTHENTICATED');
-		}
-
-		// Merge with existing context
-		const existingContext = credentials.selectedContext || {};
-		const newContext: UserContext = {
-			...existingContext,
-			...context,
-			updatedAt: new Date().toISOString()
-		};
-
-		// Save updated credentials with new context
-		const updatedCredentials: AuthCredentials = {
-			...credentials,
-			selectedContext: newContext
-		};
-
-		this.credentialStore.saveCredentials(updatedCredentials);
-	}
-
-	/**
-	 * Clear the user context
-	 */
-	async clearContext(): Promise<void> {
-		const credentials = this.getCredentials();
-		if (!credentials) {
-			throw new AuthenticationError('Not authenticated', 'NOT_AUTHENTICATED');
-		}
-
-		// Remove context from credentials
-		const { selectedContext, ...credentialsWithoutContext } = credentials;
-		this.credentialStore.saveCredentials(credentialsWithoutContext);
-	}
-
-	/**
-	 * Get the organization service instance
-	 * Uses the Supabase client with the current session or token
-	 */
-	private async getOrganizationService(): Promise<OrganizationService> {
-		if (!this.organizationService) {
-			// First check if we have credentials with a token
-			const credentials = this.getCredentials();
-			if (!credentials || !credentials.token) {
-				throw new AuthenticationError('Not authenticated', 'NOT_AUTHENTICATED');
-			}
-
-			// Initialize session if needed (this will load from our storage adapter)
-			await this.supabaseClient.initialize();
-
-			// Use the SupabaseAuthClient which now has the session
-			const supabaseClient = this.supabaseClient.getClient();
-			this.organizationService = new OrganizationService(supabaseClient as any);
-		}
-		return this.organizationService;
-	}
-
-	/**
-	 * Get all organizations for the authenticated user
-	 */
-	async getOrganizations(): Promise<Organization[]> {
-		const service = await this.getOrganizationService();
-		return service.getOrganizations();
-	}
-
-	/**
-	 * Get all briefs for a specific organization
-	 */
-	async getBriefs(orgId: string): Promise<Brief[]> {
-		const service = await this.getOrganizationService();
-		return service.getBriefs(orgId);
-	}
-
-	/**
-	 * Get a specific organization by ID
-	 */
-	async getOrganization(orgId: string): Promise<Organization | null> {
-		const service = await this.getOrganizationService();
-		return service.getOrganization(orgId);
-	}
-
-	/**
-	 * Get a specific brief by ID
-	 */
-	async getBrief(briefId: string): Promise<Brief | null> {
-		const service = await this.getOrganizationService();
-		return service.getBrief(briefId);
-	}
-
-	/**
-	 * Get all tasks for a specific brief
-	 */
-	async getTasks(briefId: string): Promise<RemoteTask[]> {
-		const service = await this.getOrganizationService();
-		return service.getTasks(briefId);
-	}
-}

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

@@ -1,37 +0,0 @@
-/**
- * Centralized authentication configuration
- */
-
-import os from 'os';
-import path from 'path';
-import { AuthConfig } from './types.js';
-
-// Single base domain for all URLs
-// Build-time: process.env.TM_PUBLIC_BASE_DOMAIN gets replaced by tsup's env option
-// Default: https://tryhamster.com for production
-const BASE_DOMAIN =
-	process.env.TM_PUBLIC_BASE_DOMAIN || // This gets replaced at build time by tsup
-	'https://tryhamster.com';
-
-/**
- * Default authentication configuration
- * All URL configuration is derived from the single BASE_DOMAIN
- */
-export const DEFAULT_AUTH_CONFIG: AuthConfig = {
-	// Base domain for all services
-	baseUrl: BASE_DOMAIN,
-
-	// Configuration directory and file paths
-	configDir: path.join(os.homedir(), '.taskmaster'),
-	configFile: path.join(os.homedir(), '.taskmaster', 'auth.json')
-};
-
-/**
- * Get merged configuration with optional overrides
- */
-export function getAuthConfig(overrides?: Partial<AuthConfig>): AuthConfig {
-	return {
-		...DEFAULT_AUTH_CONFIG,
-		...overrides
-	};
-}

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

@@ -1,575 +0,0 @@
-/**
- * Tests for CredentialStore with numeric and string timestamp handling
- */
-
-import { describe, it, expect, beforeEach, vi, afterEach } from 'vitest';
-import { CredentialStore } from './credential-store.js';
-import { AuthenticationError } from './types.js';
-import type { AuthCredentials } from './types.js';
-import fs from 'fs';
-import path from 'path';
-import os from 'os';
-
-// Mock fs module
-vi.mock('fs');
-
-// Mock logger
-const mockLogger = {
-	warn: vi.fn(),
-	info: vi.fn(),
-	debug: vi.fn(),
-	error: vi.fn()
-};
-
-vi.mock('../logger/index.js', () => ({
-	getLogger: () => mockLogger
-}));
-
-describe('CredentialStore', () => {
-	let store: CredentialStore;
-	const testDir = '/test/config';
-	const configFile = '/test/config/auth.json';
-
-	beforeEach(() => {
-		vi.clearAllMocks();
-		store = new CredentialStore({
-			configDir: testDir,
-			configFile: configFile,
-			baseUrl: 'https://api.test.com'
-		});
-	});
-
-	afterEach(() => {
-		vi.restoreAllMocks();
-	});
-
-	describe('getCredentials with timestamp migration', () => {
-		it('should handle string ISO timestamp correctly', () => {
-			const futureDate = new Date(Date.now() + 3600000); // 1 hour from now
-			const mockCredentials: AuthCredentials = {
-				token: 'test-token',
-				userId: 'user-123',
-				email: 'test@example.com',
-				expiresAt: futureDate.toISOString(),
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue(
-				JSON.stringify(mockCredentials)
-			);
-
-			const result = store.getCredentials();
-
-			expect(result).not.toBeNull();
-			expect(result?.token).toBe('test-token');
-			// The timestamp should be normalized to numeric milliseconds
-			expect(typeof result?.expiresAt).toBe('number');
-			expect(result?.expiresAt).toBe(futureDate.getTime());
-		});
-
-		it('should handle numeric timestamp correctly', () => {
-			const futureTimestamp = Date.now() + 7200000; // 2 hours from now
-			const mockCredentials = {
-				token: 'test-token',
-				userId: 'user-456',
-				email: 'test2@example.com',
-				expiresAt: futureTimestamp,
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue(
-				JSON.stringify(mockCredentials)
-			);
-
-			const result = store.getCredentials();
-
-			expect(result).not.toBeNull();
-			expect(result?.token).toBe('test-token');
-			// Numeric timestamp should remain as-is
-			expect(typeof result?.expiresAt).toBe('number');
-			expect(result?.expiresAt).toBe(futureTimestamp);
-		});
-
-		it('should reject invalid string timestamp', () => {
-			const mockCredentials = {
-				token: 'test-token',
-				userId: 'user-789',
-				expiresAt: 'invalid-date-string',
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue(
-				JSON.stringify(mockCredentials)
-			);
-
-			const result = store.getCredentials();
-
-			expect(result).toBeNull();
-			expect(mockLogger.warn).toHaveBeenCalledWith(
-				'No valid expiration time provided for token'
-			);
-		});
-
-		it('should reject NaN timestamp', () => {
-			const mockCredentials = {
-				token: 'test-token',
-				userId: 'user-nan',
-				expiresAt: NaN,
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue(
-				JSON.stringify(mockCredentials)
-			);
-
-			const result = store.getCredentials();
-
-			expect(result).toBeNull();
-			expect(mockLogger.warn).toHaveBeenCalledWith(
-				'No valid expiration time provided for token'
-			);
-		});
-
-		it('should reject Infinity timestamp', () => {
-			const mockCredentials = {
-				token: 'test-token',
-				userId: 'user-inf',
-				expiresAt: Infinity,
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue(
-				JSON.stringify(mockCredentials)
-			);
-
-			const result = store.getCredentials();
-
-			expect(result).toBeNull();
-			expect(mockLogger.warn).toHaveBeenCalledWith(
-				'No valid expiration time provided for token'
-			);
-		});
-
-		it('should handle missing expiresAt field', () => {
-			const mockCredentials = {
-				token: 'test-token',
-				userId: 'user-no-expiry',
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-				// No expiresAt field
-			};
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue(
-				JSON.stringify(mockCredentials)
-			);
-
-			const result = store.getCredentials();
-
-			expect(result).toBeNull();
-			expect(mockLogger.warn).toHaveBeenCalledWith(
-				'No valid expiration time provided for token'
-			);
-		});
-
-		it('should check token expiration correctly', () => {
-			const expiredTimestamp = Date.now() - 3600000; // 1 hour ago
-			const mockCredentials = {
-				token: 'expired-token',
-				userId: 'user-expired',
-				expiresAt: expiredTimestamp,
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue(
-				JSON.stringify(mockCredentials)
-			);
-
-			const result = store.getCredentials();
-
-			expect(result).toBeNull();
-			expect(mockLogger.warn).toHaveBeenCalledWith(
-				expect.stringContaining('Authentication token has expired'),
-				expect.any(Object)
-			);
-		});
-
-		it('should allow expired tokens when requested', () => {
-			const expiredTimestamp = Date.now() - 3600000; // 1 hour ago
-			const mockCredentials = {
-				token: 'expired-token',
-				userId: 'user-expired',
-				expiresAt: expiredTimestamp,
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue(
-				JSON.stringify(mockCredentials)
-			);
-
-			const result = store.getCredentials({ allowExpired: true });
-
-			expect(result).not.toBeNull();
-			expect(result?.token).toBe('expired-token');
-		});
-	});
-
-	describe('saveCredentials with timestamp normalization', () => {
-		beforeEach(() => {
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.mkdirSync).mockImplementation(() => undefined);
-			vi.mocked(fs.writeFileSync).mockImplementation(() => undefined);
-			vi.mocked(fs.renameSync).mockImplementation(() => undefined);
-		});
-
-		it('should normalize string timestamp to ISO string when saving', () => {
-			const futureDate = new Date(Date.now() + 3600000);
-			const credentials: AuthCredentials = {
-				token: 'test-token',
-				userId: 'user-123',
-				expiresAt: futureDate.toISOString(),
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			store.saveCredentials(credentials);
-
-			expect(fs.writeFileSync).toHaveBeenCalledWith(
-				expect.stringContaining('.tmp'),
-				expect.stringContaining('"expiresAt":'),
-				expect.any(Object)
-			);
-
-			// Check that the written data contains a valid ISO string
-			const writtenData = vi.mocked(fs.writeFileSync).mock
-				.calls[0][1] as string;
-			const parsed = JSON.parse(writtenData);
-			expect(typeof parsed.expiresAt).toBe('string');
-			expect(new Date(parsed.expiresAt).toISOString()).toBe(parsed.expiresAt);
-		});
-
-		it('should convert numeric timestamp to ISO string when saving', () => {
-			const futureTimestamp = Date.now() + 7200000;
-			const credentials: AuthCredentials = {
-				token: 'test-token',
-				userId: 'user-456',
-				expiresAt: futureTimestamp,
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			store.saveCredentials(credentials);
-
-			const writtenData = vi.mocked(fs.writeFileSync).mock
-				.calls[0][1] as string;
-			const parsed = JSON.parse(writtenData);
-			expect(typeof parsed.expiresAt).toBe('string');
-			expect(new Date(parsed.expiresAt).getTime()).toBe(futureTimestamp);
-		});
-
-		it('should reject invalid string timestamp when saving', () => {
-			const credentials: AuthCredentials = {
-				token: 'test-token',
-				userId: 'user-789',
-				expiresAt: 'invalid-date' as any,
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			let err: unknown;
-			try {
-				store.saveCredentials(credentials);
-			} catch (e) {
-				err = e;
-			}
-			expect(err).toBeInstanceOf(AuthenticationError);
-			expect((err as Error).message).toContain('Invalid expiresAt format');
-		});
-
-		it('should reject NaN timestamp when saving', () => {
-			const credentials: AuthCredentials = {
-				token: 'test-token',
-				userId: 'user-nan',
-				expiresAt: NaN as any,
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			let err: unknown;
-			try {
-				store.saveCredentials(credentials);
-			} catch (e) {
-				err = e;
-			}
-			expect(err).toBeInstanceOf(AuthenticationError);
-			expect((err as Error).message).toContain('Invalid expiresAt format');
-		});
-
-		it('should reject Infinity timestamp when saving', () => {
-			const credentials: AuthCredentials = {
-				token: 'test-token',
-				userId: 'user-inf',
-				expiresAt: Infinity as any,
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			let err: unknown;
-			try {
-				store.saveCredentials(credentials);
-			} catch (e) {
-				err = e;
-			}
-			expect(err).toBeInstanceOf(AuthenticationError);
-			expect((err as Error).message).toContain('Invalid expiresAt format');
-		});
-
-		it('should handle missing expiresAt when saving', () => {
-			const credentials: AuthCredentials = {
-				token: 'test-token',
-				userId: 'user-no-expiry',
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-				// No expiresAt
-			};
-
-			store.saveCredentials(credentials);
-
-			const writtenData = vi.mocked(fs.writeFileSync).mock
-				.calls[0][1] as string;
-			const parsed = JSON.parse(writtenData);
-			expect(parsed.expiresAt).toBeUndefined();
-		});
-
-		it('should not mutate the original credentials object', () => {
-			const originalTimestamp = Date.now() + 3600000;
-			const credentials: AuthCredentials = {
-				token: 'test-token',
-				userId: 'user-123',
-				expiresAt: originalTimestamp,
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			const originalCredentialsCopy = { ...credentials };
-
-			store.saveCredentials(credentials);
-
-			// Original object should not be modified
-			expect(credentials).toEqual(originalCredentialsCopy);
-			expect(credentials.expiresAt).toBe(originalTimestamp);
-		});
-	});
-
-	describe('corrupt file handling', () => {
-		it('should quarantine corrupt file on JSON parse error', () => {
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue('invalid json {');
-			vi.mocked(fs.renameSync).mockImplementation(() => undefined);
-
-			const result = store.getCredentials();
-
-			expect(result).toBeNull();
-			expect(fs.renameSync).toHaveBeenCalledWith(
-				configFile,
-				expect.stringContaining('.corrupt-')
-			);
-			expect(mockLogger.warn).toHaveBeenCalledWith(
-				expect.stringContaining('Quarantined corrupt auth file')
-			);
-		});
-
-		it('should handle quarantine failure gracefully', () => {
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue('invalid json {');
-			vi.mocked(fs.renameSync).mockImplementation(() => {
-				throw new Error('Permission denied');
-			});
-
-			const result = store.getCredentials();
-
-			expect(result).toBeNull();
-			expect(mockLogger.debug).toHaveBeenCalledWith(
-				expect.stringContaining('Could not quarantine corrupt file')
-			);
-		});
-	});
-
-	describe('clearCredentials', () => {
-		it('should delete the auth file when it exists', () => {
-			// Mock file exists
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.unlinkSync).mockImplementation(() => undefined);
-
-			store.clearCredentials();
-
-			expect(fs.existsSync).toHaveBeenCalledWith('/test/config/auth.json');
-			expect(fs.unlinkSync).toHaveBeenCalledWith('/test/config/auth.json');
-		});
-
-		it('should not throw when auth file does not exist', () => {
-			// Mock file does not exist
-			vi.mocked(fs.existsSync).mockReturnValue(false);
-
-			// Should not throw
-			expect(() => store.clearCredentials()).not.toThrow();
-
-			// Should not try to unlink non-existent file
-			expect(fs.unlinkSync).not.toHaveBeenCalled();
-		});
-
-		it('should throw AuthenticationError when unlink fails', () => {
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.unlinkSync).mockImplementation(() => {
-				throw new Error('Permission denied');
-			});
-
-			let err: unknown;
-			try {
-				store.clearCredentials();
-			} catch (e) {
-				err = e;
-			}
-
-			expect(err).toBeInstanceOf(AuthenticationError);
-			expect((err as Error).message).toContain('Failed to clear credentials');
-			expect((err as Error).message).toContain('Permission denied');
-		});
-	});
-
-	describe('hasValidCredentials', () => {
-		it('should return true when valid unexpired credentials exist', () => {
-			const futureDate = new Date(Date.now() + 3600000); // 1 hour from now
-			const credentials = {
-				token: 'valid-token',
-				userId: 'user-123',
-				expiresAt: futureDate.toISOString(),
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify(credentials));
-
-			expect(store.hasValidCredentials()).toBe(true);
-		});
-
-		it('should return false when credentials are expired', () => {
-			const pastDate = new Date(Date.now() - 3600000); // 1 hour ago
-			const credentials = {
-				token: 'expired-token',
-				userId: 'user-123',
-				expiresAt: pastDate.toISOString(),
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify(credentials));
-
-			expect(store.hasValidCredentials()).toBe(false);
-		});
-
-		it('should return false when no credentials exist', () => {
-			vi.mocked(fs.existsSync).mockReturnValue(false);
-
-			expect(store.hasValidCredentials()).toBe(false);
-		});
-
-		it('should return false when file contains invalid JSON', () => {
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue('invalid json {');
-			vi.mocked(fs.renameSync).mockImplementation(() => undefined);
-
-			expect(store.hasValidCredentials()).toBe(false);
-		});
-
-		it('should return false for credentials without expiry', () => {
-			const credentials = {
-				token: 'no-expiry-token',
-				userId: 'user-123',
-				tokenType: 'standard',
-				savedAt: new Date().toISOString()
-			};
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readFileSync).mockReturnValue(JSON.stringify(credentials));
-
-			// Credentials without expiry are considered invalid
-			expect(store.hasValidCredentials()).toBe(false);
-
-			// Should log warning about missing expiration
-			expect(mockLogger.warn).toHaveBeenCalledWith(
-				'No valid expiration time provided for token'
-			);
-		});
-
-		it('should use allowExpired=false by default', () => {
-			// Spy on getCredentials to verify it's called with correct params
-			const getCredentialsSpy = vi.spyOn(store, 'getCredentials');
-
-			vi.mocked(fs.existsSync).mockReturnValue(false);
-			store.hasValidCredentials();
-
-			expect(getCredentialsSpy).toHaveBeenCalledWith({ allowExpired: false });
-		});
-	});
-
-	describe('cleanupCorruptFiles', () => {
-		it('should remove old corrupt files', () => {
-			const now = Date.now();
-			const oldFile = 'auth.json.corrupt-' + (now - 8 * 24 * 60 * 60 * 1000); // 8 days old
-			const newFile = 'auth.json.corrupt-' + (now - 1000); // 1 second old
-
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readdirSync).mockReturnValue([
-				{ name: oldFile, isFile: () => true },
-				{ name: newFile, isFile: () => true },
-				{ name: 'auth.json', isFile: () => true }
-			] as any);
-			vi.mocked(fs.statSync).mockImplementation((filePath) => {
-				if (filePath.includes(oldFile)) {
-					return { mtimeMs: now - 8 * 24 * 60 * 60 * 1000 } as any;
-				}
-				return { mtimeMs: now - 1000 } as any;
-			});
-			vi.mocked(fs.unlinkSync).mockImplementation(() => undefined);
-
-			store.cleanupCorruptFiles();
-
-			expect(fs.unlinkSync).toHaveBeenCalledWith(
-				expect.stringContaining(oldFile)
-			);
-			expect(fs.unlinkSync).not.toHaveBeenCalledWith(
-				expect.stringContaining(newFile)
-			);
-		});
-
-		it('should handle cleanup errors gracefully', () => {
-			vi.mocked(fs.existsSync).mockReturnValue(true);
-			vi.mocked(fs.readdirSync).mockImplementation(() => {
-				throw new Error('Permission denied');
-			});
-
-			// Should not throw
-			expect(() => store.cleanupCorruptFiles()).not.toThrow();
-			expect(mockLogger.debug).toHaveBeenCalledWith(
-				expect.stringContaining('Error during corrupt file cleanup')
-			);
-		});
-	});
-});

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

@@ -1,265 +0,0 @@
-/**
- * Credential storage and management
- */
-
-import fs from 'fs';
-import path from 'path';
-import { AuthCredentials, AuthenticationError, AuthConfig } from './types.js';
-import { getAuthConfig } from './config.js';
-import { getLogger } from '../logger/index.js';
-
-/**
- * CredentialStore manages the persistence and retrieval of authentication credentials.
- *
- * Runtime vs Persisted Shape:
- * - When retrieved (getCredentials): expiresAt is normalized to number (milliseconds since epoch)
- * - When persisted (saveCredentials): expiresAt is stored as ISO string for readability
- *
- * This normalization ensures consistent runtime behavior while maintaining
- * human-readable persisted format in the auth.json file.
- */
-export class CredentialStore {
-	private static instance: CredentialStore | null = null;
-	private logger = getLogger('CredentialStore');
-	private config: AuthConfig;
-	// Clock skew tolerance for expiry checks (30 seconds)
-	private readonly CLOCK_SKEW_MS = 30_000;
-
-	private constructor(config?: Partial<AuthConfig>) {
-		this.config = getAuthConfig(config);
-	}
-
-	/**
-	 * Get the singleton instance of CredentialStore
-	 */
-	static getInstance(config?: Partial<AuthConfig>): CredentialStore {
-		if (!CredentialStore.instance) {
-			CredentialStore.instance = new CredentialStore(config);
-		} else if (config) {
-			// Warn if config is provided after initialization
-			const logger = getLogger('CredentialStore');
-			logger.warn(
-				'getInstance called with config after initialization; config is ignored.'
-			);
-		}
-		return CredentialStore.instance;
-	}
-
-	/**
-	 * Reset the singleton instance (useful for testing)
-	 */
-	static resetInstance(): void {
-		CredentialStore.instance = null;
-	}
-
-	/**
-	 * Get stored authentication credentials
-	 * @returns AuthCredentials with expiresAt as number (milliseconds) for runtime use
-	 */
-	getCredentials(options?: { allowExpired?: boolean }): AuthCredentials | null {
-		try {
-			if (!fs.existsSync(this.config.configFile)) {
-				return null;
-			}
-
-			const authData = JSON.parse(
-				fs.readFileSync(this.config.configFile, 'utf-8')
-			) as AuthCredentials;
-
-			// Normalize/migrate timestamps to numeric (handles both number and ISO string)
-			let expiresAtMs: number | undefined;
-			if (typeof authData.expiresAt === 'number') {
-				expiresAtMs = Number.isFinite(authData.expiresAt)
-					? authData.expiresAt
-					: undefined;
-			} else if (typeof authData.expiresAt === 'string') {
-				const parsed = Date.parse(authData.expiresAt);
-				expiresAtMs = Number.isNaN(parsed) ? undefined : parsed;
-			} else {
-				expiresAtMs = undefined;
-			}
-
-			// Validate expiration time for tokens
-			if (expiresAtMs === undefined) {
-				this.logger.warn('No valid expiration time provided for token');
-				return null;
-			}
-
-			// Update the authData with normalized timestamp
-			authData.expiresAt = expiresAtMs;
-
-			// Check if the token has expired (with clock skew tolerance)
-			const now = Date.now();
-			const allowExpired = options?.allowExpired ?? false;
-			if (now >= expiresAtMs - this.CLOCK_SKEW_MS && !allowExpired) {
-				this.logger.warn(
-					'Authentication token has expired or is about to expire',
-					{
-						expiresAt: authData.expiresAt,
-						currentTime: new Date(now).toISOString(),
-						skewWindow: `${this.CLOCK_SKEW_MS / 1000}s`
-					}
-				);
-				return null;
-			}
-
-			// Return valid token
-			return authData;
-		} catch (error) {
-			this.logger.error(
-				`Failed to read auth credentials: ${(error as Error).message}`
-			);
-
-			// Quarantine corrupt file to prevent repeated errors
-			try {
-				if (fs.existsSync(this.config.configFile)) {
-					const corruptFile = `${this.config.configFile}.corrupt-${Date.now()}-${process.pid}-${Math.random().toString(36).slice(2, 8)}`;
-					fs.renameSync(this.config.configFile, corruptFile);
-					this.logger.warn(`Quarantined corrupt auth file to: ${corruptFile}`);
-				}
-			} catch (quarantineError) {
-				// If we can't quarantine, log but don't throw
-				this.logger.debug(
-					`Could not quarantine corrupt file: ${(quarantineError as Error).message}`
-				);
-			}
-
-			return null;
-		}
-	}
-
-	/**
-	 * Save authentication credentials
-	 * @param authData - Credentials with expiresAt as number or string (will be persisted as ISO string)
-	 */
-	saveCredentials(authData: AuthCredentials): void {
-		try {
-			// Ensure directory exists
-			if (!fs.existsSync(this.config.configDir)) {
-				fs.mkdirSync(this.config.configDir, { recursive: true, mode: 0o700 });
-			}
-
-			// Add timestamp without mutating caller's object
-			authData = { ...authData, savedAt: new Date().toISOString() };
-
-			// Validate and normalize expiresAt timestamp
-			if (authData.expiresAt !== undefined) {
-				let validTimestamp: number | undefined;
-
-				if (typeof authData.expiresAt === 'number') {
-					validTimestamp = Number.isFinite(authData.expiresAt)
-						? authData.expiresAt
-						: undefined;
-				} else if (typeof authData.expiresAt === 'string') {
-					const parsed = Date.parse(authData.expiresAt);
-					validTimestamp = Number.isNaN(parsed) ? undefined : parsed;
-				}
-
-				if (validTimestamp === undefined) {
-					throw new AuthenticationError(
-						`Invalid expiresAt format: ${authData.expiresAt}`,
-						'SAVE_FAILED'
-					);
-				}
-
-				// Store as ISO string for consistency
-				authData.expiresAt = new Date(validTimestamp).toISOString();
-			}
-
-			// Save credentials atomically with secure permissions
-			const tempFile = `${this.config.configFile}.tmp`;
-			fs.writeFileSync(tempFile, JSON.stringify(authData, null, 2), {
-				mode: 0o600
-			});
-			fs.renameSync(tempFile, this.config.configFile);
-		} catch (error) {
-			throw new AuthenticationError(
-				`Failed to save auth credentials: ${(error as Error).message}`,
-				'SAVE_FAILED',
-				error
-			);
-		}
-	}
-
-	/**
-	 * Clear stored credentials
-	 */
-	clearCredentials(): void {
-		try {
-			if (fs.existsSync(this.config.configFile)) {
-				fs.unlinkSync(this.config.configFile);
-			}
-		} catch (error) {
-			throw new AuthenticationError(
-				`Failed to clear credentials: ${(error as Error).message}`,
-				'CLEAR_FAILED',
-				error
-			);
-		}
-	}
-
-	/**
-	 * Check if credentials exist and are valid
-	 */
-	hasValidCredentials(): boolean {
-		const credentials = this.getCredentials({ allowExpired: false });
-		return credentials !== null;
-	}
-
-	/**
-	 * Get configuration
-	 */
-	getConfig(): AuthConfig {
-		return { ...this.config };
-	}
-
-	/**
-	 * Clean up old corrupt auth files
-	 * Removes corrupt files older than the specified age
-	 */
-	cleanupCorruptFiles(maxAgeMs: number = 7 * 24 * 60 * 60 * 1000): void {
-		try {
-			const dir = path.dirname(this.config.configFile);
-			const baseName = path.basename(this.config.configFile);
-			const prefix = `${baseName}.corrupt-`;
-
-			if (!fs.existsSync(dir)) {
-				return;
-			}
-
-			const entries = fs.readdirSync(dir, { withFileTypes: true });
-			const now = Date.now();
-
-			for (const entry of entries) {
-				if (!entry.isFile()) continue;
-				const file = entry.name;
-
-				// Check if file matches pattern: baseName.corrupt-{timestamp}
-				if (!file.startsWith(prefix)) continue;
-				const suffix = file.slice(prefix.length);
-				if (!/^\d+$/.test(suffix)) continue; // Fixed regex, not from variable input
-
-				const filePath = path.join(dir, file);
-				try {
-					const stats = fs.statSync(filePath);
-					const age = now - stats.mtimeMs;
-
-					if (age > maxAgeMs) {
-						fs.unlinkSync(filePath);
-						this.logger.debug(`Cleaned up old corrupt file: ${file}`);
-					}
-				} catch (error) {
-					// Ignore errors for individual file cleanup
-					this.logger.debug(
-						`Could not clean up corrupt file ${file}: ${(error as Error).message}`
-					);
-				}
-			}
-		} catch (error) {
-			// Log but don't throw - this is a cleanup operation
-			this.logger.debug(
-				`Error during corrupt file cleanup: ${(error as Error).message}`
-			);
-		}
-	}
-}

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

@@ -1,28 +0,0 @@
-/**
- * Authentication module exports
- */
-
-export { AuthManager } from './auth-manager.js';
-export { CredentialStore } from './credential-store.js';
-export { OAuthService } from './oauth-service.js';
-export { SupabaseSessionStorage } from './supabase-session-storage.js';
-export type {
-	Organization,
-	Brief,
-	RemoteTask
-} from '../services/organization.service.js';
-
-export type {
-	AuthCredentials,
-	OAuthFlowOptions,
-	AuthConfig,
-	CliData,
-	UserContext
-} from './types.js';
-
-export { AuthenticationError } from './types.js';
-
-export {
-	DEFAULT_AUTH_CONFIG,
-	getAuthConfig
-} from './config.js';

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

@@ -1,403 +0,0 @@
-/**
- * OAuth 2.0 Authorization Code Flow service
- */
-
-import http from 'http';
-import { URL } from 'url';
-import crypto from 'crypto';
-import os from 'os';
-import {
-	AuthCredentials,
-	AuthenticationError,
-	OAuthFlowOptions,
-	AuthConfig,
-	CliData
-} from './types.js';
-import { CredentialStore } from './credential-store.js';
-import { SupabaseAuthClient } from '../clients/supabase-client.js';
-import { getAuthConfig } from './config.js';
-import { getLogger } from '../logger/index.js';
-import packageJson from '../../../../package.json' with { type: 'json' };
-
-export class OAuthService {
-	private logger = getLogger('OAuthService');
-	private credentialStore: CredentialStore;
-	private supabaseClient: SupabaseAuthClient;
-	private baseUrl: string;
-	private authorizationUrl: string | null = null;
-	private originalState: string | null = null;
-	private authorizationReady: Promise<void> | null = null;
-	private resolveAuthorizationReady: (() => void) | null = null;
-
-	constructor(
-		credentialStore: CredentialStore,
-		config: Partial<AuthConfig> = {}
-	) {
-		this.credentialStore = credentialStore;
-		this.supabaseClient = new SupabaseAuthClient();
-		const authConfig = getAuthConfig(config);
-		this.baseUrl = authConfig.baseUrl;
-	}
-
-	/**
-	 * Start OAuth 2.0 Authorization Code Flow with browser handling
-	 */
-	async authenticate(options: OAuthFlowOptions = {}): Promise<AuthCredentials> {
-		const {
-			openBrowser,
-			timeout = 300000, // 5 minutes default
-			onAuthUrl,
-			onWaitingForAuth,
-			onSuccess,
-			onError
-		} = options;
-
-		try {
-			// Start the OAuth flow (starts local server)
-			const authPromise = this.startFlow(timeout);
-
-			// Wait for server to be ready and URL to be generated
-			if (this.authorizationReady) {
-				await this.authorizationReady;
-			}
-
-			// Get the authorization URL
-			const authUrl = this.getAuthorizationUrl();
-
-			if (!authUrl) {
-				throw new AuthenticationError(
-					'Failed to generate authorization URL',
-					'URL_GENERATION_FAILED'
-				);
-			}
-
-			// Notify about the auth URL
-			if (onAuthUrl) {
-				onAuthUrl(authUrl);
-			}
-
-			// Open browser if callback provided
-			if (openBrowser) {
-				try {
-					await openBrowser(authUrl);
-					this.logger.debug('Browser opened successfully with URL:', authUrl);
-				} catch (error) {
-					// Log the error but don't throw - user can still manually open the URL
-					this.logger.warn('Failed to open browser automatically:', error);
-				}
-			}
-
-			// Notify that we're waiting for authentication
-			if (onWaitingForAuth) {
-				onWaitingForAuth();
-			}
-
-			// Wait for authentication to complete
-			const credentials = await authPromise;
-
-			// Notify success
-			if (onSuccess) {
-				onSuccess(credentials);
-			}
-
-			return credentials;
-		} catch (error) {
-			const authError =
-				error instanceof AuthenticationError
-					? error
-					: new AuthenticationError(
-							`OAuth authentication failed: ${(error as Error).message}`,
-							'OAUTH_FAILED',
-							error
-						);
-
-			// Notify error
-			if (onError) {
-				onError(authError);
-			}
-
-			throw authError;
-		}
-	}
-
-	/**
-	 * Start the OAuth flow (internal implementation)
-	 */
-	private async startFlow(timeout: number = 300000): Promise<AuthCredentials> {
-		const state = this.generateState();
-
-		// Store the original state for verification
-		this.originalState = state;
-
-		// Create a promise that will resolve when the server is ready
-		this.authorizationReady = new Promise<void>((resolve) => {
-			this.resolveAuthorizationReady = resolve;
-		});
-
-		return new Promise((resolve, reject) => {
-			let timeoutId: NodeJS.Timeout;
-			// Create local HTTP server for OAuth callback
-			const server = http.createServer();
-
-			// Start server on localhost only, bind to port 0 for automatic port assignment
-			server.listen(0, '127.0.0.1', () => {
-				const address = server.address();
-				if (!address || typeof address === 'string') {
-					reject(new Error('Failed to get server address'));
-					return;
-				}
-				const port = address.port;
-				const callbackUrl = `http://localhost:${port}/callback`;
-
-				// Set up request handler after we know the port
-				server.on('request', async (req, res) => {
-					const url = new URL(req.url!, `http://127.0.0.1:${port}`);
-
-					if (url.pathname === '/callback') {
-						await this.handleCallback(
-							url,
-							res,
-							server,
-							resolve,
-							reject,
-							timeoutId
-						);
-					} else {
-						// Handle other paths (favicon, etc.)
-						res.writeHead(404);
-						res.end();
-					}
-				});
-
-				// Prepare CLI data object (server handles OAuth/PKCE)
-				const cliData: CliData = {
-					callback: callbackUrl,
-					state: state,
-					name: 'Task Master CLI',
-					version: this.getCliVersion(),
-					device: os.hostname(),
-					user: os.userInfo().username,
-					platform: os.platform(),
-					timestamp: Date.now()
-				};
-
-				// Build authorization URL for CLI-specific sign-in page
-				const authUrl = new URL(`${this.baseUrl}/auth/cli/sign-in`);
-
-				// Encode CLI data as base64
-				const cliParam = Buffer.from(JSON.stringify(cliData)).toString(
-					'base64'
-				);
-
-				// Set the single CLI parameter with all encoded data
-				authUrl.searchParams.append('cli', cliParam);
-
-				// Store auth URL for browser opening
-				this.authorizationUrl = authUrl.toString();
-
-				this.logger.info(
-					`OAuth session started - ${cliData.name} v${cliData.version} on port ${port}`
-				);
-				this.logger.debug('CLI data:', cliData);
-
-				// Signal that the server is ready and URL is available
-				if (this.resolveAuthorizationReady) {
-					this.resolveAuthorizationReady();
-					this.resolveAuthorizationReady = null;
-				}
-			});
-
-			// Set timeout for authentication
-			timeoutId = setTimeout(() => {
-				if (server.listening) {
-					server.close();
-					// Clean up the readiness promise if still pending
-					if (this.resolveAuthorizationReady) {
-						this.resolveAuthorizationReady();
-						this.resolveAuthorizationReady = null;
-					}
-					reject(
-						new AuthenticationError('Authentication timeout', 'AUTH_TIMEOUT')
-					);
-				}
-			}, timeout);
-		});
-	}
-
-	/**
-	 * Handle OAuth callback
-	 */
-	private async handleCallback(
-		url: URL,
-		res: http.ServerResponse,
-		server: http.Server,
-		resolve: (value: AuthCredentials) => void,
-		reject: (error: any) => void,
-		timeoutId?: NodeJS.Timeout
-	): Promise<void> {
-		// Server now returns tokens directly instead of code
-		const type = url.searchParams.get('type');
-		const returnedState = url.searchParams.get('state');
-		const accessToken = url.searchParams.get('access_token');
-		const refreshToken = url.searchParams.get('refresh_token');
-		const expiresIn = url.searchParams.get('expires_in');
-		const error = url.searchParams.get('error');
-		const errorDescription = url.searchParams.get('error_description');
-
-		// Server handles displaying success/failure, just close connection
-		res.writeHead(200);
-		res.end();
-
-		if (error) {
-			if (server.listening) {
-				server.close();
-			}
-			reject(
-				new AuthenticationError(
-					errorDescription || error || 'Authentication failed',
-					'OAUTH_ERROR'
-				)
-			);
-			return;
-		}
-
-		// Verify state parameter for CSRF protection
-		if (returnedState !== this.originalState) {
-			if (server.listening) {
-				server.close();
-			}
-			reject(
-				new AuthenticationError('Invalid state parameter', 'INVALID_STATE')
-			);
-			return;
-		}
-
-		// Handle authorization code for PKCE flow
-		const code = url.searchParams.get('code');
-		if (code && type === 'pkce_callback') {
-			try {
-				this.logger.info('Received authorization code for PKCE flow');
-
-				// Exchange code for session using PKCE
-				const session = await this.supabaseClient.exchangeCodeForSession(code);
-
-				// Save authentication data
-				const authData: AuthCredentials = {
-					token: session.access_token,
-					refreshToken: session.refresh_token,
-					userId: session.user.id,
-					email: session.user.email,
-					expiresAt: session.expires_at
-						? new Date(session.expires_at * 1000).toISOString()
-						: undefined,
-					tokenType: 'standard',
-					savedAt: new Date().toISOString()
-				};
-
-				this.credentialStore.saveCredentials(authData);
-
-				if (server.listening) {
-					server.close();
-				}
-				// Clear timeout since authentication succeeded
-				if (timeoutId) {
-					clearTimeout(timeoutId);
-				}
-				resolve(authData);
-				return;
-			} catch (error) {
-				if (server.listening) {
-					server.close();
-				}
-				reject(error);
-				return;
-			}
-		}
-
-		// Handle direct token response from server (legacy flow)
-		if (
-			accessToken &&
-			(type === 'oauth_success' || type === 'session_transfer')
-		) {
-			try {
-				this.logger.info(`Received tokens via ${type}`);
-
-				// Create a session with the tokens and set it in Supabase client
-				const session = {
-					access_token: accessToken,
-					refresh_token: refreshToken || '',
-					expires_at: expiresIn
-						? Math.floor(Date.now() / 1000) + parseInt(expiresIn)
-						: undefined,
-					expires_in: expiresIn ? parseInt(expiresIn) : undefined,
-					token_type: 'bearer',
-					user: null as any // Will be populated by setSession
-				};
-
-				// Set the session in Supabase client
-				await this.supabaseClient.setSession(session as any);
-
-				// Get user info from the session
-				const user = await this.supabaseClient.getUser();
-
-				// Calculate expiration time
-				const expiresAt = expiresIn
-					? new Date(Date.now() + parseInt(expiresIn) * 1000).toISOString()
-					: undefined;
-
-				// Save authentication data
-				const authData: AuthCredentials = {
-					token: accessToken,
-					refreshToken: refreshToken || undefined,
-					userId: user?.id || 'unknown',
-					email: user?.email,
-					expiresAt: expiresAt,
-					tokenType: 'standard',
-					savedAt: new Date().toISOString()
-				};
-
-				this.credentialStore.saveCredentials(authData);
-
-				if (server.listening) {
-					server.close();
-				}
-				// Clear timeout since authentication succeeded
-				if (timeoutId) {
-					clearTimeout(timeoutId);
-				}
-				resolve(authData);
-			} catch (error) {
-				if (server.listening) {
-					server.close();
-				}
-				reject(error);
-			}
-		} else {
-			if (server.listening) {
-				server.close();
-			}
-			reject(new AuthenticationError('No access token received', 'NO_TOKEN'));
-		}
-	}
-
-	/**
-	 * Generate state for OAuth flow
-	 */
-	private generateState(): string {
-		return crypto.randomBytes(32).toString('base64url');
-	}
-
-	/**
-	 * Get CLI version from package.json if available
-	 */
-	private getCliVersion(): string {
-		return packageJson.version || 'unknown';
-	}
-
-	/**
-	 * Get the authorization URL (for browser opening)
-	 */
-	getAuthorizationUrl(): string | null {
-		return this.authorizationUrl;
-	}
-}

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

@@ -1,155 +0,0 @@
-/**
- * Custom storage adapter for Supabase Auth sessions in CLI environment
- * Implements the SupportedStorage interface required by Supabase Auth
- *
- * This adapter bridges Supabase's session management with our existing
- * auth.json credential storage, maintaining backward compatibility
- */
-
-import { SupportedStorage } from '@supabase/supabase-js';
-import { CredentialStore } from './credential-store.js';
-import { AuthCredentials } from './types.js';
-import { getLogger } from '../logger/index.js';
-
-const STORAGE_KEY = 'sb-taskmaster-auth-token';
-
-export class SupabaseSessionStorage implements SupportedStorage {
-	private store: CredentialStore;
-	private logger = getLogger('SupabaseSessionStorage');
-
-	constructor(store: CredentialStore) {
-		this.store = store;
-	}
-
-	/**
-	 * Build a Supabase session object from our credentials
-	 */
-	private buildSessionFromCredentials(credentials: AuthCredentials): any {
-		// Create a session object that Supabase expects
-		const session = {
-			access_token: credentials.token,
-			refresh_token: credentials.refreshToken || '',
-			expires_at: credentials.expiresAt
-				? Math.floor(new Date(credentials.expiresAt).getTime() / 1000)
-				: Math.floor(Date.now() / 1000) + 3600, // Default to 1 hour
-			token_type: 'bearer',
-			user: {
-				id: credentials.userId,
-				email: credentials.email || '',
-				aud: 'authenticated',
-				role: 'authenticated',
-				email_confirmed_at: new Date().toISOString(),
-				app_metadata: {},
-				user_metadata: {},
-				created_at: new Date().toISOString(),
-				updated_at: new Date().toISOString()
-			}
-		};
-		return session;
-	}
-
-	/**
-	 * Parse a Supabase session back to our credentials
-	 */
-	private parseSessionToCredentials(
-		sessionData: any
-	): Partial<AuthCredentials> {
-		try {
-			const session = JSON.parse(sessionData);
-			return {
-				token: session.access_token,
-				refreshToken: session.refresh_token,
-				userId: session.user?.id || 'unknown',
-				email: session.user?.email,
-				expiresAt: session.expires_at
-					? new Date(session.expires_at * 1000).toISOString()
-					: undefined
-			};
-		} catch (error) {
-			this.logger.error('Error parsing session:', error);
-			return {};
-		}
-	}
-
-	/**
-	 * Get item from storage - Supabase will request the session with a specific key
-	 */
-	getItem(key: string): string | null {
-		// Supabase uses a specific key pattern for sessions
-		if (key === STORAGE_KEY || key.includes('auth-token')) {
-			try {
-				const credentials = this.store.getCredentials({ allowExpired: true });
-				if (credentials && credentials.token) {
-					// Build and return a session object from our stored credentials
-					const session = this.buildSessionFromCredentials(credentials);
-					return JSON.stringify(session);
-				}
-			} catch (error) {
-				this.logger.error('Error getting session:', error);
-			}
-		}
-		return null;
-	}
-
-	/**
-	 * Set item in storage - Supabase will store the session with a specific key
-	 */
-	setItem(key: string, value: string): void {
-		// Only handle Supabase session keys
-		if (key === STORAGE_KEY || key.includes('auth-token')) {
-			try {
-				// Parse the session and update our credentials
-				const sessionUpdates = this.parseSessionToCredentials(value);
-				const existingCredentials = this.store.getCredentials({
-					allowExpired: true
-				});
-
-				if (sessionUpdates.token) {
-					const updatedCredentials: AuthCredentials = {
-						...existingCredentials,
-						...sessionUpdates,
-						savedAt: new Date().toISOString(),
-						selectedContext: existingCredentials?.selectedContext
-					} as AuthCredentials;
-
-					this.store.saveCredentials(updatedCredentials);
-				}
-			} catch (error) {
-				this.logger.error('Error setting session:', error);
-			}
-		}
-	}
-
-	/**
-	 * Remove item from storage - Called when signing out
-	 */
-	removeItem(key: string): void {
-		if (key === STORAGE_KEY || key.includes('auth-token')) {
-			// Don't actually remove credentials, just clear the tokens
-			// This preserves other data like selectedContext
-			try {
-				const credentials = this.store.getCredentials({ allowExpired: true });
-				if (credentials) {
-					// Keep context but clear auth tokens
-					const clearedCredentials: AuthCredentials = {
-						...credentials,
-						token: '',
-						refreshToken: undefined,
-						expiresAt: undefined
-					} as AuthCredentials;
-					this.store.saveCredentials(clearedCredentials);
-				}
-			} catch (error) {
-				this.logger.error('Error removing session:', error);
-			}
-		}
-	}
-
-	/**
-	 * Clear all session data
-	 */
-	clear(): void {
-		// Clear auth tokens but preserve context
-		this.removeItem(STORAGE_KEY);
-	}
-}

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

@@ -1,100 +0,0 @@
-/**
- * Authentication types and interfaces
- */
-
-export interface AuthCredentials {
-	token: string;
-	refreshToken?: string;
-	userId: string;
-	email?: string;
-	expiresAt?: string | number;
-	tokenType?: 'standard';
-	savedAt: string;
-	selectedContext?: UserContext;
-}
-
-export interface UserContext {
-	orgId?: string;
-	orgName?: string;
-	briefId?: string;
-	briefName?: string;
-	updatedAt: string;
-}
-
-export interface OAuthFlowOptions {
-	/** Callback to open the browser with the auth URL. If not provided, browser won't be opened */
-	openBrowser?: (url: string) => Promise<void>;
-	/** Timeout for the OAuth flow in milliseconds. Default: 300000 (5 minutes) */
-	timeout?: number;
-	/** Callback to be invoked with the authorization URL */
-	onAuthUrl?: (url: string) => void;
-	/** Callback to be invoked when waiting for authentication */
-	onWaitingForAuth?: () => void;
-	/** Callback to be invoked on successful authentication */
-	onSuccess?: (credentials: AuthCredentials) => void;
-	/** Callback to be invoked on authentication error */
-	onError?: (error: AuthenticationError) => void;
-}
-
-export interface AuthConfig {
-	baseUrl: string;
-	configDir: string;
-	configFile: string;
-}
-
-export interface CliData {
-	callback: string;
-	state: string;
-	name: string;
-	version: string;
-	device?: string;
-	user?: string;
-	platform?: string;
-	timestamp?: number;
-}
-
-/**
- * Authentication error codes
- */
-export type AuthErrorCode =
-	| 'AUTH_TIMEOUT'
-	| 'AUTH_EXPIRED'
-	| 'OAUTH_FAILED'
-	| 'OAUTH_ERROR'
-	| 'OAUTH_CANCELED'
-	| 'URL_GENERATION_FAILED'
-	| 'INVALID_STATE'
-	| 'NO_TOKEN'
-	| 'TOKEN_EXCHANGE_FAILED'
-	| 'INVALID_CREDENTIALS'
-	| 'NO_REFRESH_TOKEN'
-	| 'NOT_AUTHENTICATED'
-	| 'NETWORK_ERROR'
-	| 'CONFIG_MISSING'
-	| 'SAVE_FAILED'
-	| 'CLEAR_FAILED'
-	| 'STORAGE_ERROR'
-	| 'NOT_SUPPORTED'
-	| 'REFRESH_FAILED'
-	| 'INVALID_RESPONSE'
-	| 'PKCE_INIT_FAILED'
-	| 'PKCE_FAILED'
-	| 'CODE_EXCHANGE_FAILED'
-	| 'SESSION_SET_FAILED';
-
-/**
- * Authentication error class
- */
-export class AuthenticationError extends Error {
-	constructor(
-		message: string,
-		public code: AuthErrorCode,
-		public cause?: unknown
-	) {
-		super(message);
-		this.name = 'AuthenticationError';
-		if (cause && cause instanceof Error) {
-			this.stack = `${this.stack}\nCaused by: ${cause.stack}`;
-		}
-	}
-}

packages/tm-core/src/clients/index.ts

@@ -1,5 +0,0 @@
-/**
- * Client exports
- */
-
-export { SupabaseAuthClient } from './supabase-client.js';

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

@@ -1,312 +0,0 @@
-/**
- * Supabase authentication client for CLI auth flows
- */
-
-import {
-	createClient,
-	SupabaseClient as SupabaseJSClient,
-	User,
-	Session
-} from '@supabase/supabase-js';
-import { AuthenticationError } from '../auth/types.js';
-import { getLogger } from '../logger/index.js';
-import { SupabaseSessionStorage } from '../auth/supabase-session-storage.js';
-import { CredentialStore } from '../auth/credential-store.js';
-
-export class SupabaseAuthClient {
-	private client: SupabaseJSClient | null = null;
-	private sessionStorage: SupabaseSessionStorage;
-	private logger = getLogger('SupabaseAuthClient');
-
-	constructor() {
-		const credentialStore = CredentialStore.getInstance();
-		this.sessionStorage = new SupabaseSessionStorage(credentialStore);
-	}
-
-	/**
-	 * Get Supabase client with proper session management
-	 */
-	getClient(): SupabaseJSClient {
-		if (!this.client) {
-			// Get Supabase configuration from environment - using TM_PUBLIC prefix
-			const supabaseUrl = process.env.TM_PUBLIC_SUPABASE_URL;
-			const supabaseAnonKey = process.env.TM_PUBLIC_SUPABASE_ANON_KEY;
-
-			if (!supabaseUrl || !supabaseAnonKey) {
-				throw new AuthenticationError(
-					'Supabase configuration missing. Please set TM_PUBLIC_SUPABASE_URL and TM_PUBLIC_SUPABASE_ANON_KEY environment variables.',
-					'CONFIG_MISSING'
-				);
-			}
-
-			// Create client with custom storage adapter (similar to React Native AsyncStorage)
-			this.client = createClient(supabaseUrl, supabaseAnonKey, {
-				auth: {
-					storage: this.sessionStorage,
-					autoRefreshToken: true,
-					persistSession: true,
-					detectSessionInUrl: false
-				}
-			});
-		}
-
-		return this.client;
-	}
-
-	/**
-	 * Initialize the client and restore session if available
-	 */
-	async initialize(): Promise<Session | null> {
-		const client = this.getClient();
-
-		try {
-			// Get the current session from storage
-			const {
-				data: { session },
-				error
-			} = await client.auth.getSession();
-
-			if (error) {
-				this.logger.warn('Failed to restore session:', error);
-				return null;
-			}
-
-			if (session) {
-				this.logger.info('Session restored successfully');
-			}
-
-			return session;
-		} catch (error) {
-			this.logger.error('Error initializing session:', error);
-			return null;
-		}
-	}
-
-	/**
-	 * Sign in with PKCE flow (for CLI auth)
-	 */
-	async signInWithPKCE(): Promise<{ url: string; codeVerifier: string }> {
-		const client = this.getClient();
-
-		try {
-			// Generate PKCE challenge
-			const { data, error } = await client.auth.signInWithOAuth({
-				provider: 'github',
-				options: {
-					redirectTo:
-						process.env.TM_AUTH_CALLBACK_URL ||
-						'http://localhost:3421/auth/callback',
-					scopes: 'email'
-				}
-			});
-
-			if (error) {
-				throw new AuthenticationError(
-					`Failed to initiate PKCE flow: ${error.message}`,
-					'PKCE_INIT_FAILED'
-				);
-			}
-
-			if (!data?.url) {
-				throw new AuthenticationError(
-					'No authorization URL returned',
-					'INVALID_RESPONSE'
-				);
-			}
-
-			// Extract code_verifier from the URL or generate it
-			// Note: Supabase handles PKCE internally, we just need to handle the callback
-			return {
-				url: data.url,
-				codeVerifier: '' // Supabase manages this internally
-			};
-		} catch (error) {
-			if (error instanceof AuthenticationError) {
-				throw error;
-			}
-
-			throw new AuthenticationError(
-				`Failed to start PKCE flow: ${(error as Error).message}`,
-				'PKCE_FAILED'
-			);
-		}
-	}
-
-	/**
-	 * Exchange authorization code for session (PKCE flow)
-	 */
-	async exchangeCodeForSession(code: string): Promise<Session> {
-		const client = this.getClient();
-
-		try {
-			const { data, error } = await client.auth.exchangeCodeForSession(code);
-
-			if (error) {
-				throw new AuthenticationError(
-					`Failed to exchange code: ${error.message}`,
-					'CODE_EXCHANGE_FAILED'
-				);
-			}
-
-			if (!data?.session) {
-				throw new AuthenticationError(
-					'No session returned from code exchange',
-					'INVALID_RESPONSE'
-				);
-			}
-
-			this.logger.info('Successfully exchanged code for session');
-			return data.session;
-		} catch (error) {
-			if (error instanceof AuthenticationError) {
-				throw error;
-			}
-
-			throw new AuthenticationError(
-				`Code exchange failed: ${(error as Error).message}`,
-				'CODE_EXCHANGE_FAILED'
-			);
-		}
-	}
-
-	/**
-	 * Get the current session
-	 */
-	async getSession(): Promise<Session | null> {
-		const client = this.getClient();
-
-		try {
-			const {
-				data: { session },
-				error
-			} = await client.auth.getSession();
-
-			if (error) {
-				this.logger.warn('Failed to get session:', error);
-				return null;
-			}
-
-			return session;
-		} catch (error) {
-			this.logger.error('Error getting session:', error);
-			return null;
-		}
-	}
-
-	/**
-	 * Refresh the current session
-	 */
-	async refreshSession(): Promise<Session | null> {
-		const client = this.getClient();
-
-		try {
-			this.logger.info('Refreshing session...');
-
-			// Supabase will automatically use the stored refresh token
-			const {
-				data: { session },
-				error
-			} = await client.auth.refreshSession();
-
-			if (error) {
-				this.logger.error('Failed to refresh session:', error);
-				throw new AuthenticationError(
-					`Failed to refresh session: ${error.message}`,
-					'REFRESH_FAILED'
-				);
-			}
-
-			if (session) {
-				this.logger.info('Successfully refreshed session');
-			}
-
-			return session;
-		} catch (error) {
-			if (error instanceof AuthenticationError) {
-				throw error;
-			}
-
-			throw new AuthenticationError(
-				`Failed to refresh session: ${(error as Error).message}`,
-				'REFRESH_FAILED'
-			);
-		}
-	}
-
-	/**
-	 * Get current user from session
-	 */
-	async getUser(): Promise<User | null> {
-		const client = this.getClient();
-
-		try {
-			const {
-				data: { user },
-				error
-			} = await client.auth.getUser();
-
-			if (error) {
-				this.logger.warn('Failed to get user:', error);
-				return null;
-			}
-
-			return user;
-		} catch (error) {
-			this.logger.error('Error getting user:', error);
-			return null;
-		}
-	}
-
-	/**
-	 * Sign out and clear session
-	 */
-	async signOut(): Promise<void> {
-		const client = this.getClient();
-
-		try {
-			// Sign out with global scope to revoke all refresh tokens
-			const { error } = await client.auth.signOut({ scope: 'global' });
-
-			if (error) {
-				this.logger.warn('Failed to sign out:', error);
-			}
-
-			// Clear cached session data
-			this.sessionStorage.clear();
-		} catch (error) {
-			this.logger.error('Error during sign out:', error);
-		}
-	}
-
-	/**
-	 * Set session from external auth (e.g., from server callback)
-	 */
-	async setSession(session: Session): Promise<void> {
-		const client = this.getClient();
-
-		try {
-			const { error } = await client.auth.setSession({
-				access_token: session.access_token,
-				refresh_token: session.refresh_token
-			});
-
-			if (error) {
-				throw new AuthenticationError(
-					`Failed to set session: ${error.message}`,
-					'SESSION_SET_FAILED'
-				);
-			}
-
-			this.logger.info('Session set successfully');
-		} catch (error) {
-			if (error instanceof AuthenticationError) {
-				throw error;
-			}
-
-			throw new AuthenticationError(
-				`Failed to set session: ${(error as Error).message}`,
-				'SESSION_SET_FAILED'
-			);
-		}
-	}
-}

packages/tm-core/src/config/config-manager.spec.ts

@@ -251,11 +251,11 @@ describe('ConfigManager', () => {
 			expect(manager.getProjectRoot()).toBe(testProjectRoot);
 		});
 
-		it('should check if API is explicitly configured', () => {
-			expect(manager.isApiExplicitlyConfigured()).toBe(false);
+		it('should check if using API storage', () => {
+			expect(manager.isUsingApiStorage()).toBe(false);
 		});
 
-		it('should detect when API is explicitly configured', () => {
+		it('should detect API storage', () => {
 			// Update config for current instance
 			(manager as any).config = {
 				storage: {
@@ -265,7 +265,7 @@ describe('ConfigManager', () => {
 				}
 			};
 
-			expect(manager.isApiExplicitlyConfigured()).toBe(true);
+			expect(manager.isUsingApiStorage()).toBe(true);
 		});
 	});
 

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

@@ -6,10 +6,7 @@
  * maintainability, testability, and separation of concerns.
  */
 
-import type {
-	PartialConfiguration,
-	RuntimeStorageConfig
-} from '../interfaces/configuration.interface.js';
+import type { PartialConfiguration } from '../interfaces/configuration.interface.js';
 import { ConfigLoader } from './services/config-loader.service.js';
 import {
 	ConfigMerger,
@@ -137,28 +134,26 @@ export class ConfigManager {
 	/**
 	 * Get storage configuration
 	 */
-	getStorageConfig(): RuntimeStorageConfig {
+	getStorageConfig(): {
+		type: 'file' | 'api';
+		apiEndpoint?: string;
+		apiAccessToken?: string;
+	} {
 		const storage = this.config.storage;
 
-		// Return the configured type (including 'auto')
-		const storageType = storage?.type || 'auto';
-		const basePath = storage?.basePath ?? this.projectRoot;
-
-		if (storageType === 'api' || storageType === 'auto') {
+		if (
+			storage?.type === 'api' &&
+			storage.apiEndpoint &&
+			storage.apiAccessToken
+		) {
 			return {
-				type: storageType,
-				basePath,
-				apiEndpoint: storage?.apiEndpoint,
-				apiAccessToken: storage?.apiAccessToken,
-				apiConfigured: Boolean(storage?.apiEndpoint || storage?.apiAccessToken)
+				type: 'api',
+				apiEndpoint: storage.apiEndpoint,
+				apiAccessToken: storage.apiAccessToken
 			};
 		}
 
-		return {
-			type: storageType,
-			basePath,
-			apiConfigured: false
-		};
+		return { type: 'file' };
 	}
 
 	/**
@@ -189,10 +184,9 @@ export class ConfigManager {
 	}
 
 	/**
-	 * Check if explicitly configured to use API storage
-	 * Excludes 'auto' type
+	 * Check if using API storage
 	 */
-	isApiExplicitlyConfigured(): boolean {
+	isUsingApiStorage(): boolean {
 		return this.getStorageConfig().type === 'api';
 	}
 
@@ -225,7 +219,6 @@ export class ConfigManager {
 		await this.persistence.saveConfig(this.config);
 
 		// Re-initialize to respect precedence
-		this.initialized = false;
 		await this.initialize();
 	}
 
@@ -276,4 +269,12 @@ export class ConfigManager {
 	getConfigSources() {
 		return this.merger.getSources();
 	}
+
+	/**
+	 * Watch for configuration changes (placeholder for future)
+	 */
+	watch(_callback: (config: PartialConfiguration) => void): () => void {
+		console.warn('Configuration watching not yet implemented');
+		return () => {}; // Return no-op unsubscribe function
+	}
 }

packages/tm-core/src/index.ts

@@ -9,19 +9,19 @@ export {
 	createTaskMasterCore,
 	type TaskMasterCoreOptions,
 	type ListTasksResult
-} from './task-master-core.js';
+} from './task-master-core';
 
 // Re-export types
-export type * from './types/index.js';
+export type * from './types';
 
 // Re-export interfaces (types only to avoid conflicts)
-export type * from './interfaces/index.js';
+export type * from './interfaces';
 
 // Re-export constants
-export * from './constants/index.js';
+export * from './constants';
 
 // Re-export providers
-export * from './providers/index.js';
+export * from './providers';
 
 // Re-export storage (selectively to avoid conflicts)
 export {
@@ -29,29 +29,17 @@ export {
 	ApiStorage,
 	StorageFactory,
 	type ApiStorageConfig
-} from './storage/index.js';
-export { PlaceholderStorage, type StorageAdapter } from './storage/index.js';
+} from './storage';
+export { PlaceholderStorage, type StorageAdapter } from './storage';
 
 // Re-export parser
-export * from './parser/index.js';
+export * from './parser';
 
 // Re-export utilities
-export * from './utils/index.js';
+export * from './utils';
 
 // Re-export errors
-export * from './errors/index.js';
+export * from './errors';
 
 // Re-export entities
-export { TaskEntity } from './entities/task.entity.js';
-
-// Re-export authentication
-export {
-	AuthManager,
-	AuthenticationError,
-	type AuthCredentials,
-	type OAuthFlowOptions,
-	type AuthConfig
-} from './auth/index.js';
-
-// Re-export logger
-export { getLogger, createLogger, setGlobalLogger } from './logger/index.js';
+export { TaskEntity } from './entities/task.entity';

packages/tm-core/src/interfaces/configuration.interface.ts

@@ -3,11 +3,7 @@
  * This file defines the contract for configuration management
  */
 
-import type {
-	TaskComplexity,
-	TaskPriority,
-	StorageType
-} from '../types/index.js';
+import type { TaskComplexity, TaskPriority } from '../types/index';
 
 /**
  * Model configuration for different AI roles
@@ -78,40 +74,17 @@ export interface TagSettings {
 }
 
 /**
- * Runtime storage configuration used for storage backend selection
- * This is what getStorageConfig() returns and what StorageFactory expects
+ * Storage and persistence settings
  */
-export interface RuntimeStorageConfig {
+export interface StorageSettings {
 	/** Storage backend type */
-	type: StorageType;
-	/** Base path for file storage (if configured) */
+	type: 'file' | 'api';
+	/** Base path for file storage */
 	basePath?: string;
 	/** API endpoint for API storage (Hamster integration) */
 	apiEndpoint?: string;
 	/** Access token for API authentication */
 	apiAccessToken?: string;
-	/**
-	 * Indicates whether API is configured (has endpoint or token)
-	 * @computed Derived automatically from presence of apiEndpoint or apiAccessToken
-	 * @internal Should not be set manually - computed by ConfigManager
-	 */
-	readonly apiConfigured: boolean;
-}
-
-/**
- * Storage and persistence settings
- * Extended storage settings including file operation preferences
- */
-export interface StorageSettings
-	extends Omit<RuntimeStorageConfig, 'apiConfigured'> {
-	/** Base path for file storage */
-	basePath?: string;
-	/**
-	 * Indicates whether API is configured
-	 * @computed Derived automatically from presence of apiEndpoint or apiAccessToken
-	 * @internal Should not be set manually in user config - computed by ConfigManager
-	 */
-	readonly apiConfigured?: boolean;
 	/** Enable automatic backups */
 	enableBackup: boolean;
 	/** Maximum number of backups to retain */
@@ -415,7 +388,7 @@ export const DEFAULT_CONFIG_VALUES = {
 		NAMING_CONVENTION: 'kebab-case' as const
 	},
 	STORAGE: {
-		TYPE: 'auto' as const,
+		TYPE: 'file' as const,
 		ENCODING: 'utf8' as BufferEncoding,
 		MAX_BACKUPS: 5
 	},

packages/tm-core/src/interfaces/index.ts

@@ -4,13 +4,13 @@
  */
 
 // Storage interfaces
-export type * from './storage.interface.js';
-export * from './storage.interface.js';
+export type * from './storage.interface';
+export * from './storage.interface';
 
 // AI Provider interfaces
-export type * from './ai-provider.interface.js';
-export * from './ai-provider.interface.js';
+export type * from './ai-provider.interface';
+export * from './ai-provider.interface';
 
 // Configuration interfaces
-export type * from './configuration.interface.js';
-export * from './configuration.interface.js';
+export type * from './configuration.interface';
+export * from './configuration.interface';

packages/tm-core/src/interfaces/storage.interface.ts

@@ -3,7 +3,7 @@
  * This file defines the contract for all storage implementations
  */
 
-import type { Task, TaskMetadata } from '../types/index.js';
+import type { Task, TaskMetadata } from '../types/index';
 
 /**
  * Interface for storage operations on tasks

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

@@ -1,59 +0,0 @@
-/**
- * @fileoverview Logger factory and singleton management
- */
-
-import { Logger, type LoggerConfig } from './logger.js';
-
-// Global logger instance
-let globalLogger: Logger | null = null;
-
-// Named logger instances
-const loggers = new Map<string, Logger>();
-
-/**
- * Create a new logger instance
- */
-export function createLogger(config?: LoggerConfig): Logger {
-	return new Logger(config);
-}
-
-/**
- * Get or create a named logger instance
- */
-export function getLogger(name?: string, config?: LoggerConfig): Logger {
-	// If no name provided, return global logger
-	if (!name) {
-		if (!globalLogger) {
-			globalLogger = createLogger(config);
-		}
-		return globalLogger;
-	}
-
-	// Check if named logger exists
-	if (!loggers.has(name)) {
-		loggers.set(
-			name,
-			createLogger({
-				prefix: name,
-				...config
-			})
-		);
-	}
-
-	return loggers.get(name)!;
-}
-
-/**
- * Set the global logger instance
- */
-export function setGlobalLogger(logger: Logger): void {
-	globalLogger = logger;
-}
-
-/**
- * Clear all logger instances (useful for testing)
- */
-export function clearLoggers(): void {
-	globalLogger = null;
-	loggers.clear();
-}

packages/tm-core/src/logger/index.ts

@@ -1,8 +0,0 @@
-/**
- * @fileoverview Logger package for Task Master
- * Provides centralized logging with support for different modes and levels
- */
-
-export { Logger, LogLevel } from './logger.js';
-export type { LoggerConfig } from './logger.js';
-export { createLogger, getLogger, setGlobalLogger } from './factory.js';

packages/tm-core/src/logger/logger.ts

@@ -1,242 +0,0 @@
-/**
- * @fileoverview Core logger implementation
- */
-
-import chalk from 'chalk';
-
-export enum LogLevel {
-	SILENT = 0,
-	ERROR = 1,
-	WARN = 2,
-	INFO = 3,
-	DEBUG = 4
-}
-
-export interface LoggerConfig {
-	level?: LogLevel;
-	silent?: boolean;
-	prefix?: string;
-	timestamp?: boolean;
-	colors?: boolean;
-	// MCP mode silences all output
-	mcpMode?: boolean;
-}
-
-export class Logger {
-	private config: Required<LoggerConfig>;
-	private static readonly DEFAULT_CONFIG: Required<LoggerConfig> = {
-		level: LogLevel.WARN,
-		silent: false,
-		prefix: '',
-		timestamp: false,
-		colors: true,
-		mcpMode: false
-	};
-
-	constructor(config: LoggerConfig = {}) {
-		// Check environment variables
-		const envConfig: LoggerConfig = {};
-
-		// Check for MCP mode
-		if (
-			process.env.MCP_MODE === 'true' ||
-			process.env.TASK_MASTER_MCP === 'true'
-		) {
-			envConfig.mcpMode = true;
-		}
-
-		// Check for silent mode
-		if (
-			process.env.TASK_MASTER_SILENT === 'true' ||
-			process.env.TM_SILENT === 'true'
-		) {
-			envConfig.silent = true;
-		}
-
-		// Check for log level
-		if (process.env.TASK_MASTER_LOG_LEVEL || process.env.TM_LOG_LEVEL) {
-			const levelStr = (
-				process.env.TASK_MASTER_LOG_LEVEL ||
-				process.env.TM_LOG_LEVEL ||
-				''
-			).toUpperCase();
-			if (levelStr in LogLevel) {
-				envConfig.level = LogLevel[levelStr as keyof typeof LogLevel];
-			}
-		}
-
-		// Check for no colors
-		if (
-			process.env.NO_COLOR === 'true' ||
-			process.env.TASK_MASTER_NO_COLOR === 'true'
-		) {
-			envConfig.colors = false;
-		}
-
-		// Merge configs: defaults < constructor < environment
-		this.config = {
-			...Logger.DEFAULT_CONFIG,
-			...config,
-			...envConfig
-		};
-
-		// MCP mode overrides everything to be silent
-		if (this.config.mcpMode) {
-			this.config.silent = true;
-		}
-	}
-
-	/**
-	 * Check if logging is enabled for a given level
-	 */
-	private shouldLog(level: LogLevel): boolean {
-		if (this.config.silent || this.config.mcpMode) {
-			return false;
-		}
-		return level <= this.config.level;
-	}
-
-	/**
-	 * Format a log message
-	 */
-	private formatMessage(
-		level: LogLevel,
-		message: string,
-		...args: any[]
-	): string {
-		let formatted = '';
-
-		// Add timestamp if enabled
-		if (this.config.timestamp) {
-			const timestamp = new Date().toISOString();
-			formatted += this.config.colors
-				? chalk.gray(`[${timestamp}] `)
-				: `[${timestamp}] `;
-		}
-
-		// Add prefix if configured
-		if (this.config.prefix) {
-			formatted += this.config.colors
-				? chalk.cyan(`[${this.config.prefix}] `)
-				: `[${this.config.prefix}] `;
-		}
-
-		// Skip level indicator for cleaner output
-		// We can still color the message based on level
-		if (this.config.colors) {
-			switch (level) {
-				case LogLevel.ERROR:
-					message = chalk.red(message);
-					break;
-				case LogLevel.WARN:
-					message = chalk.yellow(message);
-					break;
-				case LogLevel.INFO:
-					// Info stays default color
-					break;
-				case LogLevel.DEBUG:
-					message = chalk.gray(message);
-					break;
-			}
-		}
-
-		// Add the message
-		formatted += message;
-
-		// Add any additional arguments
-		if (args.length > 0) {
-			formatted +=
-				' ' +
-				args
-					.map((arg) =>
-						typeof arg === 'object' ? JSON.stringify(arg, null, 2) : String(arg)
-					)
-					.join(' ');
-		}
-
-		return formatted;
-	}
-
-	/**
-	 * Log an error message
-	 */
-	error(message: string, ...args: any[]): void {
-		if (!this.shouldLog(LogLevel.ERROR)) return;
-		console.error(this.formatMessage(LogLevel.ERROR, message, ...args));
-	}
-
-	/**
-	 * Log a warning message
-	 */
-	warn(message: string, ...args: any[]): void {
-		if (!this.shouldLog(LogLevel.WARN)) return;
-		console.warn(this.formatMessage(LogLevel.WARN, message, ...args));
-	}
-
-	/**
-	 * Log an info message
-	 */
-	info(message: string, ...args: any[]): void {
-		if (!this.shouldLog(LogLevel.INFO)) return;
-		console.log(this.formatMessage(LogLevel.INFO, message, ...args));
-	}
-
-	/**
-	 * Log a debug message
-	 */
-	debug(message: string, ...args: any[]): void {
-		if (!this.shouldLog(LogLevel.DEBUG)) return;
-		console.log(this.formatMessage(LogLevel.DEBUG, message, ...args));
-	}
-
-	/**
-	 * Log a message without any formatting (raw output)
-	 * Useful for CLI output that should appear as-is
-	 */
-	log(message: string, ...args: any[]): void {
-		if (this.config.silent || this.config.mcpMode) return;
-
-		if (args.length > 0) {
-			console.log(message, ...args);
-		} else {
-			console.log(message);
-		}
-	}
-
-	/**
-	 * Update logger configuration
-	 */
-	setConfig(config: Partial<LoggerConfig>): void {
-		this.config = {
-			...this.config,
-			...config
-		};
-
-		// MCP mode always overrides to silent
-		if (this.config.mcpMode) {
-			this.config.silent = true;
-		}
-	}
-
-	/**
-	 * Get current configuration
-	 */
-	getConfig(): Readonly<Required<LoggerConfig>> {
-		return { ...this.config };
-	}
-
-	/**
-	 * Create a child logger with a prefix
-	 */
-	child(prefix: string, config?: Partial<LoggerConfig>): Logger {
-		const childPrefix = this.config.prefix
-			? `${this.config.prefix}:${prefix}`
-			: prefix;
-
-		return new Logger({
-			...this.config,
-			...config,
-			prefix: childPrefix
-		});
-	}
-}

packages/tm-core/src/mappers/TaskMapper.ts

@@ -1,170 +0,0 @@
-import { Task, Subtask } from '../types/index.js';
-import { Database, Tables } from '../types/database.types.js';
-
-type TaskRow = Tables<'tasks'>;
-type DependencyRow = Tables<'task_dependencies'>;
-
-export class TaskMapper {
-	/**
-	 * Maps database tasks to internal Task format
-	 */
-	static mapDatabaseTasksToTasks(
-		dbTasks: TaskRow[],
-		dbDependencies: DependencyRow[]
-	): Task[] {
-		if (!dbTasks || dbTasks.length === 0) {
-			return [];
-		}
-
-		// Group dependencies by task_id
-		const dependenciesByTaskId = this.groupDependenciesByTaskId(dbDependencies);
-
-		// Separate parent tasks and subtasks
-		const parentTasks = dbTasks.filter((t) => !t.parent_task_id);
-		const subtasksByParentId = this.groupSubtasksByParentId(dbTasks);
-
-		// Map parent tasks with their subtasks
-		return parentTasks.map((taskRow) =>
-			this.mapDatabaseTaskToTask(
-				taskRow,
-				subtasksByParentId.get(taskRow.id) || [],
-				dependenciesByTaskId
-			)
-		);
-	}
-
-	/**
-	 * Maps a single database task to internal Task format
-	 */
-	static mapDatabaseTaskToTask(
-		dbTask: TaskRow,
-		dbSubtasks: TaskRow[],
-		dependenciesByTaskId: Map<string, string[]>
-	): Task {
-		// Map subtasks
-		const subtasks: Subtask[] = dbSubtasks.map((subtask, index) => ({
-			id: index + 1, // Use numeric ID for subtasks
-			parentId: dbTask.id,
-			title: subtask.title,
-			description: subtask.description || '',
-			status: this.mapStatus(subtask.status),
-			priority: this.mapPriority(subtask.priority),
-			dependencies: dependenciesByTaskId.get(subtask.id) || [],
-			details: (subtask.metadata as any)?.details || '',
-			testStrategy: (subtask.metadata as any)?.testStrategy || '',
-			createdAt: subtask.created_at,
-			updatedAt: subtask.updated_at,
-			assignee: subtask.assignee_id || undefined,
-			complexity: subtask.complexity
-				? this.mapComplexityToInternal(subtask.complexity)
-				: undefined
-		}));
-
-		return {
-			id: dbTask.display_id || dbTask.id, // Use display_id if available
-			title: dbTask.title,
-			description: dbTask.description || '',
-			status: this.mapStatus(dbTask.status),
-			priority: this.mapPriority(dbTask.priority),
-			dependencies: dependenciesByTaskId.get(dbTask.id) || [],
-			details: (dbTask.metadata as any)?.details || '',
-			testStrategy: (dbTask.metadata as any)?.testStrategy || '',
-			subtasks,
-			createdAt: dbTask.created_at,
-			updatedAt: dbTask.updated_at,
-			assignee: dbTask.assignee_id || undefined,
-			complexity: dbTask.complexity
-				? this.mapComplexityToInternal(dbTask.complexity)
-				: undefined,
-			effort: dbTask.estimated_hours || undefined,
-			actualEffort: dbTask.actual_hours || undefined
-		};
-	}
-
-	/**
-	 * Groups dependencies by task ID
-	 */
-	private static groupDependenciesByTaskId(
-		dependencies: DependencyRow[]
-	): Map<string, string[]> {
-		const dependenciesByTaskId = new Map<string, string[]>();
-
-		if (dependencies) {
-			for (const dep of dependencies) {
-				const deps = dependenciesByTaskId.get(dep.task_id) || [];
-				deps.push(dep.depends_on_task_id);
-				dependenciesByTaskId.set(dep.task_id, deps);
-			}
-		}
-
-		return dependenciesByTaskId;
-	}
-
-	/**
-	 * Groups subtasks by their parent ID
-	 */
-	private static groupSubtasksByParentId(
-		tasks: TaskRow[]
-	): Map<string, TaskRow[]> {
-		const subtasksByParentId = new Map<string, TaskRow[]>();
-
-		for (const task of tasks) {
-			if (task.parent_task_id) {
-				const subtasks = subtasksByParentId.get(task.parent_task_id) || [];
-				subtasks.push(task);
-				subtasksByParentId.set(task.parent_task_id, subtasks);
-			}
-		}
-
-		// Sort subtasks by subtask_position for each parent
-		for (const subtasks of subtasksByParentId.values()) {
-			subtasks.sort((a, b) => a.subtask_position - b.subtask_position);
-		}
-
-		return subtasksByParentId;
-	}
-
-	/**
-	 * Maps database status to internal status
-	 */
-	private static mapStatus(
-		status: Database['public']['Enums']['task_status']
-	): Task['status'] {
-		switch (status) {
-			case 'todo':
-				return 'pending';
-			case 'in_progress':
-				return 'in-progress';
-			case 'done':
-				return 'done';
-			default:
-				return 'pending';
-		}
-	}
-
-	/**
-	 * Maps database priority to internal priority
-	 */
-	private static mapPriority(
-		priority: Database['public']['Enums']['task_priority']
-	): Task['priority'] {
-		switch (priority) {
-			case 'urgent':
-				return 'critical';
-			default:
-				return priority as Task['priority'];
-		}
-	}
-
-	/**
-	 * Maps numeric complexity to descriptive complexity
-	 */
-	private static mapComplexityToInternal(
-		complexity: number
-	): Task['complexity'] {
-		if (complexity <= 2) return 'simple';
-		if (complexity <= 5) return 'moderate';
-		if (complexity <= 8) return 'complex';
-		return 'very-complex';
-	}
-}

packages/tm-core/src/parser/index.ts

@@ -3,7 +3,7 @@
  * This file exports all parsing-related classes and functions
  */
 
-import type { PlaceholderTask } from '../types/index.js';
+import type { PlaceholderTask } from '../types/index';
 
 // Parser implementations will be defined here
 // export * from './prd-parser.js';

packages/tm-core/src/repositories/supabase-task-repository.ts

@@ -1,110 +0,0 @@
-import { SupabaseClient } from '@supabase/supabase-js';
-import { Task } from '../types/index.js';
-import { Database } from '../types/database.types.js';
-import { TaskMapper } from '../mappers/TaskMapper.js';
-import { AuthManager } from '../auth/auth-manager.js';
-
-export class SupabaseTaskRepository {
-	constructor(private supabase: SupabaseClient<Database>) {}
-
-	async getTasks(_projectId?: string): Promise<Task[]> {
-		// Get the current context to determine briefId
-		const authManager = AuthManager.getInstance();
-		const context = authManager.getContext();
-
-		if (!context || !context.briefId) {
-			throw new Error(
-				'No brief selected. Please select a brief first using: tm context brief'
-			);
-		}
-
-		// Get all tasks for the brief using the exact query structure
-		const { data: tasks, error } = await this.supabase
-			.from('tasks')
-			.select(`
-        *,
-        document:document_id (
-          id,
-          document_name,
-          title,
-          description
-        )
-      `)
-			.eq('brief_id', context.briefId)
-			.order('position', { ascending: true })
-			.order('subtask_position', { ascending: true })
-			.order('created_at', { ascending: true });
-
-		if (error) {
-			throw new Error(`Failed to fetch tasks: ${error.message}`);
-		}
-
-		if (!tasks || tasks.length === 0) {
-			return [];
-		}
-
-		// Get all dependencies for these tasks
-		const taskIds = tasks.map((t: any) => t.id);
-		const { data: depsData, error: depsError } = await this.supabase
-			.from('task_dependencies')
-			.select('*')
-			.in('task_id', taskIds);
-
-		if (depsError) {
-			throw new Error(
-				`Failed to fetch task dependencies: ${depsError.message}`
-			);
-		}
-
-		// Use mapper to convert to internal format
-		return TaskMapper.mapDatabaseTasksToTasks(tasks, depsData || []);
-	}
-
-	async getTask(accountId: string, taskId: string): Promise<Task | null> {
-		const { data, error } = await this.supabase
-			.from('tasks')
-			.select('*')
-			.eq('account_id', accountId)
-			.eq('id', taskId)
-			.single();
-
-		if (error) {
-			if (error.code === 'PGRST116') {
-				return null; // Not found
-			}
-			throw new Error(`Failed to fetch task: ${error.message}`);
-		}
-
-		// Get dependencies for this task
-		const { data: depsData } = await this.supabase
-			.from('task_dependencies')
-			.select('*')
-			.eq('task_id', taskId);
-
-		// Get subtasks if this is a parent task
-		const { data: subtasksData } = await this.supabase
-			.from('tasks')
-			.select('*')
-			.eq('parent_task_id', taskId)
-			.order('subtask_position', { ascending: true });
-
-		// Create dependency map
-		const dependenciesByTaskId = new Map<string, string[]>();
-		if (depsData) {
-			dependenciesByTaskId.set(
-				taskId,
-				depsData.map(
-					(d: Database['public']['Tables']['task_dependencies']['Row']) =>
-						d.depends_on_task_id
-				)
-			);
-		}
-
-		// Use mapper to convert single task
-		return TaskMapper.mapDatabaseTaskToTask(
-			data,
-			subtasksData || [],
-			dependenciesByTaskId
-		);
-	}
-}

packages/tm-core/src/repositories/task-repository.interface.ts

@@ -1,36 +0,0 @@
-import { Task, TaskTag } from '../types/index.js';
-
-export interface TaskRepository {
-	// Task operations
-	getTasks(projectId: string): Promise<Task[]>;
-	getTask(projectId: string, taskId: string): Promise<Task | null>;
-	createTask(projectId: string, task: Omit<Task, 'id'>): Promise<Task>;
-	updateTask(
-		projectId: string,
-		taskId: string,
-		updates: Partial<Task>
-	): Promise<Task>;
-	deleteTask(projectId: string, taskId: string): Promise<void>;
-
-	// Tag operations
-	getTags(projectId: string): Promise<TaskTag[]>;
-	getTag(projectId: string, tagName: string): Promise<TaskTag | null>;
-	createTag(projectId: string, tag: TaskTag): Promise<TaskTag>;
-	updateTag(
-		projectId: string,
-		tagName: string,
-		updates: Partial<TaskTag>
-	): Promise<TaskTag>;
-	deleteTag(projectId: string, tagName: string): Promise<void>;
-
-	// Bulk operations
-	bulkCreateTasks(
-		projectId: string,
-		tasks: Omit<Task, 'id'>[]
-	): Promise<Task[]>;
-	bulkUpdateTasks(
-		projectId: string,
-		updates: Array<{ id: string; updates: Partial<Task> }>
-	): Promise<Task[]>;
-	bulkDeleteTasks(projectId: string, taskIds: string[]): Promise<void>;
-}

packages/tm-core/src/services/index.ts

@@ -1,8 +0,0 @@
-/**
- * Services module exports
- * Provides business logic and service layer functionality
- */
-
-export { TaskService } from './task-service.js';
-export { OrganizationService } from './organization.service.js';
-export type { Organization, Brief } from './organization.service.js';

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

@@ -1,363 +0,0 @@
-/**
- * @fileoverview Organization and Brief management service
- * Handles fetching and managing organizations and briefs from the API
- */
-
-import { SupabaseClient } from '@supabase/supabase-js';
-import { Database } from '../types/database.types.js';
-import { TaskMasterError, ERROR_CODES } from '../errors/task-master-error.js';
-import { getLogger } from '../logger/index.js';
-
-/**
- * Organization data structure
- */
-export interface Organization {
-	id: string;
-	name: string;
-	slug: string;
-}
-
-/**
- * Brief data structure
- */
-export interface Brief {
-	id: string;
-	accountId: string;
-	documentId: string;
-	status: string;
-	createdAt: string;
-	updatedAt: string;
-}
-
-/**
- * Task data structure from the remote database
- */
-export interface RemoteTask {
-	id: string;
-	briefId: string;
-	documentId: string;
-	position: number | null;
-	subtaskPosition: number | null;
-	status: string;
-	createdAt: string;
-	updatedAt: string;
-	// Document details from join
-	document?: {
-		id: string;
-		document_name: string;
-		title: string;
-		description: string;
-	};
-}
-
-/**
- * Service for managing organizations and briefs
- */
-export class OrganizationService {
-	private logger = getLogger('OrganizationService');
-
-	constructor(private supabaseClient: SupabaseClient<Database>) {}
-
-	/**
-	 * Get all organizations for the authenticated user
-	 */
-	async getOrganizations(): Promise<Organization[]> {
-		try {
-			// The user is already authenticated via the Authorization header
-			// Query the user_accounts view/table (filtered by RLS for current user)
-			const { data, error } = await this.supabaseClient
-				.from('user_accounts')
-				.select(`
-					id,
-					name,
-					slug
-				`);
-
-			if (error) {
-				throw new TaskMasterError(
-					`Failed to fetch organizations: ${error.message}`,
-					ERROR_CODES.API_ERROR,
-					{ operation: 'getOrganizations' },
-					error
-				);
-			}
-
-			if (!data || data.length === 0) {
-				this.logger.debug('No organizations found for user');
-				return [];
-			}
-
-			// Map to our Organization interface
-			return data.map((org) => ({
-				id: org.id ?? '',
-				name: org.name ?? '',
-				slug: org.slug ?? org.id ?? '' // Use ID as fallback if slug is null
-			}));
-		} catch (error) {
-			if (error instanceof TaskMasterError) {
-				throw error;
-			}
-			throw new TaskMasterError(
-				'Failed to fetch organizations',
-				ERROR_CODES.API_ERROR,
-				{ operation: 'getOrganizations' },
-				error as Error
-			);
-		}
-	}
-
-	/**
-	 * Get a specific organization by ID
-	 */
-	async getOrganization(orgId: string): Promise<Organization | null> {
-		try {
-			const { data, error } = await this.supabaseClient
-				.from('accounts')
-				.select(`
-					id,
-					name,
-					slug
-				`)
-				.eq('id', orgId)
-				.single();
-
-			if (error) {
-				if (error.code === 'PGRST116') {
-					// No rows found
-					return null;
-				}
-				throw new TaskMasterError(
-					`Failed to fetch organization: ${error.message}`,
-					ERROR_CODES.API_ERROR,
-					{ operation: 'getOrganization', orgId },
-					error
-				);
-			}
-
-			if (!data) {
-				return null;
-			}
-
-			const accountData =
-				data as Database['public']['Tables']['accounts']['Row'];
-			return {
-				id: accountData.id,
-				name: accountData.name,
-				slug: accountData.slug || accountData.id
-			};
-		} catch (error) {
-			if (error instanceof TaskMasterError) {
-				throw error;
-			}
-			throw new TaskMasterError(
-				'Failed to fetch organization',
-				ERROR_CODES.API_ERROR,
-				{ operation: 'getOrganization', orgId },
-				error as Error
-			);
-		}
-	}
-
-	/**
-	 * Get all briefs for a specific organization
-	 */
-	async getBriefs(orgId: string): Promise<Brief[]> {
-		try {
-			const { data, error } = await this.supabaseClient
-				.from('brief')
-				.select(`
-					id,
-					account_id,
-					document_id,
-					status,
-					created_at,
-					updated_at
-				`)
-				.eq('account_id', orgId);
-
-			if (error) {
-				throw new TaskMasterError(
-					`Failed to fetch briefs: ${error.message}`,
-					ERROR_CODES.API_ERROR,
-					{ operation: 'getBriefs', orgId },
-					error
-				);
-			}
-
-			if (!data || data.length === 0) {
-				this.logger.debug(`No briefs found for organization ${orgId}`);
-				return [];
-			}
-
-			// Map to our Brief interface
-			return data.map((brief: any) => ({
-				id: brief.id,
-				accountId: brief.account_id,
-				documentId: brief.document_id,
-				status: brief.status,
-				createdAt: brief.created_at,
-				updatedAt: brief.updated_at
-			}));
-		} catch (error) {
-			if (error instanceof TaskMasterError) {
-				throw error;
-			}
-			throw new TaskMasterError(
-				'Failed to fetch briefs',
-				ERROR_CODES.API_ERROR,
-				{ operation: 'getBriefs', orgId },
-				error as Error
-			);
-		}
-	}
-
-	/**
-	 * Get a specific brief by ID
-	 */
-	async getBrief(briefId: string): Promise<Brief | null> {
-		try {
-			const { data, error } = await this.supabaseClient
-				.from('brief')
-				.select(`
-					id,
-					account_id,
-					document_id,
-					status,
-					created_at,
-					updated_at
-				`)
-				.eq('id', briefId)
-				.single();
-
-			if (error) {
-				if (error.code === 'PGRST116') {
-					// No rows found
-					return null;
-				}
-				throw new TaskMasterError(
-					`Failed to fetch brief: ${error.message}`,
-					ERROR_CODES.API_ERROR,
-					{ operation: 'getBrief', briefId },
-					error
-				);
-			}
-
-			if (!data) {
-				return null;
-			}
-
-			const briefData = data as any;
-			return {
-				id: briefData.id,
-				accountId: briefData.account_id,
-				documentId: briefData.document_id,
-				status: briefData.status,
-				createdAt: briefData.created_at,
-				updatedAt: briefData.updated_at
-			};
-		} catch (error) {
-			if (error instanceof TaskMasterError) {
-				throw error;
-			}
-			throw new TaskMasterError(
-				'Failed to fetch brief',
-				ERROR_CODES.API_ERROR,
-				{ operation: 'getBrief', briefId },
-				error as Error
-			);
-		}
-	}
-
-	/**
-	 * Validate that a user has access to an organization
-	 */
-	async validateOrgAccess(orgId: string): Promise<boolean> {
-		try {
-			const org = await this.getOrganization(orgId);
-			return org !== null;
-		} catch (error) {
-			this.logger.error(`Failed to validate org access: ${error}`);
-			return false;
-		}
-	}
-
-	/**
-	 * Validate that a user has access to a brief
-	 */
-	async validateBriefAccess(briefId: string): Promise<boolean> {
-		try {
-			const brief = await this.getBrief(briefId);
-			return brief !== null;
-		} catch (error) {
-			this.logger.error(`Failed to validate brief access: ${error}`);
-			return false;
-		}
-	}
-
-	/**
-	 * Get all tasks for a specific brief
-	 */
-	async getTasks(briefId: string): Promise<RemoteTask[]> {
-		try {
-			const { data, error } = await this.supabaseClient
-				.from('tasks')
-				.select(`
-					*,
-					document:document_id (
-						id,
-						document_name,
-						title,
-						description
-					)
-				`)
-				.eq('brief_id', briefId)
-				.order('position', { ascending: true })
-				.order('subtask_position', { ascending: true })
-				.order('created_at', { ascending: true });
-
-			if (error) {
-				throw new TaskMasterError(
-					`Failed to fetch tasks: ${error.message}`,
-					ERROR_CODES.API_ERROR,
-					{ operation: 'getTasks', briefId },
-					error
-				);
-			}
-
-			if (!data || data.length === 0) {
-				this.logger.debug(`No tasks found for brief ${briefId}`);
-				return [];
-			}
-
-			// Map to our RemoteTask interface
-			return data.map((task: any) => ({
-				id: task.id,
-				briefId: task.brief_id,
-				documentId: task.document_id,
-				position: task.position,
-				subtaskPosition: task.subtask_position,
-				status: task.status,
-				createdAt: task.created_at,
-				updatedAt: task.updated_at,
-				document: task.document
-					? {
-							id: task.document.id,
-							document_name: task.document.document_name,
-							title: task.document.title,
-							description: task.document.description
-						}
-					: undefined
-			}));
-		} catch (error) {
-			if (error instanceof TaskMasterError) {
-				throw error;
-			}
-			throw new TaskMasterError(
-				'Failed to fetch tasks',
-				ERROR_CODES.API_ERROR,
-				{ operation: 'getTasks', briefId },
-				error as Error
-			);
-		}
-	}
-}

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

@@ -3,12 +3,7 @@
  * Core service for task operations - handles business logic between storage and API
  */
 
-import type {
-	Task,
-	TaskFilter,
-	TaskStatus,
-	StorageType
-} from '../types/index.js';
+import type { Task, TaskFilter, TaskStatus } from '../types/index.js';
 import type { IStorage } from '../interfaces/storage.interface.js';
 import { ConfigManager } from '../config/config-manager.js';
 import { StorageFactory } from '../storage/storage-factory.js';
@@ -28,7 +23,7 @@ export interface TaskListResult {
 	/** The tag these tasks belong to (only present if explicitly provided) */
 	tag?: string;
 	/** Storage type being used */
-	storageType: StorageType;
+	storageType: 'file' | 'api';
 }
 
 /**
@@ -69,8 +64,8 @@ export class TaskService {
 		const storageConfig = this.configManager.getStorageConfig();
 		const projectRoot = this.configManager.getProjectRoot();
 
-		this.storage = StorageFactory.createFromStorageConfig(
-			storageConfig,
+		this.storage = StorageFactory.create(
+			{ storage: storageConfig } as any,
 			projectRoot
 		);
 
@@ -118,7 +113,7 @@ export class TaskService {
 				total: rawTasks.length,
 				filtered: filteredEntities.length,
 				tag: options.tag, // Only include tag if explicitly provided
-				storageType: this.getStorageType()
+				storageType: this.configManager.getStorageConfig().type
 			};
 		} catch (error) {
 			throw new TaskMasterError(
@@ -171,7 +166,7 @@ export class TaskService {
 		byStatus: Record<TaskStatus, number>;
 		withSubtasks: number;
 		blocked: number;
-		storageType: StorageType;
+		storageType: 'file' | 'api';
 	}> {
 		const result = await this.getTaskList({
 			tag,
@@ -339,12 +334,8 @@ export class TaskService {
 	/**
 	 * Get current storage type
 	 */
-	getStorageType(): StorageType {
-		// Prefer the runtime storage type if available to avoid exposing 'auto'
-		const s = this.storage as { getType?: () => 'file' | 'api' } | null;
-		const runtimeType = s?.getType?.();
-		return (runtimeType ??
-			this.configManager.getStorageConfig().type) as StorageType;
+	getStorageType(): 'file' | 'api' {
+		return this.configManager.getStorageConfig().type;
 	}
 
 	/**

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

@@ -1,29 +1,27 @@
 /**
- * @fileoverview API-based storage implementation using repository pattern
- * This provides storage via repository abstraction for flexibility
+ * @fileoverview API-based storage implementation for Hamster integration
+ * This provides storage via REST API instead of local file system
  */
 
 import type {
 	IStorage,
 	StorageStats
 } from '../interfaces/storage.interface.js';
-import type { Task, TaskMetadata, TaskTag } from '../types/index.js';
+import type { Task, TaskMetadata } from '../types/index.js';
 import { ERROR_CODES, TaskMasterError } from '../errors/task-master-error.js';
-import { TaskRepository } from '../repositories/task-repository.interface.js';
-import { SupabaseTaskRepository } from '../repositories/supabase-task-repository.js';
-import { SupabaseClient } from '@supabase/supabase-js';
-import { AuthManager } from '../auth/auth-manager.js';
 
 /**
  * API storage configuration
  */
 export interface ApiStorageConfig {
-	/** Supabase client instance */
-	supabaseClient?: SupabaseClient;
-	/** Custom repository implementation */
-	repository?: TaskRepository;
-	/** Project ID for scoping */
-	projectId: string;
+	/** API endpoint base URL */
+	endpoint: string;
+	/** Access token for authentication */
+	accessToken: string;
+	/** Optional project ID */
+	projectId?: string;
+	/** Request timeout in milliseconds */
+	timeout?: number;
 	/** Enable request retries */
 	enableRetry?: boolean;
 	/** Maximum retry attempts */
@@ -31,58 +29,64 @@ export interface ApiStorageConfig {
 }
 
 /**
- * ApiStorage implementation using repository pattern
- * Provides flexibility to swap between different backend implementations
+ * API response wrapper
+ */
+interface ApiResponse<T> {
+	success: boolean;
+	data?: T;
+	error?: string;
+	message?: string;
+}
+
+/**
+ * ApiStorage implementation for Hamster integration
+ * Fetches and stores tasks via REST API
  */
 export class ApiStorage implements IStorage {
-	private readonly repository: TaskRepository;
-	private readonly projectId: string;
-	private readonly enableRetry: boolean;
-	private readonly maxRetries: number;
+	private readonly config: Required<ApiStorageConfig>;
 	private initialized = false;
-	private tagsCache: Map<string, TaskTag> = new Map();
 
 	constructor(config: ApiStorageConfig) {
 		this.validateConfig(config);
 
-		// Use provided repository or create Supabase repository
-		if (config.repository) {
-			this.repository = config.repository;
-		} else if (config.supabaseClient) {
-			// TODO: SupabaseTaskRepository doesn't implement all TaskRepository methods yet
-			// Cast for now until full implementation is complete
-			this.repository = new SupabaseTaskRepository(
-				config.supabaseClient
-			) as unknown as TaskRepository;
-		} else {
-			throw new TaskMasterError(
-				'Either repository or supabaseClient must be provided',
-				ERROR_CODES.MISSING_CONFIGURATION
-			);
-		}
-
-		this.projectId = config.projectId;
-		this.enableRetry = config.enableRetry ?? true;
-		this.maxRetries = config.maxRetries ?? 3;
+		this.config = {
+			endpoint: config.endpoint.replace(/\/$/, ''), // Remove trailing slash
+			accessToken: config.accessToken,
+			projectId: config.projectId || 'default',
+			timeout: config.timeout || 30000,
+			enableRetry: config.enableRetry ?? true,
+			maxRetries: config.maxRetries || 3
+		};
 	}
 
 	/**
 	 * Validate API storage configuration
 	 */
 	private validateConfig(config: ApiStorageConfig): void {
-		if (!config.projectId) {
+		if (!config.endpoint) {
 			throw new TaskMasterError(
-				'Project ID is required for API storage',
+				'API endpoint is required for API storage',
 				ERROR_CODES.MISSING_CONFIGURATION
 			);
 		}
 
-		if (!config.repository && !config.supabaseClient) {
+		if (!config.accessToken) {
 			throw new TaskMasterError(
-				'Either repository or supabaseClient must be provided',
+				'Access token is required for API storage',
 				ERROR_CODES.MISSING_CONFIGURATION
 			);
 		}
+
+		// Validate endpoint URL format
+		try {
+			new URL(config.endpoint);
+		} catch {
+			throw new TaskMasterError(
+				'Invalid API endpoint URL',
+				ERROR_CODES.INVALID_INPUT,
+				{ endpoint: config.endpoint }
+			);
+		}
 	}
 
 	/**
@@ -92,8 +96,8 @@ export class ApiStorage implements IStorage {
 		if (this.initialized) return;
 
 		try {
-			// Load initial tags
-			await this.loadTagsIntoCache();
+			// Verify API connectivity
+			await this.verifyConnection();
 			this.initialized = true;
 		} catch (error) {
 			throw new TaskMasterError(
@@ -106,71 +110,39 @@ export class ApiStorage implements IStorage {
 	}
 
 	/**
-	 * Load tags into cache
-	 * In our API-based system, "tags" represent briefs
+	 * Verify API connection
 	 */
-	private async loadTagsIntoCache(): Promise<void> {
-		try {
-			const authManager = AuthManager.getInstance();
-			const context = authManager.getContext();
+	private async verifyConnection(): Promise<void> {
+		const response = await this.makeRequest<{ status: string }>('/health');
 
-			// If we have a selected brief, create a virtual "tag" for it
-			if (context?.briefId) {
-				// Create a virtual tag representing the current brief
-				const briefTag: TaskTag = {
-					name: context.briefId,
-					tasks: [], // Will be populated when tasks are loaded
-					metadata: {
-						briefId: context.briefId,
-						briefName: context.briefName,
-						organizationId: context.orgId
-					}
-				};
-
-				this.tagsCache.clear();
-				this.tagsCache.set(context.briefId, briefTag);
-			}
-		} catch (error) {
-			// If no brief is selected, that's okay - user needs to select one first
-			console.debug('No brief selected, starting with empty cache');
+		if (!response.success) {
+			throw new Error(`API health check failed: ${response.error}`);
 		}
 	}
 
 	/**
 	 * Load tasks from API
-	 * In our system, the tag parameter represents a brief ID
 	 */
 	async loadTasks(tag?: string): Promise<Task[]> {
 		await this.ensureInitialized();
 
 		try {
-			const authManager = AuthManager.getInstance();
-			const context = authManager.getContext();
+			const endpoint = tag
+				? `/projects/${this.config.projectId}/tasks?tag=${encodeURIComponent(tag)}`
+				: `/projects/${this.config.projectId}/tasks`;
 
-			// If no brief is selected in context, throw an error
-			if (!context?.briefId) {
-				throw new Error(
-					'No brief selected. Please select a brief first using: tm context brief <brief-id>'
-				);
+			const response = await this.makeRequest<{ tasks: Task[] }>(endpoint);
+
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to load tasks');
 			}
 
-			// Load tasks from the current brief context
-			const tasks = await this.retryOperation(() =>
-				this.repository.getTasks(this.projectId)
-			);
-
-			// Update the tag cache with the loaded task IDs
-			const briefTag = this.tagsCache.get(context.briefId);
-			if (briefTag) {
-				briefTag.tasks = tasks.map((task) => task.id);
-			}
-
-			return tasks;
+			return response.data?.tasks || [];
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to load tasks from API',
 				ERROR_CODES.STORAGE_ERROR,
-				{ operation: 'loadTasks', tag, context: 'brief-based loading' },
+				{ operation: 'loadTasks', tag },
 				error as Error
 			);
 		}
@@ -183,29 +155,15 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			if (tag) {
-				// Update tag with task IDs
-				const tagData = this.tagsCache.get(tag) || {
-					name: tag,
-					tasks: [],
-					metadata: {}
-				};
-				tagData.tasks = tasks.map((t) => t.id);
+			const endpoint = tag
+				? `/projects/${this.config.projectId}/tasks?tag=${encodeURIComponent(tag)}`
+				: `/projects/${this.config.projectId}/tasks`;
 
-				// Save or update tag
-				if (this.tagsCache.has(tag)) {
-					await this.repository.updateTag(this.projectId, tag, tagData);
-				} else {
-					await this.repository.createTag(this.projectId, tagData);
+			const response = await this.makeRequest(endpoint, 'PUT', { tasks });
+
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to save tasks');
 			}
-
-				this.tagsCache.set(tag, tagData);
-			}
-
-			// Save tasks using bulk operation
-			await this.retryOperation(() =>
-				this.repository.bulkCreateTasks(this.projectId, tasks)
-			);
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to save tasks to API',
@@ -223,17 +181,20 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			if (tag) {
-				// Check if task is in tag
-				const tagData = this.tagsCache.get(tag);
-				if (!tagData || !tagData.tasks.includes(taskId)) {
+			const endpoint = tag
+				? `/projects/${this.config.projectId}/tasks/${taskId}?tag=${encodeURIComponent(tag)}`
+				: `/projects/${this.config.projectId}/tasks/${taskId}`;
+
+			const response = await this.makeRequest<{ task: Task }>(endpoint);
+
+			if (!response.success) {
+				if (response.error?.includes('not found')) {
 					return null;
 				}
+				throw new Error(response.error || 'Failed to load task');
 			}
 
-			return await this.retryOperation(() =>
-				this.repository.getTask(this.projectId, taskId)
-			);
+			return response.data?.task || null;
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to load task from API',
@@ -251,26 +212,14 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			// Check if task exists
-			const existing = await this.repository.getTask(this.projectId, task.id);
+			const endpoint = tag
+				? `/projects/${this.config.projectId}/tasks/${task.id}?tag=${encodeURIComponent(tag)}`
+				: `/projects/${this.config.projectId}/tasks/${task.id}`;
 
-			if (existing) {
-				await this.retryOperation(() =>
-					this.repository.updateTask(this.projectId, task.id, task)
-				);
-			} else {
-				await this.retryOperation(() =>
-					this.repository.createTask(this.projectId, task)
-				);
-			}
+			const response = await this.makeRequest(endpoint, 'PUT', { task });
 
-			// Update tag if specified
-			if (tag) {
-				const tagData = this.tagsCache.get(tag);
-				if (tagData && !tagData.tasks.includes(task.id)) {
-					tagData.tasks.push(task.id);
-					await this.repository.updateTag(this.projectId, tag, tagData);
-				}
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to save task');
 			}
 		} catch (error) {
 			throw new TaskMasterError(
@@ -289,17 +238,14 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			await this.retryOperation(() =>
-				this.repository.deleteTask(this.projectId, taskId)
-			);
+			const endpoint = tag
+				? `/projects/${this.config.projectId}/tasks/${taskId}?tag=${encodeURIComponent(tag)}`
+				: `/projects/${this.config.projectId}/tasks/${taskId}`;
 
-			// Remove from tag if specified
-			if (tag) {
-				const tagData = this.tagsCache.get(tag);
-				if (tagData) {
-					tagData.tasks = tagData.tasks.filter((id) => id !== taskId);
-					await this.repository.updateTag(this.projectId, tag, tagData);
-				}
+			const response = await this.makeRequest(endpoint, 'DELETE');
+
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to delete task');
 			}
 		} catch (error) {
 			throw new TaskMasterError(
@@ -312,24 +258,21 @@ export class ApiStorage implements IStorage {
 	}
 
 	/**
-	 * List available tags (briefs in our system)
+	 * List available tags
 	 */
 	async listTags(): Promise<string[]> {
 		await this.ensureInitialized();
 
 		try {
-			const authManager = AuthManager.getInstance();
-			const context = authManager.getContext();
+			const response = await this.makeRequest<{ tags: string[] }>(
+				`/projects/${this.config.projectId}/tags`
+			);
 
-			// In our API-based system, we only have one "tag" at a time - the current brief
-			if (context?.briefId) {
-				// Ensure the current brief is in our cache
-				await this.loadTagsIntoCache();
-				return [context.briefId];
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to list tags');
 			}
 
-			// No brief selected, return empty array
-			return [];
+			return response.data?.tags || [];
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to list tags from API',
@@ -347,15 +290,19 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			if (tag) {
-				const tagData = this.tagsCache.get(tag);
-				return (tagData?.metadata as TaskMetadata) || null;
+			const endpoint = tag
+				? `/projects/${this.config.projectId}/metadata?tag=${encodeURIComponent(tag)}`
+				: `/projects/${this.config.projectId}/metadata`;
+
+			const response = await this.makeRequest<{ metadata: TaskMetadata }>(
+				endpoint
+			);
+
+			if (!response.success) {
+				return null;
 			}
 
-			// Return global metadata if no tag specified
-			// This could be stored in a special system tag
-			const systemTag = await this.repository.getTag(this.projectId, '_system');
-			return (systemTag?.metadata as TaskMetadata) || null;
+			return response.data?.metadata || null;
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to load metadata from API',
@@ -373,38 +320,14 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			if (tag) {
-				const tagData = this.tagsCache.get(tag) || {
-					name: tag,
-					tasks: [],
-					metadata: {}
-				};
-				tagData.metadata = metadata as any;
+			const endpoint = tag
+				? `/projects/${this.config.projectId}/metadata?tag=${encodeURIComponent(tag)}`
+				: `/projects/${this.config.projectId}/metadata`;
 
-				if (this.tagsCache.has(tag)) {
-					await this.repository.updateTag(this.projectId, tag, tagData);
-				} else {
-					await this.repository.createTag(this.projectId, tagData);
-				}
+			const response = await this.makeRequest(endpoint, 'PUT', { metadata });
 
-				this.tagsCache.set(tag, tagData);
-			} else {
-				// Save to system tag
-				const systemTag: TaskTag = {
-					name: '_system',
-					tasks: [],
-					metadata: metadata as any
-				};
-
-				const existing = await this.repository.getTag(
-					this.projectId,
-					'_system'
-				);
-				if (existing) {
-					await this.repository.updateTag(this.projectId, '_system', systemTag);
-				} else {
-					await this.repository.createTag(this.projectId, systemTag);
-				}
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to save metadata');
 			}
 		} catch (error) {
 			throw new TaskMasterError(
@@ -435,30 +358,14 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			// Use bulk create - repository should handle duplicates
-			await this.retryOperation(() =>
-				this.repository.bulkCreateTasks(this.projectId, tasks)
-			);
+			// First load existing tasks
+			const existingTasks = await this.loadTasks(tag);
 
-			// Update tag if specified
-			if (tag) {
-				const tagData = this.tagsCache.get(tag) || {
-					name: tag,
-					tasks: [],
-					metadata: {}
-				};
+			// Append new tasks
+			const allTasks = [...existingTasks, ...tasks];
 
-				const newTaskIds = tasks.map((t) => t.id);
-				tagData.tasks = [...new Set([...tagData.tasks, ...newTaskIds])];
-
-				if (this.tagsCache.has(tag)) {
-					await this.repository.updateTag(this.projectId, tag, tagData);
-				} else {
-					await this.repository.createTag(this.projectId, tagData);
-				}
-
-				this.tagsCache.set(tag, tagData);
-			}
+			// Save all tasks
+			await this.saveTasks(allTasks, tag);
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to append tasks to API',
@@ -480,9 +387,18 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			await this.retryOperation(() =>
-				this.repository.updateTask(this.projectId, taskId, updates)
-			);
+			// Load the task
+			const task = await this.loadTask(taskId, tag);
+
+			if (!task) {
+				throw new Error(`Task ${taskId} not found`);
+			}
+
+			// Merge updates
+			const updatedTask = { ...task, ...updates, id: taskId };
+
+			// Save updated task
+			await this.saveTask(updatedTask, tag);
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to update task via API',
@@ -507,11 +423,14 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			await this.retryOperation(() =>
-				this.repository.deleteTag(this.projectId, tag)
+			const response = await this.makeRequest(
+				`/projects/${this.config.projectId}/tags/${encodeURIComponent(tag)}`,
+				'DELETE'
 			);
 
-			this.tagsCache.delete(tag);
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to delete tag');
+			}
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to delete tag via API',
@@ -529,21 +448,15 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			const tagData = this.tagsCache.get(oldTag);
-			if (!tagData) {
-				throw new Error(`Tag ${oldTag} not found`);
+			const response = await this.makeRequest(
+				`/projects/${this.config.projectId}/tags/${encodeURIComponent(oldTag)}/rename`,
+				'POST',
+				{ newTag }
+			);
+
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to rename tag');
 			}
-
-			// Create new tag with same data
-			const newTagData = { ...tagData, name: newTag };
-			await this.repository.createTag(this.projectId, newTagData);
-
-			// Delete old tag
-			await this.repository.deleteTag(this.projectId, oldTag);
-
-			// Update cache
-			this.tagsCache.delete(oldTag);
-			this.tagsCache.set(newTag, newTagData);
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to rename tag via API',
@@ -561,17 +474,15 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			const sourceData = this.tagsCache.get(sourceTag);
-			if (!sourceData) {
-				throw new Error(`Source tag ${sourceTag} not found`);
+			const response = await this.makeRequest(
+				`/projects/${this.config.projectId}/tags/${encodeURIComponent(sourceTag)}/copy`,
+				'POST',
+				{ targetTag }
+			);
+
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to copy tag');
 			}
-
-			// Create new tag with copied data
-			const targetData = { ...sourceData, name: targetTag };
-			await this.repository.createTag(this.projectId, targetData);
-
-			// Update cache
-			this.tagsCache.set(targetTag, targetData);
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to copy tag via API',
@@ -589,22 +500,24 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			const tasks = await this.repository.getTasks(this.projectId);
-			const tags = await this.repository.getTags(this.projectId);
+			const response = await this.makeRequest<{
+				stats: StorageStats;
+			}>(`/projects/${this.config.projectId}/stats`);
 
-			const tagStats = tags.map((tag) => ({
-				tag: tag.name,
-				taskCount: tag.tasks.length,
-				lastModified: new Date().toISOString() // TODO: Get actual last modified from tag data
-			}));
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to get stats');
+			}
 
-			return {
-				totalTasks: tasks.length,
-				totalTags: tags.length,
-				storageSize: 0, // Not applicable for API storage
+			// Return stats or default values
+			return (
+				response.data?.stats || {
+					totalTasks: 0,
+					totalTags: 0,
+					storageSize: 0,
 					lastModified: new Date().toISOString(),
-				tagStats
-			};
+					tagStats: []
+				}
+			);
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to get stats from API',
@@ -622,15 +535,16 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			// Export all data
-			await this.repository.getTasks(this.projectId);
-			await this.repository.getTags(this.projectId);
+			const response = await this.makeRequest<{ backupId: string }>(
+				`/projects/${this.config.projectId}/backup`,
+				'POST'
+			);
 
-			// TODO: In a real implementation, this would:
-			// 1. Create backup data structure with tasks and tags
-			// 2. Save the backup to a storage service
-			// For now, return a backup identifier
-			return `backup-${this.projectId}-${Date.now()}`;
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to create backup');
+			}
+
+			return response.data?.backupId || 'unknown';
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to create backup via API',
@@ -644,16 +558,27 @@ export class ApiStorage implements IStorage {
 	/**
 	 * Restore from backup
 	 */
-	async restore(backupId: string): Promise<void> {
+	async restore(backupPath: string): Promise<void> {
 		await this.ensureInitialized();
 
-		// This would restore from a backup service
-		// Implementation depends on backup strategy
-		throw new TaskMasterError(
-			'Restore not implemented for API storage',
-			ERROR_CODES.NOT_IMPLEMENTED,
-			{ operation: 'restore', backupId }
+		try {
+			const response = await this.makeRequest(
+				`/projects/${this.config.projectId}/restore`,
+				'POST',
+				{ backupId: backupPath }
 			);
+
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to restore backup');
+			}
+		} catch (error) {
+			throw new TaskMasterError(
+				'Failed to restore backup via API',
+				ERROR_CODES.STORAGE_ERROR,
+				{ operation: 'restore', backupPath },
+				error as Error
+			);
+		}
 	}
 
 	/**
@@ -663,23 +588,14 @@ export class ApiStorage implements IStorage {
 		await this.ensureInitialized();
 
 		try {
-			// Delete all tasks
-			const tasks = await this.repository.getTasks(this.projectId);
-			if (tasks.length > 0) {
-				await this.repository.bulkDeleteTasks(
-					this.projectId,
-					tasks.map((t) => t.id)
+			const response = await this.makeRequest(
+				`/projects/${this.config.projectId}/clear`,
+				'POST'
 			);
-			}
 
-			// Delete all tags
-			const tags = await this.repository.getTags(this.projectId);
-			for (const tag of tags) {
-				await this.repository.deleteTag(this.projectId, tag.name);
+			if (!response.success) {
+				throw new Error(response.error || 'Failed to clear data');
 			}
-
-			// Clear cache
-			this.tagsCache.clear();
 		} catch (error) {
 			throw new TaskMasterError(
 				'Failed to clear data via API',
@@ -695,7 +611,6 @@ export class ApiStorage implements IStorage {
 	 */
 	async close(): Promise<void> {
 		this.initialized = false;
-		this.tagsCache.clear();
 	}
 
 	/**
@@ -708,21 +623,102 @@ export class ApiStorage implements IStorage {
 	}
 
 	/**
-	 * Retry an operation with exponential backoff
+	 * Make HTTP request to API
 	 */
-	private async retryOperation<T>(
-		operation: () => Promise<T>,
-		attempt: number = 1
-	): Promise<T> {
+	private async makeRequest<T>(
+		path: string,
+		method: 'GET' | 'POST' | 'PUT' | 'DELETE' = 'GET',
+		body?: unknown
+	): Promise<ApiResponse<T>> {
+		const url = `${this.config.endpoint}${path}`;
+		const controller = new AbortController();
+		const timeoutId = setTimeout(() => controller.abort(), this.config.timeout);
+
 		try {
-			return await operation();
+			const options: RequestInit = {
+				method,
+				headers: {
+					Authorization: `Bearer ${this.config.accessToken}`,
+					'Content-Type': 'application/json',
+					Accept: 'application/json'
+				},
+				signal: controller.signal
+			};
+
+			if (body && (method === 'POST' || method === 'PUT')) {
+				options.body = JSON.stringify(body);
+			}
+
+			let lastError: Error | null = null;
+			let attempt = 0;
+
+			while (attempt < this.config.maxRetries) {
+				attempt++;
+
+				try {
+					const response = await fetch(url, options);
+					const data = await response.json();
+
+					if (response.ok) {
+						return { success: true, data: data as T };
+					}
+
+					// Handle specific error codes
+					if (response.status === 401) {
+						return {
+							success: false,
+							error: 'Authentication failed - check access token'
+						};
+					}
+
+					if (response.status === 404) {
+						return {
+							success: false,
+							error: 'Resource not found'
+						};
+					}
+
+					if (response.status === 429) {
+						// Rate limited - retry with backoff
+						if (this.config.enableRetry && attempt < this.config.maxRetries) {
+							await this.delay(Math.pow(2, attempt) * 1000);
+							continue;
+						}
+					}
+
+					const errorData = data as any;
+					return {
+						success: false,
+						error:
+							errorData.error ||
+							errorData.message ||
+							`HTTP ${response.status}: ${response.statusText}`
+					};
 				} catch (error) {
-			if (this.enableRetry && attempt < this.maxRetries) {
-				const delay = Math.pow(2, attempt) * 1000;
-				await new Promise((resolve) => setTimeout(resolve, delay));
-				return this.retryOperation(operation, attempt + 1);
-			}
-			throw error;
+					lastError = error as Error;
+
+					// Retry on network errors
+					if (this.config.enableRetry && attempt < this.config.maxRetries) {
+						await this.delay(Math.pow(2, attempt) * 1000);
+						continue;
 					}
 				}
 			}
+
+			// All retries exhausted
+			return {
+				success: false,
+				error: lastError?.message || 'Request failed after retries'
+			};
+		} finally {
+			clearTimeout(timeoutId);
+		}
+	}
+
+	/**
+	 * Delay helper for retries
+	 */
+	private delay(ms: number): Promise<void> {
+		return new Promise((resolve) => setTimeout(resolve, ms));
+	}
+}

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

@@ -3,41 +3,15 @@
  */
 
 import type { IStorage } from '../interfaces/storage.interface.js';
-import type {
-	IConfiguration,
-	RuntimeStorageConfig,
-	StorageSettings
-} from '../interfaces/configuration.interface.js';
-import { FileStorage } from './file-storage/index.js';
+import type { IConfiguration } from '../interfaces/configuration.interface.js';
+import { FileStorage } from './file-storage';
 import { ApiStorage } from './api-storage.js';
 import { ERROR_CODES, TaskMasterError } from '../errors/task-master-error.js';
-import { AuthManager } from '../auth/auth-manager.js';
-import { getLogger } from '../logger/index.js';
-import { SupabaseAuthClient } from '../clients/supabase-client.js';
 
 /**
  * Factory for creating storage implementations based on configuration
  */
 export class StorageFactory {
-	/**
-	 * Create a storage implementation from runtime storage config
-	 * This is the preferred method when you have a RuntimeStorageConfig
-	 * @param storageConfig - Runtime storage configuration
-	 * @param projectPath - Project root path (for file storage)
-	 * @returns Storage implementation
-	 */
-	static createFromStorageConfig(
-		storageConfig: RuntimeStorageConfig,
-		projectPath: string
-	): IStorage {
-		// Wrap the storage config in the expected format, including projectPath
-		// This ensures ApiStorage receives the projectPath for projectId
-		return StorageFactory.create(
-			{ storage: storageConfig, projectPath } as Partial<IConfiguration>,
-			projectPath
-		);
-	}
-
 	/**
 	 * Create a storage implementation based on configuration
 	 * @param config - Configuration object
@@ -48,83 +22,15 @@ export class StorageFactory {
 		config: Partial<IConfiguration>,
 		projectPath: string
 	): IStorage {
-		const storageType = config.storage?.type || 'auto';
-
-		const logger = getLogger('StorageFactory');
+		const storageType = config.storage?.type || 'file';
 
 		switch (storageType) {
 			case 'file':
-				logger.debug('📁 Using local file storage');
 				return StorageFactory.createFileStorage(projectPath, config);
 
 			case 'api':
-				if (!StorageFactory.isHamsterAvailable(config)) {
-					const missing: string[] = [];
-					if (!config.storage?.apiEndpoint) missing.push('apiEndpoint');
-					if (!config.storage?.apiAccessToken) missing.push('apiAccessToken');
-
-					// Check if authenticated via AuthManager
-					const authManager = AuthManager.getInstance();
-					if (!authManager.isAuthenticated()) {
-						throw new TaskMasterError(
-							`API storage not fully configured (${missing.join(', ') || 'credentials missing'}). Run: tm auth login, or set the missing field(s).`,
-							ERROR_CODES.MISSING_CONFIGURATION,
-							{ storageType: 'api', missing }
-						);
-					}
-					// Use auth token from AuthManager
-					const credentials = authManager.getCredentials();
-					if (credentials) {
-						// Merge with existing storage config, ensuring required fields
-						const nextStorage: StorageSettings = {
-							...(config.storage as StorageSettings),
-							type: 'api',
-							apiAccessToken: credentials.token,
-							apiEndpoint:
-								config.storage?.apiEndpoint ||
-								process.env.HAMSTER_API_URL ||
-								'https://tryhamster.com/api'
-						};
-						config.storage = nextStorage;
-					}
-				}
-				logger.info('☁️  Using API storage');
 				return StorageFactory.createApiStorage(config);
 
-			case 'auto':
-				// Auto-detect based on authentication status
-				const authManager = AuthManager.getInstance();
-
-				// First check if API credentials are explicitly configured
-				if (StorageFactory.isHamsterAvailable(config)) {
-					logger.info('☁️  Using API storage (configured)');
-					return StorageFactory.createApiStorage(config);
-				}
-
-				// Then check if authenticated via AuthManager
-				if (authManager.isAuthenticated()) {
-					const credentials = authManager.getCredentials();
-					if (credentials) {
-						// Configure API storage with auth credentials
-						const nextStorage: StorageSettings = {
-							...(config.storage as StorageSettings),
-							type: 'api',
-							apiAccessToken: credentials.token,
-							apiEndpoint:
-								config.storage?.apiEndpoint ||
-								process.env.HAMSTER_API_URL ||
-								'https://tryhamster.com/api'
-						};
-						config.storage = nextStorage;
-						logger.info('☁️  Using API storage (authenticated)');
-						return StorageFactory.createApiStorage(config);
-					}
-				}
-
-				// Default to file storage
-				logger.debug('📁 Using local file storage');
-				return StorageFactory.createFileStorage(projectPath, config);
-
 			default:
 				throw new TaskMasterError(
 					`Unknown storage type: ${storageType}`,
@@ -149,13 +55,29 @@ export class StorageFactory {
 	 * Create API storage implementation
 	 */
 	private static createApiStorage(config: Partial<IConfiguration>): ApiStorage {
-		// Use our SupabaseAuthClient instead of creating a raw Supabase client
-		const supabaseAuthClient = new SupabaseAuthClient();
-		const supabaseClient = supabaseAuthClient.getClient();
+		const { apiEndpoint, apiAccessToken } = config.storage || {};
+
+		if (!apiEndpoint) {
+			throw new TaskMasterError(
+				'API endpoint is required for API storage',
+				ERROR_CODES.MISSING_CONFIGURATION,
+				{ storageType: 'api' }
+			);
+		}
+
+		if (!apiAccessToken) {
+			throw new TaskMasterError(
+				'API access token is required for API storage',
+				ERROR_CODES.MISSING_CONFIGURATION,
+				{ storageType: 'api' }
+			);
+		}
 
 		return new ApiStorage({
-			supabaseClient,
-			projectId: config.projectPath || '',
+			endpoint: apiEndpoint,
+			accessToken: apiAccessToken,
+			projectId: config.projectPath,
+			timeout: config.retry?.requestTimeout,
 			enableRetry: config.retry?.retryOnNetworkError,
 			maxRetries: config.retry?.retryAttempts
 		});
@@ -203,11 +125,6 @@ export class StorageFactory {
 				// File storage doesn't require additional config
 				break;
 
-			case 'auto':
-				// Auto storage is valid - it will determine the actual type at runtime
-				// No specific validation needed as it will fall back to file if API not configured
-				break;
-
 			default:
 				errors.push(`Unknown storage type: ${storageType}`);
 		}
@@ -240,8 +157,7 @@ export class StorageFactory {
 				await apiStorage.initialize();
 				return apiStorage;
 			} catch (error) {
-				const logger = getLogger('StorageFactory');
-				logger.warn(
+				console.warn(
 					'Failed to initialize API storage, falling back to file storage:',
 					error
 				);

packages/tm-core/src/subpath-exports.test.ts

@@ -1,99 +0,0 @@
-/**
- * Test file documenting subpath export usage
- * This demonstrates how consumers can use granular imports for better tree-shaking
- */
-
-import { describe, it, expect } from 'vitest';
-
-describe('Subpath Exports', () => {
-	it('should allow importing from auth subpath', async () => {
-		// Instead of: import { AuthManager } from '@tm/core';
-		// Use: import { AuthManager } from '@tm/core/auth';
-		const authModule = await import('./auth');
-		expect(authModule.AuthManager).toBeDefined();
-		expect(authModule.AuthenticationError).toBeDefined();
-	});
-
-	it('should allow importing from storage subpath', async () => {
-		// Instead of: import { FileStorage } from '@tm/core';
-		// Use: import { FileStorage } from '@tm/core/storage';
-		const storageModule = await import('./storage');
-		expect(storageModule.FileStorage).toBeDefined();
-		expect(storageModule.ApiStorage).toBeDefined();
-		expect(storageModule.StorageFactory).toBeDefined();
-	});
-
-	it('should allow importing from config subpath', async () => {
-		// Instead of: import { ConfigManager } from '@tm/core';
-		// Use: import { ConfigManager } from '@tm/core/config';
-		const configModule = await import('./config');
-		expect(configModule.ConfigManager).toBeDefined();
-	});
-
-	it('should allow importing from errors subpath', async () => {
-		// Instead of: import { TaskMasterError } from '@tm/core';
-		// Use: import { TaskMasterError } from '@tm/core/errors';
-		const errorsModule = await import('./errors');
-		expect(errorsModule.TaskMasterError).toBeDefined();
-		expect(errorsModule.ERROR_CODES).toBeDefined();
-	});
-
-	it('should allow importing from logger subpath', async () => {
-		// Instead of: import { getLogger } from '@tm/core';
-		// Use: import { getLogger } from '@tm/core/logger';
-		const loggerModule = await import('./logger');
-		expect(loggerModule.getLogger).toBeDefined();
-		expect(loggerModule.createLogger).toBeDefined();
-	});
-
-	it('should allow importing from providers subpath', async () => {
-		// Instead of: import { BaseProvider } from '@tm/core';
-		// Use: import { BaseProvider } from '@tm/core/providers';
-		const providersModule = await import('./providers');
-		expect(providersModule.BaseProvider).toBeDefined();
-	});
-
-	it('should allow importing from services subpath', async () => {
-		// Instead of: import { TaskService } from '@tm/core';
-		// Use: import { TaskService } from '@tm/core/services';
-		const servicesModule = await import('./services');
-		expect(servicesModule.TaskService).toBeDefined();
-	});
-
-	it('should allow importing from utils subpath', async () => {
-		// Instead of: import { generateId } from '@tm/core';
-		// Use: import { generateId } from '@tm/core/utils';
-		const utilsModule = await import('./utils');
-		expect(utilsModule.generateId).toBeDefined();
-	});
-});
-
-/**
- * Usage Examples for Consumers:
- *
- * 1. Import only authentication (smaller bundle):
- * ```typescript
- * import { AuthManager, AuthenticationError } from '@tm/core/auth';
- * ```
- *
- * 2. Import only storage (no auth code bundled):
- * ```typescript
- * import { FileStorage, StorageFactory } from '@tm/core/storage';
- * ```
- *
- * 3. Import only errors (minimal bundle):
- * ```typescript
- * import { TaskMasterError, ERROR_CODES } from '@tm/core/errors';
- * ```
- *
- * 4. Still support convenience imports (larger bundle but better DX):
- * ```typescript
- * import { AuthManager, FileStorage, TaskMasterError } from '@tm/core';
- * ```
- *
- * Benefits:
- * - Better tree-shaking: unused modules are not bundled
- * - Clearer dependencies: explicit about what parts of the library you use
- * - Faster builds: bundlers can optimize better with granular imports
- * - Smaller bundles: especially important for browser/edge deployments
- */

packages/tm-core/src/task-master-core.ts

@@ -10,12 +10,7 @@ import {
 } from './services/task-service.js';
 import { ERROR_CODES, TaskMasterError } from './errors/task-master-error.js';
 import type { IConfiguration } from './interfaces/configuration.interface.js';
-import type {
-	Task,
-	TaskStatus,
-	TaskFilter,
-	StorageType
-} from './types/index.js';
+import type { Task, TaskStatus, TaskFilter } from './types/index.js';
 
 /**
  * Options for creating TaskMasterCore instance
@@ -157,7 +152,7 @@ export class TaskMasterCore {
 	/**
 	 * Get current storage type
 	 */
-	getStorageType(): StorageType {
+	getStorageType(): 'file' | 'api' {
 		return this.taskService.getStorageType();
 	}
 

packages/tm-core/src/types/database.types.ts

@@ -1,491 +0,0 @@
-export type Json =
-	| string
-	| number
-	| boolean
-	| null
-	| { [key: string]: Json | undefined }
-	| Json[];
-
-export type Database = {
-	public: {
-		Tables: {
-			accounts: {
-				Row: {
-					created_at: string | null;
-					created_by: string | null;
-					email: string | null;
-					id: string;
-					is_personal_account: boolean;
-					name: string;
-					picture_url: string | null;
-					primary_owner_user_id: string;
-					public_data: Json;
-					slug: string | null;
-					updated_at: string | null;
-					updated_by: string | null;
-				};
-				Insert: {
-					created_at?: string | null;
-					created_by?: string | null;
-					email?: string | null;
-					id?: string;
-					is_personal_account?: boolean;
-					name: string;
-					picture_url?: string | null;
-					primary_owner_user_id?: string;
-					public_data?: Json;
-					slug?: string | null;
-					updated_at?: string | null;
-					updated_by?: string | null;
-				};
-				Update: {
-					created_at?: string | null;
-					created_by?: string | null;
-					email?: string | null;
-					id?: string;
-					is_personal_account?: boolean;
-					name?: string;
-					picture_url?: string | null;
-					primary_owner_user_id?: string;
-					public_data?: Json;
-					slug?: string | null;
-					updated_at?: string | null;
-					updated_by?: string | null;
-				};
-				Relationships: [];
-			};
-			brief: {
-				Row: {
-					account_id: string;
-					created_at: string;
-					created_by: string;
-					document_id: string;
-					id: string;
-					plan_generation_completed_at: string | null;
-					plan_generation_error: string | null;
-					plan_generation_started_at: string | null;
-					plan_generation_status: Database['public']['Enums']['plan_generation_status'];
-					status: Database['public']['Enums']['brief_status'];
-					updated_at: string;
-				};
-				Insert: {
-					account_id: string;
-					created_at?: string;
-					created_by: string;
-					document_id: string;
-					id?: string;
-					plan_generation_completed_at?: string | null;
-					plan_generation_error?: string | null;
-					plan_generation_started_at?: string | null;
-					plan_generation_status?: Database['public']['Enums']['plan_generation_status'];
-					status?: Database['public']['Enums']['brief_status'];
-					updated_at?: string;
-				};
-				Update: {
-					account_id?: string;
-					created_at?: string;
-					created_by?: string;
-					document_id?: string;
-					id?: string;
-					plan_generation_completed_at?: string | null;
-					plan_generation_error?: string | null;
-					plan_generation_started_at?: string | null;
-					plan_generation_status?: Database['public']['Enums']['plan_generation_status'];
-					status?: Database['public']['Enums']['brief_status'];
-					updated_at?: string;
-				};
-				Relationships: [
-					{
-						foreignKeyName: 'brief_account_id_fkey';
-						columns: ['account_id'];
-						isOneToOne: false;
-						referencedRelation: 'accounts';
-						referencedColumns: ['id'];
-					},
-					{
-						foreignKeyName: 'brief_document_id_fkey';
-						columns: ['document_id'];
-						isOneToOne: false;
-						referencedRelation: 'document';
-						referencedColumns: ['id'];
-					}
-				];
-			};
-			document: {
-				Row: {
-					account_id: string;
-					created_at: string;
-					created_by: string;
-					description: string | null;
-					document_name: string;
-					document_type: Database['public']['Enums']['document_type'];
-					file_path: string | null;
-					file_size: number | null;
-					id: string;
-					metadata: Json | null;
-					mime_type: string | null;
-					processed_at: string | null;
-					processing_error: string | null;
-					processing_status:
-						| Database['public']['Enums']['document_processing_status']
-						| null;
-					source_id: string | null;
-					source_type: string | null;
-					title: string;
-					updated_at: string;
-				};
-				Insert: {
-					account_id: string;
-					created_at?: string;
-					created_by: string;
-					description?: string | null;
-					document_name: string;
-					document_type?: Database['public']['Enums']['document_type'];
-					file_path?: string | null;
-					file_size?: number | null;
-					id?: string;
-					metadata?: Json | null;
-					mime_type?: string | null;
-					processed_at?: string | null;
-					processing_error?: string | null;
-					processing_status?:
-						| Database['public']['Enums']['document_processing_status']
-						| null;
-					source_id?: string | null;
-					source_type?: string | null;
-					title: string;
-					updated_at?: string;
-				};
-				Update: {
-					account_id?: string;
-					created_at?: string;
-					created_by?: string;
-					description?: string | null;
-					document_name?: string;
-					document_type?: Database['public']['Enums']['document_type'];
-					file_path?: string | null;
-					file_size?: number | null;
-					id?: string;
-					metadata?: Json | null;
-					mime_type?: string | null;
-					processed_at?: string | null;
-					processing_error?: string | null;
-					processing_status?:
-						| Database['public']['Enums']['document_processing_status']
-						| null;
-					source_id?: string | null;
-					source_type?: string | null;
-					title?: string;
-					updated_at?: string;
-				};
-				Relationships: [
-					{
-						foreignKeyName: 'document_account_id_fkey';
-						columns: ['account_id'];
-						isOneToOne: false;
-						referencedRelation: 'accounts';
-						referencedColumns: ['id'];
-					}
-				];
-			};
-			tasks: {
-				Row: {
-					account_id: string;
-					actual_hours: number;
-					assignee_id: string | null;
-					brief_id: string | null;
-					completed_subtasks: number;
-					complexity: number | null;
-					created_at: string;
-					created_by: string;
-					description: string | null;
-					display_id: string | null;
-					document_id: string | null;
-					due_date: string | null;
-					estimated_hours: number | null;
-					id: string;
-					metadata: Json;
-					parent_task_id: string | null;
-					position: number;
-					priority: Database['public']['Enums']['task_priority'];
-					status: Database['public']['Enums']['task_status'];
-					subtask_position: number;
-					title: string;
-					total_subtasks: number;
-					updated_at: string;
-					updated_by: string;
-				};
-				Insert: {
-					account_id: string;
-					actual_hours?: number;
-					assignee_id?: string | null;
-					brief_id?: string | null;
-					completed_subtasks?: number;
-					complexity?: number | null;
-					created_at?: string;
-					created_by: string;
-					description?: string | null;
-					display_id?: string | null;
-					document_id?: string | null;
-					due_date?: string | null;
-					estimated_hours?: number | null;
-					id?: string;
-					metadata?: Json;
-					parent_task_id?: string | null;
-					position?: number;
-					priority?: Database['public']['Enums']['task_priority'];
-					status?: Database['public']['Enums']['task_status'];
-					subtask_position?: number;
-					title: string;
-					total_subtasks?: number;
-					updated_at?: string;
-					updated_by: string;
-				};
-				Update: {
-					account_id?: string;
-					actual_hours?: number;
-					assignee_id?: string | null;
-					brief_id?: string | null;
-					completed_subtasks?: number;
-					complexity?: number | null;
-					created_at?: string;
-					created_by?: string;
-					description?: string | null;
-					display_id?: string | null;
-					document_id?: string | null;
-					due_date?: string | null;
-					estimated_hours?: number | null;
-					id?: string;
-					metadata?: Json;
-					parent_task_id?: string | null;
-					position?: number;
-					priority?: Database['public']['Enums']['task_priority'];
-					status?: Database['public']['Enums']['task_status'];
-					subtask_position?: number;
-					title?: string;
-					total_subtasks?: number;
-					updated_at?: string;
-					updated_by?: string;
-				};
-				Relationships: [
-					{
-						foreignKeyName: 'tasks_account_id_fkey';
-						columns: ['account_id'];
-						isOneToOne: false;
-						referencedRelation: 'accounts';
-						referencedColumns: ['id'];
-					},
-					{
-						foreignKeyName: 'tasks_brief_id_fkey';
-						columns: ['brief_id'];
-						isOneToOne: false;
-						referencedRelation: 'brief';
-						referencedColumns: ['id'];
-					},
-					{
-						foreignKeyName: 'tasks_document_id_fkey';
-						columns: ['document_id'];
-						isOneToOne: false;
-						referencedRelation: 'document';
-						referencedColumns: ['id'];
-					},
-					{
-						foreignKeyName: 'tasks_parent_task_id_fkey';
-						columns: ['parent_task_id'];
-						isOneToOne: false;
-						referencedRelation: 'tasks';
-						referencedColumns: ['id'];
-					}
-				];
-			};
-			task_dependencies: {
-				Row: {
-					account_id: string;
-					created_at: string;
-					depends_on_task_id: string;
-					id: string;
-					task_id: string;
-				};
-				Insert: {
-					account_id: string;
-					created_at?: string;
-					depends_on_task_id: string;
-					id?: string;
-					task_id: string;
-				};
-				Update: {
-					account_id?: string;
-					created_at?: string;
-					depends_on_task_id?: string;
-					id?: string;
-					task_id?: string;
-				};
-				Relationships: [
-					{
-						foreignKeyName: 'task_dependencies_account_id_fkey';
-						columns: ['account_id'];
-						isOneToOne: false;
-						referencedRelation: 'accounts';
-						referencedColumns: ['id'];
-					},
-					{
-						foreignKeyName: 'task_dependencies_depends_on_task_id_fkey';
-						columns: ['depends_on_task_id'];
-						isOneToOne: false;
-						referencedRelation: 'tasks';
-						referencedColumns: ['id'];
-					},
-					{
-						foreignKeyName: 'task_dependencies_task_id_fkey';
-						columns: ['task_id'];
-						isOneToOne: false;
-						referencedRelation: 'tasks';
-						referencedColumns: ['id'];
-					}
-				];
-			};
-			user_accounts: {
-				Row: {
-					id: string | null;
-					name: string | null;
-					picture_url: string | null;
-					role: string | null;
-					slug: string | null;
-				};
-				Insert: {
-					id?: string | null;
-					name?: string | null;
-					picture_url?: string | null;
-					role?: string | null;
-					slug?: string | null;
-				};
-				Update: {
-					id?: string | null;
-					name?: string | null;
-					picture_url?: string | null;
-					role?: string | null;
-					slug?: string | null;
-				};
-				Relationships: [];
-			};
-		};
-		Views: {
-			[_ in never]: never;
-		};
-		Functions: {
-			[_ in never]: never;
-		};
-		Enums: {
-			brief_status:
-				| 'draft'
-				| 'refining'
-				| 'aligned'
-				| 'delivering'
-				| 'delivered'
-				| 'done'
-				| 'archived';
-			document_processing_status: 'pending' | 'processing' | 'ready' | 'failed';
-			document_type:
-				| 'brief'
-				| 'blueprint'
-				| 'file'
-				| 'note'
-				| 'transcript'
-				| 'generated_plan'
-				| 'generated_task'
-				| 'generated_summary'
-				| 'method'
-				| 'task';
-			plan_generation_status:
-				| 'not_started'
-				| 'generating'
-				| 'completed'
-				| 'failed';
-			task_priority: 'low' | 'medium' | 'high' | 'urgent';
-			task_status: 'todo' | 'in_progress' | 'done';
-		};
-		CompositeTypes: {
-			[_ in never]: never;
-		};
-	};
-};
-
-export type Tables<
-	PublicTableNameOrOptions extends
-		| keyof (Database['public']['Tables'] & Database['public']['Views'])
-		| { schema: keyof Database },
-	TableName extends PublicTableNameOrOptions extends { schema: keyof Database }
-		? keyof (Database[PublicTableNameOrOptions['schema']]['Tables'] &
-				Database[PublicTableNameOrOptions['schema']]['Views'])
-		: never = never
-> = PublicTableNameOrOptions extends { schema: keyof Database }
-	? (Database[PublicTableNameOrOptions['schema']]['Tables'] &
-			Database[PublicTableNameOrOptions['schema']]['Views'])[TableName] extends {
-			Row: infer R;
-		}
-		? R
-		: never
-	: PublicTableNameOrOptions extends keyof (Database['public']['Tables'] &
-				Database['public']['Views'])
-		? (Database['public']['Tables'] &
-				Database['public']['Views'])[PublicTableNameOrOptions] extends {
-				Row: infer R;
-			}
-			? R
-			: never
-		: never;
-
-export type TablesInsert<
-	PublicTableNameOrOptions extends
-		| keyof Database['public']['Tables']
-		| { schema: keyof Database },
-	TableName extends PublicTableNameOrOptions extends { schema: keyof Database }
-		? keyof Database[PublicTableNameOrOptions['schema']]['Tables']
-		: never = never
-> = PublicTableNameOrOptions extends { schema: keyof Database }
-	? Database[PublicTableNameOrOptions['schema']]['Tables'][TableName] extends {
-			Insert: infer I;
-		}
-		? I
-		: never
-	: PublicTableNameOrOptions extends keyof Database['public']['Tables']
-		? Database['public']['Tables'][PublicTableNameOrOptions] extends {
-				Insert: infer I;
-			}
-			? I
-			: never
-		: never;
-
-export type TablesUpdate<
-	PublicTableNameOrOptions extends
-		| keyof Database['public']['Tables']
-		| { schema: keyof Database },
-	TableName extends PublicTableNameOrOptions extends { schema: keyof Database }
-		? keyof Database[PublicTableNameOrOptions['schema']]['Tables']
-		: never = never
-> = PublicTableNameOrOptions extends { schema: keyof Database }
-	? Database[PublicTableNameOrOptions['schema']]['Tables'][TableName] extends {
-			Update: infer U;
-		}
-		? U
-		: never
-	: PublicTableNameOrOptions extends keyof Database['public']['Tables']
-		? Database['public']['Tables'][PublicTableNameOrOptions] extends {
-				Update: infer U;
-			}
-			? U
-			: never
-		: never;
-
-export type Enums<
-	PublicEnumNameOrOptions extends
-		| keyof Database['public']['Enums']
-		| { schema: keyof Database },
-	EnumName extends PublicEnumNameOrOptions extends { schema: keyof Database }
-		? keyof Database[PublicEnumNameOrOptions['schema']]['Enums']
-		: never = never
-> = PublicEnumNameOrOptions extends { schema: keyof Database }
-	? Database[PublicEnumNameOrOptions['schema']]['Enums'][EnumName]
-	: PublicEnumNameOrOptions extends keyof Database['public']['Enums']
-		? Database['public']['Enums'][PublicEnumNameOrOptions]
-		: never;

packages/tm-core/src/types/index.ts

@@ -2,14 +2,6 @@
  * Core type definitions for Task Master
  */
 
-/**
- * Storage type options
- * - 'file': Local file system storage
- * - 'api': Remote API storage (Hamster integration)
- * - 'auto': Automatically detect based on auth status
- */
-export type StorageType = 'file' | 'api' | 'auto';
-
 // ============================================================================
 // Type Literals
 // ============================================================================
@@ -104,15 +96,6 @@ export interface TaskCollection {
 	metadata: TaskMetadata;
 }
 
-/**
- * Task tag for organizing tasks
- */
-export interface TaskTag {
-	name: string;
-	tasks: string[]; // Task IDs belonging to this tag
-	metadata: Record<string, any>;
-}
-
 // ============================================================================
 // Utility Types
 // ============================================================================

packages/tm-core/src/utils/index.ts

@@ -3,17 +3,29 @@
  * This file exports all utility functions and helper classes
  */
 
-// Export ID generation utilities
-export {
-	generateTaskId as generateId, // Alias for backward compatibility
-	generateTaskId,
-	generateSubtaskId,
-	isValidTaskId,
-	isValidSubtaskId,
-	getParentTaskId
-} from './id-generator.js';
+// Utility implementations will be defined here
+// export * from './validation.js';
+// export * from './formatting.js';
+// export * from './file-utils.js';
+// export * from './async-utils.js';
 
-// Additional utility exports
+// Placeholder exports - these will be implemented in later tasks
+
+/**
+ * Generates a unique ID for tasks
+ * @deprecated This is a placeholder function that will be properly implemented in later tasks
+ */
+export function generateTaskId(): string {
+	return `task-${Date.now()}-${Math.random().toString(36).substring(2, 9)}`;
+}
+
+/**
+ * Validates a task ID format
+ * @deprecated This is a placeholder function that will be properly implemented in later tasks
+ */
+export function isValidTaskId(id: string): boolean {
+	return typeof id === 'string' && id.length > 0;
+}
 
 /**
  * Formats a date for task timestamps

packages/tm-core/tsconfig.json

@@ -1,13 +1,12 @@
 {
 	"compilerOptions": {
 		"target": "ES2022",
-		"module": "NodeNext",
+		"module": "ESNext",
 		"lib": ["ES2022"],
 		"declaration": true,
 		"declarationMap": true,
 		"sourceMap": true,
 		"outDir": "./dist",
-		"baseUrl": ".",
 		"rootDir": "./src",
 		"strict": true,
 		"noImplicitAny": true,
@@ -24,12 +23,18 @@
 		"esModuleInterop": true,
 		"skipLibCheck": true,
 		"forceConsistentCasingInFileNames": true,
-		"moduleResolution": "NodeNext",
-		"moduleDetection": "force",
-		"types": ["node"],
+		"moduleResolution": "node",
 		"resolveJsonModule": true,
 		"isolatedModules": true,
-		"allowImportingTsExtensions": false
+		"paths": {
+			"@/*": ["./src/*"],
+			"@/types": ["./src/types"],
+			"@/providers": ["./src/providers"],
+			"@/storage": ["./src/storage"],
+			"@/parser": ["./src/parser"],
+			"@/utils": ["./src/utils"],
+			"@/errors": ["./src/errors"]
+		}
 	},
 	"include": ["src/**/*"],
 	"exclude": ["node_modules", "dist", "tests", "**/*.test.ts", "**/*.spec.ts"]

packages/tm-core/tsup.config.ts

@@ -0,0 +1,27 @@
+import { defineConfig } from 'tsup';
+
+export default defineConfig({
+	entry: {
+		index: 'src/index.ts',
+		'types/index': 'src/types/index.ts',
+		'providers/index': 'src/providers/index.ts',
+		'storage/index': 'src/storage/index.ts',
+		'parser/index': 'src/parser/index.ts',
+		'utils/index': 'src/utils/index.ts',
+		'errors/index': 'src/errors/index.ts'
+	},
+	format: ['cjs', 'esm'],
+	dts: true,
+	sourcemap: true,
+	clean: true,
+	splitting: false,
+	treeshake: true,
+	minify: false,
+	target: 'es2022',
+	tsconfig: './tsconfig.json',
+	outDir: 'dist',
+	external: ['zod'],
+	esbuildOptions(options) {
+		options.conditions = ['module'];
+	}
+});

scripts/modules/commands.js

@@ -15,13 +15,8 @@ import search from '@inquirer/search';
 import ora from 'ora'; // Import ora
 
 import { log, readJSON } from './utils.js';
-// Import new commands from @tm/cli
-import {
-	ListTasksCommand,
-	ShowCommand,
-	AuthCommand,
-	ContextCommand
-} from '@tm/cli';
+// Import new ListTasksCommand from @tm/cli
+import { ListTasksCommand } from '@tm/cli';
 
 import {
 	parsePRD,
@@ -1745,19 +1740,6 @@ function registerCommands(programInstance) {
 	// NEW: Register the new list command from @tm/cli
 	// This command handles all its own configuration and logic
 	ListTasksCommand.registerOn(programInstance);
-
-	// Register the auth command from @tm/cli
-	// Handles authentication with tryhamster.com
-	AuthCommand.registerOn(programInstance);
-
-	// Register the context command from @tm/cli
-	// Manages workspace context (org/brief selection)
-	ContextCommand.registerOn(programInstance);
-
-	// Register the show command from @tm/cli
-	// Displays detailed information about tasks
-	ShowCommand.registerOn(programInstance);
-
 	// expand command
 	programInstance
 		.command('expand')
@@ -2576,6 +2558,80 @@ ${result.result}
 			);
 		});
 
+	// show command
+	programInstance
+		.command('show')
+		.description(
+			`Display detailed information about one or more tasks${chalk.reset('')}`
+		)
+		.argument('[id]', 'Task ID(s) to show (comma-separated for multiple)')
+		.option(
+			'-i, --id <id>',
+			'Task ID(s) to show (comma-separated for multiple)'
+		)
+		.option('-s, --status <status>', 'Filter subtasks by status')
+		.option(
+			'-f, --file <file>',
+			'Path to the tasks file',
+			TASKMASTER_TASKS_FILE
+		)
+		.option(
+			'-r, --report <report>',
+			'Path to the complexity report file',
+			COMPLEXITY_REPORT_FILE
+		)
+		.option('--tag <tag>', 'Specify tag context for task operations')
+		.action(async (taskId, options) => {
+			// Initialize TaskMaster
+			const initOptions = {
+				tasksPath: options.file || true,
+				tag: options.tag
+			};
+			// Only pass complexityReportPath if user provided a custom path
+			if (options.report && options.report !== COMPLEXITY_REPORT_FILE) {
+				initOptions.complexityReportPath = options.report;
+			}
+			const taskMaster = initTaskMaster(initOptions);
+
+			const idArg = taskId || options.id;
+			const statusFilter = options.status;
+			const tag = taskMaster.getCurrentTag();
+
+			// Show current tag context
+			displayCurrentTagIndicator(tag);
+
+			if (!idArg) {
+				console.error(chalk.red('Error: Please provide a task ID'));
+				process.exit(1);
+			}
+
+			// Check if multiple IDs are provided (comma-separated)
+			const taskIds = idArg
+				.split(',')
+				.map((id) => id.trim())
+				.filter((id) => id.length > 0);
+
+			if (taskIds.length > 1) {
+				// Multiple tasks - use compact summary view with interactive drill-down
+				await displayMultipleTasksSummary(
+					taskMaster.getTasksPath(),
+					taskIds,
+					taskMaster.getComplexityReportPath(),
+					statusFilter,
+					{ projectRoot: taskMaster.getProjectRoot(), tag }
+				);
+			} else {
+				// Single task - use detailed view
+				await displayTaskById(
+					taskMaster.getTasksPath(),
+					taskIds[0],
+					taskMaster.getComplexityReportPath(),
+					statusFilter,
+					{ projectRoot: taskMaster.getProjectRoot(), tag }
+				);
+			}
+		});
+
 	// add-dependency command
 	programInstance
 		.command('add-dependency')

tests/integration/profiles/roo-files-inclusion.test.js

@@ -103,14 +103,10 @@ describe('Roo Files Inclusion in Package', () => {
 	test('source Roo files exist in public/assets directory', () => {
 		// Verify that the source files for Roo integration exist
 		expect(
-			fs.existsSync(
-				path.join(process.cwd(), 'public', 'assets', 'roocode', '.roo')
-			)
+			fs.existsSync(path.join(process.cwd(), 'public', 'assets', 'roocode', '.roo'))
 		).toBe(true);
 		expect(
-			fs.existsSync(
-				path.join(process.cwd(), 'public', 'assets', 'roocode', '.roomodes')
-			)
+			fs.existsSync(path.join(process.cwd(), 'public', 'assets', 'roocode', '.roomodes'))
 		).toBe(true);
 	});
 });

tests/integration/profiles/rules-files-inclusion.test.js

@@ -89,14 +89,10 @@ describe('Rules Files Inclusion in Package', () => {
 	test('source Roo files exist in public/assets directory', () => {
 		// Verify that the source files for Roo integration exist
 		expect(
-			fs.existsSync(
-				path.join(process.cwd(), 'public', 'assets', 'roocode', '.roo')
-			)
+			fs.existsSync(path.join(process.cwd(), 'public', 'assets', 'roocode', '.roo'))
 		).toBe(true);
 		expect(
-			fs.existsSync(
-				path.join(process.cwd(), 'public', 'assets', 'roocode', '.roomodes')
-			)
+			fs.existsSync(path.join(process.cwd(), 'public', 'assets', 'roocode', '.roomodes'))
 		).toBe(true);
 	});
 });

tests/unit/prompt-manager.test.js

@@ -145,14 +145,10 @@ describe('PromptManager', () => {
 				}
 			});
 
-			const withData = promptManager.loadPrompt('conditional-prompt', {
-				hasData: true
-			});
+			const withData = promptManager.loadPrompt('conditional-prompt', { hasData: true });
 			expect(withData.userPrompt).toBe('Data exists');
 
-			const withoutData = promptManager.loadPrompt('conditional-prompt', {
-				hasData: false
-			});
+			const withoutData = promptManager.loadPrompt('conditional-prompt', { hasData: false });
 			expect(withoutData.userPrompt).toBe('No data');
 		});
 	});
@@ -188,7 +184,7 @@ describe('PromptManager', () => {
 			expect(prompts).toBeInstanceOf(Array);
 			expect(prompts.length).toBeGreaterThan(0);
 			
-			const ids = prompts.map((p) => p.id);
+			const ids = prompts.map(p => p.id);
 			expect(ids).toContain('analyze-complexity');
 			expect(ids).toContain('expand-task');
 			expect(ids).toContain('add-task');
@@ -196,6 +192,7 @@ describe('PromptManager', () => {
 		});
 	});
 
+
 	describe('validateTemplate', () => {
 		it('should validate a correct template', () => {
 			const result = promptManager.validateTemplate('research');
@@ -205,7 +202,7 @@ describe('PromptManager', () => {
 		it('should reject invalid template', () => {
 			const result = promptManager.validateTemplate('non-existent');
 			expect(result.valid).toBe(false);
-			expect(result.error).toContain('not found');
+			expect(result.error).toContain("not found");
 		});
 	});
 });

tests/unit/scripts/modules/task-manager/analyze-task-complexity.test.js

@@ -198,13 +198,11 @@ jest.unstable_mockModule('fs', () => ({
 	default: {
 		existsSync: jest.fn(() => false),
 		readFileSync: jest.fn(),
-		writeFileSync: mockWriteFileSync,
-		unlinkSync: jest.fn()
+		writeFileSync: mockWriteFileSync
 	},
 	existsSync: jest.fn(() => false),
 	readFileSync: jest.fn(),
-	writeFileSync: mockWriteFileSync,
-	unlinkSync: jest.fn()
+	writeFileSync: mockWriteFileSync
 }));
 
 jest.unstable_mockModule(

tsup.config.ts

@@ -1,26 +1,55 @@
 import { defineConfig } from 'tsup';
-import { baseConfig, mergeConfig } from '@tm/build-config';
 
-export default defineConfig(
-	mergeConfig(baseConfig, {
+export default defineConfig({
 	entry: {
 		'task-master': 'bin/task-master.js',
 		'mcp-server': 'mcp-server/server.js'
 	},
+	format: ['esm'],
+	target: 'node18',
+	splitting: false,
+	sourcemap: true,
+	clean: true,
+	bundle: true, // Bundle everything into one file
 	outDir: 'dist',
 	publicDir: 'public',
+	// Handle TypeScript imports transparently
+	loader: {
+		'.js': 'jsx',
+		'.ts': 'ts'
+	},
+	esbuildOptions(options) {
+		options.platform = 'node';
+		// Allow importing TypeScript from JavaScript
+		options.resolveExtensions = ['.ts', '.js', '.mjs', '.json'];
+	},
 	// Bundle our monorepo packages but keep node_modules external
 	noExternal: [/@tm\/.*/],
-		// Ensure no code splitting
-		splitting: false,
-		// Better watch configuration
-		ignoreWatch: [
-			'dist',
-			'node_modules',
-			'.git',
-			'tests',
-			'*.test.*',
-			'*.spec.*'
+	external: [
+		// Keep native node modules external
+		'fs',
+		'path',
+		'child_process',
+		'crypto',
+		'os',
+		'url',
+		'util',
+		'stream',
+		'http',
+		'https',
+		'events',
+		'assert',
+		'buffer',
+		'querystring',
+		'readline',
+		'zlib',
+		'tty',
+		'net',
+		'dgram',
+		'dns',
+		'tls',
+		'cluster',
+		'process',
+		'module'
 	]
-	})
-);
+});

turbo.json

@@ -1,45 +0,0 @@
-{
-	"$schema": "https://turbo.build/schema.json",
-	"extends": ["//"],
-	"tasks": {
-		"build": {
-			"dependsOn": ["^build"],
-			"outputs": ["dist/**"],
-			"outputLogs": "new-only"
-		},
-		"dev": {
-			"cache": false,
-			"persistent": true,
-			"dependsOn": ["^build"],
-			"inputs": [
-				"$TURBO_DEFAULT$",
-				"!{packages,apps}/**/dist/**",
-				"!{packages,apps}/**/node_modules/**"
-			]
-		},
-		"test:watch": {
-			"cache": false,
-			"persistent": true,
-			"dependsOn": ["^build"]
-		},
-		"lint": {
-			"dependsOn": ["^build"],
-			"inputs": [
-				"$TURBO_DEFAULT$",
-				"!{packages,apps}/**/dist/**",
-				"!{packages,apps}/**/node_modules/**"
-			],
-			"outputLogs": "new-only"
-		},
-		"typecheck": {
-			"dependsOn": ["^build"],
-			"inputs": [
-				"$TURBO_DEFAULT$",
-				"!{packages,apps}/**/dist/**",
-				"!{packages,apps}/**/node_modules/**"
-			],
-			"outputLogs": "new-only"
-		}
-	},
-	"globalDependencies": ["turbo.json", "tsconfig.json", ".env*"]
-}