ci: pin actions, add scanners and stats card, restore required checks - #64
Merged
Merged
Conversation
…ts callers - Pin the third-party actions that run in jobs with a secret or write access to a commit SHA (S12) and declare workflow permissions on test.yml and security.yml (S11). - Add the shared secret scan and CodeQL workflows as callers. Default setup is not configured, so the repository's own CodeQL workflow is the scanner (P13). - Add the repository stats caller, writing the card to the `stats` branch, and show it in the README (P09). - Point the README badges at what they report: CI at ci-gates.yml (the workflow named CI does not exist), licence and Python version derived instead of typed. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CdiwBVuH6DCEtFPPwxbXKc
Branch protection requires "Integration Smoke Tests" and "Quick Security Scan". Both were defined only in fast-tests.yml, which is manual-only since the CI consolidation, so neither reported on pull requests and every pull request waited on them. Define them in ci-gates.yml and gate "All CI Gates Passed" on them. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CdiwBVuH6DCEtFPPwxbXKc
The card is generated on the `stats` branch by the first run of the new workflow, which can only start once the workflow is on main. Linking it now fails the link check with a 404. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CdiwBVuH6DCEtFPPwxbXKc
Contributor
🔍 PR Analysis ResultsPR: #64 | Commit: 🎨 Code Formatting✅ All files properly formatted 🔧 Code Linting✅ No linting issues found 📝 Type Checking❌ Type checking issues found Click to see type issuesFix: Add proper type annotations and resolve type errors 🔒 Security Analysis✅ No security issues detected 📊 Test Coverage Analysis✅ Coverage 81.81818181818181% meets 80% requirement 🧹 Dead Code Analysis✅ Dead code analysis completed 📋 Summary
🔧 Quick Fix Commands:# Fix formatting and auto-fixable linting issues
ruff format .
ruff check . --fix
# Run tests with coverage
pytest tests/unit/ --cov=markitdown_mcp --cov-report=term-missing
# Check security
bandit -r markitdown_mcp/This analysis was automatically generated by the PR feedback workflow. |
Contributor
🔍 CI Quality Gates SummaryOverall Status: ✅ All Passed
🔗 Quick Links🛠️ Quick Fix Commands# Fix most issues automatically
ruff format .
ruff check . --fix
# Run tests locally
pytest tests/unit/ --cov=markitdown_mcp
# Check types
mypy markitdown_mcpLast updated: 2026-09-21 19:38:18 UTC |
Contributor
🔍 PR Quality SummaryCI Status⚪ Tests: pending Metrics
Quality Checks
MCP Tools
🤖 Auto-generated by CI • Last updated: 2026-09-21 19:39 UTC |
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.
Part 2 of 3 of the Repository Quality Standard 1.18.1 rollout (pipeline).
permissionsontest.ymlandsecurity.yml(this also clears eightactions/missing-workflow-permissionscode scanning alerts).actions,python). CodeQL default setup isnot-configured, so the repository's own workflow is the scanner, per the CodeQL guide.statsbranch, becausemainis protected). The README card follows in the record pull request, once the first run has produced it.CIthat does not exist, so it now points atci-gates.yml; licence and Python badges are derived instead of typed.Integration Smoke TestsandQuick Security Scanwere defined only infast-tests.yml, manual-only since [Feature]: Simplify CI checks and consolidate required gates #58, so pull requests waited on them forever. They are now jobs ofci-gates.yml; this pull request is the first to report them.Verified locally:
ruff format --check .,ruff check ., smoke tests (7 passed) andbandit --severity-level highall pass. The workflow changes are verified by this pull request's own run.🤖 Generated with Claude Code
https://claude.ai/code/session_01CdiwBVuH6DCEtFPPwxbXKc