docs: extract installation details from CLAUDE.md and reduce duplication#29
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Refactors developer-facing documentation to keep CLAUDE.md concise (for Claude Code context efficiency) by moving detailed installation/release/launcher content into docs/installation.md, while also adding OpenSpec documentation for a ToolFactory refactor (ToolScope DSL + ToolDiscovery extraction).
Changes:
- Extracted “Installation & Distribution” guidance into
docs/installation.mdand trimmedCLAUDE.mdaccordingly. - Simplified
CLAUDE.mdarchitecture + workflow sections and removed duplicated deep dives already covered elsewhere. - Added new OpenSpec specs + an archived change proposal/design/tasks for the ToolFactory refactor.
Reviewed changes
Copilot reviewed 4 out of 10 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
docs/installation.md |
New consolidated installation/distribution documentation extracted from CLAUDE.md. |
CLAUDE.md |
Reduced size by removing detailed installation/release and deep-dive sections; added links to “Further Reading”. |
openspec/specs/tool-registration-dsl/spec.md |
New spec describing ToolScope/registrar approach (has a few inconsistencies vs current code). |
openspec/specs/tool-discovery/spec.md |
New spec describing extracted ToolDiscovery behavior. |
openspec/changes/archive/2026-03-24-refactor-toolfactory/.openspec.yaml |
Metadata for archived OpenSpec change set. |
openspec/changes/archive/2026-03-24-refactor-toolfactory/proposal.md |
Archived proposal for ToolFactory refactor. |
openspec/changes/archive/2026-03-24-refactor-toolfactory/design.md |
Archived design notes for ToolFactory refactor (some counts don’t match code). |
openspec/changes/archive/2026-03-24-refactor-toolfactory/tasks.md |
Archived task checklist (mentions runBlocking though implementation doesn’t). |
openspec/changes/archive/2026-03-24-refactor-toolfactory/specs/tool-registration-dsl/spec.md |
Archived copy of the ToolScope/registrar spec (same inconsistencies as the main spec). |
openspec/changes/archive/2026-03-24-refactor-toolfactory/specs/tool-discovery/spec.md |
Archived copy of the ToolDiscovery spec. |
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.
Summary
CLAUDE.mdfrom 453 → 165 lines (64% reduction)prerequisites into
docs/installation.mdArchitecture Overview and MCP Tools sections)
LEARNING.mdtables derivable from the codebase
unified section
LEARNING.md,docs/installation.md, andkotlin-mcp-server.instruction.mdMotivation
CLAUDE.mdis loaded into every Claude Code conversation. Keepingit focused on actionable coding guidance (build commands, tool
references, key patterns) reduces context window usage while
preserving all information Claude needs to work with the codebase.