massimo-cli 1.5.0 → 1.5.2

package/lib/frontend-openapi-generator.js

@@ -68,6 +68,7 @@ function generateFrontendImplementationFromOpenAPI ({
     useTabs: false,
     useSingleQuote: true
   })
+  const hasPathParameters = operations.some(operation => operation.path.includes('{'))
 
   writer.write('// This client was generated by Platformatic from an OpenAPI specification.')
   writer.blankLine()
@@ -132,6 +133,18 @@ function generateFrontendImplementationFromOpenAPI ({
       writer.writeLine('return output')
     })
   }
+  if (hasPathParameters) {
+    writer.newLine()
+    writer
+      .write(`function encodePathParameter(value${isTsLang ? ': unknown' : ''})${isTsLang ? ': string' : ''} `)
+      .block(() => {
+        writer.writeLine(`const encoded = encodeURIComponent(value${isTsLang ? ' as string' : ''})`)
+        writer.write('if (encoded === \'.\' || encoded === \'..\') ').block(() => {
+          writer.writeLine('throw new Error(\'Path parameters cannot be "." or ".."\')')
+        })
+        writer.writeLine('return encoded')
+      })
+  }
   writer.blankLine()
   const allOperations = []
   const originalFullResponse = fullResponse
@@ -192,7 +205,9 @@ function generateFrontendImplementationFromOpenAPI ({
       // /organizations/{orgId}/members/{memberId}
       // to
       // /organizations/${request.orgId}/members/${request.memberId}
-      const stringLiteralPath = path.replace(/\{/gm, '${' + req + "['").replace(/\}/gm, "']}")
+      const stringLiteralPath = path
+        .replace(/\{/gm, '${encodePathParameter(' + req + "['")
+        .replace(/\}/gm, "'])}")
       // GET methods need query strings instead of JSON bodies
       if (queryParams.length) {
         // query parameters should be appended to the url

package/lib/get-type.js

@@ -1,16 +1,21 @@
 import jsonpointer from 'jsonpointer'
 
-export function getType (typeDef, methodType, spec) {
+export function getType (typeDef, methodType, spec, seenRefs = new Set()) {
   if (typeDef.$ref) {
+    if (seenRefs.has(typeDef.$ref)) {
+      return 'unknown'
+    }
+    seenRefs = new Set(seenRefs)
+    seenRefs.add(typeDef.$ref)
     typeDef = jsonpointer.get(spec, typeDef.$ref.replace('#', ''))
   }
   if (typeDef.schema) {
-    return getType(typeDef.schema, methodType, spec)
+    return getType(typeDef.schema, methodType, spec, seenRefs)
   }
   if (typeDef.anyOf) {
     // recursively call this function
     const mapped = typeDef.anyOf.map(t => {
-      return getType(t, methodType, spec)
+      return getType(t, methodType, spec, seenRefs)
     })
     return mapped.join(' | ')
   }
@@ -18,7 +23,7 @@ export function getType (typeDef, methodType, spec) {
   if (typeDef.oneOf) {
     // recursively call this function
     const mapped = typeDef.oneOf.map(t => {
-      return getType(t, methodType, spec)
+      return getType(t, methodType, spec, seenRefs)
     })
 
     if (typeDef.discriminator && typeDef.discriminator.propertyName) {
@@ -56,13 +61,13 @@ export function getType (typeDef, methodType, spec) {
     // recursively call this function
     return typeDef.allOf
       .map(t => {
-        return getType(t, methodType, spec)
+        return getType(t, methodType, spec, seenRefs)
       })
       .join(' & ')
   }
   if (typeDef.type === 'array') {
     const nullable = typeDef.nullable
-    return `Array<${getType(typeDef.items, methodType, spec)}>${nullable === true ? ' | null' : ''}`
+    return `Array<${getType(typeDef.items, methodType, spec, seenRefs)}>${nullable === true ? ' | null' : ''}`
   }
   if (typeDef.enum) {
     // Note: null type represented with an enum have no types and single enum element 'null'
@@ -107,7 +112,7 @@ export function getType (typeDef, methodType, spec) {
       if (additionalPropsRequired) {
         required = required || !!additionalPropsRequired.includes(prop)
       }
-      return `'${prop}'${required ? '' : '?'}: ${getType(objProperties[prop], methodType, spec)}`
+      return `'${prop}'${required ? '' : '?'}: ${getType(objProperties[prop], methodType, spec, seenRefs)}`
     })
     if (additionalProps === true) {
       props.push('[key: string]: unknown')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "massimo-cli",
-  "version": "1.5.0",
+  "version": "1.5.2",
   "description": "A client for HTTP services.",
   "main": "index.js",
   "types": "index.d.ts",
@@ -25,7 +25,7 @@
     "graphql": "^16.8.1",
     "help-me": "^5.0.0",
     "jsonpointer": "^5.0.1",
-    "massimo": "1.5.0",
+    "massimo": "1.5.2",
     "minimist": "^1.2.8",
     "pino": "^9.9.0",
     "pino-pretty": "^13.0.0",
@@ -33,13 +33,13 @@
     "yaml": "^2.4.1"
   },
   "devDependencies": {
-    "@platformatic/composer": "3.67.0",
+    "@platformatic/composer": "3.68.0",
     "@platformatic/db": "3.67.0",
-    "@platformatic/foundation": "3.67.0",
-    "@platformatic/runtime": "3.67.0",
+    "@platformatic/foundation": "3.68.0",
+    "@platformatic/runtime": "3.68.0",
     "@platformatic/service": "3.67.0",
-    "@platformatic/sql-graphql": "3.67.0",
-    "@platformatic/sql-mapper": "3.67.0",
+    "@platformatic/sql-graphql": "3.68.0",
+    "@platformatic/sql-mapper": "3.68.0",
     "@playwright/test": "^1.42.1",
     "@types/react": "^19.1.6",
     "@types/react-dom": "^19.1.5",