Skip to content

feat: [#859] Generate Agent Skills#181

Open
krishankumar01 wants to merge 1 commit intomasterfrom
kkumar-gcc/#859
Open

feat: [#859] Generate Agent Skills#181
krishankumar01 wants to merge 1 commit intomasterfrom
kkumar-gcc/#859

Conversation

@krishankumar01
Copy link
Member

📑 Description

RelatedTo goravel/goravel#859

@coderabbitai summary

✅ Checks

  • Added test cases for my code

@coderabbitai
Copy link

coderabbitai bot commented Mar 15, 2026

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • 🚀 Review Ready

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2c39d6b3-9b11-43d6-995a-b27e18129808

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch kkumar-gcc/#859
📝 Coding Plan
  • Generate coding plan for human review comments

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.

Tip

CodeRabbit can suggest fixes for GitHub Check annotations.

Configure the reviews.tools.github-checks setting to adjust the time to wait for GitHub Checks to complete.

Copy link
Contributor

@hwbrzzl hwbrzzl left a comment

Choose a reason for hiding this comment

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

Awesome

on:
push:
branches:
- "**"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- "**"
- master
- "v1.*"

{
"output": ".ai/AGENTS.md",
"sources": [
"en/upgrade/v1.17.md",
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it necessary to contain this file in AGENTS.md? Or a single file can be generated for the upgrading doc.

- name: Determine affected prompt files
id: affected
run: |
python3 - <<'EOF'
Copy link
Contributor

Choose a reason for hiding this comment

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

Can it be moved to a single python file? And could you add a screenshot for the script?

Copy link
Contributor

Choose a reason for hiding this comment

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

How about renaming this folder to skills? Then it can be used by an AI agent locally?

Comment on lines +111 to +115
**To trigger regeneration**, go to:
[Actions → Generate Agent Skill Files → Run workflow](../../actions/workflows/generate-agents.yml)
and click **Run workflow** on this branch.

> No API tokens are spent until you manually approve the regeneration run.
Copy link
Contributor

Choose a reason for hiding this comment

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

How about creating a PR directly for the changes without approval?

"output": ".ai/prompt/route.md",
"sources": [
"en/the-basics/routing.md",
"en/the-basics/middleware.md"
Copy link
Contributor

Choose a reason for hiding this comment

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

The middleware is duplicated in the root and middleware files. Is it necessary?


---

## Arguments (v1.17)
Copy link
Contributor

Choose a reason for hiding this comment

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

There is no need to specify a version.

]
},
{
"output": ".ai/prompt/best-practices.md",
Copy link
Contributor

Choose a reason for hiding this comment

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

This file is very large. Users will spend a lot of tokens. We may generate mini prompts at first, then optimize them step by step.

Copy link
Contributor

Choose a reason for hiding this comment

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

This file is very large. How about splitting this file into different modules?


---

## Runners Interface (v1.17)
Copy link
Contributor

Choose a reason for hiding this comment

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

Ditto

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.

2 participants