chore(dev): migrate GenerateConfig trait to serde_json::Value

[pront]

Summary

GenerateConfig::generate_config() returned toml::Value due to TOML being Vector's original config format. This migrates it to serde_json::Value, which is the stated direction in #19963 (drop TOML as an internal value type).

Vector configuration

N/A

How did you test this PR?

make check-clippy and make fmt pass clean. 99 files changed.

Is this a breaking change?

• Yes
• No

Does this PR include user facing changes?

• Yes. Please add a changelog fragment based on our guidelines.
• No. A maintainer will apply the no-changelog label to this PR.

References

• Part of: Drop use of TOML as an internal configuration value #19963
• Closes: Provide generic generate_config and test_generate_config #21865

[datadog-vectordotdev]

 

⚠️ Warnings

🚦 4 Pipeline jobs failed

Test Suite | test / Tests     

🧪 20 Tests failed

config::cmd::tests::output_has_consistent_ordering from vector  

Test has failed

config::cmd::tests::output_is_a_valid_config from vector  

Test has failed

View all failed tests

Changelog | validate-changelog     

PR Title Check | Check PR Title     

View all 4 failed jobs.

ℹ️ Info

No other issues found (see more)

❄️ No new flaky tests detected

Useful? React with 👍 / 👎

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 223b72b | Docs | Datadog PR Page | Give us feedback!}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 223b72bc18

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Strip nulls before serializing generated TOML

When a default config contains an unskipped Option::None—for example, Appsignal's default tls—serde_json::to_value preserves it as JSON null. The generated value is later passed to toml::to_string by generate_example, which rejects that value as an unsupported unit, so commands such as vector generate //appsignal --format toml now fail instead of producing a config. The previous toml::Value::try_from omitted these fields, so null-valued fields need to be removed or otherwise normalized before TOML serialization.

Useful? React with 👍 / 👎.