linkgravity 1.7.2 → 1.7.4

package/bin/cli.js

@@ -14,7 +14,7 @@ const {
     LGY_SCRIPT_PATH,
 } = require('./platforms');
 
-const { python: pythonExe, isWin } = require('../npm-scripts/venv-paths');
+const { daemonPython, isWin } = require('../npm-scripts/venv-paths');
 const { isEnvironmentReady } = require('../npm-scripts/ensure-env');
 
 const cmd = process.argv[2];
@@ -203,15 +203,34 @@ function verifyStartup() {
             cwd: PM2_CWD,
         });
 
+        const baseline = getPm2Proc()?.pm2_env?.restart_time ?? 0;
+
         let settled = false;
         const finish = (ok) => {
             if (settled) return;
             settled = true;
             clearTimeout(timer);
+            clearInterval(watchdog);
             cp.kill();
             resolve(ok);
         };
 
+        // A process that exits on startup never logs anything for the stream below to match, so
+        // pm2's own counters are the only signal that it is dying and being restarted.
+        const watchdog = setInterval(() => {
+            const proc = getPm2Proc();
+            if (!proc) return;
+            const { status, restart_time: restarts = 0 } = proc.pm2_env;
+            if (status === 'errored' || restarts > baseline) {
+                console.log(
+                    `\n\n${color.yellow}❌ The daemon keeps exiting - pm2 has restarted it ` +
+                        `${restarts - baseline} time(s) (status: ${status}).${color.reset}`,
+                );
+                console.log(`   Run ${color.cyan}lgy logs${color.reset} to see why.\n`);
+                finish(false);
+            }
+        }, 2000);
+
         let timer = setTimeout(() => {
             console.log(
                 `\n\n${color.yellow}⏳ Startup verification timed out. Run 'lgy logs' to check status manually.${color.reset}`,
@@ -302,6 +321,27 @@ function findAgyBin() {
     return null;
 }
 
+// Returns null when the daemon can be started, or the reason it can't.
+function launchBlocker() {
+    const settings = getSettings();
+    const anyConfigured = Object.keys(PLATFORMS).some(
+        (key) => platformState(key, settings).configured,
+    );
+    if (!anyConfigured) {
+        return (
+            'No messenger is configured yet - set up at least one of Discord, Telegram, or ' +
+            `Slack first: ${color.cyan}lgy setup${color.reset}`
+        );
+    }
+    if (!findAgyBin()) {
+        return (
+            "Couldn't find the agy CLI (checked $AGY_BIN_PATH, ~/.local/bin/agy, and PATH). " +
+            'Install/configure agy first, or set the AGY_BIN_PATH environment variable to its location.'
+        );
+    }
+    return null;
+}
+
 function getPm2Proc() {
     const jlist = spawnSync(process.execPath, [PM2_BIN, 'jlist'], { stdio: 'pipe' });
     if (jlist.status !== 0) return null;
@@ -372,24 +412,9 @@ if (cmd === 'version' || cmd === '-v' || cmd === '--version') {
         process.exit(1);
     }
 
-    const settings = getSettings();
-    const anyConfigured = Object.keys(PLATFORMS).some(
-        (key) => platformState(key, settings).configured,
-    );
-    if (!anyConfigured) {
-        console.log(
-            `\n${color.yellow}⚠${color.reset} No messenger is configured yet - ` +
-                `set up at least one of Discord, Telegram, or Slack first: ${color.cyan}lgy setup${color.reset}\n`,
-        );
-        process.exit(1);
-    }
-
-    if (!findAgyBin()) {
-        console.log(
-            `\n${color.yellow}⚠${color.reset} Couldn't find the agy CLI ` +
-                `(checked $AGY_BIN_PATH, ~/.local/bin/agy, and PATH). Install/configure agy first, ` +
-                `or set the AGY_BIN_PATH environment variable to its location.\n`,
-        );
+    const blocker = launchBlocker();
+    if (blocker) {
+        console.log(`\n${color.yellow}⚠${color.reset} ${blocker}\n`);
         process.exit(1);
     }
 
@@ -405,7 +430,7 @@ if (cmd === 'version' || cmd === '-v' || cmd === '--version') {
         'start',
         LGY_SCRIPT_PATH,
         '--interpreter',
-        pythonExe,
+        daemonPython,
         '--name',
         LGY_PM2_NAME,
         '--update-env',
@@ -634,12 +659,18 @@ if (cmd === 'version' || cmd === '-v' || cmd === '--version') {
     repairHookRegistration({ fresh: true });
 
     if (!procBeforeUpdate) {
+        const blocker = launchBlocker();
+        if (blocker) {
+            console.log(`\n${color.yellow}⚠${color.reset} ${blocker}\n`);
+            success('Update finished - the daemon was left stopped.\n');
+            process.exit(0);
+        }
         info("Daemon wasn't running - starting it fresh...");
         runPm2([
             'start',
             LGY_SCRIPT_PATH,
             '--interpreter',
-            pythonExe,
+            daemonPython,
             '--name',
             LGY_PM2_NAME,
             '--update-env',

package/bin/pm2.js

@@ -10,6 +10,9 @@ function pm2Env() {
         ...process.env,
         // pm2 gives Python a pipe not a TTY, so it block-buffers stdout and can sit on log lines indefinitely - force line buffering.
         PYTHONUNBUFFERED: '1',
+        // Without this Python inherits the console codepage (cp949 on Korean Windows) and loguru
+        // drops every line it can't encode - including the one verifyStartup waits for.
+        PYTHONIOENCODING: 'utf-8',
         // pm2 merges --update-env rather than replacing, so a LOG_LEVEL from an earlier run
         // survives unless a value is passed every time.
         LOG_LEVEL: process.env.LOG_LEVEL || 'INFO',

package/bin/setup.js

@@ -1,6 +1,6 @@
 const p = require('@clack/prompts');
 const { spawnSync } = require('child_process');
-const { python: pythonExe } = require('../npm-scripts/venv-paths');
+const { daemonPython } = require('../npm-scripts/venv-paths');
 const { PM2_BIN, PM2_CWD, pm2Env } = require('./pm2');
 const {
     getSettings,
@@ -224,7 +224,7 @@ function startOrRestartDaemon(pm2Name, scriptPath, label) {
             'start',
             scriptPath,
             '--interpreter',
-            pythonExe,
+            daemonPython,
             '--name',
             pm2Name,
         ]);

package/hooks/hook.js

@@ -12,7 +12,9 @@ const APPROVE_PORT = 18080;
 const TIMEOUT_MS = 3600 * 1000;
 
 function emit(payload) {
-    process.stdout.write(JSON.stringify(payload));
+    // Exits explicitly: the keep-alive socket and its hour-long timer stay open after the
+    // response arrives, and agy SIGABRTs the process instead of waiting for them to expire.
+    process.stdout.write(JSON.stringify(payload), () => process.exit(0));
 }
 
 function loadApproveToken() {

package/hooks/stop_hook.js

@@ -19,7 +19,7 @@ function log(line) {
 }
 
 function emit(payload) {
-    process.stdout.write(JSON.stringify(payload));
+    process.stdout.write(JSON.stringify(payload), () => process.exit(0));
 }
 
 async function readStdin() {

package/npm-scripts/venv-paths.js

@@ -40,6 +40,10 @@ module.exports = {
     isWin,
     venvBinDir,
     python: venvBin('python'),
+    // Only for pm2's --interpreter: pm2 spawns with detached:true, which on Windows forces a
+    // console window and ignores its own windowsHide option. pythonw is GUI-subsystem so no
+    // console is ever allocated, and pm2 pipes stdio anyway so no output is lost.
+    daemonPython: venvBin(isWin ? 'pythonw' : 'python'),
     pip: venvBin('pip'),
     preCommit: venvBin('pre-commit'),
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "linkgravity",
-  "version": "1.7.2",
+  "version": "1.7.4",
   "description": "Discord/Telegram bot bridge for the Antigravity (agy) CLI, with voice interaction support",
   "scripts": {
     "start": "node npm-scripts/run-dev.js",

package/src/cogs/general_cog.py

@@ -51,7 +51,7 @@ async def fetch_models_background():
     import time
 
     try:
-        from config import AGY_BIN
+        from config import AGY_BIN, CREATION_FLAGS
 
         logger.debug(f"Starting fetch_models_background using AGY_BIN: {AGY_BIN}")
         env = os.environ.copy()
@@ -63,6 +63,7 @@ async def fetch_models_background():
             stderr=asyncio.subprocess.PIPE,
             stdin=asyncio.subprocess.DEVNULL,
             env=env,
+            creationflags=CREATION_FLAGS,
         )
         try:
             stdout, stderr = await asyncio.wait_for(p.communicate(), timeout=30.0)

package/src/config.py

@@ -1,5 +1,7 @@
 import os
 import secrets
+import shutil
+import subprocess
 from pathlib import Path
 
 from core.atomic_io import atomic_write_json, safe_load_json
@@ -158,7 +160,22 @@ MODEL_CHOICES = {
     "pro": "Gemini 3.1 Pro",
 }
 
-AGY_BIN = os.getenv("AGY_BIN_PATH", str(Path.home() / ".local/bin/agy"))
+
+def _resolve_agy_bin() -> str:
+    fallback = str(Path.home() / ".local/bin/agy")
+    for candidate in (os.getenv("AGY_BIN_PATH"), fallback):
+        if candidate and Path(candidate).is_file():
+            return candidate
+    # Windows installs agy under AppData and adds it to PATH instead.
+    return shutil.which("agy") or fallback
+
+
+# Must stay in sync with findAgyBin() in bin/cli.js.
+AGY_BIN = _resolve_agy_bin()
+
+# pm2 spawns the daemon with detached:true, so on Windows it owns no console; without this
+# flag every child it launches gets a console window of its own.
+CREATION_FLAGS = subprocess.CREATE_NO_WINDOW if os.name == "nt" else 0
 
 session_manager = SessionManager(DATA_DIR)
 

package/src/core/agy_runner.py

@@ -163,7 +163,7 @@ async def run_agy(
                 if os.name == "nt":
                     import subprocess
 
-                    kwargs["creationflags"] = subprocess.CREATE_NEW_PROCESS_GROUP
+                    kwargs["creationflags"] = subprocess.CREATE_NEW_PROCESS_GROUP | subprocess.CREATE_NO_WINDOW
                 elif hasattr(os, "setsid"):
                     kwargs["preexec_fn"] = os.setsid
 

package/src/main_discord.py

@@ -11,7 +11,7 @@ from functools import partial
 import discord
 from discord.ext import commands
 
-from config import DISCORD_TOKEN, logger, session_manager
+from config import CREATION_FLAGS, DISCORD_TOKEN, logger, session_manager
 from handlers.message_router import handle_message
 from messengers.discord_adapter import DiscordAdapter
 from messengers.registry import register_adapter
@@ -128,7 +128,7 @@ async def status_updater_task():
 
 
 def _spawn_voice_process(voice_dir: str) -> subprocess.Popen:
-    return subprocess.Popen(["node", "index.js"], cwd=voice_dir)
+    return subprocess.Popen(["node", "index.js"], cwd=voice_dir, creationflags=CREATION_FLAGS)
 
 
 async def _supervise_voice_process(voice_dir: str):

package/src/services/response.py

@@ -1,7 +1,7 @@
 from pathlib import Path
 from typing import Any
 
-from config import MAX_EMBED_LEN, MODEL_CHOICES, session_manager
+from config import MAX_EMBED_LEN, bot_settings, session_manager
 from messengers.registry import get_adapter_for_platform
 from services.discord_helpers import split_message
 from utils.utils import get_current_model
@@ -18,8 +18,8 @@ async def send_agy_response(
     adapter = get_adapter_for_platform(session.get("platform", "discord"))
     session_manager.save_sessions()
 
-    session_model = session.get("model")
-    model_display = MODEL_CHOICES.get(session_model, session_model) if session_model else get_current_model()
+    # Same precedence as the /model autocomplete in general_cog.
+    model_display = session.get("model") or bot_settings.get("default_model") or get_current_model()
 
     parts = split_message(response_text, MAX_EMBED_LEN)
     for idx, part in enumerate(parts):