Merge branch 'main' into clean-outputs

Author: Meriem-BenIsmail

CHANGELOG.md

@@ -2,6 +2,31 @@
 
 <!-- <START NEW CHANGELOG ENTRY> -->
 
+## 0.51.3
+
+([Full Changelog](https://github.com/jupyterlab/jupyterlab-git/compare/v0.51.2...73bb0d02e48e8fa5794885be900809b588d2c122))
+
+### Enhancements made
+
+- Use `serverSettings` to make API requests [#1437](https://github.com/jupyterlab/jupyterlab-git/pull/1437) ([@jtpio](https://github.com/jtpio))
+
+### Bugs fixed
+
+- Fix scrolling issue in diff viewer [#1433](https://github.com/jupyterlab/jupyterlab-git/pull/1433) ([@Meriem-BenIsmail](https://github.com/Meriem-BenIsmail))
+- Fix cloning from new host via ssh by prompting for confirmation [#1408](https://github.com/jupyterlab/jupyterlab-git/pull/1408) ([@jesuino](https://github.com/jesuino))
+
+### Maintenance and upkeep improvements
+
+- Remove unused `handler.ts` file [#1436](https://github.com/jupyterlab/jupyterlab-git/pull/1436) ([@jtpio](https://github.com/jtpio))
+
+### Contributors to this release
+
+([GitHub contributors page for this release](https://github.com/jupyterlab/jupyterlab-git/graphs/contributors?from=2025-06-12&to=2025-12-10&type=c))
+
+[@github-actions](https://github.com/search?q=repo%3Ajupyterlab%2Fjupyterlab-git+involves%3Agithub-actions+updated%3A2025-06-12..2025-12-10&type=Issues) | [@jesuino](https://github.com/search?q=repo%3Ajupyterlab%2Fjupyterlab-git+involves%3Ajesuino+updated%3A2025-06-12..2025-12-10&type=Issues) | [@jtpio](https://github.com/search?q=repo%3Ajupyterlab%2Fjupyterlab-git+involves%3Ajtpio+updated%3A2025-06-12..2025-12-10&type=Issues) | [@Meriem-BenIsmail](https://github.com/search?q=repo%3Ajupyterlab%2Fjupyterlab-git+involves%3AMeriem-BenIsmail+updated%3A2025-06-12..2025-12-10&type=Issues)
+
+<!-- <END NEW CHANGELOG ENTRY> -->
+
 ## 0.51.2
 
 ([Full Changelog](https://github.com/jupyterlab/jupyterlab-git/compare/v0.51.1...bee4e6492440dd17280fb30478420f18e7f3c416))
@@ -26,8 +51,6 @@
 
 [@ctcjab](https://github.com/search?q=repo%3Ajupyterlab%2Fjupyterlab-git+involves%3Actcjab+updated%3A2025-04-03..2025-06-12&type=Issues) | [@dependabot](https://github.com/search?q=repo%3Ajupyterlab%2Fjupyterlab-git+involves%3Adependabot+updated%3A2025-04-03..2025-06-12&type=Issues) | [@github-actions](https://github.com/search?q=repo%3Ajupyterlab%2Fjupyterlab-git+involves%3Agithub-actions+updated%3A2025-04-03..2025-06-12&type=Issues) | [@gjmooney](https://github.com/search?q=repo%3Ajupyterlab%2Fjupyterlab-git+involves%3Agjmooney+updated%3A2025-04-03..2025-06-12&type=Issues) | [@jiridanek](https://github.com/search?q=repo%3Ajupyterlab%2Fjupyterlab-git+involves%3Ajiridanek+updated%3A2025-04-03..2025-06-12&type=Issues) | [@jtpio](https://github.com/search?q=repo%3Ajupyterlab%2Fjupyterlab-git+involves%3Ajtpio+updated%3A2025-04-03..2025-06-12&type=Issues)
 
-<!-- <END NEW CHANGELOG ENTRY> -->
-
 ## 0.51.1
 
 ([Full Changelog](https://github.com/jupyterlab/jupyterlab-git/compare/v0.51.0...3dcaf17a0db3161e7256961ac8eb2a8c6f1edaab))

package.json

@@ -1,6 +1,6 @@
 {
     "name": "@jupyterlab/git",
-    "version": "0.51.2",
+    "version": "0.51.3",
     "description": "A JupyterLab extension for version control using git",
     "main": "lib/index.js",
     "types": "lib/index.d.ts",

src/__tests__/plugin.spec.ts

@@ -6,6 +6,7 @@ import { ISettingRegistry, SettingRegistry } from '@jupyterlab/settingregistry';
 import { JupyterLab } from '@jupyterlab/application';
 import { showErrorMessage } from '@jupyterlab/apputils';
 import { URLExt } from '@jupyterlab/coreutils';
+import { ServerConnection } from '@jupyterlab/services';
 import { Signal } from '@lumino/signaling';
 import {
   defaultMockedResponses,
@@ -30,6 +31,10 @@ describe('plugin', () => {
 
   beforeAll(() => {
     app = new JupyterLab() as jest.Mocked<JupyterLab>;
+    // Add the serviceManager.serverSettings mock
+    (app as any).serviceManager = {
+      serverSettings: ServerConnection.makeSettings()
+    };
     browserFactory = {
       defaultBrowser: {
         model: { pathChanged: new Signal(null), restored: Promise.resolve() }

src/__tests__/utils.ts

@@ -1,3 +1,4 @@
+import { ServerConnection } from '@jupyterlab/services';
 import { ReadonlyJSONObject } from '@lumino/coreutils';
 import { Git } from '../tokens';
 
@@ -44,8 +45,7 @@ export const defaultMockedResponses: {
   stash: {
     body: () => ({
       code: 0,
-      message: '',
-      command: ''
+      stashes: []
     })
   },
   status: {
@@ -68,13 +68,15 @@ export function mockedRequestAPI(
   endPoint?: string,
   method?: string,
   body?: ReadonlyJSONObject | null,
-  namespace?: string
+  namespace?: string,
+  serverSettings?: ServerConnection.ISettings
 ) => Promise<any> {
   const mockedImplementation = (
     url?: string,
     method?: string,
     body?: ReadonlyJSONObject | null,
-    namespace?: string
+    namespace?: string,
+    serverSettings?: ServerConnection.ISettings
   ) => {
     mockedResponses = mockedResponses ?? {};
     const path = mockedResponses.path ?? DEFAULT_REPOSITORY_PATH;

src/git.ts

@@ -20,6 +20,7 @@ export const AUTH_ERROR_MESSAGES = [
  * @param method HTML method; default 'GET'
  * @param body JSON object to be passed as body or null; default null
  * @param namespace API namespace; default 'git'
+ * @param serverSettings Optional server connection settings; if not provided, uses ServerConnection.makeSettings()
  * @returns The response body interpreted as JSON
  *
  * @throws {Git.GitResponseError} If the server response is not ok
@@ -29,10 +30,11 @@ export async function requestAPI<T>(
   endPoint = '',
   method = 'GET',
   body: Partial<ReadonlyJSONObject> | null = null,
-  namespace = 'git'
+  namespace = 'git',
+  serverSettings?: ServerConnection.ISettings
 ): Promise<T> {
   // Make request to Jupyter API
-  const settings = ServerConnection.makeSettings();
+  const settings = serverSettings ?? ServerConnection.makeSettings();
   const requestUrl = URLExt.join(
     settings.baseUrl,
     namespace, // API Namespace

src/handler.ts

@@ -86,7 +86,7 @@ async function activate(
   translator: ITranslator | null
 ): Promise<IGitExtension> {
   let settings: ISettingRegistry.ISettings | undefined = undefined;
-  let serverSettings: Git.IServerSettings;
+  let gitServerSettings: Git.IServerSettings;
   translator = translator ?? nullTranslator;
   const trans = translator.load('jupyterlab_git');
 
@@ -98,9 +98,10 @@ async function activate(
       trans.__('Failed to load settings for the Git Extension.\n%1', error)
     );
   }
+  const serverSettings = app.serviceManager.serverSettings;
   try {
-    serverSettings = await getServerSettings(trans);
-    const { frontendVersion, gitVersion, serverVersion } = serverSettings;
+    gitServerSettings = await getServerSettings(trans, serverSettings);
+    const { frontendVersion, gitVersion, serverVersion } = gitServerSettings;
 
     // Version validation
     if (!gitVersion) {
@@ -145,7 +146,12 @@ async function activate(
     return null;
   }
   // Create the Git model
-  const gitExtension = new GitExtension(docmanager, app.docRegistry, settings);
+  const gitExtension = new GitExtension(
+    docmanager,
+    app.docRegistry,
+    settings,
+    serverSettings
+  );
 
   const onPathChanged = (
     model: FileBrowserModel,