@kaitranntt/ccs 7.65.2 → 7.65.3-dev.10

package/scripts/completion/ccs.zsh

@@ -1,319 +1,41 @@
 #compdef ccs
 
 # Zsh completion for CCS (Claude Code Switch)
-# Compatible with zsh 5.0+
-#
-# Installation:
-#   Add to ~/.zshrc:
-#     fpath=(~/.zsh/completion $fpath)
-#     autoload -Uz compinit && compinit
-#     source /path/to/ccs/scripts/completion/ccs.zsh
-#
-#   Or install system-wide:
-#     sudo cp scripts/completion/ccs.zsh /usr/local/share/zsh/site-functions/_ccs
-
-# Set up completion styles for better formatting and colors
-zstyle ':completion:*:*:ccs:*:commands' list-colors '=(#b)(auth|api|cliproxy|doctor|env|sync|update)([[:space:]]#--[[:space:]]#*)==0\;34=2\;37'
-zstyle ':completion:*:*:ccs:*:proxy-profiles' list-colors '=(#b)(gemini|codex|agy|qwen)([[:space:]]#--[[:space:]]#*)==0\;35=2\;37'
-zstyle ':completion:*:*:ccs:*:model-profiles' list-colors '=(#b)(default|glm|kimi|[^[:space:]]##)([[:space:]]#--[[:space:]]#*)==0\;32=2\;37'
-zstyle ':completion:*:*:ccs:*:account-profiles' list-colors '=(#b)([^[:space:]]##)([[:space:]]#--[[:space:]]#*)==0\;33=2\;37'
-zstyle ':completion:*:*:ccs:*' group-name ''
-zstyle ':completion:*:*:ccs:*:descriptions' format $'\n%B%F{yellow}── %d ──%f%b'
-zstyle ':completion:*:*:ccs:*' list-separator '  --  '
-zstyle ':completion:*:*:ccs:*' list-rows-first true
-zstyle ':completion:*:*:ccs:*' menu select
 
 _ccs() {
-  local -a commands proxy_profiles settings_profiles_described account_profiles_described cliproxy_variants_described
-  local curcontext="$curcontext" state line
-  typeset -A opt_args
-
-  # Define top-level commands
-  commands=(
-    'auth:Manage multiple Claude accounts'
-    'api:Manage API profiles (create/remove)'
-    'cliproxy:Manage CLIProxy variants and binary'
-    'doctor:Run health check and diagnostics'
-    'env:Export env vars for third-party tools'
-    'sync:Sync delegation commands and skills'
-    'update:Update CCS to latest version'
-  )
-
-  # Define CLIProxy hardcoded profiles (OAuth providers)
-  proxy_profiles=(
-    'gemini:Google Gemini (OAuth)'
-    'codex:OpenAI Codex (OAuth)'
-    'agy:Antigravity (OAuth)'
-    'qwen:Qwen Code (OAuth)'
-    'iflow:iFlow (OAuth)'
-    'kiro:Kiro (OAuth)'
-    'ghcp:GitHub Copilot (OAuth)'
-    'claude:Claude Direct (OAuth)'
-  )
-
-  # Define known settings profiles with descriptions
-  local -A profile_descriptions
-  profile_descriptions=(
-    'default' 'Default Claude Sonnet 4.5'
-    'glm'     'GLM-4.6 (cost-optimized)'
-    'kimi'    'Kimi for Coding (long-context)'
-  )
-
-  # Load settings-based profiles from config.json
-  if [[ -f ~/.ccs/config.json ]]; then
-    local -a raw_settings_profiles
-    raw_settings_profiles=(${(f)"$(jq -r '.profiles | keys[]' ~/.ccs/config.json 2>/dev/null)"})
-
-    for profile in $raw_settings_profiles; do
-      local desc="${profile_descriptions[$profile]:-Settings-based profile}"
-      settings_profiles_described+=("${profile}:${desc}")
-    done
+  local current
+  current="${words[CURRENT]}"
+
+  local -a tokens_before_current
+  if (( CURRENT > 2 )); then
+    tokens_before_current=("${words[@]:2:$((CURRENT-2))}")
+  else
+    tokens_before_current=()
   fi
 
-  # Load account-based profiles from profiles.json
-  if [[ -f ~/.ccs/profiles.json ]]; then
-    local -a raw_account_profiles
-    raw_account_profiles=(${(f)"$(jq -r '.profiles | keys[]' ~/.ccs/profiles.json 2>/dev/null)"})
-
-    for profile in $raw_account_profiles; do
-      account_profiles_described+=("${profile}:Account-based profile")
-    done
-  fi
-
-  # Load cliproxy variants from config.json
-  if [[ -f ~/.ccs/config.json ]]; then
-    local -a raw_cliproxy_variants
-    raw_cliproxy_variants=(${(f)"$(jq -r '.cliproxy | keys[]' ~/.ccs/config.json 2>/dev/null)"})
-
-    for variant in $raw_cliproxy_variants; do
-      cliproxy_variants_described+=("${variant}:CLIProxy variant")
-    done
-  fi
-
-  _arguments -C \
-    '(- *)'{-h,--help}'[Show help message]' \
-    '(- *)'{-v,--version}'[Show version information]' \
-    '(- *)'{-sc,--shell-completion}'[Install shell completion]' \
-    '1: :->command' \
-    '*:: :->args'
-
-  case $state in
-    command)
-      _describe -t commands 'commands' commands
-      _describe -t proxy-profiles 'CLIProxy profiles' proxy_profiles
-      _describe -t model-profiles 'model profiles' settings_profiles_described
-      _describe -t account-profiles 'account profiles' account_profiles_described
-      _describe -t cliproxy-variants 'CLIProxy variants' cliproxy_variants_described
-      ;;
-
-    args)
-      case $words[1] in
-        auth)
-          _ccs_auth
-          ;;
-        api)
-          _ccs_api
-          ;;
-        cliproxy)
-          _ccs_cliproxy
-          ;;
-        update)
-          _arguments \
-            '--force[Force reinstall current version]' \
-            '--beta[Install from dev channel]' \
-            '--dev[Install from dev channel]' \
-            '(- *)'{-h,--help}'[Show help]'
-          ;;
-        doctor)
-          _arguments \
-            '(- *)'{-h,--help}'[Show help for doctor command]'
-          ;;
-        env)
-          _arguments \
-            '--format[Output format]:format:(openai anthropic raw)' \
-            '--shell[Shell syntax]:shell:(auto bash zsh fish powershell)' \
-            '(- *)'{-h,--help}'[Show help]' \
-            '1:profile:($proxy_profiles ${(k)settings_profiles_described})'
-          ;;
-        gemini|codex|agy|qwen)
-          _arguments \
-            '--auth[Authenticate only]' \
-            '--config[Change model configuration]' \
-            '--logout[Clear authentication]' \
-            '--headless[Headless auth (for SSH)]' \
-            '(- *)'{-h,--help}'[Show help]'
-          ;;
-        --shell-completion|-sc)
-          _arguments \
-            '--bash[Install for bash]' \
-            '--zsh[Install for zsh]' \
-            '--fish[Install for fish]' \
-            '--powershell[Install for PowerShell]'
-          ;;
-        *)
-          _message 'Claude CLI arguments'
-          ;;
-      esac
-      ;;
-  esac
+  local -a suggestions
+  suggestions=("${(@f)$(__ccs_completion_run "${current}" "${tokens_before_current[@]}")}")
+  compadd -- "${suggestions[@]}"
 }
 
-_ccs_api() {
-  local curcontext="$curcontext" state line
-  typeset -A opt_args
-
-  local -a api_commands settings_profiles
-
-  api_commands=(
-    'create:Create new API profile (interactive)'
-    'list:List all API profiles'
-    'remove:Remove an API profile'
-  )
-
-  if [[ -f ~/.ccs/config.json ]]; then
-    settings_profiles=(${(f)"$(jq -r '.profiles | keys[]' ~/.ccs/config.json 2>/dev/null)"})
+__ccs_completion_run() {
+  local current="$1"
+  shift || true
+
+  local script_path script_dir repo_root repo_cli
+  script_path="${(%):-%N}"
+  script_dir="${script_path:A:h}"
+  repo_root="${script_dir:h:h}"
+  repo_cli="${repo_root}/dist/ccs.js"
+  if [[ ! -f "${repo_cli}" ]]; then
+    repo_cli="${repo_root}/bin/ccs.js"
   fi
-
-  _arguments -C \
-    '(- *)'{-h,--help}'[Show help for api commands]' \
-    '1: :->subcommand' \
-    '*:: :->subargs'
-
-  case $state in
-    subcommand)
-      _describe -t api-commands 'api commands' api_commands
-      ;;
-
-    subargs)
-      case $words[1] in
-        create)
-          _arguments \
-            '1:profile name:' \
-            '--base-url[API base URL]:url:' \
-            '--api-key[API key]:key:' \
-            '--model[Default model]:model:' \
-            '--force[Overwrite existing profile]' \
-            {--yes,-y}'[Skip prompts]'
-          ;;
-        list)
-          ;;
-        remove|delete|rm)
-          _arguments \
-            '1:profile:($settings_profiles)' \
-            {--yes,-y}'[Skip confirmation]'
-          ;;
-      esac
-      ;;
-  esac
-}
-
-_ccs_cliproxy() {
-  local curcontext="$curcontext" state line
-  typeset -A opt_args
-
-  local -a cliproxy_commands cliproxy_variants providers
-
-  cliproxy_commands=(
-    'create:Create new CLIProxy variant profile'
-    'list:List all CLIProxy variant profiles'
-    'remove:Remove a CLIProxy variant profile'
-  )
-
-  providers=(gemini codex agy qwen)
-
-  if [[ -f ~/.ccs/config.json ]]; then
-    cliproxy_variants=(${(f)"$(jq -r '.cliproxy | keys[]' ~/.ccs/config.json 2>/dev/null)"})
+  if [[ -f "${repo_cli}" ]]; then
+    node "${repo_cli}" __complete --shell zsh --current "${current}" -- "$@" 2>/dev/null
+    return 0
   fi
 
-  _arguments -C \
-    '(- *)'{-h,--help}'[Show help for cliproxy commands]' \
-    '--install[Install specific version]:version:' \
-    '--latest[Install latest version]' \
-    '1: :->subcommand' \
-    '*:: :->subargs'
-
-  case $state in
-    subcommand)
-      _describe -t cliproxy-commands 'cliproxy commands' cliproxy_commands
-      ;;
-
-    subargs)
-      case $words[1] in
-        create)
-          _arguments \
-            '1:variant name:' \
-            '--provider[Provider name]:provider:($providers)' \
-            '--model[Model name]:model:' \
-            '--force[Overwrite existing variant]' \
-            {--yes,-y}'[Skip prompts]'
-          ;;
-        list|ls)
-          ;;
-        remove|delete|rm)
-          _arguments \
-            '1:variant:($cliproxy_variants)' \
-            {--yes,-y}'[Skip confirmation]'
-          ;;
-      esac
-      ;;
-  esac
-}
-
-_ccs_auth() {
-  local curcontext="$curcontext" state line
-  typeset -A opt_args
-
-  local -a auth_commands account_profiles
-
-  auth_commands=(
-    'create:Create new profile and login'
-    'list:List all saved profiles'
-    'show:Show profile details'
-    'remove:Remove saved profile'
-    'default:Set default profile'
-  )
-
-  if [[ -f ~/.ccs/profiles.json ]]; then
-    account_profiles=(${(f)"$(jq -r '.profiles | keys[]' ~/.ccs/profiles.json 2>/dev/null)"})
+  if (( $+commands[ccs] )); then
+    ccs __complete --shell zsh --current "${current}" -- "$@" 2>/dev/null
   fi
-
-  _arguments -C \
-    '(- *)'{-h,--help}'[Show help for auth commands]' \
-    '1: :->subcommand' \
-    '*:: :->subargs'
-
-  case $state in
-    subcommand)
-      _describe -t auth-commands 'auth commands' auth_commands
-      ;;
-
-    subargs)
-      case $words[1] in
-        create)
-          _message 'new profile name'
-          _arguments '--force[Allow overwriting existing profile]'
-          ;;
-        list)
-          _arguments \
-            '--verbose[Show additional details]' \
-            '--json[Output in JSON format]'
-          ;;
-        show)
-          _arguments \
-            '1:profile:($account_profiles)' \
-            '--json[Output in JSON format]'
-          ;;
-        remove)
-          _arguments \
-            '1:profile:($account_profiles)' \
-            {--yes,-y}'[Skip confirmation prompts]'
-          ;;
-        default)
-          _arguments '1:profile:($account_profiles)'
-          ;;
-      esac
-      ;;
-  esac
 }
-
-_ccs "$@"

package/scripts/github/normalize-ai-review-output.mjs

@@ -65,7 +65,9 @@ function cleanMultilineText(value) {
 }
 
 function escapeMarkdown(value) {
-  return String(value).replace(/\\/g, '\\\\').replace(/([`*_{}[\]<>|])/g, '\\$1');
+  return String(value)
+    .replace(/\\/g, '\\\\')
+    .replace(/([`*_{}[\]<>|])/g, '\\$1');
 }
 
 function escapeMarkdownText(value) {
@@ -155,7 +157,8 @@ function normalizeRenderingMetadata(raw) {
   if (packetIncludedFiles !== null) metadata.packetIncludedFiles = packetIncludedFiles;
   if (packetTotalFiles !== null) metadata.packetTotalFiles = packetTotalFiles;
   if (packetOmittedFiles !== null) metadata.packetOmittedFiles = packetOmittedFiles;
-  if (scopeLabel === 'reviewable files' || scopeLabel === 'changed files') metadata.scopeLabel = scopeLabel;
+  if (scopeLabel === 'reviewable files' || scopeLabel === 'changed files')
+    metadata.scopeLabel = scopeLabel;
 
   return metadata;
 }
@@ -203,7 +206,8 @@ function formatScopeSummary(rendering) {
     typeof rendering.selectedChanges === 'number' &&
     typeof rendering.reviewableChanges === 'number'
   ) {
-    const changeLabel = scopeLabel === 'reviewable files' ? 'reviewable changed lines' : 'changed lines';
+    const changeLabel =
+      scopeLabel === 'reviewable files' ? 'reviewable changed lines' : 'changed lines';
     return `${fileScope}; ${rendering.selectedChanges}/${rendering.reviewableChanges} ${changeLabel}`;
   }
 
@@ -254,7 +258,8 @@ function formatReviewContext(rendering) {
     parts.push(`packet ${rendering.packetIncludedFiles}/${rendering.packetTotalFiles}`);
   }
 
-  const runtimeBudget = formatCombinedBudget(rendering) || formatTimeBudget(rendering) || formatTurnBudget(rendering);
+  const runtimeBudget =
+    formatCombinedBudget(rendering) || formatTimeBudget(rendering) || formatTurnBudget(rendering);
   if (runtimeBudget) {
     parts.push(runtimeBudget);
   }
@@ -301,7 +306,10 @@ function describeIncompleteOutcome({ reason, rendering, turnsUsed, status }) {
     return `The ${reviewLabel} ended before it could produce validated structured output within the available ${combinedBudget} runtime budget.`;
   }
 
-  if (classifyFallbackReason(reason) === 'missing' || classifyFallbackReason(reason) === 'invalid_json') {
+  if (
+    classifyFallbackReason(reason) === 'missing' ||
+    classifyFallbackReason(reason) === 'invalid_json'
+  ) {
     return `The ${reviewLabel} ended without validated structured output, so the normalizer published the safe fallback comment instead.`;
   }
 
@@ -344,7 +352,10 @@ function normalizeChecklistRows(fieldName, labelField, raw) {
 
   const rows = [];
   for (const [index, item] of raw.entries()) {
-    const label = validatePlainTextField(`${fieldName}[${index}].${labelField}`, item?.[labelField]);
+    const label = validatePlainTextField(
+      `${fieldName}[${index}].${labelField}`,
+      item?.[labelField]
+    );
     if (!label.ok) return label;
 
     const notes = validatePlainTextField(`${fieldName}[${index}].notes`, item?.notes);
@@ -479,16 +490,19 @@ function formatHotspotFiles(files) {
 function formatRemainingCoverage(rendering) {
   if (
     typeof rendering.reviewableFiles !== 'number' ||
-    (typeof rendering.packetIncludedFiles !== 'number' && typeof rendering.selectedFiles !== 'number')
+    (typeof rendering.packetIncludedFiles !== 'number' &&
+      typeof rendering.selectedFiles !== 'number')
   ) {
     return null;
   }
 
-  const coveredFiles = typeof rendering.packetIncludedFiles === 'number'
-    ? rendering.packetIncludedFiles
-    : rendering.selectedFiles;
+  const coveredFiles =
+    typeof rendering.packetIncludedFiles === 'number'
+      ? rendering.packetIncludedFiles
+      : rendering.selectedFiles;
   const remainingFiles = Math.max(rendering.reviewableFiles - coveredFiles, 0);
-  const packetOmittedFiles = typeof rendering.packetOmittedFiles === 'number' ? rendering.packetOmittedFiles : 0;
+  const packetOmittedFiles =
+    typeof rendering.packetOmittedFiles === 'number' ? rendering.packetOmittedFiles : 0;
   const hasChangeCounts =
     packetOmittedFiles === 0 &&
     typeof rendering.selectedChanges === 'number' &&
@@ -540,7 +554,11 @@ export function normalizeStructuredOutput(raw) {
   if (!overallRationale.ok) return overallRationale;
 
   const findings = Array.isArray(parsed.findings) ? parsed.findings : null;
-  const securityChecklist = normalizeChecklistRows('securityChecklist', 'check', parsed.securityChecklist);
+  const securityChecklist = normalizeChecklistRows(
+    'securityChecklist',
+    'check',
+    parsed.securityChecklist
+  );
   if (!securityChecklist.ok) return securityChecklist;
 
   const ccsCompliance = normalizeChecklistRows('ccsCompliance', 'rule', parsed.ccsCompliance);
@@ -582,7 +600,11 @@ export function normalizeStructuredOutput(raw) {
     if (finding && Object.hasOwn(finding, 'line')) {
       if (finding.line === null) {
         line = null;
-      } else if (typeof finding.line === 'number' && Number.isInteger(finding.line) && finding.line > 0) {
+      } else if (
+        typeof finding.line === 'number' &&
+        Number.isInteger(finding.line) &&
+        finding.line > 0
+      ) {
         line = finding.line;
       } else {
         return { ok: false, reason: `findings[${index}].line is invalid` };
@@ -668,7 +690,9 @@ function renderFindingReference(finding) {
 }
 
 function getOrderedFindings(findings) {
-  return SEVERITY_ORDER.flatMap((severity) => findings.filter((finding) => finding.severity === severity));
+  return SEVERITY_ORDER.flatMap((severity) =>
+    findings.filter((finding) => finding.severity === severity)
+  );
 }
 
 function renderTopFindings(findings) {
@@ -726,37 +750,65 @@ function renderDetailedFindings(findings) {
 
 export function renderStructuredReview(review, { model, rendering: renderOptions } = {}) {
   const rendering = mergeRenderingMetadata(review?.rendering, renderOptions);
-  const lines = [
-    '### Verdict',
-    '',
-    `**${ASSESSMENTS[review.overallAssessment]}** — ${renderInlineText(review.overallRationale)}`,
-    '',
-    renderInlineText(review.summary),
-  ];
+  const lines = ['### 📋 Summary', '', renderInlineText(review.summary)];
   const reviewContext = formatReviewContext(rendering);
 
   if (reviewContext) {
     lines.push('', reviewContext);
   }
 
-  lines.push('', '### Top Findings', '', ...renderTopFindings(review.findings));
-  lines.push(...renderSection(`### Detailed Findings (${review.findings.length})`, renderDetailedFindings(review.findings)));
+  lines.push('', '### 🔍 Findings');
+  if (review.findings.length === 0) {
+    lines.push('', 'No confirmed issues found after reviewing the diff and surrounding code.');
+  } else {
+    for (const severity of SEVERITY_ORDER) {
+      const scopedFindings = review.findings.filter((finding) => finding.severity === severity);
+      if (scopedFindings.length === 0) continue;
+
+      lines.push('', SEVERITY_HEADERS[severity], '');
+      for (const finding of scopedFindings) {
+        const snippets = Array.isArray(finding.snippets) ? finding.snippets : [];
+        lines.push(
+          `- **${renderCode(renderFindingReference(finding))} — ${renderInlineText(finding.title)}**`
+        );
+        lines.push(`  Problem: ${renderInlineText(finding.what)}`);
+        lines.push(`  Why it matters: ${renderInlineText(finding.why)}`);
+        lines.push(`  Suggested fix: ${renderInlineText(finding.fix)}`);
+
+        if (snippets.length > 0) {
+          lines.push('');
+          lines.push(...renderFindingSnippets(snippets));
+        }
+
+        lines.push('');
+      }
+    }
+
+    if (lines[lines.length - 1] === '') {
+      lines.pop();
+    }
+  }
+
   lines.push(
     ...renderSection(
-      `### Security Checklist (${review.securityChecklist.length})`,
+      '### 🔒 Security Checklist',
       renderChecklistTable('Check', 'check', review.securityChecklist)
     )
   );
   lines.push(
     ...renderSection(
-      `### CCS Compliance (${review.ccsCompliance.length})`,
+      '### 📊 CCS Compliance',
       renderChecklistTable('Rule', 'rule', review.ccsCompliance)
     )
   );
-  lines.push(...renderSection(`### Informational (${review.informational.length})`, renderBulletSection(review.informational)));
-  lines.push(...renderSection(`### What's Done Well (${review.strengths.length})`, renderBulletSection(review.strengths)));
+  lines.push(...renderSection('### 💡 Informational', renderBulletSection(review.informational)));
+  lines.push(...renderSection("### ✅ What's Done Well", renderBulletSection(review.strengths)));
 
   lines.push(
+    '',
+    '### 🎯 Overall Assessment',
+    '',
+    `**${ASSESSMENTS[review.overallAssessment]}** — ${renderInlineText(review.overallRationale)}`,
     '',
     `> 🤖 Reviewed by \`${model}\``
   );
@@ -784,7 +836,9 @@ export function renderIncompleteReview({
   ];
 
   if (rendering.mode) {
-    lines.push(`- Review mode: ${renderCode(rendering.mode)} (${escapeMarkdownText(REVIEW_MODE_DETAILS[rendering.mode])})`);
+    lines.push(
+      `- Review mode: ${renderCode(rendering.mode)} (${escapeMarkdownText(REVIEW_MODE_DETAILS[rendering.mode])})`
+    );
   }
   const scopeSummary = formatScopeSummary(rendering);
   if (scopeSummary) {
@@ -804,7 +858,9 @@ export function renderIncompleteReview({
   }
   const remainingCoverage = formatRemainingCoverage(rendering);
   if (remainingCoverage) {
-    lines.push(`- Remaining reviewable scope not fully covered: ${escapeMarkdownText(remainingCoverage)}`);
+    lines.push(
+      `- Remaining reviewable scope not fully covered: ${escapeMarkdownText(remainingCoverage)}`
+    );
   }
   lines.push(`- Manual follow-up: ${escapeMarkdownText(formatFallbackFollowUp(rendering))}`);
   if (runtimeTools?.length) {
@@ -814,7 +870,12 @@ export function renderIncompleteReview({
     lines.push(`- Turns used: ${turnsUsed}`);
   }
 
-  lines.push('', `Re-run \`/review\` or inspect [the workflow run](${runUrl}).`, '', `> 🤖 Reviewed by \`${model}\``);
+  lines.push(
+    '',
+    `Re-run \`/review\` or inspect [the workflow run](${runUrl}).`,
+    '',
+    `> 🤖 Reviewed by \`${model}\``
+  );
   return lines.join('\n');
 }
 
@@ -857,15 +918,16 @@ export function writeReviewFromEnv(env = process.env) {
   fs.writeFileSync(outputFile, `${content}\n`, 'utf8');
 
   if (!validation.ok) {
-    console.warn(`::warning::AI review output normalization fell back to incomplete comment: ${validation.reason}`);
+    console.warn(
+      `::warning::AI review output normalization fell back to incomplete comment: ${validation.reason}`
+    );
   }
 
   return { usedFallback: !validation.ok, content };
 }
 
 const isMain =
-  process.argv[1] &&
-  path.resolve(process.argv[1]) === path.resolve(fileURLToPath(import.meta.url));
+  process.argv[1] && path.resolve(process.argv[1]) === path.resolve(fileURLToPath(import.meta.url));
 
 if (isMain) {
   writeReviewFromEnv();