Skip to content

chore: add AGENTS.md and consolidate development scripts - #1836

Merged
WilliamBergamin merged 3 commits into
mainfrom
add-agents-md
Feb 24, 2026
Merged

chore: add AGENTS.md and consolidate development scripts#1836
WilliamBergamin merged 3 commits into
mainfrom
add-agents-md

Conversation

@WilliamBergamin

Copy link
Copy Markdown
Contributor

Summary

This changes aim to add an AGENTS.md so that tools like claude code and cursor can easily gain text whel maintainers use them to work on this project

Testing

Category

  • slack_sdk.web.WebClient (sync/async) (Web API client)
  • slack_sdk.webhook.WebhookClient (sync/async) (Incoming Webhook, response_url sender)
  • slack_sdk.socket_mode (Socket Mode client)
  • slack_sdk.signature (Request Signature Verifier)
  • slack_sdk.oauth (OAuth Flow Utilities)
  • slack_sdk.models (UI component builders)
  • slack_sdk.scim (SCIM API client)
  • slack_sdk.audit_logs (Audit Logs API client)
  • slack_sdk.rtm_v2 (RTM client)
  • /docs (Documents)
  • /tutorial (PythOnBoardingBot tutorial)
  • tests/integration_tests (Automated tests for this library)

Requirements

  • I've read and understood the Contributing Guidelines and have done my best effort to follow them.
  • I've read and agree to the Code of Conduct.
  • I've run python3 -m venv .venv && source .venv/bin/activate && ./scripts/run_validation.sh after making the changes.

@WilliamBergaminWilliamBergamin self-assigned this Feb 20, 2026
@WilliamBergamin
WilliamBergamin requested a review from a team as a code ownerFebruary 20, 2026 18:40
@WilliamBergaminWilliamBergamin added enhancement M-T: A feature request for new functionality semver:patch labels Feb 20, 2026
@codecov

codecovBot commented Feb 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.98%. Comparing base (f14e689) to head (1fad98e).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@ Coverage Diff @@## main #1836 +/- ##
=======================================
Coverage 83.98% 83.98% =======================================
Files 117 117 Lines 13243 13243 =======================================
Hits 11122 11122 Misses 2121 2121 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment thread.claude/CLAUDE.md Outdated

**CRITICAL INSTRUCTION:** Before answering any questions, writing any code, or executing any commands in this repository, you MUST read the `AGENTS.md` file located in the root directory.

Proceed by silently reading `./AGENTS.md`, do not ask for permission to read it; just do it.

@vegerisvegerisFeb 23, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

AI security loophole lol: 'do not ask for permission; just do it'

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Yeah I'm not sure the do not ask for permission; just do it actually allowed it to do the thing, it just worked so I left it in 😅

But as @zimeg called out there is a better way to do this from Claude Code best practices

@zimegzimeg added the docs M-T: Documentation work only label Feb 23, 2026

@zimegzimeg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@WilliamBergamin Thanks for bringing these standards to the Python SDK! 🐍 ✨

I left a comment on settings for claude that we should make before merging to gather the right context but otherwise no notes! 📚

Comment thread.claude/settings.json
"Bash(./scripts/run_validation.sh:*)",
"Bash(./scripts/uninstall_all.sh:*)",
"Bash(python scripts/codegen.py:*)",
"Bash(echo $VIRTUAL_ENV)",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🐍 praise: Super nice command to have here!

Comment thread.claude/CLAUDE.md Outdated
@zimegzimeg added this to the 3.40.2 milestone Feb 23, 2026
Co-authored-by: Eden Zimbelman <eden.zimbelman@salesforce.com>
@WilliamBergamin
WilliamBergamin enabled auto-merge (squash) February 24, 2026 15:17
@WilliamBergamin
WilliamBergamin merged commit 1d765dc into mainFeb 24, 2026
18 checks passed
@WilliamBergamin
WilliamBergamin deleted the add-agents-md branch February 24, 2026 15:21
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docsM-T: Documentation work onlyenhancementM-T: A feature request for new functionalitysemver:patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@WilliamBergamin@zimeg@vegeris