Skip to content

Add step to install Genkit Agent Skills to get-started.mdx#260

Open
chrisraygill wants to merge 1 commit intogenkit-ai:mainfrom
chrisraygill:add-skills-cli-step
Open

Add step to install Genkit Agent Skills to get-started.mdx#260
chrisraygill wants to merge 1 commit intogenkit-ai:mainfrom
chrisraygill:add-skills-cli-step

Conversation

@chrisraygill
Copy link
Copy Markdown
Collaborator

This PR adds a step to install Genkit Agent Skills using the skills CLI to the Get started guide.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds a new section to the 'Get Started' documentation for installing Genkit Agent Skills using a CLI command. The review feedback suggests clarifying the Node.js requirement for users of other languages and correcting the package name to use the proper @genkit-ai scope while providing examples for multiple package managers.


### Install Genkit Agent Skills

To provide your AI assistant with structured knowledge about Genkit, you can install the Genkit Agent Skills using the skills CLI:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The npx command requires Node.js, which is not listed as a prerequisite for Go, Dart, or Python users in this guide. Since this section is visible to all users regardless of their chosen language, please add a brief note clarifying that Node.js is required for this step.

To provide your AI assistant with structured knowledge about Genkit, you can install the Genkit Agent Skills using the skills CLI (requires Node.js):

To provide your AI assistant with structured knowledge about Genkit, you can install the Genkit Agent Skills using the skills CLI:

```bash
npx skills add genkit-ai/skills
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Per the general rules, documentation should specify supported package managers like npm or pnpm. Additionally, please verify the package name; Genkit packages are typically scoped under @genkit-ai. Using the scoped name @genkit-ai/skills ensures the correct CLI is installed and avoids potential conflicts with unrelated packages named skills.

npx @genkit-ai/skills add genkit-ai/skills
# or
pnpm dlx @genkit-ai/skills add genkit-ai/skills
References
  1. In documentation, specify supported package managers like npm or pnpm for installation instructions, rather than using generic phrases like 'your favorite package manager'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant