@vizzly-testing/cli 0.27.1 → 0.28.0

package/dist/server/routers/projects.js

@@ -1,10 +1,9 @@
 /**
  * Projects Router
- * Handles project management and builds endpoints
+ * Handles project listing and builds endpoints
  */
 
 import * as output from '../../utils/output.js';
-import { parseJsonBody } from '../middleware/json-parser.js';
 import { sendError, sendServiceUnavailable, sendSuccess } from '../middleware/response.js';
 
 /**
@@ -40,105 +39,6 @@ export function createProjectsRouter({
       }
     }
 
-    // List project directory mappings
-    if (req.method === 'GET' && pathname === '/api/projects/mappings') {
-      try {
-        const mappings = await projectService.listMappings();
-        sendSuccess(res, {
-          mappings
-        });
-        return true;
-      } catch (error) {
-        output.debug('Error listing project mappings:', {
-          error: error.message
-        });
-        sendError(res, 500, error.message);
-        return true;
-      }
-    }
-
-    // Create or update project mapping
-    if (req.method === 'POST' && pathname === '/api/projects/mappings') {
-      try {
-        const body = await parseJsonBody(req);
-        const {
-          directory,
-          projectSlug,
-          organizationSlug,
-          token,
-          projectName
-        } = body;
-        const mapping = await projectService.createMapping(directory, {
-          projectSlug,
-          organizationSlug,
-          token,
-          projectName
-        });
-        sendSuccess(res, {
-          success: true,
-          mapping
-        });
-        return true;
-      } catch (error) {
-        output.debug('Error creating project mapping:', {
-          error: error.message
-        });
-        sendError(res, 500, error.message);
-        return true;
-      }
-    }
-
-    // Delete project mapping
-    if (req.method === 'DELETE' && pathname.startsWith('/api/projects/mappings/')) {
-      try {
-        const directory = decodeURIComponent(pathname.replace('/api/projects/mappings/', ''));
-        await projectService.removeMapping(directory);
-        sendSuccess(res, {
-          success: true,
-          message: 'Mapping deleted'
-        });
-        return true;
-      } catch (error) {
-        output.debug('Error deleting project mapping:', {
-          error: error.message
-        });
-        sendError(res, 500, error.message);
-        return true;
-      }
-    }
-
-    // Get recent builds for current project
-    if (req.method === 'GET' && pathname === '/api/builds/recent') {
-      if (!projectService) {
-        sendServiceUnavailable(res, 'Project service');
-        return true;
-      }
-      try {
-        const currentDir = process.cwd();
-        const mapping = await projectService.getMapping(currentDir);
-        if (!mapping || !mapping.projectSlug || !mapping.organizationSlug) {
-          sendError(res, 400, 'No project configured for this directory');
-          return true;
-        }
-        const limit = parseInt(parsedUrl.searchParams.get('limit') || '10', 10);
-        const branch = parsedUrl.searchParams.get('branch') || undefined;
-        const builds = await projectService.getRecentBuilds(mapping.projectSlug, mapping.organizationSlug, {
-          limit,
-          branch
-        });
-        sendSuccess(res, {
-          builds
-        });
-        return true;
-      } catch (error) {
-        output.debug('Error fetching recent builds:', {
-          error: error.message
-        });
-        sendError(res, 500, error.message);
-        return true;
-      }
-    }
-
     // Get builds for a specific project (used by /builds page)
     const projectBuildsMatch = pathname.match(/^\/api\/projects\/([^/]+)\/([^/]+)\/builds$/);
     if (req.method === 'GET' && projectBuildsMatch) {

package/dist/services/project-service.js

@@ -4,29 +4,22 @@
  *
  * Provides the interface expected by src/server/routers/projects.js:
  * - listProjects() - Returns [] if not authenticated
- * - listMappings() - Returns [] if no mappings
- * - getMapping(directory) - Returns null if not found
- * - createMapping(directory, projectData) - Throws on invalid input
- * - removeMapping(directory) - Throws on invalid directory
  * - getRecentBuilds(projectSlug, organizationSlug, options) - Returns [] if not authenticated
  *
  * Error handling:
  * - API methods (listProjects, getRecentBuilds) return empty arrays when not authenticated
- * - Local methods (listMappings, getMapping) never require authentication
- * - Validation errors (createMapping, removeMapping) throw with descriptive messages
  */
 
 import { createAuthClient } from '../auth/client.js';
 import * as projectOps from '../project/operations.js';
 import { getApiUrl } from '../utils/environment-config.js';
-import { deleteProjectMapping, getAuthTokens, getProjectMapping, getProjectMappings, saveProjectMapping } from '../utils/global-config.js';
+import { getAuthTokens } from '../utils/global-config.js';
 
 /**
  * Create a project service instance
  * @param {Object} [options]
  * @param {string} [options.apiUrl] - API base URL (defaults to VIZZLY_API_URL or https://app.vizzly.dev)
  * @param {Object} [options.httpClient] - Injectable HTTP client (for testing)
- * @param {Object} [options.mappingStore] - Injectable mapping store (for testing)
  * @param {Function} [options.getAuthTokens] - Injectable token getter (for testing)
  * @returns {Object} Project service
  */
@@ -39,15 +32,6 @@ export function createProjectService(options = {}) {
     baseUrl: apiUrl
   });
 
-  // Create mapping store adapter for global config
-  // Allow injection for testing
-  let mappingStore = options.mappingStore || {
-    getMappings: getProjectMappings,
-    getMapping: getProjectMapping,
-    saveMapping: saveProjectMapping,
-    deleteMapping: deleteProjectMapping
-  };
-
   // Allow injection of getAuthTokens for testing
   let tokenGetter = options.getAuthTokens || getAuthTokens;
 
@@ -80,38 +64,6 @@ export function createProjectService(options = {}) {
         apiClient: null
       });
     },
-    /**
-     * List all project mappings
-     * @returns {Promise<Array>} Array of project mappings
-     */
-    async listMappings() {
-      return projectOps.listMappings(mappingStore);
-    },
-    /**
-     * Get project mapping for a specific directory
-     * @param {string} directory - Directory path
-     * @returns {Promise<Object|null>} Project mapping or null
-     */
-    async getMapping(directory) {
-      return projectOps.getMapping(mappingStore, directory);
-    },
-    /**
-     * Create or update project mapping
-     * @param {string} directory - Directory path
-     * @param {Object} projectData - Project data
-     * @returns {Promise<Object>} Created mapping
-     */
-    async createMapping(directory, projectData) {
-      return projectOps.createMapping(mappingStore, directory, projectData);
-    },
-    /**
-     * Remove project mapping
-     * @param {string} directory - Directory path
-     * @returns {Promise<void>}
-     */
-    async removeMapping(directory) {
-      return projectOps.removeMapping(mappingStore, directory);
-    },
     /**
      * Get recent builds for a project
      * Returns empty array if not authenticated (projectOps handles null oauthClient)

package/dist/utils/config-loader.js

@@ -2,7 +2,7 @@ import { resolve } from 'node:path';
 import { cosmiconfigSync } from 'cosmiconfig';
 import { validateVizzlyConfigWithDefaults } from './config-schema.js';
 import { getApiToken, getApiUrl, getBuildName, getParallelId } from './environment-config.js';
-import { getAccessToken, getProjectMapping } from './global-config.js';
+import { getAccessToken } from './global-config.js';
 import * as output from './output.js';
 const DEFAULT_CONFIG = {
   // API Configuration
@@ -73,29 +73,7 @@ export async function loadConfig(configPath = null, cliOverrides = {}) {
   // Merge validated file config
   mergeConfig(config, validatedFileConfig);
 
-  // 3. Check project mapping for current directory (if no CLI flag)
-  if (!cliOverrides.token) {
-    const currentDir = process.cwd();
-    const projectMapping = await getProjectMapping(currentDir);
-    if (projectMapping?.token) {
-      // Handle both string tokens and token objects (backward compatibility)
-      let token;
-      if (typeof projectMapping.token === 'string') {
-        token = projectMapping.token;
-      } else if (typeof projectMapping.token === 'object' && projectMapping.token.token) {
-        // Handle nested token object from old API responses
-        token = projectMapping.token.token;
-      } else {
-        token = String(projectMapping.token);
-      }
-      config.apiKey = token;
-      config.projectSlug = projectMapping.projectSlug;
-      config.organizationSlug = projectMapping.organizationSlug;
-      output.debug('config', `linked to ${projectMapping.projectSlug} (${projectMapping.organizationSlug})`);
-    }
-  }
-
-  // 4. Override with environment variables (higher priority than fallbacks)
+  // 3. Override with environment variables (higher priority than fallbacks)
   const envApiKey = getApiToken();
   const envApiUrl = getApiUrl();
   const envBuildName = getBuildName();
@@ -111,20 +89,19 @@ export async function loadConfig(configPath = null, cliOverrides = {}) {
   }
   if (envParallelId) config.parallelId = envParallelId;
 
-  // 5. Apply CLI overrides (highest priority)
+  // 4. Apply CLI overrides (highest priority)
   if (cliOverrides.token) {
     output.debug('config', 'using token from --token flag');
   }
   applyCLIOverrides(config, cliOverrides);
 
-  // 6. Fall back to user auth token if no other token found
+  // 5. Fall back to user auth token if no other token found
   // This enables interactive commands (builds, comparisons, approve, etc.)
   // to work without a project token when the user is logged in
   if (!config.apiKey) {
     let userToken = await getAccessToken();
     if (userToken) {
       config.apiKey = userToken;
-      config.isUserAuth = true; // Flag to indicate this is user auth, not project token
       output.debug('config', 'using token from user login');
     }
   }

package/dist/utils/context.js

@@ -10,7 +10,7 @@
 
 import { existsSync, readFileSync } from 'node:fs';
 import { homedir } from 'node:os';
-import { dirname, join } from 'node:path';
+import { join } from 'node:path';
 
 /**
  * Get dynamic context about the current Vizzly state
@@ -63,20 +63,6 @@ export function getContext() {
       // Ignore
     }
 
-    // Check for project mapping (from vizzly project:select)
-    // Traverse up to find project config, with bounds check for Windows compatibility
-    let projectMapping = null;
-    let checkPath = cwd;
-    let prevPath = null;
-    while (checkPath && checkPath !== prevPath) {
-      if (globalConfig.projects?.[checkPath]) {
-        projectMapping = globalConfig.projects[checkPath];
-        break;
-      }
-      prevPath = checkPath;
-      checkPath = dirname(checkPath);
-    }
-
     // Check for OAuth login (from vizzly login)
     let isLoggedIn = !!globalConfig.auth?.accessToken;
     let userName = globalConfig.auth?.user?.name || globalConfig.auth?.user?.email;
@@ -92,13 +78,7 @@ export function getContext() {
         value: `running on :${serverPort}`
       });
     }
-    if (projectMapping) {
-      items.push({
-        type: 'success',
-        label: 'Project',
-        value: `${projectMapping.projectName} (${projectMapping.organizationSlug})`
-      });
-    } else if (isLoggedIn && userName) {
+    if (isLoggedIn && userName) {
       items.push({
         type: 'success',
         label: 'Logged in',
@@ -114,7 +94,7 @@ export function getContext() {
       items.push({
         type: 'info',
         label: 'Not connected',
-        value: 'run vizzly login or project:select'
+        value: 'run vizzly login'
       });
     }
     if (baselineCount > 0) {
@@ -153,8 +133,7 @@ export function getDetailedContext() {
       port: null
     },
     project: {
-      hasConfig: false,
-      mapping: null
+      hasConfig: false
     },
     auth: {
       loggedIn: false,
@@ -204,19 +183,6 @@ export function getDetailedContext() {
       // Ignore
     }
 
-    // Check for project mapping
-    // Traverse up to find project config, with bounds check for Windows compatibility
-    let checkPath = cwd;
-    let prevPath = null;
-    while (checkPath && checkPath !== prevPath) {
-      if (globalConfig.projects?.[checkPath]) {
-        context.project.mapping = globalConfig.projects[checkPath];
-        break;
-      }
-      prevPath = checkPath;
-      checkPath = dirname(checkPath);
-    }
-
     // Check auth status
     context.auth.loggedIn = !!globalConfig.auth?.accessToken;
     context.auth.userName = globalConfig.auth?.user?.name || globalConfig.auth?.user?.email;

package/dist/utils/global-config.js

@@ -6,7 +6,7 @@
 import { existsSync } from 'node:fs';
 import { chmod, mkdir, readFile, writeFile } from 'node:fs/promises';
 import { homedir } from 'node:os';
-import { dirname, join, parse } from 'node:path';
+import { join } from 'node:path';
 import * as output from './output.js';
 
 /**
@@ -197,79 +197,4 @@ export async function getAccessToken() {
   if (!valid) return null;
   let auth = await getAuthTokens();
   return auth?.accessToken || null;
-}
-
-/**
- * Get project mapping for a directory
- * Walks up the directory tree to find the closest mapping
- * @param {string} directoryPath - Absolute path to project directory
- * @returns {Promise<Object|null>} Project data or null
- */
-export async function getProjectMapping(directoryPath) {
-  const config = await loadGlobalConfig();
-  if (!config.projects) {
-    return null;
-  }
-
-  // Walk up the directory tree looking for a mapping
-  let currentPath = directoryPath;
-  const {
-    root
-  } = parse(currentPath);
-  while (currentPath !== root) {
-    if (config.projects[currentPath]) {
-      return config.projects[currentPath];
-    }
-
-    // Move to parent directory
-    const parentPath = dirname(currentPath);
-    if (parentPath === currentPath) {
-      // We've reached the root
-      break;
-    }
-    currentPath = parentPath;
-  }
-  return null;
-}
-
-/**
- * Save project mapping for a directory
- * @param {string} directoryPath - Absolute path to project directory
- * @param {Object} projectData - Project configuration
- * @param {string} projectData.token - Project API token (vzt_...)
- * @param {string} projectData.projectSlug - Project slug
- * @param {string} projectData.organizationSlug - Organization slug
- * @param {string} projectData.projectName - Project name
- */
-export async function saveProjectMapping(directoryPath, projectData) {
-  const config = await loadGlobalConfig();
-  if (!config.projects) {
-    config.projects = {};
-  }
-  config.projects[directoryPath] = {
-    ...projectData,
-    createdAt: new Date().toISOString()
-  };
-  await saveGlobalConfig(config);
-}
-
-/**
- * Get all project mappings
- * @returns {Promise<Object>} Map of directory paths to project data
- */
-export async function getProjectMappings() {
-  const config = await loadGlobalConfig();
-  return config.projects || {};
-}
-
-/**
- * Delete project mapping for a directory
- * @param {string} directoryPath - Absolute path to project directory
- */
-export async function deleteProjectMapping(directoryPath) {
-  const config = await loadGlobalConfig();
-  if (config.projects?.[directoryPath]) {
-    delete config.projects[directoryPath];
-    await saveGlobalConfig(config);
-  }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vizzly-testing/cli",
-  "version": "0.27.1",
+  "version": "0.28.0",
   "description": "Visual review platform for UI developers and designers",
   "keywords": [
     "visual-testing",