Skip to content

docs: fix review feedback on connection check, diagram, graphExpand status, and prerequisites#107

Closed
Copilot wants to merge 2 commits intodocs/skill-cli-firstfrom
copilot/sub-pr-105
Closed

docs: fix review feedback on connection check, diagram, graphExpand status, and prerequisites#107
Copilot wants to merge 2 commits intodocs/skill-cli-firstfrom
copilot/sub-pr-105

Conversation

Copy link
Contributor

Copilot AI commented Feb 24, 2026

Five documentation inconsistencies flagged in review, addressed across skill specs, vision doc, and skill README.

Changes

  • Connection check (skill/SKILL.md, skill/skills/raged/SKILL.md): Replace raged query health check with raged collections --api "$RAGED_URL"query requires embeddings to be running; collections only hits the API
  • Diagram (docs/04-claude-skills.md): Replace node dist/index.js query with raged query to match CLI-first convention used throughout the rest of the docs
  • graphExpand status (docs/00-vision.md): Correct "has been removed" to reflect current reality — the field is still active but experimental, transitioning to Apache AGE
  • Prerequisites (skill/README.md): Add explicit raged CLI install step; previously listed Node/npm but not the binary users actually need on PATH

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…nd, and prerequisites

Co-authored-by: mfittko <326798+mfittko@users.noreply.github.com>
Copilot AI changed the title [WIP] Align CLI-first docs and note Apache AGE graph transition docs: fix review feedback on connection check, diagram, graphExpand status, and prerequisites Feb 24, 2026
Copy link
Owner

@mfittko mfittko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Closing per maintainer request; changes are being handled in another PR.

@mfittko
Copy link
Owner

mfittko commented Feb 24, 2026

Closing this duplicate PR per maintainer request; we are continuing all fixes directly in #105.

@mfittko mfittko closed this Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants