Skip to content

fix(expo-deployment): unify reference paths to ./references/ (plural)#51

Open
xiaolai wants to merge 1 commit into
expo:mainfrom
xiaolai:fix/nlpm-deployment-reference-paths
Open

fix(expo-deployment): unify reference paths to ./references/ (plural)#51
xiaolai wants to merge 1 commit into
expo:mainfrom
xiaolai:fix/nlpm-deployment-reference-paths

Conversation

@xiaolai

@xiaolai xiaolai commented Apr 27, 2026

Copy link
Copy Markdown

Automated audit: This PR was generated by NLPM, a natural language programming linter, running via claude-code-action. Please evaluate the diff on its merits.

Bug

plugins/expo/skills/expo-deployment/SKILL.md has an inconsistency in its reference paths. The top-level References section (lines 16–20) correctly uses ./references/ (plural), but five in-body See … links use ./reference/ (singular):

  • Line 123: See ./reference/testflight.md
  • Line 124: See ./reference/ios-app-store.md
  • Line 130: See ./reference/play-store.md
  • Line 136: See ./reference/workflows.md
  • Line 165: See ./reference/workflows.md

The references/ directory exists; reference/ does not. Claude follows these links to fetch supplementary docs at load time — the singular-form links silently resolve to nothing, making half the contextual references unreachable.

Fix: Replace all ./reference/ occurrences in the body text with ./references/ to match the actual directory name and the top-level References section.

Co-Authored-By: Claude Code <noreply@anthropic.com>
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.

1 participant