Set up Cursor Cloud dev environment#219
Conversation
Co-authored-by: Liu Ziming <[email protected]>
|
|
|
This PR has been automatically marked as stale because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 days of this comment. |
|
📋 CLA Reminder Thank you for this PR! Before it can be merged, please sign the Contributor License Agreement (CLA). Once signed, the CLA status will update automatically and we can proceed with approval. Automated notification by github-manager-bot |
ralf0131
left a comment
There was a problem hiding this comment.
Summary
This PR adds an AGENTS.md file with Cursor Cloud dev environment instructions for the LoongSuite Python monorepo — toolchain overview, common commands, and non-obvious gotchas.
The documentation is well-structured and accurate based on the repo setup. No code issues found.
Notes
⚠️ CLA is not signed yet — this PR cannot be approved/merged until the contributor signs the CLA. See the CLA reminder comment above.- ❗ The
changelogCI check is failing. This PR likely needs a changelog entry (e.g.,.changes/orCHANGES.mddepending on the repo convention). Please check the CI log and add the required changelog file.
Automated review by github-manager-bot
Summary
Sets up the development environment for this OpenTelemetry Python Contrib / LoongSuite instrumentation monorepo and documents it for future Cursor Cloud agents. No product code is changed.
AGENTS.mdwith a## Cursor Cloud specific instructionssection describing theuv+toxworkflow, common commands, required system build libs, and non-obvious gotchas.Environment setup performed
uvand system build libs (build-essential pkg-config libpq-dev default-libmysqlclient-dev librdkafka-dev libsnappy-dev python3-dev) needed to compile native deps (psycopg2, mysqlclient, etc.).uv syncbuilds the dev virtualenv at.venv/(all workspace packages +tox/tox-uv/pre-commit).uvif missing, thenuv sync.Verification
.venv/bin/tox -e ruff.venv/bin/tox -e py312-test-instrumentation-requests.venv/bin/tox -c tox-loongsuite.ini -e py312-test-loongsuite-instrumentation-langchain-latestRequestsInstrumentor+ConsoleSpanExporterGET example.comhttp.status_code: 200Hello-world span output
hello_world_span.log
Gotcha discovered
The
opentelemetry-instrumentauto-instrumentation CLI currently fails in theuv syncvenv (core API/SDK pinned to upstreammainis ahead of the bundled logging instrumentor:LogRecord.__init__() got an unexpected keyword argument 'context'). Per-packagetoxenvs are unaffected (they pinCORE_REPO_SHA). Use programmatic instrumentation for quick smoke tests. Documented inAGENTS.md.To show artifacts inline, enable in settings.