feat: Implement KeycloakService for managing Keycloak clients and tokens

- Added KeycloakService to handle client creation and token fetching.
- Introduced KeycloakError for better error handling specific to Keycloak operations.
- Integrated KeycloakService into ToolsService for untrust client functionality.
- Enhanced OasConversionService and OasDereferenceService with improved error logging.
- Refactored OasInputService to utilize RemoteSpecificationService for fetching specifications.
- Created PostmanConversionService for converting OpenAPI specifications to Postman collections.
- Added ZipService for creating ZIP files from entries.
- Introduced utility functions for sanitizing file names.
- Updated error handling and logging across various services.

Author: pasibun

.github/workflows/go-ci.yml .github/workflows/node-ci.yml.github/workflows/go-ci.yml renamed to .github/workflows/node-ci.yml

@@ -1,43 +1,37 @@
-name: Go CI/CD
+name: Node CI/CD
 
-on: push
+on:
+  push:
+  pull_request:
 
 jobs:
   lint:
     name: Lint
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - name: Setup Go
-        uses: actions/setup-go@v5
-        with:
-          go-version: '1.24'
-      - name: Install golangci-lint
-        run: go install github.com/golangci/golangci-lint/cmd/golangci-lint@latest
-      - name: Run golangci-lint
-        run: golangci-lint run --timeout 5m
-
-  test:
-    name: Test
-    runs-on: ubuntu-latest
-    needs: lint
-    steps:
-      - uses: actions/checkout@v4
-      - name: Setup Go
-        uses: actions/setup-go@v5
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
         with:
-          go-version: '1.21'
-      - name: Run tests
-        run: go test -coverprofile coverage.out ./...
+          node-version: 20
+          cache: npm
+
+      - name: Install dependencies
+        run: npm install --ignore-scripts
+
+      - name: Run lint
+        run: npm run lint
 
   build-and-push:
     name: Build & Push Docker Image
     runs-on: ubuntu-latest
-    needs: test
+    needs: lint
     permissions:
       packages: write
     steps:
       - uses: actions/checkout@v4
+
       - name: Login to GitHub Container Registry
         uses: docker/login-action@v3
         with:

api/openapi.json

@@ -31,7 +31,7 @@
           "content": {
             "application/json": {
               "schema": {
-                "$ref": "#/components/schemas/ArazzoInput"
+                "$ref": "#/components/schemas/OASInput"
               }
             }
           }

api/openapi.yaml

@@ -27,7 +27,7 @@ paths:
         content:
           application/json:
             schema:
-              $ref: "#/components/schemas/ArazzoInput"
+              $ref: "#/components/schemas/OASInput"
       responses:
         "200":
           content:
@@ -285,16 +285,6 @@ components:
         API-Version:
           $ref: "#/headers/API-Version"
   schemas:
-    ArazzoInput:
-      example:
-        arazzoBody: arazzoBody
-        arazzoUrl: arazzoUrl
-      properties:
-        arazzoBody:
-          type: string
-        arazzoUrl:
-          type: string
-      type: object
     OASInput:
       example:
         oasUrl: oasUrl

biome.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "https://biomejs.dev/schemas/1.7.2/schema.json",
+  "$schema": "https://biomejs.dev/schemas/2.3.4/schema.json",
   "formatter": {
     "indentStyle": "space",
     "indentWidth": 2,

controllers/Controller.js

@@ -2,6 +2,7 @@ const fs = require("node:fs");
 const path = require("node:path");
 const config = require("../config");
 const Service = require("../services/Service");
+const logger = require("../logger");
 
 class Controller {
   static getStatusText(status) {
@@ -77,6 +78,13 @@ class Controller {
       problem.invalidParams = invalidParams;
     }
 
+    logger.error(`Request failed (${status} ${Controller.getStatusText(status)}): ${reason}`, {
+      detail,
+      invalidParams,
+      errorMessage: error?.message,
+      stack: error?.stack,
+    });
+
     response.status(status).json(problem);
   }
 
@@ -126,23 +134,23 @@ class Controller {
       return params;
     }
     const result = { ...params };
-    if (!Object.prototype.hasOwnProperty.call(result, bodyName)) {
+    if (!Object.hasOwn(result, bodyName)) {
       result[bodyName] = value;
     }
-    if (!Object.prototype.hasOwnProperty.call(result, "body")) {
+    if (!Object.hasOwn(result, "body")) {
       result.body = value;
     }
     const sanitizedName = Service.sanitizeOperationId(bodyName);
-    if (sanitizedName && !Object.prototype.hasOwnProperty.call(result, sanitizedName)) {
+    if (sanitizedName && !Object.hasOwn(result, sanitizedName)) {
       result[sanitizedName] = value;
     }
     const lowerCaseName = bodyName.charAt(0).toLowerCase() + bodyName.slice(1);
-    if (lowerCaseName && !Object.prototype.hasOwnProperty.call(result, lowerCaseName)) {
+    if (lowerCaseName && !Object.hasOwn(result, lowerCaseName)) {
       result[lowerCaseName] = value;
     }
     if (value && typeof value === "object" && !Array.isArray(value)) {
       Object.keys(value).forEach((key) => {
-        if (!Object.prototype.hasOwnProperty.call(result, key)) {
+        if (!Object.hasOwn(result, key)) {
           result[key] = value[key];
         }
       });
@@ -181,7 +189,7 @@ class Controller {
         }
         const declaredProperties = schemaDefinition?.properties ? Object.keys(schemaDefinition.properties) : [];
         const allowAdditional = (() => {
-          if (schemaDefinition && Object.prototype.hasOwnProperty.call(schemaDefinition, "additionalProperties")) {
+          if (schemaDefinition && Object.hasOwn(schemaDefinition, "additionalProperties")) {
             return schemaDefinition.additionalProperties !== false;
           }
           return false;

expressServer.js

@@ -82,7 +82,7 @@ class ExpressServer {
     this.port = port;
     this.app = express();
     try {
-      this.schema = jsYaml.safeLoad(fs.readFileSync(openApiYaml));
+      this.schema = jsYaml.load(fs.readFileSync(openApiYaml, "utf8"));
       if (this.schema?.components) {
         const { components } = this.schema;
         const componentMirrors = [
@@ -136,11 +136,11 @@ class ExpressServer {
     this.app.use(express.json());
     this.app.use(express.urlencoded({ extended: false }));
     this.app.use(cookieParser());
-    this.app.use((req, res, next) => {
+    this.app.use((_req, res, next) => {
       res.set("API-Version", this.schema.info.version);
       next();
     });
-    this.app.get("/openapi.json", (req, res) => res.json(this.schema));
+    this.app.get("/openapi.json", (_req, res) => res.json(this.schema));
     this.app.get("/login-redirect", (req, res) => {
       res.status(200);
       res.json(req.query);
@@ -235,7 +235,7 @@ class ExpressServer {
 
   launch() {
     // eslint-disable-next-line no-unused-vars
-    this.app.use((err, req, res, next) => {
+    this.app.use((err, req, res, _next) => {
       // format errors using RFC 7807 Problem Details format
       const status = err.status || 500;
       const problemDetails = {