Skip to content

fix: issues on postkit skill, repo migration changes, minor docs - #50

Open
Yasirunet wants to merge 1 commit into
mainfrom
fix/skill-path-issue
Open

fix: issues on postkit skill, repo migration changes, minor docs#50
Yasirunet wants to merge 1 commit into
mainfrom
fix/skill-path-issue

Conversation

@Yasirunet

Copy link
Copy Markdown
Contributor

fix: ship PostKit skills via npx skills + migrate repo to postkitstack org

Branch: fix/skill-path-issuedevelopment

Summary

  • Fixes agent skill packaging: npx skills add postkitstack/Postkit previously delivered zero PostKit skills and instead offered the repo's own internal contributor tooling, because agent/skills/ is not a directory the skills CLI searches.
  • Marks the seven internal .claude/skills/ workflows as internal so they are no longer shipped to end users, and fixes two of them that failed YAML parsing.
  • Migrates every GitHub reference from appritechnologies/postkit to the new postkitstack/Postkit org.

Changes

  • agent/skills/skills/: moved all four public skills (postkit-migrate, postkit-setup, postkit-schema, postkit-auth) with zero content changes. skills/ is a default discovery location for the skills CLI; agent/skills/ is not — it appears in the CLI only as an install destination for deepagents/eve, never as a source. The recursive fallback scan is gated on skills.length === 0, and .claude/skills/ was already yielding matches, so the public skills were never reachable.
  • .claude/skills/*/SKILL.md (7 files): added metadata: internal: true, which the skills CLI honours by excluding the skill from discovery, --list, and interactive install. Claude Code ignores the field, so /bugfix, /create-pr etc. keep working locally, and contributors can still fetch them with INSTALL_INTERNAL_SKILLS=1 or an explicit --skill <name>.
  • .claude/skills/{bugfix,create-feature}/SKILL.md: quoted descriptions containing : — unquoted, YAML parsed them as nested mappings and the CLI skipped both files with a parse error.
  • Docs — install guidance: replaced the recommended --all -y command in CLAUDE.md and docs/docs/agent-skills/overview.md with an explicit four---skill form. --all is shorthand for --skill '*' --agent '*' -y, so it both bypasses the internal-skill filter and fans the install out across every supported agent directory.
  • Docs — corrections in agent-skills/overview.md: the Scope table listed ./<agent>/skills/; real project paths are agent-specific (.claude/skills/ for Claude Code, .agents/skills/ shared by Cursor/Codex/Cline/Gemini CLI/Copilot, .windsurf/skills/, .roo/skills/). Added a "Verify the Install" step using npx skills list.
  • Docs — new Public vs Internal Skills section in CLAUDE.md: documents the two skill directories, the required internal flag, and the colon-quoting rule. Cross-referenced from "Adding a New Skill" and the Skills Registry, the two places a contributor would otherwise add a skill to the wrong directory.
  • Org migration (README.md, cli/README.md, cli/package.json, docs/docusaurus.config.ts, docs/src/pages/index.tsx, docs/docs/getting-started/installation.md, docs/docs/modules/db/troubleshooting.md): all GitHub URLs now point to postkitstack/Postkit, normalising the previously inconsistent postkit/Postkit casing. Also updates organizationName/projectName and both editUrls in the Docusaurus config, and corrects cd postkit/clicd Postkit/cli so the from-source clone instructions work on case-sensitive filesystems.

Type of Change

  • feat: New feature
  • fix: Bug fix
  • refactor: Code refactoring (no functional change)
  • test: Adding or updating tests
  • docs: Documentation changes
  • chore: Build, tooling, or CI changes

Test Plan

  • Unit tests pass (npm run test) — not run; no CLI source touched (changes are skill markdown, docs, and package metadata)
  • E2E tests pass (npm run test:e2e) (if applicable) — not applicable for the same reason
  • Build succeeds (npm run build) — docusaurus build for the docs site, which is meaningful here because onBrokenLinks: 'throw' is set and would fail on any broken cross-link
  • Manually tested: ran the real npx skills@1.5.22 add <repo> --list against a snapshot of the branch — returns exactly the four postkit-* skills with no warnings (before: 5 internal skills plus 2 YAML parse errors, and none of the public ones). Verified the documented CI command installs exactly those four into .claude/skills/ and nothing else; verified INSTALL_INTERNAL_SKILLS=1 still surfaces all 11 for contributors. Confirmed every documented agent id (claude-code, cursor, codex, cline, gemini-cli, github-copilot, windsurf, roo) and flag (-a, -g, -y, --skill, --agent, --copy, --list) exists in the CLI, and that npx skills list is a real subcommand. Checked no stale agent/ references remain in CI workflows, .gitignore, or build config.

Breaking Changes

  • No breaking changes

Note for reviewers: two related items were deliberately left out of scope — docs/blog/authors.yml still carries the appritechnologies author profile URL and title: Appri Technologies (author identity, a branding decision rather than a path move), and the published npm package remains @appritech/postkit (unaffected by the GitHub org move; renaming it is a coordinated npm change).

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