@minimalcorp/tsunagi 0.1.31 → 0.1.33

package/dist/server/lib/repositories/repository.js

@@ -1,16 +1,20 @@
 import { prisma } from '../db.js';
-// リポジトリ一覧取得
-export async function getRepos() {
-    const repos = await prisma.repository.findMany({
-        orderBy: { createdAt: 'desc' },
-    });
-    return repos.map((repo) => ({
+function mapRepo(repo) {
+    return {
         id: repo.id,
         owner: repo.owner,
         repo: repo.repo,
         cloneUrl: repo.cloneUrl,
+        order: repo.order,
         createdAt: repo.createdAt.toISOString(),
-    }));
+    };
+}
+// リポジトリ一覧取得（表示順）
+export async function getRepos() {
+    const repos = await prisma.repository.findMany({
+        orderBy: [{ order: 'asc' }, { createdAt: 'asc' }],
+    });
+    return repos.map(mapRepo);
 }
 // リポジトリ取得
 export async function getRepo(owner, repo) {
@@ -19,15 +23,9 @@ export async function getRepo(owner, repo) {
     });
     if (!repository)
         return null;
-    return {
-        id: repository.id,
-        owner: repository.owner,
-        repo: repository.repo,
-        cloneUrl: repository.cloneUrl,
-        createdAt: repository.createdAt.toISOString(),
-    };
+    return mapRepo(repository);
 }
-// リポジトリ作成
+// リポジトリ作成（表示順は末尾に追加）
 export async function createRepo(repo) {
     const existing = await prisma.repository.findFirst({
         where: { owner: repo.owner, repo: repo.repo },
@@ -35,20 +33,16 @@ export async function createRepo(repo) {
     if (existing) {
         throw new Error(`Repository ${repo.owner}/${repo.repo} already exists`);
     }
+    const last = await prisma.repository.findFirst({ orderBy: { order: 'desc' } });
     const newRepo = await prisma.repository.create({
         data: {
             owner: repo.owner,
             repo: repo.repo,
             cloneUrl: repo.cloneUrl,
+            order: last ? last.order + 1 : 0,
         },
     });
-    return {
-        id: newRepo.id,
-        owner: newRepo.owner,
-        repo: newRepo.repo,
-        cloneUrl: newRepo.cloneUrl,
-        createdAt: newRepo.createdAt.toISOString(),
-    };
+    return mapRepo(newRepo);
 }
 // リポジトリ更新
 export async function updateRepo(id, updates) {
@@ -59,15 +53,19 @@ export async function updateRepo(id, updates) {
         where: { id },
         data: {
             ...(updates.cloneUrl && { cloneUrl: updates.cloneUrl }),
+            ...(updates.order !== undefined && { order: updates.order }),
         },
     });
-    return {
-        id: updatedRepo.id,
-        owner: updatedRepo.owner,
-        repo: updatedRepo.repo,
-        cloneUrl: updatedRepo.cloneUrl,
-        createdAt: updatedRepo.createdAt.toISOString(),
-    };
+    return mapRepo(updatedRepo);
+}
+/**
+ * 表示順を一括更新する。
+ * 渡された id の並びをそのまま order 0..n-1 に割り当てるため、
+ * タスクの order のような玉突き処理は不要。
+ */
+export async function reorderRepos(repoIds) {
+    await prisma.$transaction(repoIds.map((id, index) => prisma.repository.update({ where: { id }, data: { order: index } })));
+    return getRepos();
 }
 // リポジトリ削除
 export async function deleteRepo(id) {

package/dist/server/lib/worktree-manager.js

@@ -7,6 +7,18 @@ import { promisify } from 'util';
 import { normalizeBranchName } from './branch-utils.js';
 const execAsync = promisify(exec);
 const WORKSPACES_ROOT = path.join(os.homedir(), '.tsunagi', 'workspaces');
+/**
+ * サーバープロセスにはTTYが無いため、git/sshが認証情報を対話的に要求すると
+ * リクエストがハングしてしまう。プロンプトを禁止して即座に失敗させる。
+ */
+const NON_INTERACTIVE_GIT_ENV = {
+    GIT_TERMINAL_PROMPT: '0',
+    GIT_SSH_COMMAND: 'ssh -o BatchMode=yes',
+};
+function nonInteractiveGit(baseDir) {
+    const git = baseDir ? simpleGit(baseDir) : simpleGit();
+    return git.env({ ...process.env, ...NON_INTERACTIVE_GIT_ENV });
+}
 // bare repositoryのパスを取得
 function getBareRepoPath(owner, repo) {
     return path.join(WORKSPACES_ROOT, owner, repo, '.bare');
@@ -51,10 +63,10 @@ export async function initBareRepository(owner, repo, cloneUrl) {
     // 親ディレクトリを作成
     await fs.mkdir(path.dirname(bareRepoPath), { recursive: true });
     // bare repositoryとしてクローン
-    const git = simpleGit();
+    const git = nonInteractiveGit();
     await git.clone(cloneUrl, bareRepoPath, ['--bare']);
     // 標準的なfetch refspecを設定
-    const bareGit = simpleGit(bareRepoPath);
+    const bareGit = nonInteractiveGit(bareRepoPath);
     await bareGit.addConfig('remote.origin.fetch', '+refs/heads/*:refs/remotes/origin/*');
     // Remote tracking branchesを作成
     await bareGit.fetch();

package/dist/server/pty-manager.js

@@ -2,6 +2,26 @@ import * as pty from 'node-pty';
 import * as os from 'os';
 const SCROLLBACK_MAX_BYTES = 256 * 1024; // 256KB
 const SESSION_GC_INTERVAL_MS = 30 * 60 * 1000; // 30分
+/**
+ * apps/cli が Fastify サーバー起動時に自分で注入する環境変数
+ * （TSUNAGI_SERVER_PORT, NODE_ENV, TSUNAGI_NEXT_PORT, TSUNAGI_OUTER_NODE_ENV）。
+ * サーバープロセス自身の制御用に後から足された値のため、内部ターミナルには継承しない。
+ * 外側 Terminal 由来の環境変数はこのリストと無関係に、これまで通り全て継承する
+ * （PORT はここでは扱わない。apps/cli は generic な PORT を上書きせず、専用キー
+ * TSUNAGI_SERVER_PORT で自身の待受ポートを伝えるため、ユーザーが Terminal で明示した
+ * PORT はそのまま内部ターミナルまで届く）。
+ *
+ * NODE_ENV は Fastify/Next 等が直接参照するため、PORT と違って専用キーへ逃がせず、
+ * サーバープロセス自身には強制的に 'production' が入る。apps/cli はサーバー起動前の
+ * 元の値（外側 Terminal 由来、未設定なら undefined）を TSUNAGI_OUTER_NODE_ENV に退避して
+ * 渡してくるので、ここで NODE_ENV を一旦除外した後、退避値があれば NODE_ENV として復元する。
+ */
+const TSUNAGI_INJECTED_ENV_KEYS = [
+    'TSUNAGI_SERVER_PORT',
+    'NODE_ENV',
+    'TSUNAGI_NEXT_PORT',
+    'TSUNAGI_OUTER_NODE_ENV',
+];
 class PtyManager {
     sessions = new Map();
     /** sessionId → 最後に input を送信した socket.id */
@@ -22,13 +42,25 @@ class PtyManager {
             throw new Error(`Session already exists: ${sessionId}`);
         }
         const shell = process.env.SHELL || (os.platform() === 'win32' ? 'cmd.exe' : 'bash');
+        // tsunagi 自身が注入した環境変数は内部ターミナルに継承しない（TSUNAGI_INJECTED_ENV_KEYS 参照）。
+        // /settings 等で明示的に同名キーを設定した場合は env（dbEnv 経由）が後勝ちで上書きするため、
+        // カスケードの優先順位・挙動は変わらない。
+        const baseEnv = { ...process.env };
+        for (const key of TSUNAGI_INJECTED_ENV_KEYS) {
+            delete baseEnv[key];
+        }
+        // 外側 Terminal に元々 NODE_ENV があった場合は復元する。なければ未設定のままにする
+        // （tsunagi が強制した 'production' を内部ターミナルに漏らさないため）。
+        if (process.env.TSUNAGI_OUTER_NODE_ENV !== undefined) {
+            baseEnv.NODE_ENV = process.env.TSUNAGI_OUTER_NODE_ENV;
+        }
         const ptyProcess = pty.spawn(shell, [], {
             name: 'xterm-256color',
             cols: 80,
             rows: 24,
             cwd,
             env: {
-                ...process.env,
+                ...baseEnv,
                 ...env,
                 TERM: 'xterm-256color',
                 COLORTERM: 'truecolor',

package/dist/server/routes/repos.js

@@ -7,17 +7,39 @@ import * as worktreeManager from '../lib/worktree-manager.js';
 import { getEnv } from '../lib/repositories/environment.js';
 import { createRepo } from '../lib/repositories/repository.js';
 const WORKSPACES_ROOT = path.join(os.homedir(), '.tsunagi', 'workspaces');
-// Git URLからowner/repoを抽出
+/**
+ * Git URLからowner/repoを抽出する。
+ * 前後の空白・末尾のスラッシュ・www. など、コピペで紛れ込みがちな差異は吸収する。
+ * 対応形式:
+ *   https://github.com/owner/repo(.git)
+ *   git@github.com:owner/repo(.git)
+ *   ssh://git@github.com/owner/repo(.git)
+ */
 function parseGitUrl(gitUrl) {
-    const httpsMatch = gitUrl.match(/https?:\/\/github\.com\/([^/]+)\/([^/]+?)(\.git)?$/);
-    const sshMatch = gitUrl.match(/git@github\.com:([^/]+)\/([^/]+?)(\.git)?$/);
-    const match = httpsMatch || sshMatch;
-    if (!match)
-        return null;
-    return {
-        owner: match[1],
-        repo: match[2],
-    };
+    const normalized = gitUrl.trim().replace(/\/+$/, '');
+    const patterns = [
+        /^https?:\/\/(?:www\.)?github\.com\/([^/]+)\/([^/]+?)(?:\.git)?$/,
+        /^git@github\.com:([^/]+)\/([^/]+?)(?:\.git)?$/,
+        /^ssh:\/\/git@github\.com\/([^/]+)\/([^/]+?)(?:\.git)?$/,
+    ];
+    for (const pattern of patterns) {
+        const match = normalized.match(pattern);
+        if (match)
+            return { owner: match[1], repo: match[2], url: normalized };
+    }
+    return null;
+}
+/** 認証が無くて失敗したときに出る git のメッセージ */
+function isGitAuthError(error) {
+    const message = (error instanceof Error ? error.message : String(error)).toLowerCase();
+    return (message.includes('could not read username') ||
+        message.includes('could not read password') ||
+        message.includes('authentication failed') ||
+        message.includes('terminal prompts disabled') ||
+        message.includes('permission denied') ||
+        // private repository に未認証でアクセスすると GitHub は 404 を返す
+        message.includes('repository not found') ||
+        message.includes('not found'));
 }
 export async function reposRoutes(fastify) {
     // GET /repos
@@ -51,6 +73,27 @@ export async function reposRoutes(fastify) {
             return reply.status(500).send({ error: 'Failed to create repository' });
         }
     });
+    // PUT /repos/reorder - トップページの列の並び順を一括更新
+    fastify.put('/repos/reorder', async (request, reply) => {
+        try {
+            const { repoIds } = request.body;
+            if (!Array.isArray(repoIds) || repoIds.length === 0) {
+                return reply.status(400).send({ error: 'Missing required field: repoIds' });
+            }
+            const existing = await repoRepo.getRepos();
+            const existingIds = new Set(existing.map((r) => r.id));
+            const unknown = repoIds.filter((id) => !existingIds.has(id));
+            if (unknown.length > 0) {
+                return reply.status(404).send({ error: `Unknown repository ids: ${unknown.join(', ')}` });
+            }
+            const repos = await repoRepo.reorderRepos(repoIds);
+            return reply.status(200).send({ data: repos });
+        }
+        catch (error) {
+            fastify.log.error(error, 'PUT /repos/reorder error');
+            return reply.status(500).send({ error: 'Failed to reorder repositories' });
+        }
+    });
     // GET /repos/:owner/:repo
     fastify.get('/repos/:owner/:repo', async (request, reply) => {
         try {
@@ -149,9 +192,11 @@ export async function reposRoutes(fastify) {
             }
             const parsed = parseGitUrl(gitUrl);
             if (!parsed) {
-                return reply.status(400).send({ error: 'Invalid Git URL format' });
+                return reply.status(400).send({
+                    error: 'Invalid Git URL format. Expected https://github.com/owner/repo or git@github.com:owner/repo',
+                });
             }
-            const { owner, repo } = parsed;
+            const { owner, repo, url: normalizedUrl } = parsed;
             const envVars = await getEnv('global');
             if (envVars.GITHUB_PAT) {
                 try {
@@ -162,20 +207,49 @@ export async function reposRoutes(fastify) {
                 }
             }
             const bareRepoPath = path.join(os.homedir(), '.tsunagi', 'workspaces', owner, repo, '.bare');
-            try {
-                await worktreeManager.initBareRepository(owner, repo, gitUrl);
-            }
-            catch (error) {
+            const cleanupBareRepo = async () => {
                 try {
                     await fs.rm(bareRepoPath, { recursive: true, force: true });
                 }
                 catch (cleanupError) {
                     fastify.log.warn(cleanupError, 'Failed to cleanup bare repository after clone failure');
                 }
-                throw error;
+            };
+            const sshUrl = `git@github.com:${owner}/${repo}.git`;
+            const isHttpsUrl = normalizedUrl.startsWith('http');
+            let clonedUrl = normalizedUrl;
+            try {
+                await worktreeManager.initBareRepository(owner, repo, normalizedUrl);
             }
-            const newRepo = await createRepo({ owner, repo, cloneUrl: gitUrl });
-            return reply.status(200).send({ data: { repository: { ...newRepo, bareRepoPath } } });
+            catch (error) {
+                await cleanupBareRepo();
+                // private repository を HTTPS で clone するには認証が必要。
+                // tsunagi は SSH agent 前提で動くため、認証エラーなら SSH で自動的に試し直す。
+                if (!isHttpsUrl || !isGitAuthError(error))
+                    throw error;
+                fastify.log.info({ owner, repo }, 'HTTPS clone failed with an auth error, retrying over SSH');
+                try {
+                    await worktreeManager.initBareRepository(owner, repo, sshUrl);
+                    clonedUrl = sshUrl;
+                }
+                catch (sshError) {
+                    await cleanupBareRepo();
+                    const httpsMessage = error instanceof Error ? error.message : String(error);
+                    const sshMessage = sshError instanceof Error ? sshError.message : String(sshError);
+                    throw new Error(`Failed to clone ${owner}/${repo}. ` +
+                        `HTTPS では認証情報が無いため失敗し、SSH でも失敗しました。` +
+                        `SSH鍵(ssh-agent)を設定するか、リポジトリへのアクセス権を確認してください。\n` +
+                        `HTTPS: ${httpsMessage.trim()}\nSSH: ${sshMessage.trim()}`);
+                }
+            }
+            const newRepo = await createRepo({ owner, repo, cloneUrl: clonedUrl });
+            return reply.status(200).send({
+                data: {
+                    repository: { ...newRepo, bareRepoPath },
+                    // HTTPSで認証できずSSHで clone し直した場合、UI側で利用者に知らせる
+                    fallbackToSsh: clonedUrl !== normalizedUrl,
+                },
+            });
         }
         catch (error) {
             fastify.log.error(error, 'Failed to clone repository');

package/dist/server/routes/terminal.js

@@ -7,9 +7,9 @@ import { getEnv } from '../lib/repositories/environment.js';
 import { ensureClaudeOnboardingCompleted } from '../lib/claude-config-guard.js';
 // サーバーはプロジェクトルートから起動されるため process.cwd() でルートを取得
 const TSUNAGI_EDITOR_PATH = path.resolve(process.cwd(), 'scripts/monaco-editor.sh');
-// Fastify(API) の公開ポート。index.ts と同じ既定値。monaco-editor.sh など PTY 内の
-// プロセスが API を叩く際の同一ホスト向けベース URL に使う。
-const SERVER_PORT = Number(process.env.PORT) || 2791;
+// Fastify(API) の公開ポート。index.ts と同じ既定値・同じ環境変数(TSUNAGI_SERVER_PORT)を見る。
+// monaco-editor.sh など PTY 内のプロセスが API を叩く際の同一ホスト向けベース URL に使う。
+const SERVER_PORT = Number(process.env.TSUNAGI_SERVER_PORT) || 2791;
 /**
  * 再接続時の画面復元用 scrollback から「端末への問い合わせ（クエリ）」シーケンスを除去する。
  *