@jonnyhoo/ccs 1.0.0

package/scripts/bump-version.sh

@@ -0,0 +1,153 @@
+#!/usr/bin/env bash
+# =============================================================================
+# DEPRECATED: This script is kept for emergency manual releases only.
+#
+# Releases are now automated via semantic-release:
+#   - Merge to main  → stable release (npm @latest)
+#   - Merge to dev   → dev release (npm @dev)
+#
+# Version is determined automatically from conventional commits:
+#   - feat: commit   → MINOR bump (5.0.2 → 5.1.0)
+#   - fix: commit    → PATCH bump (5.0.2 → 5.0.3)
+#   - feat!: commit  → MAJOR bump (5.0.2 → 6.0.0)
+#
+# See: docs/version-management.md
+# =============================================================================
+#
+# Legacy usage (emergency only): ./scripts/bump-version.sh [major|minor|patch]
+
+set -euo pipefail
+
+# Show deprecation warning
+echo "============================================================="
+echo "[!] DEPRECATED: This script is for emergency use only."
+echo ""
+echo "    Releases are now automated via semantic-release."
+echo "    Simply merge to 'main' or 'dev' branch."
+echo ""
+echo "    Version is determined from conventional commits:"
+echo "      feat: commit   -> MINOR (5.0.2 -> 5.1.0)"
+echo "      fix: commit    -> PATCH (5.0.2 -> 5.0.3)"
+echo "      feat!: commit  -> MAJOR (5.0.2 -> 6.0.0)"
+echo "============================================================="
+echo ""
+
+# Require explicit confirmation
+read -p "Continue with manual bump? (y/N) " -n 1 -r
+echo
+if [[ ! $REPLY =~ ^[Yy]$ ]]; then
+    echo "Cancelled. Use conventional commits + merge to main instead."
+    exit 0
+fi
+echo ""
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+CCS_DIR="$(dirname "$SCRIPT_DIR")"
+VERSION_FILE="$CCS_DIR/VERSION"
+
+# Check VERSION file exists
+if [[ ! -f "$VERSION_FILE" ]]; then
+    echo "[X] Error: VERSION file not found at $VERSION_FILE"
+    exit 1
+fi
+
+# Read current version
+CURRENT_VERSION=$(cat "$VERSION_FILE")
+echo "Current version: $CURRENT_VERSION"
+
+# Parse version
+if [[ ! "$CURRENT_VERSION" =~ ^([0-9]+)\.([0-9]+)\.([0-9]+)$ ]]; then
+    echo "[X] Error: Invalid version format in VERSION file"
+    echo "Expected: MAJOR.MINOR.PATCH (e.g., 1.2.3)"
+    exit 1
+fi
+
+MAJOR="${BASH_REMATCH[1]}"
+MINOR="${BASH_REMATCH[2]}"
+PATCH="${BASH_REMATCH[3]}"
+
+# Determine bump type
+BUMP_TYPE="${1:-patch}"
+
+case "$BUMP_TYPE" in
+    major)
+        MAJOR=$((MAJOR + 1))
+        MINOR=0
+        PATCH=0
+        ;;
+    minor)
+        MINOR=$((MINOR + 1))
+        PATCH=0
+        ;;
+    patch)
+        PATCH=$((PATCH + 1))
+        ;;
+    *)
+        echo "[X] Error: Invalid bump type '$BUMP_TYPE'"
+        echo "Usage: $0 [major|minor|patch]"
+        exit 1
+        ;;
+esac
+
+NEW_VERSION="$MAJOR.$MINOR.$PATCH"
+
+echo "New version: $NEW_VERSION"
+echo ""
+echo "This will update hardcoded versions in:"
+echo "  1. VERSION file"
+echo "  2. package.json (via sync-version.js)"
+echo "  3. installers/install.sh"
+echo "  4. installers/install.ps1"
+echo ""
+echo "Note: lib/ccs and lib/ccs.ps1 are now bootstraps"
+echo "      (delegate to Node.js, no version hardcoded)"
+echo ""
+
+# Already confirmed above in deprecation warning
+
+# Update VERSION file
+echo "$NEW_VERSION" > "$VERSION_FILE"
+echo "[OK] Updated VERSION file to $NEW_VERSION"
+
+# Note: lib/ccs and lib/ccs.ps1 are now lightweight bootstraps
+# They delegate to Node.js via npx - no version variable needed
+# Version is determined by the npm package at runtime
+
+# Update installers/install.sh
+INSTALL_SH="$CCS_DIR/installers/install.sh"
+if [[ -f "$INSTALL_SH" ]]; then
+    sed -i.bak "s/^CCS_VERSION=\".*\"/CCS_VERSION=\"$NEW_VERSION\"/" "$INSTALL_SH"
+    rm -f "$INSTALL_SH.bak"
+    echo "[OK] Updated installers/install.sh"
+else
+    echo "[!] installers/install.sh not found, skipping"
+fi
+
+# Update installers/install.ps1
+INSTALL_PS1="$CCS_DIR/installers/install.ps1"
+if [[ -f "$INSTALL_PS1" ]]; then
+    sed -i.bak "s/^\\\$CcsVersion = \".*\"/\\\$CcsVersion = \"$NEW_VERSION\"/" "$INSTALL_PS1"
+    rm -f "$INSTALL_PS1.bak"
+    echo "[OK] Updated installers/install.ps1"
+else
+    echo "[!] installers/install.ps1 not found, skipping"
+fi
+
+# Sync version to package.json
+echo "Syncing version to package.json..."
+if node "$SCRIPT_DIR/sync-version.js"; then
+    echo "[OK] Synced version to package.json"
+else
+    echo "[X] Error: Failed to sync version to package.json"
+    exit 1
+fi
+
+echo ""
+echo "[OK] Version bumped to $NEW_VERSION"
+echo ""
+echo "Next steps:"
+echo "  1. Review changes: git diff"
+echo "  2. Commit: git add VERSION package.json installers/install.sh installers/install.ps1"
+echo "  3. Commit: git commit -m \"chore: bump version to $NEW_VERSION\""
+echo "  4. Tag: git tag v$NEW_VERSION"
+echo "  5. Push: git push origin main && git push origin v$NEW_VERSION"

package/scripts/check-executables.js

@@ -0,0 +1,18 @@
+#!/usr/bin/env node
+const fs = require('fs');
+const path = require('path');
+
+// Ensure executables have correct permissions (Unix only)
+if (process.platform !== 'win32') {
+  const executables = [
+    path.join(__dirname, '..', 'bin', 'ccs.js'),
+    path.join(__dirname, '..', 'lib', 'ccs'),
+  ];
+
+  for (const file of executables) {
+    if (fs.existsSync(file)) {
+      fs.chmodSync(file, '755');
+      console.log(`✓ Set executable: ${path.basename(file)}`);
+    }
+  }
+}

package/scripts/clean-dist.js

@@ -0,0 +1,46 @@
+#!/usr/bin/env node
+
+/**
+ * Clean dist directory while preserving UI bundle
+ * The UI bundle is built separately and should not be deleted during regular builds
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+const DIST_DIR = path.join(__dirname, '../dist');
+const TSCONFIG_BUILDINFO = path.join(__dirname, '../tsconfig.tsbuildinfo');
+
+// Directories to preserve (from UI build)
+const PRESERVE = new Set(['ui']);
+
+function cleanDist() {
+  // Remove tsconfig.tsbuildinfo
+  if (fs.existsSync(TSCONFIG_BUILDINFO)) {
+    fs.unlinkSync(TSCONFIG_BUILDINFO);
+  }
+
+  // If dist doesn't exist, nothing to clean
+  if (!fs.existsSync(DIST_DIR)) {
+    return;
+  }
+
+  const entries = fs.readdirSync(DIST_DIR, { withFileTypes: true });
+
+  for (const entry of entries) {
+    // Skip preserved directories
+    if (PRESERVE.has(entry.name)) {
+      continue;
+    }
+
+    const fullPath = path.join(DIST_DIR, entry.name);
+
+    if (entry.isDirectory()) {
+      fs.rmSync(fullPath, { recursive: true, force: true });
+    } else {
+      fs.unlinkSync(fullPath);
+    }
+  }
+}
+
+cleanDist();

package/scripts/code-reviewer.ts

@@ -0,0 +1,217 @@
+#!/usr/bin/env bun
+/**
+ * AI Code Reviewer for CCS CLI
+ *
+ * Fetches PR diff, calls Claude via CLIProxyAPI, posts review as comment.
+ * Runs on self-hosted runner with localhost access to CLIProxyAPI:8317.
+ * Posts as ccs-agy-reviewer[bot] via GitHub App token.
+ *
+ * Usage: bun run scripts/code-reviewer.ts <PR_NUMBER>
+ * Env: CLIPROXY_API_KEY, GITHUB_REPOSITORY, GH_TOKEN
+ */
+
+import { $ } from 'bun';
+
+// Types
+interface PRContext {
+  number: number;
+  title: string;
+  body: string;
+  baseRef: string;
+  headRef: string;
+  files: Array<{ path: string; additions: number; deletions: number }>;
+  diff: string;
+}
+
+// Config
+const MAX_DIFF_LINES = 10000;
+const CLIPROXY_URL = process.env.CLIPROXY_URL || 'http://localhost:8317';
+const MODEL = process.env.REVIEW_MODEL || 'gemini-claude-opus-4-5-thinking';
+
+// System prompt for code review - new style
+const CODE_REVIEWER_SYSTEM_PROMPT = `You are the CCS AGY Code Reviewer, an expert AI assistant reviewing pull requests for the CCS CLI project.
+
+## Review Guidelines
+- Focus ONLY on changes in this PR - don't suggest unrelated improvements
+- Be concise - no fluff, no excessive praise
+- Provide specific file:line references for issues
+- Verify claims before making them (check if patterns exist, check actual code)
+- Avoid over-engineering suggestions for simple fixes
+
+## Check For
+1. **Bugs**: Logic errors, edge cases, null handling, race conditions
+2. **Security**: Injection, auth bypass, secrets exposure, data leaks
+3. **Performance**: N+1 queries, missing indexes, inefficient algorithms
+4. **TypeScript**: Proper typing, no \`any\`, null safety
+5. **Consistency**: Similar patterns exist elsewhere that need same fix?
+
+## Output Format
+Structure your response EXACTLY like this (no code fences, render as markdown):
+
+## 🔍 Code Review
+
+**Verdict**: [✅ Approve | ✅ Approve with suggestions | ⚠️ Request changes]
+
+### Summary
+[1-2 sentences on what the PR does and if it's correct]
+
+### ✅ What's Good
+- [Bullet points, 2-4 items max]
+
+### ⚠️ Issues Found
+| File:Line | Issue | Severity |
+|-----------|-------|----------|
+| \`file.ts:123\` | Description | 🔴 High / 🟡 Medium / 🟢 Low |
+
+(If no issues, write "None - LGTM")
+
+### 💡 Suggestions (Optional)
+- [Only if truly valuable, max 2 items]
+
+IMPORTANT: Output ONLY the markdown review. No JSON, no code blocks wrapping the review.`;
+
+// Fetch PR context
+async function getPRContext(prNumber: number, repo: string): Promise<PRContext> {
+  $.throws(true);
+
+  // Get PR metadata
+  const prJson =
+    await $`gh pr view ${prNumber} --repo ${repo} --json number,title,body,baseRefName,headRefName,files`.text();
+  const pr = JSON.parse(prJson);
+
+  // Get diff
+  let diff = await $`gh pr diff ${prNumber} --repo ${repo}`.text();
+
+  // Truncate if too large
+  const lines = diff.split('\n');
+  if (lines.length > MAX_DIFF_LINES) {
+    diff = lines.slice(0, MAX_DIFF_LINES).join('\n') + '\n\n[DIFF TRUNCATED - exceeded 10k lines]';
+  }
+
+  return {
+    number: pr.number,
+    title: pr.title,
+    body: pr.body || '',
+    baseRef: pr.baseRefName,
+    headRef: pr.headRefName,
+    files: pr.files || [],
+    diff,
+  };
+}
+
+// Call Claude via CLIProxyAPI
+async function callClaude(context: PRContext, repo: string): Promise<string> {
+  const apiKey = process.env.CLIPROXY_API_KEY;
+  if (!apiKey) throw new Error('CLIPROXY_API_KEY not set');
+
+  const userMessage = `REPO: ${repo}
+PR NUMBER: ${context.number}
+
+## Pull Request: ${context.title}
+
+### Description
+${context.body || '(No description provided)'}
+
+### Changed Files
+${context.files.map((f) => `- ${f.path} (+${f.additions}/-${f.deletions})`).join('\n')}
+
+### Diff
+\`\`\`diff
+${context.diff}
+\`\`\`
+
+Review this PR following the guidelines. Refer to the project's CLAUDE.md and docs/ folder for conventions.`;
+
+  const response = await fetch(`${CLIPROXY_URL}/v1/messages`, {
+    method: 'POST',
+    headers: {
+      'x-api-key': apiKey,
+      'anthropic-version': '2023-06-01',
+      'content-type': 'application/json',
+    },
+    body: JSON.stringify({
+      model: MODEL,
+      max_tokens: 4096,
+      system: CODE_REVIEWER_SYSTEM_PROMPT,
+      messages: [{ role: 'user', content: userMessage }],
+    }),
+  });
+
+  if (!response.ok) {
+    const error = await response.text();
+    throw new Error(`CLIProxyAPI error: ${response.status} - ${error}`);
+  }
+
+  const data = (await response.json()) as { content: Array<{ text: string }> };
+  const content = data.content[0]?.text;
+
+  if (!content) {
+    throw new Error('Empty response from Claude');
+  }
+
+  return content;
+}
+
+// Post review as PR comment
+async function postReview(prNumber: number, repo: string, reviewContent: string): Promise<void> {
+  // Use gh pr comment to post the review
+  await $`gh pr comment ${prNumber} --repo ${repo} --body ${reviewContent}`;
+}
+
+// Check if already reviewed this PR (avoid spam)
+async function hasRecentReview(prNumber: number, repo: string): Promise<boolean> {
+  try {
+    const comments =
+      await $`gh api repos/${repo}/issues/${prNumber}/comments --jq '[.[] | select(.body | contains("🔍 Code Review"))] | length'`.text();
+    return parseInt(comments.trim(), 10) > 0;
+  } catch {
+    return false;
+  }
+}
+
+// Main
+async function main() {
+  const prNumber = parseInt(process.argv[2], 10);
+  const repo = process.env.GITHUB_REPOSITORY || 'kaitranntt/ccs';
+  const forceReview = process.argv.includes('--force');
+
+  if (!prNumber || isNaN(prNumber)) {
+    console.error('Usage: bun run scripts/code-reviewer.ts <PR_NUMBER> [--force]');
+    process.exit(1);
+  }
+
+  console.log(`[i] Reviewing PR #${prNumber} in ${repo}`);
+
+  try {
+    // Check for existing review (avoid spam)
+    if (!forceReview && (await hasRecentReview(prNumber, repo))) {
+      console.log('[i] Already reviewed this PR. Use --force to review again.');
+      process.exit(0);
+    }
+
+    // 1. Get PR context
+    console.log('[i] Fetching PR context...');
+    const context = await getPRContext(prNumber, repo);
+    console.log(`[i] PR: "${context.title}" (${context.files.length} files changed)`);
+
+    const diffLines = context.diff.split('\n').length;
+    if (diffLines > MAX_DIFF_LINES) {
+      console.log(`[!] Diff too large (${diffLines} lines), truncated to ${MAX_DIFF_LINES}`);
+    }
+
+    // 2. Call Claude
+    console.log(`[i] Calling Claude (${MODEL}) for review...`);
+    const reviewContent = await callClaude(context, repo);
+    console.log('[i] Review generated');
+
+    // 3. Post review as comment
+    console.log('[i] Posting review to PR...');
+    await postReview(prNumber, repo, reviewContent);
+    console.log('[OK] Review posted successfully');
+  } catch (error) {
+    console.error('[X] Review failed:', error);
+    process.exit(1);
+  }
+}
+
+main();