@reykjavik/webtools 0.1.10 → 0.1.11

package/CHANGELOG.md

@@ -4,6 +4,12 @@
 
 - ... <!-- Add new lines here. -->
 
+## 0.1.11
+
+_2024-02-14_
+
+- fix: Issue with `SiteImprove` import `next/script` in Next.js < 13
+
 ## 0.1.10
 
 _2024-02-14_

package/esm/http.d.ts

@@ -1,4 +1,4 @@
-import { ServerResponse } from 'http';
+import type { ServerResponse } from 'http';
 /** The client should continue the request or ignore the response if the request is already finished. */
 export declare const HTTP_100_Continue = 100;
 /** Response to an Upgrade request header from the client and indicates the protocol the server is switching to. */

package/esm/next/SiteImprove.js

@@ -2,8 +2,12 @@ import React, { useEffect } from 'react';
 import { Router } from 'next/router.js';
 import NextScript from 'next/script.js';
 import { useCookieHubConsent } from '../CookieHubConsent.js';
-// Fixes an issue with `next/script` types when `pkgJson.type === "module"`
-const Script = NextScript;
+// Fixes an issue with `next/script`'s types and mixture of default and named exports.
+// This workaround doesn't seem to be necessary in Next.js 13.5 (pages router), but
+// is definitely needed for the webpack bundler used by Next.js 11. (v12 is untested.)
+const Script = ('__esModule' in NextScript && 'default' in NextScript
+    ? NextScript.default
+    : NextScript);
 // END: Mock typing of SiteImprove's event tracking API
 // --------------------------------------------------------------------------
 //

package/esm/next/http.d.ts

@@ -1,7 +1,7 @@
 /// <reference types="node" resolution-mode="require"/>
 import React, { FunctionComponent } from 'react';
-import { Cleanup } from '@reykjavik/hanna-utils';
-import { ServerResponse } from 'http';
+import type { Cleanup } from '@reykjavik/hanna-utils';
+import type { ServerResponse } from 'http';
 import type { AppType } from 'next/app.js';
 import type { HTTP_ERROR_ALL, TTLConfig } from '../http.js';
 export * from '../http.js';

package/http.d.ts

@@ -1,4 +1,4 @@
-import { ServerResponse } from 'http';
+import type { ServerResponse } from 'http';
 /** The client should continue the request or ignore the response if the request is already finished. */
 export declare const HTTP_100_Continue = 100;
 /** Response to an Upgrade request header from the client and indicates the protocol the server is switching to. */

package/next/SiteImprove.js

@@ -31,8 +31,12 @@ const react_1 = __importStar(require("react"));
 const router_js_1 = require("next/router.js");
 const script_js_1 = __importDefault(require("next/script.js"));
 const CookieHubConsent_js_1 = require("../CookieHubConsent.js");
-// Fixes an issue with `next/script` types when `pkgJson.type === "module"`
-const Script = script_js_1.default;
+// Fixes an issue with `next/script`'s types and mixture of default and named exports.
+// This workaround doesn't seem to be necessary in Next.js 13.5 (pages router), but
+// is definitely needed for the webpack bundler used by Next.js 11. (v12 is untested.)
+const Script = ('__esModule' in script_js_1.default && 'default' in script_js_1.default
+    ? script_js_1.default.default
+    : script_js_1.default);
 // END: Mock typing of SiteImprove's event tracking API
 // --------------------------------------------------------------------------
 //

package/next/http.d.ts

@@ -1,7 +1,7 @@
 /// <reference types="node" />
 import React, { FunctionComponent } from 'react';
-import { Cleanup } from '@reykjavik/hanna-utils';
-import { ServerResponse } from 'http';
+import type { Cleanup } from '@reykjavik/hanna-utils';
+import type { ServerResponse } from 'http';
 import type { AppType } from 'next/app.js';
 import type { HTTP_ERROR_ALL, TTLConfig } from '../http.js';
 export * from '../http.js';

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@reykjavik/webtools",
-	"version": "0.1.10",
+	"version": "0.1.11",
 	"description": "Misc. JS/TS helpers used by Reykjavík City's web dev teams.",
 	"main": "index.js",
 	"repository": "ssh://git@github.com:reykjavikcity/webtools.git",