Skip to content

Add Harbor adapter that wraps headless Corbits exec - #687

Open
TheGreatAxios wants to merge 2 commits into
mainfrom
cl-6923-harbor-agent-adapter-wrapping-corbits-exec
Open

Add Harbor adapter that wraps headless Corbits exec#687
TheGreatAxios wants to merge 2 commits into
mainfrom
cl-6923-harbor-agent-adapter-wrapping-corbits-exec

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • Adds a thin Harbor BaseInstalledAgent under evals/harbor/ that runs product corbits exec with --dangerously-skip-permissions --force (no second agent loop).
  • Translates Harbor kwargs/env into a temporary --config settings.json (required baseURL + API key); ships argv unit tests and a trivial Harbor task scaffold for later dry-run.

Verification

  • PYTHONPATH=. python3 -m unittest evals.harbor.tests.test_argv -v passes
  • No product src/ changes; full Harbor CLI dry-run still needs a Linux ELF binary + Harbor install (tracked separately)

toward CL-6923

@linear-code

linear-code Bot commented Aug 25, 2026

Copy link
Copy Markdown
@TheGreatAxios
TheGreatAxios force-pushed the main branch 2 times, most recently from ab4779a to 8b115bf Compare August 27, 2026 23:46
@TheGreatAxios
TheGreatAxios force-pushed the cl-6923-harbor-agent-adapter-wrapping-corbits-exec branch from c91eda8 to 5fd886d Compare August 28, 2026 02:23
TheGreatAxios

This comment was marked as outdated.

Harbor needs a thin installed-agent bridge so Terminal-Bench and similar
cells can drive product Corbits without a second agent loop. Credentials
and baseURL go through a temporary settings.json for --config only;
end-to-end dry-run remains deferred until a Linux binary and Harbor CLI
are available on a suitable host.
The CLI has no --version flag, so install verification now uses --help and the version probe records the help banner. Run the exec command through environment.exec so the real non-zero exit code lands in context metadata instead of a hardcoded 1. Drop the cross-provider API key fallback and the no-op --force exec flag.
@TheGreatAxios
TheGreatAxios force-pushed the cl-6923-harbor-agent-adapter-wrapping-corbits-exec branch from f1273f0 to 746297d Compare August 28, 2026 19:14
TheGreatAxios

This comment was marked as outdated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant