fix(persona-kit): repair opencode one-shot invocation by khaliqgant · Pull Request #270 · AgentWorkforce/workforce

khaliqgant · 2026-07-15T12:13:02Z

Closes#269

Summary

stop passing unsupported --dir to opencode run; the runtime already spawns the child with the requested cwd
stop overriding the generated persona agent model with the CLI --model flag
normalize bare OpenCode persona model IDs to opencode/<model> in the generated opencode.json agent config while preserving already-qualified values
add regression coverage using a truthy working directory and bare deepseek-v4-flash-free model

Why

The non-interactive path was adding two stale CLI arguments after buildInteractiveSpec had already moved OpenCode model selection into opencode.json. Current OpenCode rejects --dir, and its --model parser requires provider/model values. The child process already receives cwd through spawnAndCapture, while the generated agent config carries the persona model selected by --agent. Since OpenCode agent configs also require provider/model-id, bare model IDs are normalized to the harness provider before serialization.

Validation

pnpm --filter @agentworkforce/persona-kit test (287 passed)
pnpm --filter @agentworkforce/persona-kit lint
pnpm --filter @agentworkforce/persona-kit typecheck

Downstream note

The daytona-monitor, cloudflare-monitor, gcp-watcher, and neon-monitor personas in AgentWorkforce/agents use the OpenCode harness with bare model strings. Before this fix, they would be exposed to the same invalid provider/model resolution on affected OpenCode versions; this PR does not modify that repository.

gemini-code-assist · 2026-07-15T12:13:05Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

cursor · 2026-07-15T12:13:07Z

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

coderabbitai · 2026-07-15T12:13:23Z

📝 Walkthrough

Walkthrough

The opencode non-interactive command now selects models through generated opencode.json agent configuration, omits unsupported --dir and redundant --model arguments, and uses default output formatting. A test verifies the resulting command and configuration.

Changes

Opencode harness execution

Layer / File(s)	Summary
Update opencode invocation arguments `packages/persona-kit/src/interactive-spec.ts`	The opencode command no longer passes `--model` or `--dir`; it uses generated agent configuration and includes `--format default`.
Verify generated opencode specification `packages/persona-kit/src/interactive-spec.test.ts`	Tests verify the binary, arguments, omitted flags, and model recorded in `opencode.json`.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers:willwashburn, claude

Poem

I’m a rabbit with a config to share,
No dangling flags in the command air.
The model now rests where agents belong,
Default format makes output strong.
Hop, hop—tests confirm the run!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes remove the invalid `--dir` and `--model` usage and preserve model selection in config as required by `#269`.
Out of Scope Changes check	✅ Passed	The PR stays focused on the OpenCode invocation fix and its regression coverage.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly summarizes the main fix: repairing the OpenCode one-shot invocation in persona-kit.
Description check	✅ Passed	The description matches the changeset and explains the OpenCode invocation fix and added regression coverage.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch fix/opencode-noninteractive-cli-269

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

cubic-dev-ai

No issues found across 2 files

_{Re-trigger cubic}

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:35cad039da

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

khaliqgant · 2026-07-15T12:13:02Z

Closes#269

Summary

stop passing unsupported --dir to opencode run; the runtime already spawns the child with the requested cwd
stop overriding the generated persona agent model with the CLI --model flag
normalize bare OpenCode persona model IDs to opencode/<model> in the generated opencode.json agent config while preserving already-qualified values
add regression coverage using a truthy working directory and bare deepseek-v4-flash-free model

Why

The non-interactive path was adding two stale CLI arguments after buildInteractiveSpec had already moved OpenCode model selection into opencode.json. Current OpenCode rejects --dir, and its --model parser requires provider/model values. The child process already receives cwd through spawnAndCapture, while the generated agent config carries the persona model selected by --agent. Since OpenCode agent configs also require provider/model-id, bare model IDs are normalized to the harness provider before serialization.

Validation

pnpm --filter @agentworkforce/persona-kit test (287 passed)
pnpm --filter @agentworkforce/persona-kit lint
pnpm --filter @agentworkforce/persona-kit typecheck

Downstream note

The daytona-monitor, cloudflare-monitor, gcp-watcher, and neon-monitor personas in AgentWorkforce/agents use the OpenCode harness with bare model strings. Before this fix, they would be exposed to the same invalid provider/model resolution on affected OpenCode versions; this PR does not modify that repository.

gemini-code-assist · 2026-07-15T12:13:05Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

cursor · 2026-07-15T12:13:07Z

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

coderabbitai · 2026-07-15T12:13:23Z

📝 Walkthrough

Walkthrough

The opencode non-interactive command now selects models through generated opencode.json agent configuration, omits unsupported --dir and redundant --model arguments, and uses default output formatting. A test verifies the resulting command and configuration.

Changes

Opencode harness execution

Layer / File(s)	Summary
Update opencode invocation arguments `packages/persona-kit/src/interactive-spec.ts`	The opencode command no longer passes `--model` or `--dir`; it uses generated agent configuration and includes `--format default`.
Verify generated opencode specification `packages/persona-kit/src/interactive-spec.test.ts`	Tests verify the binary, arguments, omitted flags, and model recorded in `opencode.json`.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers:willwashburn, claude

Poem

I’m a rabbit with a config to share,
No dangling flags in the command air.
The model now rests where agents belong,
Default format makes output strong.
Hop, hop—tests confirm the run!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes remove the invalid `--dir` and `--model` usage and preserve model selection in config as required by `#269`.
Out of Scope Changes check	✅ Passed	The PR stays focused on the OpenCode invocation fix and its regression coverage.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly summarizes the main fix: repairing the OpenCode one-shot invocation in persona-kit.
Description check	✅ Passed	The description matches the changeset and explains the OpenCode invocation fix and added regression coverage.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch fix/opencode-noninteractive-cli-269

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

cubic-dev-ai

No issues found across 2 files

_{Re-trigger cubic}

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:35cad039da

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

khaliqgant · 2026-07-15T12:13:02Z

Closes#269

Summary

stop passing unsupported --dir to opencode run; the runtime already spawns the child with the requested cwd
stop overriding the generated persona agent model with the CLI --model flag
normalize bare OpenCode persona model IDs to opencode/<model> in the generated opencode.json agent config while preserving already-qualified values
add regression coverage using a truthy working directory and bare deepseek-v4-flash-free model

Why

The non-interactive path was adding two stale CLI arguments after buildInteractiveSpec had already moved OpenCode model selection into opencode.json. Current OpenCode rejects --dir, and its --model parser requires provider/model values. The child process already receives cwd through spawnAndCapture, while the generated agent config carries the persona model selected by --agent. Since OpenCode agent configs also require provider/model-id, bare model IDs are normalized to the harness provider before serialization.

Validation

pnpm --filter @agentworkforce/persona-kit test (287 passed)
pnpm --filter @agentworkforce/persona-kit lint
pnpm --filter @agentworkforce/persona-kit typecheck

Downstream note

The daytona-monitor, cloudflare-monitor, gcp-watcher, and neon-monitor personas in AgentWorkforce/agents use the OpenCode harness with bare model strings. Before this fix, they would be exposed to the same invalid provider/model resolution on affected OpenCode versions; this PR does not modify that repository.

gemini-code-assist · 2026-07-15T12:13:05Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

cursor · 2026-07-15T12:13:07Z

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

coderabbitai · 2026-07-15T12:13:23Z

📝 Walkthrough

Walkthrough

The opencode non-interactive command now selects models through generated opencode.json agent configuration, omits unsupported --dir and redundant --model arguments, and uses default output formatting. A test verifies the resulting command and configuration.

Changes

Opencode harness execution

Layer / File(s)	Summary
Update opencode invocation arguments `packages/persona-kit/src/interactive-spec.ts`	The opencode command no longer passes `--model` or `--dir`; it uses generated agent configuration and includes `--format default`.
Verify generated opencode specification `packages/persona-kit/src/interactive-spec.test.ts`	Tests verify the binary, arguments, omitted flags, and model recorded in `opencode.json`.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers:willwashburn, claude

Poem

I’m a rabbit with a config to share,
No dangling flags in the command air.
The model now rests where agents belong,
Default format makes output strong.
Hop, hop—tests confirm the run!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes remove the invalid `--dir` and `--model` usage and preserve model selection in config as required by `#269`.
Out of Scope Changes check	✅ Passed	The PR stays focused on the OpenCode invocation fix and its regression coverage.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly summarizes the main fix: repairing the OpenCode one-shot invocation in persona-kit.
Description check	✅ Passed	The description matches the changeset and explains the OpenCode invocation fix and added regression coverage.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch fix/opencode-noninteractive-cli-269

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

cubic-dev-ai

No issues found across 2 files

_{Re-trigger cubic}

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:35cad039da

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

khaliqgant · 2026-07-15T12:13:02Z

Closes#269

Summary

stop passing unsupported --dir to opencode run; the runtime already spawns the child with the requested cwd
stop overriding the generated persona agent model with the CLI --model flag
normalize bare OpenCode persona model IDs to opencode/<model> in the generated opencode.json agent config while preserving already-qualified values
add regression coverage using a truthy working directory and bare deepseek-v4-flash-free model

Why

The non-interactive path was adding two stale CLI arguments after buildInteractiveSpec had already moved OpenCode model selection into opencode.json. Current OpenCode rejects --dir, and its --model parser requires provider/model values. The child process already receives cwd through spawnAndCapture, while the generated agent config carries the persona model selected by --agent. Since OpenCode agent configs also require provider/model-id, bare model IDs are normalized to the harness provider before serialization.

Validation

pnpm --filter @agentworkforce/persona-kit test (287 passed)
pnpm --filter @agentworkforce/persona-kit lint
pnpm --filter @agentworkforce/persona-kit typecheck

Downstream note

The daytona-monitor, cloudflare-monitor, gcp-watcher, and neon-monitor personas in AgentWorkforce/agents use the OpenCode harness with bare model strings. Before this fix, they would be exposed to the same invalid provider/model resolution on affected OpenCode versions; this PR does not modify that repository.

gemini-code-assist · 2026-07-15T12:13:05Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

cursor · 2026-07-15T12:13:07Z

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

coderabbitai · 2026-07-15T12:13:23Z

📝 Walkthrough

Walkthrough

The opencode non-interactive command now selects models through generated opencode.json agent configuration, omits unsupported --dir and redundant --model arguments, and uses default output formatting. A test verifies the resulting command and configuration.

Changes

Opencode harness execution

Layer / File(s)	Summary
Update opencode invocation arguments `packages/persona-kit/src/interactive-spec.ts`	The opencode command no longer passes `--model` or `--dir`; it uses generated agent configuration and includes `--format default`.
Verify generated opencode specification `packages/persona-kit/src/interactive-spec.test.ts`	Tests verify the binary, arguments, omitted flags, and model recorded in `opencode.json`.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers:willwashburn, claude

Poem

I’m a rabbit with a config to share,
No dangling flags in the command air.
The model now rests where agents belong,
Default format makes output strong.
Hop, hop—tests confirm the run!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes remove the invalid `--dir` and `--model` usage and preserve model selection in config as required by `#269`.
Out of Scope Changes check	✅ Passed	The PR stays focused on the OpenCode invocation fix and its regression coverage.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly summarizes the main fix: repairing the OpenCode one-shot invocation in persona-kit.
Description check	✅ Passed	The description matches the changeset and explains the OpenCode invocation fix and added regression coverage.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch fix/opencode-noninteractive-cli-269

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

cubic-dev-ai

No issues found across 2 files

_{Re-trigger cubic}

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:35cad039da

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

khaliqgant · 2026-07-15T12:13:02Z

Closes#269

Summary

stop passing unsupported --dir to opencode run; the runtime already spawns the child with the requested cwd
stop overriding the generated persona agent model with the CLI --model flag
normalize bare OpenCode persona model IDs to opencode/<model> in the generated opencode.json agent config while preserving already-qualified values
add regression coverage using a truthy working directory and bare deepseek-v4-flash-free model

Why

The non-interactive path was adding two stale CLI arguments after buildInteractiveSpec had already moved OpenCode model selection into opencode.json. Current OpenCode rejects --dir, and its --model parser requires provider/model values. The child process already receives cwd through spawnAndCapture, while the generated agent config carries the persona model selected by --agent. Since OpenCode agent configs also require provider/model-id, bare model IDs are normalized to the harness provider before serialization.

Validation

pnpm --filter @agentworkforce/persona-kit test (287 passed)
pnpm --filter @agentworkforce/persona-kit lint
pnpm --filter @agentworkforce/persona-kit typecheck

Downstream note

The daytona-monitor, cloudflare-monitor, gcp-watcher, and neon-monitor personas in AgentWorkforce/agents use the OpenCode harness with bare model strings. Before this fix, they would be exposed to the same invalid provider/model resolution on affected OpenCode versions; this PR does not modify that repository.

gemini-code-assist · 2026-07-15T12:13:05Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

cursor · 2026-07-15T12:13:07Z

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

coderabbitai · 2026-07-15T12:13:23Z

📝 Walkthrough

Walkthrough

The opencode non-interactive command now selects models through generated opencode.json agent configuration, omits unsupported --dir and redundant --model arguments, and uses default output formatting. A test verifies the resulting command and configuration.

Changes

Opencode harness execution

Layer / File(s)	Summary
Update opencode invocation arguments `packages/persona-kit/src/interactive-spec.ts`	The opencode command no longer passes `--model` or `--dir`; it uses generated agent configuration and includes `--format default`.
Verify generated opencode specification `packages/persona-kit/src/interactive-spec.test.ts`	Tests verify the binary, arguments, omitted flags, and model recorded in `opencode.json`.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers:willwashburn, claude

Poem

I’m a rabbit with a config to share,
No dangling flags in the command air.
The model now rests where agents belong,
Default format makes output strong.
Hop, hop—tests confirm the run!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes remove the invalid `--dir` and `--model` usage and preserve model selection in config as required by `#269`.
Out of Scope Changes check	✅ Passed	The PR stays focused on the OpenCode invocation fix and its regression coverage.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly summarizes the main fix: repairing the OpenCode one-shot invocation in persona-kit.
Description check	✅ Passed	The description matches the changeset and explains the OpenCode invocation fix and added regression coverage.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch fix/opencode-noninteractive-cli-269

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

cubic-dev-ai

No issues found across 2 files

_{Re-trigger cubic}

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:35cad039da

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

khaliqgant · 2026-07-15T12:13:02Z

Closes#269

Summary

stop passing unsupported --dir to opencode run; the runtime already spawns the child with the requested cwd
stop overriding the generated persona agent model with the CLI --model flag
normalize bare OpenCode persona model IDs to opencode/<model> in the generated opencode.json agent config while preserving already-qualified values
add regression coverage using a truthy working directory and bare deepseek-v4-flash-free model

Why

The non-interactive path was adding two stale CLI arguments after buildInteractiveSpec had already moved OpenCode model selection into opencode.json. Current OpenCode rejects --dir, and its --model parser requires provider/model values. The child process already receives cwd through spawnAndCapture, while the generated agent config carries the persona model selected by --agent. Since OpenCode agent configs also require provider/model-id, bare model IDs are normalized to the harness provider before serialization.

Validation

pnpm --filter @agentworkforce/persona-kit test (287 passed)
pnpm --filter @agentworkforce/persona-kit lint
pnpm --filter @agentworkforce/persona-kit typecheck

Downstream note

The daytona-monitor, cloudflare-monitor, gcp-watcher, and neon-monitor personas in AgentWorkforce/agents use the OpenCode harness with bare model strings. Before this fix, they would be exposed to the same invalid provider/model resolution on affected OpenCode versions; this PR does not modify that repository.

gemini-code-assist · 2026-07-15T12:13:05Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

cursor · 2026-07-15T12:13:07Z

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

coderabbitai · 2026-07-15T12:13:23Z

📝 Walkthrough

Walkthrough

The opencode non-interactive command now selects models through generated opencode.json agent configuration, omits unsupported --dir and redundant --model arguments, and uses default output formatting. A test verifies the resulting command and configuration.

Changes

Opencode harness execution

Layer / File(s)	Summary
Update opencode invocation arguments `packages/persona-kit/src/interactive-spec.ts`	The opencode command no longer passes `--model` or `--dir`; it uses generated agent configuration and includes `--format default`.
Verify generated opencode specification `packages/persona-kit/src/interactive-spec.test.ts`	Tests verify the binary, arguments, omitted flags, and model recorded in `opencode.json`.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers:willwashburn, claude

Poem

I’m a rabbit with a config to share,
No dangling flags in the command air.
The model now rests where agents belong,
Default format makes output strong.
Hop, hop—tests confirm the run!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes remove the invalid `--dir` and `--model` usage and preserve model selection in config as required by `#269`.
Out of Scope Changes check	✅ Passed	The PR stays focused on the OpenCode invocation fix and its regression coverage.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly summarizes the main fix: repairing the OpenCode one-shot invocation in persona-kit.
Description check	✅ Passed	The description matches the changeset and explains the OpenCode invocation fix and added regression coverage.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch fix/opencode-noninteractive-cli-269

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

cubic-dev-ai

No issues found across 2 files

_{Re-trigger cubic}

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:35cad039da

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

khaliqgant · 2026-07-15T12:13:02Z

Closes#269

Summary

stop passing unsupported --dir to opencode run; the runtime already spawns the child with the requested cwd
stop overriding the generated persona agent model with the CLI --model flag
normalize bare OpenCode persona model IDs to opencode/<model> in the generated opencode.json agent config while preserving already-qualified values
add regression coverage using a truthy working directory and bare deepseek-v4-flash-free model

Why

The non-interactive path was adding two stale CLI arguments after buildInteractiveSpec had already moved OpenCode model selection into opencode.json. Current OpenCode rejects --dir, and its --model parser requires provider/model values. The child process already receives cwd through spawnAndCapture, while the generated agent config carries the persona model selected by --agent. Since OpenCode agent configs also require provider/model-id, bare model IDs are normalized to the harness provider before serialization.

Validation

pnpm --filter @agentworkforce/persona-kit test (287 passed)
pnpm --filter @agentworkforce/persona-kit lint
pnpm --filter @agentworkforce/persona-kit typecheck

Downstream note

The daytona-monitor, cloudflare-monitor, gcp-watcher, and neon-monitor personas in AgentWorkforce/agents use the OpenCode harness with bare model strings. Before this fix, they would be exposed to the same invalid provider/model resolution on affected OpenCode versions; this PR does not modify that repository.

gemini-code-assist · 2026-07-15T12:13:05Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

cursor · 2026-07-15T12:13:07Z

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

coderabbitai · 2026-07-15T12:13:23Z

📝 Walkthrough

Walkthrough

The opencode non-interactive command now selects models through generated opencode.json agent configuration, omits unsupported --dir and redundant --model arguments, and uses default output formatting. A test verifies the resulting command and configuration.

Changes

Opencode harness execution

Layer / File(s)	Summary
Update opencode invocation arguments `packages/persona-kit/src/interactive-spec.ts`	The opencode command no longer passes `--model` or `--dir`; it uses generated agent configuration and includes `--format default`.
Verify generated opencode specification `packages/persona-kit/src/interactive-spec.test.ts`	Tests verify the binary, arguments, omitted flags, and model recorded in `opencode.json`.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers:willwashburn, claude

Poem

I’m a rabbit with a config to share,
No dangling flags in the command air.
The model now rests where agents belong,
Default format makes output strong.
Hop, hop—tests confirm the run!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes remove the invalid `--dir` and `--model` usage and preserve model selection in config as required by `#269`.
Out of Scope Changes check	✅ Passed	The PR stays focused on the OpenCode invocation fix and its regression coverage.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly summarizes the main fix: repairing the OpenCode one-shot invocation in persona-kit.
Description check	✅ Passed	The description matches the changeset and explains the OpenCode invocation fix and added regression coverage.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch fix/opencode-noninteractive-cli-269

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

cubic-dev-ai

No issues found across 2 files

_{Re-trigger cubic}

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:35cad039da

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

khaliqgant · 2026-07-15T12:13:02Z

Closes#269

Summary

stop passing unsupported --dir to opencode run; the runtime already spawns the child with the requested cwd
stop overriding the generated persona agent model with the CLI --model flag
normalize bare OpenCode persona model IDs to opencode/<model> in the generated opencode.json agent config while preserving already-qualified values
add regression coverage using a truthy working directory and bare deepseek-v4-flash-free model

Why

The non-interactive path was adding two stale CLI arguments after buildInteractiveSpec had already moved OpenCode model selection into opencode.json. Current OpenCode rejects --dir, and its --model parser requires provider/model values. The child process already receives cwd through spawnAndCapture, while the generated agent config carries the persona model selected by --agent. Since OpenCode agent configs also require provider/model-id, bare model IDs are normalized to the harness provider before serialization.

Validation

pnpm --filter @agentworkforce/persona-kit test (287 passed)
pnpm --filter @agentworkforce/persona-kit lint
pnpm --filter @agentworkforce/persona-kit typecheck

Downstream note

The daytona-monitor, cloudflare-monitor, gcp-watcher, and neon-monitor personas in AgentWorkforce/agents use the OpenCode harness with bare model strings. Before this fix, they would be exposed to the same invalid provider/model resolution on affected OpenCode versions; this PR does not modify that repository.

gemini-code-assist · 2026-07-15T12:13:05Z

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

cursor · 2026-07-15T12:13:07Z

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

coderabbitai · 2026-07-15T12:13:23Z

📝 Walkthrough

Walkthrough

The opencode non-interactive command now selects models through generated opencode.json agent configuration, omits unsupported --dir and redundant --model arguments, and uses default output formatting. A test verifies the resulting command and configuration.

Changes

Opencode harness execution

Layer / File(s)	Summary
Update opencode invocation arguments `packages/persona-kit/src/interactive-spec.ts`	The opencode command no longer passes `--model` or `--dir`; it uses generated agent configuration and includes `--format default`.
Verify generated opencode specification `packages/persona-kit/src/interactive-spec.test.ts`	Tests verify the binary, arguments, omitted flags, and model recorded in `opencode.json`.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers:willwashburn, claude

Poem

I’m a rabbit with a config to share,
No dangling flags in the command air.
The model now rests where agents belong,
Default format makes output strong.
Hop, hop—tests confirm the run!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes remove the invalid `--dir` and `--model` usage and preserve model selection in config as required by `#269`.
Out of Scope Changes check	✅ Passed	The PR stays focused on the OpenCode invocation fix and its regression coverage.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly summarizes the main fix: repairing the OpenCode one-shot invocation in persona-kit.
Description check	✅ Passed	The description matches the changeset and explains the OpenCode invocation fix and added regression coverage.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch fix/opencode-noninteractive-cli-269

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

cubic-dev-ai

No issues found across 2 files

_{Re-trigger cubic}

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:35cad039da

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

fix(persona-kit): repair opencode one-shot invocation
35cad03

cubic-dev-aiBot reviewed Jul 15, 2026
View reviewed changes

chatgpt-codex-connectorBot reviewed Jul 15, 2026
View reviewed changes

Comment threadpackages/persona-kit/src/interactive-spec.ts

fix(persona-kit): normalize bare opencode agent models
f6eefef

khaliqgant merged commit af47512 into mainJul 15, 2026
3 checks passed

khaliqgant deleted the fix/opencode-noninteractive-cli-269 branch July 15, 2026 12:27

coderabbitaiBot mentioned this pull request Jul 16, 2026
fix: deliver harness prompts off argv #274
Merged

fix(persona-kit): repair opencode one-shot invocation
35cad03

cubic-dev-aiBot reviewed Jul 15, 2026
View reviewed changes

chatgpt-codex-connectorBot reviewed Jul 15, 2026
View reviewed changes

Comment threadpackages/persona-kit/src/interactive-spec.ts

fix(persona-kit): normalize bare opencode agent models
f6eefef

khaliqgant merged commit af47512 into mainJul 15, 2026
3 checks passed

khaliqgant deleted the fix/opencode-noninteractive-cli-269 branch July 15, 2026 12:27

coderabbitaiBot mentioned this pull request Jul 16, 2026
fix: deliver harness prompts off argv #274
Merged

fix(persona-kit): repair opencode one-shot invocation
35cad03

cubic-dev-aiBot reviewed Jul 15, 2026
View reviewed changes

chatgpt-codex-connectorBot reviewed Jul 15, 2026
View reviewed changes

Comment threadpackages/persona-kit/src/interactive-spec.ts

fix(persona-kit): normalize bare opencode agent models
f6eefef

khaliqgant merged commit af47512 into mainJul 15, 2026
3 checks passed

khaliqgant deleted the fix/opencode-noninteractive-cli-269 branch July 15, 2026 12:27

coderabbitaiBot mentioned this pull request Jul 16, 2026
fix: deliver harness prompts off argv #274
Merged

fix(persona-kit): repair opencode one-shot invocation
35cad03

cubic-dev-aiBot reviewed Jul 15, 2026
View reviewed changes

chatgpt-codex-connectorBot reviewed Jul 15, 2026
View reviewed changes

Comment threadpackages/persona-kit/src/interactive-spec.ts

fix(persona-kit): normalize bare opencode agent models
f6eefef

khaliqgant merged commit af47512 into mainJul 15, 2026
3 checks passed

khaliqgant deleted the fix/opencode-noninteractive-cli-269 branch July 15, 2026 12:27

coderabbitaiBot mentioned this pull request Jul 16, 2026
fix: deliver harness prompts off argv #274
Merged

fix(persona-kit): repair opencode one-shot invocation
35cad03

cubic-dev-aiBot reviewed Jul 15, 2026
View reviewed changes

chatgpt-codex-connectorBot reviewed Jul 15, 2026
View reviewed changes

Comment threadpackages/persona-kit/src/interactive-spec.ts

fix(persona-kit): normalize bare opencode agent models
f6eefef

khaliqgant merged commit af47512 into mainJul 15, 2026
3 checks passed

khaliqgant deleted the fix/opencode-noninteractive-cli-269 branch July 15, 2026 12:27

coderabbitaiBot mentioned this pull request Jul 16, 2026
fix: deliver harness prompts off argv #274
Merged

fix(persona-kit): repair opencode one-shot invocation
35cad03

cubic-dev-aiBot reviewed Jul 15, 2026
View reviewed changes

chatgpt-codex-connectorBot reviewed Jul 15, 2026
View reviewed changes

Comment threadpackages/persona-kit/src/interactive-spec.ts

fix(persona-kit): normalize bare opencode agent models
f6eefef

khaliqgant merged commit af47512 into mainJul 15, 2026
3 checks passed

khaliqgant deleted the fix/opencode-noninteractive-cli-269 branch July 15, 2026 12:27

coderabbitaiBot mentioned this pull request Jul 16, 2026
fix: deliver harness prompts off argv #274
Merged

fix(persona-kit): repair opencode one-shot invocation
35cad03

cubic-dev-aiBot reviewed Jul 15, 2026
View reviewed changes

chatgpt-codex-connectorBot reviewed Jul 15, 2026
View reviewed changes

Comment threadpackages/persona-kit/src/interactive-spec.ts

fix(persona-kit): normalize bare opencode agent models
f6eefef

khaliqgant merged commit af47512 into mainJul 15, 2026
3 checks passed

khaliqgant deleted the fix/opencode-noninteractive-cli-269 branch July 15, 2026 12:27

coderabbitaiBot mentioned this pull request Jul 16, 2026
fix: deliver harness prompts off argv #274
Merged

fix(persona-kit): repair opencode one-shot invocation
35cad03

cubic-dev-aiBot reviewed Jul 15, 2026
View reviewed changes

chatgpt-codex-connectorBot reviewed Jul 15, 2026
View reviewed changes

Comment threadpackages/persona-kit/src/interactive-spec.ts

fix(persona-kit): normalize bare opencode agent models
f6eefef

khaliqgant merged commit af47512 into mainJul 15, 2026
3 checks passed

khaliqgant deleted the fix/opencode-noninteractive-cli-269 branch July 15, 2026 12:27

coderabbitaiBot mentioned this pull request Jul 16, 2026
fix: deliver harness prompts off argv #274
Merged

Uh oh!

Conversation

khaliqgant commented Jul 15, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Why

Validation

Downstream note

Uh oh!

gemini-code-assistBot commented Jul 15, 2026

Uh oh!

cursorBot commented Jul 15, 2026

Uh oh!

coderabbitaiBot commented Jul 15, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Uh oh!

cubic-dev-aiBot left a comment

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connectorBot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

khaliqgant commented Jul 15, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Why

Validation

Downstream note

Uh oh!

gemini-code-assistBot commented Jul 15, 2026

Uh oh!

cursorBot commented Jul 15, 2026

Uh oh!

coderabbitaiBot commented Jul 15, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Uh oh!

cubic-dev-aiBot left a comment

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connectorBot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

khaliqgant commented Jul 15, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Why

Validation

Downstream note

Uh oh!

gemini-code-assistBot commented Jul 15, 2026

khaliqgant commented Jul 15, 2026•
edited
Loading

coderabbitaiBot commented Jul 15, 2026•
edited
Loading

khaliqgant commented Jul 15, 2026•
edited
Loading

coderabbitaiBot commented Jul 15, 2026•
edited
Loading

khaliqgant commented Jul 15, 2026•
edited
Loading

coderabbitaiBot commented Jul 15, 2026•
edited
Loading

khaliqgant commented Jul 15, 2026•
edited
Loading

coderabbitaiBot commented Jul 15, 2026•
edited
Loading