prjct-cli 1.6.1 → 1.6.3

package/CHANGELOG.md

@@ -1,5 +1,71 @@
 # Changelog
 
+## [1.6.3] - 2026-02-07
+
+### Bug Fixes
+
+- replace console.error with logger in setup.ts (PRJ-74) (#126)
+
+
+## [1.6.5] - 2026-02-07
+
+### Bug Fixes
+
+- **Replace console.error with logger in setup.ts**: 12 `console.error` warning calls replaced with `log.warn` for Gemini, Cursor, Windsurf, Antigravity, migration, status line, Context7, symlink, and gitignore warnings. User-facing chalk output preserved. Fatal direct-run error kept as `console.error`.
+
+## [1.6.4] - 2026-02-06
+
+### Bug Fixes
+
+- **Replace console.error with logger in routes.ts**: Server routes now use the centralized `log` module instead of raw `console.error` for JSON read/write and context read errors. Production remains quiet by default; enable with `PRJCT_DEBUG=1`.
+
+## [1.6.3] - 2026-02-06
+
+### Improvements
+
+- **Typed event bus payloads**: Added 15 typed payload interfaces (`SessionStartedPayload`, `SnapshotCreatedPayload`, etc.) and an `EventMap` type mapping event strings to their payloads. Convenience `emit` methods now enforce required fields at compile time. Backward compatible — callers with `Record<string, unknown>` still work.
+
+### Implementation Details
+- `core/types/bus.ts`: Added `EventMap` interface and all payload types
+- `core/bus/bus.ts`: Generic overload `emit<K extends keyof EventMap>()`, typed convenience methods
+- `core/types/index.ts`: Exported all new payload types
+
+### Test Plan
+
+#### For QA
+1. `bun run typecheck` — zero errors
+2. `emit.sessionStarted({})` → TS error for missing fields (IntelliSense works)
+3. `bun test` — all 416 tests pass
+
+#### For Users
+- **What changed:** Event emit methods have typed payloads with autocomplete
+- **Breaking changes:** None
+
+## [1.6.2] - 2026-02-06
+
+### Improvements
+
+- **Diff preview before sync overwrites**: `prjct sync --dry-run` (or `--preview`) now shows what would change in CLAUDE.md without applying changes. Cancelling interactive sync restores the original file. Previously, files were written before the diff was shown, so cancel/preview still applied changes.
+
+### Implementation Details
+- Added `--dry-run` CLI flag as alias for `--preview`
+- Added `restoreOriginal()` helper that writes back saved CLAUDE.md content on cancel/preview
+- Non-interactive mode (LLM) restores original and returns JSON — apply with `prjct sync --yes`
+
+### Test Plan
+
+#### For QA
+1. `prjct sync --dry-run` — diff shown, CLAUDE.md NOT modified
+2. `prjct sync` → cancel — CLAUDE.md restored
+3. `prjct sync` → approve — changes kept
+4. `prjct sync --yes` — applied directly
+5. `prjct sync --json` — JSON returned, CLAUDE.md restored
+
+#### For Users
+- **What changed:** `--dry-run` flag works correctly; cancel restores original
+- **How to use:** `prjct sync --dry-run` to preview, `prjct sync --yes` to apply
+- **Breaking changes:** None
+
 ## [1.6.1] - 2026-02-06
 
 ### Bug Fixes

package/core/bus/bus.ts

@@ -11,7 +11,27 @@ import fs from 'node:fs/promises'
 import path from 'node:path'
 import { getErrorMessage } from '../errors'
 import pathManager from '../infrastructure/path-manager'
-import { type EventCallback, type EventData, EventTypes } from '../types'
+import {
+  type AnalysisCompletedPayload,
+  type EventCallback,
+  type EventData,
+  type EventMap,
+  EventTypes,
+  type FeaturePayload,
+  type GitCommitPayload,
+  type GitPushPayload,
+  type IdeaCapturedPayload,
+  type ProjectInitializedPayload,
+  type ProjectSyncedPayload,
+  type SessionCompletedPayload,
+  type SessionPausedPayload,
+  type SessionResumedPayload,
+  type SessionStartedPayload,
+  type SnapshotCreatedPayload,
+  type SnapshotRestoredPayload,
+  type TaskCompletedPayload,
+  type TaskCreatedPayload,
+} from '../types'
 import log from '../utils/logger'
 
 class EventBus {
@@ -77,8 +97,10 @@ class EventBus {
   }
 
   /**
-   * Emit an event
+   * Emit an event with typed payload
    */
+  async emit<K extends keyof EventMap>(event: K, data: EventMap[K]): Promise<void>
+  async emit(event: string, data: Record<string, unknown>): Promise<void>
   async emit(event: string, data: Record<string, unknown> = {}): Promise<void> {
     const timestamp = new Date().toISOString()
     const eventData: EventData = {
@@ -233,37 +255,34 @@ class EventBus {
 // Singleton instance
 const eventBus = new EventBus()
 
-// Convenience methods for common events
+// Convenience methods for common events (typed payloads)
 const emit = {
-  sessionStarted: (data: Record<string, unknown>) =>
-    eventBus.emit(EventTypes.SESSION_STARTED, data),
-  sessionPaused: (data: Record<string, unknown>) => eventBus.emit(EventTypes.SESSION_PAUSED, data),
-  sessionResumed: (data: Record<string, unknown>) =>
-    eventBus.emit(EventTypes.SESSION_RESUMED, data),
-  sessionCompleted: (data: Record<string, unknown>) =>
+  sessionStarted: (data: SessionStartedPayload) => eventBus.emit(EventTypes.SESSION_STARTED, data),
+  sessionPaused: (data: SessionPausedPayload) => eventBus.emit(EventTypes.SESSION_PAUSED, data),
+  sessionResumed: (data: SessionResumedPayload) => eventBus.emit(EventTypes.SESSION_RESUMED, data),
+  sessionCompleted: (data: SessionCompletedPayload) =>
     eventBus.emit(EventTypes.SESSION_COMPLETED, data),
 
-  taskCreated: (data: Record<string, unknown>) => eventBus.emit(EventTypes.TASK_CREATED, data),
-  taskCompleted: (data: Record<string, unknown>) => eventBus.emit(EventTypes.TASK_COMPLETED, data),
+  taskCreated: (data: TaskCreatedPayload) => eventBus.emit(EventTypes.TASK_CREATED, data),
+  taskCompleted: (data: TaskCompletedPayload) => eventBus.emit(EventTypes.TASK_COMPLETED, data),
 
-  featureAdded: (data: Record<string, unknown>) => eventBus.emit(EventTypes.FEATURE_ADDED, data),
-  featureShipped: (data: Record<string, unknown>) =>
-    eventBus.emit(EventTypes.FEATURE_SHIPPED, data),
+  featureAdded: (data: FeaturePayload) => eventBus.emit(EventTypes.FEATURE_ADDED, data),
+  featureShipped: (data: FeaturePayload) => eventBus.emit(EventTypes.FEATURE_SHIPPED, data),
 
-  ideaCaptured: (data: Record<string, unknown>) => eventBus.emit(EventTypes.IDEA_CAPTURED, data),
+  ideaCaptured: (data: IdeaCapturedPayload) => eventBus.emit(EventTypes.IDEA_CAPTURED, data),
 
-  snapshotCreated: (data: Record<string, unknown>) =>
+  snapshotCreated: (data: SnapshotCreatedPayload) =>
     eventBus.emit(EventTypes.SNAPSHOT_CREATED, data),
-  snapshotRestored: (data: Record<string, unknown>) =>
+  snapshotRestored: (data: SnapshotRestoredPayload) =>
     eventBus.emit(EventTypes.SNAPSHOT_RESTORED, data),
 
-  commitCreated: (data: Record<string, unknown>) => eventBus.emit(EventTypes.COMMIT_CREATED, data),
-  pushCompleted: (data: Record<string, unknown>) => eventBus.emit(EventTypes.PUSH_COMPLETED, data),
+  commitCreated: (data: GitCommitPayload) => eventBus.emit(EventTypes.COMMIT_CREATED, data),
+  pushCompleted: (data: GitPushPayload) => eventBus.emit(EventTypes.PUSH_COMPLETED, data),
 
-  projectInitialized: (data: Record<string, unknown>) =>
+  projectInitialized: (data: ProjectInitializedPayload) =>
     eventBus.emit(EventTypes.PROJECT_INITIALIZED, data),
-  projectSynced: (data: Record<string, unknown>) => eventBus.emit(EventTypes.PROJECT_SYNCED, data),
-  analysisCompleted: (data: Record<string, unknown>) =>
+  projectSynced: (data: ProjectSyncedPayload) => eventBus.emit(EventTypes.PROJECT_SYNCED, data),
+  analysisCompleted: (data: AnalysisCompletedPayload) =>
     eventBus.emit(EventTypes.ANALYSIS_COMPLETED, data),
 }
 

package/core/commands/analysis.ts

@@ -347,8 +347,18 @@ export class AnalysisCommands extends PrjctCommandsBase {
           return { success: true, message: 'No changes' }
         }
 
+        // Helper to restore original CLAUDE.md (undo sync's write)
+        const restoreOriginal = async () => {
+          if (existingContent != null) {
+            await fs.writeFile(claudeMdPath, existingContent, 'utf-8')
+          }
+        }
+
         // Non-interactive mode: return JSON for LLM to handle
         if (isNonInteractive) {
+          // Restore original — LLM will call `prjct sync --yes` to apply
+          await restoreOriginal()
+
           // Build a plain-text diff summary for LLM to show user
           const diffSummary = {
             added: diff.added.map((s) => ({ name: s.name, lineCount: s.lineCount })),
@@ -388,8 +398,9 @@ export class AnalysisCommands extends PrjctCommandsBase {
         // Show diff preview (interactive mode)
         console.log(formatDiffPreview(diff))
 
-        // Preview-only mode - don't apply
+        // Preview-only mode (--preview / --dry-run) - restore and don't apply
         if (options.preview) {
+          await restoreOriginal()
           return {
             success: true,
             isPreview: true,
@@ -411,7 +422,8 @@ export class AnalysisCommands extends PrjctCommandsBase {
         })
 
         if (response.action === 'cancel' || !response.action) {
-          out.warn('Sync cancelled')
+          await restoreOriginal()
+          out.warn('Sync cancelled — no changes applied')
           return { success: false, message: 'Cancelled by user' }
         }
 
@@ -424,12 +436,13 @@ export class AnalysisCommands extends PrjctCommandsBase {
             initial: true,
           })
           if (!confirm.apply) {
-            out.warn('Sync cancelled')
+            await restoreOriginal()
+            out.warn('Sync cancelled — no changes applied')
             return { success: false, message: 'Cancelled by user' }
           }
         }
 
-        // Changes already applied from dry-run, just show success
+        // User approved — changes already applied by sync
         out.done('Changes applied')
         return this.showSyncResult(result, startTime)
       }

package/core/index.ts

@@ -162,7 +162,7 @@ async function main(): Promise<void> {
         sync: () =>
           commands.sync(process.cwd(), {
             aiTools: options.agents ? String(options.agents).split(',') : undefined,
-            preview: options.preview === true,
+            preview: options.preview === true || options['dry-run'] === true,
             yes: options.yes === true,
             json: options.json === true,
             package: options.package ? String(options.package) : undefined,

package/core/infrastructure/setup.ts

@@ -27,6 +27,7 @@ import { dependencyValidator } from '../services/dependency-validator'
 import { isNotFoundError } from '../types/fs'
 import type { AIProviderConfig, AIProviderName } from '../types/provider'
 import { fileExists } from '../utils/fs-helpers'
+import log from '../utils/logger'
 import { getPackageRoot, VERSION } from '../utils/version'
 import {
   detectAllProviders,
@@ -270,7 +271,7 @@ async function installGeminiRouter(): Promise<boolean> {
     }
     return false
   } catch (error) {
-    console.error(`Gemini router warning: ${(error as Error).message}`)
+    log.warn(`Gemini router warning: ${(error as Error).message}`)
     return false
   }
 }
@@ -340,7 +341,7 @@ async function installGeminiGlobalConfig(): Promise<{ success: boolean; action:
     await fs.writeFile(globalConfigPath, updatedContent, 'utf-8')
     return { success: true, action: 'updated' }
   } catch (error) {
-    console.error(`Gemini config warning: ${(error as Error).message}`)
+    log.warn(`Gemini config warning: ${(error as Error).message}`)
     return { success: false, action: null }
   }
 }
@@ -373,7 +374,7 @@ export async function installAntigravitySkill(): Promise<{
 
     // Read template content
     if (!(await fileExists(templatePath))) {
-      console.error('Antigravity SKILL.md template not found')
+      log.warn('Antigravity SKILL.md template not found')
       return { success: false, action: null }
     }
 
@@ -384,7 +385,7 @@ export async function installAntigravitySkill(): Promise<{
 
     return { success: true, action: skillExists ? 'updated' : 'created' }
   } catch (error) {
-    console.error(`Antigravity skill warning: ${(error as Error).message}`)
+    log.warn(`Antigravity skill warning: ${(error as Error).message}`)
     return { success: false, action: null }
   }
 }
@@ -465,7 +466,7 @@ export async function installCursorProject(projectRoot: string): Promise<{
     result.success = result.rulesCreated || result.commandsCreated
     return result
   } catch (error) {
-    console.error(`Cursor installation warning: ${(error as Error).message}`)
+    log.warn(`Cursor installation warning: ${(error as Error).message}`)
     return result
   }
 }
@@ -515,7 +516,7 @@ async function addCursorToGitignore(projectRoot: string): Promise<boolean> {
     await fs.writeFile(gitignorePath, newContent, 'utf-8')
     return true
   } catch (error) {
-    console.error(`Gitignore update warning: ${(error as Error).message}`)
+    log.warn(`Gitignore update warning: ${(error as Error).message}`)
     return false
   }
 }
@@ -615,7 +616,7 @@ export async function installWindsurfProject(projectRoot: string): Promise<{
     result.success = result.rulesCreated || result.workflowsCreated
     return result
   } catch (error) {
-    console.error(`Windsurf installation warning: ${(error as Error).message}`)
+    log.warn(`Windsurf installation warning: ${(error as Error).message}`)
     return result
   }
 }
@@ -665,7 +666,7 @@ async function addWindsurfToGitignore(projectRoot: string): Promise<boolean> {
     await fs.writeFile(gitignorePath, newContent, 'utf-8')
     return true
   } catch (error) {
-    console.error(`Gitignore update warning: ${(error as Error).message}`)
+    log.warn(`Gitignore update warning: ${(error as Error).message}`)
     return false
   }
 }
@@ -738,7 +739,7 @@ async function migrateProjectsCliVersion(): Promise<void> {
     // Silently fail if projects directory doesn't exist
     if (!isNotFoundError(error)) {
       // Log unexpected errors but don't crash - migration is optional
-      console.error(`Migration warning: ${(error as Error).message}`)
+      log.warn(`Migration warning: ${(error as Error).message}`)
     }
   }
 }
@@ -912,7 +913,7 @@ echo "prjct"
     // Silently fail if directories don't exist
     if (!isNotFoundError(error)) {
       // Log unexpected errors but don't crash - status line is optional
-      console.error(`Status line warning: ${(error as Error).message}`)
+      log.warn(`Status line warning: ${(error as Error).message}`)
     }
   }
 }
@@ -965,7 +966,7 @@ async function installContext7MCP(): Promise<void> {
     }
   } catch (error) {
     // Non-fatal error, just log
-    console.error(`Context7 MCP setup warning: ${(error as Error).message}`)
+    log.warn(`Context7 MCP setup warning: ${(error as Error).message}`)
   }
 }
 
@@ -1018,7 +1019,7 @@ async function ensureStatusLineSymlink(linkPath: string, targetPath: string): Pr
     } catch (copyError) {
       // Both symlink and copy failed - log if unexpected error
       if (!isNotFoundError(copyError)) {
-        console.error(`Symlink fallback warning: ${(copyError as Error).message}`)
+        log.warn(`Symlink fallback warning: ${(copyError as Error).message}`)
       }
     }
   }

package/core/server/routes.ts

@@ -12,6 +12,7 @@ import { Hono } from 'hono'
 import * as jsonc from 'jsonc-parser'
 import pathManager from '../infrastructure/path-manager'
 import { isNotFoundError } from '../types/fs'
+import log from '../utils/logger'
 
 // Storage paths relative to project data directory
 const STORAGE_PATHS = {
@@ -34,7 +35,7 @@ async function readJsonFile<T>(filePath: string): Promise<T | null> {
   } catch (error) {
     // ENOENT or parse error - expected for new projects
     if (!isNotFoundError(error) && !(error instanceof SyntaxError)) {
-      console.error(`JSON read error: ${(error as Error).message}`)
+      log.error(`JSON read error: ${(error as Error).message}`)
     }
     return null
   }
@@ -49,7 +50,7 @@ async function writeJsonFile(filePath: string, data: unknown): Promise<boolean>
     await fs.writeFile(filePath, `${JSON.stringify(data, null, 2)}\n`, 'utf-8')
     return true
   } catch (error) {
-    console.error(`JSON write error: ${(error as Error).message}`)
+    log.error(`JSON write error: ${(error as Error).message}`)
     return false
   }
 }
@@ -165,7 +166,7 @@ export function createRoutes(projectId: string, _projectPath: string): Hono {
     } catch (error) {
       // ENOENT - context file doesn't exist yet (expected)
       if (!isNotFoundError(error)) {
-        console.error(`Context read error: ${(error as Error).message}`)
+        log.error(`Context read error: ${(error as Error).message}`)
       }
       return c.text('', 200, { 'Content-Type': 'text/markdown' })
     }

package/core/types/bus.ts

@@ -58,7 +58,7 @@ export const EventTypes = {
 export type BusEventType = (typeof EventTypes)[keyof typeof EventTypes]
 
 /**
- * Event data payload
+ * Event data payload (base interface for all events)
  */
 export interface EventData {
   type: string
@@ -67,6 +67,117 @@ export interface EventData {
   [key: string]: unknown
 }
 
+// =============================================================================
+// Typed Event Payloads
+// =============================================================================
+
+export interface SessionStartedPayload {
+  sessionId: string
+  task: unknown
+  projectId: string | null
+}
+
+export interface SessionPausedPayload {
+  sessionId: string
+  task: unknown
+  duration: number
+  projectId: string | null
+}
+
+export interface SessionResumedPayload {
+  sessionId: string
+  task: unknown
+  projectId: string | null
+}
+
+export interface SessionCompletedPayload {
+  sessionId: string
+  task: unknown
+  duration: number
+  metrics: unknown
+  projectId: string | null
+}
+
+export interface TaskCreatedPayload {
+  taskId: string
+  description: string
+  [key: string]: unknown
+}
+
+export interface TaskCompletedPayload {
+  taskId: string
+  [key: string]: unknown
+}
+
+export interface FeaturePayload {
+  [key: string]: unknown
+}
+
+export interface IdeaCapturedPayload {
+  [key: string]: unknown
+}
+
+export interface SnapshotCreatedPayload {
+  hash: string
+  message: string
+  timestamp: string
+  filesCount: number
+  projectId: string | null
+}
+
+export interface SnapshotRestoredPayload {
+  hash: string
+  filesCount: number
+  timestamp: string
+  projectId: string | null
+}
+
+export interface GitCommitPayload {
+  [key: string]: unknown
+}
+
+export interface GitPushPayload {
+  [key: string]: unknown
+}
+
+export interface ProjectInitializedPayload {
+  [key: string]: unknown
+}
+
+export interface ProjectSyncedPayload {
+  [key: string]: unknown
+}
+
+export interface AnalysisCompletedPayload {
+  [key: string]: unknown
+}
+
+/**
+ * Maps event type strings to their typed payloads
+ */
+export interface EventMap {
+  [EventTypes.SESSION_STARTED]: SessionStartedPayload
+  [EventTypes.SESSION_PAUSED]: SessionPausedPayload
+  [EventTypes.SESSION_RESUMED]: SessionResumedPayload
+  [EventTypes.SESSION_COMPLETED]: SessionCompletedPayload
+  [EventTypes.TASK_CREATED]: TaskCreatedPayload
+  [EventTypes.TASK_COMPLETED]: TaskCompletedPayload
+  [EventTypes.TASK_UPDATED]: Record<string, unknown>
+  [EventTypes.FEATURE_ADDED]: FeaturePayload
+  [EventTypes.FEATURE_SHIPPED]: FeaturePayload
+  [EventTypes.FEATURE_UPDATED]: FeaturePayload
+  [EventTypes.IDEA_CAPTURED]: IdeaCapturedPayload
+  [EventTypes.IDEA_PROMOTED]: Record<string, unknown>
+  [EventTypes.SNAPSHOT_CREATED]: SnapshotCreatedPayload
+  [EventTypes.SNAPSHOT_RESTORED]: SnapshotRestoredPayload
+  [EventTypes.COMMIT_CREATED]: GitCommitPayload
+  [EventTypes.PUSH_COMPLETED]: GitPushPayload
+  [EventTypes.PROJECT_INITIALIZED]: ProjectInitializedPayload
+  [EventTypes.PROJECT_SYNCED]: ProjectSyncedPayload
+  [EventTypes.ANALYSIS_COMPLETED]: AnalysisCompletedPayload
+  [EventTypes.ALL]: Record<string, unknown>
+}
+
 /**
  * Event callback handler
  */

package/core/types/index.ts

@@ -118,10 +118,26 @@ export type {
 // Bus Types
 // =============================================================================
 export type {
+  AnalysisCompletedPayload,
   BusEventType,
   EventCallback,
   EventData,
+  EventMap,
   EventSubscription,
+  FeaturePayload,
+  GitCommitPayload,
+  GitPushPayload,
+  IdeaCapturedPayload,
+  ProjectInitializedPayload,
+  ProjectSyncedPayload,
+  SessionCompletedPayload,
+  SessionPausedPayload,
+  SessionResumedPayload,
+  SessionStartedPayload,
+  SnapshotCreatedPayload,
+  SnapshotRestoredPayload,
+  TaskCompletedPayload,
+  TaskCreatedPayload,
 } from './bus'
 export { EventTypes } from './bus'
 // =============================================================================

package/dist/bin/prjct.mjs

@@ -2092,6 +2092,45 @@ var init_editors_config = __esm({
   }
 });
 
+// core/utils/logger.ts
+function getLogLevel() {
+  const debugEnv = process.env.PRJCT_DEBUG || process.env.DEBUG || "";
+  if (!debugEnv) return { level: -1, name: "disabled" };
+  if (TRUTHY_VALUES.has(debugEnv) || debugEnv.includes("prjct")) {
+    return { level: LEVELS.debug, name: "debug" };
+  }
+  const levelValue = LEVELS[debugEnv] ?? -1;
+  const levelName = levelValue >= 0 ? debugEnv : "disabled";
+  return { level: levelValue, name: levelName };
+}
+function createLogMethod(levelThreshold, prefix, method) {
+  return currentLevel >= levelThreshold ? (...args2) => console[method](prefix, ...args2) : noop;
+}
+var LEVELS, TRUTHY_VALUES, currentLevel, currentLevelName, noop, logger, logger_default;
+var init_logger = __esm({
+  "core/utils/logger.ts"() {
+    "use strict";
+    LEVELS = { error: 0, warn: 1, info: 2, debug: 3 };
+    TRUTHY_VALUES = /* @__PURE__ */ new Set(["1", "true", "*"]);
+    __name(getLogLevel, "getLogLevel");
+    ({ level: currentLevel, name: currentLevelName } = getLogLevel());
+    noop = /* @__PURE__ */ __name(() => {
+    }, "noop");
+    __name(createLogMethod, "createLogMethod");
+    logger = {
+      error: createLogMethod(LEVELS.error, "[prjct:error]", "error"),
+      warn: createLogMethod(LEVELS.warn, "[prjct:warn]", "warn"),
+      info: createLogMethod(LEVELS.info, "[prjct:info]", "log"),
+      debug: createLogMethod(LEVELS.debug, "[prjct:debug]", "log"),
+      // Check if logging is enabled (useful for expensive log prep)
+      isEnabled: /* @__PURE__ */ __name(() => currentLevel >= 0, "isEnabled"),
+      // Get current level name (pre-computed, no runtime lookup)
+      level: /* @__PURE__ */ __name(() => currentLevelName, "level")
+    };
+    logger_default = logger;
+  }
+});
+
 // core/utils/branding.ts
 import chalk from "chalk";
 var SPINNER_FRAMES, SPINNER_SPEED, branding, branding_default;
@@ -19160,7 +19199,13 @@ var init_analysis2 = __esm({
               output_default.done("No changes detected (context is up to date)");
               return { success: true, message: "No changes" };
             }
+            const restoreOriginal = /* @__PURE__ */ __name(async () => {
+              if (existingContent != null) {
+                await fs36.writeFile(claudeMdPath, existingContent, "utf-8");
+              }
+            }, "restoreOriginal");
             if (isNonInteractive) {
+              await restoreOriginal();
               const diffSummary = {
                 added: diff.added.map((s) => ({ name: s.name, lineCount: s.lineCount })),
                 modified: diff.modified.map((s) => ({ name: s.name, lineCount: s.lineCount })),
@@ -19193,6 +19238,7 @@ var init_analysis2 = __esm({
             }
             console.log(formatDiffPreview(diff));
             if (options.preview) {
+              await restoreOriginal();
               return {
                 success: true,
                 isPreview: true,
@@ -19211,7 +19257,8 @@ var init_analysis2 = __esm({
               ]
             });
             if (response.action === "cancel" || !response.action) {
-              output_default.warn("Sync cancelled");
+              await restoreOriginal();
+              output_default.warn("Sync cancelled \u2014 no changes applied");
               return { success: false, message: "Cancelled by user" };
             }
             if (response.action === "diff") {
@@ -19224,7 +19271,8 @@ ${formatFullDiff(diff)}`);
                 initial: true
               });
               if (!confirm.apply) {
-                output_default.warn("Sync cancelled");
+                await restoreOriginal();
+                output_default.warn("Sync cancelled \u2014 no changes applied");
                 return { success: false, message: "Cancelled by user" };
               }
             }
@@ -21052,45 +21100,6 @@ var init_ai_tools = __esm({
   }
 });
 
-// core/utils/logger.ts
-function getLogLevel() {
-  const debugEnv = process.env.PRJCT_DEBUG || process.env.DEBUG || "";
-  if (!debugEnv) return { level: -1, name: "disabled" };
-  if (TRUTHY_VALUES.has(debugEnv) || debugEnv.includes("prjct")) {
-    return { level: LEVELS.debug, name: "debug" };
-  }
-  const levelValue = LEVELS[debugEnv] ?? -1;
-  const levelName = levelValue >= 0 ? debugEnv : "disabled";
-  return { level: levelValue, name: levelName };
-}
-function createLogMethod(levelThreshold, prefix, method) {
-  return currentLevel >= levelThreshold ? (...args2) => console[method](prefix, ...args2) : noop;
-}
-var LEVELS, TRUTHY_VALUES, currentLevel, currentLevelName, noop, logger2, logger_default;
-var init_logger = __esm({
-  "core/utils/logger.ts"() {
-    "use strict";
-    LEVELS = { error: 0, warn: 1, info: 2, debug: 3 };
-    TRUTHY_VALUES = /* @__PURE__ */ new Set(["1", "true", "*"]);
-    __name(getLogLevel, "getLogLevel");
-    ({ level: currentLevel, name: currentLevelName } = getLogLevel());
-    noop = /* @__PURE__ */ __name(() => {
-    }, "noop");
-    __name(createLogMethod, "createLogMethod");
-    logger2 = {
-      error: createLogMethod(LEVELS.error, "[prjct:error]", "error"),
-      warn: createLogMethod(LEVELS.warn, "[prjct:warn]", "warn"),
-      info: createLogMethod(LEVELS.info, "[prjct:info]", "log"),
-      debug: createLogMethod(LEVELS.debug, "[prjct:debug]", "log"),
-      // Check if logging is enabled (useful for expensive log prep)
-      isEnabled: /* @__PURE__ */ __name(() => currentLevel >= 0, "isEnabled"),
-      // Get current level name (pre-computed, no runtime lookup)
-      level: /* @__PURE__ */ __name(() => currentLevelName, "level")
-    };
-    logger_default = logger2;
-  }
-});
-
 // core/services/context-generator.ts
 import fs39 from "node:fs/promises";
 import path43 from "node:path";
@@ -24672,7 +24681,7 @@ async function installGeminiRouter() {
     }
     return false;
   } catch (error) {
-    console.error(`Gemini router warning: ${error.message}`);
+    logger_default.warn(`Gemini router warning: ${error.message}`);
     return false;
   }
 }
@@ -24721,7 +24730,7 @@ ${templateContent}`;
     await fs47.writeFile(globalConfigPath, updatedContent, "utf-8");
     return { success: true, action: "updated" };
   } catch (error) {
-    console.error(`Gemini config warning: ${error.message}`);
+    logger_default.warn(`Gemini config warning: ${error.message}`);
     return { success: false, action: null };
   }
 }
@@ -24734,14 +24743,14 @@ async function installAntigravitySkill() {
     await fs47.mkdir(prjctSkillDir, { recursive: true });
     const skillExists = await fileExists(skillMdPath);
     if (!await fileExists(templatePath)) {
-      console.error("Antigravity SKILL.md template not found");
+      logger_default.warn("Antigravity SKILL.md template not found");
       return { success: false, action: null };
     }
     const templateContent = await fs47.readFile(templatePath, "utf-8");
     await fs47.writeFile(skillMdPath, templateContent, "utf-8");
     return { success: true, action: skillExists ? "updated" : "created" };
   } catch (error) {
-    console.error(`Antigravity skill warning: ${error.message}`);
+    logger_default.warn(`Antigravity skill warning: ${error.message}`);
     return { success: false, action: null };
   }
 }
@@ -24782,7 +24791,7 @@ async function installCursorProject(projectRoot) {
     result.success = result.rulesCreated || result.commandsCreated;
     return result;
   } catch (error) {
-    console.error(`Cursor installation warning: ${error.message}`);
+    logger_default.warn(`Cursor installation warning: ${error.message}`);
     return result;
   }
 }
@@ -24821,7 +24830,7 @@ ${entriesToAdd.join("\n")}
     await fs47.writeFile(gitignorePath, newContent, "utf-8");
     return true;
   } catch (error) {
-    console.error(`Gitignore update warning: ${error.message}`);
+    logger_default.warn(`Gitignore update warning: ${error.message}`);
     return false;
   }
 }
@@ -24873,7 +24882,7 @@ async function installWindsurfProject(projectRoot) {
     result.success = result.rulesCreated || result.workflowsCreated;
     return result;
   } catch (error) {
-    console.error(`Windsurf installation warning: ${error.message}`);
+    logger_default.warn(`Windsurf installation warning: ${error.message}`);
     return result;
   }
 }
@@ -24912,7 +24921,7 @@ ${entriesToAdd.join("\n")}
     await fs47.writeFile(gitignorePath, newContent, "utf-8");
     return true;
   } catch (error) {
-    console.error(`Gitignore update warning: ${error.message}`);
+    logger_default.warn(`Gitignore update warning: ${error.message}`);
     return false;
   }
 }
@@ -24956,7 +24965,7 @@ async function migrateProjectsCliVersion() {
     }
   } catch (error) {
     if (!isNotFoundError(error)) {
-      console.error(`Migration warning: ${error.message}`);
+      logger_default.warn(`Migration warning: ${error.message}`);
     }
   }
 }
@@ -25080,7 +25089,7 @@ echo "prjct"
     await ensureStatusLineSettings(settingsPath, claudeStatusLinePath);
   } catch (error) {
     if (!isNotFoundError(error)) {
-      console.error(`Status line warning: ${error.message}`);
+      logger_default.warn(`Status line warning: ${error.message}`);
     }
   }
 }
@@ -25112,7 +25121,7 @@ async function installContext7MCP() {
       await fs47.writeFile(mcpConfigPath, JSON.stringify(context7Config, null, 2), "utf-8");
     }
   } catch (error) {
-    console.error(`Context7 MCP setup warning: ${error.message}`);
+    logger_default.warn(`Context7 MCP setup warning: ${error.message}`);
   }
 }
 async function installStatusLineModules(sourceDir, destDir) {
@@ -25150,7 +25159,7 @@ async function ensureStatusLineSymlink(linkPath, targetPath) {
       }
     } catch (copyError) {
       if (!isNotFoundError(copyError)) {
-        console.error(`Symlink fallback warning: ${copyError.message}`);
+        logger_default.warn(`Symlink fallback warning: ${copyError.message}`);
       }
     }
   }
@@ -25188,6 +25197,7 @@ var init_setup = __esm({
     init_dependency_validator();
     init_fs();
     init_fs_helpers();
+    init_logger();
     init_version();
     init_ai_provider();
     init_command_installer();
@@ -28604,7 +28614,7 @@ var require_package = __commonJS({
   "package.json"(exports, module) {
     module.exports = {
       name: "prjct-cli",
-      version: "1.6.1",
+      version: "1.6.3",
       description: "Context layer for AI agents. Project context for Claude Code, Gemini CLI, and more.",
       main: "core/index.ts",
       bin: {
@@ -28812,7 +28822,7 @@ async function main() {
         // Setup
         sync: /* @__PURE__ */ __name(() => commands.sync(process.cwd(), {
           aiTools: options.agents ? String(options.agents).split(",") : void 0,
-          preview: options.preview === true,
+          preview: options.preview === true || options["dry-run"] === true,
           yes: options.yes === true,
           json: options.json === true,
           package: options.package ? String(options.package) : void 0
@@ -29058,7 +29068,7 @@ import chalk19 from "chalk";
 // core/server/server.ts
 import { Hono as Hono3 } from "hono";
 import { cors } from "hono/cors";
-import { logger } from "hono/logger";
+import { logger as logger2 } from "hono/logger";
 
 // core/utils/runtime.ts
 function detectRuntime() {
@@ -29076,6 +29086,7 @@ __name(isBun, "isBun");
 // core/server/routes.ts
 init_path_manager();
 init_fs();
+init_logger();
 import fs7 from "node:fs/promises";
 import path7 from "node:path";
 import { Hono } from "hono";
@@ -29095,7 +29106,7 @@ async function readJsonFile(filePath) {
     return errors.length > 0 ? null : result;
   } catch (error) {
     if (!isNotFoundError(error) && !(error instanceof SyntaxError)) {
-      console.error(`JSON read error: ${error.message}`);
+      logger_default.error(`JSON read error: ${error.message}`);
     }
     return null;
   }
@@ -29108,7 +29119,7 @@ async function writeJsonFile(filePath, data) {
 `, "utf-8");
     return true;
   } catch (error) {
-    console.error(`JSON write error: ${error.message}`);
+    logger_default.error(`JSON write error: ${error.message}`);
     return false;
   }
 }
@@ -29198,7 +29209,7 @@ function createRoutes(projectId, _projectPath) {
       return c.text(content, 200, { "Content-Type": "text/markdown" });
     } catch (error) {
       if (!isNotFoundError(error)) {
-        console.error(`Context read error: ${error.message}`);
+        logger_default.error(`Context read error: ${error.message}`);
       }
       return c.text("", 200, { "Content-Type": "text/markdown" });
     }
@@ -29704,7 +29715,7 @@ function createServer(config) {
     );
   }
   if (config.enableLogging !== false) {
-    app.use("*", logger());
+    app.use("*", logger2());
   }
   app.get("/health", (c) => c.json({ status: "ok", timestamp: (/* @__PURE__ */ new Date()).toISOString() }));
   app.get(

package/dist/core/infrastructure/setup.js

@@ -719,6 +719,39 @@ __name(isNotFoundError, "isNotFoundError");
 // core/infrastructure/setup.ts
 init_fs_helpers();
 
+// core/utils/logger.ts
+var LEVELS = { error: 0, warn: 1, info: 2, debug: 3 };
+var TRUTHY_VALUES = /* @__PURE__ */ new Set(["1", "true", "*"]);
+function getLogLevel() {
+  const debugEnv = process.env.PRJCT_DEBUG || process.env.DEBUG || "";
+  if (!debugEnv) return { level: -1, name: "disabled" };
+  if (TRUTHY_VALUES.has(debugEnv) || debugEnv.includes("prjct")) {
+    return { level: LEVELS.debug, name: "debug" };
+  }
+  const levelValue = LEVELS[debugEnv] ?? -1;
+  const levelName = levelValue >= 0 ? debugEnv : "disabled";
+  return { level: levelValue, name: levelName };
+}
+__name(getLogLevel, "getLogLevel");
+var { level: currentLevel, name: currentLevelName } = getLogLevel();
+var noop = /* @__PURE__ */ __name(() => {
+}, "noop");
+function createLogMethod(levelThreshold, prefix, method) {
+  return currentLevel >= levelThreshold ? (...args) => console[method](prefix, ...args) : noop;
+}
+__name(createLogMethod, "createLogMethod");
+var logger = {
+  error: createLogMethod(LEVELS.error, "[prjct:error]", "error"),
+  warn: createLogMethod(LEVELS.warn, "[prjct:warn]", "warn"),
+  info: createLogMethod(LEVELS.info, "[prjct:info]", "log"),
+  debug: createLogMethod(LEVELS.debug, "[prjct:debug]", "log"),
+  // Check if logging is enabled (useful for expensive log prep)
+  isEnabled: /* @__PURE__ */ __name(() => currentLevel >= 0, "isEnabled"),
+  // Get current level name (pre-computed, no runtime lookup)
+  level: /* @__PURE__ */ __name(() => currentLevelName, "level")
+};
+var logger_default = logger;
+
 // core/utils/version.ts
 var import_node_fs = __toESM(require("node:fs"));
 var import_node_path = __toESM(require("node:path"));
@@ -1574,7 +1607,7 @@ async function installGeminiRouter() {
     }
     return false;
   } catch (error) {
-    console.error(`Gemini router warning: ${error.message}`);
+    logger_default.warn(`Gemini router warning: ${error.message}`);
     return false;
   }
 }
@@ -1624,7 +1657,7 @@ ${templateContent}`;
     await import_promises4.default.writeFile(globalConfigPath, updatedContent, "utf-8");
     return { success: true, action: "updated" };
   } catch (error) {
-    console.error(`Gemini config warning: ${error.message}`);
+    logger_default.warn(`Gemini config warning: ${error.message}`);
     return { success: false, action: null };
   }
 }
@@ -1638,14 +1671,14 @@ async function installAntigravitySkill() {
     await import_promises4.default.mkdir(prjctSkillDir, { recursive: true });
     const skillExists = await fileExists(skillMdPath);
     if (!await fileExists(templatePath)) {
-      console.error("Antigravity SKILL.md template not found");
+      logger_default.warn("Antigravity SKILL.md template not found");
       return { success: false, action: null };
     }
     const templateContent = await import_promises4.default.readFile(templatePath, "utf-8");
     await import_promises4.default.writeFile(skillMdPath, templateContent, "utf-8");
     return { success: true, action: skillExists ? "updated" : "created" };
   } catch (error) {
-    console.error(`Antigravity skill warning: ${error.message}`);
+    logger_default.warn(`Antigravity skill warning: ${error.message}`);
     return { success: false, action: null };
   }
 }
@@ -1688,7 +1721,7 @@ async function installCursorProject(projectRoot) {
     result.success = result.rulesCreated || result.commandsCreated;
     return result;
   } catch (error) {
-    console.error(`Cursor installation warning: ${error.message}`);
+    logger_default.warn(`Cursor installation warning: ${error.message}`);
     return result;
   }
 }
@@ -1728,7 +1761,7 @@ ${entriesToAdd.join("\n")}
     await import_promises4.default.writeFile(gitignorePath, newContent, "utf-8");
     return true;
   } catch (error) {
-    console.error(`Gitignore update warning: ${error.message}`);
+    logger_default.warn(`Gitignore update warning: ${error.message}`);
     return false;
   }
 }
@@ -1783,7 +1816,7 @@ async function installWindsurfProject(projectRoot) {
     result.success = result.rulesCreated || result.workflowsCreated;
     return result;
   } catch (error) {
-    console.error(`Windsurf installation warning: ${error.message}`);
+    logger_default.warn(`Windsurf installation warning: ${error.message}`);
     return result;
   }
 }
@@ -1823,7 +1856,7 @@ ${entriesToAdd.join("\n")}
     await import_promises4.default.writeFile(gitignorePath, newContent, "utf-8");
     return true;
   } catch (error) {
-    console.error(`Gitignore update warning: ${error.message}`);
+    logger_default.warn(`Gitignore update warning: ${error.message}`);
     return false;
   }
 }
@@ -1870,7 +1903,7 @@ async function migrateProjectsCliVersion() {
     }
   } catch (error) {
     if (!isNotFoundError(error)) {
-      console.error(`Migration warning: ${error.message}`);
+      logger_default.warn(`Migration warning: ${error.message}`);
     }
   }
 }
@@ -1996,7 +2029,7 @@ echo "prjct"
     await ensureStatusLineSettings(settingsPath, claudeStatusLinePath);
   } catch (error) {
     if (!isNotFoundError(error)) {
-      console.error(`Status line warning: ${error.message}`);
+      logger_default.warn(`Status line warning: ${error.message}`);
     }
   }
 }
@@ -2029,7 +2062,7 @@ async function installContext7MCP() {
       await import_promises4.default.writeFile(mcpConfigPath, JSON.stringify(context7Config, null, 2), "utf-8");
     }
   } catch (error) {
-    console.error(`Context7 MCP setup warning: ${error.message}`);
+    logger_default.warn(`Context7 MCP setup warning: ${error.message}`);
   }
 }
 __name(installContext7MCP, "installContext7MCP");
@@ -2069,7 +2102,7 @@ async function ensureStatusLineSymlink(linkPath, targetPath) {
       }
     } catch (copyError) {
       if (!isNotFoundError(copyError)) {
-        console.error(`Symlink fallback warning: ${copyError.message}`);
+        logger_default.warn(`Symlink fallback warning: ${copyError.message}`);
       }
     }
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prjct-cli",
-  "version": "1.6.1",
+  "version": "1.6.3",
   "description": "Context layer for AI agents. Project context for Claude Code, Gemini CLI, and more.",
   "main": "core/index.ts",
   "bin": {