Repository navigation
ci: add required Articulate security workflow - #107
Closed
catzo-agent[bot] wants to merge 1 commit into
Closed
catzo-agent[bot] wants to merge 1 commit into
catzo-agent[bot] wants to merge 1 commit into
Conversation
Contributor
|
This is a public repo and can't run that workflow |
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.
The repo has no
.github/workflows/security.yml, which fails the Art verify rulegithub:missing_security_workflow. This PR adds that file, calling the shared Articulate security workflow so CodeQL, dependency review and the Claude security review now run on PRs and pushes tomain. I could not runart verifyin this environment, so there is no real after-fix verify output yet.Changes
.github/workflows/security.yml, copied from the template in the rule's documentation. It callsarticulate/security/.github/workflows/security.yml@v3withsecrets: inheritand leaves all inputs at their defaults.service.json,verify.ignoreoverrides or.npmrcwere added. The repo has no Node project, so the.npmrcscanner bug doesn't apply here.Verify evidence:
Before (output you observed on a fresh snapshot of
7786322):FAIL - Combined - github - Missing required security workflow file .github/workflows/security.yml.and3 tests, 2 passed, 0 warnings, 1 failure, 0 exceptions.After: not available. Neither the
artCLI nor an Art MCP verify tool is installed here, and neither areopaorconftest.What I checked instead: I applied the logic of the current
policy/github.regorules by hand to the files in the repo:missing_security_workflow: fails onorigin/main(3 workflows, none named security), passes on this branch (4 workflows, one issecurity.yml).incorrect_security_workflow: passes, because the file references@v3.node_lockfile_bypass: passes, because no workflow runsnpm,yarnorpnpminstall.This is a hand check, not an
art verifyrecord. Please runart verifyon this branch before marking the PR ready.Limits and notes:
anthropic/claude-opus-5-5).{ "repo": "articulate/docker-ruby", "pr_url": null, "head_sha": "fdc6bd0e50ed0e555ff76c36f80b966ca664e04c", "base_sha": "7786322ff0959876210fce96a722e4ecbcb61862", "fixes": [{"rule_id": "github:missing_security_workflow", "change": "added .github/workflows/security.yml calling articulate/security/.github/workflows/security.yml@v3"}], "overrides": [], "before_verify": "1 failure (github:missing_security_workflow), 0 warnings, 2 passed (observed snapshot output)", "after_verify": "not run: art CLI/MCP unavailable; manual evaluation of github.rego rules shows 0 failures, 0 warnings", "tests": ["manual github.rego rule evaluation before/after"], "unresolved": ["Run art verify on this branch to get a real passing record", "PR URL is assigned when the platform publishes the branch"] }🤖 Generated with Claude Code
Agent session: 📊 Datadog · this job
Model:
anthropic/claude-opus-5-5