Skip to content

chore: fix state-store adoc duplicate include and incomplete operation list - #25728

Merged
gnodet merged 3 commits into
apache:mainfrom
gnodet:quick-fix/state-store-adoc-fixes
Aug 25, 2026
Merged

gnodet merged 3 commits into
apache:mainfrom
gnodet:quick-fix/state-store-adoc-fixes

Conversation

@gnodet

@gnodet gnodet commented Aug 25, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Claude Code on behalf of gnodet

Two minor documentation fixes for the camel-state-store component merged in PR #22158 (CAMEL-23239):

  1. Fix duplicate include — component-configure-options.adoc was included in both the component-configure options section and the auto-generated component options section. Removed it from the manually-managed component-configure options section (keeping it in the component options section where the build plugin expects it), matching the standard component doc pattern.

  2. Complete operation list — the "When to use" section listed only put, get, delete, contains, keys but the component also supports putIfAbsent, size, and clear. Updated to list all 8 operations.

Test plan

  • Documentation-only change — no code impact
  • Build passes with no uncommitted generated changes

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

@gnodet gnodet added the docs label Aug 25, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🌟 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.

gnodet and others added 2 commits August 25, 2026 19:16
…n list

Remove duplicate component-configure-options include from the component
options block, and add missing operations (putIfAbsent, size, clear) to
the "When to use" section.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The build plugin (camel-package-maven-plugin) auto-generates the
"component options" section includes. Move the component-configure-options
include from the manually-managed section to the auto-generated section
to avoid the duplicate and satisfy the CI uncommitted-changes check.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@davsclaus
davsclaus force-pushed the quick-fix/state-store-adoc-fixes branch from b774be1 to d717f9b Compare August 25, 2026 17:16
Update the generated catalog copy to match the source adoc changes
(updated operation list and fixed duplicate include).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • catalog/camel-catalog
  • components/camel-state-store/camel-state-store

🔬 Scalpel shadow comparison — Scalpel: 13 tested, 28 compile-only — current: 12 all tested

Maveniverse Scalpel detected 41 affected modules (current approach: 12).

⚠️ Modules only in Scalpel (29)
  • apache-camel
  • camel-allcomponents
  • camel-catalog
  • camel-catalog-console
  • camel-catalog-lucene
  • camel-catalog-maven
  • camel-catalog-suggest
  • camel-componentdsl
  • camel-csimple-maven-plugin
  • camel-endpointdsl
  • camel-endpointdsl-support
  • camel-itest
  • camel-jbang-core
  • camel-jbang-it
  • camel-jbang-main
  • camel-jbang-plugin-edit
  • camel-jbang-plugin-generate
  • camel-jbang-plugin-kubernetes
  • camel-jbang-plugin-test
  • camel-kamelet-main
  • camel-launcher
  • camel-report-maven-plugin
  • camel-route-parser
  • camel-yaml-dsl
  • camel-yaml-dsl-deserializers
  • camel-yaml-dsl-maven-plugin
  • coverage
  • docs
  • dummy-component

Skip-tests mode would test 13 modules (2 direct + 11 downstream), skip tests for 28 (generated code, meta-modules)

Modules Scalpel would test (13)
  • camel-catalog
  • camel-jbang-mcp
  • camel-jbang-plugin-mcp
  • camel-jbang-plugin-route-parser
  • camel-jbang-plugin-tui
  • camel-jbang-plugin-validate
  • camel-launcher-container
  • camel-state-store
  • camel-state-store-caffeine
  • camel-state-store-infinispan
  • camel-state-store-redis
  • camel-yaml-dsl-validator
  • camel-yaml-dsl-validator-maven-plugin
Modules with tests skipped (28)
  • apache-camel
  • camel-allcomponents
  • camel-catalog-console
  • camel-catalog-lucene
  • camel-catalog-maven
  • camel-catalog-suggest
  • camel-componentdsl
  • camel-csimple-maven-plugin
  • camel-endpointdsl
  • camel-endpointdsl-support
  • camel-itest
  • camel-jbang-core
  • camel-jbang-it
  • camel-jbang-main
  • camel-jbang-plugin-edit
  • camel-jbang-plugin-generate
  • camel-jbang-plugin-kubernetes
  • camel-jbang-plugin-test
  • camel-kamelet-main
  • camel-launcher
  • camel-report-maven-plugin
  • camel-route-parser
  • camel-yaml-dsl
  • camel-yaml-dsl-deserializers
  • camel-yaml-dsl-maven-plugin
  • coverage
  • docs
  • dummy-component

ℹ️ Shadow mode — Scalpel observes but does not affect test execution. Learn more

All tested modules (41 modules)
  • Camel :: All Components Sync point
  • Camel :: Assembly
  • Camel :: Catalog :: CSimple Maven Plugin (deprecated)
  • Camel :: Catalog :: Camel Catalog
  • Camel :: Catalog :: Camel Report Maven Plugin
  • Camel :: Catalog :: Camel Route Parser
  • Camel :: Catalog :: Console
  • Camel :: Catalog :: Dummy Component
  • Camel :: Catalog :: Lucene (deprecated)
  • Camel :: Catalog :: Maven
  • Camel :: Catalog :: Suggest
  • Camel :: Component DSL
  • Camel :: Coverage
  • Camel :: Docs
  • Camel :: Endpoint DSL
  • Camel :: Endpoint DSL :: Support
  • Camel :: Integration Tests
  • Camel :: JBang :: Core
  • Camel :: JBang :: Integration tests
  • Camel :: JBang :: MCP
  • Camel :: JBang :: Main
  • Camel :: JBang :: Plugin :: Edit
  • Camel :: JBang :: Plugin :: Generate
  • Camel :: JBang :: Plugin :: Kubernetes
  • Camel :: JBang :: Plugin :: MCP
  • Camel :: JBang :: Plugin :: Route Parser
  • Camel :: JBang :: Plugin :: TUI
  • Camel :: JBang :: Plugin :: Testing
  • Camel :: JBang :: Plugin :: Validate
  • Camel :: Kamelet Main
  • Camel :: Launcher
  • Camel :: Launcher :: Container
  • Camel :: State Store
  • Camel :: State Store :: Caffeine
  • Camel :: State Store :: Infinispan
  • Camel :: State Store :: Redis
  • Camel :: YAML DSL
  • Camel :: YAML DSL :: Deserializers
  • Camel :: YAML DSL :: Maven Plugins
  • Camel :: YAML DSL :: Validator
  • Camel :: YAML DSL :: Validator Maven Plugin

⚙️ View full build and test results

@gnodet gnodet added this to the 4.23.0 milestone Aug 25, 2026
@gnodet gnodet self-assigned this Aug 25, 2026
@gnodet
gnodet merged commit 239475b into apache:main Aug 25, 2026
6 checks passed
@gnodet
gnodet deleted the quick-fix/state-store-adoc-fixes branch August 25, 2026 18:39
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