trident-git 0.2.0 → 0.2.2

package/README.md

@@ -125,6 +125,8 @@ npx trident-git --port 3200
 npx trident-git --dev
 ```
 
+Published npm packages are expected to include a prebuilt `.next` output, so `npx trident-git` does not build on the end user's machine.
+
 ### Installation
 
 ```bash

package/bin/trident-git.mjs

@@ -2,6 +2,7 @@
 
 import { spawn } from "node:child_process";
 import fs from "node:fs";
+import { createRequire } from "node:module";
 import net from "node:net";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
@@ -9,7 +10,8 @@ import { fileURLToPath } from "node:url";
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 const APP_ROOT = path.resolve(__dirname, "..");
-const NEXT_BIN = path.join(APP_ROOT, "node_modules", "next", "dist", "bin", "next");
+const require = createRequire(import.meta.url);
+const NEXT_BIN = require.resolve("next/dist/bin/next");
 const DEFAULT_PORT = 3100;
 
 function printHelp() {
@@ -106,16 +108,12 @@ function runNext(args) {
   });
 }
 
-async function ensureBuildExists() {
+function ensureBuildExists() {
   const buildIdPath = path.join(APP_ROOT, ".next", "BUILD_ID");
-  if (fs.existsSync(buildIdPath)) {
-    return;
-  }
-
-  console.log("No production build found. Building trident-git...");
-  const exitCode = await runNext(["build", "--webpack"]);
-  if (exitCode !== 0) {
-    process.exit(exitCode);
+  if (!fs.existsSync(buildIdPath)) {
+    throw new Error(
+      "No production build found in this package (.next/BUILD_ID is missing). The npm package must be published with prebuilt assets.",
+    );
   }
 }
 
@@ -140,7 +138,7 @@ async function main() {
       process.exit(await runNext(["dev", "--webpack", "-p", String(port)]));
     }
 
-    await ensureBuildExists();
+    ensureBuildExists();
     console.log(`Starting trident-git on http://localhost:${port}`);
     process.exit(await runNext(["start", "-p", String(port)]));
   } catch (error) {

package/{next.config.ts → next.config.mjs}

@@ -1,6 +1,5 @@
-import type { NextConfig } from "next";
-
-const nextConfig: NextConfig = {
+/** @type {import('next').NextConfig} */
+const nextConfig = {
   // Disable Turbopack for production builds as keytar is a native module
   // that requires webpack's externals configuration
   experimental: {
@@ -11,20 +10,20 @@ const nextConfig: NextConfig = {
     if (isServer) {
       // Externalize keytar - it's a native module that should be loaded at runtime
       config.externals = config.externals || [];
-      config.externals.push('keytar');
+      config.externals.push("keytar");
     }
     return config;
   },
   images: {
     remotePatterns: [
       {
-        protocol: 'https',
-        hostname: 'www.google.com',
-        pathname: '/s2/favicons',
+        protocol: "https",
+        hostname: "www.google.com",
+        pathname: "/s2/favicons",
       },
     ],
   },
-  serverExternalPackages: ['keytar'],
+  serverExternalPackages: ["keytar"],
 };
 
 export default nextConfig;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "trident-git",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "private": false,
   "description": "A modern, web-based Git client.",
   "bin": {
@@ -8,17 +8,19 @@
   },
   "files": [
     "bin",
-    "src",
+    ".next/BUILD_ID",
+    ".next/server",
+    ".next/static",
+    ".next/*.json",
+    ".next/*.js",
     "public",
-    "next.config.ts",
-    "postcss.config.mjs",
-    "tsconfig.json",
-    "eslint.config.mjs",
+    "next.config.mjs",
     "README.md"
   ],
   "scripts": {
     "dev": "npx gfport 3100 -- next dev --webpack -p {PORT}",
     "build": "next build --webpack",
+    "prepack": "rm -rf .next && npm run build",
     "start": "next start -p 3100",
     "lint": "eslint",
     "cli": "node ./bin/trident-git.mjs",

package/eslint.config.mjs

@@ -1,18 +0,0 @@
-import { defineConfig, globalIgnores } from "eslint/config";
-import nextVitals from "eslint-config-next/core-web-vitals";
-import nextTs from "eslint-config-next/typescript";
-
-const eslintConfig = defineConfig([
-  ...nextVitals,
-  ...nextTs,
-  // Override default ignores of eslint-config-next.
-  globalIgnores([
-    // Default ignores of eslint-config-next:
-    ".next/**",
-    "out/**",
-    "build/**",
-    "next-env.d.ts",
-  ]),
-]);
-
-export default eslintConfig;

package/postcss.config.mjs

@@ -1,7 +0,0 @@
-const config = {
-  plugins: {
-    "@tailwindcss/postcss": {},
-  },
-};
-
-export default config;

package/src/app/api/credentials/route.ts

@@ -1,113 +0,0 @@
-import { NextResponse } from 'next/server';
-import { z } from 'zod';
-import {
-  getAllCredentials,
-  createGitHubCredential,
-  createGitLabCredential,
-  updateCredential,
-  deleteCredential,
-} from '@/lib/credentials';
-
-// GET - List all credentials
-export async function GET() {
-  try {
-    const credentials = await getAllCredentials();
-    return NextResponse.json(credentials);
-  } catch (error) {
-    console.error('Failed to get credentials:', error);
-    return NextResponse.json({ error: 'Failed to get credentials' }, { status: 500 });
-  }
-}
-
-// POST - Create a new credential
-const createGitHubSchema = z.object({
-  type: z.literal('github'),
-  token: z.string().min(1, 'Token is required'),
-});
-
-const createGitLabSchema = z.object({
-  type: z.literal('gitlab'),
-  serverUrl: z.string().url('Invalid server URL'),
-  token: z.string().min(1, 'Token is required'),
-});
-
-const createSchema = z.discriminatedUnion('type', [createGitHubSchema, createGitLabSchema]);
-
-export async function POST(request: Request) {
-  try {
-    const body = await request.json();
-    const data = createSchema.parse(body);
-
-    let result;
-    if (data.type === 'github') {
-      result = await createGitHubCredential(data.token);
-    } else {
-      result = await createGitLabCredential(data.serverUrl, data.token);
-    }
-
-    if (!result.success) {
-      return NextResponse.json({ error: result.error }, { status: 400 });
-    }
-
-    return NextResponse.json(result.credential);
-  } catch (error) {
-    if (error instanceof z.ZodError) {
-      return NextResponse.json({ error: error.issues[0].message }, { status: 400 });
-    }
-    console.error('Failed to create credential:', error);
-    return NextResponse.json({ error: 'Failed to create credential' }, { status: 500 });
-  }
-}
-
-// PUT - Update a credential
-const updateSchema = z.object({
-  id: z.string().min(1, 'Credential ID is required'),
-  token: z.string().min(1, 'Token is required'),
-});
-
-export async function PUT(request: Request) {
-  try {
-    const body = await request.json();
-    const { id, token } = updateSchema.parse(body);
-
-    const result = await updateCredential(id, token);
-
-    if (!result.success) {
-      return NextResponse.json({ error: result.error }, { status: 400 });
-    }
-
-    return NextResponse.json(result.credential);
-  } catch (error) {
-    if (error instanceof z.ZodError) {
-      return NextResponse.json({ error: error.issues[0].message }, { status: 400 });
-    }
-    console.error('Failed to update credential:', error);
-    return NextResponse.json({ error: 'Failed to update credential' }, { status: 500 });
-  }
-}
-
-// DELETE - Delete a credential
-const deleteSchema = z.object({
-  id: z.string().min(1, 'Credential ID is required'),
-});
-
-export async function DELETE(request: Request) {
-  try {
-    const body = await request.json();
-    const { id } = deleteSchema.parse(body);
-
-    const result = await deleteCredential(id);
-
-    if (!result.success) {
-      return NextResponse.json({ error: result.error }, { status: 400 });
-    }
-
-    return NextResponse.json({ success: true });
-  } catch (error) {
-    if (error instanceof z.ZodError) {
-      return NextResponse.json({ error: error.issues[0].message }, { status: 400 });
-    }
-    console.error('Failed to delete credential:', error);
-    return NextResponse.json({ error: 'Failed to delete credential' }, { status: 500 });
-  }
-}

package/src/app/api/custom-scripts/route.ts

@@ -1,203 +0,0 @@
-import { NextResponse } from 'next/server';
-import { spawn, ChildProcessWithoutNullStreams } from 'node:child_process';
-import fs from 'node:fs';
-import { z } from 'zod';
-import { GitService } from '@/lib/git';
-
-export const runtime = 'nodejs';
-
-type ExecutionStatus = 'running' | 'completed' | 'failed' | 'canceled';
-
-interface ScriptExecution {
-  id: string;
-  status: ExecutionStatus;
-  cancelRequested: boolean;
-  output: string;
-  exitCode: number | null;
-  signal: NodeJS.Signals | null;
-  startedAt: string;
-  finishedAt: string | null;
-  process: ChildProcessWithoutNullStreams | null;
-}
-
-const executions = new Map<string, ScriptExecution>();
-const MAX_OUTPUT_LENGTH = 500_000;
-const FINISHED_EXECUTION_TTL_MS = 30 * 60 * 1000;
-
-const startSchema = z.object({
-  command: z.literal('start'),
-  repoPath: z.string().min(1),
-  branchRef: z.string().min(1),
-  scriptContent: z.string(),
-});
-
-const statusSchema = z.object({
-  command: z.literal('status'),
-  executionId: z.string().min(1),
-});
-
-const cancelSchema = z.object({
-  command: z.literal('cancel'),
-  executionId: z.string().min(1),
-});
-
-const requestSchema = z.discriminatedUnion('command', [startSchema, statusSchema, cancelSchema]);
-
-function normalizeBranchForCheckout(branchRef: string): string {
-  return branchRef.startsWith('remotes/') ? branchRef.slice('remotes/'.length) : branchRef;
-}
-
-function appendOutput(execution: ScriptExecution, text: string) {
-  execution.output += text;
-  if (execution.output.length > MAX_OUTPUT_LENGTH) {
-    execution.output = execution.output.slice(execution.output.length - MAX_OUTPUT_LENGTH);
-  }
-}
-
-function toResponsePayload(execution: ScriptExecution) {
-  return {
-    executionId: execution.id,
-    status: execution.status,
-    cancelRequested: execution.cancelRequested,
-    output: execution.output,
-    exitCode: execution.exitCode,
-    signal: execution.signal,
-    startedAt: execution.startedAt,
-    finishedAt: execution.finishedAt,
-  };
-}
-
-function cleanupFinishedExecutions() {
-  const now = Date.now();
-  for (const [id, execution] of executions.entries()) {
-    if (execution.status === 'running') continue;
-    if (!execution.finishedAt) continue;
-    const finishedAtMs = new Date(execution.finishedAt).getTime();
-    if (Number.isNaN(finishedAtMs)) continue;
-    if (now - finishedAtMs > FINISHED_EXECUTION_TTL_MS) {
-      executions.delete(id);
-    }
-  }
-}
-
-export async function POST(request: Request) {
-  try {
-    cleanupFinishedExecutions();
-
-    const body = await request.json();
-    const payload = requestSchema.parse(body);
-
-    if (payload.command === 'status') {
-      const execution = executions.get(payload.executionId);
-      if (!execution) {
-        return NextResponse.json({ error: 'Execution not found' }, { status: 404 });
-      }
-
-      return NextResponse.json({ success: true, ...toResponsePayload(execution) });
-    }
-
-    if (payload.command === 'cancel') {
-      const execution = executions.get(payload.executionId);
-      if (!execution) {
-        return NextResponse.json({ error: 'Execution not found' }, { status: 404 });
-      }
-
-      if (execution.status === 'running' && execution.process) {
-        execution.cancelRequested = true;
-        appendOutput(execution, '\n[info] Cancel requested...\n');
-        execution.process.kill('SIGTERM');
-
-        setTimeout(() => {
-          if (execution.process && execution.cancelRequested) {
-            execution.process.kill('SIGKILL');
-          }
-        }, 1500);
-      }
-
-      return NextResponse.json({ success: true, ...toResponsePayload(execution) });
-    }
-
-    const { repoPath, branchRef, scriptContent } = payload;
-
-    if (!fs.existsSync(repoPath)) {
-      return NextResponse.json({ error: `Path not found: ${repoPath}` }, { status: 404 });
-    }
-
-    if (!fs.statSync(repoPath).isDirectory()) {
-      return NextResponse.json({ error: 'Repository path must be a directory' }, { status: 400 });
-    }
-
-    const git = new GitService(repoPath);
-    const branches = await git.getBranches();
-    const checkoutBranch = normalizeBranchForCheckout(branchRef);
-    const currentBranch = branches.current;
-
-    if (!currentBranch || currentBranch !== checkoutBranch) {
-      await git.checkout(checkoutBranch);
-    }
-
-    const executionId = crypto.randomUUID();
-    const child = spawn('bash', ['-s'], {
-      cwd: repoPath,
-      env: process.env,
-      stdio: 'pipe',
-    });
-
-    const execution: ScriptExecution = {
-      id: executionId,
-      status: 'running',
-      cancelRequested: false,
-      output: '',
-      exitCode: null,
-      signal: null,
-      startedAt: new Date().toISOString(),
-      finishedAt: null,
-      process: child,
-    };
-
-    executions.set(executionId, execution);
-
-    const onData = (chunk: Buffer | string) => {
-      appendOutput(execution, typeof chunk === 'string' ? chunk : chunk.toString('utf-8'));
-    };
-
-    child.stdout.on('data', onData);
-    child.stderr.on('data', onData);
-
-    child.on('error', (error) => {
-      appendOutput(execution, `\n[error] ${error.message}\n`);
-      execution.status = execution.cancelRequested ? 'canceled' : 'failed';
-      execution.finishedAt = new Date().toISOString();
-      execution.process = null;
-    });
-
-    child.on('close', (code, signal) => {
-      execution.exitCode = code;
-      execution.signal = signal;
-      execution.finishedAt = new Date().toISOString();
-      execution.process = null;
-
-      if (execution.cancelRequested) {
-        execution.status = 'canceled';
-        return;
-      }
-
-      execution.status = code === 0 ? 'completed' : 'failed';
-    });
-
-    child.stdin.write(scriptContent);
-    child.stdin.end();
-
-    return NextResponse.json({
-      success: true,
-      ...toResponsePayload(execution),
-      checkedOutBranch: checkoutBranch,
-      previousBranch: currentBranch,
-    });
-  } catch (error) {
-    if (error instanceof z.ZodError) {
-      return NextResponse.json({ error: error.issues }, { status: 400 });
-    }
-    return NextResponse.json({ error: (error as Error).message }, { status: 500 });
-  }
-}

package/src/app/api/fs/route.ts

@@ -1,75 +0,0 @@
-import { NextResponse } from 'next/server';
-import fs from 'fs';
-import path from 'path';
-import os from 'os';
-
-export async function GET(request: Request) {
-  const { searchParams } = new URL(request.url);
-  const requestedPath = searchParams.get('path');
-
-  // Default to home directory
-  const currentPath = requestedPath ? requestedPath : os.homedir();
-
-  try {
-      // Security check: though this is local app, basic sanity check
-      // For now, allow reading anywhere as it's a dev tool/local tool.
-      
-      const stats = await fs.promises.stat(currentPath);
-      if (!stats.isDirectory()) {
-          return NextResponse.json({ error: 'Not a directory' }, { status: 400 });
-      }
-
-      const items = await fs.promises.readdir(currentPath, { withFileTypes: true });
-      
-      // Let's verify if they are git repos
-      const directories = items.filter(item => item.isDirectory());
-
-      const contents = [];
-      const BATCH_SIZE = 50;
-
-      for (let i = 0; i < directories.length; i += BATCH_SIZE) {
-          const batch = directories.slice(i, i + BATCH_SIZE);
-          const batchResults = await Promise.all(batch.map(async (item) => {
-              const itemPath = path.join(currentPath, item.name);
-              let isRepo = false;
-              try {
-                  // Check for .git directory inside
-                  await fs.promises.access(path.join(itemPath, '.git'), fs.constants.F_OK);
-                  isRepo = true;
-              } catch (e) {}
-
-              return {
-                  name: item.name,
-                  path: itemPath,
-                  isRepo
-              };
-          }));
-          contents.push(...batchResults);
-      }
-
-      // Sort: Visible folders first, then Repos first within that group, then alphabetical
-      contents.sort((a, b) => {
-          const aHidden = a.name.startsWith('.');
-          const bHidden = b.name.startsWith('.');
-
-          // Visible first
-          if (!aHidden && bHidden) return -1;
-          if (aHidden && !bHidden) return 1;
-
-          // Repos first
-          if (a.isRepo && !b.isRepo) return -1;
-          if (!a.isRepo && b.isRepo) return 1;
-
-          return a.name.localeCompare(b.name);
-      });
-
-      return NextResponse.json({
-          path: currentPath,
-          folders: contents,
-          parent: path.dirname(currentPath)
-      });
-      
-  } catch (error) {
-      return NextResponse.json({ error: (error as Error).message }, { status: 500 });
-  }
-}