Skip to content

docs(agents): add internal meta-issue-creation and meta-pr-creation skills - #919

Open
coderdan wants to merge 2 commits into
mainfrom
docs/meta-agent-skills
Open

docs(agents): add internal meta-issue-creation and meta-pr-creation skills#919
coderdan wants to merge 2 commits into
mainfrom
docs/meta-agent-skills

Conversation

@coderdan

@coderdan coderdan commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Agents working on this repo now file issues and open PRs regularly, with no documented structure — quality has depended on the session. This adds two internal skills setting the conventions: a required issue shape (Background / Problem / Proposal) written for a reader who is new to CipherStash, and the branch/commit/PR authoring rules (signed commits, the changeset checklist, PR body sections).

They live in .claude/skills/, not skills/ — everything under skills/ ships to customers inside the stash npm tarball (packages/cli/tsup.config.ts copies the whole directory), and these are internal-only. The meta- prefix marks them as repo-process skills, distinct from the customer-facing stash-* set.

Changes

  • .claude/skills/meta-issue-creation/SKILL.md — pre-filing checks (search existing issues open and closed, verify claims against code, pick the right repo), the required section structure, dumbed-down wording rules, cross-linking in both directions.
  • .claude/skills/meta-pr-creation/SKILL.md — branch/commit conventions, signed-commit verification (%G?) and the re-sign rebase recipe, the AGENTS.md pre-PR checklist by pointer, PR body structure, force-push etiquette.

Both encode two reference traps from recent sessions: bare #N autolinks to an unrelated issue unless a GitHub reference is intended, and internal Linear (CIP-) numbers must never appear on GitHub.

Verification

Docs-only; no build or tests apply. Checked that .claude/skills/ is outside every tarball copy path (tsup.config.ts copies skills/ and doctrine/ only). No changeset: .claude/ is internal and ships nowhere, per the changeset rule these skills themselves document.

Related

Conventions distilled from the sessions around #895/#897 (skill corrections), #896 (hazard comments), and #918 (issue filing).

…kills

Agent-authored issues and PRs on this repo have had no documented
structure. These two skills capture the conventions that worked in
practice: Background / Problem / Proposal issue sections written for a
reader new to CipherStash, signed-commit verification and the rebase
recipe for an unsigned commit, the changeset/skills/meta-file
checklist by pointer to AGENTS.md, and the reference traps (bare #N
autolinks; Linear CIP- numbers never appear on GitHub).

They live in .claude/skills/ deliberately: skills/ ships to customers
inside the stash tarball via packages/cli/tsup.config.ts, and these
are internal-only.
@coderdan
coderdan requested a review from a team as a code owner August 19, 2026 03:33
@changeset-bot

changeset-bot Bot commented Aug 19, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: c87e7eb

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderdan
coderdan marked this pull request as draft August 19, 2026 03:50
@coderdan
coderdan marked this pull request as ready for review August 19, 2026 04:04
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