fix: remove --bare flag that breaks auth

[kris-hansen]

PR Type:

Bug fix

---

PR Description:

• Removed the --bare flag from the command line arguments in the buildArgsAgentic function to fix authentication issues.
• Updated test cases to reflect the removal of the --bare flag.
• The --bare flag was causing 'Not logged in' errors by skipping OAuth/credential loading.

---

PR Main Files Walkthrough:

files:

• utils/models/claudecode.go: Removed the --bare flag from the args array in the buildArgsAgentic function. This change ensures that OAuth/credential loading is not skipped, thereby fixing the authentication issues.
• utils/models/claudecode_test.go: Updated multiple test cases in TestClaudeCodeBuildArgsAgentic to remove the --bare flag from the contains array. This aligns the test expectations with the updated behavior of the buildArgsAgentic function.

---

User Description:

The --bare flag skips OAuth/credential loading, causing 'Not logged in' errors when running Claude Code in -p mode.

Symptoms: claude -p --bare ... returns:

{"result":"Not logged in · Please run /login"}

Root cause: --bare was added based on research suggesting it provides 'deterministic subprocess behavior' but it also skips auth loading.

Fix: Remove --bare, keep other flags (-p, --dangerously-skip-permissions, --allowedTools, --output-format json).

[preston-ai]

PR Analysis

• 🎯 Main theme: Removal of the --bare flag to fix authentication issues in the buildArgsAgentic function.
• 📝 PR summary: This PR addresses a bug where the --bare flag was causing authentication issues by skipping OAuth/credential loading. The flag has been removed from the buildArgsAgentic function and corresponding test cases have been updated to reflect this change.
• 📌 Type of PR: Bug fix
• 🧪 Relevant tests added: Yes
• ✨ Focused PR: Yes, because the changes are specifically targeted at removing the --bare flag to resolve authentication issues, and the updates are consistently applied across the relevant files.
• ⏱️ Estimated effort to review [1-5]: 2, because the changes are straightforward, primarily involving the removal of a single flag and updating test cases accordingly.
• 🔒 Security concerns: No security concerns found

PR Feedback

• 💡 General suggestions: The PR effectively resolves the authentication issue by removing the problematic --bare flag. Ensure that the removal does not inadvertently affect other functionalities that relied on the deterministic behavior provided by the flag.

How to use

Instructions

To invoke the Preston AI, add a comment using one of the following commands:
/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/add_docs: Generate docstring for new components introduced in the PR.
/generate_labels: Generate labels for the PR based on the PR's contents.

[preston-ai]

Consider adding a comment or a log entry to indicate the removal of the --bare flag and its impact on authentication, to help future developers understand the reason behind this change. [medium]