Pull request overview

Updates the st.table docstring examples to showcase a more feature-rich key-value table example that demonstrates Streamlit’s Markdown capabilities (material icons, links, and badges), while removing a redundant older example.

Changes:

• Replaced the prior “auto-hidden headers” key-value example with a badges/material-icons example.
• Reduced the total number of st.table docstring examples from 6 to 5 by removing the redundant badges example that previously appeared later.

Summary

This PR is a docs-only change to the st.table docstring in lib/streamlit/elements/table.py. It consolidates two examples by promoting the more feature-rich "badges" example (previously Example 6) into the Example 4 slot, replacing the simpler key-value dict example. The old Example 6 is then removed as redundant, reducing the total example count from 6 to 5. The example format is also converted from doctest (>>>) to .. code-block:: python, consistent with project docstring guidelines. No runtime code, frontend, protobuf, or test changes are included.

All three reviewers (claude-4.6-opus-high-thinking, gemini-3.1-pro, gpt-5.3-codex-high) agreed on the scope and nature of the change. The sole point of contention was the .. output:: URL for Example 4 (see Recommendations).

Code Quality

The docstring edit is clean and well-structured. The new Example 4 uses the recommended .. code-block:: python format with :filename: directive and .. output:: reference, consistent with the other examples in the file and with the project's docstring guidelines in lib/streamlit/AGENTS.md. The auto-hidden-header behavior is still implicitly demonstrated because the example data remains a flat dict.

All three reviewers agreed the code quality is good with no structural or maintainability issues.

Test Coverage

No test changes needed. This is a pure documentation change with no behavioral impact. All three reviewers concurred.

Backwards Compatibility

No concerns. Only docstring content was modified — no API signatures, defaults, or runtime behavior changed. All three reviewers agreed.

Security & Risk

No security concerns. No runtime code is modified. All three reviewers agreed this is zero-risk from a security perspective.

External test recommendation

• Recommend external_test: No
• Triggered categories: None
• Evidence:
  • lib/streamlit/elements/table.py: Only docstring text was changed; no runtime, network, auth, embedding, or asset-serving code was touched.
• Suggested external_test focus areas: N/A
• Confidence: High
• Assumptions and gaps: None — the change is confined to documentation strings.

All three reviewers unanimously agreed no external test coverage is needed.

Accessibility

No frontend changes; not applicable. All three reviewers agreed.

Recommendations

1. Update the .. output:: URL for Example 4 (line 332): The URL https://doc-table-auto-header.streamlit.app/ was carried over from the old simple key-value Example 4, but the example code is now the badges example previously at https://doc-table-key-value-badges.streamlit.app/. All three reviewers flagged this mismatch — gemini-3.1-pro requested changes for it, while claude-4.6-opus-high-thinking and gpt-5.3-codex-high noted it as a follow-up. This is a minor but real inconsistency: the output preview URL should match the example content so the docs render the correct output. Consider updating the URL to https://doc-table-key-value-badges.streamlit.app/ (or the appropriate deployed URL for this example).

Verdict

APPROVED: Clean docs-only change that consolidates redundant examples and improves instructional value. The output URL mismatch (see inline comment) is worth fixing but is not blocking — the majority of reviewers approved and no critical issues were raised.

Consolidated from 3 of 3 expected AI reviews (claude-4.6-opus-high-thinking, gemini-3.1-pro, gpt-5.3-codex-high). Consolidation by claude-4.6-opus-high-thinking.

This review also includes 1 inline comment(s) on specific code lines.

Summary

This PR consolidates the st.table docstring examples by replacing the simple key-value Example 4 ("Display key-value data with auto-hidden headers") with the more feature-rich badges example (previously Example 6, "Display key-value data with badges"), and removing the now-redundant Example 6. This reduces the total examples from 6 to 5 while showcasing more of st.table's capabilities (material icons, Markdown links, color badges, border="horizontal", width="content") in a single example.

All three reviewers agreed on the nature and scope of the change.

Code Quality

The change is clean and well-structured. All three reviewers agreed on this point:

• The new Example 4 uses the .. code-block:: python format with :filename: directive, consistent with the other examples. The old Example 6 used the >>> doctest format, so this change also improves format consistency.
• The example title accurately describes the content.
• Indentation and docstring structure follow existing patterns.

Resolved disagreement — output URL: One reviewer (gemini-3.1-pro) flagged that the .. output:: URL (doc-table-auto-header.streamlit.app/) no longer matches the new example content and requested it be updated. However, the commit history explicitly shows this was intentionally preserved: the URL was updated in a second commit and then reverted in a third commit titled "Revert URL to doc-table-auto-header as requested". The deployed example app at that URL will be updated separately to reflect the new content. This is not a valid concern.

Test Coverage

No test changes are needed. All three reviewers agreed this is a pure docstring edit with no functional code, API surface, or runtime behavior changes.

Backwards Compatibility

No backwards compatibility concerns. All three reviewers agreed: no API signatures, behavior, or defaults are modified.

Security & Risk

No security concerns. All three reviewers agreed: this is a documentation-only change with no functional code modifications.

External test recommendation

• Recommend external_test: No
• Triggered categories: None
• Evidence:
  • lib/streamlit/elements/table.py: Only docstring text was changed; no functional code, routing, auth, WebSocket, embedding, asset serving, or security header modifications.
• Suggested external_test focus areas: N/A
• Confidence: High
• Assumptions and gaps: None — the change is clearly scoped to documentation only.

All three reviewers unanimously agreed no external test coverage is needed.

Accessibility

No accessibility impact. This is a docstring-only change that does not affect any rendered frontend components.

Recommendations

No blocking issues. No non-blocking follow-up changes are required for merge readiness.

Reviewer agreement summary

The sole point of disagreement — whether the output URL should be updated — is resolved by the commit history, which shows the URL was explicitly reverted at the author's request. All other findings are unanimous.

Verdict

APPROVED: Clean documentation consolidation that removes a redundant example, improves docstring format consistency, and introduces no functional changes. The sole objection (output URL mismatch) was investigated and found to be intentionally preserved per commit history.

Consolidated AI review by claude-4.6-opus-high-thinking. Individual reviews from: claude-4.6-opus-high-thinking, gemini-3.1-pro, gpt-5.3-codex-high.