Skip to content

Conversation

@sestinj
Copy link
Contributor

@sestinj sestinj commented Sep 5, 2025

Description

so we can use custom agents when starting them remotely


Summary by cubic

Adds a --config flag to cn remote to launch remote sessions with a custom agent/config. The request now includes agent and config fields, and the PR review workflow uses this flag.

  • New Features
    • Accepts --config on cn remote; forwards value to API as agent and config.
    • Applies common CLI options to remote via addCommonOptions.
    • Adds tests for config handling and updates the GitHub workflow to pass a default config.

@sestinj sestinj requested a review from a team as a code owner September 5, 2025 03:27
@sestinj sestinj requested review from tomasz-stefaniak and removed request for a team September 5, 2025 03:27
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Sep 5, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 4 files

React with 👍 or 👎 to teach cubic. You can also tag @cubic-dev-ai to give feedback, ask questions, or re-run the review.

idempotencyKey: testIdempotencyKey,
branchName: testBranch,
agent: undefined,
config: undefined,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Expecting config to be undefined in parsed JSON is incorrect; JSON.stringify omits undefined properties so the key will be absent. Update the test to not expect this property when config is not provided.

Prompt for AI agents
Address the following comment on extensions/cli/src/commands/remote.test.ts at line 240:

<comment>Expecting config to be undefined in parsed JSON is incorrect; JSON.stringify omits undefined properties so the key will be absent. Update the test to not expect this property when config is not provided.</comment>

<file context>
@@ -236,6 +236,71 @@ describe(&quot;remote command&quot;, () =&gt; {
       idempotencyKey: testIdempotencyKey,
       branchName: testBranch,
+      agent: undefined,
+      config: undefined,
+    });
+  });
</file context>

@sestinj sestinj merged commit 7b89364 into main Sep 10, 2025
66 of 77 checks passed
@sestinj sestinj deleted the nate/cn-remote branch September 10, 2025 21:33
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PRs Sep 10, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Sep 10, 2025
@github-actions github-actions bot added the tier 2 Important feature that adds new capabilities to the platform or improves critical user journeys label Sep 10, 2025
@sestinj
Copy link
Contributor Author

sestinj commented Sep 11, 2025

🎉 This PR is included in version 1.13.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

released size:M This PR changes 30-99 lines, ignoring generated files. tier 2 Important feature that adds new capabilities to the platform or improves critical user journeys

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants