refactor(cli): remove AI scaffolding and skills management - #27
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
There was a problem hiding this comment.
🔵 Needs a closer look
This is a broad breaking-surface refactor (commands, embedded assets, tests, docs) that should get a final human pass to confirm no missed integration points remain.
Pull request overview
This PR removes the CLI’s AI/agent scaffolding and bundled skill distribution features, updating the CLI implementation, embedded resources pipeline, tests, and documentation to match the new (breaking) product surface.
Changes:
- Removes the
one skillscommand surface, the bundled skills payload, and the exportedpackages/cli/pkg/agentskillsAPI. - Stops generating workspace agent docs (
AGENTS.md,CLAUDE.md,.one/…) and strips retired agent assets from embedded templates/resources. - Updates docs/site copy and reference snapshots to reflect the new behavior and JSON envelopes.
File summaries
| File | Description |
|---|---|
| Taskfile.yml | Updates task docs and inputs for the new bundled-resources layout; removes skills docs from verification. |
| README.md | Removes “install skills” onboarding and reframes agent usage around manifest + JSON output. |
| packages/templates/starlight-docs/CLAUDE.md | Removes template-level agent guide. |
| packages/templates/go-lib/README.md.hbs | Removes references to .one/agents guidance and simplifies layout guidance. |
| packages/skills/one-migrate/references/INDEX.md | Removes retired skill reference content. |
| packages/skills/one-cli/references/INDEX.md | Removes retired skill reference content. |
| packages/skills/one-cli/references/env-consumption.md | Removes retired skill reference content. |
| packages/skills/one-cli/references/dependencies.md | Removes retired skill reference content. |
| packages/cli/tools/verify-cli-references/main.go | Stops scanning skills docs; keeps doc scanning focused on docs + templates. |
| packages/cli/tools/sync-resources/main.go | Stops bundling packages/skills; filters retired agent assets from embedded templates. |
| packages/cli/tools/sync-resources/main_test.go | Adds coverage to ensure retired agent assets are not bundled. |
| packages/cli/tools/gen-error-codes/main.go | Removes the “Agent 文档 / Skills” error-code group from generated docs. |
| packages/cli/tests/e2e/snapshot_e2e_create_test.go | Ensures one create no longer generates agent docs/.one and asserts absence. |
| packages/cli/tests/e2e/snapshot_e2e_create_preset_test.go | Ensures preset create also does not generate agent docs/.one. |
| packages/cli/tests/e2e/snapshot_e2e_configure_test.go | Removes e2e coverage for the retired skills install command. |
| packages/cli/tests/e2e/e2e_helpers_test.go | Removes skills-install volatility keys and updates helper docs accordingly. |
| packages/cli/testdata/reference/skills-install-claude-code.json | Removes retired snapshot payload. |
| packages/cli/testdata/reference/help/skills.txt | Removes retired help snapshot. |
| packages/cli/testdata/reference/help/skills_install.txt | Removes retired help snapshot. |
| packages/cli/testdata/reference/create-preset-fullstack.json | Removes skills field from create preset JSON reference. |
| packages/cli/testdata/reference/create-default.json | Removes skills field from create JSON reference. |
| packages/cli/testdata/reference/add-go-api.json | Removes ai_guides payload from add JSON reference. |
| packages/cli/pkg/agentskills/types.go | Removes exported agent-skill install API types. |
| packages/cli/pkg/agentskills/detect.go | Removes exported agent detection/path resolution code. |
| packages/cli/pkg/agentskills/agentskills_test.go | Removes tests for the deleted exported agentskills API. |
| packages/cli/internal/transport/cobra/create/result.go | Removes skills payload from create envelope and TTY guidance. |
| packages/cli/internal/transport/cobra/create/preset.go | Removes skills payload from preset-create envelope. |
| packages/cli/internal/transport/cobra/create/create.go | Removes UI copy referring to skill selection and removes skills payload emission. |
| packages/cli/internal/transport/cobra/add/cmd.go | Removes ai_guides from add envelope and removes AI module dependency. |
| packages/cli/internal/resources/bundled/bundled.go | Removes embedded skills filesystem; bundled assets are now registry + templates + web UI. |
| packages/cli/internal/platform/i18n/locales/zh-CN.json | Removes i18n strings for retired skills commands. |
| packages/cli/internal/platform/i18n/locales/en-US.json | Removes i18n strings for retired skills commands. |
| packages/cli/internal/platform/errors/codes.go | Removes AI/skills error codes and their definitions. |
| packages/cli/internal/platform/errors/codes_test.go | Removes deleted AI/skills codes from completeness tests. |
| packages/cli/internal/modules/skills/skills_test.go | Removes tests for the deleted skills installation module. |
| packages/cli/internal/modules/preset/codes.go | Removes docs reference to SKILL.md; points to preset docs generically. |
| packages/cli/internal/modules/creation/workspace_files.go | Stops generating AGENTS.md/CLAUDE.md/.one/agents on workspace create. |
| packages/cli/internal/modules/creation/workspace_files_test.go | Updates test commentary to remove agent-doc generation claims. |
| packages/cli/internal/modules/creation/workspace_content.go | Removes agent-doc content generators and AI module dependency. |
| packages/cli/internal/modules/creation/service.go | Removes AI refresh from project add flow. |
| packages/cli/internal/modules/creation/preset.go | Removes AI refresh from preset application flow. |
| packages/cli/internal/core/template/render.go | Skips .one and agent instruction files when rendering embedded templates. |
| packages/cli/internal/bootstrap/cli/root.go | Unregisters the skills command group from the CLI root. |
| packages/cli/internal/bootstrap/cli/root_test.go | Updates known/removed subcommand lists to reflect skills removal. |
| CONTRIBUTING.md | Updates contributor docs to remove skills guidance and adjust bundle description. |
| CHANGELOG.md | Documents breaking removals for skills distribution and generated agent docs. |
| ARCHITECTURE.md | Updates module/resource descriptions to remove ai/skills components. |
| apps/docs/src/data/examples.ts | Updates prompt copy to refer to One CLI directly (not “one-cli skill”). |
| apps/docs/src/app/docs/tutorials-sidebar.tsx | Removes “skills install” tutorial link from docs sidebar. |
| apps/docs/src/app/docs/docs-sidebar.tsx | Removes “Skills” page link from docs sidebar. |
| apps/docs/src/app/[lang]/templates/page.tsx | Updates SEO copy to remove “one-cli skill” language. |
| apps/docs/src/app/[lang]/blog/page.tsx | Updates blog landing description to remove “agent skills” framing. |
| apps/docs/src/app/(home)/hero-canvas.tsx | Renames “AI context” module to “CLI interface” in homepage hero content. |
| apps/docs/public/llms.txt | Removes bundled-skill mentions and skills links from LLM index. |
| apps/docs/public/install.ps1 | Updates post-install instruction to one --help instead of one skills install. |
| apps/docs/design/README.md | Adds archived-design disclaimer to prevent restoring retired skills/agent features. |
| apps/docs/content/tutorials/zh/meta.json | Removes skills-install tutorial from tutorial index. |
| apps/docs/content/tutorials/zh/json-output-error-codes.mdx | Removes skill dependency in agent guidance section. |
| apps/docs/content/tutorials/zh/first-workspace.mdx | Removes generated AGENTS/CLAUDE and skills-install sections. |
| apps/docs/content/tutorials/zh/configure-profiles.mdx | Removes “bundled skill” phrasing; keeps error-code guidance. |
| apps/docs/content/tutorials/en/meta.json | Removes skills-install tutorial from tutorial index. |
| apps/docs/content/tutorials/en/json-output-error-codes.mdx | Removes skill dependency in agent guidance section. |
| apps/docs/content/tutorials/en/first-workspace.mdx | Removes generated AGENTS/CLAUDE and skills-install sections. |
| apps/docs/content/tutorials/en/configure-profiles.mdx | Removes “bundled skill” phrasing; keeps error-code guidance. |
| apps/docs/content/docs/zh/quick-start.md | Updates “Claude help” link to AI-native guidance instead of skills install. |
| apps/docs/content/docs/zh/meta.json | Removes “skills” from docs nav ordering. |
| apps/docs/content/docs/zh/manifest.md | Removes manifest field docs tied to AI provider generation. |
| apps/docs/content/docs/zh/installation.md | Removes skills-install steps and cleanup instructions. |
| apps/docs/content/docs/zh/error-codes.md | Removes AI/skills error code group; adds/adjusts run-usage guidance section. |
| apps/docs/content/docs/zh/create.md | Removes skills language and removes skills from example envelope. |
| apps/docs/content/docs/zh/cli-overview.md | Removes skills command documentation and skills install section. |
| apps/docs/content/docs/zh/ai-native.md | Updates “AI-native” guidance to team-maintained docs instead of generated agent harness. |
| apps/docs/content/docs/zh/add.md | Removes ai_guides from example output and removes AI-doc refresh claims. |
| apps/docs/content/docs/en/quick-start.md | Updates “Claude help” link to AI-native guidance instead of skills install. |
| apps/docs/content/docs/en/meta.json | Removes “skills” from docs nav ordering. |
| apps/docs/content/docs/en/manifest.md | Removes manifest field docs tied to AI provider generation. |
| apps/docs/content/docs/en/installation.md | Removes skills-install steps and cleanup instructions. |
| apps/docs/content/docs/en/error-codes.md | Removes AI/skills error code group. |
| apps/docs/content/docs/en/create.md | Removes skills language and removes skills from example envelope. |
| apps/docs/content/docs/en/cli-overview.md | Removes skills command documentation and skills install section. |
| apps/docs/content/docs/en/ai-native.md | Updates “AI-native” guidance to team-maintained docs instead of generated agent harness. |
| apps/docs/content/docs/en/add.md | Removes ai_guides from example output and removes AI-doc refresh claims. |
| apps/docs/content/blog/zh/template-governance-for-ai-workspaces.md | Reframes agent guidance as team-maintained instructions (not bundled skill). |
| apps/docs/content/blog/zh/preset-id-boundary.md | Removes skills from “small surface” command list. |
| apps/docs/content/blog/zh/one-cli-vs-general-scaffolding.md | Reframes skills as team-maintained project guidance. |
| apps/docs/content/blog/zh/manifest-as-contract.md | Reframes skills as team-maintained project guidance. |
| apps/docs/content/blog/zh/coding-agent-workspace-setup.md | Removes “One CLI skill rule” phrasing; keeps toolchain-based guidance. |
| apps/docs/content/blog/zh/ai-native-monorepo-cli.md | Reframes skills as team-maintained project guidance. |
| apps/docs/content/blog/zh/agentic-engineering.md | Replaces skills reference with AI-native guidance reference. |
| apps/docs/content/blog/zh/agent-skill-context.md | Marks the post as historical design and adds pointer to current workflow. |
| apps/docs/content/blog/en/template-governance-for-ai-workspaces.md | Reframes agent guidance as team-maintained instructions (not bundled skill). |
| apps/docs/content/blog/en/preset-id-boundary.md | Removes skills from “small surface” command list. |
| apps/docs/content/blog/en/one-cli-vs-general-scaffolding.md | Reframes skills as team-maintained project guidance. |
| apps/docs/content/blog/en/manifest-as-contract.md | Reframes skills as team-maintained project guidance. |
| apps/docs/content/blog/en/coding-agent-workspace-setup.md | Removes “One CLI skill rule” phrasing; keeps toolchain-based guidance. |
| apps/docs/content/blog/en/ai-native-monorepo-cli.md | Reframes skills as team-maintained project guidance. |
| apps/docs/content/blog/en/agentic-engineering.md | Replaces skills reference with AI-native guidance reference. |
| apps/docs/content/blog/en/agent-skill-context.md | Marks the post as historical design and adds pointer to current workflow. |
| .gitignore | Removes ignored bundled skills/ path; bundle is now registry + templates + web. |
| .github/workflows/ci.yml | Updates CI comment to reflect new bundled tree shape. |
| .agents/skills/vercel-react-best-practices/rules/server-serialization.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/server-parallel-nested-fetching.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/server-parallel-fetching.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/server-no-shared-module-state.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/server-dedup-props.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/server-cache-react.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/server-cache-lru.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/server-auth-actions.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/server-after-nonblocking.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-use-ref-transient-values.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-use-deferred-value.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-transitions.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-split-combined-hooks.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-simple-expression-in-memo.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-no-inline-components.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-move-effect-to-event.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-memo.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-memo-with-default-value.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-lazy-state-init.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-functional-setstate.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-derived-state.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-derived-state-no-effect.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-dependencies.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rerender-defer-reads.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rendering-usetransition-loading.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rendering-svg-precision.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rendering-script-defer-async.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rendering-resource-hints.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rendering-hydration-suppress-warning.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rendering-hydration-no-flicker.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rendering-hoist-jsx.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rendering-content-visibility.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rendering-conditional-render.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rendering-animate-svg-wrapper.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/rendering-activity.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-tosorted-immutable.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-set-map-lookups.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-request-idle-callback.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-min-max-loop.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-length-check-first.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-index-maps.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-hoist-regexp.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-flatmap-filter.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-early-exit.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-combine-iterations.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-cache-storage.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-cache-property-access.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/js-cache-function-results.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/client-swr-dedup.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/client-passive-event-listeners.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/client-localstorage-schema.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/client-event-listeners.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/bundle-preload.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/bundle-dynamic-imports.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/bundle-defer-third-party.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/bundle-conditional.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/bundle-barrel-imports.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/bundle-analyzable-paths.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/async-suspense-boundaries.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/async-parallel.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/async-dependencies.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/async-defer-await.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/async-cheap-condition-before-await.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/async-api-routes.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/advanced-use-latest.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/advanced-init-once.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/advanced-event-handler-refs.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/advanced-effect-event-deps.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/_template.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/rules/_sections.md | Removes retired agent skill content. |
| .agents/skills/vercel-react-best-practices/metadata.json | Removes retired agent skill metadata file. |
| .agents/skills/tdd/tests.md | Removes retired agent skill content. |
| .agents/skills/tdd/SKILL.md | Removes retired agent skill content. |
| .agents/skills/tdd/mocking.md | Removes retired agent skill content. |
| .agents/skills/tdd/agents/openai.yaml | Removes retired agent skill agent mapping. |
| .agents/skills/improve-codebase-architecture/agents/openai.yaml | Removes retired agent skill agent mapping. |
| .agents/skills/full-output-enforcement/SKILL.md | Removes retired agent skill content. |
Review details
Suppressed comments (1)
packages/cli/tools/verify-cli-references/main.go:130
- The repo no longer contains a top-level CLAUDE.md, but docFiles() still includes it in the top-level scan list. Dropping it keeps the scan contract aligned with the refactor and avoids carrying a stale path.
- Files reviewed: 228/238 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: caorushizi <[email protected]>
This branch was successfully deployed
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.
No description provided.