Refactor config.json to restructure default settings and included patterns#110
Refactor config.json to restructure default settings and included patterns#110Tendy24 wants to merge 4 commits intodocker:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces a new .snapshots directory with configuration and documentation for the Snapshots for AI tool. The changes establish default settings for snapshot generation and define patterns for files to be handled by the tool.
Key changes:
- Added comprehensive configuration structure with default prompt and file inclusion settings
- Created extensive pattern list covering build artifacts, media files, databases, and development tools
- Introduced sponsor information and project documentation
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
.snapshots/sponsors.md |
Adds sponsorship information, support options, and community links for the project |
.snapshots/readme.md |
Provides basic documentation explaining the purpose and contents of snapshot files |
.snapshots/config.json |
Defines default settings and a comprehensive list of file patterns (149 lines) for the snapshot tool |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| ### Join the GBTI Network!!! 🙏🙏🙏 | ||
|
|
||
| The GBTI Network is a community of developers who are passionate about open source and community-driven development. Members enjoy access to exclussive tools, resources, a private MineCraft server, a listing in our members directory, co-op opportunities and more. |
There was a problem hiding this comment.
The word "exclussive" is misspelled. It should be "exclusive" with one 's'.
| The GBTI Network is a community of developers who are passionate about open source and community-driven development. Members enjoy access to exclussive tools, resources, a private MineCraft server, a listing in our members directory, co-op opportunities and more. | |
| The GBTI Network is a community of developers who are passionate about open source and community-driven development. Members enjoy access to exclusive tools, resources, a private MineCraft server, a listing in our members directory, co-op opportunities and more. |
|
|
||
| - Visit [Codeable](https://www.codeable.io/developers/?ref=z8h3e) to hire your next team member. | ||
|
|
||
| ### Lead positive reviews on our marketplace listing ⭐⭐⭐⭐⭐ |
There was a problem hiding this comment.
The word "Lead" should be "Leave" in the context of leaving positive reviews.
| ### Lead positive reviews on our marketplace listing ⭐⭐⭐⭐⭐ | |
| ### Leave positive reviews on our marketplace listing ⭐⭐⭐⭐⭐ |
| ".npmrc", | ||
| ".git", | ||
| ".gitignore", | ||
| "gradle", | ||
| "gradlew", | ||
| "gradlew.*", | ||
| "node_modules", | ||
| ".snapshots", | ||
| ".idea", | ||
| ".vscode", | ||
| "*.log", | ||
| "*.tmp", | ||
| "target", | ||
| "dist", | ||
| "build", | ||
| ".DS_Store", | ||
| "*.bak", | ||
| "*.swp", | ||
| "*.swo", | ||
| "*.lock", | ||
| "*.iml", | ||
| "coverage", | ||
| "*.min.js", | ||
| "*.min.css", | ||
| "__pycache__", | ||
| ".marketing", | ||
| ".env", | ||
| ".env.*", | ||
| "*.jpg", | ||
| "*.jpeg", | ||
| "*.png", | ||
| "*.gif", | ||
| "*.bmp", | ||
| "*.tiff", | ||
| "*.ico", | ||
| "*.svg", | ||
| "*.webp", | ||
| "*.psd", | ||
| "*.ai", | ||
| "*.eps", | ||
| "*.indd", | ||
| "*.raw", | ||
| "*.cr2", | ||
| "*.nef", | ||
| "*.mp4", | ||
| "*.mov", | ||
| "*.avi", | ||
| "*.wmv", | ||
| "*.flv", | ||
| "*.mkv", | ||
| "*.webm", | ||
| "*.m4v", | ||
| "*.wfp", | ||
| "*.prproj", | ||
| "*.aep", | ||
| "*.psb", | ||
| "*.xcf", | ||
| "*.sketch", | ||
| "*.fig", | ||
| "*.xd", | ||
| "*.db", | ||
| "*.sqlite", | ||
| "*.sqlite3", | ||
| "*.mdb", | ||
| "*.accdb", | ||
| "*.frm", | ||
| "*.myd", | ||
| "*.myi", | ||
| "*.ibd", | ||
| "*.dbf", | ||
| "*.rdb", | ||
| "*.aof", | ||
| "*.pdb", | ||
| "*.sdb", | ||
| "*.s3db", | ||
| "*.ddb", | ||
| "*.db-shm", | ||
| "*.db-wal", | ||
| "*.sqlitedb", | ||
| "*.sql.gz", | ||
| "*.bak.sql", | ||
| "dump.sql", | ||
| "dump.rdb", | ||
| "*.vsix", | ||
| "*.jar", | ||
| "*.war", | ||
| "*.ear", | ||
| "*.zip", | ||
| "*.tar", | ||
| "*.tar.gz", | ||
| "*.tgz", | ||
| "*.rar", | ||
| "*.7z", | ||
| "*.exe", | ||
| "*.dll", | ||
| "*.so", | ||
| "*.dylib", | ||
| "*.app", | ||
| "*.dmg", | ||
| "*.iso", | ||
| "*.msi", | ||
| "*.deb", | ||
| "*.rpm", | ||
| "*.apk", | ||
| "*.aab", | ||
| "*.ipa", | ||
| "*.pkg", | ||
| "*.nupkg", | ||
| "*.snap", | ||
| "*.whl", | ||
| "*.gem", | ||
| "*.pyc", | ||
| "*.pyo", | ||
| "*.pyd", | ||
| "*.class", | ||
| "*.o", | ||
| "*.obj", | ||
| "*.lib", | ||
| "*.a", | ||
| "*.map", | ||
| ".npmrc" |
There was a problem hiding this comment.
The field name is "included_patterns" but the patterns listed appear to be primarily exclusion patterns (items to ignore like .git, node_modules, *.log, build artifacts, etc.). This creates confusion about the intent - should these patterns be included or excluded from snapshots? Consider renaming to "excluded_patterns" or clarifying the intended behavior in the documentation.
| "*.map", | ||
| ".npmrc" |
There was a problem hiding this comment.
The pattern ".npmrc" is duplicated - it appears both on line 27 and line 147. Remove one of the duplicate entries.
| "*.map", | |
| ".npmrc" | |
| "*.map" |
1 participant
No description provided.