Optimize AI agent setup by lukasmasuch · Pull Request #13800 · streamlit/streamlit

lukasmasuch · 2026-02-03T14:09:48Z

Summary

Updates AI agent configurations and improves development tooling:

AI Agent Configs: Updated configurations for Claude Code, Cursor, Copilot, and Codex
Shared Skills: Codex now uses a symlink to .claude/skills/ for skill auto-discovery
PostToolUse Hook: Added async hook to auto-fix Python files after edits (ruff check + format)
Makefile Fix: Corrected ruff lint/format order in make check and make autofix (lint first, then format per ruff docs)
Documentation: Updated AGENTS.md files with development guidelines

Changes

AI Agent Configurations

.claude/settings.json: Command permissions and hooks configuration
.codex/skills → .claude/skills/: Symlink for shared skill auto-discovery
.cursor/rules/: Updated Cursor rules
.github/copilot-instructions.md: Updated Copilot instructions

New Hook: `post_edit_autofix.sh`

Async PostToolUse hook that runs after Edit/Write operations on Python files:

Runs ruff check --fix then ruff format (correct order)
Respects ruff exclusions from pyproject.toml
Never fails on unfixable errors (just applies what it can)
Runs asynchronously so Claude continues working

Makefile Improvements

Fixed make check and make autofix to run ruff check --fix before ruff format
This follows ruff's recommended order: lint fixes can modify code, so formatting should run last

AGENTS.md Documentation Updates

lib/streamlit/AGENTS.md: Added theming and layout guidance - theming/layout calculations must be done in the frontend, not the Python backend; avoid using get_option("theme.primaryColor") in backend code
lib/tests/AGENTS.md: Added guidance on imports - imports should be at the top-level of test files unless there is a specific reason (e.g., integration requirements, circular imports)

Testing Plan

No additional tests needed. Configuration files are static and the hook has been manually tested.

Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

- Added test file co-location conventions to Testing Strategy section in AGENTS.md - Created .claude/settings.json with Claude Code permissions and plansDirectory setting - Created .cursor/cli.json with Cursor CLI permissions - Created .codex/config.toml with Codex skills configuration - Updated gitignores to use whitelist pattern (allow specific files instead of blacklist) - Generated GitHub Copilot and Cursor rule files from updated AGENTS.md Co-Authored-By: Claude (claude-haiku-4-5) <noreply@anthropic.com>

snyk-io · 2026-02-03T14:10:10Z

✅ Snyk checks have passed. No issues have been found so far.

Status	Scanner	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues
✅	Licenses	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

github-actions · 2026-02-03T14:11:24Z

✅ PR preview is ready!

Name	Link
📦 Wheel file	https://core-previews.s3-us-west-2.amazonaws.com/pr-13800/streamlit-1.53.1-py3-none-any.whl
📦 `@streamlit/component-v2-lib`	Download from artifacts
🕹️ Preview app	pr-13800.streamlit.app (☁️ Deploy here if not accessible)

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.}

.claude/settings.json

Copilot

Pull request overview

This PR adds AI agent configuration files and enhances documentation by adding test file co-location conventions to help AI agents understand the project structure better.

Changes:

Added detailed test file location conventions to the Testing Strategy section across three documentation files (AGENTS.md, .github/copilot-instructions.md, .cursor/rules/overview.mdc)
Created configuration files for three AI coding assistants: Claude Code (.claude/settings.json), Cursor (.cursor/cli.json), and Codex (.codex/config.toml) with appropriate command permissions
Updated gitignore files for .claude/, .cursor/, and .codex/ directories to use whitelist patterns for better security

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
AGENTS.md	Added test file co-location details to Testing Strategy section
.github/copilot-instructions.md	Added test file co-location details to Testing Strategy section (identical to AGENTS.md)
.cursor/rules/overview.mdc	Added test file co-location details to Testing Strategy section (identical to AGENTS.md)
.cursor/cli.json	New file: Cursor CLI configuration with shell command permissions for make, uv, yarn, and gh
.cursor/.gitignore	Updated to use whitelist pattern, ignoring all files by default and explicitly allowing specific tracked files
.codex/config.toml	New file: Codex configuration registering two skills from .claude/skills/ directory
.codex/.gitignore	New file: Whitelist pattern gitignore allowing only .gitignore and config.toml
.claude/settings.json	Added plansDirectory and permissions configuration with allow/ask lists for bash commands
.claude/.gitignore	Updated to use whitelist pattern, explicitly allowing specific files in hooks/ and skills/ directories

AGENTS.md

.github/copilot-instructions.md

.cursor/rules/overview.mdc

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Run `ruff check --fix` before `ruff format` per ruff's recommended order: linting fixes can modify code (e.g., removing unused imports), so formatting should run last to ensure consistent style. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Replace explicit skill configurations in .codex/config.toml with a symlink to .claude/skills/. Codex auto-discovers skills in .codex/skills/ and follows symlinks, so this provides a single source of truth. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

The config file is no longer needed since skills are auto-discovered via the symlink to .claude/skills/. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Adds an async hook that runs `ruff check --fix` and `ruff format` on Python files after Edit/Write operations. The hook: - Runs asynchronously so Claude continues working - Only processes .py files - Respects ruff's exclusions from pyproject.toml - Never fails on unfixable lint errors (just applies what it can) Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

- Remove async flag to prevent race conditions with concurrent edits - Reduce timeout to 5s (ruff is very fast) - Add Cursor afterFileEdit hook using the same shared script - Update script to handle both Claude and Cursor input formats Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Move ruff-specific comments closer to the ruff commands and remove outdated async reference. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Use ${CLAUDE_PROJECT_DIR:-$CURSOR_PROJECT_DIR} to support running the hook in both Claude Code and Cursor environments. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

lukasmasuch · 2026-02-03T15:11:55Z

.codex/skills

just a symlink to the .claude/skills folder

lukasmasuch · 2026-02-03T15:12:32Z

Makefile

-		echo "=== Python: format (ruff) ===" && \
-		uv run ruff format $$PY_FILES && \
-		echo "" && \
 		echo "=== Python: lint (ruff) ===" && \
 		uv run ruff check --fix $$PY_FILES && \
 		echo "" && \


the recommended way in ruff is to first apply check fixes and then the formatting

lukasmasuch · 2026-02-03T15:13:08Z

Makefile

+		echo "=== Python: format (ruff) ===" && \
+		uv run ruff format $$PY_FILES && \
+		echo "" && \


the recommended way in ruff is to first apply ruff check auto fixes and then the formatting.

lukasmasuch · 2026-02-03T15:13:59Z

.claude/hooks/post_edit_autofix.sh

Ruff format and check is extremely fast, especially on single files. Therefore, I think its fine to always run this in auto-fix mode after every file write or edit

github-actions · 2026-02-03T15:19:30Z

📉 Frontend coverage change detected

The frontend unit test (vitest) coverage has decreased by 0.0500%

Current PR: 86.3800% (13706 lines, 1866 missed)
Latest develop: 86.4300% (13706 lines, 1859 missed)

💡 Consider adding more unit tests to maintain or improve coverage.

📊 View detailed coverage comparison

Document that theming and layout calculations should be done in the frontend, not the Python backend. Using get_option("theme.primaryColor") is unreliable since themes can be configured in multiple ways. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Move the theming and layout section from lib/AGENTS.md to the more specific lib/streamlit/AGENTS.md, which contains guidelines specific to Streamlit library development. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

.claude/hooks/post_edit_autofix.sh

Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Imports should be at the top-level of test files unless there is a specific reason to place them inside test functions. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Exit silently if CLAUDE_PROJECT_DIR/CURSOR_PROJECT_DIR is unset or doesn't exist, preventing the script from running in the wrong directory. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

…olocation-docs

Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

sfc-gh-bnisco · 2026-02-04T18:05:36Z

.claude/settings.json

+      "Bash(uv sync *)",
+      "Bash(uv pip *)",
+      "Bash(yarn *)",
+      "Bash(gh *)",


suggestion: We should limit the gh subcommands to read-only commands.

Copilot AI review requested due to automatic review settings February 3, 2026 14:09

lukasmasuch requested a review from a team as a code owner February 3, 2026 14:09

lukasmasuch added security-assessment-completed Security assessment has been completed for PR change:other PR contains other type of change impact:internal PR changes only affect internal code labels Feb 3, 2026

Copilot started reviewing on behalf of lukasmasuch February 3, 2026 14:11 View session

cursor bot reviewed Feb 3, 2026

View reviewed changes

.claude/settings.json Show resolved Hide resolved

Copilot AI reviewed Feb 3, 2026

View reviewed changes

AGENTS.md Outdated Show resolved Hide resolved

.github/copilot-instructions.md Outdated Show resolved Hide resolved

.cursor/rules/overview.mdc Outdated Show resolved Hide resolved

lukasmasuch and others added 10 commits February 3, 2026 15:17

Update AGENTS.md

5d9415e

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update .github/copilot-instructions.md

ab3c7d3

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update .cursor/rules/overview.mdc

9766f3f

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Remove .codex/config.toml

b67ac28

The config file is no longer needed since skills are auto-discovered via the symlink to .claude/skills/. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Clean up post_edit_autofix.sh comments

7fe13b5

Move ruff-specific comments closer to the ruff commands and remove outdated async reference. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Support both Claude and Cursor project dir env vars

d43e4c6

Use ${CLAUDE_PROJECT_DIR:-$CURSOR_PROJECT_DIR} to support running the hook in both Claude Code and Cursor environments. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

lukasmasuch commented Feb 3, 2026

View reviewed changes

.codex/skills

Copy link

Collaborator Author

lukasmasuch Feb 3, 2026

Choose a reason for hiding this comment

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

just a symlink to the .claude/skills folder

lukasmasuch commented Feb 3, 2026

View reviewed changes

lukasmasuch changed the title ~~Add AI agent configurations and test file co-location docs~~ Optimize AI agent setup Feb 3, 2026

lukasmasuch and others added 2 commits February 3, 2026 16:21

graphite-app bot reviewed Feb 3, 2026

View reviewed changes

.claude/hooks/post_edit_autofix.sh Outdated Show resolved Hide resolved

lukasmasuch and others added 5 commits February 3, 2026 16:27

Simplify theming guidance wording

13bf904

Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Remove 'only' from theming guidance

4dcbd7a

Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Add guidance on top-level imports in test files

d9b8a28

Imports should be at the top-level of test files unless there is a specific reason to place them inside test functions. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Add AppTest exception for in-function imports

b3018a9

Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Add guard for missing project directory in autofix hook

d5dc3af

Exit silently if CLAUDE_PROJECT_DIR/CURSOR_PROJECT_DIR is unset or doesn't exist, preventing the script from running in the wrong directory. Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

lukasmasuch added the ai-review If applied to PR or issue will run AI review workflow label Feb 3, 2026

github-actions bot removed the ai-review If applied to PR or issue will run AI review workflow label Feb 3, 2026

lukasmasuch and others added 2 commits February 3, 2026 18:03

Merge remote-tracking branch 'origin/develop' into lukasmasuch/test-c…

cccfb7d

…olocation-docs

Remove .cursor/cli.json

b9f3568

Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

streamlit deleted a comment from github-actions bot Feb 3, 2026

lukasmasuch and others added 3 commits February 3, 2026 18:09

Remove cli.json from .cursor/.gitignore

40be195

Co-Authored-By: Claude (claude-opus-4-5) <noreply@anthropic.com>

Update

0ad52d5

Merge branch 'develop' into lukasmasuch/test-colocation-docs

d722c33

sfc-gh-bnisco reviewed Feb 4, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Optimize AI agent setup#13800

Optimize AI agent setup#13800
lukasmasuch wants to merge 23 commits intodevelopfrom
lukasmasuch/test-colocation-docs

lukasmasuch commented Feb 3, 2026 •

edited

Loading

Uh oh!

snyk-io bot commented Feb 3, 2026 •

edited

Loading

Uh oh!

github-actions bot commented Feb 3, 2026 •

edited

Loading

Uh oh!

cursor bot left a comment

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lukasmasuch Feb 3, 2026

Uh oh!

lukasmasuch Feb 3, 2026

Uh oh!

lukasmasuch Feb 3, 2026

Uh oh!

lukasmasuch Feb 3, 2026

Uh oh!

github-actions bot commented Feb 3, 2026 •

edited

Loading

Uh oh!

Uh oh!

sfc-gh-bnisco Feb 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

lukasmasuch commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

AI Agent Configurations

New Hook: post_edit_autofix.sh

Makefile Improvements

AGENTS.md Documentation Updates

Testing Plan

Uh oh!

snyk-io bot commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Snyk checks have passed. No issues have been found so far.

Uh oh!

github-actions bot commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ PR preview is ready!

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lukasmasuch Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

lukasmasuch Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

lukasmasuch Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

lukasmasuch Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📉 Frontend coverage change detected

Uh oh!

Uh oh!

sfc-gh-bnisco Feb 4, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

lukasmasuch commented Feb 3, 2026 •

edited

Loading

New Hook: `post_edit_autofix.sh`

snyk-io bot commented Feb 3, 2026 •

edited

Loading

github-actions bot commented Feb 3, 2026 •

edited

Loading

github-actions bot commented Feb 3, 2026 •

edited

Loading