feat: Fix issues in generated headers

languages/cpp/templates/declarations/default.c

@@ -1,6 +1,6 @@
-        /* 
-        ${method.name}
-        ${method.description}
-        ${method.params.annotations}${if.deprecated} * @deprecated ${method.deprecation}${end.if.deprecated}
-        */
-        virtual int32_t ${method.Name}( ${method.signature.params}${if.result}${if.params}, ${end.if.params}${method.result.type}& ${method.result.name}${end.if.result} ) = 0;
+    /* 
+    ${method.name}
+    ${method.description}
+    ${method.params.annotations}${if.deprecated} * @deprecated ${method.deprecation}${end.if.deprecated}
+    */
+    virtual int32_t ${method.Name}( ${method.signature.params}${if.result}${if.params}, ${end.if.params}${method.result.type}& ${method.result.name}${end.if.result} ) = 0;

languages/cpp/templates/declarations/event.c

@@ -1,10 +1,8 @@
-        /* ${method.name} - ${method.description} */
-        struct I${method.Name}Notification {
-            virtual void ${method.Name} ( ${event.signature.callback.params}${if.event.params}, ${end.if.event.params}${method.result.properties} ) = 0;  
-        };
-
-        virtual int32_t subscribe ( ${event.signature.params}${if.event.params}, ${end.if.event.params}I${method.Name}Notification& notification ) = 0;
-        virtual int32_t unsubscribe ( I${method.Name}Notification& notification ) = 0;
-
-        // signature callback params: ${event.signature.callback.params}
-        // method result properties : ${method.result.properties}
+    /* ${method.name} - ${method.description} */
+    struct I${method.Name}Notification {
+        virtual void ${method.Name} ( ${event.signature.callback.params}${if.event.params}, ${end.if.event.params}${method.result.properties} ) = 0;  
+    };
+    // signature callback params: ${event.signature.callback.params}
+    // method result properties : ${method.result.properties}
+    virtual int32_t subscribe ( ${event.signature.params}${if.event.params}, ${end.if.event.params}I${method.Name}Notification& notification ) = 0;
+    virtual int32_t unsubscribe ( I${method.Name}Notification& notification ) = 0;

languages/cpp/templates/declarations/polymorphic-pull-event.c

@@ -1,8 +1,8 @@
-/* ${method.name} - ${method.description} */
-struct I{method.Name}Notification {
-    virtual void ${method.Name} ( ${method.pulls.param.type}& value ) = 0;  
-};
-        
-virtual int32_t subscribe ( I{method.Name}Notification& notification ) = 0;
-virtual int32_t unsubscribe ( I{method.Name}Notification& notification ) = 0;
+    /* ${method.name} - ${method.description} */
+    struct I{method.Name}Notification {
+        virtual void ${method.Name} ( ${method.pulls.param.type}& value ) = 0;  
+    };
+
+    virtual int32_t subscribe ( I{method.Name}Notification& notification ) = 0;
+    virtual int32_t unsubscribe ( I{method.Name}Notification& notification ) = 0;
 

languages/cpp/templates/declarations/polymorphic-pull.c

@@ -1,2 +1,2 @@
-/* ${method.name} - ${method.description} */
-virtual int32_t ${method.Name}( ${method.signature.params} ) = 0;
+    /* ${method.name} - ${method.description} */
+    virtual int32_t ${method.Name}( ${method.signature.params} ) = 0;

languages/cpp/templates/declarations/polymorphic-reducer.c

@@ -1,5 +1,5 @@
-/*
- * ${method.summary}
- * ${method.params}
- */
-virtual int32_t ${method.Name}(${method.signature.params}${if.result.properties}${if.params}, ${end.if.params}${end.if.result.properties}${method.result.properties}) = 0;
+    /*
+    * ${method.summary}
+    * ${method.params}
+    */
+    virtual int32_t ${method.Name}(${method.signature.params}${if.result.properties}${if.params}, ${end.if.params}${end.if.result.properties}${method.result.properties}) = 0;

languages/cpp/templates/declarations/property.h

@@ -1,5 +1,5 @@
-        /*
-        * ${method.description}
-        * ${method.params}
-        */
-        virtual int32_t ${method.Name} (${method.signature.params}${if.params}, ${end.if.params}${method.result.properties}) = 0;
+    /*
+    * ${method.description}
+    * ${method.params}
+    */
+virtual int32_t ${method.Name} (${method.signature.params}${if.params}, ${end.if.params}${method.result.properties}) = 0;

languages/cpp/templates/declarations/setter.h

@@ -1,5 +1,5 @@
-        /*
-        ${method.rpc.name}
-        ${method.description}
-        */
-        virtual int32_t ${method.Name}( ${method.signature.params} ) = 0;
+    /*
+    ${method.rpc.name}
+    ${method.description}
+    */
+    virtual int32_t ${method.Name}( ${method.signature.params} ) = 0;

languages/cpp/templates/modules/include/module.h

@@ -25,19 +25,19 @@
 namespace ${info.Title} {
 
 // Enums
-/* ${ENUMS} */
+    /* ${ENUMS} */
 
 // Types
 /* ${TYPES} */
 
-    struct I${info.Title} {
+struct I${info.Title} {
 
-        virtual ~I${info.Title} = default;
+    virtual ~I${info.Title} = default;
 
-    // Methods & Events
-    /* ${DECLARATIONS} */
+// Methods & Events
+/* ${DECLARATIONS} */
 
-    };
+};
 
 } //namespace ${info.Title}
 }${end.if.declarations}

languages/cpp/templates/types/additionalProperties.h

@@ -1 +1 @@
-std::unordered_map<std::string,${type}> title;
+${title} value;

languages/cpp/templates/types/array.h

@@ -1 +1 @@
-std::vector<${type}> ${title};
+std::vector<${type}>

languages/cpp/templates/types/null.h

@@ -1 +0,0 @@
-nullptr

languages/cpp/templates/types/property.h

@@ -1 +1 @@
-${if.optional}std::optional<${end.if.optional}${title}${if.optional}>${end.if.optional} ${property}; // ${summary}
+${if.optional}std::optional<${end.if.optional}${title}${if.optional}>${end.if.optional} ${property};

languages/cpp/templates/types/tuple.h

@@ -1,6 +1,2 @@
 /* ${title} */
-struct ${title} {
-
-    ${properties}
-
-};
+std::pair<${type},${type}> ${title};

src/macrofier/engine.mjs

@@ -376,7 +376,7 @@ const promoteSchema = (location, property, title, document, destinationPath) =>
 }
 
 // only consider sub-objects and sub enums to be sub-schemas
-const isSubSchema = (schema) => schema.type === 'object' || (schema.type === 'string' && schema.enum) || (schema.type === 'array' && schema.items)
+const isSubSchema = (schema) => schema.type === 'object' || (schema.type === 'string' && schema.enum) // || (schema.type === 'array' && schema.items)
 
 const promoteAndNameSubSchemas = (obj) => {
   // make a copy so we don't polute our inputs
@@ -506,7 +506,7 @@ const generateMacros = (obj, templates, languages, options = {}) => {
     macros.types[dir] = getTemplate('/sections/types', templates).replace(/\$\{schema.list\}/g, schemasArray.filter(x => !x.enum).map(s => s.body).filter(body => body).join('\n'))
     macros.enums[dir] = getTemplate('/sections/enums', templates).replace(/\$\{schema.list\}/g, schemasArray.filter(x => x.enum).map(s => s.body).filter(body => body).join('\n'))
   })
-
+  state.typeTemplateDir = 'types'
   const examples = generateExamples(obj, templates, languages)
   const allMethodsArray = generateMethods(obj, examples, templates)
   const methodsArray = allMethodsArray.filter(m => m.body && !m.event && (!options.hideExcluded || !m.excluded))
@@ -1483,7 +1483,7 @@ function generateResultParams(result, json, templates, {name = ''}={}) {
     const template = getTemplate('/parameters/result', templates)
     return Object.entries(result.properties).map( ([name, type]) => template
                                                                       .replace(/\$\{method\.param\.name\}/g, name)
-                                                                      .replace(/\$\{method\.param\.type\}/g, types.getSchemaType(type, json, {name: name, moduleTitle: moduleTitle, result: true}))
+                                                                      .replace(/\$\{method\.param\.type\}/g, types.getSchemaType(type, json, {name: name, moduleTitle: moduleTitle, result: true, namespace: true}))
     ).join(', ') // most languages separate params w/ a comma, so leaving this here for now  
   }
   // tuples get unwrapped
@@ -1492,14 +1492,14 @@ function generateResultParams(result, json, templates, {name = ''}={}) {
     const template = getTemplate('/parameters/result', templates)
     return result.items.map( (type) => template
                                         .replace(/\$\{method\.param\.name\}/g, type['x-property'])
-                                        .replace(/\$\{method\.param\.type\}/g, types.getSchemaType(type, json, {name: name, moduleTitle: moduleTitle, result: true}))
+                                        .replace(/\$\{method\.param\.type\}/g, types.getSchemaType(type, json, {name: name, moduleTitle: moduleTitle, result: true, namespace: true}))
     ).join(', ')
   }
   // everything else is just output as-is
   else {
-    const template = getTemplate('/parameters/result', templates)
-    const type = types.getSchemaType(result, json, {name: name, moduleTitle: moduleTitle, result: true})
 
+    const template = getTemplate('/parameters/result', templates)
+    const type = types.getSchemaType(result, json, {name: name, moduleTitle: moduleTitle, result: true, namespace: true})
     if (type === 'undefined') {
       console.log(`Warning: undefined type for ${name}`)
     }