Skip to content

feat(@schematics/angular): add schematics to generate ai context files #30763

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 13, 2025

Conversation

JeanMeche
Copy link
Member

@JeanMeche JeanMeche commented Jul 22, 2025

  • ng generate ai-config to prompt support tools.
  • ng generate ai-config --tool=gemini to specify the tool.
  • ng new will prompt to config AI tools.
  • ng new --ai-config=gemini to create a new project with AI configuration.

Supported AI tools: gemini, claude, copilot, windsurf, jetbrains, cursor.

@JeanMeche JeanMeche force-pushed the generate-ai-file branch 2 times, most recently from 70ae7ec to 288dcfd Compare July 22, 2025 16:53
@JeanMeche JeanMeche marked this pull request as ready for review July 29, 2025 13:36
@dgp1130 dgp1130 added action: review The PR is still awaiting reviews from at least one requested reviewer target: minor This PR is targeted for the next minor release labels Jul 31, 2025
@dgp1130 dgp1130 requested a review from clydin July 31, 2025 04:16
@JeanMeche JeanMeche force-pushed the generate-ai-file branch 2 times, most recently from 3dc7023 to 5ae2b77 Compare August 1, 2025 12:11
@JeanMeche JeanMeche removed the request for review from AndrewKushnir August 1, 2025 12:12
clydin
clydin previously requested changes Aug 1, 2025
@JeanMeche JeanMeche force-pushed the generate-ai-file branch 2 times, most recently from 048a5d7 to 19858e9 Compare August 4, 2025 14:16
Copy link
Collaborator

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

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

Some minor things and the commit message is still outdated.

@alan-agius4 alan-agius4 removed the request for review from clydin August 13, 2025 08:30
Copy link
Collaborator

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

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

Mostly NITs, but I think there is a bug with the "none" value.

"tool": {
"type": "array",
"uniqueItems": true,
"x-prompt": "Which AI tools do you want to configure with Angular best practices? https://angular.dev/ai/develop-with-ai",
Copy link
Collaborator

Choose a reason for hiding this comment

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

NIT: As a follow up can we improve the prompt to include some more details

  "x-prompt": {
        "message": "Which AI tools do you want to configure with Angular best practices? https://angular.dev/ai/develop-with-ai",
        "type": "list",
        "items": [
          {
            "value": "none",
            "label": "None"
          },
          {
            "value": "copilot",
            "label": "Copilot powered IDEs     [ https://https://code.visualstudio.com/docs/copilot/copilot-customization#_custom-instructions ]"
          },
          {
            "value": "cursor",
            "label": "Cursor                   [ https://docs.cursor.com/en/context/rules                                                      ]"
          }
        ]
       ...
      },

Copy link
Member Author

Choose a reason for hiding this comment

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

Will take care of it in a follow up.

Copy link
Collaborator

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

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

There are still a couple of mistakes in the commit message (fixed below)

* `ng generate ai-config` to prompt support tools.
* `ng generate ai-config --tool=gemini` to specify the tool.
* `ng new` will prompt to config AI tools.
* `ng new --ai-config=gemini` to create a new project with AI configuration.

Supported AI tools: gemini, claude, copilot, windsurf, jetbrains, cursor.

@alan-agius4 alan-agius4 changed the title feat(@angular-devkit/schematics): add schematics to generate ai context files. feat(@schematics/angular): add schematics to generate ai context files Aug 13, 2025
@JeanMeche JeanMeche force-pushed the generate-ai-file branch 2 times, most recently from 95ce010 to f0f59fc Compare August 13, 2025 10:49
@alan-agius4 alan-agius4 removed the action: review The PR is still awaiting reviews from at least one requested reviewer label Aug 13, 2025
Copy link
Collaborator

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

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

2 comments to fix the prompt, as I highly doubt it works right now.

@JeanMeche JeanMeche force-pushed the generate-ai-file branch 2 times, most recently from 151c778 to 37bd025 Compare August 13, 2025 13:21
@alan-agius4 alan-agius4 removed the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label Aug 13, 2025
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Aug 13, 2025
@alan-agius4 alan-agius4 force-pushed the generate-ai-file branch 3 times, most recently from 0db379b to f9270a7 Compare August 13, 2025 13:37
* `ng generate ai-config` to prompt support tools.
* `ng generate ai-config --tool=gemini` to specify the tool.
* `ng new` will prompt to config AI tools.
* `ng new --ai-config=gemini` to create a new project with AI configuration.

Supported AI tools: gemini, claude, copilot, windsurf, jetbrains, cursor.
@alan-agius4 alan-agius4 merged commit d80dae2 into angular:main Aug 13, 2025
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: @angular-devkit/schematics area: @schematics/angular detected: feature PR contains a feature commit target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants