chore: slim public repo to distribution-only (smoke + notify CI, remove .internal)#19
Merged
Merged
Conversation
Remove the .internal/ manifest generator and reduce CI to smoke tests plus the docs notification. The build/validate gate already runs upstream in the private dev repo before each release, and the release sync never touches .internal/, .github/, README, or LICENSE — so the public repo only needs the vendor dot-folders + their payload, minimal CI, and hand-owned README/LICENSE. - Delete .internal/ (generator, schemas, package.json, build scripts) - ci.yml: drop the validate job and npm build steps; smoke-test the committed layout only (claude plugin validate; codex install assertion); pin Node 22; drop the npm cache (no lockfile) - notify-docs.yml: add a paths filter so README/LICENSE/.github edits don't trigger the docs sync - update .github/workflows/README.md and tidy .gitignore Both Claude and Codex smoke tests pass against the committed layout. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
Turns the public repo into a lean distribution mirror. Skills, commands, and vendor manifests are generated and synced in from a private repo via automated "chore(release): publish content" PRs, and the build/validate gate already runs upstream there before each release. So the public repo only needs the vendor dot-folders + their payload, minimal CI, and hand-owned README/LICENSE.
Changes
.internal/— the manifest generator, schemas,package.json, and build scripts. The dev→public release sync never touches.internal/(not in the release allowlist), so removing it here is permanent and safe.ci.yml→ smoke tests only — dropped thevalidatejob and allnpm ci/npm run buildsteps (no generator to run; manifests are committed). Now just installs the Claude/Codex CLIs and validates the committed layout. Pinned Node 22; dropped the npm cache (no lockfile in this repo anymore).notify-docs.yml— added apaths:filter so the docsrepository_dispatchonly fires on published content (skills, commands, vendor manifests,skills.sh.json). EditingREADME.md/LICENSE/.github/no longer triggers a docs sync..github/workflows/README.md— updated to describe the distribution-mirror model..gitignore— reworded the comment that referenced.internal/(rules unchanged).Kept untouched:
.claude-plugin/,.cursor-plugin/,.agents/,skills/,commands/,assets/,skills.sh.json,README.md,LICENSE,LICENSE-MIT.Verification
Both vendor smoke tests pass locally against the committed layout (the same commands CI runs):
claude plugin validate ./→ ✔ Validation passedcodex plugin marketplace add ./ && codex plugin add commercetools@commercetools && codex plugin list→commercetools@commercetools installed, enabledNo
.internalreferences remain; both workflow YAMLs parse cleanly.Notes
.github/is not in the dev release allowlist, so these workflow edits won't be overwritten by future release PRs.assets/commercetools-symbol.svgis referenced by the Cursor manifest and is intentionally kept (hand-maintained — not part of the release sync).notify-docs(changed paths don't match the new content filter).🤖 Generated with Claude Code