aws-samples / aws-samples/sample-agentic-platform

[pyproject] Clean up stale uv.workspace member paths

Open
#48 0 comments 0 reactions 1 assignee Claimed by @ManojRamani View on GitHub
Dominant language
Python
Stars
133
Forks
54
PR merge metrics
No merged PRs in 30d

Description

The `[tool.uv.workspace]` members list in root `pyproject.toml` references paths that don't match the actual directory layout:

\`\`\`toml
members = [
\"labs/module4/notebooks/mcp\", # does not exist
\"labs/module4/mcp_server/weather_server\", # does not exist
\"labs/module4/notebooks/common/mcp_server/weather_server\" # does not exist
]
\`\`\`

The only MCP server \`pyproject.toml\` in the repo lives at \`labs/module4/notebooks/mcp_servers/weather_server/pyproject.toml\`.

\`uv sync\` may be silently ignoring these missing entries (or warning on them), which makes the workspace definition misleading for new contributors.

### Suggested fix
Replace the three stale paths with the single correct one:

\`\`\`toml
[tool.uv.workspace]
members = [\"labs/module4/notebooks/mcp_servers/weather_server\"]
\`\`\`

…or remove them entirely if the weather server is intended to be standalone.

### Context
Discovered while planning a restructure that adds \`labs/\` as a new workspace member (to host a shared \`labs_common\` package for model-ID config). Flagging separately so that cleanup can be scoped on its own.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.