[DEV-101] Document how to add a new agent harness in the agent-harness package README - #92
Merged
Conversation
…the agent-harness package README
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.mdcovering all acceptance criteria:Contents
src/harnesses/<name>.ts, export fromharnesses/index.ts, re-export fromsrc/index.ts,registerHarnessinregistry.ts, plus alias inHARNESS_ALIASESsupportedModes(fail-closed viaassertModeSupported),supportsMaxTurns,constrainedModeAllowsExternalTools(noted accurately that no built-in sets it today),promptFlag,imageInput/buildImageArgsUSAGE_LIMIT_PATTERNSreferencing Claude Code session-limit and opencode/AI SDK examples in the module header, including documented gaps (no-timer spend limits, opencode--print-logshidden errors) and whole-line anchoring to avoid transcript false positivesdetect-max-turns.ts, incomplete-implementation (stderr exclusion rationale), open-questions detection; prompt/input conventions (argv over stdin, ENXIO/TUI pitfalls,workingDirvscwd)kimi.tsminimal,claude-code.tsfull-featured,codex.tsnative images; test updates incl. the exact-count gotcha inregistry.test.tsgemini→antigravityprecedent),workingDir(opencode --dir, Grok--cwd), sandbox nesting guardbun run test,typecheck,lintVerification (all pass): typecheck ✅ · lint ✅ (pre-existing warnings only) · 275 tests ✅
This PR was automatically created by @devintern/code