Skip to content

docs: add desloppify scorecard and score badge#27

Merged
samtuckerdavis merged 1 commit intomainfrom
feat/desloppify-scorecard
Apr 22, 2026
Merged

docs: add desloppify scorecard and score badge#27
samtuckerdavis merged 1 commit intomainfrom
feat/desloppify-scorecard

Conversation

@samtuckerdavis
Copy link
Copy Markdown

@samtuckerdavis samtuckerdavis commented Apr 18, 2026

Adds the current desloppify mechanical scan score as a badge and embeds the full scorecard image. Score: 24.7/100. Scorecard tracked as scorecard.png in repo root.\n\nNote: the strict score is 24.7 because desloppify scanning itself triggers self-referential wontfix items. The objective/verified score is 98.8.

Summary by CodeRabbit

  • Documentation
    • Added a new code quality badge to the README linking to the project score image.
    • Introduced a "Code Quality" section that embeds the score image with descriptive alt text and fixed display sizing for clear visual metrics.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 18, 2026

📝 Walkthrough

Walkthrough

README badges updated to include a new “desloppify score” shields.io badge linking to scorecard.png, and a new "Code Quality" section was added embedding scorecard.png via an <img> tag. Campaign: documentation update.

Changes

Cohort / File(s) Summary
Documentation
README.md
Appended "desloppify score" shields.io badge; added "Code Quality" section embedding scorecard.png with alt text and fixed width.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 8
✅ Passed checks (8 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: adding a desloppify scorecard and score badge to documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
No Hardcoded Secrets Or Credentials ✅ Passed PR modifies only README.md with desloppify badge and Code Quality section. Credential scan found no matches for sensitive patterns. All changes are documentation and image references with no hardcoded secrets.
No Speciesist Idioms ✅ Passed PR #27 focuses on documentation/badge additions with limited scope. Speciesist idioms appear only in the Advocacy language detector section as examples of problematic language the tool catches.
No Tier 3 Data Committed ✅ Passed PR additions to README.md and scorecard.png contain no Tier 3 data. Public feature descriptions and code quality badge only—no activist identities, legal communications, operational security, investigative materials, PII, NGO financials, or credentials.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/desloppify-scorecard
  • 🛠️ fix NAV violations: Commit on current branch
  • 🛠️ fix NAV violations: Create PR

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 18, 2026

Auto-Merge Gate Check — Blocked

PR labeled needs-human-review — one or more gates failed.

Failed Gates

  • Classification label missing — PR needs wave0-auto or level-0 label
  • PR author 'stuckvgn' is not a known agent bot
  • CI checks not all green: Incomplete: tests-core, ci-contracts, package-smoke, check, lint, arch-contracts, typecheck, tests-full, Analyze (python), Code Quality Gate (score >= 70).

How to unblock

  • Fix the failing gates and push a new commit — this workflow re-runs automatically.
  • If CI is still running, wait for it to complete.
  • If NAV config is absent, open a task to install the no-animal-violence suite.

Permanent block

  • Add the human-gate label to permanently disable auto-merge for this PR.

Auto-Merge Gate Check — 2026-04-22T14:27:57.886Z

@samtuckerdavis
Copy link
Copy Markdown
Author

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 21, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Line 250: The <img> tag referencing "scorecard.png" in README.md is missing an
alt attribute; update the HTML image tag to include a descriptive alt text
(e.g., alt="project scorecard showing X metrics") so the element for
"scorecard.png" is accessible and satisfies MD045; locate the img tag and add an
appropriate alt string that concisely describes the image content.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 88bf2962-35b5-4816-82c2-ef58724e505e

📥 Commits

Reviewing files that changed from the base of the PR and between 4aa174f and cc0b41b.

⛔ Files ignored due to path filters (1)
  • scorecard.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • README.md

Comment thread README.md Outdated
@samtuckerdavis samtuckerdavis enabled auto-merge (squash) April 21, 2026 21:28
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@samtuckerdavis samtuckerdavis force-pushed the feat/desloppify-scorecard branch from cc0b41b to 938f0cc Compare April 22, 2026 14:25
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Line 1: The README badge references a relative scorecard.png which breaks on
PyPI; update the badge URL in the README markdown line (the badge link that
currently points to scorecard.png) to use the full GitHub URL
(https://github.com/Open-Paws/desloppify/blob/main/scorecard.png) so the image
resolves outside the repo and renders correctly on PyPI.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 78cbb4e6-987d-4209-b9fc-3eba1148a1b1

📥 Commits

Reviewing files that changed from the base of the PR and between cc0b41b and 938f0cc.

⛔ Files ignored due to path filters (1)
  • scorecard.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • README.md

Comment thread README.md
@@ -1,4 +1,4 @@
[![PyPI version](https://img.shields.io/pypi/v/desloppify)](https://pypi.org/project/desloppify/) [![License: MIT](https://img.shields.io/badge/license-MIT-green)](LICENSE) [![Python 3.11+](https://img.shields.io/badge/python-3.11%2B-blue)](https://www.python.org/downloads/) [![Last commit](https://img.shields.io/github/last-commit/Open-Paws/desloppify)](https://github.com/Open-Paws/desloppify/commits/main)
[![PyPI version](https://img.shields.io/pypi/v/desloppify)](https://pypi.org/project/desloppify/) [![License: MIT](https://img.shields.io/badge/license-MIT-green)](LICENSE) [![Python 3.11+](https://img.shields.io/badge/python-3.11%2B-blue)](https://www.python.org/downloads/) [![Last commit](https://img.shields.io/github/last-commit/Open-Paws/desloppify)](https://github.com/Open-Paws/desloppify/commits/main) [![desloppify score](https://img.shields.io/badge/desloppify-24.7%2F100-red)](scorecard.png)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Description: Check if scorecard.png is included in the package distribution

# Check if setup.py or pyproject.toml includes scorecard.png in package_data or data_files
fd -e py -e toml 'setup|pyproject' --exec grep -l 'scorecard.png' {}

Repository: Open-Paws/desloppify

Length of output: 46


🏁 Script executed:

# Check if scorecard.png exists in the repository
fd scorecard.png

Repository: Open-Paws/desloppify

Length of output: 46


🏁 Script executed:

# Examine the package configuration to understand how assets are handled
git ls-files | head -20

Repository: Open-Paws/desloppify

Length of output: 722


🏁 Script executed:

# Check pyproject.toml and setup.py content if they exist
fd -e toml -e py 'setup|pyproject' -x cat {}

Repository: Open-Paws/desloppify

Length of output: 14847


🏁 Script executed:

# Explicitly check for scorecard.png existence with verbose output
fd scorecard.png -v

Repository: Open-Paws/desloppify

Length of output: 294


🏁 Script executed:

# Also check with find as backup
find . -name "scorecard.png" -type f

Repository: Open-Paws/desloppify

Length of output: 102


🏁 Script executed:

# Check the README to see if score metrics are explained in the new scorecard section
sed -n '172,180p' README.md

Repository: Open-Paws/desloppify

Length of output: 497


🏁 Script executed:

# Also check what the badge currently links to in line 1
sed -n '1p' README.md

Repository: Open-Paws/desloppify

Length of output: 567


Fix badge link for PyPI distribution.

The badge links to a relative scorecard.png, which breaks when PyPI renders the README—the file isn't included in the package distribution (absent from pyproject.toml's package-data). Use the full GitHub URL instead: https://github.com/Open-Paws/desloppify/blob/main/scorecard.png.

🧰 Tools
🪛 markdownlint-cli2 (0.22.0)

[warning] 1-1: First line in a file should be a top-level heading

(MD041, first-line-heading, first-line-h1)

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` at line 1, The README badge references a relative scorecard.png
which breaks on PyPI; update the badge URL in the README markdown line (the
badge link that currently points to scorecard.png) to use the full GitHub URL
(https://github.com/Open-Paws/desloppify/blob/main/scorecard.png) so the image
resolves outside the repo and renders correctly on PyPI.

@samtuckerdavis samtuckerdavis merged commit a85d44b into main Apr 22, 2026
14 checks passed
@samtuckerdavis samtuckerdavis deleted the feat/desloppify-scorecard branch April 22, 2026 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant