Skip to content

docs: Add AGENTS.md#5579

Open
sentrivana wants to merge 8 commits intomasterfrom
ivana/agents-md
Open

docs: Add AGENTS.md#5579
sentrivana wants to merge 8 commits intomasterfrom
ivana/agents-md

Conversation

@sentrivana
Copy link
Contributor

@sentrivana sentrivana commented Mar 3, 2026

Description

Add a tailored AGENTS.md.

Tested with Claude fixing a small issue in the Celery integration. It correctly identified where to make the change, where to add the tests, and how to run the correct test suite.

Issues

Closes #5576

Reminders

@sentrivana sentrivana marked this pull request as ready for review March 3, 2026 10:07
@sentrivana sentrivana requested a review from a team as a code owner March 3, 2026 10:07
AGENTS.md Outdated

## Type Checking

- Always run mypy from the main `.venv`. If there is no `.venv`, create it and install `pip install -r requirements-lint.txt -r requirements-test.txt`
Copy link
Contributor

Choose a reason for hiding this comment

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

We should create a tox env that only runs mypy imo, feel like it's easy for a virtual env to be polluted.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I like the idea. Added ruff and mypy as tox envs and tweaked the file.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Bug Fixes 🐛

  • (celery) Propagate user-set headers by sentrivana in #5581

Documentation 📚

  • Add AGENTS.md by sentrivana in #5579
  • Add set_attribute example to changelog by sentrivana in #5578

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026

Codecov Results 📊

1707 passed | ⏭️ 184 skipped | Total: 1891 | Pass Rate: 90.27% | Execution Time: 2m 21s

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 11872 uncovered lines.


Generated by Codecov Action

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Copy link
Member

@sl0thentr0py sl0thentr0py left a comment

Choose a reason for hiding this comment

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

nice, do we want to keep .cursor or remove it now?

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.

Add AGENTS.md

3 participants