Skip to content

feat(cli): recommend adding arkenv agent skill as a next step#944

Merged
yamcodes merged 3 commits into
mainfrom
942-skills-should-be-integrated-into-the-cli-1
May 12, 2026
Merged

feat(cli): recommend adding arkenv agent skill as a next step#944
yamcodes merged 3 commits into
mainfrom
942-skills-should-be-integrated-into-the-cli-1

Conversation

@yamcodes
Copy link
Copy Markdown
Owner

@yamcodes yamcodes commented May 12, 2026

Fixes #942

Adds a new recommended next step to the CLI output that suggests installing the ArkEnv agent skill. The command is now dynamically generated based on the detected package manager (e.g., pnpm dlx, bunx, yarn dlx, or npx).

Summary by CodeRabbit

Release Notes

  • New Features
    • CLI scaffolding now displays an additional recommended step guiding users to install the ArkEnv Agent Skill using their package manager after project setup completes.

Review Change Stack

@yamcodes yamcodes added enhancement New feature or request @arkenv/cli Issues or Pull Requests involving the ArkEnv CLI skills Issues or Pull Requests related to ArkEnv Agent Skills labels May 12, 2026
@yamcodes yamcodes linked an issue May 12, 2026 that may be closed by this pull request
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 12, 2026

🦋 Changeset detected

Latest commit: 818846c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@arkenv/cli Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@arkenv-bot
Copy link
Copy Markdown
Contributor

arkenv-bot Bot commented May 12, 2026

🤖 Hi @yamcodes, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 12, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 6bbe19be-4862-4896-aa65-851af23b5207

📥 Commits

Reviewing files that changed from the base of the PR and between 817b78e and 818846c.

📒 Files selected for processing (3)
  • .changeset/solid-bananas-smile.md
  • packages/cli/src/index.ts
  • packages/cli/src/scaffold.ts

Walkthrough

The CLI adds a package-manager-aware getDlxCommand(pm) helper and logs a third post-scaffold step instructing users to install the ArkEnv Agent Skill using the derived dlx command; the changeset documents this recommended next step.

Changes

CLI Post-Scaffolding Guidance

Layer / File(s) Summary
Package-manager dlx command helper
packages/cli/src/scaffold.ts
Adds exported getDlxCommand(pm: string) returning pnpm dlx, yarn dlx, bunx, or npx fallback.
Post-scaffolding instruction uses dlx command
packages/cli/src/index.ts
Reformats ./scaffold import to named imports and adds a third log.step that computes dlx from packageManager and logs ${dlx} skills add yamcodes/arkenv as a post-scaffold instruction.
Changeset documents agent-skill recommendation
.changeset/solid-bananas-smile.md
Bumps @arkenv/cli patch and adds the human-readable note: “Add ArkEnv Agent Skill as recommended next step”.

Sequence Diagram(s)

sequenceDiagram
  participant CLI
  participant getDlxCommand
  participant UserShell
  CLI->>getDlxCommand: getDlxCommand(packageManager)
  getDlxCommand-->>CLI: dlx command string
  CLI->>UserShell: log step `${dlx} skills add yamcodes/arkenv`
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • yamcodes/arkenv#937: Adds Agent Skills documentation and standardizes the installation commands referenced by the CLI guidance.
  • yamcodes/arkenv#944: Also modifies CLI post-scaffolding guidance to recommend installing the ArkEnv Agent Skill.

Poem

🐰 I hopped through code with a cheery trill,
A helper to run dlx, quiet and still.
Now scaffolds end with a helpful call,
"Run skills add" — one step for all.
Hop on, install, let's answer the thrill!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'feat(cli): recommend adding arkenv agent skill as a next step' directly and clearly summarizes the main change: adding a recommendation for installing the ArkEnv agent skill as a post-scaffolding next step.
Linked Issues check ✅ Passed The PR directly addresses issue #942 by implementing the requested feature to integrate skills into the CLI as a recommended next step, which is the primary objective.
Out of Scope Changes check ✅ Passed The changes are narrowly scoped to adding CLI guidance for installing the ArkEnv agent skill, which is directly within the scope of issue #942 requirements.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 942-skills-should-be-integrated-into-the-cli-1

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 12, 2026

Open in StackBlitz

arkenv

npm i https://pkg.pr.new/arkenv@944

@arkenv/bun-plugin

npm i https://pkg.pr.new/@arkenv/bun-plugin@944

@arkenv/cli

npm i https://pkg.pr.new/@arkenv/cli@944

@arkenv/fumadocs-ui

npm i https://pkg.pr.new/@arkenv/fumadocs-ui@944

@arkenv/vite-plugin

npm i https://pkg.pr.new/@arkenv/vite-plugin@944

commit: e117780

@arkenv-bot
Copy link
Copy Markdown
Contributor

arkenv-bot Bot commented May 12, 2026

🤖 I'm sorry @yamcodes, but I was unable to process your request. Please see the logs for more details.

@arkenv-bot
Copy link
Copy Markdown
Contributor

arkenv-bot Bot commented May 12, 2026

📦 Bundle Size Report

No results found

All size limits passed!

@github-actions github-actions Bot added the docs Improvements or additions to documentation label May 12, 2026
@yamcodes
Copy link
Copy Markdown
Owner Author

Good enough for now but keep an eye on #924

@yamcodes yamcodes closed this May 12, 2026
@yamcodes yamcodes reopened this May 12, 2026
@yamcodes yamcodes merged commit 718fb4b into main May 12, 2026
12 checks passed
@yamcodes yamcodes deleted the 942-skills-should-be-integrated-into-the-cli-1 branch May 12, 2026 08:35
@arkenv-bot
Copy link
Copy Markdown
Contributor

arkenv-bot Bot commented May 12, 2026

🤖 Hi @yamcodes, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

@arkenv-bot arkenv-bot Bot mentioned this pull request May 12, 2026
@arkenv-bot
Copy link
Copy Markdown
Contributor

arkenv-bot Bot commented May 12, 2026

🤖 I'm sorry @yamcodes, but I was unable to process your request. Please see the logs for more details.

@arkenv-bot
Copy link
Copy Markdown
Contributor

arkenv-bot Bot commented May 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Warning

Rate limit exceeded

@yamcodes has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 18 minutes and 18 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered by pushing new commits to this PR or by re-running the workflow.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

We enforce hourly and daily rate limits to ensure stability and fair usage of resources.

yamcodes pushed a commit that referenced this pull request May 12, 2026
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @arkenv/cli@0.0.4

### Patch Changes

- #### Add ArkEnv Agent Skill as recommended next step
_[`#944`](#944)
[`718fb4b`](718fb4b)
[@yamcodes](https://github.com/yamcodes)_
- #### Adapt CLI templates for framework plugins (Vite/Bun)
_[`#943`](#943)
[`eba2f83`](eba2f83)
[@yamcodes](https://github.com/yamcodes)_

## @arkenv/fumadocs-ui@0.0.8

### Patch Changes

- #### Fix hamburger menu fade-in behavior on mobile
_[`#935`](#935)
[`f1ee3de`](f1ee3de)
[@yamcodes](https://github.com/yamcodes)_

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

@arkenv/cli Issues or Pull Requests involving the ArkEnv CLI docs Improvements or additions to documentation enhancement New feature or request skills Issues or Pull Requests related to ArkEnv Agent Skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Skills should be integrated into the CLI

1 participant