-
Notifications
You must be signed in to change notification settings - Fork 444
Copilot/open application #542
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
base: main
Are you sure you want to change the base?
Copilot/open application #542
Conversation
Co-authored-by: krroomm4-coder <[email protected]>
Co-authored-by: krroomm4-coder <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR enhances the GitHub workflow to automatically close issues with non-descriptive titles, expanding beyond single-word detection to include multiple languages and various forms of vague titles.
Key Changes:
- Extended detection logic to identify short titles (< 10 characters with < 3 words) and vague phrases
- Added multi-language support for detecting non-descriptive titles in English, Arabic, Chinese, and Japanese
- Improved documentation with detailed comments explaining the workflow's behavior
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // Check if title contains only very common/vague words | ||
| const vaguePhrases = [ | ||
| /^(open|close|start|stop|launch|run|help|issue|problem|bug|error|fix)\s*(the|an|a)?\s*(app|application|program|cli|tool|issue)?$/i, | ||
| /^(فتح|إغلاق|مساعدة|مشكلة|خطأ)\s*([اأإ]?[تل]?تطبيق|[اأإ]?[تل]?برنامج)?$/, // Arabic |
Copilot
AI
Nov 12, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Arabic regex pattern has an error: [اأإ]?[تل]?تطبيق won't correctly match "التطبيق" (with definite article). The pattern should be [اأإ]?[لت]?تطبيق or better yet ([اأإ]لتطبيق|تطبيق) to match both "تطبيق" and "التطبيق" forms. The same issue applies to "برنامج".
| /^(فتح|إغلاق|مساعدة|مشكلة|خطأ)\s*([اأإ]?[تل]?تطبيق|[اأإ]?[تل]?برنامج)?$/, // Arabic | |
| /^(فتح|إغلاق|مساعدة|مشكلة|خطأ)\s*(([اأإ]لتطبيق|تطبيق)|([اأإ]لبرنامج|برنامج))?$/, // Arabic |
| # 1. Single-word titles (e.g., "bug", "help") | ||
| # 2. Very short titles (< 10 characters) with fewer than 3 words | ||
| # 3. Vague phrases in multiple languages (English, Arabic, Chinese, Japanese) | ||
| # Examples: "open app", "فتح اتطبيق", "打开应用", etc. |
Copilot
AI
Nov 12, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling error in the Arabic example: "فتح اتطبيق" should be "فتح التطبيق" (the definite article "ال" is misspelled as "ات").
| # Examples: "open app", "فتح اتطبيق", "打开应用", etc. | |
| # Examples: "open app", "فتح التطبيق", "打开应用", etc. |
تطبيق بذكا الصطناعي يدلعك