Refresh Vite+ skill and publish workflow hardening#6
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR refreshes the Vite+ skill to match current upstream naming/command guidance and hardens the skills publish workflow by separating review from publish and gating publish behind a protected GitHub Environment.
Changes:
- Renames the skill from
viteplustovite-plus(new tile nameuinaf/vite-plus) and updates skill metadata accordingly. - Updates Vite+ guidance across the skill docs/references/evals (commands, CI/CD, bootstrap/editor behavior, caching/task inputs, known issues).
- Hardens publishing by introducing a secretless review job and gating publish behind the
releaseEnvironment, while pinning publish-path actions to SHAs.
Reviewed changes
Copilot reviewed 29 out of 29 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| skills/viteplus/tile.json | Removes the old uinaf/viteplus tile metadata as part of the rename. |
| skills/viteplus/agents/openai.yaml | Removes the old viteplus agent interface definition. |
| skills/vite-plus/tile.json | Adds the new uinaf/vite-plus tile metadata and skill identifier. |
| skills/vite-plus/SKILL.md | Renames the skill and refreshes Vite+ workflow guidance and command surface. |
| skills/vite-plus/references/testing.md | Updates testing guidance wording to Vite+ terminology. |
| skills/vite-plus/references/packages.md | Updates packages guidance wording to Vite+ terminology. |
| skills/vite-plus/references/monorepos.md | Updates monorepo guidance and adds cache/input guidance for tasks. |
| skills/vite-plus/references/known-issues.md | Refreshes known-issues framing and removes outdated hook-runner section. |
| skills/vite-plus/references/commands.md | Expands/updates command reference (env/install/rebuild, preview, check flags). |
| skills/vite-plus/references/ci-cd.md | Updates CI/CD guidance wording to Vite+ terminology and expectations. |
| skills/vite-plus/references/bootstrap.md | Updates bootstrap guidance (create/migrate editor behavior, packaged docs guidance). |
| skills/vite-plus/evals/vite-plus-version-upgrade-workflow/task.md | Updates eval task text to Vite+ naming and new script name. |
| skills/vite-plus/evals/vite-plus-version-upgrade-workflow/criteria.json | Updates eval criteria text to Vite+ naming. |
| skills/vite-plus/evals/test-import-and-coverage-migration/task.md | Updates eval task text to Vite+ naming. |
| skills/vite-plus/evals/test-import-and-coverage-migration/criteria.json | Updates eval criteria text to Vite+ naming. |
| skills/vite-plus/evals/monorepo-build-chain-orchestration/task.md | Updates eval task text to Vite+ naming. |
| skills/vite-plus/evals/monorepo-build-chain-orchestration/criteria.json | Updates eval criteria text to Vite+ naming. |
| skills/vite-plus/evals/library-packaging-and-hook-configuration/task.md | Updates eval task text to Vite+ naming. |
| skills/vite-plus/evals/library-packaging-and-hook-configuration/criteria.json | Updates eval criteria text to Vite+ naming. |
| skills/vite-plus/evals/config-consolidation-and-type-aware-lint/task.md | Updates eval task text to Vite+ naming. |
| skills/vite-plus/evals/config-consolidation-and-type-aware-lint/criteria.json | Updates eval criteria text to Vite+ naming. |
| skills/vite-plus/evals/ci-cd-migration-to-vite-plus/task.md | Updates eval task text to Vite+ naming. |
| skills/vite-plus/evals/ci-cd-migration-to-vite-plus/criteria.json | Updates eval criteria text to Vite+ naming. |
| skills/vite-plus/agents/openai.yaml | Adds the new vite-plus agent interface definition and prompt. |
| skills/uinaf-design-system/references/repo-docs.md | Updates references to point to the Vite+ skill and terminology. |
| skills/gh-release-pipeline/references/workflows.md | Updates a workflow snippet label from VitePlus to Vite+. |
| scripts/sync/skills.json | Updates the synced skill name from viteplus to vite-plus. |
| docs/distribution.md | Documents the new review→environment-gated publish flow and token storage in release. |
| .github/workflows/publish-skills.yml | Splits review/publish, gates publish behind release, and pins actions to SHAs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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.
Summary
Refresh the Vite+ skill package and harden the skill publishing workflow.
Changed
viteplustovite-plusand updated catalog metadata.vp dev,vp preview,vp check --no-lint, create/migrate editor behavior, packaged docs, and task input cache guidance.Risks
uinaf/viteplustouinaf/vite-plus; consumers of the old name need to move to the new tile.agents/openai.yamltile lint placement warning remains unchanged.Verification
npx tessl skill review skills/vite-plusnpx tessl tile lint skills/vite-plus./scripts/skills/review.shgit grepfor staleviteplusskill slugsgit diff --checkComplexity
Neutral: this keeps the skill surface aligned with the upstream tool name and current Vite+ behavior without adding new workflow machinery.