Skip to content

Conversation

@sarthakNITT
Copy link

@sarthakNITT sarthakNITT commented Jan 19, 2026

Description

  • Fixes the docs setup command in CONTRIBUTING.mdx.
  • Replaces npm run start with npm run dev to correctly run Mintlify docs locally (mintlify dev) on http://localhost:3000.

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Continue Tasks

Status Task Actions
▶️ Queued Update docs on PR View

Powered by Continue


Summary by cubic

Use npm run dev instead of npm run start to run Mintlify docs locally at http://localhost:3000. Also cleans up formatting and stabilizes a few CLI/UI tests; no functional changes.

Written for commit 32a1335. Summary will update on new commits.

@sarthakNITT sarthakNITT requested a review from a team as a code owner January 19, 2026 16:53
@sarthakNITT sarthakNITT requested review from RomneyDa and removed request for a team January 19, 2026 16:53
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jan 19, 2026
@github-actions
Copy link

github-actions bot commented Jan 19, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

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.

No issues found across 2 files

@sarthakNITT
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Jan 19, 2026
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 37 files (changes from recent commits).

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="extensions/vscode/src/diff/vertical/handler.ts">

<violation number="1" location="extensions/vscode/src/diff/vertical/handler.ts:377">
P2: Undo-stop options are ignored—options object is not passed to `editor.edit`, so undo stops aren’t disabled.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

await this.editor.edit((editBuilder) => {
editBuilder.replace(this.range, replaceContent),
{ undoStopAfter: false, undoStopBefore: false };
(editBuilder.replace(this.range, replaceContent),
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot Jan 19, 2026

Choose a reason for hiding this comment

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

P2: Undo-stop options are ignored—options object is not passed to editor.edit, so undo stops aren’t disabled.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At extensions/vscode/src/diff/vertical/handler.ts, line 377:

<comment>Undo-stop options are ignored—options object is not passed to `editor.edit`, so undo stops aren’t disabled.</comment>

<file context>
@@ -374,8 +374,8 @@ export class VerticalDiffHandler implements vscode.Disposable {
     await this.editor.edit((editBuilder) => {
-      editBuilder.replace(this.range, replaceContent),
-        { undoStopAfter: false, undoStopBefore: false };
+      (editBuilder.replace(this.range, replaceContent),
+        { undoStopAfter: false, undoStopBefore: false });
     });
</file context>
Fix with Cubic

Copy link
Collaborator

@RomneyDa RomneyDa left a comment

Choose a reason for hiding this comment

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

@sarthakNITT looks like there's a large diff here and the described changes to contributing.mdx are just a couple lines, also it's to contributing.md I'm gonna close this to be safe, could you open another PR with the proposed changes if needed?

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Jan 20, 2026
@RomneyDa RomneyDa closed this Jan 20, 2026
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Jan 20, 2026
@github-actions github-actions bot locked and limited conversation to collaborators Jan 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants