linkgravity 1.5.11 → 1.5.13

package/bin/cli.js

@@ -45,8 +45,14 @@ function runPm2(args, silent = true) {
     const result = spawnSync(process.execPath, [PM2_BIN, ...args], {
         stdio: stdioOpt,
         cwd: path.join(__dirname, '..'),
-        // pm2 gives Python a pipe not a TTY, so it block-buffers stdout and can sit on log lines indefinitely - force line buffering.
-        env: { ...process.env, PYTHONUNBUFFERED: '1' },
+        env: {
+            ...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',
+            // Version managers (fnm, nvm) put node on PATH from a shell hook the daemon never runs,
+            // so the bot's own `node` lookup for voice-service would fail without this.
+            PATH: `${path.dirname(process.execPath)}${path.delimiter}${process.env.PATH || ''}`,
+        },
     });
 
     if (result.error) {

package/package.json

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

package/src/api/ui_routes.py

@@ -6,18 +6,13 @@ import uuid
 
 from aiohttp import web
 
+from api.server import is_tool_allowed
 from config import APPROVAL_TIMEOUT_SEC, MAX_EMBED_LEN, logger, session_manager
 from messengers.base import ScopeOption
 from messengers.registry import get_adapter_for_platform, get_adapter_for_thread
 from utils.utils import split_message
 
 
-def is_tool_allowed(tool_name, tool_input):
-    from api.server import is_tool_allowed as is_tool_allowed_orig
-
-    return is_tool_allowed_orig(tool_name, tool_input)
-
-
 async def send_ordered(target_thread_id, send_coro_factory):
     """Routes through the same per-conversation stream queue as the answer
     text, so tool-call messages can't arrive out of order. Falls back to

package/src/approval/tool_formatter.py

@@ -57,9 +57,6 @@ def format_tool_display(tool_name: str, tool_input: dict) -> tuple[str, str, dic
 
 
 def format_bash_display(sub_cmd: str) -> tuple[str, str, dict]:
-    """
-    Formats a single bash sub-command for display.
-    """
     is_long = "\n" in sub_cmd or len(sub_cmd) > 50
     display_cmd = sub_cmd.split("\n")[0][:50] + "..." if is_long else sub_cmd
     tool_msg_text = f"● Bash({display_cmd})"

package/src/cogs/voice/enrollment.py

@@ -91,8 +91,6 @@ class SampleConfirmView(discord.ui.View):
 
 
 class EnrollmentManager:
-    """Owns /sound's recording flow - see _commit_enrollment."""
-
     def __init__(self, bot, voice_state: dict, play_audio, bot_settings: dict, save_bot_settings, logger):
         self.bot = bot
         self._voice_state = voice_state  # shared with VoiceCog
@@ -109,11 +107,7 @@ class EnrollmentManager:
     def stop(self):
         self.cleanup_stale_enrollments.cancel()
 
-    def is_enrolling(self, user_id: str) -> bool:
-        return user_id in self._enrollment
-
     async def handle_voice_service_down(self):
-        """Called when Node dies."""
         stale_user_ids = list(self._enrollment.keys())
         for uid in stale_user_ids:
             session = self._enrollment.pop(uid, None)
@@ -181,7 +175,6 @@ class EnrollmentManager:
             self.logger.warning(f"Failed to send enrollment status message: {e}")
 
     async def start_wake_word_recording(self, interaction: discord.Interaction, word: str) -> bool:
-        """Called by /sound's wake_word param."""
         guild_id = interaction.guild_id
         user_id = str(interaction.user.id)
 
@@ -241,7 +234,6 @@ class EnrollmentManager:
         return True
 
     async def handle_enroll_sample(self, user_id: str, audio_bytes: bytes):
-        """Called via /enroll_sample for each captured sample."""
         session = self._enrollment.get(user_id)
         if not session:
             return  # stray sample - recording already finished/cancelled/expired

package/src/cogs/voice/stt_session.py

@@ -19,13 +19,11 @@ class SttSessionTracker:
         self._partial_msg = {}
 
     def is_active(self, guild_id: str) -> bool:
-        """Whether the "stay awake" window is still open for this guild."""
         active_duration = self.bot_settings.get("active_timer", 60)
         last_active = self._last_active_time.get(str(guild_id), 0)
         return (time.time() - last_active) < active_duration
 
     def mark_tts_finished(self, guild_id: str):
-        """Called via /tts_finished once the spoken reply finishes playing."""
         self.extend_active_window(str(guild_id))
 
     def extend_active_window(self, guild_id: str):

package/src/core/platform_health.py

@@ -22,7 +22,3 @@ def set_status(platform: str, status: str, detail: str = "") -> None:
         "at": datetime.now().isoformat(),
     }
     atomic_write_json(path, data)
-
-
-def get_all() -> dict:
-    return safe_load_json(_path(), {})

package/src/core/session_manager.py

@@ -7,8 +7,6 @@ from core.atomic_io import atomic_write_json, safe_load_json
 
 
 class SessionManager:
-    """Manages conversation state, async streaming queues, and user approval states."""
-
     def __init__(self, data_dir: Path):
         self.session_file = data_dir / "sessions.json"
         self.persistent_file = data_dir / "persistent_tools.json"
@@ -120,9 +118,6 @@ class SessionManager:
         if conv_id:
             self.active_approval_by_conv[conv_id] = approval_key
 
-    def get_pending_approval(self, approval_key: str) -> asyncio.Future | None:
-        return self.pending_approvals.get(approval_key)
-
     def get_pending_approval_by_conv(self, conv_id: str) -> asyncio.Future | None:
         """Looks up whichever approval is CURRENTLY active for a given
         conversation - for callers that only have the stable

package/src/main_discord.py

@@ -1,6 +1,3 @@
-"""Discord bot setup. Runs in the same process as Telegram (see main.py),
-which starts both concurrently when both platforms are enabled."""
-
 import asyncio
 import atexit
 import os

package/src/main_slack.py

@@ -1,6 +1,3 @@
-"""Slack bot setup. Runs in the same process as Discord/Telegram (see
-main.py), which starts all enabled platforms concurrently."""
-
 import asyncio
 import re
 import uuid

package/src/main_telegram.py

@@ -1,6 +1,3 @@
-"""Telegram bot setup. Runs in the same process as Discord (see main.py),
-which starts both concurrently when both platforms are enabled."""
-
 import asyncio
 import uuid
 from datetime import datetime

package/src/messengers/base.py

@@ -1,6 +1,4 @@
-"""Core messenger interface. Every backend (Discord now, Slack/Telegram
-later) implements MessengerAdapter; business logic never touches
-platform SDK types directly.
+"""Core messenger interface. Business logic never touches platform SDK types directly.
 
 Futures for approval/question prompts are owned by business logic, not
 the adapter - the same future can also be resolved by a typed reply,
@@ -17,7 +15,7 @@ from typing import Any
 
 @dataclass
 class IncomingAttachment:
-    """A file on an inbound message; reader defers fetching bytes until needed."""
+    """reader defers fetching bytes until the attachment is actually read."""
 
     filename: str
     content_type: str | None
@@ -144,10 +142,6 @@ class MessengerAdapter(ABC):
     ) -> PromptHandle:
         raise NotImplementedError
 
-    @property
-    def supports_voice(self) -> bool:
-        return isinstance(self, VoiceCapable)
-
 
 class VoiceCapable(ABC):
     @abstractmethod

package/src/messengers/registry.py

@@ -1,7 +1,6 @@
-"""Per-platform messenger adapter registry, populated once at startup by
-main.py. Both bots now share one process, so callers must say which
-platform they mean - either directly (voice/Discord-only code) or by
-looking up which platform a given thread_id/conversation belongs to."""
+"""Per-platform messenger adapter registry, populated once at startup by main.py. All platforms
+share one process, so callers must say which platform they mean - either directly (voice/
+Discord-only code) or by looking up which platform a thread_id belongs to."""
 
 from messengers.base import MessengerAdapter
 
@@ -24,12 +23,3 @@ def get_adapter_for_thread(thread_id: str) -> MessengerAdapter:
     session = session_manager.get_session(thread_id) or {}
     platform = session.get("platform", "discord")  # pre-multi-platform sessions have no tag - assume discord
     return get_adapter_for_platform(platform)
-
-
-def get_adapter_for_conv_id(conv_id: str) -> MessengerAdapter | None:
-    from config import session_manager
-
-    for _thread_id, sess in session_manager.get_all_sessions().items():
-        if sess.get("conversation_id") == conv_id:
-            return get_adapter_for_platform(sess.get("platform", "discord"))
-    return None

package/src/messengers/slack_adapter.py

@@ -46,8 +46,8 @@ def decode_conversation_id(conversation_id: str) -> tuple[str, str] | None:
 
 def latest_channel_session(channel: str) -> tuple[str, dict] | None:
     """Most recently created Slack session in a channel - used as a fallback for un-threaded
-    messages (users rarely bother clicking "Reply in thread") and for /model, /credit, which
-    can't target a specific thread since Slack slash commands can't be invoked inside one."""
+    messages (users rarely bother clicking "Reply in thread") and for /model, which can't target
+    a specific thread since Slack slash commands can't be invoked inside one."""
     candidates = [
         (cid, s)
         for cid, s in session_manager.get_all_sessions().items()
@@ -59,8 +59,6 @@ def latest_channel_session(channel: str) -> tuple[str, dict] | None:
 
 
 class SlackConversationRef:
-    """conversation_ref for Slack - a channel + the thread_ts all replies go under."""
-
     __slots__ = ("channel", "thread_ts")
 
     def __init__(self, channel: str, thread_ts: str):
@@ -77,8 +75,6 @@ class SlackConversationRef:
 
 
 class SlackMessageRef:
-    """message_ref for edit_message - a specific message within a channel."""
-
     __slots__ = ("channel", "ts")
 
     def __init__(self, channel: str, ts: str):

package/voice-service/logger.js

@@ -1,6 +1,8 @@
 const originalLog = console.log;
 const originalError = console.error;
 
+const debugEnabled = (process.env.LOG_LEVEL || 'INFO').toUpperCase() === 'DEBUG';
+
 function getTimestamp() {
     const now = new Date();
     const offset = now.getTimezoneOffset() * 60000;
@@ -14,3 +16,8 @@ console.log = function (...args) {
 console.error = function (...args) {
     originalError(`${getTimestamp()} ERROR Voice:`, ...args);
 };
+console.debug = function (...args) {
+    if (debugEnabled) {
+        originalLog(`${getTimestamp()} DEBUG Voice:`, ...args);
+    }
+};

package/voice-service/receiver.js

@@ -148,7 +148,7 @@ function setupReceiver(connection, guildId, client) {
                 const dynamicThreshold = isBotPlaying ? baseThreshold * 3 : baseThreshold;
                 if (rms > dynamicThreshold) {
                     if (interruptTTS(guildId)) {
-                        console.log(
+                        console.debug(
                             `[VAD] Loud voice detected (${Math.round(rms)}), interrupting TTS (Threshold: ${dynamicThreshold})`,
                         );
                         hasInterrupted = true;
@@ -236,15 +236,18 @@ function setupReceiver(connection, guildId, client) {
                 const threshold = wakeThresholdFor(userId);
                 wakeConfirmed = bestWakeScoreName !== null;
                 matchedWakeWord = wakeConfirmed ? bestWakeScoreName : null;
-                console.log(
-                    wakeConfirmed
-                        ? `[Wake] ${userId}: CONFIRMED (score ${bestWakeScore.toFixed(3)} for ` +
-                              `"${bestWakeScoreName}", threshold ${threshold})`
-                        : `[Wake] ${userId}: no match (score ${bestWakeScore.toFixed(3)}, ` +
-                              `threshold ${threshold}; diagnostic-only closeness ` +
-                              `${bestDiagScore.toFixed(3)} for "${bestDiagScoreName ?? 'n/a'}" - ` +
-                              `different scoring config, not directly comparable to the threshold)`,
-                );
+                if (wakeConfirmed) {
+                    console.log(
+                        `[Wake] ${userId}: CONFIRMED (score ${bestWakeScore.toFixed(3)} for ` +
+                            `"${bestWakeScoreName}", threshold ${threshold})`,
+                    );
+                } else {
+                    console.debug(
+                        `[Wake] ${userId}: no match (threshold ${threshold}; diagnostic-only ` +
+                            `closeness ${bestDiagScore.toFixed(3)} for "${bestDiagScoreName ?? 'n/a'}" - ` +
+                            `different scoring config, not directly comparable to the threshold)`,
+                    );
+                }
             }
 
             const pcmBuffer = Buffer.concat(chunks);

package/voice-service/tts.js

@@ -14,7 +14,7 @@ function interruptTTS(guildId) {
 
     if (player && player.state.status !== AudioPlayerStatus.Idle) {
         player.stop();
-        console.log(`[VAD] Interrupted TTS in guild ${guildId}`);
+        console.debug(`[VAD] Interrupted TTS in guild ${guildId}`);
         interrupted = true;
     }