@in-the-loop-labs/pair-review 2.1.1 → 2.2.0

package/README.md

@@ -195,6 +195,9 @@ pair-review --local [path]
 | `-h`, `--help` | Show help message with full CLI documentation |
 | `-l`, `--local [path]` | Review local uncommitted changes. Optional path defaults to current directory |
 | `--model <name>` | Override the AI model for any provider. Model availability depends on provider configuration. |
+| `--register` | Register `pair-review://` URL scheme handler (macOS only) |
+| `--unregister` | Unregister `pair-review://` URL scheme handler (macOS only) |
+| `--command <cmd>` | Custom CLI command for `--register` (default: `npx @in-the-loop-labs/pair-review`) |
 | `-v`, `--version` | Show version number |
 
 ### Examples
@@ -204,6 +207,8 @@ pair-review 123                        # Review PR #123 in current repo
 pair-review https://github.com/owner/repo/pull/456
 pair-review --local                    # Review uncommitted local changes
 pair-review 123 --ai                   # Auto-run AI analysis
+pair-review --register                 # Register pair-review:// URL scheme (macOS)
+pair-review --register --command "node bin/pair-review.js"  # Custom command
 ```
 
 ## Configuration

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@in-the-loop-labs/pair-review",
-  "version": "2.1.1",
+  "version": "2.2.0",
   "description": "Your AI-powered code review partner - Close the feedback loop with AI coding agents",
   "main": "src/server.js",
   "bin": {

package/plugin/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "pair-review",
-  "version": "2.1.1",
+  "version": "2.2.0",
   "description": "pair-review app integration — Open PRs and local changes in the pair-review web UI, run server-side AI analysis, and address review feedback. Requires the pair-review MCP server.",
   "author": {
     "name": "in-the-loop-labs",

package/plugin-code-critic/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "code-critic",
-  "version": "2.1.1",
+  "version": "2.2.0",
   "description": "AI-powered code review analysis — Run three-level AI analysis and implement-review-fix loops directly in your coding agent. Works standalone, no server required.",
   "author": {
     "name": "in-the-loop-labs",

package/public/css/pr.css

@@ -4846,6 +4846,16 @@ tr.line-range-start .d2h-code-line-ctn {
   opacity: 0.7;
 }
 
+/* Assisted-by footer toggle (extends .remember-toggle) */
+.assisted-by-toggle {
+  margin-top: 8px;
+}
+
+.assisted-by-toggle.disabled {
+  opacity: 0.5;
+  pointer-events: none;
+}
+
 .review-type-options {
   display: flex;
   flex-direction: column;

package/public/js/components/ReviewModal.js

@@ -3,11 +3,24 @@
  * Review Submission Modal Component
  * Allows users to submit their review with comments to GitHub
  */
+
+const ASSISTED_BY_STORAGE_KEY = 'pair-review-assisted-by';
+const DEFAULT_ASSISTED_BY_URL = 'https://github.com/in-the-loop-labs/pair-review';
+
 class ReviewModal {
   constructor() {
     this.modal = null;
     this.isVisible = false;
     this.isSubmitting = false;
+    this.assistedByUrl = DEFAULT_ASSISTED_BY_URL;
+    fetch('/api/config')
+      .then(res => res.ok ? res.json() : null)
+      .then(data => {
+        if (data?.assisted_by_url) {
+          this.assistedByUrl = data.assisted_by_url;
+        }
+      })
+      .catch(() => {});  // Use default on failure
     this.createModal();
     this.setupEventListeners();
   }
@@ -69,6 +82,11 @@ class ReviewModal {
                 placeholder="Leave a comment about this pull request..."
                 rows="2"
               ></textarea>
+              <label class="remember-toggle assisted-by-toggle" id="assisted-by-toggle">
+                <input type="checkbox" id="assisted-by-checkbox" />
+                <span class="toggle-switch"></span>
+                <span class="toggle-label">Append pair-review footer</span>
+              </label>
             </div>
             
             <div class="review-type-section">
@@ -183,6 +201,9 @@ class ReviewModal {
       if (e.target.matches('input[name="review-event"]')) {
         window.reviewModal?.updateTextareaState();
       }
+      if (e.target.matches('#assisted-by-checkbox')) {
+        window.reviewModal?.handleAssistedByToggle();
+      }
     });
   }
 
@@ -194,6 +215,7 @@ class ReviewModal {
   updateTextareaState() {
     const textarea = this.modal?.querySelector('#review-body-modal');
     const selectedOption = this.modal?.querySelector('input[name="review-event"]:checked');
+    const toggle = this.modal?.querySelector('#assisted-by-toggle');
 
     if (!textarea || !selectedOption) return;
 
@@ -204,9 +226,15 @@ class ReviewModal {
     if (isDraft) {
       textarea.title = 'Review summary is not included with draft reviews';
       textarea.classList.add('disabled-textarea');
+      if (toggle) {
+        toggle.classList.add('disabled');
+      }
     } else {
       textarea.title = '';
       textarea.classList.remove('disabled-textarea');
+      if (toggle) {
+        toggle.classList.remove('disabled');
+      }
     }
   }
 
@@ -235,6 +263,9 @@ class ReviewModal {
     // Update textarea state (ensures it's enabled since COMMENT is selected by default)
     this.updateTextareaState();
 
+    // Restore assisted-by toggle from localStorage
+    this.restoreAssistedByToggle();
+
     // Clear any errors or warnings
     this.hideError();
     this.updateLargeReviewWarning(0);
@@ -431,6 +462,10 @@ class ReviewModal {
     if (this.isSubmitting) return;
     
     const reviewBody = this.modal.querySelector('#review-body-modal').value.trim();
+    const assistedByCheckbox = this.modal.querySelector('#assisted-by-checkbox');
+    const finalBody = assistedByCheckbox?.checked
+      ? reviewBody + this.getAssistedByFooter()
+      : reviewBody;
     const selectedOption = this.modal.querySelector('input[name="review-event"]:checked');
     const reviewEvent = selectedOption ? selectedOption.value : 'COMMENT';
     // Count BOTH line-level (.user-comment-row) and file-level (.file-comment-card.user-comment) comments
@@ -480,7 +515,7 @@ class ReviewModal {
         },
         body: JSON.stringify({
           event: reviewEvent,
-          body: reviewBody
+          body: finalBody
         })
       });
       
@@ -590,7 +625,6 @@ class ReviewModal {
     const textarea = this.modal?.querySelector('#review-body-modal');
     if (!textarea) return;
 
-    // Get AI summary from the AI panel
     const summary = window.aiPanel?.getSummary?.();
     if (!summary) {
       if (window.toast) {
@@ -613,6 +647,35 @@ class ReviewModal {
     }
   }
 
+  /**
+   * Get the "assisted by" footer string
+   */
+  getAssistedByFooter() {
+    const url = this.assistedByUrl || DEFAULT_ASSISTED_BY_URL;
+    return `\n\n---\n_Review assisted by [pair-review](${url})_`;
+  }
+
+  /**
+   * Restore the assisted-by toggle state from localStorage
+   */
+  restoreAssistedByToggle() {
+    const checkbox = this.modal?.querySelector('#assisted-by-checkbox');
+    if (!checkbox) return;
+
+    const stored = localStorage.getItem(ASSISTED_BY_STORAGE_KEY);
+    checkbox.checked = stored !== 'false';
+  }
+
+  /**
+   * Handle the assisted-by checkbox toggle
+   */
+  handleAssistedByToggle() {
+    const checkbox = this.modal?.querySelector('#assisted-by-checkbox');
+    if (!checkbox) return;
+
+    localStorage.setItem(ASSISTED_BY_STORAGE_KEY, String(checkbox.checked));
+  }
+
 }
 
 // Initialize when DOM is ready if not already initialized

package/src/config.js

@@ -24,7 +24,8 @@ const DEFAULT_CONFIG = {
   enable_chat: true,  // When true, enables the chat panel feature (requires Pi AI provider)
   chat: { enable_shortcuts: true },  // Chat panel settings (enable_shortcuts: show action shortcut buttons)
   providers: {},  // Custom provider configurations (overrides built-in defaults)
-  monorepos: {}  // Monorepo configurations: { "owner/repo": { path: "~/path/to/clone" } }
+  monorepos: {},  // Monorepo configurations: { "owner/repo": { path: "~/path/to/clone" } }
+  assisted_by_url: "https://github.com/in-the-loop-labs/pair-review"  // URL for "Review assisted by" footer link
 };
 
 /**

package/src/github/parser.js

@@ -30,7 +30,7 @@ class PRArgumentParser {
     // Check if input is a PR number
     const prNumber = parseInt(input);
     if (isNaN(prNumber) || prNumber <= 0) {
-      throw new Error('Invalid input format. Expected: PR number, GitHub URL (https://github.com/owner/repo/pull/number), or Graphite URL (https://app.graphite.com/github/pr/owner/repo/number)');
+      throw new Error('Invalid input format. Expected: PR number, GitHub URL (https://github.com/owner/repo/pull/number), Graphite URL (https://app.graphite.com/github/pr/owner/repo/number), or pair-review:// URL');
     }
 
     // Parse repository from current directory's git remote
@@ -59,6 +59,15 @@ class PRArgumentParser {
       normalizedUrl = 'https://' + normalizedUrl;
     }
 
+    // Check if input is a pair-review:// protocol URL
+    if (normalizedUrl.startsWith('pair-review://')) {
+      try {
+        return this.parseProtocolURL(normalizedUrl);
+      } catch (e) {
+        return null;
+      }
+    }
+
     // Check if input is a GitHub URL
     if (normalizedUrl.startsWith('https://github.com/')) {
       try {
@@ -114,6 +123,22 @@ class PRArgumentParser {
     return this._createPRInfo(owner, repo, numberStr, 'Graphite');
   }
 
+  /**
+   * Parse pair-review:// protocol URL to extract owner, repo, and PR number
+   * @param {string} url - Protocol URL (e.g., pair-review://pr/owner/repo/123)
+   * @returns {Object} Parsed information { owner, repo, number }
+   */
+  parseProtocolURL(url) {
+    const match = url.match(/^pair-review:\/\/pr\/([^\/]+)\/([^\/]+)\/(\d+)(?:\/.*)?$/);
+
+    if (!match) {
+      throw new Error('Invalid pair-review:// URL format. Expected: pair-review://pr/owner/repo/number');
+    }
+
+    const [, owner, repo, numberStr] = match;
+    return this._createPRInfo(owner, repo, numberStr, 'pair-review://');
+  }
+
   /**
    * Create and validate PR info object from parsed components
    * @param {string} owner - Repository owner
@@ -129,7 +154,9 @@ class PRArgumentParser {
     if (isNaN(number) || number <= 0) {
       const exampleUrl = source === 'GitHub'
         ? 'https://github.com/owner/repo/pull/number'
-        : 'https://app.graphite.com/github/pr/owner/repo/number';
+        : source === 'pair-review://'
+          ? 'pair-review://pr/owner/repo/number'
+          : 'https://app.graphite.com/github/pr/owner/repo/number';
       throw new Error(`Invalid ${source} URL format. Expected: ${exampleUrl}`);
     }
 

package/src/main.js

@@ -16,6 +16,7 @@ const simpleGit = require('simple-git');
 const { getGeneratedFilePatterns } = require('./git/gitattributes');
 const { getEmoji: getCategoryEmoji } = require('./utils/category-emoji');
 const open = (...args) => import('open').then(({default: open}) => open(...args));
+const { registerProtocolHandler, unregisterProtocolHandler } = require('./protocol-handler');
 
 let db = null;
 
@@ -117,6 +118,9 @@ OPTIONS:
                             (automatic in GitHub Actions)
     --yolo                  Allow AI providers full system access (skip read-only
                             restrictions). Analogous to --dangerously-skip-permissions
+    --register [--command <cmd>]  Register pair-review:// URL scheme handler (macOS)
+                                Default command: npx @in-the-loop-labs/pair-review
+    --unregister                Unregister pair-review:// URL scheme handler (macOS)
     -v, --version           Show version number and exit
 
 EXAMPLES:
@@ -125,6 +129,8 @@ EXAMPLES:
     pair-review --local                # Review uncommitted local changes
     pair-review 123 --ai               # Auto-run AI analysis
     pair-review --ai-review            # CI mode: auto-detect PR, submit review
+    pair-review --register                     # Register URL scheme handler
+    pair-review --register --command "node bin/pair-review.js"  # Custom command
 
 ENVIRONMENT VARIABLES:
     GITHUB_TOKEN            GitHub Personal Access Token (takes precedence over config file)
@@ -188,6 +194,9 @@ const KNOWN_FLAGS = new Set([
   '-l', '--local',
   '--mcp',
   '--model',
+  '--register',
+  '--unregister',
+  '--command',
   '--use-checkout',
   '--yolo',
   '-v', '--version'
@@ -243,9 +252,14 @@ function parseArgs(args) {
         i++; // Skip next argument since we consumed it
       }
       // localPath will be resolved to cwd if not provided
-    } else if (arg === '--configure' || arg === '-h' || arg === '--help' || arg === '--mcp' || arg === '-v' || arg === '--version') {
+    } else if (arg === '--configure' || arg === '-h' || arg === '--help' || arg === '--mcp' || arg === '-v' || arg === '--version' || arg === '--register' || arg === '--unregister') {
       // Skip flags that are handled earlier in main()
       continue;
+    } else if (arg === '--command') {
+      // --command flag consumed by --register handler, skip it and its value
+      if (i + 1 < args.length && !args[i + 1].startsWith('-')) {
+        i++; // Skip the next argument (the command value)
+      }
     } else if (arg.startsWith('-')) {
       // Unknown flag - collect for error reporting
       unknownFlags.push(arg);
@@ -346,6 +360,26 @@ AI PROVIDERS:
       process.exit(0);
     }
 
+    // Handle protocol handler registration
+    if (args.includes('--register')) {
+      const cmdIdx = args.indexOf('--command');
+      let command;
+      if (cmdIdx !== -1) {
+        if (cmdIdx + 1 < args.length && !args[cmdIdx + 1].startsWith('-')) {
+          command = args[cmdIdx + 1];
+        } else {
+          throw new Error('--command flag requires a command string');
+        }
+      }
+      await registerProtocolHandler({ command });
+      process.exit(0);
+    }
+
+    if (args.includes('--unregister')) {
+      await unregisterProtocolHandler();
+      process.exit(0);
+    }
+
     // Load configuration
     const { config, isFirstRun } = await loadConfig();
 

package/src/protocol-handler.js

@@ -0,0 +1,122 @@
+// SPDX-License-Identifier: GPL-3.0-or-later
+const fs = require('fs');
+const path = require('path');
+const { execSync } = require('child_process');
+const { getConfigDir } = require('./config');
+const logger = require('./utils/logger');
+
+const LSREGISTER = '/System/Library/Frameworks/CoreServices.framework/Frameworks/LaunchServices.framework/Support/lsregister';
+
+// Default dependencies (overridable for testing)
+const defaults = {
+  fs,
+  execSync,
+  getConfigDir,
+  logger,
+};
+
+/**
+ * Register a macOS custom URL scheme handler for pair-review://
+ * Creates an AppleScript .app that forwards URLs to the pair-review CLI.
+ *
+ * @param {object} [options]
+ * @param {string} [options.command] - CLI command to invoke (default: npx @in-the-loop-labs/pair-review)
+ * @param {object} [options._deps] - Internal: dependency overrides for testing
+ */
+function registerProtocolHandler({ command, _deps } = {}) {
+  const deps = { ...defaults, ..._deps };
+
+  if (process.platform !== 'darwin') {
+    deps.logger.warn('Protocol handler registration is only supported on macOS');
+    return;
+  }
+
+  const shell = process.env.SHELL || '/bin/zsh';
+  const resolvedCommand = command || 'npx @in-the-loop-labs/pair-review';
+
+  // Escape characters special inside AppleScript double-quoted strings
+  const escapeAS = (s) => s.replace(/\\/g, '\\\\').replace(/"/g, '\\"');
+  const safeShell = escapeAS(shell);
+  const safeCommand = escapeAS(resolvedCommand);
+
+  const appleScriptSource = [
+    'on run',
+    '\t-- No-op: launched directly without a URL',
+    'end run',
+    '',
+    'on open location theURL',
+    `\tdo shell script "${safeShell} -l -c \\"${safeCommand} " & quoted form of theURL & " > /dev/null 2>&1 &\\""`,
+    'end open location',
+  ].join('\n');
+
+  const appPath = path.join(deps.getConfigDir(), 'PairReview.app');
+
+  // Remove existing .app if present
+  deps.fs.rmSync(appPath, { recursive: true, force: true });
+
+  // Compile the AppleScript into an .app bundle
+  deps.execSync(`osacompile -o "${appPath}"`, { input: appleScriptSource });
+
+  // Mutate Info.plist to declare the URL scheme
+  const plistPath = path.join(appPath, 'Contents', 'Info.plist');
+  let plist = deps.fs.readFileSync(plistPath, 'utf-8');
+
+  const urlSchemeEntries = [
+    '\t<key>CFBundleIdentifier</key>',
+    '\t<string>com.pair-review.launcher</string>',
+    '\t<key>CFBundleURLTypes</key>',
+    '\t<array>',
+    '\t\t<dict>',
+    '\t\t\t<key>CFBundleURLName</key>',
+    '\t\t\t<string>pair-review URL</string>',
+    '\t\t\t<key>CFBundleURLSchemes</key>',
+    '\t\t\t<array>',
+    '\t\t\t\t<string>pair-review</string>',
+    '\t\t\t</array>',
+    '\t\t</dict>',
+    '\t</array>',
+  ].join('\n');
+
+  // Insert before the closing </dict> that precedes </plist>
+  plist = plist.replace('</dict>\n</plist>', `${urlSchemeEntries}\n</dict>\n</plist>`);
+  deps.fs.writeFileSync(plistPath, plist);
+
+  deps.execSync(`"${LSREGISTER}" -R -f "${appPath}"`);
+
+  // console.log intentional: CLI output, not server-side (see CLAUDE.md logging convention)
+  console.log('Registered pair-review:// URL scheme handler');
+  console.log(`Command: ${shell} -l -c '${resolvedCommand} <url>'`);
+}
+
+/**
+ * Unregister the macOS custom URL scheme handler for pair-review://
+ * Removes the .app bundle and deregisters from Launch Services.
+ *
+ * @param {object} [options]
+ * @param {object} [options._deps] - Internal: dependency overrides for testing
+ */
+function unregisterProtocolHandler({ _deps } = {}) {
+  const deps = { ...defaults, ..._deps };
+
+  if (process.platform !== 'darwin') {
+    deps.logger.warn('Protocol handler registration is only supported on macOS');
+    return;
+  }
+
+  const appPath = path.join(deps.getConfigDir(), 'PairReview.app');
+
+  // Attempt to deregister from Launch Services (may fail if not registered)
+  try {
+    deps.execSync(`"${LSREGISTER}" -u "${appPath}"`);
+  } catch {
+    // Ignore — handler may not be registered
+  }
+
+  // Remove the .app bundle
+  deps.fs.rmSync(appPath, { recursive: true, force: true });
+
+  // console.log intentional: CLI output, not server-side (see CLAUDE.md logging convention)
+  console.log('Unregistered pair-review:// URL scheme handler');
+}
+
+module.exports = { registerProtocolHandler, unregisterProtocolHandler };

package/src/routes/config.js

@@ -40,7 +40,8 @@ router.get('/api/config', (req, res) => {
     is_running_via_npx: isRunningViaNpx(),
     enable_chat: config.enable_chat !== false,
     chat_enable_shortcuts: config.chat?.enable_shortcuts !== false,
-    pi_available: getCachedAvailability('pi')?.available || false
+    pi_available: getCachedAvailability('pi')?.available || false,
+    assisted_by_url: config.assisted_by_url || 'https://github.com/in-the-loop-labs/pair-review'
   });
 });