Ensure --output json is always added to CLI commands

[Copilot]

🎯 Aim

The runCliCommand function executes commands without ensuring JSON output format, leading to inconsistent and unparseable responses.

📷 Result

N/A - Internal logic change, no UI impact

✅ What was done

• Modified runCliCommand in src/util.ts to append --output json when not explicitly specified
• Preserves user's explicit --output choice (text/csv/md) when present

Implementation:

export async function runCliCommand(command: string): Promise<string> {
    // Ensure --output json is always present
    if (!command.includes('--output')) {
        command = `${command} --output json`;
    }
    
    return new Promise((resolve, reject) => {
        // ... existing code
    });
}

Behavior:

• m365 status → m365 status --output json
• m365 status --output text → m365 status --output text (unchanged)

🔗 Related issue

Closes: #[issue_number]

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Ensure --output json is always added to CLI commands</issue_title>
<issue_description>Description:

Summary

Currently, the runCliCommand function in src/util.ts executes commands without ensuring that the output format is JSON. This could lead to inconsistent output formats that are harder to parse.

Current Behavior

The runCliCommand function executes commands as-is:

const subprocess = spawn(command, {
    shell: true,
    timeout: 120000,
});

Proposed Enhancement

Modify runCliCommand to automatically append --output json to commands if it's not already present. This would ensure:

• Consistent JSON output for easier parsing
• Better error handling and response processing
• More reliable integration with MCP server responses

Example Implementation

export async function runCliCommand(command: string): Promise<string> {
    // Ensure --output json is always present
    if (!command.includes('--output')) {
        command = `${command} --output json`;
    }
    
    return new Promise((resolve, reject) => {
        const subprocess = spawn(command, {
            shell: true,
            timeout: 120000,
        });
        // ... rest of implementation
    });
}

Benefits

• Guarantees parseable JSON responses
• Reduces potential errors from text parsing
  </issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Ensure --output json is always added to CLI commands #72

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.