Skip to content

feat: update research skills part 8#54

Merged
markusha77 merged 3 commits into
ChatAndBuild:mainfrom
sixiang-svg:update-research-part-8
Mar 25, 2026
Merged

feat: update research skills part 8#54
markusha77 merged 3 commits into
ChatAndBuild:mainfrom
sixiang-svg:update-research-part-8

Conversation

@sixiang-svg
Copy link
Copy Markdown
Contributor

New Skill Contribution

Skill Info

  • Skill ID:
  • Category:
  • Description:

Checklist

  • SKILL.md has valid YAML frontmatter with all required fields
  • Skill ID matches the directory name
  • Category is one of: productivity, development, communication, writing, research, other
  • Instructions are clear and self-contained
  • Instructions are under 4000 tokens
  • No external URLs referenced in instructions
  • Tested the skill with an AI agent and it produces good results
  • Added usage examples in the frontmatter

Testing

Describe how you tested this skill:

Notes

Any additional context or notes for reviewers.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Mar 18, 2026

Greptile Summary

This PR standardises the YAML frontmatter of 20 Research category skills — adding requires: [] and examples fields, reordering fields for consistency, and trimming verbose descriptions. The majority of changes are clean housekeeping improvements that align each skill with the expected schema.

Issues found:

  • tavily-web/SKILL.md — Two problems: (1) an embedded external URL (https://github.com/BenedictKing/tavily-web) that violates the repository's "no external URLs" checklist requirement; (2) Step 1 of the guide reads "Install the skill using the command above" but no such command exists anywhere in the document, making the step non-functional.
  • scientific-brainstorming/SKILL.md — The Phase 2 instruction still references references/brainstorming_methods.md (line 99), but the section header was renamed to just ### references in this same PR, creating an internal inconsistency.
  • scikit-bio/SKILL.md — Three external URLs remain in the "Additional Resources" section at the bottom, which conflicts with the PR checklist requirement of no external URLs in instructions.
  • segment-combiner/SKILL.md — The example bash command hardcodes the absolute path /root/.claude/skills/segment-combiner/scripts/combine_segments.py, which is tied to a single specific installation environment and will fail for all other users.

Confidence Score: 3/5

  • Mostly safe to merge — the majority of files are clean frontmatter updates, but four files have notable issues that should be addressed before merging.
  • 16 out of 20 files are clean and well-formed. The four flagged files include a broken step in tavily-web (step references a non-existent command), an external URL policy violation in both tavily-web and scikit-bio, an internal reference inconsistency in scientific-brainstorming, and an environment-specific hardcoded path in segment-combiner. These are not critical runtime errors but do reduce the quality and reliability of the skill definitions.
  • Pay close attention to skills/Research/tavily-web/SKILL.md (broken step + external URL) and skills/Research/scikit-bio/SKILL.md (external URLs).

Important Files Changed

Filename Overview
skills/Research/scientific-brainstorming/SKILL.md Frontmatter updated with requires and examples; resource section header renamed from references/brainstorming_methods.md to references, but the inline reference at line 99 was not updated to match.
skills/Research/scikit-bio/SKILL.md Detailed bioinformatics skill with good coverage; however, three external URLs remain at the end of the file, which violates the project's "no external URLs" requirement.
skills/Research/segment-combiner/SKILL.md Frontmatter updated; example bash command contains a hardcoded absolute path (/root/.claude/skills/…) that is environment-specific and will break in other installations.
skills/Research/seo-cannibalization-detector/SKILL.md Frontmatter standardised; the opening "Use this skill when / Do not use this skill when" block reads as generic template boilerplate but the core skill content is otherwise detailed and actionable.
skills/Research/seo-keyword-strategist/SKILL.md Frontmatter updated; similar generic boilerplate section at the top as the SEO cannibalization skill; core content is detailed and useful.
skills/Research/tavily-web/SKILL.md Frontmatter updated but the skill body is very thin: it contains a broken "command above" reference, an external URL violating project guidelines, and largely defers to an external GitHub repository for examples and troubleshooting.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[PR #54: 20 Research SKILL.md files] --> B{Frontmatter updated?}
    B -->|Yes - all files| C[Added requires, examples fields]
    C --> D{Issues found?}
    D -->|scientific-brainstorming| E["⚠️ Stale inline reference\nreferences/brainstorming_methods.md\nvs renamed header 'references'"]
    D -->|scikit-bio| F["⚠️ 3 External URLs\nin instructions body"]
    D -->|tavily-web| G["❌ External URL\n+ broken 'command above' reference"]
    D -->|segment-combiner| H["⚠️ Hardcoded absolute path\n/root/.claude/skills/..."]
    D -->|14 other files| I["✅ Clean - no issues"]
Loading

Comments Outside Diff (4)

  1. skills/Research/scikit-bio/SKILL.md, line 248-250 (link)

    P2 External URLs referenced in skill instructions

    The PR checklist explicitly requires "No external URLs referenced in instructions," but this section contains three external URLs:

    • https://scikit.bio/docs/latest/
    • https://github.com/scikit-bio/scikit-bio
    • https://forum.qiime2.org

    These should be removed or replaced with a note to consult external documentation without embedding specific URLs.

  2. skills/Research/tavily-web/SKILL.md, line 30 (link)

    P2 External URL violates PR checklist requirement

    The PR checklist explicitly states "No external URLs referenced in instructions," but this line embeds a direct link to https://github.com/BenedictKing/tavily-web. This should be removed.

  3. skills/Research/tavily-web/SKILL.md, line 26 (link)

    P1 Broken reference to a non-existent command

    Step 1 reads "Install the skill using the command above," but there is no command anywhere above this line in the document. This appears to be leftover template text where an installation command (e.g., a claude skill install snippet) was never added, leaving the step both misleading and incomplete.

  4. skills/Research/segment-combiner/SKILL.md, line 70-73 (link)

    P2 Hardcoded environment-specific absolute path

    The example uses the absolute path /root/.claude/skills/segment-combiner/scripts/combine_segments.py, which is tied to a specific installation directory (/root/.claude/). This will not work in any other environment or for users with a different installation path. Consider replacing it with a relative path or a placeholder that is environment-agnostic:

Last reviewed commit: "Fix typo in performa..."

Comment thread skills/Research/swissweather/SKILL.md Outdated
Comment thread skills/Research/scikit-hep-analysis/SKILL.md Outdated
@markusha77 markusha77 merged commit eea3e4d into ChatAndBuild:main Mar 25, 2026
3 checks passed
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.

3 participants