Skip to content

CAMEL-23298: Add missing PQC and OCSF dataformats in model and fix A..Z ordering#22491

Merged
oscerd merged 2 commits intomainfrom
ci-issue-CAMEL-23298
Apr 8, 2026
Merged

CAMEL-23298: Add missing PQC and OCSF dataformats in model and fix A..Z ordering#22491
oscerd merged 2 commits intomainfrom
ci-issue-CAMEL-23298

Conversation

@oscerd
Copy link
Copy Markdown
Contributor

@oscerd oscerd commented Apr 8, 2026

Summary

Claude Code on behalf of Andrea Cosentino

  • Add missing PQC (Post-Quantum Cryptography) dataformat to DataFormatsDefinition, DataFormatTransformerDefinition, DataFormatClause, and DataFormatBuilderFactory
  • Add missing OCSF (Open Cybersecurity Schema Framework) dataformat to DataFormatsDefinition and DataFormatTransformerDefinition
  • Fix A..Z ordering of pgp and pqc entries in MarshalDefinition, UnmarshalDefinition, and DataFormatTransformerDefinition (they were appended at the end instead of their proper alphabetical position)

Test plan

  • camel-core-model module builds and all tests pass
  • Verify PQC and OCSF dataformats appear in YAML DSL schema after full catalog regeneration
  • Verify Java DSL marshal().pqc() compiles correctly in downstream usage

🤖 Generated with Claude Code

@oscerd oscerd requested review from Croway and davsclaus April 8, 2026 12:44
@github-actions github-actions bot added the core label Apr 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@davsclaus
Copy link
Copy Markdown
Contributor

you may need to rebuild yaml-dsl also

@oscerd
Copy link
Copy Markdown
Contributor Author

oscerd commented Apr 8, 2026

I'll have a look. Hopefully before tomorrow for the release from @gzurowski

oscerd and others added 2 commits April 8, 2026 18:08
….Z ordering

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
….Z ordering

Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

🧪 CI tested the following changed modules:

  • catalog/camel-catalog
  • components/camel-netty-http
  • components/camel-netty
  • core/camel-core-model
  • core/camel-xml-io
  • core/camel-yaml-io
  • dsl/camel-yaml-dsl/camel-yaml-dsl-deserializers
  • dsl/camel-yaml-dsl/camel-yaml-dsl

ℹ️ Dependent modules were not tested because the total number of affected modules exceeded the threshold (50). Use the test-dependents label to force testing all dependents.

⚠️ Some tests are disabled on GitHub Actions (@DisabledIfSystemProperty(named = "ci.env.name")) and require manual verification:

  • components/camel-netty: 1 test(s) disabled on GitHub Actions
Build reactor — dependencies compiled but only changed modules were tested (8 modules)
  • Camel :: Catalog :: Camel Catalog
  • Camel :: Core Model
  • Camel :: Netty
  • Camel :: Netty HTTP
  • Camel :: XML IO
  • Camel :: YAML DSL
  • Camel :: YAML DSL :: Deserializers
  • Camel :: YAML IO

⚙️ View full build and test results

@oscerd oscerd merged commit 191db3c into main Apr 8, 2026
6 checks passed
@oscerd oscerd deleted the ci-issue-CAMEL-23298 branch April 8, 2026 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants