@adobe/spacecat-shared-content-client 1.2.1 → 1.2.3

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+# [@adobe/spacecat-shared-content-client-v1.2.3](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-content-client-v1.2.2...@adobe/spacecat-shared-content-client-v1.2.3) (2024-11-23)
+
+
+### Bug Fixes
+
+* **deps:** update external fixes ([#454](https://github.com/adobe/spacecat-shared/issues/454)) ([325cf8d](https://github.com/adobe/spacecat-shared/commit/325cf8dded5fcabadaf7d8fdd510d33aeafd08a7))
+
+# [@adobe/spacecat-shared-content-client-v1.2.2](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-content-client-v1.2.1...@adobe/spacecat-shared-content-client-v1.2.2) (2024-11-20)
+
+
+### Bug Fixes
+
+* **deps:** update dependency @adobe/spacecat-helix-content-sdk to v1.3.5 ([#445](https://github.com/adobe/spacecat-shared/issues/445)) ([a4a1027](https://github.com/adobe/spacecat-shared/commit/a4a1027c5ac7b2c2d342a37eca15d7a85a4582cc))
+
 # [@adobe/spacecat-shared-content-client-v1.2.1](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-content-client-v1.2.0...@adobe/spacecat-shared-content-client-v1.2.1) (2024-11-16)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-content-client",
-  "version": "1.2.1",
+  "version": "1.2.3",
   "description": "Shared modules of the Spacecat Services - Content Client",
   "type": "module",
   "engines": {
@@ -36,7 +36,7 @@
   "dependencies": {
     "@adobe/helix-universal": "5.0.6",
     "@adobe/spacecat-shared-data-access": "1.50.0",
-    "@adobe/spacecat-helix-content-sdk": "1.2.7",
+    "@adobe/spacecat-helix-content-sdk": "1.3.5",
     "@adobe/spacecat-shared-utils": "1.22.4",
     "graph-data-structure": "4.2.0"
   },
@@ -47,6 +47,6 @@
     "nock": "13.5.6",
     "sinon": "19.0.2",
     "sinon-chai": "4.0.0",
-    "typescript": "5.6.3"
+    "typescript": "5.7.2"
   }
 }

package/src/clients/content-client.js

@@ -264,8 +264,8 @@ export default class ContentClient {
     this.log.info(`Getting page metadata for ${this.site.getId()} and path ${path}`);
 
     const docPath = this.#resolveDocPath(path);
-    const document = await this.rawClient.read(docPath);
-    const metadata = document.getMetadata();
+    const document = this.rawClient.getDocument(docPath);
+    const metadata = await document.getMetadata();
 
     this.#logDuration('getPageMetadata', startTime);
 
@@ -284,8 +284,8 @@ export default class ContentClient {
     this.log.info(`Updating page metadata for ${this.site.getId()} and path ${path}`);
 
     const docPath = this.#resolveDocPath(path);
-    const document = await this.rawClient.read(docPath);
-    const originalMetadata = document.getMetadata();
+    const document = this.rawClient.getDocument(docPath);
+    const originalMetadata = await document.getMetadata();
 
     let mergedMetadata;
     if (overwrite) {
@@ -296,10 +296,7 @@ export default class ContentClient {
 
     const response = await document.updateMetadata(mergedMetadata);
     if (response?.status !== 200) {
-      const saveResponse = await this.rawClient.save(document);
-      if (saveResponse.status !== 200) {
-        throw new Error(`Failed to update metadata for path ${path}`);
-      }
+      throw new Error(`Failed to update metadata for path ${path}`);
     }
 
     this.#logDuration('updatePageMetadata', startTime);
@@ -313,7 +310,8 @@ export default class ContentClient {
 
     this.log.info(`Getting redirects for ${this.site.getId()}`);
 
-    const redirects = await this.rawClient.getRedirects();
+    const redirectsFile = this.rawClient.getRedirects();
+    const redirects = await redirectsFile.get();
     this.#logDuration('getRedirects', startTime);
 
     return redirects;
@@ -328,8 +326,8 @@ export default class ContentClient {
 
     this.log.info(`Updating redirects for ${this.site.getId()}`);
 
-    const currentRedirects = await this.getRedirects();
-
+    const redirectsFile = this.rawClient.getRedirects();
+    const currentRedirects = await redirectsFile.get();
     // validate combination of existing and new redirects
     const cleanNewRedirects = removeDuplicatedRedirects(currentRedirects, redirects, this.log);
     if (cleanNewRedirects.length === 0) {
@@ -342,7 +340,7 @@ export default class ContentClient {
       return;
     }
 
-    const response = await this.rawClient.appendRedirects(noCycleRedirects);
+    const response = await redirectsFile.append(noCycleRedirects);
     if (response.status !== 200) {
       throw new Error('Failed to update redirects');
     }