Skip to content

Add screenshot-janitor to Plugins & Extensions#60

Open
MECoban wants to merge 3 commits into
jmanhype:mainfrom
MECoban:add-screenshot-janitor
Open

Add screenshot-janitor to Plugins & Extensions#60
MECoban wants to merge 3 commits into
jmanhype:mainfrom
MECoban:add-screenshot-janitor

Conversation

@MECoban

@MECoban MECoban commented Jun 14, 2026

Copy link
Copy Markdown

Adds screenshot-janitor to the Plugins & Extensions table.

Per-session screenshot cleanup for Claude Code — before you leave a session it asks once whether to move that session's screenshots to the Trash. macOS + Linux, recoverable deletion, zero dependencies. Follows the existing table format.

@qodo-code-review

qodo-code-review Bot commented Jun 14, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0) 🎨 UX issues (0) 🔗 Cross-repo conflicts (0)

Grey Divider


Remediation recommended

1. Stale plugin count ✓ Resolved 🐞 Bug ≡ Correctness
Description
Adding screenshot-janitor increases the Plugins & Extensions entries to 5, but the README Status
metric still reports Plugins listed | 4, making the published count incorrect.
Code

README.md[33]

+| [screenshot-janitor](https://github.com/MECoban/screenshot-janitor) | MECoban | Per-session screenshot cleanup — asks to move a session's screenshots to the Trash before you leave (macOS + Linux, zero dependencies). |
Evidence
The Status table’s plugin count is 4, while the Plugins & Extensions table shows 5 plugin rows after
this PR’s addition.

README.md[7-13]
README.md[25-34]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The README `Status` table still reports `Plugins listed | 4` even though the PR adds a 5th plugin entry.
## Issue Context
This inconsistency was introduced by adding a new row to the `Plugins & Extensions` table without updating the summary metrics.
## Fix Focus Areas
- README.md[7-13]
- README.md[25-34]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools



Informational

2. Stale last-updated value ✓ Resolved 🐞 Bug ⚙ Maintainability
Description
The README Last updated metric remains 2025-Q2 even though this PR changes the curated list, so
the freshness indicator becomes misleading.
Code

README.md[33]

+| [screenshot-janitor](https://github.com/MECoban/screenshot-janitor) | MECoban | Per-session screenshot cleanup — asks to move a session's screenshots to the Trash before you leave (macOS + Linux, zero dependencies). |
Evidence
The PR adds a new plugin row, but the Status table still claims the list was last updated in
2025-Q2.

README.md[7-13]
README.md[25-34]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The README `Status` table includes a `Last updated` field, but it was not updated alongside this PR’s content change.
## Issue Context
Since this PR modifies the README’s curated entries, the `Last updated` value should be bumped to reflect the latest edit period.
## Fix Focus Areas
- README.md[7-13]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Qodo Logo

@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Add screenshot-janitor to Plugins & Extensions list
📝 Documentation 🕐 Less than 5 minutes

Grey Divider

Walkthroughs

Description
• Add screenshot-janitor entry to the Plugins & Extensions table.
• Document per-session screenshot cleanup behavior, platforms, and zero dependencies.
High-Level Assessment

The PR’s approach (adding a single row that matches the existing table format) is the most appropriate. Only consider ensuring the table’s ordering/conventions (e.g., alphabetical or grouped) remain consistent.

Grey Divider

File Changes

Documentation (1)
README.md Add screenshot-janitor row to Plugins & Extensions table +1/-0

Add screenshot-janitor row to Plugins & Extensions table

• Adds a new table entry linking to MECoban/screenshot-janitor and summarizes its purpose (per-session screenshot cleanup), supported platforms (macOS + Linux), and lack of dependencies.

README.md


Grey Divider

Qodo Logo

@MECoban

MECoban commented Jun 14, 2026

Copy link
Copy Markdown
Author

Thanks for the automated review 🙏 — addressed both findings: bumped Plugins listed 4 → 5 and updated Last updated to 2026-Q2 in the Status table.

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