Skip to content
This repository was archived by the owner on Jul 24, 2026. It is now read-only.

Prepare edison-detectord for crates.io publishing#5

Merged
IliaMManolov merged 2 commits into
masterfrom
cargo_publish
Jul 14, 2026
Merged

Prepare edison-detectord for crates.io publishing#5
IliaMManolov merged 2 commits into
masterfrom
cargo_publish

Conversation

@IliaMManolov

@IliaMManolov IliaMManolov commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Prepares the library crate for its first crates.io release under the name edison-detectord (detectord alone was deemed too generic to claim).

Changes

  • Rename mcp_detector_libedison-detectord: package name, crates/ directory, all use mcp_detector_lib:: imports across the workspace, and README/architecture doc references.
  • Fix publish blocker: readme = "README.md" pointed at a file that doesn't exist in the crate directory; now points at the repo README.
  • Metadata: add repository/homepage pointing at this repo; swap the command-line-utilities category for config (this is a library).
  • License: sync to AGPL-3.0-only and add the LICENSE file, matching the other Edison Watch repos (was MIT OR Apache-2.0 with no license file).

The other three crates (mcp_detector_daemon, mcp_backend, mcp_quarantine) remain publish = false and only had their path deps/imports updated.

Verification

  • cargo test --workspace: 115 tests pass
  • cargo publish --dry-run -p edison-detectord: packages and verifies cleanly
  • The name edison-detectord is unclaimed on crates.io

After merge: cargo publish -p edison-detectord (no dependency ordering — the lib has no path deps).

🤖 Generated with Claude Code

https://claude.ai/code/session_01QNXkCRxAip5uDbzdjjSi7t


Summary by cubic

Prepares the library for its first crates.io release as edison-detectord. Renames the crate, fixes publish metadata, aligns licensing across the workspace, and updates cargo-deny to allow AGPL for first-party crates.

  • Refactors

    • Rename mcp_detector_libedison-detectord (crate dir, package name, all imports/usages, examples/tests, docs, and watcher thread name).
    • Fix publish blocker by pointing readme to the repo README.md; add repository and homepage; change category to config.
    • Switch license to AGPL-3.0-only and add LICENSE; update workspace and dependent crates to use the new crate and license.
    • Keep mcp_backend, mcp_quarantine, and mcp_detector_daemon as publish = false.
    • Allow AGPL-3.0-only for first-party crates in deny.toml (cargo-deny).
  • Migration

    • Downstream code should import the crate as edison_detectord.
    • After merge: cargo publish -p edison-detectord (no dependency ordering needed).

Written for commit 4ce08dc. Summary will update on new commits.

Review in cubic

- Rename mcp_detector_lib to edison-detectord (crate, directory, imports, docs)
- Add repository/homepage metadata and point readme at repo README
- Sync license to AGPL-3.0-only and add LICENSE file, matching other Edison Watch repos
- Swap command-line-utilities category for config (it's a library)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QNXkCRxAip5uDbzdjjSi7t
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 46 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@IliaMManolov
IliaMManolov merged commit cc10ba2 into master Jul 14, 2026
11 checks passed
@IliaMManolov
IliaMManolov deleted the cargo_publish branch July 14, 2026 13:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant