@link-assistant/hive-mind 2.1.7 → 2.1.8

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @link-assistant/hive-mind
 
+## 2.1.8
+
+### Patch Changes
+
+- 11dab3c: Handle Telegram and CLI commands where a GitHub issue or pull request URL is immediately followed by a long option marker.
+
 ## 2.1.7
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@link-assistant/hive-mind",
-  "version": "2.1.7",
+  "version": "2.1.8",
   "description": "AI-powered issue solver and hive mind for collaborative problem solving",
   "main": "src/hive.mjs",
   "type": "module",

package/src/argument-normalization.lib.mjs

@@ -0,0 +1,23 @@
+const TYPOGRAPHIC_LONG_OPTION_DASHES = /[\u2013\u2014]/g;
+
+const GITHUB_ISSUE_OR_PR_WITH_JOINED_OPTION = new RegExp(['^(', '(?:https?://)?', '(?:www\\.)?', '(?:github\\.com/)?', '[A-Za-z0-9_.-]+/[A-Za-z0-9_.-]+/(?:issues|pull)/\\d+', ')', '(--[A-Za-z][A-Za-z0-9-]*(?:=.*)?)', '$'].join(''));
+
+export const normalizeTypographicOptionDashes = value => {
+  if (typeof value !== 'string') return value;
+  return value.replace(TYPOGRAPHIC_LONG_OPTION_DASHES, '--');
+};
+
+export const splitJoinedGitHubLongOptionArg = value => {
+  const normalized = normalizeTypographicOptionDashes(value);
+  if (typeof normalized !== 'string') return [normalized];
+
+  const match = normalized.match(GITHUB_ISSUE_OR_PR_WITH_JOINED_OPTION);
+  if (!match) return [normalized];
+
+  return [match[1], match[2]];
+};
+
+export const normalizeCliArgs = args => {
+  if (!Array.isArray(args)) return [];
+  return args.flatMap(splitJoinedGitHubLongOptionArg);
+};

package/src/cli-arguments.lib.mjs

@@ -1,8 +1,10 @@
 import { getenv, makeConfig, yargs as linoYargs } from 'lino-arguments';
 
+import { normalizeCliArgs } from './argument-normalization.lib.mjs';
 import { enhanceUnknownArgumentError } from './option-suggestions.lib.mjs';
 
 export { getenv };
+export { normalizeCliArgs, normalizeTypographicOptionDashes, splitJoinedGitHubLongOptionArg } from './argument-normalization.lib.mjs';
 
 export const hideBin = argv => argv.slice(2);
 
@@ -53,7 +55,7 @@ export function addCliCompatibilityAliases(parsed, { positionalAliases = [] } =
 }
 
 export function parseCliArgumentsWithLino({ argv = process.argv, commandName = 'cli', createYargsConfig, positionalAliases = [], lenv = { enabled: true }, env = { enabled: false }, getenv: getenvOptions = { enabled: true } } = {}) {
-  const fullArgv = ensureFullArgv(argv, commandName);
+  const fullArgv = normalizeCliArgs(ensureFullArgv(argv, commandName));
   let configuredParser = null;
   let parsed;
 

package/src/hive.mjs

@@ -18,9 +18,9 @@ if (earlyArgs.includes('--version')) {
 if (earlyArgs.includes('--help') || earlyArgs.includes('-h')) {
   try {
     // Load minimal modules needed for help
-    const { getLinoYargsFactory, hideBin } = await import('./cli-arguments.lib.mjs');
+    const { getLinoYargsFactory, hideBin, normalizeCliArgs } = await import('./cli-arguments.lib.mjs');
     const yargs = getLinoYargsFactory();
-    const rawArgs = hideBin(process.argv);
+    const rawArgs = normalizeCliArgs(hideBin(process.argv));
     // Reuse createYargsConfig from shared module to avoid duplication
     const { createYargsConfig } = await import('./hive.config.lib.mjs');
     const helpYargs = createYargsConfig(yargs(rawArgs)).version(false);
@@ -62,7 +62,7 @@ if (isRunningDirectly) {
       30000, // 30 second timeout
       'loading command-stream'
     );
-    const { parseCliArgumentsWithLino, hideBin } = await import('./cli-arguments.lib.mjs');
+    const { parseCliArgumentsWithLino, hideBin, normalizeCliArgs } = await import('./cli-arguments.lib.mjs');
     const path = (await withTimeout(use('path'), 30000, 'loading path')).default;
     const fs = (await withTimeout(use('fs'), 30000, 'loading fs')).promises;
     // Import shared library functions
@@ -225,7 +225,7 @@ if (isRunningDirectly) {
     }
 
     // Configure command line arguments - GitHub URL as positional argument
-    const rawArgs = hideBin(process.argv);
+    const rawArgs = normalizeCliArgs(hideBin(process.argv));
     // Use .parse() instead of .argv to ensure .strict() mode works correctly
     // When you use .argv, strict mode doesn't trigger properly
     // See: https://github.com/yargs/yargs/issues - .strict() only works with .parse()
@@ -248,7 +248,7 @@ if (isRunningDirectly) {
 
     try {
       argv = parseCliArgumentsWithLino({
-        argv: process.argv,
+        argv: ['node', 'hive', ...rawArgs],
         commandName: 'hive',
         createYargsConfig,
         positionalAliases: ['github-url'],

package/src/solve.config.lib.mjs

@@ -11,7 +11,7 @@ import { enhanceErrorMessage, detectMalformedFlags } from './option-suggestions.
 import { defaultModels, buildModelOptionDescription, resolveDefaultFallbackModel, resolveRuntimeDefaultModel } from './models/index.mjs';
 import { validateBranchName } from './solve.branch.lib.mjs';
 import { resolveEscalationConfig, isEscalateEnabled, DEFAULT_ESCALATE_RANGE } from './solve.escalate.lib.mjs';
-import { getLinoYargsFactory, hideBin, parseCliArgumentsWithLino } from './cli-arguments.lib.mjs';
+import { getLinoYargsFactory, hideBin, normalizeCliArgs, parseCliArgumentsWithLino } from './cli-arguments.lib.mjs';
 
 // Re-export for use by telegram-bot.mjs (avoids extra import lines there)
 export { detectMalformedFlags };
@@ -737,7 +737,7 @@ export const createYargsConfig = yargsInstance => {
 
 // Parse command line arguments - now needs yargs and hideBin passed in
 export const parseArguments = async (yargs = getLinoYargsFactory(), hideBinFn = hideBin) => {
-  const rawArgs = hideBinFn(process.argv);
+  const rawArgs = normalizeCliArgs(hideBinFn(process.argv));
 
   // Issue #1092: Detect malformed flag patterns BEFORE yargs parsing
   // This catches cases like "-- model" which yargs silently treats as positional arguments
@@ -776,7 +776,7 @@ export const parseArguments = async (yargs = getLinoYargsFactory(), hideBinFn =
     try {
       yargsInstance = createYargsConfig(yargs());
       argv = parseCliArgumentsWithLino({
-        argv: process.argv,
+        argv: ['node', 'solve', ...rawArgs],
         commandName: 'solve',
         createYargsConfig,
         positionalAliases: ['issue-url'],

package/src/telegram-solve-command.lib.mjs

@@ -8,6 +8,7 @@
  * @see https://github.com/link-assistant/hive-mind/issues/1618
  */
 
+import { normalizeCliArgs } from './argument-normalization.lib.mjs';
 import { enhanceUnknownArgumentError } from './option-suggestions.lib.mjs';
 
 export const TOOL_SOLVE_COMMAND_ALIASES = Object.freeze({
@@ -29,16 +30,13 @@ export function parseCommandArgs(text) {
     return [];
   }
 
-  // Replace em-dash with double-dash to fix Telegram auto-replacement.
-  const normalizedArgsText = argsText.replace(/—/g, '--');
-
   const args = [];
   let currentArg = '';
   let inQuotes = false;
   let quoteChar = null;
 
-  for (let i = 0; i < normalizedArgsText.length; i++) {
-    const char = normalizedArgsText[i];
+  for (let i = 0; i < argsText.length; i++) {
+    const char = argsText[i];
 
     if ((char === '"' || char === "'") && !inQuotes) {
       inQuotes = true;
@@ -60,7 +58,7 @@ export function parseCommandArgs(text) {
     args.push(currentArg);
   }
 
-  return args;
+  return normalizeCliArgs(args);
 }
 
 function toCamelCaseOptionName(name) {