@magnusekdahl/parallix 1.2.0 → 1.3.0

package/lib/core/gitignore.js

@@ -1,112 +1,65 @@
-const fs = require('fs');
-const path = require('path');
-
-/**
- * Workflow-generated paths that must be gitignored in enterprise repositories.
- * These are paths created by the px workflow toolkit that should never be
- * committed to version control.
- */
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+const node_fs_1 = __importDefault(require("node:fs"));
+const node_path_1 = __importDefault(require("node:path"));
 const WORKFLOW_ENTRIES = [
-  '.workflow/',
-  '.sessions/',
-  '.forgejo-local/',
-  'workflow/.cache/',
-  'workflow/.sessions/',
-  'workflow/config/agents.local.json',
-  'agents.local.json',
+    '.workflow/',
+    '.sessions/',
+    '.forgejo-local/',
+    'workflow/.cache/',
+    'workflow/.sessions/',
+    'workflow/config/agents.local.json',
+    'agents.local.json',
 ];
-
-/**
- * Ensure that a repository's .gitignore contains all workflow-generated paths.
- *
- * Handles:
- * - Missing .gitignore: creates one with all 7 entries
- * - Existing .gitignore: appends only missing entries, skips duplicates
- * - Symlinked .gitignore: skips modification, logs warning
- * - Non-git directories: skips gracefully without crashing
- *
- * @param {string} rootDir - Path to the repository root
- * @param {object} options - Injected dependencies
- * @param {Function} [options.existsSyncFn] - fs.existsSync replacement
- * @param {Function} [options.lstatSyncFn] - fs.lstatSync replacement
- * @param {Function} [options.readFileSyncFn] - fs.readFileSync replacement
- * @param {Function} [options.writeFileSyncFn] - fs.writeFileSync replacement
- * @param {Function} [options.logFn] - Logger function
- * @returns {{ ok: boolean, created: boolean, appended: number, skipped: boolean, reason?: string }}
- */
 function ensureWorkflowGitignore(rootDir, options = {}) {
-  const {
-    existsSyncFn = fs.existsSync,
-    lstatSyncFn = fs.lstatSync,
-    readFileSyncFn = fs.readFileSync,
-    writeFileSyncFn = fs.writeFileSync,
-    logFn = () => {},
-  } = options;
-
-  const gitignorePath = path.join(rootDir, '.gitignore');
-  const gitDirPath = path.join(rootDir, '.git');
-
-  // Check if this is a git repository
-  if (!existsSyncFn(gitDirPath)) {
-    logFn(`[gitignore] Skipping ${rootDir}: not a git repository (no .git directory)`);
-    return { ok: true, created: false, appended: 0, skipped: true, reason: 'not-a-git-repo' };
-  }
-
-  // Check if .gitignore exists
-  if (!existsSyncFn(gitignorePath)) {
-    // Create new .gitignore with all entries
-    const content = WORKFLOW_ENTRIES.join('\n') + '\n';
-    writeFileSyncFn(gitignorePath, content, 'utf8');
-    logFn(`[gitignore] Created ${gitignorePath} with ${WORKFLOW_ENTRIES.length} entries`);
-    return { ok: true, created: true, appended: WORKFLOW_ENTRIES.length, skipped: false };
-  }
-
-  // .gitignore exists - check for symlink
-  try {
-    const stat = lstatSyncFn(gitignorePath);
-    if (stat.isSymbolicLink()) {
-      logFn(`[gitignore] Skipping ${gitignorePath}: symbolic link, not modifying`);
-      return { ok: true, created: false, appended: 0, skipped: true, reason: 'symlink' };
+    const { existsSyncFn = node_fs_1.default.existsSync, lstatSyncFn = node_fs_1.default.lstatSync, readFileSyncFn = node_fs_1.default.readFileSync, writeFileSyncFn = node_fs_1.default.writeFileSync, logFn = () => { }, } = options;
+    const gitignorePath = node_path_1.default.join(rootDir, '.gitignore');
+    const gitDirPath = node_path_1.default.join(rootDir, '.git');
+    if (!existsSyncFn(gitDirPath)) {
+        logFn(`[gitignore] Skipping ${rootDir}: not a git repository (no .git directory)`);
+        return { ok: true, created: false, appended: 0, skipped: true, reason: 'not-a-git-repo' };
     }
-  } catch (_) {
-    // If we can't stat, fall through to read - it might exist but be inaccessible
-  }
-
-  // Read existing .gitignore
-  let existingContent;
-  try {
-    existingContent = readFileSyncFn(gitignorePath, 'utf8');
-  } catch (err) {
-    logFn(`[gitignore] Could not read ${gitignorePath}: ${/** @type{Error} */(err).message}`);
-    return { ok: false, created: false, appended: 0, skipped: false, reason: `read-error: ${/** @type{Error} */(err).message}` };
-  }
-
-  // Parse existing entries
-  const existingLines = existingContent.split(/\r?\n/);
-  const existingEntries = new Set(
-    existingLines
-      /** @param {string} line */
-      .map(/** @param {string} line */(line) => line.trim())
-      /** @param {string} line */
-      .filter(/** @param {string} line */(line) => line.length > 0 && !line.startsWith('#'))
-  );
-
-  // Find missing entries
-  const missingEntries = WORKFLOW_ENTRIES.filter(entry => !existingEntries.has(entry));
-
-  if (missingEntries.length === 0) {
-    logFn(`[gitignore] ${gitignorePath} already contains all ${WORKFLOW_ENTRIES.length} workflow entries`);
-    return { ok: true, created: false, appended: 0, skipped: false };
-  }
-
-  // Append missing entries
-  const newContent = existingContent + '\n' + missingEntries.join('\n') + '\n';
-  writeFileSyncFn(gitignorePath, newContent, 'utf8');
-  logFn(`[gitignore] Appended ${missingEntries.length} missing entries to ${gitignorePath}`);
-
-  return { ok: true, created: false, appended: missingEntries.length, skipped: false };
+    if (!existsSyncFn(gitignorePath)) {
+        const content = [...WORKFLOW_ENTRIES].join('\n') + '\n';
+        writeFileSyncFn(gitignorePath, content, 'utf8');
+        logFn(`[gitignore] Created ${gitignorePath} with ${WORKFLOW_ENTRIES.length} entries`);
+        return { ok: true, created: true, appended: WORKFLOW_ENTRIES.length, skipped: false };
+    }
+    try {
+        const stat = lstatSyncFn(gitignorePath);
+        if (stat.isSymbolicLink()) {
+            logFn(`[gitignore] Skipping ${gitignorePath}: symbolic link, not modifying`);
+            return { ok: true, created: false, appended: 0, skipped: true, reason: 'symlink' };
+        }
+    }
+    catch (_err) {
+        // If we can't stat, fall through to read
+    }
+    let existingContent;
+    try {
+        existingContent = readFileSyncFn(gitignorePath, 'utf8');
+    }
+    catch (err) {
+        const message = err instanceof Error ? err.message : String(err);
+        logFn(`[gitignore] Could not read ${gitignorePath}: ${message}`);
+        return { ok: false, created: false, appended: 0, skipped: false, reason: `read-error: ${message}` };
+    }
+    const existingLines = existingContent.split(/\r?\n/);
+    const existingEntries = new Set(existingLines
+        .map((line) => line.trim())
+        .filter((line) => line.length > 0 && !line.startsWith('#')));
+    const missingEntries = WORKFLOW_ENTRIES.filter(entry => !existingEntries.has(entry));
+    if (missingEntries.length === 0) {
+        logFn(`[gitignore] ${gitignorePath} already contains all ${WORKFLOW_ENTRIES.length} workflow entries`);
+        return { ok: true, created: false, appended: 0, skipped: false };
+    }
+    const newContent = existingContent + '\n' + [...missingEntries].join('\n') + '\n';
+    writeFileSyncFn(gitignorePath, newContent, 'utf8');
+    logFn(`[gitignore] Appended ${missingEntries.length} missing entries to ${gitignorePath}`);
+    return { ok: true, created: false, appended: missingEntries.length, skipped: false };
 }
-
+ensureWorkflowGitignore.WORKFLOW_ENTRIES = WORKFLOW_ENTRIES;
+ensureWorkflowGitignore.ensureWorkflowGitignore = ensureWorkflowGitignore;
 module.exports = ensureWorkflowGitignore;
-module.exports.ensureWorkflowGitignore = ensureWorkflowGitignore;
-module.exports.WORKFLOW_ENTRIES = WORKFLOW_ENTRIES;

package/lib/core/gitignore.ts

@@ -0,0 +1,103 @@
+import fs from 'node:fs';
+import path from 'node:path';
+
+const WORKFLOW_ENTRIES = [
+  '.workflow/',
+  '.sessions/',
+  '.forgejo-local/',
+  'workflow/.cache/',
+  'workflow/.sessions/',
+  'workflow/config/agents.local.json',
+  'agents.local.json',
+] as const;
+
+interface GitignoreResult {
+  ok: boolean;
+  created: boolean;
+  appended: number;
+  skipped: boolean;
+  reason?: string;
+}
+
+interface EnsureOptions {
+  existsSyncFn?: typeof fs.existsSync;
+  lstatSyncFn?: typeof fs.lstatSync;
+  readFileSyncFn?: typeof fs.readFileSync;
+  writeFileSyncFn?: typeof fs.writeFileSync;
+  logFn?: (msg: string) => void;
+}
+
+type EnsureWorkflowGitignoreFn = {
+  (rootDir: string, options?: EnsureOptions): GitignoreResult;
+  WORKFLOW_ENTRIES: typeof WORKFLOW_ENTRIES;
+  ensureWorkflowGitignore: EnsureWorkflowGitignoreFn;
+};
+
+function ensureWorkflowGitignore(rootDir: string, options: EnsureOptions = {}): GitignoreResult {
+  const {
+    existsSyncFn = fs.existsSync,
+    lstatSyncFn = fs.lstatSync,
+    readFileSyncFn = fs.readFileSync,
+    writeFileSyncFn = fs.writeFileSync,
+    logFn = () => {},
+  } = options;
+
+  const gitignorePath = path.join(rootDir, '.gitignore');
+  const gitDirPath = path.join(rootDir, '.git');
+
+  if (!existsSyncFn(gitDirPath)) {
+    logFn(`[gitignore] Skipping ${rootDir}: not a git repository (no .git directory)`);
+    return { ok: true, created: false, appended: 0, skipped: true, reason: 'not-a-git-repo' };
+  }
+
+  if (!existsSyncFn(gitignorePath)) {
+    const content = [...WORKFLOW_ENTRIES].join('\n') + '\n';
+    writeFileSyncFn(gitignorePath, content, 'utf8');
+    logFn(`[gitignore] Created ${gitignorePath} with ${WORKFLOW_ENTRIES.length} entries`);
+    return { ok: true, created: true, appended: WORKFLOW_ENTRIES.length, skipped: false };
+  }
+
+  try {
+    const stat = lstatSyncFn(gitignorePath);
+    if (stat.isSymbolicLink()) {
+      logFn(`[gitignore] Skipping ${gitignorePath}: symbolic link, not modifying`);
+      return { ok: true, created: false, appended: 0, skipped: true, reason: 'symlink' };
+    }
+  } catch (_err: unknown) {
+    // If we can't stat, fall through to read
+  }
+
+  let existingContent: string;
+  try {
+    existingContent = readFileSyncFn(gitignorePath, 'utf8');
+  } catch (err: unknown) {
+    const message = err instanceof Error ? err.message : String(err);
+    logFn(`[gitignore] Could not read ${gitignorePath}: ${message}`);
+    return { ok: false, created: false, appended: 0, skipped: false, reason: `read-error: ${message}` };
+  }
+
+  const existingLines = existingContent.split(/\r?\n/);
+  const existingEntries = new Set(
+    existingLines
+      .map((line: string) => line.trim())
+      .filter((line: string) => line.length > 0 && !line.startsWith('#'))
+  );
+
+  const missingEntries = WORKFLOW_ENTRIES.filter(entry => !existingEntries.has(entry));
+
+  if (missingEntries.length === 0) {
+    logFn(`[gitignore] ${gitignorePath} already contains all ${WORKFLOW_ENTRIES.length} workflow entries`);
+    return { ok: true, created: false, appended: 0, skipped: false };
+  }
+
+  const newContent = existingContent + '\n' + [...missingEntries].join('\n') + '\n';
+  writeFileSyncFn(gitignorePath, newContent, 'utf8');
+  logFn(`[gitignore] Appended ${missingEntries.length} missing entries to ${gitignorePath}`);
+
+  return { ok: true, created: false, appended: missingEntries.length, skipped: false };
+}
+
+(ensureWorkflowGitignore as EnsureWorkflowGitignoreFn).WORKFLOW_ENTRIES = WORKFLOW_ENTRIES;
+(ensureWorkflowGitignore as EnsureWorkflowGitignoreFn).ensureWorkflowGitignore = ensureWorkflowGitignore as EnsureWorkflowGitignoreFn;
+
+export = ensureWorkflowGitignore;