Merge pull request #231 from jshemas/commonjs-fix

adding main and types back into the package.json
jshemas · Jun 29, 2024 · a819cd2 · a819cd2
2 parents d7fe9ac + 5e897d1
commit a819cd2
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Change Log
 
+## 6.6.2
+
+- Fixed issue with `package.json` `exports` was not working in `CommonJs` projects.
+
 ## 6.6.1
 
 - Use `node16` for `module`/`moduleResolution` ESM build

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,13 +1,15 @@
 {
   "name": "open-graph-scraper",
   "description": "Node.js scraper module for Open Graph and Twitter Card info",
-  "version": "6.6.1",
+  "version": "6.6.2",
   "license": "MIT",
+  "main": "./dist/cjs/index.js",
+  "types": "./types/index.d.ts",
   "exports": {
     ".": {
-      "require": "./dist/cjs/index.js",
+      "types": "./types/index.d.ts",
       "import": "./dist/esm/index.js",
-      "types": "./types/index.d.ts"
+      "require": "./dist/cjs/index.js"
     }
   },
   "scripts": {

diff --git a/tests/integration/basic.spec.ts b/tests/integration/basic.spec.ts
@@ -224,7 +224,8 @@ describe('basic', function () {
     });
   });
 
-  it('vimeo.com should return open graph data', function () {
+  // vimeo.com is now blocking OGS even when using the "User Agent" header
+  it.skip('vimeo.com should return open graph data', function () {
     return ogsRequire({ url: 'https://vimeo.com/232889838' }).then(function ({ error, result, response }) {
       console.log('error:', error);
       console.log('result:', result);

diff --git a/tests/unit/static.spec.ts b/tests/unit/static.spec.ts
@@ -218,6 +218,7 @@ describe('static check meta tags', function () {
   it('jsonLD', function () {
     const metaHTML = `<html><head>
     <script type="application/ld+json">{"foo":"bar"}</script>
+    <script type="application/ld+json"></script>
     </head></html>`;
 
     mockAgent.get('http://www.test.com')