[docs] Update documentation for features from 2026-05-04#1132
Open
danielmeppiel wants to merge 1 commit intomainfrom
Open
[docs] Update documentation for features from 2026-05-04#1132danielmeppiel wants to merge 1 commit intomainfrom
danielmeppiel wants to merge 1 commit intomainfrom
Conversation
…uting - Fix skill integration section: primary target is now .agents/skills/ (converged for Copilot, Cursor, OpenCode, Codex, Gemini); Claude keeps .claude/skills/. Document --legacy-skill-paths escape hatch. - Fix uninstall table: skill folder path updated to .agents/skills/ - Add Cursor slash command support (Cursor 1.6+) to auto-detection note - Add Performance and Progress UI section documenting parallel BFS, persistent two-tier cache (git + ETag HTTP), live ASCII progress bar, elapsed time on all exit paths, and per-phase timing in --verbose - Update --verbose description to mention per-phase timing Refs: #1116 (perf+ux overhaul), #1046 (Cursor slash commands) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the apm install and integration-path documentation to reflect recently shipped behavior (install performance/UI work, Cursor command deployment, and converged skill routing), keeping the CLI reference aligned with current defaults.
Changes:
- Adds a new
apm installsection describing parallel resolution/download, caching, progress UI, and timing output. - Updates Cursor auto-detection notes to mention Cursor 1.6+ slash command deployment to
.cursor/commands/. - Updates skill path documentation to reflect converged deployment to
.agents/skills/with a--legacy-skill-pathsescape hatch, and adjusts the uninstall table accordingly.
Show a summary per file
| File | Description |
|---|---|
| docs/src/content/docs/reference/cli-commands.md | Documents new apm install performance/progress behaviors and updates integration path guidance (Cursor commands + converged skill routing). |
Copilot's findings
Comments suppressed due to low confidence (3)
docs/src/content/docs/reference/cli-commands.md:175
- The 'Live progress' bullet reads as unconditional, but the install TUI/progress bar is gated (TTY-only by default, disabled in CI, and controllable via
APM_PROGRESS=auto|always|never). Consider documenting those conditions here so readers don't expect a progress bar in non-interactive/CI logs.
- **In-run dedup**: Multiple subdirectory deps from the same repo share one clone per install.
- **Live progress**: An ASCII progress bar shows resolution and download progress in real time.
- **Elapsed time**: Total elapsed time is shown on every exit path (including errors).
docs/src/content/docs/reference/cli-commands.md:351
- This section updates the primary skill path to
.agents/skills/, but earlier in this page the 'Dependency Types' section still states that skills are transformed to.github/agents/*.agent.mdfor VSCode. Please reconcile that earlier statement with the converged skill deployment behavior to avoid internal doc contradictions.
Skills are copied to the converged cross-client directory by default:
- **Primary (converged)**: `.agents/skills/{skill-name}/` — Shared by Copilot, Cursor, OpenCode, Codex, and Gemini
- **Claude**: `.claude/skills/{skill-name}/` — Claude retains its own per-client path
- Pass `--legacy-skill-paths` (or set `APM_LEGACY_SKILL_PATHS=1`) to restore the old per-client layout (`.github/skills/`, `.cursor/skills/`, etc.)
docs/src/content/docs/reference/cli-commands.md:407
- The uninstall 'What Gets Removed' table now calls out
.agents/skills/...as the converged location, but the table still lists per-client skill directories (e.g..cursor/skills/,.opencode/skills/,.gemini/skills/) below. If those are only present when--legacy-skill-paths/APM_LEGACY_SKILL_PATHS=1is used, consider labeling them as legacy-only (or otherwise clarifying when they apply).
| Integrated chatmodes | `.github/agents/*.agent.md` |
| Claude commands | `.claude/commands/*.md` |
| Skill folders | `.agents/skills/{folder-name}/` (converged); `.claude/skills/` for Claude |
| Integrated hooks | `.github/hooks/*.json` |
- Files reviewed: 1/1 changed files
- Comments generated: 1
| `apm install` uses parallel dependency resolution and download for fast installs: | ||
|
|
||
| - **Parallel BFS resolution**: Dependencies are resolved level-by-level in parallel batches. | ||
| - **Persistent two-tier cache**: Git repos and HTTP packages are cached on disk; cache hits are validated with `rev-parse HEAD` (git) or ETag revalidation (HTTP), so repeated installs skip redundant network fetches. |
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.
Documentation Updates - 2026-05-04
This PR updates
docs/src/content/docs/reference/cli-commands.mdbased on features merged in the last 24 hours.Features Documented
apm installperformance + UX overhaul: parallel BFS, persistent two-tier cache, live progress bar, elapsed time, per-phase--verbosetiming (from perf+ux: comprehensive overhaul of apm install (cache, parallel BFS, UX) #1116).prompt.md->.cursor/commands/*.md(from feat(cursor): add slash command support for Cursor 1.6+ #1046, shipped in v0.12.1).agents/skills/for Copilot, Cursor, OpenCode, Codex, Gemini (from [FEATURE] Support.agents/skillsas anapm installtarget for shared skill deployment #737, shipped in v0.12.0)Changes Made
docs/src/content/docs/reference/cli-commands.md:.github/skills/to.agents/skills/; added--legacy-skill-pathsescape hatch note.github/skills/to.agents/skills/.cursor/commands/*.md)Merged PRs Referenced
.agents/skillsas anapm installtarget for shared skill deployment #737 - skill routing convergence to.agents/skills/Notes
PRs #1127 and #1129 were documentation-only PRs that already updated their respective docs pages. PRs #1121, #1124, #1125 were CI/test fixes with no user-facing documentation impact. PR #1112 was the 0.12.0 release cut.
Note
🔒 Integrity filter blocked 2 items
The following items were blocked because they don't meet the GitHub integrity level.
search_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".pull_request_read: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".To allow these resources, lower
min-integrityin your GitHub frontmatter: