juno-code 1.0.47 → 1.0.50

package/dist/templates/extensions/pi/juno-skill-preprocessor.ts

@@ -0,0 +1,239 @@
+/**
+ * Juno Skill Preprocessor — Pi Extension
+ *
+ * Adds variable substitution ($1, $2, $ARGUMENTS, $@, ${@:N}, ${@:N:L})
+ * and shell directive execution (!`command`) to Pi skill invocations.
+ *
+ * This extension intercepts /skill: commands via the "input" event,
+ * BEFORE Pi's internal _expandSkillCommand() runs. It reads the skill
+ * file, processes variables and shell directives, wraps the result in
+ * <skill> tags, and returns the fully expanded text.
+ *
+ * Shell directives only execute when the skill's frontmatter contains:
+ *   enable-shell-directives: true
+ *
+ * Variable substitution always runs when arguments are provided.
+ *
+ * Shipped via juno-code's SkillInstaller to .pi/extensions/.
+ */
+import type { ExtensionAPI, InputEvent } from "@mariozechner/pi-coding-agent";
+import { execSync } from "child_process";
+import { existsSync, readFileSync } from "fs";
+import { dirname, join } from "path";
+
+const SHELL_DIRECTIVE_REGEX = /!`([^`]+)`/g;
+const DEFAULT_SHELL_TIMEOUT = 5000;
+
+/** Directories to search for skill files, relative to project root. */
+const SKILL_DIRS = [".pi/skills", ".claude/skills"];
+
+/**
+ * Find a skill's SKILL.md file by name, searching known skill directories.
+ * Checks both {dir}/{name}/SKILL.md and {dir}/{name}.md patterns.
+ */
+function findSkillFile(skillName: string, cwd: string): string | null {
+	for (const dir of SKILL_DIRS) {
+		const candidates = [join(cwd, dir, skillName, "SKILL.md"), join(cwd, dir, `${skillName}.md`)];
+		for (const candidate of candidates) {
+			if (existsSync(candidate)) return candidate;
+		}
+	}
+	return null;
+}
+
+/**
+ * Parse YAML-like frontmatter from a skill file.
+ * Returns frontmatter key-value pairs and the body text after the frontmatter block.
+ */
+function parseFrontmatter(content: string): { frontmatter: Record<string, string | boolean>; body: string } {
+	const match = content.match(/^---\s*\n([\s\S]*?)\n---\s*\n?([\s\S]*)$/);
+	if (!match) return { frontmatter: {}, body: content };
+
+	const yaml = match[1];
+	const body = match[2] ?? '';
+	const frontmatter: Record<string, string | boolean> = {};
+
+	for (const rawLine of yaml!.split("\n")) {
+		const colonIndex = rawLine.indexOf(":");
+		if (colonIndex === -1) continue;
+		const key = rawLine.slice(0, colonIndex).trim();
+		const value = rawLine.slice(colonIndex + 1).trim();
+		if (value === "true") {
+			frontmatter[key] = true;
+		} else if (value === "false") {
+			frontmatter[key] = false;
+		} else {
+			frontmatter[key] = value;
+		}
+	}
+
+	return { frontmatter, body };
+}
+
+/**
+ * Parse command arguments respecting single and double quotes.
+ * Handles escape characters with backslash.
+ *
+ * Examples:
+ *   'hello world'      → ["hello", "world"]
+ *   '"hello world"'    → ["hello world"]
+ *   "'hello' \"world\"" → ["hello", "world"]
+ */
+function parseCommandArgs(input: string): string[] {
+	if (!input.trim()) return [];
+
+	const args: string[] = [];
+	let current = "";
+	let inSingle = false;
+	let inDouble = false;
+	let escape = false;
+
+	for (const char of input) {
+		if (escape) {
+			current += char;
+			escape = false;
+			continue;
+		}
+		if (char === "\\") {
+			escape = true;
+			continue;
+		}
+		if (char === '"' && !inSingle) {
+			inDouble = !inDouble;
+			continue;
+		}
+		if (char === "'" && !inDouble) {
+			inSingle = !inSingle;
+			continue;
+		}
+		if (char === " " && !inSingle && !inDouble) {
+			if (current) {
+				args.push(current);
+				current = "";
+			}
+			continue;
+		}
+		current += char;
+	}
+	if (current) args.push(current);
+	return args;
+}
+
+/**
+ * Substitute argument placeholders in content.
+ *
+ * Supports (1-indexed, aligned with bash and Pi's prompt-templates):
+ *   $1, $2, ...   — positional arguments
+ *   $@             — all arguments joined by space
+ *   $ARGUMENTS     — all arguments joined by space (alias)
+ *   ${@:N}         — arguments from Nth position onwards
+ *   ${@:N:L}       — L arguments starting from position N
+ */
+function substituteArgs(content: string, args: string[]): string {
+	let result = content;
+
+	// Replace $1, $2, etc. FIRST (before wildcards to prevent re-substitution)
+	result = result.replace(/\$(\d+)/g, (_, num) => {
+		const index = parseInt(num, 10) - 1;
+		return args[index] ?? "";
+	});
+
+	// Replace ${@:start} or ${@:start:length} (bash-style, 1-indexed)
+	result = result.replace(/\$\{@:(\d+)(?::(\d+))?\}/g, (_, startStr, lengthStr) => {
+		let start = parseInt(startStr, 10) - 1;
+		if (start < 0) start = 0;
+		if (lengthStr) {
+			const length = parseInt(lengthStr, 10);
+			return args.slice(start, start + length).join(" ");
+		}
+		return args.slice(start).join(" ");
+	});
+
+	const allArgs = args.join(" ");
+	result = result.replace(/\$ARGUMENTS/g, allArgs);
+	result = result.replace(/\$@/g, allArgs);
+
+	return result;
+}
+
+/**
+ * Process shell directives (!`command`) by executing them and inlining stdout.
+ * On error: replaces with [Error executing: command].
+ * Timeout: DEFAULT_SHELL_TIMEOUT ms (configurable).
+ */
+function processShellDirectives(content: string, cwd: string, timeout: number = DEFAULT_SHELL_TIMEOUT): string {
+	return content.replace(SHELL_DIRECTIVE_REGEX, (_, command: string) => {
+		try {
+			return execSync(command, {
+				cwd,
+				timeout,
+				encoding: "utf-8",
+				stdio: ["pipe", "pipe", "pipe"],
+			}).trim();
+		} catch {
+			return `[Error executing: ${command}]`;
+		}
+	});
+}
+
+/**
+ * Juno Skill Preprocessor Extension
+ *
+ * Intercepts /skill: commands via the "input" event (before Pi's internal
+ * _expandSkillCommand runs), applies variable substitution and shell
+ * directive processing, then returns the fully expanded skill block.
+ */
+export default function junoSkillPreprocessor(pi: ExtensionAPI) {
+	pi.on("input", (event: InputEvent) => {
+		const text = typeof event.text === "string" ? event.text : "";
+		if (!text.startsWith("/skill:")) return { action: "continue" };
+
+		const cwd = process.cwd();
+
+		// Parse skill name and arguments from the command
+		const spaceIndex = text.indexOf(" ");
+		const skillName = spaceIndex === -1 ? text.slice(7) : text.slice(7, spaceIndex);
+		const argsString = spaceIndex === -1 ? "" : text.slice(spaceIndex + 1).trim();
+		const args = parseCommandArgs(argsString);
+
+		// Find the skill file on disk
+		const skillPath = findSkillFile(skillName, cwd);
+		if (!skillPath) return { action: "continue" }; // Unknown skill — let Pi handle it
+
+		try {
+			const content = readFileSync(skillPath, "utf-8");
+			const { frontmatter, body } = parseFrontmatter(content);
+			let processedBody = body.trim();
+
+			// Substitute variable placeholders with provided arguments
+			if (args.length > 0) {
+				processedBody = substituteArgs(processedBody, args);
+			}
+
+			// Execute shell directives (only when explicitly opted in via frontmatter)
+			if (frontmatter["enable-shell-directives"] === true) {
+				processedBody = processShellDirectives(processedBody, cwd);
+			}
+
+			// Build the <skill> block (matches Pi's _expandSkillCommand format)
+			const baseDir = dirname(skillPath);
+			const skillBlock = [
+				`<skill name="${skillName}" location="${skillPath}">`,
+				`References are relative to ${baseDir}.`,
+				"",
+				processedBody,
+				"</skill>",
+			].join("\n");
+
+			// Return transformed text — Pi's _expandSkillCommand will see this
+			// doesn't start with /skill: and pass it through unchanged.
+			return { action: "transform", text: skillBlock };
+		} catch {
+			// On error, let Pi's native expansion handle it
+			return { action: "continue" };
+		}
+	});
+}
+
+// Export internals for testing
+export { findSkillFile, parseCommandArgs, parseFrontmatter, processShellDirectives, substituteArgs };

package/dist/templates/scripts/install_requirements.sh

@@ -13,7 +13,7 @@
 #    - If inside venv: installs into venv
 #    - If externally managed Python detected: uses pipx or creates temporary venv
 #    - If outside venv (non-managed): uses --system flag for system-wide installation
-# 6. Installs required packages: juno-kanban, roundtable-ai
+# 6. Installs required packages: juno-kanban
 # 7. Reports if requirements are already satisfied
 #
 # Usage: ./install_requirements.sh
@@ -54,13 +54,13 @@ NC='\033[0m' # No Color
 # Required packages
 # Note: requests and python-dotenv are required by github.py
 # slack_sdk is required by Slack integration scripts (slack_fetch.py, slack_respond.py)
-REQUIRED_PACKAGES=("juno-kanban" "roundtable-ai" "requests" "python-dotenv" "slack_sdk")
+REQUIRED_PACKAGES=("juno-kanban" "requests" "python-dotenv" "slack_sdk")
 
 # Version check cache configuration
 # This ensures we don't check PyPI on every run (performance optimization per Task RTafs5)
 VERSION_CHECK_CACHE_DIR="${HOME}/.juno_code"
 VERSION_CHECK_CACHE_FILE="${VERSION_CHECK_CACHE_DIR}/.version_check_cache"
-VERSION_CHECK_INTERVAL_HOURS=24  # Check for updates once per day
+VERSION_CHECK_INTERVAL_HOURS="${VERSION_CHECK_INTERVAL_HOURS:-24}"  # Check for updates once per day (override via env var)
 
 # Logging functions
 log_info() {
@@ -402,6 +402,38 @@ is_externally_managed_python() {
     return 1  # Not externally managed
 }
 
+# Function to upgrade pip to latest version inside the active venv
+# Why: venv ships with the pip version bundled in the Python distribution,
+# which can be months/years behind. Old pip may fail to resolve modern
+# dependency metadata or miss security fixes. Upgrading pip is fast (<2s)
+# and prevents hard-to-debug install failures downstream.
+upgrade_pip_in_venv() {
+    if ! is_in_virtualenv; then
+        return 0  # Only upgrade pip inside a venv
+    fi
+
+    log_info "Upgrading pip to latest version in venv..."
+
+    # Prefer uv for speed, fall back to pip itself
+    if command -v uv &>/dev/null; then
+        if uv pip install --upgrade pip --quiet 2>/dev/null; then
+            local pip_ver
+            pip_ver=$(python3 -m pip --version 2>/dev/null | awk '{print $2}' || echo "unknown")
+            log_success "pip upgraded to v$pip_ver (via uv)"
+            return 0
+        fi
+    fi
+
+    # Fall back to pip self-upgrade
+    if python3 -m pip install --upgrade pip --quiet 2>/dev/null; then
+        local pip_ver
+        pip_ver=$(python3 -m pip --version 2>/dev/null | awk '{print $2}' || echo "unknown")
+        log_success "pip upgraded to v$pip_ver"
+    else
+        log_warning "Could not upgrade pip (non-fatal, continuing with current version)"
+    fi
+}
+
 # Function to install packages using pipx
 install_with_pipx() {
     log_info "Installing packages using 'pipx' (recommended for Python applications)..."
@@ -492,6 +524,9 @@ install_with_uv() {
             log_error "Virtual environment activation script not found"
             return 1
         fi
+
+        # Upgrade pip to latest version in venv
+        upgrade_pip_in_venv
     fi
 
     local failed_packages=()
@@ -562,6 +597,9 @@ install_with_pip() {
         source "$venv_path/bin/activate"
         log_success "Activated virtual environment"
         python_cmd="python"  # Use the venv's python
+
+        # Upgrade pip to latest version in venv
+        upgrade_pip_in_venv
     fi
 
     local failed_packages=()

package/dist/templates/scripts/kanban.sh

@@ -170,12 +170,30 @@ ensure_python_environment() {
 # Get the directory where this script is located
 SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
 
-# Navigate to project root (parent of scripts directory)
-PROJECT_ROOT="$( cd "$SCRIPT_DIR/../.." && pwd )"
+# Auto-discover project root by walking up the directory tree looking for .juno_task/
+# This makes kanban.sh work from any installation depth (e.g. .juno_task/scripts/,
+# .claude/skills/ralph-loop/scripts/, etc.) without a hardcoded relative path.
+PROJECT_ROOT=""
+_dir="$SCRIPT_DIR"
+while [[ "$_dir" != "/" ]]; do
+    if [[ -d "$_dir/.juno_task" ]]; then
+        PROJECT_ROOT="$_dir"
+        break
+    fi
+    _dir="$( cd "$_dir/.." && pwd )"
+done
+if [[ -z "$PROJECT_ROOT" ]]; then
+    echo "ERROR: Could not find project root (no .juno_task/ directory found above $SCRIPT_DIR)" >&2
+    exit 1
+fi
 
 # Change to project root
 cd "$PROJECT_ROOT"
 
+# Export JUNO_TASK_ROOT so juno-kanban resolves .juno_task paths from project root,
+# not from wherever the calling agent happens to be. Respects existing override.
+export JUNO_TASK_ROOT="${JUNO_TASK_ROOT:-$PROJECT_ROOT}"
+
 # Arrays to store normalized arguments (declared at script level for proper handling)
 declare -a NORMALIZED_GLOBAL_FLAGS=()
 declare -a NORMALIZED_COMMAND_ARGS=()
@@ -192,7 +210,7 @@ normalize_arguments() {
     local found_command=false
 
     # Known subcommands
-    local commands="create search get show update archive mark list merge"
+    local commands="create search get show update archive mark list merge ready deps order"
 
     while [[ $# -gt 0 ]]; do
         case $1 in
@@ -207,7 +225,7 @@ normalize_arguments() {
                 fi
                 ;;
             # Global flags that don't take a value
-            -p|--pretty|--raw|-v|--verbose|-h|--help|--version)
+            -p|--pretty|--raw|-v|--verbose|--version)
                 NORMALIZED_GLOBAL_FLAGS+=("$1")
                 shift
                 ;;