claude-flow 2.0.0-alpha.78 → 2.0.0-alpha.79

package/src/cli/simple-commands/init/templates/github-safe-enhanced.js

@@ -0,0 +1,333 @@
+#!/usr/bin/env node
+
+/**
+ * Enhanced Safe GitHub CLI Helper - Production Ready
+ * Uses the comprehensive GitHubCliSafe wrapper for secure command execution
+ * 
+ * Prevents:
+ * - Timeout issues with large content
+ * - Command injection attacks  
+ * - Process resource leaks
+ * - Rate limiting issues
+ * - Input validation bypasses
+ * 
+ * Usage:
+ *   ./github-safe-enhanced.js issue comment 123 "Message with `backticks` and $(dangerous) content"
+ *   ./github-safe-enhanced.js pr create --title "Title" --body "Complex body with special chars"
+ *   ./github-safe-enhanced.js issue create --title "Bug Report" --body "Long description..." --labels "bug,urgent"
+ *   ./github-safe-enhanced.js release create v1.0.0 --title "Release v1.0.0" --body "Release notes..."
+ */
+
+// Import the production-ready GitHub CLI safety wrapper
+import { githubCli, GitHubCliError, GitHubCliTimeoutError, GitHubCliValidationError } from '../../../utils/github-cli-safety-wrapper.js';
+
+const args = process.argv.slice(2);
+
+/**
+ * Show help message
+ */
+function showHelp() {
+  console.log(`
+🛡️  Enhanced Safe GitHub CLI Helper
+
+FEATURES:
+  ✅ Injection attack prevention
+  ✅ Timeout handling with cleanup
+  ✅ Input validation and sanitization
+  ✅ Rate limiting protection
+  ✅ Comprehensive error handling
+  ✅ Process resource management
+
+USAGE:
+  ./github-safe-enhanced.js <command> <subcommand> [arguments] [options]
+
+COMMANDS:
+  Issue Operations:
+    issue create --title "Title" --body "Body" [--labels "label1,label2"] [--assignees "user1,user2"]
+    issue comment <number> "Comment body"
+    
+  Pull Request Operations:
+    pr create --title "Title" --body "Body" [--base branch] [--head branch] [--draft]
+    pr comment <number> "Comment body"
+    
+  Release Operations:
+    release create <tag> --title "Title" --body "Body" [--prerelease] [--draft]
+
+OPTIONS:
+  --timeout <ms>     Override default timeout (30000ms)
+  --verbose          Enable detailed logging
+  --dry-run         Show what would be executed without running
+  --help            Show this help message
+
+EXAMPLES:
+  # Create issue with special characters safely
+  ./github-safe-enhanced.js issue create \\
+    --title "Bug: Login fails with special chars" \\
+    --body "Steps: 1. Enter \`user@domain.com\` 2. Use password with \$(special) chars" \\
+    --labels "bug,high-priority"
+
+  # Add comment with code blocks
+  ./github-safe-enhanced.js issue comment 123 \\
+    "Fixed in commit abc123. Test with: \`npm test && npm run build\`"
+
+  # Create PR with complex body
+  ./github-safe-enhanced.js pr create \\
+    --title "Feature: Add authentication" \\
+    --body "## Changes\\n- Added JWT auth\\n- Updated tests\\n\\n\`\`\`js\\nconst token = jwt.sign(payload);\\n\`\`\`" \\
+    --base main --head feature/auth
+
+  # Create release with detailed notes
+  ./github-safe-enhanced.js release create v2.1.0 \\
+    --title "Version 2.1.0 - Security Update" \\
+    --body "## Features\\n- Enhanced security\\n- Bug fixes\\n\\n## Breaking Changes\\nNone"
+
+SECURITY FEATURES:
+  🔒 Blocks dangerous patterns: \$(cmd), \`cmd\`, eval(), exec()
+  🔒 Prevents command chaining: &&, ||, ;
+  🔒 Validates file sizes and input lengths
+  🔒 Uses secure temporary files with restricted permissions
+  🔒 Implements proper process cleanup and timeout handling
+`);
+}
+
+/**
+ * Parse command line arguments into structured format
+ */
+function parseArguments(args) {
+  if (args.length === 0 || args.includes('--help')) {
+    showHelp();
+    process.exit(0);
+  }
+
+  const [command, subcommand, ...restArgs] = args;
+  const options = {};
+  const positionalArgs = [];
+  
+  // Parse flags and options
+  for (let i = 0; i < restArgs.length; i++) {
+    const arg = restArgs[i];
+    
+    if (arg.startsWith('--')) {
+      const flagName = arg.substring(2);
+      const nextArg = restArgs[i + 1];
+      
+      if (nextArg && !nextArg.startsWith('--')) {
+        options[flagName] = nextArg;
+        i++; // Skip the next argument
+      } else {
+        options[flagName] = true;
+      }
+    } else {
+      positionalArgs.push(arg);
+    }
+  }
+  
+  return { command, subcommand, positionalArgs, options };
+}
+
+/**
+ * Execute GitHub CLI command safely
+ */
+async function executeCommand(command, subcommand, positionalArgs, options) {
+  try {
+    // Handle dry-run mode
+    if (options['dry-run']) {
+      console.log('🔍 DRY RUN MODE - Would execute:');
+      console.log(`Command: ${command} ${subcommand}`);
+      console.log(`Arguments:`, positionalArgs);
+      console.log(`Options:`, options);
+      return;
+    }
+    
+    // Configure GitHub CLI wrapper
+    const cliOptions = {
+      timeout: parseInt(options.timeout) || 30000,
+      enableLogging: options.verbose || false
+    };
+    
+    let result;
+    
+    // Route to appropriate method based on command
+    if (command === 'issue') {
+      result = await handleIssueCommand(subcommand, positionalArgs, options, cliOptions);
+    } else if (command === 'pr') {
+      result = await handlePRCommand(subcommand, positionalArgs, options, cliOptions);
+    } else if (command === 'release') {
+      result = await handleReleaseCommand(subcommand, positionalArgs, options, cliOptions);
+    } else {
+      throw new Error(`Unsupported command: ${command}`);
+    }
+    
+    // Output result
+    console.log('✅ Command executed successfully');
+    if (options.verbose && result.stdout) {
+      console.log('Output:', result.stdout);
+    }
+    
+  } catch (error) {
+    console.error('❌ Command failed:', error.message);
+    
+    if (error instanceof GitHubCliTimeoutError) {
+      console.error('💡 Try increasing timeout with --timeout <ms>');
+    } else if (error instanceof GitHubCliValidationError) {
+      console.error('💡 Input validation failed. Check for dangerous characters or patterns.');
+    }
+    
+    if (options.verbose && error.details) {
+      console.error('Details:', error.details);
+    }
+    
+    process.exit(1);
+  }
+}
+
+/**
+ * Handle issue commands
+ */
+async function handleIssueCommand(subcommand, positionalArgs, options, cliOptions) {
+  if (subcommand === 'create') {
+    if (!options.title || !options.body) {
+      throw new Error('Issue creation requires --title and --body options');
+    }
+    
+    return await githubCli.createIssue({
+      title: options.title,
+      body: options.body,
+      labels: options.labels ? options.labels.split(',') : [],
+      assignees: options.assignees ? options.assignees.split(',') : [],
+      ...cliOptions
+    });
+    
+  } else if (subcommand === 'comment') {
+    const [issueNumber] = positionalArgs;
+    const body = positionalArgs[1] || options.body;
+    
+    if (!issueNumber || !body) {
+      throw new Error('Issue comment requires issue number and body');
+    }
+    
+    return await githubCli.addIssueComment(parseInt(issueNumber), body, cliOptions);
+    
+  } else {
+    throw new Error(`Unsupported issue subcommand: ${subcommand}`);
+  }
+}
+
+/**
+ * Handle PR commands
+ */
+async function handlePRCommand(subcommand, positionalArgs, options, cliOptions) {
+  if (subcommand === 'create') {
+    if (!options.title || !options.body) {
+      throw new Error('PR creation requires --title and --body options');
+    }
+    
+    return await githubCli.createPR({
+      title: options.title,
+      body: options.body,
+      base: options.base || 'main',
+      head: options.head,
+      draft: options.draft || false,
+      ...cliOptions
+    });
+    
+  } else if (subcommand === 'comment') {
+    const [prNumber] = positionalArgs;
+    const body = positionalArgs[1] || options.body;
+    
+    if (!prNumber || !body) {
+      throw new Error('PR comment requires PR number and body');
+    }
+    
+    return await githubCli.addPRComment(parseInt(prNumber), body, cliOptions);
+    
+  } else {
+    throw new Error(`Unsupported PR subcommand: ${subcommand}`);
+  }
+}
+
+/**
+ * Handle release commands
+ */
+async function handleReleaseCommand(subcommand, positionalArgs, options, cliOptions) {
+  if (subcommand === 'create') {
+    const [tag] = positionalArgs;
+    
+    if (!tag || !options.title || !options.body) {
+      throw new Error('Release creation requires tag, --title, and --body');
+    }
+    
+    return await githubCli.createRelease({
+      tag,
+      title: options.title,
+      body: options.body,
+      prerelease: options.prerelease || false,
+      draft: options.draft || false,
+      ...cliOptions
+    });
+    
+  } else {
+    throw new Error(`Unsupported release subcommand: ${subcommand}`);
+  }
+}
+
+/**
+ * Main execution
+ */
+async function main() {
+  try {
+    // Parse arguments
+    const { command, subcommand, positionalArgs, options } = parseArguments(args);
+    
+    // Check GitHub CLI availability
+    const isAvailable = await githubCli.checkGitHubCli();
+    if (!isAvailable) {
+      console.error('❌ GitHub CLI is not installed or not in PATH');
+      console.error('💡 Install from: https://cli.github.com/');
+      process.exit(1);
+    }
+    
+    // Check authentication (unless dry-run)
+    if (!options['dry-run']) {
+      const isAuthenticated = await githubCli.checkAuthentication();
+      if (!isAuthenticated) {
+        console.error('❌ GitHub CLI is not authenticated');
+        console.error('💡 Run: gh auth login');
+        process.exit(1);
+      }
+    }
+    
+    // Execute command
+    await executeCommand(command, subcommand, positionalArgs, options);
+    
+  } catch (error) {
+    console.error('❌ Unexpected error:', error.message);
+    if (args.includes('--verbose')) {
+      console.error('Stack trace:', error.stack);
+    }
+    process.exit(1);
+  }
+}
+
+// Handle process cleanup
+process.on('SIGINT', async () => {
+  console.log('\n🛑 Received interrupt signal, cleaning up...');
+  await githubCli.cleanup();
+  process.exit(0);
+});
+
+process.on('SIGTERM', async () => {
+  console.log('\n🛑 Received termination signal, cleaning up...');
+  await githubCli.cleanup();
+  process.exit(0);
+});
+
+// Run if called directly
+if (import.meta.main || process.argv[1].endsWith('github-safe-enhanced.js')) {
+  main().catch(error => {
+    console.error('💥 Fatal error:', error.message);
+    process.exit(1);
+  });
+}
+
+export { executeCommand, handleIssueCommand, handlePRCommand, handleReleaseCommand };

package/src/cli/simple-commands/init/validation/config-validator.js

@@ -1,3 +1,4 @@
+import { promises as fs } from 'fs';
 // config-validator.js - Configuration file validation
 
 export class ConfigValidator {
@@ -20,7 +21,7 @@ export class ConfigValidator {
 
     try {
       // Check if file exists
-      const stat = await Deno.stat(roomodesPath);
+      const stat = await fs.stat(roomodesPath);
       if (!stat.isFile) {
         result.success = false;
         result.errors.push('.roomodes exists but is not a file');
@@ -28,7 +29,7 @@ export class ConfigValidator {
       }
 
       // Read and parse JSON
-      const content = await Deno.readTextFile(roomodesPath);
+      const content = await fs.readFile(roomodesPath, 'utf8');
 
       try {
         const config = JSON.parse(content);
@@ -71,7 +72,7 @@ export class ConfigValidator {
     const claudeMdPath = `${this.workingDir}/CLAUDE.md`;
 
     try {
-      const content = await Deno.readTextFile(claudeMdPath);
+      const content = await fs.readFile(claudeMdPath, 'utf8');
       result.content = content;
 
       // Check for required sections
@@ -123,7 +124,7 @@ export class ConfigValidator {
     const memoryDataPath = `${this.workingDir}/memory/claude-flow-data.json`;
 
     try {
-      const content = await Deno.readTextFile(memoryDataPath);
+      const content = await fs.readFile(memoryDataPath, 'utf8');
 
       try {
         const data = JSON.parse(content);
@@ -162,7 +163,7 @@ export class ConfigValidator {
     const coordinationPath = `${this.workingDir}/coordination.md`;
 
     try {
-      const content = await Deno.readTextFile(coordinationPath);
+      const content = await fs.readFile(coordinationPath, 'utf8');
       result.content = content;
 
       // Check for required sections
@@ -203,7 +204,7 @@ export class ConfigValidator {
     const executablePath = `${this.workingDir}/claude-flow`;
 
     try {
-      const stat = await Deno.stat(executablePath);
+      const stat = await fs.stat(executablePath);
 
       if (!stat.isFile) {
         result.success = false;
@@ -220,7 +221,7 @@ export class ConfigValidator {
       }
 
       // Read and validate content
-      const content = await Deno.readTextFile(executablePath);
+      const content = await fs.readFile(executablePath, 'utf8');
 
       // Check for required elements
       if (content.includes('#!/usr/bin/env')) {

package/src/cli/simple-commands/init/validation/health-checker.js

@@ -1,3 +1,4 @@
+import { promises as fs } from 'fs';
 // health-checker.js - System health checks for SPARC initialization
 
 export class HealthChecker {
@@ -108,7 +109,7 @@ export class HealthChecker {
         const templatePath = `${this.workingDir}/${template}`;
 
         try {
-          const content = await Deno.readTextFile(templatePath);
+          const content = await fs.readFile(templatePath, 'utf8');
           if (content.length < 50) {
             result.templates.corrupted.push(template);
           } else {
@@ -277,7 +278,7 @@ export class HealthChecker {
     try {
       // Check if mode exists in .roomodes
       const roomodesPath = `${this.workingDir}/.roomodes`;
-      const content = await Deno.readTextFile(roomodesPath);
+      const content = await fs.readFile(roomodesPath, 'utf8');
       const config = JSON.parse(content);
 
       return !!(config.modes && config.modes[mode]);
@@ -326,7 +327,7 @@ export class HealthChecker {
     try {
       // Compare .roomodes with slash commands
       const roomodesPath = `${this.workingDir}/.roomodes`;
-      const content = await Deno.readTextFile(roomodesPath);
+      const content = await fs.readFile(roomodesPath, 'utf8');
       const config = JSON.parse(content);
 
       if (config.modes) {
@@ -368,7 +369,7 @@ export class HealthChecker {
 
     try {
       const claudePath = `${this.workingDir}/CLAUDE.md`;
-      const content = await Deno.readTextFile(claudePath);
+      const content = await fs.readFile(claudePath, 'utf8');
 
       // Check if mentioned commands exist
       const mentionedCommands = ['claude-flow sparc', 'npm run build', 'npm run test'];
@@ -405,7 +406,7 @@ export class HealthChecker {
     try {
       // Check if memory structure matches documentation
       const memoryDataPath = `${this.workingDir}/memory/claude-flow-data.json`;
-      const data = JSON.parse(await Deno.readTextFile(memoryDataPath));
+      const data = JSON.parse(await fs.readFile(memoryDataPath, 'utf8'));
 
       // Basic structure validation
       if (!data.agents || !data.tasks) {

package/src/cli/simple-commands/init/validation/mode-validator.js

@@ -1,3 +1,4 @@
+import { promises as fs } from 'fs';
 // mode-validator.js - SPARC mode functionality testing
 
 export class ModeValidator {
@@ -143,7 +144,7 @@ export class ModeValidator {
     try {
       // Try to get modes from .roomodes
       const roomodesPath = `${this.workingDir}/.roomodes`;
-      const content = await Deno.readTextFile(roomodesPath);
+      const content = await fs.readFile(roomodesPath, 'utf8');
       const config = JSON.parse(content);
 
       if (config.modes && typeof config.modes === 'object') {
@@ -210,7 +211,7 @@ export class ModeValidator {
     try {
       // Read .roomodes and validate mode config
       const roomodesPath = `${this.workingDir}/.roomodes`;
-      const content = await Deno.readTextFile(roomodesPath);
+      const content = await fs.readFile(roomodesPath, 'utf8');
       const config = JSON.parse(content);
 
       if (!config.modes || !config.modes[modeName]) {
@@ -355,7 +356,7 @@ export class ModeValidator {
 
     try {
       const workflowPath = `${this.workingDir}/.roo/workflows/${filename}`;
-      const content = await Deno.readTextFile(workflowPath);
+      const content = await fs.readFile(workflowPath, 'utf8');
 
       // Parse JSON
       const workflow = JSON.parse(content);

package/src/cli/simple-commands/init/validation/post-init-validator.js

@@ -1,3 +1,4 @@
+import { promises as fs } from 'fs';
 // post-init-validator.js - Post-initialization verification checks
 
 export class PostInitValidator {
@@ -30,7 +31,7 @@ export class PostInitValidator {
       const filePath = `${this.workingDir}/${file.path}`;
 
       try {
-        const stat = await Deno.stat(filePath);
+        const stat = await fs.stat(filePath);
 
         // Check if it exists and is a file
         if (!stat.isFile) {
@@ -62,7 +63,7 @@ export class PostInitValidator {
 
         // Try to read the file
         try {
-          await Deno.readTextFile(filePath);
+          await fs.readFile(filePath, 'utf8');
           result.files[file.path] = { status: 'ok', size: stat.size };
         } catch (readError) {
           result.success = false;
@@ -117,7 +118,7 @@ export class PostInitValidator {
       const dirPath = `${this.workingDir}/${dir}`;
 
       try {
-        const stat = await Deno.stat(dirPath);
+        const stat = await fs.stat(dirPath);
         if (!stat.isDirectory) {
           result.success = false;
           result.errors.push(`Expected directory but found file: ${dir}`);
@@ -135,7 +136,7 @@ export class PostInitValidator {
       const dirPath = `${this.workingDir}/${dir}`;
 
       try {
-        await Deno.stat(dirPath);
+        await fs.stat(dirPath);
       } catch {
         if (dir.includes('.roo') || dir.includes('sparc')) {
           result.warnings.push(`Optional SPARC directory missing: ${dir}`);
@@ -227,7 +228,7 @@ export class PostInitValidator {
       const itemPath = `${this.workingDir}/${item.path}`;
 
       try {
-        const stat = await Deno.stat(itemPath);
+        const stat = await fs.stat(itemPath);
         const actualMode = stat.mode & 0o777;
         const expectedMode = item.requiredMode;
 
@@ -265,7 +266,7 @@ export class PostInitValidator {
 
     for (const dir of expectedDirs) {
       try {
-        await Deno.stat(`${this.workingDir}/memory/${dir}`);
+        await fs.stat(`${this.workingDir}/memory/${dir}`);
         structure.dirs.push(dir);
       } catch {
         structure.valid = false;
@@ -274,7 +275,7 @@ export class PostInitValidator {
 
     for (const file of expectedFiles) {
       try {
-        await Deno.stat(`${this.workingDir}/memory/${file}`);
+        await fs.stat(`${this.workingDir}/memory/${file}`);
         structure.files.push(file);
       } catch {
         structure.valid = false;
@@ -294,7 +295,7 @@ export class PostInitValidator {
 
     for (const dir of expectedDirs) {
       try {
-        await Deno.stat(`${this.workingDir}/coordination/${dir}`);
+        await fs.stat(`${this.workingDir}/coordination/${dir}`);
         structure.dirs.push(dir);
       } catch {
         structure.valid = false;
@@ -315,7 +316,7 @@ export class PostInitValidator {
 
     for (const dir of expectedDirs) {
       try {
-        await Deno.stat(`${this.workingDir}/.claude/${dir}`);
+        await fs.stat(`${this.workingDir}/.claude/${dir}`);
         structure.dirs.push(dir);
       } catch {
         structure.valid = false;
@@ -325,7 +326,7 @@ export class PostInitValidator {
     // Check if there are any command files
     try {
       const entries = [];
-      for await (const entry of Deno.readDir(`${this.workingDir}/.claude/commands`)) {
+      for await (const entry of fs.readdir(`${this.workingDir}/.claude/commands`)) {
         if (entry.isFile && entry.name.endsWith('.js')) {
           entries.push(entry.name);
         }
@@ -341,7 +342,7 @@ export class PostInitValidator {
 
   async checkSparcExists() {
     try {
-      await Deno.stat(`${this.workingDir}/.roomodes`);
+      await fs.stat(`${this.workingDir}/.roomodes`);
       return true;
     } catch {
       return false;
@@ -358,7 +359,7 @@ export class PostInitValidator {
 
     // Check .roomodes file
     try {
-      const stat = await Deno.stat(`${this.workingDir}/.roomodes`);
+      const stat = await fs.stat(`${this.workingDir}/.roomodes`);
       structure.hasRoomodes = stat.isFile;
     } catch {
       structure.valid = false;
@@ -366,14 +367,14 @@ export class PostInitValidator {
 
     // Check .roo directory
     try {
-      const stat = await Deno.stat(`${this.workingDir}/.roo`);
+      const stat = await fs.stat(`${this.workingDir}/.roo`);
       structure.hasRooDir = stat.isDirectory;
 
       if (structure.hasRooDir) {
         const expectedDirs = ['templates', 'workflows', 'modes', 'configs'];
         for (const dir of expectedDirs) {
           try {
-            await Deno.stat(`${this.workingDir}/.roo/${dir}`);
+            await fs.stat(`${this.workingDir}/.roo/${dir}`);
             structure.dirs.push(dir);
           } catch {
             // Optional subdirectories

package/src/cli/simple-commands/init/validation/test-runner.js

@@ -1,6 +1,7 @@
 // test-runner.js - Test runner for validation and rollback systems
 
 import { ValidationSystem } from './index.js';
+import { promises as fs } from 'fs';
 import { RollbackSystem } from '../rollback/index.js';
 import { printSuccess, printError, printWarning } from '../../../utils.js';
 
@@ -410,9 +411,9 @@ export class ValidationTestRunner {
 
   async createTestFiles() {
     try {
-      await Deno.mkdir(`${this.workingDir}/test-temp`, { recursive: true });
-      await Deno.writeTextFile(`${this.workingDir}/test-temp/CLAUDE.md`, '# Test CLAUDE.md');
-      await Deno.writeTextFile(`${this.workingDir}/test-temp/memory-bank.md`, '# Test Memory Bank');
+      await fs.mkdir(`${this.workingDir}/test-temp`, { recursive: true });
+      await fs.writeFile(`${this.workingDir}/test-temp/CLAUDE.md`, '# Test CLAUDE.md', 'utf8');
+      await fs.writeFile(`${this.workingDir}/test-temp/memory-bank.md`, '# Test Memory Bank', 'utf8');
     } catch {
       // Test files creation failed - not critical for testing
     }
@@ -420,7 +421,7 @@ export class ValidationTestRunner {
 
   async cleanupTestFiles() {
     try {
-      await Deno.remove(`${this.workingDir}/test-temp`, { recursive: true });
+      await fs.unlink(`${this.workingDir}/test-temp`, { recursive: true });
     } catch {
       // Cleanup failed - not critical
     }
@@ -437,9 +438,8 @@ export class ValidationTestRunner {
         },
       };
 
-      await Deno.writeTextFile(
-        `${this.workingDir}/test-roomodes`,
-        JSON.stringify(testConfig, null, 2),
+      await fs.writeFile(
+        `${this.workingDir}/test-roomodes`, JSON.stringify(testConfig, null, 2, 'utf8'),
       );
     } catch {
       // Test config creation failed - not critical
@@ -448,7 +448,7 @@ export class ValidationTestRunner {
 
   async cleanupTestConfigs() {
     try {
-      await Deno.remove(`${this.workingDir}/test-roomodes`);
+      await fs.unlink(`${this.workingDir}/test-roomodes`);
     } catch {
       // Cleanup failed - not critical
     }
@@ -457,7 +457,7 @@ export class ValidationTestRunner {
   async createTestSparcConfig() {
     try {
       await this.createTestConfigs();
-      await Deno.mkdir(`${this.workingDir}/test-roo`, { recursive: true });
+      await fs.mkdir(`${this.workingDir}/test-roo`, { recursive: true });
     } catch {
       // Test SPARC config creation failed - not critical
     }
@@ -466,7 +466,7 @@ export class ValidationTestRunner {
   async cleanupTestSparcConfig() {
     try {
       await this.cleanupTestConfigs();
-      await Deno.remove(`${this.workingDir}/test-roo`, { recursive: true });
+      await fs.unlink(`${this.workingDir}/test-roo`, { recursive: true });
     } catch {
       // Cleanup failed - not critical
     }