Skip to content

Conversation

@zzstoatzz
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings June 8, 2025 00:02
Copy link
Contributor

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 restructures the Slackbot example into a standalone slackbot library, centralizes imports under the new package, and updates tooling and documentation to reference the library module.

  • Register slackbot as an optional dependency and UV workspace in the root pyproject.toml
  • Update CLI invocations (Justfile, Dockerfile) and Pydantic usage to use slackbot module paths
  • Adjust example project metadata and docs to reflect the new package name

Reviewed Changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pyproject.toml Added slackbot optional-dep and configured UV workspace
justfile Updated run-slackbot target to use -m slackbot
examples/slackbot/src/slackbot/*.py Changed all relative imports to slackbot.*
examples/slackbot/src/slackbot/api.py Switched to model_validate and updated import paths
examples/slackbot/src/slackbot/main.py Updated import and UV module reference
examples/slackbot/pyproject.toml Added project metadata for the new slackbot package
examples/slackbot/README.md Renamed entry point to __main__.py
examples/slackbot/Dockerfile.slackbot Bumped base image, updated UV commands
examples/slackbot/CLAUDE.md Updated start command in docs
Comments suppressed due to low confidence (2)

examples/slackbot/CLAUDE.md:40

  • The module path examples.slackbot does not match the library entry point. Update this to uv run -m slackbot to align with the package namespace.
uv run -m examples.slackbot

examples/slackbot/pyproject.toml:63

  • The UV source is labeled marvin, but the package is named slackbot. Rename the key to slackbot so the UV workspace can locate the correct module.
marvin = { workspace = true }

@zzstoatzz zzstoatzz merged commit 5b22dd6 into main Jun 8, 2025
2 checks passed
@zzstoatzz zzstoatzz deleted the organize-slackbot branch June 8, 2025 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants