Skip to content

fix(pdf): correct case-sensitive file references in SKILL.md - #538

Open
Lubrsy706 wants to merge 1 commit into
anthropics:mainfrom
Lubrsy706:fix/pdf-case-sensitive-references
Open

Lubrsy706 wants to merge 1 commit into
anthropics:mainfrom
Lubrsy706:fix/pdf-case-sensitive-references

Conversation

@Lubrsy706

Copy link
Copy Markdown

Summary

  • Fix 8 case-sensitivity mismatches in skills/pdf/SKILL.md
  • REFERENCE.md → reference.md (4 occurrences)
  • FORMS.md → forms.md (4 occurrences)

The actual files are lowercase (forms.md, reference.md), but SKILL.md references them in uppercase. This breaks on case-sensitive filesystems (Linux, macOS APFS case-sensitive) where the skill may fail to resolve these references.

Closes #375

Test plan

  • Verified actual filenames are lowercase: forms.md, reference.md
  • All 8 references updated to match actual filenames
  • No other files affected

🤖 Generated with Claude Code

SKILL.md references FORMS.md and REFERENCE.md in uppercase, but the
actual files are forms.md and reference.md (lowercase). This breaks on
case-sensitive filesystems (Linux, macOS APFS case-sensitive).

Fixes 8 occurrences: 4x REFERENCE.md → reference.md, 4x FORMS.md → forms.md

Closes anthropics#375
@Lubrsy706
Lubrsy706 force-pushed the fix/pdf-case-sensitive-references branch from 666e73e to c3235fc Compare March 6, 2026 15:22
Jah-yee added a commit to Jah-yee/skills-1 that referenced this pull request Mar 6, 2026
The actual files are lowercase (forms.md, reference.md), but SKILL.md
referenced them in uppercase. This breaks on case-sensitive filesystems
(Linux, macOS APFS case-sensitive).

Fixes: anthropics#538
@Lubrsy706

Copy link
Copy Markdown
Author

Friendly ping: this PR has been ready for review since March 6, 2026. If there is anything blocking it or if you would prefer changes before merge, let me know and I can update it. Would appreciate a review when someone has time.

@Lubrsy706

Copy link
Copy Markdown
Author

Following up on this PR. The fix should be ready for review; please let me know if you'd like any updates or additional test coverage from my side.

@Lubrsy706

Copy link
Copy Markdown
Author

Following up on this PR as well. The change should be ready for review from my side; if you would prefer any updates or additional coverage before merge, let me know and I can handle it. Would appreciate review when someone has time.

@Lubrsy706

Copy link
Copy Markdown
Author

Following up on this PR. The case-sensitivity fix in the PDF skill docs should be ready for review from my side; if you would like any adjustments or extra validation, I can handle that. Would appreciate a review when someone has time.

@stefanv

stefanv commented Apr 29, 2026

Copy link
Copy Markdown

This PR correctly addresses an issue in the PDF skill: namely, that it incorrectly refers to filenames (REFERENCE.md instead of reference.md).

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.

Fix case sensitivity issues in pdf skill references

2 participants