Skip to content

[DEV-101] Document how to add a new agent harness in the agent-harness package README - #92

Merged
danii1 merged 1 commit into
mainfrom
feature/dev-101
Aug 27, 2026
Merged

[DEV-101] Document how to add a new agent harness in the agent-harness package README#92
danii1 merged 1 commit into
mainfrom
feature/dev-101

Conversation

@danii1

@danii1 danii1 commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Task: DEV-101

Summary: Document how to add a new agent harness in the agent-harness package README

Implementation Details

I have full context now. Writing the README:
These typecheck errors are all pre-existing source files missing Node/Bun ambient types — looks like dependencies aren't installed in this worktree. Installing and re-running:
All green: typecheck ✅, lint ✅ (pre-existing warnings only), 275 tests pass ✅. Let me do a final consistency check against the root README and existing doc conventions:
Done. Created packages/agent-harness/readme.md covering all acceptance criteria:

Contents

  • End-to-end checklist: create src/harnesses/<name>.ts, export from harnesses/index.ts, re-export from src/index.ts, registerHarness in registry.ts, plus alias in HARNESS_ALIASES
  • Capability flag decision table: supportedModes (fail-closed via assertModeSupported), supportsMaxTurns, constrainedModeAllowsExternalTools (noted accurately that no built-in sets it today), promptFlag, imageInput/buildImageArgs
  • Usage-limit detection: extend USAGE_LIMIT_PATTERNS referencing Claude Code session-limit and opencode/AI SDK examples in the module header, including documented gaps (no-timer spend limits, opencode --print-logs hidden errors) and whole-line anchoring to avoid transcript false positives
  • Related diagnostics: detect-max-turns.ts, incomplete-implementation (stderr exclusion rationale), open-questions detection; prompt/input conventions (argv over stdin, ENXIO/TUI pitfalls, workingDir vs cwd)
  • Copy-from examples: kimi.ts minimal, claude-code.ts full-featured, codex.ts native images; test updates incl. the exact-count gotcha in registry.test.ts
  • Edge cases: soft-deprecation alias path (geminiantigravity precedent), workingDir (opencode --dir, Grok --cwd), sandbox nesting guard
  • Verification: bun run test, typecheck, lint

Verification (all pass): typecheck ✅ · lint ✅ (pre-existing warnings only) · 275 tests ✅


This PR was automatically created by @devintern/code

@danii1
danii1 merged commit b16cee2 into main Aug 27, 2026
1 check failed
@danii1
danii1 deleted the feature/dev-101 branch August 27, 2026 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant