-
Notifications
You must be signed in to change notification settings - Fork 8
feat(ai-config): add agent choices for skills/instruction locations #1664
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
Changes from all commits
Commits
Show all changes
55 commits
Select commit
Hold shift + click to select a range
52cd22f
Initial plan
Copilot 2c18475
feat: make ai-config agent-agnostic with --agent and --skills-dir opt…
Copilot c67709e
refactor: improve analytics event and clean up skillsDir resolution
Copilot d4478a9
feat: update angular schematics ai-config to accept --agent and --ski…
Copilot 5784c4a
feat: enhance ai-config to prompt for agent selection and improve ski…
Marina-L-Stoyanova 39eff3d
feat: enhance ai-config command to handle agent selection and improve…
Marina-L-Stoyanova 6c5d717
feat: make ai-config agent-agnostic by supporting multiple agents and…
Marina-L-Stoyanova 14e062d
feat: refactor ai-config to support agent-based skills configuration …
Marina-L-Stoyanova 026fb9e
Merge branch 'master' of https://github.com/IgniteUI/igniteui-cli int…
Marina-L-Stoyanova 3669211
feat: add AI agent configuration options to project creation
Marina-L-Stoyanova f2f39a9
change():Moving skills to ai-config skills directory and updating the…
Marina-L-Stoyanova 1379243
chore: enhance AI configuration options with labels and improved sele…
Marina-L-Stoyanova 4fb65da
feat: update AI configuration options to include 'none' and improve p…
Marina-L-Stoyanova baee30a
feat: enhance AI configuration by adding instruction file handling an…
Marina-L-Stoyanova 56ccdf8
Removing skills and instructions from _base templates
Marina-L-Stoyanova 96f010b
Optimizing react ai-config template
Marina-L-Stoyanova 9ca5b16
fix: remove unnecessary whitespace in PromptSession unit tests
Marina-L-Stoyanova 100f1e4
chore: remove outdated AGENTS.md file from React template
Marina-L-Stoyanova fd28dde
feat: refactor AI agent selection to use promptForAgents function
Marina-L-Stoyanova f2656af
feat: add InquirerWrapper checkbox spy to new command tests
Marina-L-Stoyanova 62a33a4
feat: update AI agent configuration and help command documentation
Marina-L-Stoyanova 9d5d801
feat: enhance AI tooling configuration with agent selection and instr…
Marina-L-Stoyanova d30a1cd
feat: refactor AI configuration to use promptForAgents and update ski…
Marina-L-Stoyanova a892c47
feat: add required option to promptForAgents and update tests for ana…
Marina-L-Stoyanova ae4b10b
feat: apply review comments
Marina-L-Stoyanova ee143b8
Apply suggestion from @damyanpetev
Marina-L-Stoyanova dd7dff4
Apply suggestion from @damyanpetev
Marina-L-Stoyanova e9f9424
refactor: update skills directory resolution and clean up tests
Marina-L-Stoyanova 2b5318e
chore():updating ai-config templates
Marina-L-Stoyanova bfc819f
feat: add template file resolution for AGENTS.md and update related t…
Marina-L-Stoyanova 713f181
feat: enhance ai-config template validation and add file existence ch…
Marina-L-Stoyanova af0430a
chore: cleanup ai-config partials
damyanpetev d9ec603
refactor: call ai-config schematic w/ task for prompt reuse
damyanpetev 829fde0
refactor: move dirs inside copy logic
damyanpetev 1f8670b
refactor: restore `configureAI()` to call `configure()` entry and mov…
Copilot 215bf05
refactor: cleanup agent options and dir exports
damyanpetev 6e548d1
fix: new command argument name and adjust descriptions
damyanpetev f6916ea
refactor: constants for AI config project ID and skills directory
damyanpetev 61b414b
Merge branch 'master' into copilot/update-ai-config-agent-agnostic
damyanpetev e72529a
fix: agents schematics param
damyanpetev 22ad345
chore: schematics descr and redundant default
damyanpetev d13a60c
feat: add AI agents configuration tests to new command
Marina-L-Stoyanova c4cd106
test(schematics): fix async tests
damyanpetev 53ef9c6
test(schematics): fix broken text expects/setup
damyanpetev 750d75b
fix(schematics,new): filter out hidden projects
damyanpetev f8d1271
refactor(schematics,new): run full ai-config schematic for args/prompts
damyanpetev fa0fb97
doc(readme): missed arg rename
damyanpetev a271689
revert(schematics,new): early name use for move
damyanpetev 11f6d71
fix: update Google Analytics tracking for AI configuration and adjust…
Marina-L-Stoyanova e22cb40
Merge branch 'copilot/update-ai-config-agent-agnostic' of https://git…
Marina-L-Stoyanova 2824045
fix: update Google Analytics tracking for AI configuration to use cor…
Marina-L-Stoyanova e2da748
refactor(ai-config): TTY check for prompt and defaults handling
damyanpetev faffbaa
test(new): fix ai-config spies
damyanpetev 46dff06
chore: unused import
damyanpetev 3231e02
Merge branch 'master' into copilot/update-ai-config-agent-agnostic
damyanpetev File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
3 changes: 0 additions & 3 deletions
3
packages/cli/templates/react/igr-ts/projects/_base/files/__dot__claude/CLAUDE.md
This file was deleted.
Oops, something went wrong.
3 changes: 0 additions & 3 deletions
3
...mplates/react/igr-ts/projects/_base/files/__dot__github/copilot-instructions.md
This file was deleted.
Oops, something went wrong.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
38 changes: 38 additions & 0 deletions
38
packages/cli/templates/react/igr-ts/projects/ai-config/index.ts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,38 @@ | ||
| import { ControlExtraConfiguration, defaultDelimiters, ProjectTemplate } from "@igniteui/cli-core"; | ||
| import * as path from "path"; | ||
|
|
||
| // currently reusing hidden project impl as components/views pipeline go through registerInProject | ||
| // ideally would define a separate type/category for those partial files | ||
| export class IgrTsAiConfigPartial implements ProjectTemplate { | ||
| public id: string = "ai-config"; | ||
| public name = "ai-config"; | ||
| public description = "Ignite UI CLI AI config for React partial project files"; | ||
| public framework: string = "react"; | ||
| public projectType: string = "tsx"; | ||
| public dependencies: string[] = []; | ||
| public hasExtraConfiguration: boolean = false; | ||
| public isHidden: boolean = true; | ||
| public delimiters = defaultDelimiters; | ||
|
|
||
| public get templatePaths(): string[] { | ||
| return [path.join(__dirname, "files")]; | ||
| } | ||
|
|
||
| public generateConfig(_name: string, _theme: string, ..._options: any[]): {[key: string]: any} { | ||
| return { /* partials not using Util.processTemplates atm */ }; | ||
| } | ||
|
|
||
| public installModules(): void { | ||
| throw new Error("Method not implemented."); | ||
| } | ||
| public async upgradeIgniteUIPackages(_projectPath: string, _packagePath: string): Promise<boolean> { | ||
| throw new Error("Method not implemented."); | ||
| } | ||
| public getExtraConfiguration(): ControlExtraConfiguration[] { | ||
| throw new Error("Method not implemented."); | ||
| } | ||
| public setExtraConfiguration(_extraConfigKeys: {}) { | ||
| throw new Error("Method not implemented."); | ||
| } | ||
| } | ||
| export default new IgrTsAiConfigPartial(); |
3 changes: 0 additions & 3 deletions
3
...cli/templates/webcomponents/igc-ts/projects/_base/files/__dot__claude/CLAUDE.md
This file was deleted.
Oops, something went wrong.
3 changes: 0 additions & 3 deletions
3
...webcomponents/igc-ts/projects/_base/files/__dot__github/copilot-instructions.md
This file was deleted.
Oops, something went wrong.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
40 changes: 40 additions & 0 deletions
40
packages/cli/templates/webcomponents/igc-ts/projects/ai-config/index.ts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,40 @@ | ||
| import { ControlExtraConfiguration, defaultDelimiters, ProjectTemplate } from "@igniteui/cli-core"; | ||
| import * as path from "path"; | ||
|
|
||
| // currently reusing hidden project impl as components/views pipeline go through registerInProject | ||
| // ideally would define a separate type/category for those partial files | ||
| export class IgcTsAiConfigPartial implements ProjectTemplate { | ||
|
|
||
| public id: string = "ai-config"; | ||
| public name = "ai-config"; | ||
| public description = "Ignite UI CLI AI config for Web Components partial project files"; | ||
| public framework: string = "webcomponents"; | ||
| public projectType: string = "igc-ts"; | ||
| public dependencies: string[]; | ||
| public hasExtraConfiguration: boolean = false; | ||
| public isHidden: boolean = true; | ||
| public delimiters = defaultDelimiters; | ||
|
|
||
| public get templatePaths(): string[] { | ||
| return [path.join(__dirname, "files")]; | ||
| } | ||
|
|
||
| installModules(): void { | ||
| throw new Error("Method not implemented."); | ||
| } | ||
| upgradeIgniteUIPackages(_projectPath: string, _packagePath: string): Promise<boolean> { | ||
| throw new Error("Method not implemented."); | ||
| } | ||
| generateConfig(_name: string, _theme: string, ..._options: any[]): { [key: string]: any; } { | ||
| throw new Error("Method not implemented."); | ||
| } | ||
| getExtraConfiguration(): ControlExtraConfiguration[] { | ||
| throw new Error("Method not implemented."); | ||
| } | ||
| setExtraConfiguration(_extraConfigKeys: {}) { | ||
| throw new Error("Method not implemented."); | ||
|
damyanpetev marked this conversation as resolved.
|
||
| } | ||
| } | ||
| export default new IgcTsAiConfigPartial(); | ||
|
|
||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.