[PR #720] [MERGED] fix: remove broken rules/ directory references in python-expert skill #13168

New Issue

GiteaMirror · 2026-06-21T05:41:50-05:00

GiteaMirror commented

2026-06-21 05:41:50 -05:00

📋 Pull Request Information

Original PR: https://github.com/Shubhamsaboo/awesome-llm-apps/pull/720
Author: @Shubhamsaboo
Created: 4/15/2026
Status: ✅ Merged
Merged: 4/15/2026
Merged by: @Shubhamsaboo

Base: main ← Head: claude/cool-bell-fcHUs

📝 Commits (1)

5b50395 fix: remove broken rules/ directory references in python-expert skill

📊 Changes

2 files changed (+10 additions, -28 deletions)

View changed files

📝 awesome_agent_skills/python-expert/AGENTS.md (+0 -17)
📝 awesome_agent_skills/python-expert/SKILL.md (+10 -11)

📄 Description

Summary

Fixes #660.

The python-expert skill referenced a rules/ directory that does not exist. SKILL.md linked to 8 non-existent rule files and AGENTS.md ended each rule section with a "Full details" link pointing into the same missing directory. The "References" section also listed the missing directory.

Fix

AGENTS.md already contains the complete rule content inline with examples, so the referenced deep-dive files are redundant. This PR:

AGENTS.md: removes the 8 broken [➡️ Full details: ...](rules/...) links and the missing-directory bullet in "References"
SKILL.md: changes the 8 "Available Rules" bullets to link directly into AGENTS.md via section anchors, and rewrites the "How to Use This Skill" paragraph and Quick Start to stop pointing at the non-existent rules/ directory

Net diff: AGENTS.md -17 lines, SKILL.md +10/-11 lines.

Test plan

Grep for rules/ inside awesome_agent_skills/python-expert/ returns no matches
All [Rule Name](AGENTS.md#anchor) links in SKILL.md point at existing section headings in AGENTS.md
AGENTS.md still renders with all 8 rules fully explained (content was inline, not in the removed links)

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Shubhamsaboo/awesome-llm-apps/pull/720 **Author:** [@Shubhamsaboo](https://github.com/Shubhamsaboo) **Created:** 4/15/2026 **Status:** ✅ Merged **Merged:** 4/15/2026 **Merged by:** [@Shubhamsaboo](https://github.com/Shubhamsaboo) **Base:** `main` ← **Head:** `claude/cool-bell-fcHUs` --- ### 📝 Commits (1) - [`5b50395`](https://github.com/Shubhamsaboo/awesome-llm-apps/commit/5b503953550069b5784971e145c39623a02fb018) fix: remove broken rules/ directory references in python-expert skill ### 📊 Changes **2 files changed** (+10 additions, -28 deletions) <details> <summary>View changed files</summary> 📝 `awesome_agent_skills/python-expert/AGENTS.md` (+0 -17) 📝 `awesome_agent_skills/python-expert/SKILL.md` (+10 -11) </details> ### 📄 Description ## Summary Fixes #660. The `python-expert` skill referenced a `rules/` directory that does not exist. `SKILL.md` linked to 8 non-existent rule files and `AGENTS.md` ended each rule section with a "Full details" link pointing into the same missing directory. The "References" section also listed the missing directory. ## Fix `AGENTS.md` already contains the complete rule content inline with examples, so the referenced deep-dive files are redundant. This PR: - **AGENTS.md**: removes the 8 broken `[➡️ Full details: ...](rules/...)` links and the missing-directory bullet in "References" - **SKILL.md**: changes the 8 "Available Rules" bullets to link directly into `AGENTS.md` via section anchors, and rewrites the "How to Use This Skill" paragraph and Quick Start to stop pointing at the non-existent `rules/` directory Net diff: `AGENTS.md -17 lines`, `SKILL.md +10/-11 lines`. ## Test plan - [x] Grep for `rules/` inside `awesome_agent_skills/python-expert/` returns no matches - [x] All `[Rule Name](AGENTS.md#anchor)` links in SKILL.md point at existing section headings in AGENTS.md - [x] AGENTS.md still renders with all 8 rules fully explained (content was inline, not in the removed links) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2026-06-21 05:41:50 -05:00

GiteaMirror closed this issue

2026-06-21 05:41:51 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/awesome-llm-apps#13168