-
Notifications
You must be signed in to change notification settings - Fork 2
Fix all findings from post-integration codebase review #26
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,6 +1,6 @@ | ||||||
| --- | ||||||
| title: Skills | ||||||
| description: Complete reference for all 22 CodeForge skills — domain knowledge packs for frameworks, patterns, and workflows. | ||||||
| description: Complete reference for all 38 CodeForge skills — domain knowledge packs for frameworks, patterns, and workflows. | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. “Complete reference for all 38 skills” overstates current page content. Line 3 claims full coverage of all 38 skills, but this page currently documents a smaller subset. Please either add the missing skill entries or soften the description to avoid a false completeness claim. Suggested wording fallback-description: Complete reference for all 38 CodeForge skills — domain knowledge packs for frameworks, patterns, and workflows.
+description: Reference for CodeForge skills — domain knowledge packs for frameworks, patterns, and workflows.As per coding guidelines, user-facing documentation changes must be reflected accurately in relevant pages under 📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||
| sidebar: | ||||||
| order: 3 | ||||||
| --- | ||||||
|
|
||||||
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.
Agent count claim and on-page reference content are out of sync.
Lines 3/8 now state 21 agents, but the detailed reference matrix/sections in this page still cover fewer agents. Please reconcile by adding the missing agent docs or adjusting the claim until the full set is documented.
As per coding guidelines, user-facing documentation changes must be reflected accurately in relevant pages under
docs/.Also applies to: 8-8
🤖 Prompt for AI Agents