Add a `--configuration` option to the index binary #628

augmentcode · 2026-02-03T14:25:03Z

--configuration returns before the --url override block later in index_main, so invocations like ... --url <...> --configuration will print the non-overridden URL (and other derived values) rather than the effective configuration; the Makefile now calls this combination in sandbox-index. Is that the intended behavior for the new option?

Severity: medium

_{🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.}

cubic-dev-ai · 2026-02-03T14:25:32Z

P2: --configuration should be a read-only operation, but it still constructs Output before returning, which creates/scans the output directory. Consider moving the configuration-print/exit path before Output initialization so it doesn’t touch the filesystem when only dumping the configuration.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At src/index/index.cc, line 105: <comment>`--configuration` should be a read-only operation, but it still constructs `Output` before returning, which creates/scans the output directory. Consider moving the configuration-print/exit path before `Output` initialization so it doesn’t touch the filesystem when only dumping the configuration.</comment> <file context> @@ -102,9 +102,10 @@ static auto index_main(const std::string_view &program, - if (app.contains("verbose")) { - sourcemeta::core::prettify(raw_configuration, std::cerr); - std::cerr << "\n"; + if (app.contains("configuration")) { + sourcemeta::core::prettify(raw_configuration, std::cout); + std::cout << "\n"; </file context>

augmentcode · 2026-02-03T14:25:03Z

In --configuration mode the JSON is written to stdout, but the program also prints the version banner to stdout at startup, so the overall stdout stream isn’t valid JSON (the new tests work around this via tail -n +2). If --configuration is meant for scripting/pipe use, this mixed output could be surprising.

Severity: low

_{🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.}

cubic-dev-ai · 2026-02-03T14:25:32Z

P2: The pipeline only checks tail's exit status, so a failure in the index command can be masked. Run the command separately (or capture its output) so errexit stops the test on failure.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At test/cli/index/enterprise/sourcemeta-std.sh, line 17: <comment>The pipeline only checks `tail`'s exit status, so a failure in the index command can be masked. Run the command separately (or capture its output) so `errexit` stops the test on failure.</comment> <file context> @@ -0,0 +1,54 @@ +} +EOF + +"$1" "$TMP/one.json" "$TMP/output" --configuration | tail -n +2 > "$TMP/output.txt" + +cat << EOF > "$TMP/expected.txt" </file context>

-Original file line number
+Diff line change
@@ Expand Up / @@ -82,6 +82,9 @@ test-ui: node_modules @@
     .PHONY: sandbox-index
     sandbox-index: compile
+    	$(PREFIX)/bin/sourcemeta-one-index \
+    		$(SANDBOX)/one-$(SANDBOX_CONFIGURATION).json \
+    		$(OUTPUT)/sandbox --url $(SANDBOX_URL) --configuration
     	$(PREFIX)/bin/sourcemeta-one-index \
     		$(SANDBOX)/one-$(SANDBOX_CONFIGURATION).json \
     		$(OUTPUT)/sandbox --url $(SANDBOX_URL) --profile
@@ Expand Down Expand Up / @@ -142,6 +145,7 @@ docs: mkdocs.yml @@
     .PHONY: public
     public:
+    	$(PREFIX)/bin/sourcemeta-one-index $(PUBLIC)/one.json $(OUTPUT)/public --configuration
     	$(PREFIX)/bin/sourcemeta-one-index $(PUBLIC)/one.json $(OUTPUT)/public --verbose
     	$(PREFIX)/bin/sourcemeta-one-server $(OUTPUT)/public 8000
@@ Expand Down @@

-Original file line number
+Diff line change
@@ Expand Up / @@ -102,9 +102,10 @@ static auto index_main(const std::string_view &program, @@
       const auto raw_configuration{sourcemeta::one::Configuration::read(
           configuration_path, SOURCEMETA_ONE_COLLECTIONS)};
-      if (app.contains("verbose")) {
-        sourcemeta::core::prettify(raw_configuration, std::cerr);
-        std::cerr << "\n";
+      if (app.contains("configuration")) {
+        sourcemeta::core::prettify(raw_configuration, std::cout);
+        std::cout << "\n";
+        return EXIT_SUCCESS;
       }
       auto configuration{sourcemeta::one::Configuration::parse(raw_configuration)};
@@ Expand Down Expand Up / @@ -600,6 +601,7 @@ auto main(int argc, char *argv[]) noexcept -> int { @@
         app.option("concurrency", {"c"});
         app.flag("verbose", {"v"});
         app.flag("profile", {"p"});
+        app.flag("configuration", {"g"});
         app.parse(argc, argv);
         const std::string_view program{argv[0]};
@@ Expand Down @@

-Original file line number
+Diff line change
@@ Expand Up / @@ -30,11 +30,14 @@ if(ONE_INDEX) @@
       sourcemeta_one_test_cli(common index rebuild-to-empty)
       sourcemeta_one_test_cli(common index verbose-long)
       sourcemeta_one_test_cli(common index verbose-short)
+      sourcemeta_one_test_cli(common index configuration-long)
+      sourcemeta_one_test_cli(common index configuration-short)
       sourcemeta_one_test_cli(common index no-base-uri)
       if(ONE_ENTERPRISE)
         sourcemeta_one_test_cli(enterprise index no-options)
         sourcemeta_one_test_cli(enterprise index no-output)
+        sourcemeta_one_test_cli(enterprise index sourcemeta-std)
       else()
         sourcemeta_one_test_cli(community index no-options)
         sourcemeta_one_test_cli(community index no-output)
@@ Expand Down @@

-Original file line number
+Diff line change
@@ -0,0 +1,55 @@
+    #!/bin/sh
+    set -o errexit
+    set -o nounset
+    TMP="$(mktemp -d)"
+    clean() { rm -rf "$TMP"; }
+    trap clean EXIT
+    cat << EOF > "$TMP/one.json"
+    {
+      "url": "https://sourcemeta.com/",
+      "contents": {
+        "example": {
+          "contents": {
+            "schemas": {
+              "baseUri": "https://example.com/",
+              "path": "./schemas"
+            }
+          }
+        }
+      }
+    }
+    EOF
+    mkdir "$TMP/schemas"
+    cat << 'EOF' > "$TMP/schemas/foo.json"
+    {
+      "$schema": "http://json-schema.org/draft-07/schema#",
+      "$id": "https://example.com/foo"
+    }
+    EOF
+    "$1" "$TMP/one.json" "$TMP/output" --configuration | tail -n +2 > "$TMP/output.txt"
+    cat << EOF > "$TMP/expected.txt"
+    {
+      "url": "https://sourcemeta.com/",
+      "contents": {
+        "example": {
+          "contents": {
+            "schemas": {
+              "baseUri": "https://example.com/",
+              "path": "$(realpath "$TMP")/schemas"
+            }
+          }
+        }
+      },
+      "html": {
+        "name": "Sourcemeta",
+        "description": "The next-generation JSON Schema platform"
+      }
+    }
+    EOF
+    diff "$TMP/output.txt" "$TMP/expected.txt"

-Original file line number
+Diff line change
@@ -0,0 +1,55 @@
+    #!/bin/sh
+    set -o errexit
+    set -o nounset
+    TMP="$(mktemp -d)"
+    clean() { rm -rf "$TMP"; }
+    trap clean EXIT
+    cat << EOF > "$TMP/one.json"
+    {
+      "url": "https://sourcemeta.com/",
+      "contents": {
+        "example": {
+          "contents": {
+            "schemas": {
+              "baseUri": "https://example.com/",
+              "path": "./schemas"
+            }
+          }
+        }
+      }
+    }
+    EOF
+    mkdir "$TMP/schemas"
+    cat << 'EOF' > "$TMP/schemas/foo.json"
+    {
+      "$schema": "http://json-schema.org/draft-07/schema#",
+      "$id": "https://example.com/foo"
+    }
+    EOF
+    "$1" "$TMP/one.json" "$TMP/output" -g | tail -n +2 > "$TMP/output.txt"
+    cat << EOF > "$TMP/expected.txt"
+    {
+      "url": "https://sourcemeta.com/",
+      "contents": {
+        "example": {
+          "contents": {
+            "schemas": {
+              "baseUri": "https://example.com/",
+              "path": "$(realpath "$TMP")/schemas"
+            }
+          }
+        }
+      },
+      "html": {
+        "name": "Sourcemeta",
+        "description": "The next-generation JSON Schema platform"
+      }
+    }
+    EOF
+    diff "$TMP/output.txt" "$TMP/expected.txt"

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Add a `--configuration` option to the index binary #628

Uh oh!

Diff view

Diff view

There are no files selected for viewing

augmentcode bot Feb 3, 2026

Uh oh!

cubic-dev-ai bot Feb 3, 2026 •

edited

Loading

Uh oh!

augmentcode bot Feb 3, 2026

Uh oh!

cubic-dev-ai bot Feb 3, 2026 •

edited

Loading

Uh oh!

Uh oh!

Add a --configuration option to the index binary #628

Uh oh!

Add a --configuration option to the index binary #628

Uh oh!

Uh oh!

Diff view

Diff view

There are no files selected for viewing

augmentcode bot Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai bot Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

augmentcode bot Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai bot Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Add a `--configuration` option to the index binary #628

Add a `--configuration` option to the index binary #628

cubic-dev-ai bot Feb 3, 2026 •

edited

Loading

cubic-dev-ai bot Feb 3, 2026 •

edited

Loading