R package methods check
Skill maxwellkonnaris/agents_skills/.agents/skills/r-package-methods-check
agentic workflows and skills
npx -y skills add maxwellkonnaris/agents_skills --skill r-package-methods-checkAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 0 stars0 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
What its author says it does
Copied from the file, not written here
Review R package method changes for statistical correctness, numerical robustness, API stability, tests, and docs.
SKILL.md
2.9 KB, 573 tokens by cl100k_base, as published. Nobody here has run it
Review the changed R package code with emphasis on methodological correctness, not style.
Use this for method changes, bug fixes that may affect inference, numerical refactors, API changes, or pre-PR review of an R package.
Focus on these questions:
- Statistical correctness
- Are the assumptions in the code consistent with the stated method?
- Are parameterizations clear and internally consistent?
- Are transformations, offsets, scales, and links handled correctly?
- Are quantities on the correct scale when returned, printed, or plotted?
- Are defaults scientifically reasonable?
- Are estimated quantities distinguishable from inputs, hyperparameters, and diagnostics?
- Are uncertainty-related quantities labeled clearly and computed on the intended scale?
- Are implicit assumptions hidden in preprocessing, normalization, filtering, or pseudocounts?
- Numerical stability and edge-case handling
- unstable subtraction, division, exponentiation, or log operations
- underflow/overflow risk
- division by zero or near-zero quantities
- unguarded
log,exp,softmax, or likelihood calculations - singular or near-singular matrix operations
- poor behavior for very small sample sizes
- problems induced by extreme counts, sparsity, or heavy skew
- failure to handle ties, all-zero rows, empty groups, or one-level factors
- silent recycling, coercion, or factor conversion issues
- missing handling for
NA,NaN,Inf,-Inf
- Package behavior
- Are exported vs internal functions appropriate?
- Are function names, arguments, and defaults coherent?
- Are argument checks informative and early?
- Are return values stable and documented?
- Is backward compatibility preserved where reasonable?
- Are S3/S4 methods registered correctly if relevant?
- Are dependencies necessary and minimal?
- Are examples safe, fast, and reproducible?
- Tests
- Propose the smallest regression and edge-case tests that would catch real breakage.
- Prefer one minimal regression test over broad test-suite rewrites unless clearly needed.
- Documentation
- Flag roxygen, examples, README, vignettes, and NEWS items that no longer match behavior.
Return exactly these sections:
Summary
2-4 sentences on what changed and the main risk.
Major issues
For each issue, give:
- Issue
- Why it matters
- Smallest fix
Suggested tests
List the minimal tests to add or update.
Documentation updates
List docs or examples that should be updated.
Nice-to-have improvements
Optional cleanup that is not required for correctness.
Behavior:
- Be skeptical about scientific correctness, not just syntax.
- Prefer the smallest correct patch.
- Distinguish correctness issues from maintainability issues.
- If uncertain, say exactly what is uncertain.
- Review first; do not start editing unless explicitly asked.
What ships with it
Read from the repository
Just SKILL.md. No reference files, no scripts.