Create electron-angular-native.chatmode.md #36
+284
−0
Conversation
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
## Summary This adds a new chatmode for reviewing Electron desktop apps with: - Node.js (main) - Angular (renderer) - Native integration (e.g., AppleScript, shell, exiftool) The mode enforces best practices in IPC security, async/await consistency, error handling, performance, and Angular subscription management. ## File - `chatmodes/electron-angular-native.chatmode.md` ## Motivation To help teams building hybrid Electron apps (Node + Angular + native automation) perform rigorous code audits using GitHub Copilot Chat.
There was a problem hiding this comment.
Pull Request Overview
Adds a new chat mode for reviewing Electron desktop apps that use Node.js in the main process, Angular in the renderer, and native integrations.
- Introduces
electron-angular-native.chatmode.mdwith frontmatter and tool definitions - Defines coding conventions, security/performance guidelines, and a review checklist
- Provides templates for feature examples and output formatting
Comments suppressed due to low confidence (3)
chatmodes/electron-angular-native.chatmode.md:1
- The frontmatter is missing a
namefield, which may be required to register this chat mode. Please addname: electron-angular-nativeunder thedescriptionkey.
---
chatmodes/electron-angular-native.chatmode.md:2
- [nitpick] The term
(render)is inconsistent with the later use ofRenderer Process. Consider renaming to(renderer)for consistency.
description: "Code Review Mode tailored for Electron app with Node.js backend (main), Angular frontend (render), and native integration layer (e.g., AppleScript, shell, or native tooling). Services in other repos are not reviewed here."
chatmodes/electron-angular-native.chatmode.md:203
- The Feature sections B–E are placeholders without content. Either populate these with examples or remove them to prevent confusion.
### Feature B
| - Node.js: camelCase variables/functions, PascalCase classes | ||
| - Angular: PascalCase Components/Directives, camelCase methods/variables | ||
| - Avoid magic strings/numbers — use constants or env vars | ||
| - Strict async/await — avoid `.then()`, `.Result`, `.Wait()`, or callback mixing |
There was a problem hiding this comment.
References to .Result and .Wait() are specific to .NET and not relevant for JavaScript/TypeScript environments. Consider removing these to avoid confusion.
Suggested change
| - Strict async/await — avoid `.then()`, `.Result`, `.Wait()`, or callback mixing | |
| - Strict async/await — avoid `.then()` or callback mixing |
Copilot uses AI. Check for mistakes.
|
Hi team 👋 Thanks for building this amazing repo. This PR contributes a code review chatmode for hybrid Electron desktop apps (Node.js + Angular + native integrations). Would appreciate a quick review when time permits 🙌 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
This adds a new chatmode for reviewing Electron desktop apps with:
The mode enforces best practices in IPC security, async/await consistency, error handling, performance, and Angular subscription management.
File
chatmodes/electron-angular-native.chatmode.mdMotivation
To help teams building hybrid Electron apps (Node + Angular + native automation) perform rigorous code audits using GitHub Copilot Chat.
name: Pull Request
about: Submit a contribution to the awesome-copilot repository
title: ''
labels: needs-review
assignees: ''
Pull Request Checklist
node update-readme.jsand verified thatREADME.mdis up to date.Description
Type of Contribution
Additional Notes
By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.