addressing-pr-review-c…
Address all valid review comments on a PR for the current branch in the streamlit/streamlit repo. Covers both inline review comments and general PR (issue)…
Review internal documentation (*.md files) against the current codebase state and propose updates for outdated or incorrect information.
$ npx -y skills add streamlit/streamlit --skill updating-internal-docs --agent claude-codeHow it fires
How this skill gets triggered: by you, by Claude, or both.
/updating-internal-docsContext preview
The summary Claude sees to decide when to auto-load this skill.
Review internal documentation (*.md files) against the current codebase state and propose updates for outdated or incorrect information.
name: updating-internal-docs description: Review internal documentation (*.md files) against the current codebase state and propose updates for outdated or incorrect information.
Review internal documentation files against the actual codebase state and propose fixes for outdated, incorrect, or missing information.
Priority files (most likely to contain codebase-specific instructions):
**Files to skip** (synced copies, updated separately):
If you edit a source `AGENTS.md` or `scripts/assets/code-review-instructions.md`, run `uv run python scripts/generate_agent_rules.py` so generated copies stay in sync.
When a PR **adds or changes a Streamlit feature** (new widget, API change, deprecation, new capability), check if the bundled skills need updates:
For periodic reviews, treat recently merged PRs as leads for documentation drift. Inspect those diffs, then verify the current code before updating docs. A merge does not by itself require a bundled-skill update; apply the prominence and scope rules in `lib/streamlit/.agents/skills/AGENTS.md`.
Common triggers for bundled skill updates:
# Check path exists: test -e path && echo ok || echo missing
# Check URL reachable: curl -sI -o /dev/null -w "%{http_code}" <url>| Tool | Config file | |------|-------------| | TypeScript, React, Vite, Vitest, ESLint, oxfmt, Emotion | `frontend/package.json` | | Yarn | `frontend/package.json` (`packageManager` field) | | Python, Ruff, mypy, pytest | `pyproject.toml` | | Node.js | `.nvmrc` |
| Type | Description | |------|-------------| | OUTDATED | Info no longer accurate (old make targets, renamed files) | | INCORRECT | Factually wrong (wrong paths, invalid commands) | | VERSION_MISMATCH | Documented version differs from actual | | MISSING | Important info not documented | | BROKEN_LINK | Links to non-existent resources | | INCONSISTENT | Conflicts with other docs |
1. **Enumerate**: Find all markdown documentation files 2. **Verify**: Cross-reference documented commands, paths, and examples against the codebase 3. **Report**: Present findings grouped by priority 4. **Fix**: Apply changes after user approval
List all issues and let the user choose which to fix:
Documentation Review: {SCOPE}
═══════════════════════════════════════════════════════════════
Found {N} issues across {M} files:
1. [OUTDATED] AGENTS.md:42
Current: `make python-check`
Actual: Command renamed to `make python-lint`
2. [INCORRECT] wiki/testing.md:15
Current: Tests in `lib/tests/unit/`
Actual: Path is `lib/tests/streamlit/`
3. [BROKEN_LINK] CONTRIBUTING.md:88
Current: Link to `./docs/setup.md`
Actual: File does not exist
Which issues should I fix?
Recommended: "all"
Options: "1" | "1,2,3" | "all" | "skip 3"Repo: streamlit/streamlit
Address all valid review comments on a PR for the current branch in the streamlit/streamlit repo. Covers both inline review comments and general PR (issue)…
Assesses whether branch or PR changes are high-risk for externally hosted or embedded Streamlit usage and recommends whether external e2e coverage with…
Validates all code changes before committing by running format, lint, type, and unit test checks. Use after making backend (Python) or frontend (TypeScript)…
Creates a draft pull request on GitHub with proper labels, branch naming, and description formatting. Use when changes are ready to be submitted as a PR to the…
Debug Streamlit frontend and backend changes using make debug with hot-reload. Use when testing code changes, investigating bugs, checking UI behavior, or…
Lists available make commands for Streamlit development. Use for build, test, lint, or format tasks.