Phase 1: progressive-disclosure refactor — graphql-developer (591 → 100 lines) - #23
Closed
olehsvyrydov wants to merge 1 commit into
Closed
Phase 1: progressive-disclosure refactor — graphql-developer (591 → 100 lines)#23olehsvyrydov wants to merge 1 commit into
olehsvyrydov wants to merge 1 commit into
Conversation
… 100 lines) Move the ~497-line Expertise block into references/graphql-expertise.md (versions, core concepts: schema design/resolvers/Apollo Server/Federation/DataLoader/ subscriptions/security/performance, project structure). Lean SKILL.md keeps trigger, context, doc lookup, standards, checklist, anti-patterns. Fences verified. No content lost.
There was a problem hiding this comment.
Pull request overview
Refactors the graphql-developer skill to follow the repo’s progressive-disclosure pattern by keeping SKILL.md lean and moving the large “Expertise” content into an on-demand reference document.
Changes:
- Replaced the inlined ~500-line “Expertise” section in
SKILL.mdwith a short “Deep-dive references” router. - Added
references/graphql-expertise.mdcontaining the extracted GraphQL expertise material (versions, core concepts, examples, structure).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| claude/skills/architecture/graphql/graphql-developer/SKILL.md | Removes the large embedded expertise block and replaces it with a reference pointer to keep the skill lean. |
| claude/skills/architecture/graphql/graphql-developer/references/graphql-expertise.md | New reference file containing the extracted deep GraphQL expertise content for on-demand loading. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Owner
Author
|
Superseded by the agent-consolidation work: graphql-developer is being folded into solution-architect (/arch) as a reference rather than refactored as a standalone agent. |
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
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.
What
Slice-4 refactor #18. Splits
graphql-developer(591 lines) into a leanSKILL.md(100 lines) + 1references/file (the ~497-line Expertise block):graphql-expertise.md— versions, core concepts (schema design, resolvers, Apollo Server/Federation, DataLoader, subscriptions, security, performance), project structure.Lean
SKILL.mdkeeps trigger, context, doc lookup, standards, checklist, anti-patterns.Why
Same pattern as #5–#22 — content-preserving move;
SKILL.md< ~500 lines; deep content loaded on demand. No content lost (591 → 100 + 1 ref); fences verified balanced/non-nested.Review focus
Content preservation; coherent lean router.