feat(autodoc): add autodoc-executor reusable workflow - #65
Merged
Conversation
…audit pipeline Adds a purpose-built reusable workflow that runs Claude Code via AWS Bedrock for the issue_autodoc.yml pipeline in dotCMS/core. Separating this from the general-purpose claude-executor keeps AUTODOC_* secret passthrough and the report artifact upload out of the code-review path. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
🤖 Bedrock Review —
|
jdcmsd
marked this pull request as draft
July 31, 2026 15:51
sfreudenthaler
marked this pull request as ready for review
July 31, 2026 19:57
…to input - Pin actions/checkout, configure-aws-credentials, claude-code-action, and upload-artifact to full commit SHAs (Semgrep supply-chain findings) - Move AUTODOC_DOTCMS_SITE_FOLDER from secrets to a plain string input — it is non-sensitive config, not a credential, and is easier to trace as a var Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
🤖 Bedrock Review —
|
sfreudenthaler
previously requested changes
Aug 4, 2026
sfreudenthaler
left a comment
Member
There was a problem hiding this comment.
getting closer. a few questions about unhappy path and one requested change for the base url
…y defaults AUTODOC_DOTCMS_BASE_URL is not a credential — it is config. Move it from secrets to a required string input (autodoc_dotcms_base_url) so it appears in Actions logs and is easy to verify without special access. Also clarify the intent of the two optional inputs that default to empty string: - model_id: empty → --model omitted → action uses its built-in default model - autodoc_dotcms_site_folder: empty → navFolder not set → content lands on SYSTEM_HOST Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
🤖 Bedrock Review —
|
jdcmsd
dismissed
sfreudenthaler’s stale review
August 6, 2026 18:14
All the conversations are marked as resolved, seems time to press forward.
sfreudenthaler
approved these changes
Aug 7, 2026
sfreudenthaler
left a comment
Member
There was a problem hiding this comment.
changes look good. let'r rip
This was referenced Aug 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
relates to: dotCMS/core#36842
Summary
autodoc-executor.yml, a purpose-built reusable workflow for the dotCMS documentation audit pipeline (issue_autodoc.ymlindotCMS/core)AUTODOC_*secrets as env vars so Claude can call the dotCMS AI search and workflow APIs during execution/tmp/autodoc-report.md) as an artifact for the calling workflow'sfinalizejob to consumeclaude-executor.ymlto avoid adding autodoc-specific secrets and artifact upload to the general-purpose code-review pathWhy a new executor instead of extending
claude-executorclaude-executor.ymlis general-purpose (code review). Autodoc needs two things it doesn't: (1) pass-through ofAUTODOC_*secrets as env vars to Claude, and (2) upload of the written report as an artifact. Grafting those ontoclaude-executorwould pollute it with pipeline-specific concerns.Dependency
dotCMS/coreissue_autodoc.ymlwill reference this at@v3.5.0— please tag after merge.Test plan
v3.5.0dotCMS/corePR (referencesautodoc-executor.yml@v3.5.0)dotCMS/coreand confirm the Actions run completes: OIDC assumption succeeds, Claude writes report, artifact is uploaded, finalize posts comment🤖 Generated with Claude Code