Skip to content

Commit 096e86f

Browse files
DavisVaughanDavisVaughan
committed
Add revdep info and claude skill for categorizing them
1 parent 29c5518 commit 096e86f

File tree

5 files changed

+1518
-1437
lines changed

5 files changed

+1518
-1437
lines changed

‎.claude/skills/categorize-revdep-issues/SKILL.md‎

Lines changed: 115 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,115 @@
1+
---
2+
name: categorize-revdep-issues
3+
description: Write revdep/issue.md to group packages by common error categories
4+
---
5+
6+
# Categorize Reverse Dependency Issues
7+
8+
Write `revdep/issue.md` to group packages by common error categories based on the error output in `revdep/problems.md` and `revdep/failures.md`.
9+
10+
## What This Skill Does
11+
12+
This skill analyzes reverse dependency check results and writes out an issue tracking file to group packages by their error types, making it easier to:
13+
- Identify common problems across packages
14+
- Understand which issues are dplyr-related vs external dependencies
15+
- Prioritize fixes based on error patterns
16+
- Provide clear guidance to package maintainers
17+
18+
## Files Involved
19+
20+
**Input files:**
21+
- `revdep/README.md` - The `New problems` section lists the packages to group
22+
- `revdep/problems.md` - Detailed error output for packages with test/check issues
23+
- `revdep/failures.md` - Installation failures and error messages
24+
25+
**Output file:**
26+
- `revdep/issue.md` - File to write packages grouped by error category to, may need to be created
27+
28+
## Process
29+
30+
### Step 1: Read All Input Files
31+
32+
Read all input files in parallel to understand the complete error landscape.
33+
34+
### Step 2: Analyze and Categorize Errors
35+
36+
Group packages by their primary error cause. Common categories include:
37+
38+
**Issues related to changes in our package:**
39+
- Missing exports
40+
- Defunct functions
41+
- Defunct function arguments
42+
- Missing Rd cross-references to removed functions
43+
44+
**Issues unrelated to changes we have made:**
45+
- Missing exports from other packages (e.g., `purrr::at_depth`)
46+
- Dependency issues
47+
- Compilation failures (C++, Rust)
48+
- Other packages that passed or have unclear status
49+
50+
### Step 3: Create Category Structure
51+
52+
For each category:
53+
1. Create an H2 header with a descriptive category name
54+
2. Write a brief description explaining:
55+
- What the issue is
56+
- Why it's happening
57+
- Suggested solution or migration path
58+
3. List all affected packages in checkbox format
59+
60+
### Step 4: Write the New File
61+
62+
Format requirements:
63+
64+
```markdown
65+
## Category Name
66+
67+
Brief description of the issue and solution approach.
68+
69+
* [ ] [package1](url1)
70+
* [ ] [package2](url2)
71+
...
72+
```
73+
74+
If you must include package specific comments, do so like:
75+
76+
```markdown
77+
## Category Name
78+
79+
Brief description of the issue and solution approach.
80+
81+
* [ ] [package1](url1)
82+
* Comments about package1.
83+
* [ ] [package2](url2)
84+
...
85+
```
86+
87+
Do not include package specific comments in the `Brief description` section.
88+
89+
**Critical constraints:**
90+
- MUST preserve ALL packages from the `README.md`'s `New problems` section
91+
- Each package appears in exactly ONE category
92+
- Maintain checkbox format: `* [ ] [package](url)`
93+
- Order categories by relevance (common issues first)
94+
- Use "Other issues requiring investigation" for unclear cases
95+
96+
## Quality Checks
97+
98+
After completing the rewrite, verify:
99+
- [ ] Total package count in new file matches the `README.md`'s `New problems` section
100+
- [ ] Each package from original appears exactly once in new file
101+
- [ ] Each category has a clear H2 header and description
102+
- [ ] Packages are in checkbox format with correct URLs
103+
- [ ] Categories are logically ordered
104+
- [ ] Common patterns are grouped together
105+
106+
## Example Category
107+
108+
```markdown
109+
## Missing export: `dplyr::id`
110+
111+
Packages using `dplyr::id` which is not exported. These packages get "no visible binding for global variable 'id'" errors.
112+
113+
* [ ] [packageA](https://github.com/userA/packageA)
114+
* [ ] [packageB](https://github.com/userB/packageB)
115+
```

‎revdep/README.md‎

Lines changed: 38 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -1,54 +1,43 @@
11
# Revdeps
22

3-
## Failed to check (40)
3+
## Failed to check (15)
44

5-
|package |version |error |warning |note |
6-
|:--------------|:-------|:-----|:-------|:----|
7-
|ADAM |? | | | |
8-
|AGread |? | | | |
9-
|AMARETTO |? | | | |
10-
|amplican |? | | | |
11-
|anomalize |? | | | |
12-
|bayesmodels |? | | | |
13-
|BMTME |? | | | |
14-
|brendaDb |? | | | |
15-
|BUSpaRse |? | | | |
16-
|cattonum |? | | | |
17-
|ceRNAnetsim |? | | | |
18-
|COMPASS |? | | | |
19-
|cort |? | | | |
20-
|ctDNAtools |? | | | |
21-
|CytoML |? | | | |
22-
|datastructures |? | | | |
23-
|DeLorean |? | | | |
24-
|DepecheR |? | | | |
25-
|destiny |? | | | |
26-
|DiffBind |? | | | |
27-
|diffman |? | | | |
28-
|diffrprojects |? | | | |
29-
|dynfrail |? | | | |
30-
|epiphy |? | | | |
31-
|evaluator |? | | | |
32-
|expstudies |? | | | |
33-
|fipe |? | | | |
34-
|foieGras |? | | | |
35-
|ImputeRobust |? | | | |
36-
|IsoCorrectoR |? | | | |
37-
|loon.ggplot |? | | | |
38-
|loon.shiny |? | | | |
39-
|mafs |? | | | |
40-
|MarketMatching |? | | | |
41-
|modeltime.h2o |? | | | |
42-
|Platypus |? | | | |
43-
|SCtools |? | | | |
44-
|sknifedatar |? | | | |
45-
|tidyfit |? | | | |
46-
|vivid |? | | | |
5+
|package |version |error |warning |note |
6+
|:-------------------|:-------|:-----|:-------|:----|
7+
|amberr |1.2.0 |1 | | |
8+
|bayesdfa |1.3.4 |1 | | |
9+
|ClustAssess |? | | | |
10+
|CytoML |? | | | |
11+
|DSMolgenisArmadillo |? | | | |
12+
|dsTidyverse |? | | | |
13+
|dsTidyverseClient |? | | | |
14+
|flowWorkspace |? | | | |
15+
|galvanizer |0.5.3 |1 | | |
16+
|h3o |0.3.0 |1 | | |
17+
|loon.shiny |? | | | |
18+
|NanoMethViz |? | | | |
19+
|rshift |3.1.2 |1 | | |
20+
|tidyjson |0.3.2 |1 | | |
21+
|TriDimRegression |1.0.3 |1 | | |
4722

48-
## New problems (2)
49-
50-
|package |version |error |warning |note |
51-
|:---------|:-------|:------|:-------|:----|
52-
|[covidcast](problems.md#covidcast)|0.5.0 |__+1__ | |1 |
53-
|[scGOclust](problems.md#scgoclust)|0.1.0 |__+1__ | | |
23+
## New problems (17)
5424

25+
|package |version |error |warning |note |
26+
|:----------|:-------|:------|:-------|:----|
27+
|[blob](problems.md#blob)|1.2.4 |__+1__ | | |
28+
|[crmPack](problems.md#crmpack)|2.0.1 |__+1__ | | |
29+
|[diceR](problems.md#dicer)|3.1.0 |__+1__ | | |
30+
|[feisr](problems.md#feisr)|1.3.0 |__+3__ | | |
31+
|[ggstats](problems.md#ggstats)|0.11.0 |__+2__ | | |
32+
|[klassR](problems.md#klassr)|1.0.4 |__+1__ | | |
33+
|[merTools](problems.md#mertools)|0.6.3 |__+2__ | | |
34+
|[myTAI](problems.md#mytai)|2.3.4 |__+1__ | | |
35+
|[orderly](problems.md#orderly)|2.0.0 |__+1__ | | |
36+
|[purrr](problems.md#purrr)|1.2.0 |__+2__ | | |
37+
|[rainette](problems.md#rainette)|0.3.1.1 |__+2__ | | |
38+
|[riskmetric](problems.md#riskmetric)|0.2.5 |__+2__ | | |
39+
|[rlang](problems.md#rlang)|1.1.6 |__+1__ | |1 |
40+
|[svytest](problems.md#svytest)|1.1.0 |__+1__ | | |
41+
|[testthat](problems.md#testthat)|3.3.1 |__+1__ | | |
42+
|[tibble](problems.md#tibble)|3.3.0 |__+1__ | | |
43+
|[wk](problems.md#wk)|0.9.4 |__+1__ | | |

‎revdep/cran.md‎

Lines changed: 69 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,84 @@
11
## revdepcheck results
22

3-
We checked 4392 reverse dependencies (4359 from CRAN + 33 from Bioconductor), comparing R CMD check results across CRAN and dev versions of this package.
3+
We checked 5569 reverse dependencies (5566 from CRAN + 3 from Bioconductor), comparing R CMD check results across CRAN and dev versions of this package.
44

5-
* We saw 2 new problems
6-
* We failed to check 7 packages
5+
* We saw 17 new problems
6+
* We failed to check 12 packages
77

88
Issues with CRAN packages are summarised below.
99

1010
### New problems
1111
(This reports the first line of each new failure)
1212

13-
* covidcast
13+
* blob
14+
checking tests ... ERROR
15+
16+
* crmPack
17+
checking examples ... ERROR
18+
19+
* diceR
20+
checking tests ... ERROR
21+
22+
* feisr
23+
checking examples ... ERROR
24+
checking tests ... ERROR
25+
checking re-building of vignette outputs ... ERROR
26+
27+
* ggstats
28+
checking examples ... ERROR
29+
checking tests ... ERROR
30+
31+
* klassR
32+
checking tests ... ERROR
33+
34+
* merTools
35+
checking examples ... ERROR
36+
checking tests ... ERROR
37+
38+
* myTAI
1439
checking re-building of vignette outputs ... ERROR
1540

16-
* scGOclust
41+
* orderly
42+
checking tests ... ERROR
43+
44+
* purrr
45+
checking examples ... ERROR
46+
checking tests ... ERROR
47+
48+
* rainette
49+
checking tests ... ERROR
1750
checking re-building of vignette outputs ... ERROR
1851

52+
* riskmetric
53+
checking tests ... ERROR
54+
checking re-building of vignette outputs ... ERROR
55+
56+
* rlang
57+
checking tests ... ERROR
58+
59+
* svytest
60+
checking re-building of vignette outputs ... ERROR
61+
62+
* testthat
63+
checking tests ... ERROR
64+
65+
* tibble
66+
checking re-building of vignette outputs ... ERROR
67+
68+
* wk
69+
checking tests ... ERROR
70+
1971
### Failed to check
2072

21-
* ImputeRobust (NA)
22-
* loon.ggplot (NA)
23-
* loon.shiny (NA)
24-
* MarketMatching (NA)
25-
* Platypus (NA)
26-
* tidyfit (NA)
27-
* vivid (NA)
73+
* amberr (NA)
74+
* bayesdfa (NA)
75+
* ClustAssess (NA)
76+
* DSMolgenisArmadillo (NA)
77+
* dsTidyverse (NA)
78+
* dsTidyverseClient (NA)
79+
* galvanizer (NA)
80+
* h3o (NA)
81+
* loon.shiny (NA)
82+
* rshift (NA)
83+
* tidyjson (NA)
84+
* TriDimRegression (NA)

0 commit comments

Comments
 (0)