Add support for R breakpoints

extensions/positron-python/src/client/positron-supervisor.d.ts

@@ -373,27 +373,25 @@ export type CommBackendMessage =
  * Disposing the `DapComm` automatically disposes of the nested `Comm`.
  */
 export interface DapComm {
-    /** The `targetName` passed to the constructor. */
+    /** The `targetName` passed to `create()`. */
     readonly targetName: string;
 
-    /** The `debugType` passed to the constructor. */
+    /** The `debugType` passed to `create()`. */
     readonly debugType: string;
 
-    /** The `debugName` passed to the constructor. */
+    /** The `debugName` passed to `create()`. */
     readonly debugName: string;
 
     /**
      * The comm for the DAP.
      * Use it to receive messages or make notifications and requests.
-     * Defined after `createServerComm()` has been called.
      */
-    readonly comm?: Comm;
+    readonly comm: Comm;
 
     /**
      * The port on which the DAP server is listening.
-     * Defined after `createServerComm()` has been called.
      */
-    readonly serverPort?: number;
+    readonly port: number;
 
     /**
      * Handle a message received via `this.comm.receiver`.
@@ -416,6 +414,12 @@ export interface DapComm {
      */
     handleMessage(msg: any): Promise<boolean>;
 
+    /** Connect to the DAP server. */
+    connect(): Promise<void>;
+
+    /** Disconnect from the DAP server. */
+    disconnect(): Promise<void>;
+
     /**
      * Dispose of the underlying comm.
      * Must be called if the DAP comm is no longer in use.

extensions/positron-r/package.json

@@ -700,12 +700,19 @@
         "when": "isRPackage"
       }
     ],
+    "breakpoints": [
+      {
+        "language": "r"
+      }
+    ],
     "debuggers": [
       {
         "type": "ark",
         "label": "R Debugger",
         "languages": ["r"],
-        "supportsUiLaunch": false
+        "supportsUiLaunch": false,
+        "sendBreakpointsOnAllSaves": true,
+        "verifyBreakpointsInDirtyDocuments": true
       }
     ],
     "notebookRenderer": [
@@ -955,7 +962,7 @@
   },
   "positron": {
     "binaryDependencies": {
-      "ark": "0.1.223"
+      "ark": "0.1.224"
     },
     "minimumRVersion": "4.2.0",
     "minimumRenvVersion": "1.0.9"

extensions/positron-r/package.nls.json

@@ -86,6 +86,6 @@
 	"r.walkthrough.migrateFromRStudio.workspaces.description": "  \n[Open an existing folder](command:workbench.action.files.openFolder)",
 	"r.walkthrough.migrateFromRStudio.formatting.title": "Formatting your R code",
 	"r.walkthrough.migrateFromRStudio.formatting.description": "  \n[Configure Air to format on save](command:r.walkthrough.formatOnSave)",
-  "r.welcome.views.debugger.content": "Positron currently provides limited debugging support for R code, but you can use R's native debugging features in Positron.\n[Learn more](https://positron.posit.co/guide-r.html#debugging)"
+	"r.welcome.views.debugger.content": "Positron provides first-class debugging support for R\n[Learn more](https://positron.posit.co/guide-r.html#debugging)"
 
 }

extensions/positron-r/src/positron-supervisor.d.ts

@@ -381,27 +381,25 @@ export type CommBackendMessage =
  * Disposing the `DapComm` automatically disposes of the nested `Comm`.
  */
 export interface DapComm {
-	/** The `targetName` passed to the constructor. */
+	/** The `targetName` passed to `create()`. */
 	readonly targetName: string;
 
-	/** The `debugType` passed to the constructor. */
+	/** The `debugType` passed to `create()`. */
 	readonly debugType: string;
 
-	/** The `debugName` passed to the constructor. */
+	/** The `debugName` passed to `create()`. */
 	readonly debugName: string;
 
 	/**
 	 * The comm for the DAP.
 	 * Use it to receive messages or make notifications and requests.
-	 * Defined after `createServerComm()` has been called.
 	 */
-	readonly comm?: Comm;
+	readonly comm: Comm;
 
 	/**
 	 * The port on which the DAP server is listening.
-	 * Defined after `createServerComm()` has been called.
 	 */
-	readonly serverPort?: number;
+	readonly port: number;
 
 	/**
 	 * Handle a message received via `this.comm.receiver`.
@@ -424,6 +422,12 @@ export interface DapComm {
 	 */
 	handleMessage(msg: any): Promise<boolean>;
 
+	/** Connect to the DAP server. */
+	connect(): Promise<void>;
+
+	/** Disconnect from the DAP server. */
+	disconnect(): Promise<void>;
+
 	/**
 	 * Dispose of the underlying comm.
 	 * Must be called if the DAP comm is no longer in use.

extensions/positron-r/src/session-manager.ts

@@ -165,11 +165,11 @@ export class RSessionManager implements vscode.Disposable {
 	 * the safer `activateConsoleSession()`.
 	 */
 	private async activateSession(session: RSession, reason: string): Promise<void> {
-		await session.activateLsp(reason);
+		await session.activateServices(reason);
 	}
 
 	private async deactivateSession(session: RSession, reason: string): Promise<void> {
-		await session.deactivateLsp(reason);
+		await session.deactivateServices(reason);
 	}
 
 	/**