docs: rename to Docker Agent Action and sync README#230
Open
Sayt-0 wants to merge 2 commits into
Open
Conversation
- Rename title to 'Docker Agent Action' and update repo references to docker/docker-agent-action - Add missing inputs (skip-summary, org-membership-token, auth-org, skip-auth) - Add missing outputs (cagent-version, security-blocked, input-risk-level) - Fix prompt-suspicious description (content is stripped, not just detected) - Fix broken links: point Security Policy to .github/SECURITY.md, remove disabled Discussions link
docker-agent
reviewed
Jun 4, 2026
Contributor
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
This is a clean documentation-only PR. All changes in README.md were reviewed against the source of truth (action.yml) and confirmed correct:
- Action rename — all 7
uses: docker/cagent-action@VERSIONreferences correctly updated todocker/docker-agent-action@VERSION✅ - Outputs table — all 10 outputs from
action.ymlare present and accurately described (exit-code,output-file,cagent-version,mcp-gateway-installed,execution-time,verbose-log-file,security-blocked,secrets-detected,prompt-suspicious,input-risk-level) ✅ - New inputs —
skip-summary,org-membership-token,auth-org,skip-authdescriptions matchaction.yml✅ prompt-suspiciousdescription — updated wording matchesaction.ymlexactly ✅- Link fixes —
security/README.md→.github/SECURITY.md(file confirmed to exist), issues URL updated todocker/docker-agent-action, dead Discussions link removed ✅
No issues found in the changed code.
derekmisler
reviewed
Jun 4, 2026
| - **Prompt Injection Detection**: Warns about suspicious patterns in user prompts | ||
| - **Automatic Incident Response**: Creates security issues and fails workflows when secrets are detected | ||
|
|
||
| See [security/README.md](security/README.md) for complete security documentation. |
Contributor
There was a problem hiding this comment.
I would suggest merging #228 first, it's the "real" security doc that was accidentally removed in a previous PR
Comment on lines
+120
to
+122
| | `org-membership-token` | PAT with `read:org` scope for org membership authorization checks | No | - | | ||
| | `auth-org` | GitHub organization to check membership against (used with `org-membership-token`) | No | - | | ||
| | `skip-auth` | Skip the built-in authorization check (when the caller already performed its own) | No | `false` | |
Contributor
There was a problem hiding this comment.
I don't think we need to include these. The top two are deprecated and the last one should only be used internally by the PR review workflow
…puts - Point Security Policy links to SECURITY.md (the doc restored in #228) - Remove org-membership-token/auth-org (deprecated) and skip-auth (internal PR-review-workflow use only) from the inputs table
Member
Author
|
Thx @derekmisler! I fixed the Security Policy links to point to |
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
3 participants
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.
Summary
uses:/ issue links todocker/docker-agent-action(repo is being renamed)action.ymlChanges
Inputs added:
skip-summary,org-membership-token,auth-org,skip-authOutputs added:
cagent-version,security-blocked,input-risk-levelFixed:
prompt-suspiciousdescription; Security Policy now points to.github/SECURITY.md; removed dead Discussions link (Discussions disabled on the repo)