@podium/layout - npm Package Compare versions

Comparing version 5.1.4 to 5.1.5

lib/layout.js

@@ -8,9 +8,5 @@ /* eslint-disable consistent-return */
     HttpIncoming,
-    // @ts-ignore
     template,
-    // @ts-ignore
     isFunction,
-    // @ts-ignore
     pathnameBuilder,
-    // @ts-ignore
     uriIsRelative,
@@ -44,6 +40,6 @@ AssetCss,
  * @property {string} pathname - (required) layout pathname
- * @property {Console | import('abslog').AbstractLogger} [logger] - A logger to use when provided. Can be the console object if console logging is desired but can also be any Log4j compatible logging object as well. Nothing is logged if no logger is provided. (default null)
+ * @property {import('abslog').AbstractLoggerOptions} [logger] - A logger to use when provided. Can be the console object if console logging is desired but can also be any Log4j compatible logging object as well. Nothing is logged if no logger is provided. (default null)
  * @property {LayoutContext} [context] - Options to be passed on to the internal `@podium/context` constructor. See that module for details. (default null)
  * @property {LayoutClientOptions} [client] - Options to be passed on to the internal `@podium/client` constructor. See that module for details. (default null)
- * @property {import("@podium/proxy").default.PodiumProxyOptions} [proxy] - Options to be passed on to the internal `@podium/proxy` constructor. See that module for details. (default null)
+ * @property {import('@podium/proxy').PodiumProxyOptions} [proxy] - Options to be passed on to the internal `@podium/proxy` constructor. See that module for details. (default null)
  *
@@ -121,3 +117,3 @@ *
      *
-     * @type {import('abslog').AbstractLogger}
+     * @type {import('abslog').ValidLogger}
      */
@@ -153,3 +149,3 @@ log;
      *
-     * @type {LayoutContext}
+     * @type {Context}
      */
@@ -301,3 +297,2 @@ context;
         this.httpProxy.metrics.pipe(this.metrics);
-        // @ts-ignore
         this.context.metrics.pipe(this.metrics);
@@ -307,5 +302,3 @@ this.client.metrics.pipe(this.metrics);
         // Register proxy endpoints
-        // @ts-ignore
         this.client.registry.on('set', (key, item) => {
-            // @ts-ignore
             this.httpProxy.register(key, item.newVal);
@@ -570,3 +563,2 @@ });
 
-        // @ts-ignore
         if (context) await this.context.process(incoming);
@@ -597,3 +589,3 @@ if (proxy) await this.httpProxy.process(incoming);
             const incoming = new HttpIncoming(req, res, res.locals);
-            // @ts-ignore
+            // @ts-expect-error The API here is a bit odd. You assign a string, but the `set` constructs a URL from it.
             incoming.url = `${req.protocol}://${req.get('host')}${
@@ -671,7 +663,3 @@ req.originalUrl
         if (uri) {
-            // @ts-ignore
-            return uriIsRelative(uri)
-                ? // @ts-ignore
-                  pathnameBuilder(pathname, uri)
-                : uri;
+            return uriIsRelative(uri) ? pathnameBuilder(pathname, uri) : uri;
         }
@@ -678,0 +666,0 @@ return uri;

package.json

 {
   "name": "@podium/layout",
-  "version": "5.1.4",
+  "version": "5.1.5",
   "type": "module",
@@ -35,3 +35,3 @@ "description": "Module for composing full page layouts out of page fragments in a micro frontend architecture.",
     "lint:fix": "eslint --fix .",
-    "test": "tap --disable-coverage --allow-empty-coverage",
+    "test": "tap --disable-coverage --allow-empty-coverage && tsc --project tsconfig.test.json",
     "test:snapshots": "tap --snapshot --disable-coverage --allow-empty-coverage",
@@ -42,8 +42,8 @@ "types": "tsc --declaration --emitDeclarationOnly && ./fixup.sh"
     "@metrics/client": "2.5.2",
-    "@podium/client": "5.0.10",
-    "@podium/context": "5.0.4",
-    "@podium/proxy": "5.0.8",
-    "@podium/schemas": "5.0.0",
-    "@podium/utils": "5.0.3",
-    "abslog": "2.4.2",
+    "@podium/client": "5.0.21",
+    "@podium/context": "5.0.13",
+    "@podium/proxy": "5.0.16",
+    "@podium/schemas": "5.0.2",
+    "@podium/utils": "5.0.6",
+    "abslog": "2.4.4",
     "ajv": "8.12.0",
@@ -55,3 +55,3 @@ "lodash.merge": "4.6.2",
     "@babel/eslint-parser": "7.23.3",
-    "@podium/podlet": "5.0.6",
+    "@podium/podlet": "5.1.4",
     "@podium/test-utils": "2.5.2",
@@ -61,4 +61,4 @@ "@semantic-release/changelog": "6.0.3",
     "@semantic-release/git": "10.0.1",
-    "@semantic-release/github": "9.2.4",
-    "@semantic-release/npm": "11.0.1",
+    "@semantic-release/github": "9.2.6",
+    "@semantic-release/npm": "11.0.3",
     "@semantic-release/release-notes-generator": "12.1.0",
@@ -75,3 +75,3 @@ "@types/node": "^20.10.5",
     "prettier": "3.1.0",
-    "semantic-release": "22.0.8",
+    "semantic-release": "22.0.12",
     "stoppable": "1.1.0",
@@ -78,0 +78,0 @@ "supertest": "6.3.4",

types/layout.d.ts

@@ -7,6 +7,6 @@ /**
  * @property {string} pathname - (required) layout pathname
- * @property {Console | import('abslog').AbstractLogger} [logger] - A logger to use when provided. Can be the console object if console logging is desired but can also be any Log4j compatible logging object as well. Nothing is logged if no logger is provided. (default null)
+ * @property {import('abslog').AbstractLoggerOptions} [logger] - A logger to use when provided. Can be the console object if console logging is desired but can also be any Log4j compatible logging object as well. Nothing is logged if no logger is provided. (default null)
  * @property {LayoutContext} [context] - Options to be passed on to the internal `@podium/context` constructor. See that module for details. (default null)
  * @property {LayoutClientOptions} [client] - Options to be passed on to the internal `@podium/client` constructor. See that module for details. (default null)
- * @property {import("@podium/proxy").default.PodiumProxyOptions} [proxy] - Options to be passed on to the internal `@podium/proxy` constructor. See that module for details. (default null)
+ * @property {import('@podium/proxy').PodiumProxyOptions} [proxy] - Options to be passed on to the internal `@podium/proxy` constructor. See that module for details. (default null)
  *
@@ -69,5 +69,5 @@ *
      *
-     * @type {import('abslog').AbstractLogger}
+     * @type {import('abslog').ValidLogger}
      */
-    log: import('abslog').AbstractLogger;
+    log: import('abslog').ValidLogger;
     /**
@@ -99,5 +99,5 @@ * An instance of the `@podium/proxy` module
      *
-     * @type {LayoutContext}
+     * @type {Context}
      */
-    context: LayoutContext;
+    context: Context;
     /**
@@ -170,3 +170,3 @@ * Property that holds information about a Cascading Style Sheet related to a layout.
     metrics: import("@metrics/client");
-    client: Client;
+    client: any;
     /**
@@ -414,3 +414,3 @@ * Set relative or absolute URLs to Cascading Style Sheets (CSS) assets for the layout.
      */
-    logger?: Console | import('abslog').AbstractLogger;
+    logger?: import('abslog').AbstractLoggerOptions;
     /**
@@ -427,3 +427,3 @@ * - Options to be passed on to the internal `@podium/context` constructor. See that module for details. (default null)
      */
-    proxy?: import("@podium/proxy").default.PodiumProxyOptions;
+    proxy?: import('@podium/proxy').PodiumProxyOptions;
 };
@@ -507,9 +507,9 @@ export type LayoutContext = {
 };
+import Context from '@podium/context';
 import { AssetCss } from '@podium/utils';
 import { AssetJs } from '@podium/utils';
-import Client from '@podium/client';
 import { HttpIncoming } from '@podium/utils';
 declare global {
   namespace Express {
-    interface Response {
+    export interface Response {
       podiumSend(fragment: string, ...args: unknown[]): Response;
@@ -516,0 +516,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

130741

increased by0.09%

Worsened metrics

Lines of code

1135

decreased by-1.05%

Dependency changes

• + Added@podium/client@5.0.21(transitive)

• + Added@podium/context@5.0.13(transitive)

• + Added@podium/proxy@5.0.16(transitive)

• + Added@podium/schemas@5.0.2(transitive)

• + Added@podium/utils@5.0.6(transitive)

• + Addedabslog@2.4.4(transitive)

• + Addedajv@8.13.0(transitive)

• + Addedlru-cache@10.2.2(transitive)

• + Addedundici@6.16.1(transitive)

• - Removed@podium/client@5.0.10(transitive)

• - Removed@podium/context@5.0.4(transitive)

• - Removed@podium/proxy@5.0.8(transitive)

• - Removed@podium/schemas@5.0.0(transitive)

• - Removed@podium/utils@5.0.3(transitive)

• - Removedabslog@2.4.2(transitive)

• - Removedlru-cache@10.2.0(transitive)

• - Removedundici@6.21.0(transitive)

• Updated@podium/client@5.0.21

• Updated@podium/context@5.0.13

• Updated@podium/proxy@5.0.16

• Updated@podium/schemas@5.0.2

• Updated@podium/utils@5.0.6

• Updatedabslog@2.4.4