Skip to content

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