apex-ruby 1.0.7 → 1.0.9

data/ext/apex_ext/apex_src/src/plugins.c

@@ -92,7 +92,8 @@ static char *apex_git_toplevel(void) {
     return out;
 }
 
-struct apex_plugin {
+
+typedef struct apex_plugin {
     char *id;
     char *title;
     char *author;
@@ -101,7 +102,8 @@ struct apex_plugin {
     char *repo;
     apex_plugin_phase_mask phases;
     int priority;
-    char *handler_command;
+
+    char *handler_command; /* External command to be executed */
     int timeout_ms;
     /* Declarative regex support */
     char *pattern;
@@ -112,14 +114,20 @@ struct apex_plugin {
     char *dir_path;
     /* Per-plugin support directory (used for APEX_SUPPORT_DIR) */
     char *support_dir;
+    /* A custom callback to transform the passed text. Executed only if handler_command and has_regex are not set. */
+    char *(*callback)(const char *text, const char *id_plugin, apex_plugin_phase_mask phase, const struct apex_options *options);
     struct apex_plugin *next;
-};
+} apex_plugin;
 
 struct apex_plugin_manager {
     struct apex_plugin *pre_parse;
     struct apex_plugin *post_render;
 };
 
+static apex_plugin *init_plugin(void) {
+    return calloc(1, sizeof(struct apex_plugin));
+}
+
 static void free_plugin(struct apex_plugin *p) {
     while (p) {
         struct apex_plugin *next = p->next;
@@ -137,6 +145,7 @@ static void free_plugin(struct apex_plugin *p) {
         if (p->has_regex) {
             regfree(&p->regex);
         }
+        p->callback = NULL;
         free(p);
         p = next;
     }
@@ -211,6 +220,31 @@ static bool plugin_id_exists(struct apex_plugin *head, const char *id) {
     return false;
 }
 
+bool apex_plugin_register(apex_plugin_manager *manager, const char *id, apex_plugin_phase_mask phase, char *(*callback)(const char *text,  const char *id_plugin, apex_plugin_phase_mask phase, const struct apex_options *options)) {
+    apex_plugin *plugin;
+    plugin = init_plugin();
+    if (!plugin || !manager) {
+        return false;
+    }
+
+    plugin->id = id != NULL ? strdup(id) : NULL;
+    plugin->callback = callback;
+    plugin->phases = phase;
+
+    if (plugin->phases & APEX_PLUGIN_PHASE_PRE_PARSE) {
+        if (!plugin_id_exists(manager->pre_parse, plugin->id)) {
+            append_plugin_sorted(&manager->pre_parse, plugin);
+            return true;
+        }
+    } else if (plugin->phases & APEX_PLUGIN_PHASE_POST_RENDER) {
+        if (!plugin_id_exists(manager->post_render, plugin->id)) {
+            append_plugin_sorted(&manager->post_render, plugin);
+            return true;
+        }
+    }
+    return false;
+}
+
 /* Determine base support directory for plugins, creating it if needed.
  * This follows XDG conventions: $XDG_CONFIG_HOME/apex/support or
  * $HOME/.config/apex/support.
@@ -355,7 +389,7 @@ static void load_plugins_from_dir(apex_plugin_manager *manager,
                     continue;
                 }
 
-                struct apex_plugin *p = calloc(1, sizeof(struct apex_plugin));
+                struct apex_plugin *p = init_plugin();
                 if (!p) {
                     apex_free_metadata(merged);
                     continue;
@@ -478,7 +512,7 @@ static void load_plugins_from_dir(apex_plugin_manager *manager,
         }
 
         const char *final_id = id ? id : ent->d_name;
-        struct apex_plugin *p = calloc(1, sizeof(struct apex_plugin));
+        struct apex_plugin *p = init_plugin();
         if (!p) {
             apex_free_metadata(meta);
             continue;
@@ -568,66 +602,72 @@ apex_plugin_manager *apex_plugins_load(const apex_options *options) {
     apex_plugin_manager *manager = calloc(1, sizeof(apex_plugin_manager));
     if (!manager) return NULL;
 
-    /* Project-scoped (current working directory): CWD/.apex/plugins */
-    char cwd[1024];
-    cwd[0] = '\0';
-    if (getcwd(cwd, sizeof(cwd)) != NULL && cwd[0] != '\0') {
-        char *cwd_proj_dir = dup_join(cwd, ".apex/plugins");
-        if (cwd_proj_dir) {
-            load_plugins_from_dir(manager, cwd_proj_dir);
-            free(cwd_proj_dir);
-        }
+    if (options->plugin_register) {
+        options->plugin_register(manager, options);
     }
 
-    /* Project-scoped (explicit base_directory): base_directory/.apex/plugins */
-    if (options->base_directory && options->base_directory[0] != '\0') {
-        char *proj_dir = dup_join(options->base_directory, ".apex/plugins");
-        if (proj_dir) {
-            load_plugins_from_dir(manager, proj_dir);
-            free(proj_dir);
+    if (options->allow_external_plugin_detection) {
+        /* Project-scoped (current working directory): CWD/.apex/plugins */
+        char cwd[1024];
+        cwd[0] = '\0';
+        if (getcwd(cwd, sizeof(cwd)) != NULL && cwd[0] != '\0') {
+            char *cwd_proj_dir = dup_join(cwd, ".apex/plugins");
+            if (cwd_proj_dir) {
+                load_plugins_from_dir(manager, cwd_proj_dir);
+                free(cwd_proj_dir);
+            }
+        }
+
+        /* Project-scoped (explicit base_directory): base_directory/.apex/plugins */
+        if (options->base_directory && options->base_directory[0] != '\0') {
+            char *proj_dir = dup_join(options->base_directory, ".apex/plugins");
+            if (proj_dir) {
+                load_plugins_from_dir(manager, proj_dir);
+                free(proj_dir);
+            }
         }
-    }
 
-    /* Project-scoped (Git repository root): <git top>/\.apex/plugins
-     * Only used when the current directory is inside the work tree.
-     */
-    char *git_root = apex_git_toplevel();
-    if (git_root && git_root[0] != '\0' && cwd[0] != '\0') {
-        size_t root_len = strlen(git_root);
-        /* Ensure the Git root is a parent of (or equal to) the current directory. */
-        if (strncmp(cwd, git_root, root_len) == 0 &&
-            (cwd[root_len] == '/' || cwd[root_len] == '\0')) {
-            /* Avoid re-loading if git_root is the same as base_directory. */
-            if (!options->base_directory ||
-                strcmp(git_root, options->base_directory) != 0) {
-                char *git_proj_dir = dup_join(git_root, ".apex/plugins");
-                if (git_proj_dir) {
-                    load_plugins_from_dir(manager, git_proj_dir);
-                    free(git_proj_dir);
+        /* Project-scoped (Git repository root): <git top>/\.apex/plugins
+         * Only used when the current directory is inside the work tree.
+         */
+        char *git_root = apex_git_toplevel();
+        if (git_root && git_root[0] != '\0' && cwd[0] != '\0') {
+            size_t root_len = strlen(git_root);
+            /* Ensure the Git root is a parent of (or equal to) the current directory. */
+            if (strncmp(cwd, git_root, root_len) == 0 &&
+                (cwd[root_len] == '/' || cwd[root_len] == '\0')) {
+                /* Avoid re-loading if git_root is the same as base_directory. */
+                if (!options->base_directory ||
+                    strcmp(git_root, options->base_directory) != 0) {
+                    char *git_proj_dir = dup_join(git_root, ".apex/plugins");
+                    if (git_proj_dir) {
+                        load_plugins_from_dir(manager, git_proj_dir);
+                        free(git_proj_dir);
+                    }
+                    }
                 }
-            }
+            free(git_root);
         }
-        free(git_root);
-    }
 
-    /* User-global: $XDG_CONFIG_HOME/apex/plugins or $HOME/.config/apex/plugins */
-    const char *xdg = getenv("XDG_CONFIG_HOME");
-    char *global_dir = NULL;
-    if (xdg && *xdg) {
-        char buf[1024];
-        snprintf(buf, sizeof(buf), "%s/apex/plugins", xdg);
-        global_dir = strdup(buf);
-    } else {
-        const char *home = getenv("HOME");
-        if (home && *home) {
+        /* User-global: $XDG_CONFIG_HOME/apex/plugins or $HOME/.config/apex/plugins */
+        const char *xdg = getenv("XDG_CONFIG_HOME");
+        char *global_dir = NULL;
+        if (xdg && *xdg) {
             char buf[1024];
-            snprintf(buf, sizeof(buf), "%s/.config/apex/plugins", home);
+            snprintf(buf, sizeof(buf), "%s/apex/plugins", xdg);
             global_dir = strdup(buf);
+        } else {
+            const char *home = getenv("HOME");
+            if (home && *home) {
+                char buf[1024];
+                snprintf(buf, sizeof(buf), "%s/.config/apex/plugins", home);
+                global_dir = strdup(buf);
+            }
+        }
+        if (global_dir) {
+            load_plugins_from_dir(manager, global_dir);
+            free(global_dir);
         }
-    }
-    if (global_dir) {
-        load_plugins_from_dir(manager, global_dir);
-        free(global_dir);
     }
 
     if (!manager->pre_parse && !manager->post_render) {
@@ -861,6 +901,8 @@ char *apex_plugins_run_text_phase(apex_plugin_manager *manager,
             }
         } else if (p->has_regex) {
             next = apply_regex_replacement(p, current);
+        } else if (p->callback) {
+            next = p->callback(current, p->id, phase, options);
         }
 
         if (do_profile) {

data/ext/apex_ext/apex_src/src/plugins.h

@@ -7,16 +7,6 @@
 extern "C" {
 #endif
 
-/* Plugin phases */
-typedef enum {
-    APEX_PLUGIN_PHASE_PRE_PARSE  = 1 << 0,
-    APEX_PLUGIN_PHASE_BLOCK      = 1 << 1,
-    APEX_PLUGIN_PHASE_INLINE     = 1 << 2,
-    APEX_PLUGIN_PHASE_POST_RENDER= 1 << 3
-} apex_plugin_phase_mask;
-
-typedef struct apex_plugin_manager apex_plugin_manager;
-
 /* Discover and load plugins from project and user config dirs.
  * Returns NULL if no plugins are found or an error occurs. */
 apex_plugin_manager *apex_plugins_load(const apex_options *options);

data/ext/apex_ext/apex_src/src/pretty_html.c

@@ -56,7 +56,7 @@ static char *extract_tag_name(const char *tag_start, bool *is_closing, bool *is_
     const char *name_start = p;
     while (*p && !isspace((unsigned char)*p) && *p != '>' && *p != '/') p++;
 
-    int len = p - name_start;
+    size_t len = (size_t)(p - name_start);
     if (len == 0) return NULL;
 
     char *name = malloc(len + 1);

data/ext/apex_ext/apex_src/tests/fixtures/metadata/mmd-metadata.md

@@ -0,0 +1,5 @@
+Title: A MultiMarkdown Test
+Author: Brett Terpstra
+Random Key: Bargle
+
+This is a test of MultiMarkdown metadata. [%randomkey]

data/ext/apex_ext/apex_src/tests/fixtures/metadata/pandoc-meta.md

@@ -0,0 +1,4 @@
+% Pandoc Metadata
+% Brett Terpstra
+
+This is a test of Pandoc metadata.

data/ext/apex_ext/apex_src/tests/fixtures/metadata/yaml-frontmatter.md

@@ -0,0 +1,6 @@
+---
+title: A YAML test
+author: Brett Terpstra
+random_key: Flargle
+---
+This is a test of YAML Front Matter. [%randomkey]

data/ext/apex_ext/apex_src/tests/metadata_cli_test.sh

@@ -0,0 +1,119 @@
+#!/usr/bin/env bash
+
+# CLI tests for --info, --extract-meta, and --extract-meta-value using tests/fixtures/metadata.
+
+set -euo pipefail
+
+ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
+APEX_BIN="${APEX_BIN:-$ROOT/build/apex}"
+FIXTURES="$ROOT/tests/fixtures/metadata"
+YAML="$FIXTURES/yaml-frontmatter.md"
+MMD="$FIXTURES/mmd-metadata.md"
+PANDOC="$FIXTURES/pandoc-meta.md"
+
+if [[ ! -x "$APEX_BIN" ]]; then
+	echo "Error: $APEX_BIN not found or not executable. Build Apex first (cmake --build build --target apex_cli)." >&2
+	exit 1
+fi
+
+for f in "$YAML" "$MMD" "$PANDOC"; do
+	if [[ ! -f "$f" ]]; then
+		echo "Error: missing fixture $f" >&2
+		exit 1
+	fi
+done
+
+TMPDIR="${TMPDIR:-/tmp}"
+WORK="$(mktemp -d "${TMPDIR}/apex-metadata-cli-XXXXXX")"
+trap 'rm -rf "$WORK"' EXIT
+
+echo "== Testing -i / --info (no input files; info on stdout) =="
+
+INFO_OUT="$WORK/info_stdout.txt"
+# Close stdin so the CLI does not wait for interactive input in some environments.
+"$APEX_BIN" -i </dev/null >"$INFO_OUT"
+grep -q '^version:' "$INFO_OUT"
+grep -q 'plugins:' "$INFO_OUT"
+echo "-i without files: stdout contains version and plugins section."
+
+echo
+echo "== Testing -i with a file (info on stderr, HTML on stdout) =="
+
+STDOUT="$WORK/out.html"
+STDERR="$WORK/info.err"
+"$APEX_BIN" -i "$YAML" >"$STDOUT" 2>"$STDERR"
+grep -q '^version:' "$STDERR" || {
+	echo "metadata_cli_test: expected version line on stderr when using -i with a file" >&2
+	exit 1
+}
+grep -q '<p>' "$STDOUT" || {
+	echo "metadata_cli_test: expected HTML fragment on stdout with -i and a file" >&2
+	exit 1
+}
+echo "-i with file: version/config on stderr, conversion on stdout."
+
+echo
+echo "== Testing --extract-meta (YAML fixture) =="
+
+META_OUT="$WORK/meta.yaml"
+"$APEX_BIN" --extract-meta "$YAML" >"$META_OUT"
+grep -q '^title: A YAML test$' "$META_OUT" || {
+	echo "metadata_cli_test: --extract-meta missing expected title from YAML fixture" >&2
+	exit 1
+}
+grep -q '^random_key: Flargle$' "$META_OUT" || {
+	echo "metadata_cli_test: --extract-meta missing random_key from YAML fixture" >&2
+	exit 1
+}
+echo "--extract-meta on YAML front matter: expected keys present."
+
+echo
+echo "== Testing -e / --extract-meta-value =="
+
+val="$("$APEX_BIN" -e title "$YAML")"
+if [[ "$val" != "A YAML test" ]]; then
+	echo "metadata_cli_test: -e title from YAML expected 'A YAML test', got '$val'" >&2
+	exit 1
+fi
+
+val="$("$APEX_BIN" --extract-meta-value random_key "$YAML")"
+if [[ "$val" != "Flargle" ]]; then
+	echo "metadata_cli_test: -e random_key expected 'Flargle', got '$val'" >&2
+	exit 1
+fi
+
+val="$("$APEX_BIN" -e randomkey "$MMD")"
+if [[ "$val" != "Bargle" ]]; then
+	echo "metadata_cli_test: -e randomkey (MMD) expected 'Bargle', got '$val'" >&2
+	exit 1
+fi
+
+val="$("$APEX_BIN" -e title "$PANDOC")"
+if [[ "$val" != "Pandoc Metadata" ]]; then
+	echo "metadata_cli_test: -e title (Pandoc) expected 'Pandoc Metadata', got '$val'" >&2
+	exit 1
+fi
+
+if "$APEX_BIN" -e definitely_missing_key_xyz "$YAML" 2>"$WORK/miss.err"; then
+	echo "metadata_cli_test: expected non-zero exit for missing metadata key" >&2
+	exit 1
+fi
+grep -q "not found" "$WORK/miss.err" || {
+	echo "metadata_cli_test: expected error message for missing key on stderr" >&2
+	exit 1
+}
+echo "-e returns values and exit 1 when key is absent."
+
+echo
+echo "== Testing --combine --extract-meta (merge order, later file wins) =="
+
+COMBINED="$WORK/combined_meta.yaml"
+"$APEX_BIN" --combine --extract-meta "$MMD" "$YAML" >"$COMBINED"
+grep -q '^title: A YAML test$' "$COMBINED" || {
+	echo "metadata_cli_test: combined extract expected last file to win for title" >&2
+	exit 1
+}
+echo "--combine --extract-meta: later file overrides title as expected."
+
+echo
+echo "All metadata CLI tests passed."

data/ext/apex_ext/apex_src/tests/test_custom_plugins.c

@@ -0,0 +1,78 @@
+//
+// Created by Sbarex on 10/02/26.
+//
+
+#include "test_helpers.h"
+#include "apex/apex.h"
+#include <stdlib.h>
+
+
+/* cmark-gfm headers */
+#include "cmark-gfm.h"
+#include <string.h>
+
+static char * my_plugin_callback(const char *text, __attribute__((unused)) const char *id_plugin, apex_plugin_phase_mask phase_mask, __attribute__((unused)) const apex_options *options) {
+    if (phase_mask & APEX_PLUGIN_PHASE_PRE_PARSE) {
+        const char *suffix = "\n_Hello Sbarex_\n";
+        size_t len1 = strlen(text);
+        size_t len2 = strlen(suffix);
+
+        char *result = malloc(len1 + len2 + 1);
+        if (!result) return NULL;
+
+        memcpy(result, text, len1);
+        memcpy(result + len1, suffix, len2 + 1); // include '\0'
+        return result;
+    } else if (phase_mask & APEX_PLUGIN_PHASE_POST_RENDER) {
+        const char *suffix = "\n<p>Everything is fine</p>";
+        size_t len1 = strlen(text);
+        size_t len2 = strlen(suffix);
+
+        char *result = malloc(len1 + len2 + 1);
+        if (!result) return NULL;
+
+        memcpy(result, text, len1);
+        memcpy(result + len1, suffix, len2 + 1);
+        return result;
+    }
+    return NULL;
+}
+
+static void my_plugin_register(apex_plugin_manager *manager, __attribute__((unused)) const apex_options *options) {
+    printf(COLOR_GREEN "✓" COLOR_RESET " Custom plugin register callback called\n");
+
+    /* Attach to appropriate phase lists, enforcing per-list id uniqueness */
+    if (apex_plugin_register(manager, "my_plugin", APEX_PLUGIN_PHASE_PRE_PARSE, my_plugin_callback)) {
+        printf(COLOR_GREEN "✓" COLOR_RESET " Custom plugin has been registered for pre parse\n");
+    } else {
+        printf(COLOR_RED "✗" COLOR_RESET " Custom plugin has not been registered for pre parse\n");
+    }
+
+    /* Attach to appropriate phase lists, enforcing per-list id uniqueness */
+    if (apex_plugin_register(manager, "my_plugin", APEX_PLUGIN_PHASE_POST_RENDER, my_plugin_callback)) {
+        printf(COLOR_GREEN "✓" COLOR_RESET " Custom plugin has been registered for post render\n");
+    } else {
+        printf(COLOR_RED "✗" COLOR_RESET " Custom plugin has not been registered for post render\n");
+    }
+}
+
+void test_custom_plugins(void) {
+    int suite_failures = suite_start();
+    print_suite_title("Custom plugins Tests", false, true);
+
+    apex_options opts = apex_options_default();
+    opts.enable_plugins = true;
+    opts.allow_external_plugin_detection = false;
+    opts.plugin_register = my_plugin_register;
+
+    const char *s = "# Custom plugin callback test\n\n";
+
+    char *html;
+    html = apex_markdown_to_html(s, strlen(s), &opts);
+    assert_contains(html, "<em>Hello Sbarex</em>", "Custom pre parse plugin executed");
+    assert_contains(html, "<p>Everything is fine</p>", "Custom post render plugin executed");
+    apex_free_string(html);
+
+    bool had_failures = suite_end(suite_failures);
+    print_suite_title("Cmark Callbacks Tests", had_failures, false);
+}

data/ext/apex_ext/apex_src/tests/test_extensions.c

@@ -2246,6 +2246,33 @@ void test_mixed_lists(void) {
     }
     apex_free_string(html);
 
+    /* Regression: alpha lists with nested sublists should stay intact and not leak marker tokens */
+    unified_opts = apex_options_for_mode(APEX_MODE_UNIFIED);
+    const char *alpha_with_nested_bullets = "a. Test\nb. Test\n\t- Test\n\t- Test\nc. Test\n";
+    html = apex_markdown_to_html(alpha_with_nested_bullets, strlen(alpha_with_nested_bullets), &unified_opts);
+    assert_contains(html, "<ol style=\"list-style-type: lower-alpha\">", "Alpha list keeps lower-alpha style with nested bullets");
+    assert_contains(html, "<ul>", "Nested bullet list rendered");
+    assert_not_contains(html, "[apex-alpha-list:", "Alpha marker token removed from output");
+    apex_free_string(html);
+
+    const char *alpha_with_nested_ordered = "a. Test\nb. Test\n\t1. Nested one\n\t2. Nested two\nc. Test\n";
+    html = apex_markdown_to_html(alpha_with_nested_ordered, strlen(alpha_with_nested_ordered), &unified_opts);
+    assert_contains(html, "<ol style=\"list-style-type: lower-alpha\">", "Alpha list keeps style with nested ordered list");
+    assert_not_contains(html, "[apex-alpha-list:", "No leaked alpha marker token after nested ordered list");
+    apex_free_string(html);
+
+    const char *alpha_with_nested_alpha = "a. Test\nb. Test\n\tc. Test\n\td. Test\ne. Test\n";
+    html = apex_markdown_to_html(alpha_with_nested_alpha, strlen(alpha_with_nested_alpha), &unified_opts);
+    assert_contains(html, "<ol style=\"list-style-type: lower-alpha\">", "Alpha list keeps style with nested alpha sublist");
+    assert_not_contains(html, "[apex-alpha-list:", "No leaked alpha marker token after nested alpha sublist");
+    apex_free_string(html);
+
+    const char *numeric_with_nested_ordered = "1. Test\n2. Test\n\t3. Test\n\t4. Test\n5. Test\n";
+    html = apex_markdown_to_html(numeric_with_nested_ordered, strlen(numeric_with_nested_ordered), &unified_opts);
+    assert_contains(html, "<ol start=\"3\">", "Numeric nested ordered sublist renders as nested ordered list");
+    assert_not_contains(html, "2. Test\n    3. Test", "Nested ordered items are not flattened into parent text");
+    apex_free_string(html);
+
     bool had_failures = suite_end(suite_failures);
     print_suite_title("Mixed List Markers Tests", had_failures, false);
 }

data/ext/apex_ext/apex_src/tests/test_metadata.c

@@ -7,6 +7,8 @@
 #include "../src/extensions/metadata.h"
 #include <string.h>
 #include <stdlib.h>
+#include <stdio.h>
+#include <assert.h>
 
 void test_metadata(void) {
     int suite_failures = suite_start();
@@ -96,6 +98,46 @@ void test_metadata(void) {
     print_suite_title("Metadata Tests", had_failures, false);
 }
 
+/**
+ * YAML serialization helpers used by the CLI
+ */
+void test_metadata_yaml_emit(void) {
+    int suite_failures = suite_start();
+    print_suite_title("Metadata YAML emit Tests", false, true);
+
+    apex_metadata_item *m = apex_parse_command_metadata("title=Hello World");
+    assert(m != NULL);
+    FILE *fp = tmpfile();
+    assert(fp != NULL);
+    apex_metadata_fprint_yaml_document(fp, m);
+    rewind(fp);
+    char buf[800];
+    size_t n = fread(buf, 1, sizeof(buf) - 1, fp);
+    buf[n] = '\0';
+    fclose(fp);
+    assert_contains(buf, "---", "yaml document has markers");
+    assert_contains(buf, "title:", "yaml title key");
+    assert_contains(buf, "Hello World", "yaml title value");
+    apex_free_metadata(m);
+
+    apex_metadata_item *a = apex_parse_command_metadata("x=1");
+    apex_metadata_item *b = apex_parse_command_metadata("x=2");
+    apex_metadata_item *mg = apex_merge_metadata(a, b, NULL);
+    apex_free_metadata(a);
+    apex_free_metadata(b);
+    fp = tmpfile();
+    apex_metadata_fprint_yaml_mapping(fp, mg);
+    rewind(fp);
+    n = fread(buf, 1, sizeof(buf) - 1, fp);
+    buf[n] = '\0';
+    fclose(fp);
+    assert_contains(buf, "x: 2", "merge: later metadata wins for yaml mapping");
+    apex_free_metadata(mg);
+
+    bool had_failures = suite_end(suite_failures);
+    print_suite_title("Metadata YAML emit Tests", had_failures, false);
+}
+
 /**
  * Test MultiMarkdown metadata keys
  */

data/ext/apex_ext/apex_src/tests/test_output.c

@@ -34,6 +34,13 @@ void test_toc(void) {
     assert_contains(html, "Section", "MMD TOC includes headers");
     apex_free_string(html);
 
+    /* MultiMarkdown mode should process {{TOC}} even with marked extensions disabled */
+    apex_options mmd_opts = apex_options_for_mode(APEX_MODE_MULTIMARKDOWN);
+    html = apex_markdown_to_html(mmd_toc, strlen(mmd_toc), &mmd_opts);
+    assert_contains(html, "<nav class=\"toc\">", "MMD mode renders TOC marker");
+    assert_contains(html, "Section", "MMD mode TOC includes headers");
+    apex_free_string(html);
+
     /* Test TOC with depth range */
     const char *depth_toc = "# H1\n\n{{TOC:2-3}}\n\n## H2\n\n### H3\n\n#### H4";
     html = apex_markdown_to_html(depth_toc, strlen(depth_toc), &opts);
@@ -390,6 +397,82 @@ void test_terminal_output(void) {
     test_result(out != NULL && strstr(out, "plain") != NULL, "terminal_width set still produces terminal output");
     if (out) apex_free_string(out);
 
+    /* apex_resolve_local_image_path */
+    {
+        char *rp = apex_resolve_local_image_path("img/a.png", "/tmp/proj");
+        test_result(rp != NULL && strcmp(rp, "/tmp/proj/img/a.png") == 0,
+                    "apex_resolve_local_image_path joins base_directory");
+        free(rp);
+        rp = apex_resolve_local_image_path("/abs/foo.png", "/any");
+        test_result(rp != NULL && strcmp(rp, "/abs/foo.png") == 0,
+                    "apex_resolve_local_image_path keeps absolute paths");
+        free(rp);
+    }
+
+    /* Terminal images: inline rendering uses isatty(STDOUT_FILENO). Test output is
+     * captured to a string; under a normal CI pipe or non-TTY, stdout is not a TTY
+     * so these exercises never call curl or imgcat/chafa/viu/catimg. (Running the
+     * suite in an interactive terminal with those tools on PATH could take a
+     * different path for remote URLs.) */
+
+    /* Remote images: link-style fallback when not inline (non-TTY: no download/viewer) */
+    opts = apex_options_default();
+    opts.output_format = APEX_OUTPUT_TERMINAL;
+    out = apex_markdown_to_html("![r](https://ex.com/x.png)", 28, &opts);
+    assert_contains(out, "https://ex.com/x.png", "Remote image URL in fallback");
+    test_result(strstr(out, "![") == NULL, "Remote image fallback uses link style not markdown image");
+    apex_free_string(out);
+
+    /* http:// same as https for fallback */
+    opts = apex_options_default();
+    opts.output_format = APEX_OUTPUT_TERMINAL;
+    out = apex_markdown_to_html("![h](http://ex.com/y.png)", 26, &opts);
+    assert_contains(out, "http://ex.com/y.png", "http remote image URL in fallback");
+    test_result(strstr(out, "![") == NULL, "http remote image uses link style not markdown image");
+    apex_free_string(out);
+
+    /* terminal256: same link-style fallback for images */
+    opts = apex_options_default();
+    opts.output_format = APEX_OUTPUT_TERMINAL256;
+    out = apex_markdown_to_html("![z](https://ex.com/z.png)", 28, &opts);
+    assert_contains(out, "https://ex.com/z.png", "terminal256 remote image URL in fallback");
+    test_result(strstr(out, "![") == NULL, "terminal256 remote image uses link style");
+    apex_free_string(out);
+
+    /* Empty alt still emits (url) */
+    opts = apex_options_default();
+    opts.output_format = APEX_OUTPUT_TERMINAL;
+    out = apex_markdown_to_html("![](https://ex.com/e.png)", 25, &opts);
+    assert_contains(out, "https://ex.com/e.png", "Empty-alt remote image URL in fallback");
+    test_result(strstr(out, "![") == NULL, "Empty-alt image uses link style");
+    apex_free_string(out);
+
+    /* terminal_image_width is ignored when not inlining (non-TTY); must not crash */
+    opts = apex_options_default();
+    opts.output_format = APEX_OUTPUT_TERMINAL;
+    opts.terminal_image_width = 72;
+    out = apex_markdown_to_html("![w](https://ex.com/w.png)", 28, &opts);
+    test_result(out != NULL && strstr(out, "https://ex.com/w.png") != NULL,
+                "terminal_image_width set with non-TTY still yields link-style remote image");
+    apex_free_string(out);
+
+    /* terminal_inline_images false: link-style fallback */
+    opts = apex_options_default();
+    opts.output_format = APEX_OUTPUT_TERMINAL;
+    opts.terminal_inline_images = false;
+    out = apex_markdown_to_html("![z](local.png)", 17, &opts);
+    assert_contains(out, "local.png", "Disabled inline: URL shown like a link");
+    test_result(strstr(out, "![") == NULL, "terminal_inline_images off uses link style not markdown image");
+    apex_free_string(out);
+
+    /* Missing local file: link-style fallback (non-TTY) */
+    opts = apex_options_default();
+    opts.output_format = APEX_OUTPUT_TERMINAL;
+    out = apex_markdown_to_html("![missing](_/apex_test_missing_image_99.png)", 45, &opts);
+    assert_contains(out, "apex_test_missing_image_99.png", "Missing file: URL in link-style output");
+    test_result(strstr(out, "![") == NULL, "Missing local image uses link style not markdown image");
+    apex_free_string(out);
+
     bool had_failures = suite_end(suite_failures);
     print_suite_title("Terminal Output Tests", had_failures, false);
 }