Skip to content

Conversation

@elasticdotventures
Copy link
Member

  • sessionId= should not be required
  • example with stdio
  • added install
  • --tldr checkpoint 1
  • --tldr checkpoint 2
  • added count_tokens
  • --max-tokens CLI parameter is now implemented for the Test subcommand.
  • added 内容被截断
  • improved readme
  • list_crate_items
  • list_crate_items
  • checkpoint 1, broken
  • checkpoint 2, working
  • checkpoint, moved list_crate_items moved to tools
  • checkpoint, 1 test fails
  • added version tool
  • --tldr added tag stripping
  • checkpoint, syn in - but missing
  • list_crate_items appears to work!
  • tdlr didn't work with stdio
  • added tldr.rs
  • added --max_tokens ### to stdio mode
  • wip: testing pkgx & docker builds
  • feat: pkgx mcp registry

Copilot AI review requested due to automatic review settings November 15, 2025 23:04
Copilot finished reviewing on behalf of elasticdotventures November 15, 2025 23:05
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR renames the project from cratedocs-mcp to rust-cargo-docs-rag-mcp and adds comprehensive build/deployment infrastructure including Docker support, pkgx integration, and versioning automation via Cocogitto.

Key changes:

  • Project renamed across all configuration files and documentation
  • Added Docker containerization with Alpine-based images and GitHub Container Registry publishing
  • Integrated pkgx pantry for alternative installation method

Reviewed Changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/tools/docs/docs.rs Updated User-Agent headers to reflect new repository URL
Cargo.toml Updated package name to rust-cargo-docs-rag-mcp and bumped version to 0.3.0
Cargo.lock Updated package entry to match new name and version
README.md Comprehensive documentation overhaul with pkgx and Docker instructions
Dockerfile Multi-stage Alpine-based build for containerized deployment
docker/entrypoint.sh Entrypoint script for configurable container runtime modes
.dockerignore Standard exclusion patterns for Docker builds
justfile Added pkgx install and Docker build/run targets
scripts/set-version.sh Version bumping automation script for Cocogitto
cog.toml Cocogitto configuration for automated versioning
CHANGELOG.md Initial changelog documenting v0.3.0 changes
.github/workflows/docker.yml CI/CD workflow for Docker image building and MCP registry publishing

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -0,0 +1,11 @@
# Changelog

## [0.3.0] - 2025-11-12
Copy link

Copilot AI Nov 15, 2025

Choose a reason for hiding this comment

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

The release date is set to a future date (2025-11-12) which is inconsistent with typical changelog practices. This should either be the actual release date or use "Unreleased" if the version hasn't been released yet.

Suggested change
## [0.3.0] - 2025-11-12
## [0.3.0] - Unreleased

Copilot uses AI. Check for mistakes.
@@ -0,0 +1,16 @@
#!/bin/sh
Copy link

Copilot AI Nov 15, 2025

Choose a reason for hiding this comment

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

The entrypoint script should be marked as executable in the repository. After creating this file, run chmod +x docker/entrypoint.sh and commit the permission change, or ensure the Dockerfile's chmod +x /entrypoint.sh at line 29 is sufficient for runtime.

Copilot uses AI. Check for mistakes.
Copy link
Member Author

Choose a reason for hiding this comment

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

@copilot open a new pull request to apply changes based on this feedback

Copy link

Copilot AI commented Nov 15, 2025

@elasticdotventures I've opened a new pull request, #10, to work on those changes. Once the pull request is ready, I'll request review from you.

@elasticdotventures elasticdotventures merged commit 9584b65 into main Nov 15, 2025
2 checks passed
@elasticdotventures elasticdotventures deleted the feat/build-pkgx-install branch November 15, 2025 23:09
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