@tenonhq/sincronia-core 0.0.74 → 0.0.76

package/dist/initSystem/orchestrator.js

@@ -113,6 +113,31 @@ async function runLoginPhase(plugin, context) {
         return;
     // Core plugin: retry loop with specific error messages
     if (plugin.name === "core") {
+        // If .env already has all required credentials, offer to skip login
+        const hasAllCreds = context.env.SN_INSTANCE && context.env.SN_USER && context.env.SN_PASSWORD;
+        if (hasAllCreds) {
+            Logger_1.logger.info("Found existing credentials:");
+            Logger_1.logger.info("  Instance: " + context.env.SN_INSTANCE);
+            Logger_1.logger.info("  User:     " + context.env.SN_USER);
+            Logger_1.logger.info("");
+            const useExisting = await inquirer_1.default.prompt([{
+                    type: "confirm",
+                    name: "confirmed",
+                    message: "Use these credentials?",
+                    default: true,
+                }]);
+            if (useExisting.confirmed) {
+                Logger_1.logger.info("Validating credentials...");
+                const result = await (0, corePlugin_1.validateCoreLogin)(context);
+                if (result === true) {
+                    Logger_1.logger.success(chalk_1.default.green("✓ Connected to " + context.env.SN_INSTANCE));
+                    return;
+                }
+                Logger_1.logger.error(chalk_1.default.red("✗ " + result));
+                Logger_1.logger.info("Please re-enter your credentials.");
+                context.env.SN_PASSWORD = "";
+            }
+        }
         while (true) {
             await collectLoginHooks(hooks, context);
             // Run per-hook validation

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tenonhq/sincronia-core",
-  "version": "0.0.74",
+  "version": "0.0.76",
   "description": "Next-gen file syncer",
   "license": "GPL-3.0",
   "main": "./dist/index.js",

package/dist/Watcher.js

@@ -1,51 +0,0 @@
-"use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.startWatching = startWatching;
-exports.stopWatching = stopWatching;
-const chokidar_1 = __importDefault(require("chokidar"));
-const logMessages_1 = require("./logMessages");
-const Logger_1 = require("./Logger");
-const lodash_1 = require("lodash");
-const FileUtils_1 = require("./FileUtils");
-const appUtils_1 = require("./appUtils");
-const recentEdits_1 = require("./recentEdits");
-const DEBOUNCE_MS = 300;
-let pushQueue = [];
-let watcher = undefined;
-const processQueue = (0, lodash_1.debounce)(async () => {
-    if (pushQueue.length > 0) {
-        //dedupe pushes
-        const toProcess = Array.from(new Set([...pushQueue]));
-        pushQueue = [];
-        const fileContexts = toProcess
-            .map(FileUtils_1.getFileContextFromPath)
-            .filter((ctx) => !!ctx);
-        const buildables = (0, appUtils_1.groupAppFiles)(fileContexts);
-        const updateResults = await (0, appUtils_1.pushFiles)(buildables);
-        updateResults.forEach((res, index) => {
-            (0, logMessages_1.logFilePush)(fileContexts[index], res);
-            if (res.success) {
-                (0, recentEdits_1.writeRecentEdit)(fileContexts[index]);
-            }
-        });
-    }
-}, DEBOUNCE_MS);
-function startWatching(directory) {
-    watcher = chokidar_1.default.watch(directory);
-    watcher.on("change", fileChanged);
-    watcher.on("error", (error) => {
-        Logger_1.logger.error(`Watcher error: ${error.message}`);
-    });
-}
-async function fileChanged(path) {
-    pushQueue.push(path);
-    processQueue();
-}
-function stopWatching() {
-    if (watcher) {
-        watcher.close();
-    }
-}

package/dist/tests/watcher.test.js

@@ -1,228 +0,0 @@
-"use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-function createMockWatcher() {
-    const handlers = {};
-    const mock = {
-        _handlers: handlers,
-        on: jest.fn((event, handler) => {
-            if (!handlers[event])
-                handlers[event] = [];
-            handlers[event].push(handler);
-            return mock;
-        }),
-        close: jest.fn(),
-        _emit: (event, ...args) => {
-            (handlers[event] || []).forEach((h) => h(...args));
-        },
-    };
-    return mock;
-}
-let latestMockWatcher;
-jest.mock("chokidar", () => ({
-    watch: jest.fn(() => {
-        latestMockWatcher = createMockWatcher();
-        return latestMockWatcher;
-    }),
-}));
-// Controllable debounce — captures fn, tests trigger manually
-let capturedProcessQueue;
-jest.mock("lodash", () => {
-    const actual = jest.requireActual("lodash");
-    return {
-        ...actual,
-        debounce: jest.fn((fn) => {
-            capturedProcessQueue = fn;
-            const wrapper = jest.fn((...args) => {
-                // Don't call automatically — tests call capturedProcessQueue()
-            });
-            wrapper.cancel = jest.fn();
-            wrapper.flush = jest.fn(() => fn());
-            return wrapper;
-        }),
-    };
-});
-jest.mock("../FileUtils", () => ({
-    getFileContextFromPath: jest.fn(),
-}));
-jest.mock("../appUtils", () => ({
-    groupAppFiles: jest.fn(),
-    pushFiles: jest.fn(),
-}));
-jest.mock("../logMessages", () => ({
-    logFilePush: jest.fn(),
-}));
-jest.mock("../Logger", () => ({
-    logger: {
-        info: jest.fn(),
-        error: jest.fn(),
-        warn: jest.fn(),
-        debug: jest.fn(),
-        success: jest.fn(),
-    },
-}));
-// --- Imports (after mocks) ---
-const chokidar_1 = __importDefault(require("chokidar"));
-const Watcher_1 = require("../Watcher");
-const FileUtils_1 = require("../FileUtils");
-const appUtils_1 = require("../appUtils");
-const logMessages_1 = require("../logMessages");
-const Logger_1 = require("../Logger");
-// --- Test Fixtures ---
-const MOCK_FILE_PATH = "/project/src/sys_script_include/TestScript/script.js";
-const MOCK_FILE_PATH_2 = "/project/src/sys_script_include/OtherScript/script.js";
-const makeFileContext = (overrides = {}) => ({
-    filePath: MOCK_FILE_PATH,
-    ext: ".js",
-    sys_id: "abc123",
-    name: "TestScript",
-    scope: "x_test",
-    tableName: "sys_script_include",
-    targetField: "script",
-    ...overrides,
-});
-const MOCK_BUILDABLE = {
-    table: "sys_script_include",
-    sysId: "abc123",
-    fields: {},
-};
-const MOCK_PUSH_SUCCESS = {
-    success: true,
-    message: "Pushed successfully",
-};
-// --- Tests ---
-describe("Watcher", () => {
-    beforeEach(() => {
-        jest.clearAllMocks();
-    });
-    afterEach(() => {
-        (0, Watcher_1.stopWatching)();
-    });
-    describe("startWatching", () => {
-        it("calls chokidar.watch with the given directory", () => {
-            (0, Watcher_1.startWatching)("/project/src");
-            expect(chokidar_1.default.watch).toHaveBeenCalledWith("/project/src");
-        });
-        it("registers change event handler", () => {
-            (0, Watcher_1.startWatching)("/project/src");
-            expect(latestMockWatcher.on).toHaveBeenCalledWith("change", expect.any(Function));
-        });
-        it("registers error event handler", () => {
-            (0, Watcher_1.startWatching)("/project/src");
-            expect(latestMockWatcher.on).toHaveBeenCalledWith("error", expect.any(Function));
-        });
-    });
-    describe("file change processing", () => {
-        it("processes a changed file through the full pipeline", async () => {
-            const ctx = makeFileContext();
-            FileUtils_1.getFileContextFromPath.mockReturnValue(ctx);
-            appUtils_1.groupAppFiles.mockReturnValue([MOCK_BUILDABLE]);
-            appUtils_1.pushFiles.mockResolvedValue([MOCK_PUSH_SUCCESS]);
-            (0, Watcher_1.startWatching)("/project/src");
-            latestMockWatcher._emit("change", MOCK_FILE_PATH);
-            // Manually trigger the debounced processQueue
-            await capturedProcessQueue();
-            expect(FileUtils_1.getFileContextFromPath).toHaveBeenCalledWith(MOCK_FILE_PATH, 0, [MOCK_FILE_PATH]);
-            expect(appUtils_1.groupAppFiles).toHaveBeenCalledWith([ctx]);
-            expect(appUtils_1.pushFiles).toHaveBeenCalledWith([MOCK_BUILDABLE]);
-            expect(logMessages_1.logFilePush).toHaveBeenCalledWith(ctx, MOCK_PUSH_SUCCESS);
-        });
-        it("deduplicates multiple changes to the same file", async () => {
-            const ctx = makeFileContext();
-            FileUtils_1.getFileContextFromPath.mockReturnValue(ctx);
-            appUtils_1.groupAppFiles.mockReturnValue([MOCK_BUILDABLE]);
-            appUtils_1.pushFiles.mockResolvedValue([MOCK_PUSH_SUCCESS]);
-            (0, Watcher_1.startWatching)("/project/src");
-            // Emit the same file path three times before processing
-            latestMockWatcher._emit("change", MOCK_FILE_PATH);
-            latestMockWatcher._emit("change", MOCK_FILE_PATH);
-            latestMockWatcher._emit("change", MOCK_FILE_PATH);
-            await capturedProcessQueue();
-            // Should only process the file once due to Set dedup
-            expect(FileUtils_1.getFileContextFromPath).toHaveBeenCalledTimes(1);
-        });
-        it("batches changes to multiple files", async () => {
-            const ctx1 = makeFileContext();
-            const ctx2 = makeFileContext({
-                filePath: MOCK_FILE_PATH_2,
-                sys_id: "def456",
-                name: "OtherScript",
-            });
-            FileUtils_1.getFileContextFromPath
-                .mockReturnValueOnce(ctx1)
-                .mockReturnValueOnce(ctx2);
-            appUtils_1.groupAppFiles.mockReturnValue([MOCK_BUILDABLE]);
-            appUtils_1.pushFiles.mockResolvedValue([MOCK_PUSH_SUCCESS]);
-            (0, Watcher_1.startWatching)("/project/src");
-            latestMockWatcher._emit("change", MOCK_FILE_PATH);
-            latestMockWatcher._emit("change", MOCK_FILE_PATH_2);
-            await capturedProcessQueue();
-            expect(FileUtils_1.getFileContextFromPath).toHaveBeenCalledTimes(2);
-            expect(appUtils_1.groupAppFiles).toHaveBeenCalledWith([ctx1, ctx2]);
-        });
-        it("filters out files where getFileContextFromPath returns undefined", async () => {
-            const ctx = makeFileContext();
-            FileUtils_1.getFileContextFromPath
-                .mockReturnValueOnce(undefined)
-                .mockReturnValueOnce(ctx);
-            appUtils_1.groupAppFiles.mockReturnValue([MOCK_BUILDABLE]);
-            appUtils_1.pushFiles.mockResolvedValue([MOCK_PUSH_SUCCESS]);
-            (0, Watcher_1.startWatching)("/project/src");
-            latestMockWatcher._emit("change", "/project/src/unknown/file.txt");
-            latestMockWatcher._emit("change", MOCK_FILE_PATH);
-            await capturedProcessQueue();
-            // groupAppFiles should only receive the valid context
-            expect(appUtils_1.groupAppFiles).toHaveBeenCalledWith([ctx]);
-        });
-        it("does not process when queue is empty", async () => {
-            (0, Watcher_1.startWatching)("/project/src");
-            // Trigger processQueue without emitting any events
-            await capturedProcessQueue();
-            expect(FileUtils_1.getFileContextFromPath).not.toHaveBeenCalled();
-            expect(appUtils_1.groupAppFiles).not.toHaveBeenCalled();
-            expect(appUtils_1.pushFiles).not.toHaveBeenCalled();
-        });
-        it("calls logFilePush for each result paired with its file context", async () => {
-            const ctx1 = makeFileContext();
-            const ctx2 = makeFileContext({
-                filePath: MOCK_FILE_PATH_2,
-                sys_id: "def456",
-                name: "OtherScript",
-            });
-            const result1 = { success: true, message: "ok" };
-            const result2 = { success: false, message: "failed" };
-            FileUtils_1.getFileContextFromPath
-                .mockReturnValueOnce(ctx1)
-                .mockReturnValueOnce(ctx2);
-            appUtils_1.groupAppFiles.mockReturnValue([MOCK_BUILDABLE, MOCK_BUILDABLE]);
-            appUtils_1.pushFiles.mockResolvedValue([result1, result2]);
-            (0, Watcher_1.startWatching)("/project/src");
-            latestMockWatcher._emit("change", MOCK_FILE_PATH);
-            latestMockWatcher._emit("change", MOCK_FILE_PATH_2);
-            await capturedProcessQueue();
-            expect(logMessages_1.logFilePush).toHaveBeenCalledTimes(2);
-            expect(logMessages_1.logFilePush).toHaveBeenCalledWith(ctx1, result1);
-            expect(logMessages_1.logFilePush).toHaveBeenCalledWith(ctx2, result2);
-        });
-    });
-    describe("error handling", () => {
-        it("logs error when chokidar emits error event", () => {
-            (0, Watcher_1.startWatching)("/project/src");
-            const error = new Error("Watch failed");
-            latestMockWatcher._emit("error", error);
-            expect(Logger_1.logger.error).toHaveBeenCalledWith("Watcher error: Watch failed");
-        });
-    });
-    describe("stopWatching", () => {
-        it("calls watcher.close() when watcher exists", () => {
-            (0, Watcher_1.startWatching)("/project/src");
-            (0, Watcher_1.stopWatching)();
-            expect(latestMockWatcher.close).toHaveBeenCalled();
-        });
-        it("does not throw when called before startWatching", () => {
-            expect(() => (0, Watcher_1.stopWatching)()).not.toThrow();
-        });
-    });
-});