Repository navigation
Conversation
Co-authored-by: palewire <9993+palewire@users.noreply.github.com>
Co-authored-by: palewire <9993+palewire@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
Three moderate review findings remain unresolved.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Adds project-neutral Context7 indexing defaults and optional refresh automation to generated repositories.
Changes:
- Adds schema-backed documentation indexing configuration.
- Adds opt-in Context7 refresh workflow.
- Documents setup and packages
context7.jsonin source distributions. - Records the feature in the changelog.
File summaries
| File | Summary | Final review comments |
|---|---|---|
TEMPLATE_SETUP.md |
Documents Context7 setup and maintenance. | None |
MANIFEST.in |
Includes context7.json in source distributions. |
None |
context7.json |
Defines documentation indexing and exclusions. | Moderate: add remaining default operational filename variants. 1 vote |
CHANGELOG.md |
Records the Context7 feature. | None |
.github/workflows/context7-refresh.yml |
Provides opt-in refresh automation. | Moderate: isolate concurrency by ref. 3 votes; Moderate: keep workflow paths aligned with configurable documentation paths. 1 vote |
Review details
Suppressed comments (2)
.github/workflows/context7-refresh.yml:8
- The setup instructions invite maintainers to move the documentation paths in
context7.json, but this workflow still watches onlydocs/**andREADME.md. After moving the docs, subsequent documentation changes will not trigger the advertised automatic refresh. Please make the setup checklist require updating this workflow'spathsas well, or choose a trigger strategy that follows the configured documentation locations.
- "docs/**"
context7.json:4
- Because
excludeFilesis supplied, Context7 no longer relies on its default filename exclusions. A generated repository that usesLICENSE.md,license.md,CHANGELOG.mdx, or the lowercase variants will therefore index those operational files, contrary to the setup text's claim that operational files are excluded. Add the remaining default filename variants (while retaining the template's extensionlessLICENSE) so this default remains project-neutral.
"excludeFiles": [
- Files reviewed: 5/5 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
@copilot Please fix the three outstanding review findings on this pull request in a cloud coding session:
Please make the changes on this PR branch, run the relevant validation, and report the commit and results here. Preserve the opt-in behavior and project-neutral defaults.**** |
Co-authored-by: palewire <9993+palewire@users.noreply.github.com>
Fixed in Validation:
|
Adds project-neutral Context7 indexing defaults and optional refresh automation to generated repositories.
Changes
context7.json.docs/and README content while excluding operational files.{ "$schema": "https://context7.com/schema/context7.json", "folders": ["docs"] }Automation
CONTEXT7_API_KEYsecret.Documentation
context7.jsonin source distributions and records the feature in the changelog.Verification