[Repo Assist] fix: replace console.group/time with VS Code OutputChannel for debug logging

[github-actions]

🤖 This is an automated pull request from Repo Assist.

Problem

Closes #14

When "phpcbf.debug": true is set, the extension crashes with:

TypeError: console.group is not a function

This occurs because VS Code's extension host does not expose console.group(), console.time(), console.timeEnd(), or console.groupEnd() in some versions/environments.

Additionally, the debug output was silently discarded:

• phpcbfError was set inside an async exit callback but checked synchronously right after the Promise setup (always false), so the stdout listener for error output was never registered — error output from phpcbf was completely swallowed.
• Exit code 3 set phpcbfError = true but never called resolve()/reject(), leaving the Promise permanently unresolved (hang), and in strict mode the undeclared phpcbfError assignment would throw a ReferenceError.

Root cause

The extension used browser-style console.group/console.time APIs that are not part of VS Code's extension host sandbox.

Fix

• Add a _debugLog(message) helper on the PHPCBF class that writes to a VS Code OutputChannel (View → Output → phpcbf), with a console.log fallback for testing outside VS Code.
• Replace all console.group, console.time, console.timeEnd, console.groupEnd calls with _debugLog().
• Replace stderr console.log with _debugLog() so stderr is always visible in the Output panel (not just in debug mode).
• Remove the broken phpcbfError stdout-capture pattern; stdout is now streamed directly to _debugLog when debug is enabled.
• Fix exit code 3: call reject() so the Promise settles and show a window.showErrorMessage notification.
• Create the OutputChannel in activate() and push it to context.subscriptions for proper cleanup.

Trade-offs

Debug output now appears in View → Output → phpcbf instead of the developer console. This is the VS Code-recommended approach and produces cleaner, always-accessible output for users.

Test Status

✅ All 7 unit tests pass (npm run test:unit).

No new tests were added for this change — the debug path requires a live VS Code extension host and phpcbf binary. The fix eliminates the crash that was reported and removes dead code.

Generated by Repo Assist · ◷

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@346204513ecfa08b81566450d7d599556807389f