@mcp-abap-adt/adt-clients 30.0.0 → 31.0.0

package/README.md

@@ -91,12 +91,12 @@ This package interacts with external packages **ONLY through interfaces**:
 
   | package | what this package takes from it |
   |---|---|
-  | [`@mcp-abap-adt/interfaces-adt`](https://www.npmjs.com/package/@mcp-abap-adt/interfaces-adt) `^15.1.0` | the capability atoms, every object's config, `IAdtResponse`, `IAnalyse`, `IAdtAnalyseOptions`, `IResultStrategy`, `ADT_NO_FAILURE`, `ADT_TASK_TYPE` |
-  | [`@mcp-abap-adt/interfaces-adt-connection`](https://www.npmjs.com/package/@mcp-abap-adt/interfaces-adt-connection) `^2.0.0` | `IAbapConnection`, `IAdtWireResponse`, `IAbapRequestOptions`, `ITimeoutConfig`, the connection capability atoms, `ADT_SESSION_ERROR` |
-  | [`@mcp-abap-adt/interfaces-network`](https://www.npmjs.com/package/@mcp-abap-adt/interfaces-network) `^2.0.0` | `IWebSocketTransport` and its four companions, `HttpError` |
-  | [`@mcp-abap-adt/interfaces-utils`](https://www.npmjs.com/package/@mcp-abap-adt/interfaces-utils) `^1.1.0` | `ILogger`, `LogLevel`, `XmlNode` |
-  | [`@mcp-abap-adt/interfaces-auth`](https://www.npmjs.com/package/@mcp-abap-adt/interfaces-auth) — **dev only** | `IAuthProvider`, for the test helpers that open a session |
-  | [`@mcp-abap-adt/interfaces-auth-sap`](https://www.npmjs.com/package/@mcp-abap-adt/interfaces-auth-sap) `^1.0.0` — **dev only** | `ISapConfig`, to build a connector in one unit test |
+  | [`@mcp-abap-adt/interfaces-adt`](https://www.npmjs.com/package/@mcp-abap-adt/interfaces-adt) `^16.0.0` | the capability atoms, every object's config, `IAdtResponse`, `IAnalyse`, `IAdtAnalyseOptions`, `IResultStrategy`, `ADT_NO_FAILURE`, `ADT_TASK_TYPE` |
+  | [`@mcp-abap-adt/interfaces-adt-connection`](https://www.npmjs.com/package/@mcp-abap-adt/interfaces-adt-connection) `^3.0.0` | `IAbapConnection`, `IAdtWireResponse`, `IAbapRequestOptions`, `ITimeoutConfig`, the connection capability atoms, `ADT_SESSION_ERROR` |
+  | [`@mcp-abap-adt/interfaces-network`](https://www.npmjs.com/package/@mcp-abap-adt/interfaces-network) `^2.0.1` | `IWebSocketTransport` and its four companions, `HttpError` |
+  | [`@mcp-abap-adt/interfaces-utils`](https://www.npmjs.com/package/@mcp-abap-adt/interfaces-utils) `^1.1.1` | `ILogger`, `LogLevel`, `XmlNode` |
+  | [`@mcp-abap-adt/interfaces-auth`](https://www.npmjs.com/package/@mcp-abap-adt/interfaces-auth) `^7.5.0` — **dev only** | `IAuthProvider`, for the test helpers that open a session |
+  | [`@mcp-abap-adt/interfaces-auth-sap`](https://www.npmjs.com/package/@mcp-abap-adt/interfaces-auth-sap) `^3.3.0` — **dev only** | `ISapConfig`, to build a connector in one unit test |
 
   Taking them by name is what keeps a consumer off one release rate for every contract: `interfaces-utils` has had two releases ever, and a package that needs only `ILogger` should move at that pace rather than at ADT's. The `@mcp-abap-adt/interfaces` facade is **deleted** — npm still serves 51.0.0 to anyone pinned to it and nothing further ships there.
 
@@ -162,7 +162,7 @@ npm install @mcp-abap-adt/adt-clients
 
 6. **AbapDebugger, AmdpDebugger, MemorySnapshots**
    - Standalone, from `@mcp-abap-adt/adt-clients/runtime` or the root: `new AbapDebugger(connection, logger, results?, { onConflict? })`
-   - One member per request, contracts `IAbapDebugger`, `IAmdpDebugger`, `IMemorySnapshots` from `@mcp-abap-adt/interfaces-adt` (contract 15 pairs with clients 29)
+   - One member per request, contracts `IAbapDebugger`, `IAmdpDebugger`, `IMemorySnapshots` from `@mcp-abap-adt/interfaces-adt` (contract 16 pairs with clients 31)
    - **The identity states its mode.** `{ debuggingMode: 'user', requestUser, terminalId, ideId }`, or `{ debuggingMode: 'terminal', terminalId, ideId }`; `attach` takes the identity, not the user. A terminal-mode listener catches a run that carries its ids: `run(target, { debuggee: { terminalId, ideId } })` sends them in `SAP-PASSPORT` (since 29.1.0)
    - **Breakpoints are replaced, not added to.** `setBreakpoints` and `syncBreakpoints` replace the identity's set, and `stopListener` deletes it: arm the whole set in one call
    - **Two sessions.** The ABAP listener holds one stateful connection; each debuggee caught is attached on a new one, routed to its application server. The AMDP debugger answers events on one session and takes commands on another
@@ -217,8 +217,11 @@ const config = {
 
 // Three things, all stated by you: which system (the connector), which
 // credential (the provider), and which wire (the transport). Since
-// @mcp-abap-adt/connection 6.0.0 nothing is inferred and there is no factory —
-// for ABAP Cloud take AdtCloudConnector + CloudHttpTransport instead.
+// @mcp-abap-adt/connection 6.0.0 nothing is inferred — for ABAP Cloud take
+// AdtCloudConnector + CloudHttpTransport instead; connection 16's
+// createConnection({ system, transport, … }) states the same two facts. One
+// instance is one SAP session: after disconnect() a new session is a new
+// instance.
 const connection = new AdtOnPremConnector(
   config,
   new BasicAuthProvider(config.username, config.password),
@@ -857,7 +860,7 @@ everything else is not.
 
 ### Single Definition Site: the contract packages
 
-Since **7.5.0**, every contract type is **defined once**, in the contract packages — today `@mcp-abap-adt/interfaces-adt` (`^15.1.0`) for the ADT contracts and `@mcp-abap-adt/interfaces-adt-connection` (`^2.0.0`) for the connection. This package declares no copies of its own — every `IXxxConfig`, the capability atoms, the option and response types and the cross-cutting shared types all live there.
+Since **7.5.0**, every contract type is **defined once**, in the contract packages — today `@mcp-abap-adt/interfaces-adt` (`^16.0.0`) for the ADT contracts and `@mcp-abap-adt/interfaces-adt-connection` (`^3.0.0`) for the connection. This package declares no copies of its own — every `IXxxConfig`, the capability atoms, the option and response types and the cross-cutting shared types all live there.
 
 **Import them from the package that owns them:**
 
@@ -937,7 +940,7 @@ contracts (`IAbapDebugger`, `IAmdpDebugger`, `IMemorySnapshots`, 13.1.0). This
 package no longer declares or exports any contract type — every contract name it uses is
 sourced from the contract packages.
 
-> **Version pairing.** `adt-clients` 29 pairs with `interfaces-adt` 15 (breakpoints lose `skipCount` and gain `clientId`), and `adt-strategies` 0.10.0 declares the same range. Because the types are sourced rather than copied, `@mcp-abap-adt/interfaces-adt` and `@mcp-abap-adt/interfaces-adt-connection` are hard peers of this package's public API. A major bump there implies a bump here; keep the two in step rather than letting a resolver pick a mismatched pair.
+> **Version pairing.** `adt-clients` 31 pairs with `interfaces-adt` 16 and `interfaces-adt-connection` 3 (the per-request `signal` and `ADT_REQUEST_ERROR` are gone), and `adt-strategies` 0.11.0 declares the same ranges. Because the types are sourced rather than copied, `@mcp-abap-adt/interfaces-adt` and `@mcp-abap-adt/interfaces-adt-connection` are hard peers of this package's public API. A major bump there implies a bump here; keep the two in step rather than letting a resolver pick a mismatched pair.
 
 ### Naming Conventions
 
@@ -974,6 +977,9 @@ See [Architecture Documentation](docs/architecture/ARCHITECTURE.md#type-system-a
 One document per breaking release, each written for a consumer on the contract
 before it:
 
+- **[30.x → 31.0.0](docs/usage/MIGRATION-31.md)** — `interfaces-adt` 16 and
+  `interfaces-adt-connection` 3: the per-request `signal`, `ADT_REQUEST_ERROR`
+  and `IAdtError.refusal` are gone; code written for 30.x compiles unchanged.
 - **[29.x → 30.0.0](docs/usage/MIGRATION-30.md)** — `getRequest()` gains
   `sortAndCompress` and `moveObjects`; `ITransportResults` has two more
   required slots, `sortedAndCompressed` and `movedObjects`.

package/dist/core/shared/sqlQuery.d.ts

@@ -10,6 +10,12 @@ import type { IGetSqlQueryParams } from './types';
 /**
  * Execute freestyle SQL query via SAP ADT Data Preview API
  *
+ * **No deadline of the library's.** The request carries `timeout: 0` — none:
+ * a query over large tables runs as long as it runs, and how long to wait is
+ * the caller's, set on the connection it hands in. Until 31.0.0 this set
+ * `getTimeout('long')` (`SAP_TIMEOUT_LONG`, 120 s by default), which a caller
+ * applying its own deadline only where the request had none never got to.
+ *
  * @param connection - ABAP connection
  * @param params - SQL query parameters
  * @returns Query results

package/dist/core/shared/sqlQuery.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"sqlQuery.d.ts","sourceRoot":"","sources":["../../../src/core/shared/sqlQuery.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EACV,eAAe,EACf,gBAAgB,EACjB,MAAM,yCAAyC,CAAC;AAGjD,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,SAAS,CAAC;AAElD;;;;;;GAMG;AACH,wBAAsB,WAAW,CAC/B,UAAU,EAAE,eAAe,EAC3B,MAAM,EAAE,kBAAkB,GACzB,OAAO,CAAC,gBAAgB,CAAC,CAc3B"}
+{"version":3,"file":"sqlQuery.d.ts","sourceRoot":"","sources":["../../../src/core/shared/sqlQuery.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EACV,eAAe,EACf,gBAAgB,EACjB,MAAM,yCAAyC,CAAC;AAEjD,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,SAAS,CAAC;AAElD;;;;;;;;;;;;GAYG;AACH,wBAAsB,WAAW,CAC/B,UAAU,EAAE,eAAe,EAC3B,MAAM,EAAE,kBAAkB,GACzB,OAAO,CAAC,gBAAgB,CAAC,CAgB3B"}

package/dist/core/shared/sqlQuery.js

@@ -9,10 +9,15 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getSqlQuery = getSqlQuery;
 const contentTypes_1 = require("../../constants/contentTypes");
-const timeouts_1 = require("../../utils/timeouts");
 /**
  * Execute freestyle SQL query via SAP ADT Data Preview API
  *
+ * **No deadline of the library's.** The request carries `timeout: 0` — none:
+ * a query over large tables runs as long as it runs, and how long to wait is
+ * the caller's, set on the connection it hands in. Until 31.0.0 this set
+ * `getTimeout('long')` (`SAP_TIMEOUT_LONG`, 120 s by default), which a caller
+ * applying its own deadline only where the request had none never got to.
+ *
  * @param connection - ABAP connection
  * @param params - SQL query parameters
  * @returns Query results
@@ -23,7 +28,9 @@ async function getSqlQuery(connection, params) {
     return connection.makeAdtRequest({
         url,
         method: 'POST',
-        timeout: (0, timeouts_1.getTimeout)('long'),
+        // No deadline: the contract requires a number, and 0 is what the wires
+        // read as "do not abort". How long to wait is the caller's.
+        timeout: 0,
         data: params.sql_query,
         headers: {
             'Content-Type': contentTypes_1.CT_SOURCE,

package/dist/core/shared/tableContents.d.ts

@@ -27,6 +27,11 @@ export declare function getTableColumns(connection: IAbapConnection, tableName:
  * statement the caller could not reach: not the column list, not an ordering,
  * not a `WHERE`. Read the columns with {@link getTableColumns} and write the
  * statement you want.
+ *
+ * **No deadline of the library's**, as for the freestyle query: the request
+ * carries `timeout: 0` — none — and how long to wait is the caller's, set on the
+ * connection it hands in. Until 31.0.0 this set `getTimeout('long')`
+ * (`SAP_TIMEOUT_LONG`, 120 s by default).
  */
 export declare function getTableContents(connection: IAbapConnection, params: IGetTableContentsParams): Promise<IAdtWireResponse>;
 //# sourceMappingURL=tableContents.d.ts.map

package/dist/core/shared/tableContents.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"tableContents.d.ts","sourceRoot":"","sources":["../../../src/core/shared/tableContents.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EACV,eAAe,EACf,gBAAgB,EACjB,MAAM,yCAAyC,CAAC;AAIjD,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,SAAS,CAAC;AAIvD;;;;;;GAMG;AACH,wBAAsB,eAAe,CACnC,UAAU,EAAE,eAAe,EAC3B,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC,gBAAgB,CAAC,CAS3B;AAED;;;;;;;;;GASG;AACH,wBAAsB,gBAAgB,CACpC,UAAU,EAAE,eAAe,EAC3B,MAAM,EAAE,uBAAuB,GAC9B,OAAO,CAAC,gBAAgB,CAAC,CAc3B"}
+{"version":3,"file":"tableContents.d.ts","sourceRoot":"","sources":["../../../src/core/shared/tableContents.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EACV,eAAe,EACf,gBAAgB,EACjB,MAAM,yCAAyC,CAAC;AAIjD,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,SAAS,CAAC;AAIvD;;;;;;GAMG;AACH,wBAAsB,eAAe,CACnC,UAAU,EAAE,eAAe,EAC3B,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC,gBAAgB,CAAC,CAS3B;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAsB,gBAAgB,CACpC,UAAU,EAAE,eAAe,EAC3B,MAAM,EAAE,uBAAuB,GAC9B,OAAO,CAAC,gBAAgB,CAAC,CAgB3B"}

package/dist/core/shared/tableContents.js

@@ -41,6 +41,11 @@ async function getTableColumns(connection, tableName) {
  * statement the caller could not reach: not the column list, not an ordering,
  * not a `WHERE`. Read the columns with {@link getTableColumns} and write the
  * statement you want.
+ *
+ * **No deadline of the library's**, as for the freestyle query: the request
+ * carries `timeout: 0` — none — and how long to wait is the caller's, set on the
+ * connection it hands in. Until 31.0.0 this set `getTimeout('long')`
+ * (`SAP_TIMEOUT_LONG`, 120 s by default).
  */
 async function getTableContents(connection, params) {
     const tableName = params.table_name.toUpperCase();
@@ -48,7 +53,9 @@ async function getTableContents(connection, params) {
     return connection.makeAdtRequest({
         url: `/sap/bc/adt/datapreview/ddic?rowNumber=${maxRows}&ddicEntityName=${encodeURIComponent(tableName)}`,
         method: 'POST',
-        timeout: (0, timeouts_1.getTimeout)('long'),
+        // No deadline: the contract requires a number, and 0 is what the wires
+        // read as "do not abort". How long to wait is the caller's.
+        timeout: 0,
         data: params.sql_query,
         headers: {
             Accept: ACCEPT_HEADER,

package/docs/usage/MIGRATION-31.md

@@ -0,0 +1,38 @@
+# Migrating to 31.0.0
+
+31.0.0 moves the contract packages a major: `@mcp-abap-adt/interfaces-adt`
+`^16.0.0` and `@mcp-abap-adt/interfaces-adt-connection` `^3.0.0`. Their types
+are this package's public API, which is why this is a major too. No member
+answers differently.
+
+## 1. Declare the new ranges
+
+If your package imports the contracts, declare the same ranges, so npm installs
+one copy of each — `npm ls @mcp-abap-adt/interfaces-adt` and
+`npm ls @mcp-abap-adt/interfaces-adt-connection` must each show one version.
+`@mcp-abap-adt/adt-strategies` 0.11.0 declares the same ranges.
+
+```json
+"@mcp-abap-adt/interfaces-adt": "^16.0.0",
+"@mcp-abap-adt/interfaces-adt-connection": "^3.0.0"
+```
+
+## 2. What the contracts removed
+
+`interfaces-adt` 16.0.0 removes `IAdtClientOptions.signal` and
+`IAdtError.refusal`; `interfaces-adt-connection` 3.0.0 removes
+`IAbapRequestOptions.signal`, `ADT_REQUEST_ERROR` and `AdtRequestErrorCode`. All
+of them arrived in 15.4.0 / 2.1.0, which 30.x did not declare — so code written
+for 30.x compiles unchanged. Code that took them up drops them: no `signal` in
+client or request options, no read of `refusal`, no match on
+`ADT_REQUEST_ERROR`. An `IAbapConnection` implementation of your own drops the
+same.
+
+## 3. The connection you build (if it is `@mcp-abap-adt/connection` 16)
+
+Not this package's contract, but what a consumer moving with it meets: one
+connection instance is one SAP session. `connect()` after `disconnect()` — or
+after a lost session — rejects, and a second `disconnect()` rejects
+`NOT_CONNECTED`; a new session is a new instance, built with the constructors
+or `createConnection({ system, transport, … })`. Anything built over the old
+instance (an `AdtClient`, a handler) is built again over the new one.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mcp-abap-adt/adt-clients",
-  "version": "30.0.0",
+  "version": "31.0.0",
   "description": "ADT clients for SAP ABAP systems - AdtClient and AdtRuntimeClient",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -117,21 +117,21 @@
     "node": ">=18.0.0"
   },
   "dependencies": {
-    "@mcp-abap-adt/interfaces-adt": "^15.3.0",
-    "@mcp-abap-adt/interfaces-adt-connection": "^2.0.0",
-    "@mcp-abap-adt/interfaces-network": "^2.0.0",
-    "@mcp-abap-adt/interfaces-utils": "^1.1.0",
-    "@mcp-abap-adt/logger": "^0.4.0",
+    "@mcp-abap-adt/interfaces-adt": "^16.0.0",
+    "@mcp-abap-adt/interfaces-adt-connection": "^3.0.0",
+    "@mcp-abap-adt/interfaces-network": "^2.0.1",
+    "@mcp-abap-adt/interfaces-utils": "^1.1.1",
+    "@mcp-abap-adt/logger": "^0.4.1",
     "axios": "^1.18.1",
     "fast-xml-parser": "^5.9.3"
   },
   "devDependencies": {
     "@biomejs/biome": "^2.4.4",
     "@mcp-abap-adt/adt-strategies": "^0.10.0",
-    "@mcp-abap-adt/auth-providers": "^5.0.1",
-    "@mcp-abap-adt/connection": "^10.0.2",
-    "@mcp-abap-adt/interfaces-auth": "^3.0.0",
-    "@mcp-abap-adt/interfaces-auth-sap": "^1.1.0",
+    "@mcp-abap-adt/auth-providers": "^6.0.2",
+    "@mcp-abap-adt/connection": "^16.0.0",
+    "@mcp-abap-adt/interfaces-auth": "^7.5.0",
+    "@mcp-abap-adt/interfaces-auth-sap": "^3.3.0",
     "@types/jest": "^30.0.0",
     "@types/node": "^25.3.3",
     "dotenv": "^17.3.1",