Skip to content

Rename agent-driver.ts to target-driver.ts (#1735) - #1741

Merged
suleimansh merged 1 commit into
mainfrom
1735-target-driver
Aug 28, 2026
Merged

Rename agent-driver.ts to target-driver.ts (#1735)#1741
suleimansh merged 1 commit into
mainfrom
1735-target-driver

Conversation

@suleimansh

Copy link
Copy Markdown
Contributor

Closes #1735.

agent-driver.ts shared its name with the agent-driver package it imports. It is the run-target axis (--run-on), so it is now target-driver.ts, next to driver-cli.ts which owns the CLI axis.


🤖 curated · Fable 5, effort high

  • git mv of the six files (.ts, .test.ts, .SPEC.md, .BUG-ANALYSIS.md, .test.SPEC.md, .test.BUG-ANALYSIS.md).
  • createAgentDrivercreateTargetDriver, CreateAgentDriverOptionsCreateTargetDriverOptions; the one import in cli.ts.
  • The doc comment said "on top of the agent axis" and "falls through to the local agent driver" — "agent" there is the pre-D5 word for the CLI choice, the ambiguity Rename agent-driver.ts, it collides with the package #1735 is about. Now "the CLI axis" and "the driver for the chosen CLI" (same in the test's header).
  • SPEC files: renamed only; they never named the file.

No behaviour change. Proof: framework typecheck, 1471 node tests + 864 dashboard tests green.

The module turns the run target (--run-on) into the Driver that runs an
agent's turns; it shared its name with the `agent-driver` package it
imports. `createAgentDriver` is `createTargetDriver`. No behaviour change.

Claude-Session: https://claude.ai/code/session_013KV2g51oWteQWwhuTTwiZX

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
@suleimansh
suleimansh merged commit 6b0268a into main Aug 28, 2026
2 checks passed
@suleimansh
suleimansh deleted the 1735-target-driver branch August 28, 2026 20:24
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.

Rename agent-driver.ts, it collides with the package

1 participant