Skip to content

fix: add missing YAML frontmatter to ship reference docs#182

Open
xiaolai wants to merge 1 commit intoComposioHQ:masterfrom
xiaolai:fix/nlpm-ship-reference-frontmatter
Open

fix: add missing YAML frontmatter to ship reference docs#182
xiaolai wants to merge 1 commit intoComposioHQ:masterfrom
xiaolai:fix/nlpm-ship-reference-frontmatter

Conversation

@xiaolai
Copy link
Copy Markdown

@xiaolai xiaolai commented Apr 21, 2026

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

Three reference command files in the ship/ plugin were missing YAML frontmatter entirely:

  • ship/commands/ship-deployment.md
  • ship/commands/ship-ci-review-loop.md
  • ship/commands/ship-error-handling.md

Claude Code requires a YAML frontmatter block (with at minimum a description field) to register and describe command files. Without it, Claude Code may enumerate these files as commands but cannot describe them, and standalone invocation fails silently. The parent ship.md correctly references these as sub-documents, but the missing frontmatter creates ambiguity for the runtime.

Fix

Added minimal YAML frontmatter to each file:

---
description: "Reference: <purpose> for /ship — <detail>. Not a standalone command."
allowed-tools: []
---

The allowed-tools: [] explicitly marks these as internal reference documents that don't need direct tool access (the parent ship.md declares the full tool budget). The descriptions clarify their role as sub-documents referenced by /ship.

Impact

Without this fix, users who attempt to invoke these files directly (e.g., /ship-deployment) get no useful feedback. The frontmatter makes Claude Code's behavior deterministic for these files.

Claude Code requires YAML frontmatter with at least a `description` field
to register command files. The three ship reference docs (ship-deployment,
ship-ci-review-loop, ship-error-handling) were missing frontmatter entirely,
causing Claude Code to fail to describe them while potentially enumerating
them as commands. Added minimal frontmatter marking them as internal
reference docs with `allowed-tools: []`.

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