Skip to content

feat: add Claude Code harness setup and diagnostics#3722

Merged
Sayt-0 merged 1 commit into
mainfrom
feat/claude-code-harness-setup
Jul 18, 2026
Merged

feat: add Claude Code harness setup and diagnostics#3722
Sayt-0 merged 1 commit into
mainfrom
feat/claude-code-harness-setup

Conversation

@Sayt-0

@Sayt-0 Sayt-0 commented Jul 17, 2026

Copy link
Copy Markdown
Member

Summary

  • Add an interactive Claude Code harness setup path backed by the official claude CLI and its existing Claude subscription login.
  • Diagnose Claude Code installation and authentication when an agent configuration uses a claude-code harness.
  • Document harness model selection, effort levels, authentication inheritance, limitations, and security behavior.

Changes

Area Details
Setup Add a fourth docker agent setup path for Claude Code, with an optional model override, configurable effort, explicit login confirmation, and generation of claude-code-agent.yaml.
Doctor Check Claude Code installation, version, and safe authentication metadata only when the supplied configuration declares a claude-code harness.
Configuration Accept xhigh for Claude Code harness effort across config versions v9, v10, v11, latest, and the JSON schema.
Harness-only configs Do not require an unrelated provider API key or Docker Model Runner when every agent delegates to an external coding harness.
Documentation Cover root and sub-agent usage, login inheritance, model defaults, effort configuration, toolset limitations, and permission behavior.

Security

  • Invoke the Claude CLI directly without a shell.
  • Never read, copy, print, or store Claude credentials.
  • Retain only non-identifying authentication metadata from claude auth status --json.
  • Run claude auth login --claudeai only after explicit confirmation.
  • Warn that the current harness uses --dangerously-skip-permissions.
  • Recommend an isolated git worktree for harness runs.
  • Clarify that Docker Agent sandbox mode does not automatically carry the host Claude CLI or its login.

Validation

  • go build ./...
  • go test ./... -count=1
  • golangci-lint run
  • go run ./lint .
  • Hugo documentation build
  • Real docker agent doctor smoke test with an authenticated Claude Code CLI

The full test suite passes with proxy environment variables unset. The documentation link checker still reports pre-existing external 403/404 responses and an unrelated existing MCP anchor.

@aheritier aheritier added area/agent For work that has to do with the general agent loop/agentic features of the app area/cli CLI commands, flags, output formatting area/config For configuration parsing, YAML, environment variables area/docs Documentation changes kind/feat PR adds a new feature (maps to feat:). Use on PRs only. labels Jul 17, 2026
@Sayt-0
Sayt-0 marked this pull request as ready for review July 17, 2026 18:26
@Sayt-0
Sayt-0 requested a review from a team as a code owner July 17, 2026 18:26
@Sayt-0
Sayt-0 enabled auto-merge July 17, 2026 18:26
@Sayt-0
Sayt-0 merged commit 188d3de into main Jul 18, 2026
21 checks passed
@Sayt-0
Sayt-0 deleted the feat/claude-code-harness-setup branch July 18, 2026 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/agent For work that has to do with the general agent loop/agentic features of the app area/cli CLI commands, flags, output formatting area/config For configuration parsing, YAML, environment variables area/docs Documentation changes kind/feat PR adds a new feature (maps to feat:). Use on PRs only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants