docs: fix renamed-repo links, drop stale assessment artifacts#2058
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
💤 Files with no reviewable changes (2)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThis PR updates documentation to standardize repository references to ChangesDocumentation Updates
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@EVALUATION.md`:
- Line 98: The document has inconsistent agent/skill/command counts: the
sentence "Run `ecc install --profile full` to get all 61 agents, 246 skills, and
76 commands." conflicts with earlier "28/116/59" references and headings like
"28/116/59 missing"; update all occurrences so the same counts are used
throughout (search for the exact strings "61 agents, 246 skills, and 76
commands" and "28/116/59" and replace or reconcile them), and adjust any
headings or "missing" lines to reflect the chosen canonical totals so the page
is internally consistent.
🪄 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: defaults
Review profile: CHILL
Plan: Pro
Run ID: 3d1222f3-2dda-4d76-85f0-02035ed9b559
📒 Files selected for processing (3)
CONTRIBUTING.mdEVALUATION.mdREPO-ASSESSMENT.md
💤 Files with no reviewable changes (1)
- REPO-ASSESSMENT.md
CONTRIBUTING.md still pointed at the old `affaan-m/everything-claude-code` repo URL in the Quick Start fork instructions and in the Issues link at the bottom. Both relied on GitHub's silent rename-redirect, but the literal `cd everything-claude-code` after `gh repo fork` would land in the wrong directory now that the repo is `affaan-m/ECC`. REPO-ASSESSMENT.md and EVALUATION.md were both 2026-03-21 personal fork-audit artifacts written from one user's specific install. They describe the project as a fork at `Infiniteyieldai/everything-claude-code` v1.9.0 with 28 agents / 116 skills / 59 commands and pin the recommended mode at "use as upstream tracker". None of that is true anymore (this IS the upstream, v2.0.0-rc.1, currently 61 / 246 / 76). EVALUATION.md in particular still references a defunct branch (`claude/evaluate-repo-comparison-ASZ9Y`) and describes a "Current Setup" of zero installed components as if it were universal, which it is not. Neither file is referenced by anything else in the repo (`rg` confirmed) and they actively mislead new contributors and visitors. Delete both. A targeted line-by-line refresh of EVALUATION.md was considered but rejected: bringing only the totals up to date (61/246/76) would leave the rest of the document — v1.9.0 references, branch metadata, the zero-component "Current Setup" — internally inconsistent (CodeRabbit flagged this on the first revision of this PR). Wholesale removal is the honest fix. Translated copies (e.g. docs/pt-BR/README.md still has the 28/116/59 numbers) are intentionally left for a follow-up i18n PR to keep this diff small.
b3679ad to
f473ba1
Compare
|
Good catch — you were right that updating only line 98 left the rest of Reworked in |
Summary
Two cleanup fixes:
CONTRIBUTING.md— replace two staleaffaan-m/everything-claude-codereferences (Quick Start fork instructions + bottom Issues link) withaffaan-m/ECC. The literalcd everything-claude-codeaftergh repo forkin Quick Start would have landed users in the wrong directory; the issues link works only via GitHub's rename-redirect.REPO-ASSESSMENT.mdandEVALUATION.md— delete both. They are 2026-03-21 personal fork-audit artifacts that describe this project as a fork atInfiniteyieldai/everything-claude-codev1.9.0 with 28 agents / 116 skills / 59 commands and recommend "use as upstream tracker". None of that is true anymore (this IS the upstream, v2.0.0-rc.1, currently 61/246/76).EVALUATION.mdalso references a defunct branch (claude/evaluate-repo-comparison-ASZ9Y) and describes a "Current Setup" of zero installed components as if it were universal. Neither file is referenced by anything else (rgconfirmed).Revision history
EVALUATION.md(61/246/76) and deleted onlyREPO-ASSESSMENT.md.EVALUATION.mdinternally inconsistent — line 98 quoted the new totals while lines 35-37, 61, 68, 76 still said 28/116/59, plus the v1.9.0 and defunct-branch references remained. The honest fix is to deleteEVALUATION.mdfor the same reasonREPO-ASSESSMENT.mdis being deleted: it is a stale personal audit, not maintained documentation.Out of scope
AGENTS.mdalready has the current counts (recent commitd7813494 chore: sync catalog counts after PR triage).docs/pt-BR/README.mdstill has the 28/116/59 line) are intentionally left for a follow-up i18n PR to keep this diff small and focused.Test plan
git diff --stat: only 3 intended file changes (1 modified, 2 deleted)rg REPO-ASSESSMENTreturns zero remaining referencesrg EVALUATIONagainst the rest of the repo returns only an unrelatedEVALUATION_SCHEMA_VERSIONconstant inscripts/lib/skill-improvement/evaluate.jsrg "affaan-m/everything-claude-code"againstCONTRIBUTING.mdreturns zero matches after the change (the slugeverything-claude-codeis still used as the plugin name in many other places — those are intentionally untouched)Summary by CodeRabbit