fix: update documentation links and remove test comment in main.tf

Author: virgofx

__tests__/utils/string.test.ts

@@ -176,5 +176,68 @@ describe('utils/string', () => {
       const result = renderTemplate(template, variables);
       expect(result).toBe('Item first and second');
     });
+
+    it('should handle undefined values by leaving placeholders unchanged', () => {
+      const template = 'Hello, {{name}} and {{greeting}}!';
+      const variables = { name: 'World', greeting: undefined };
+      const result = renderTemplate(template, variables);
+      expect(result).toBe('Hello, World and {{greeting}}!');
+    });
+
+    it('should handle all undefined values', () => {
+      const template = '{{greeting}}, {{name}}!';
+      const variables = { greeting: undefined, name: undefined };
+      const result = renderTemplate(template, variables);
+      expect(result).toBe('{{greeting}}, {{name}}!');
+    });
+
+    it('should handle mixed defined and undefined values', () => {
+      const template = 'Module: {{module}}, Version: {{version}}, Author: {{author}}';
+      const variables = { module: 'vpc-endpoint', version: undefined, author: 'TechPivot' };
+      const result = renderTemplate(template, variables);
+      expect(result).toBe('Module: vpc-endpoint, Version: {{version}}, Author: TechPivot');
+    });
+
+    it('should handle empty string vs undefined distinction', () => {
+      const template = 'A{{key1}}B{{key2}}C';
+      const variables = { key1: '', key2: undefined };
+      const result = renderTemplate(template, variables);
+      expect(result).toBe('AB{{key2}}C');
+    });
+
+    it('should handle undefined values in complex templates', () => {
+      const template = 'Path: {{path}}, Command: {{cmd}}, Options: {{opts}}';
+      const variables = { path: '/opt/bin/terraform', cmd: undefined, opts: '--verbose' };
+      const result = renderTemplate(template, variables);
+      expect(result).toBe('Path: /opt/bin/terraform, Command: {{cmd}}, Options: --verbose');
+    });
+
+    it('should handle null values by leaving placeholders unchanged', () => {
+      const template = 'Hello, {{name}} and {{greeting}}!';
+      const variables = { name: 'World', greeting: null };
+      const result = renderTemplate(template, variables);
+      expect(result).toBe('Hello, World and {{greeting}}!');
+    });
+
+    it('should handle all null values', () => {
+      const template = '{{greeting}}, {{name}}!';
+      const variables = { greeting: null, name: null };
+      const result = renderTemplate(template, variables);
+      expect(result).toBe('{{greeting}}, {{name}}!');
+    });
+
+    it('should handle mixed null, undefined, and defined values', () => {
+      const template = 'Module: {{module}}, Version: {{version}}, Author: {{author}}, License: {{license}}';
+      const variables = { module: 'vpc-endpoint', version: null, author: 'TechPivot', license: undefined };
+      const result = renderTemplate(template, variables);
+      expect(result).toBe('Module: vpc-endpoint, Version: {{version}}, Author: TechPivot, License: {{license}}');
+    });
+
+    it('should handle empty string vs null vs undefined distinction', () => {
+      const template = 'A{{key1}}B{{key2}}C{{key3}}D';
+      const variables = { key1: '', key2: null, key3: undefined };
+      const result = renderTemplate(template, variables);
+      expect(result).toBe('AB{{key2}}C{{key3}}D');
+    });
   });
 });

src/types/metadata.types.ts

@@ -14,7 +14,7 @@ import type { Config } from '@/types/config.types';
  * comprehensive mapping of all action inputs, which then drives the automatic
  * config generation process.
  *
- * @see {@link /workspaces/terraform-module-releaser/src/utils/metadata.ts} for usage
+ * @see utils/metadata.ts for usage
  * @see {@link https://docs.github.com/en/actions/reference/metadata-syntax-for-github-actions#inputs} GitHub Actions input reference
  */
 export interface ActionInputMetadata {

src/utils/constants.ts

@@ -10,7 +10,7 @@
  * - `modules_aws_s3_bucket_v1.0.0` (using '_')
  * - `modules.aws.s3.bucket.v1.0.0` (using '.')
  *
- * The default separator is '/' as defined in action.yml.dddd
+ * The default separator is '/' as defined in action.yml.
  */
 export const VALID_TAG_DIRECTORY_SEPARATORS = ['-', '_', '/', '.'];
 

src/utils/string.ts

@@ -61,6 +61,7 @@ export function removeLeadingCharacters(input: string, charactersToRemove: strin
  *
  * @param template The template string containing placeholders in the format `{{key}}`.
  * @param variables An object where keys correspond to placeholder names and values are their replacements.
+ *                   If a value is undefined or null, the placeholder will be left unchanged.
  * @returns The rendered string with placeholders replaced.
  *
  * @example
@@ -70,9 +71,18 @@ export function removeLeadingCharacters(input: string, charactersToRemove: strin
  * @example
  * // Returns "Hi, There!"
  * renderTemplate("{{greeting}}, {{name}}!", { greeting: "Hi", name: "There" })
+ *
+ * @example
+ * // Returns "Hello, {{name}}!" (undefined value leaves placeholder unchanged)
+ * renderTemplate("Hello, {{name}}!", { name: undefined })
+ *
+ * @example
+ * // Returns "Hello, {{name}}!" (null value leaves placeholder unchanged)
+ * renderTemplate("Hello, {{name}}!", { name: null })
  */
-export function renderTemplate(template: string, variables: Record<string, string>): string {
+export function renderTemplate(template: string, variables: Record<string, string | undefined | null>): string {
   return template.replace(/\{\{(\w+)\}\}/g, (placeholder, key) => {
-    return key in variables ? variables[key] : placeholder;
+    const value = variables[key];
+    return value !== undefined && value !== null ? value : placeholder;
   });
 }

src/wiki.ts

@@ -309,8 +309,8 @@ async function generateWikiTerraformModule(terraformModule: TerraformModule): Pr
 
   const usage = renderTemplate(config.wikiUsageTemplate, {
     module_name: terraformModule.name,
-    latest_tag: terraformModule.getLatestTag() ?? '',
-    latest_tag_version_number: terraformModule.getLatestTagVersionNumber() ?? '',
+    latest_tag: terraformModule.getLatestTag(),
+    latest_tag_version_number: terraformModule.getLatestTagVersionNumber(),
     module_source: moduleSource,
     module_name_terraform: terraformModule.name.replace(/[^a-zA-Z0-9]/g, '_').toLowerCase(),
   });

tf-modules/vpc-endpoint/main.tf

@@ -1,5 +1,4 @@
 # Noop
-# Test
 
 locals {
   endpoints = { for k, v in var.endpoints : k => v }