Skip to content

docs(skills): document test_server() in the bundled shiny-for-python skill - #2491

Merged
schloerke merged 3 commits into
mainfrom
schloerke/skill-test-server-docs
Sep 11, 2026
Merged

schloerke merged 3 commits into
mainfrom
schloerke/skill-test-server-docs

Conversation

@schloerke

Copy link
Copy Markdown
Collaborator

Summary

#2470 added shiny.testserver.test_server() but the bundled shiny-for-python Agent Skill did not mention it, so coding agents using the skill still reach for Playwright (or hand-built sessions) when only server logic needs testing.

  • New references/test-server.md: what test_server() accepts, set_inputs/get_output/get_export, TestServerValue statuses (ok/error/silent), module scopes, client data defaults, fixtures/snapshots, test_server_async, common mistakes.
  • Index row in SKILL.md (Testing & observability) plus a mention in the umbrella description.
  • Cross-link from references/testing.md pointing server-logic tests at test_server().
  • CHANGELOG entry.

Validation

  • pytest tests/pytest/test_packaging.py passes.
  • uvx --from skills-ref agentskills validate shiny/.agents/skills/shiny-for-python reports a valid skill.

…skill

Adds a test-server reference topic covering test_server(), value statuses,
module scopes, client data, fixtures, and test_server_async, plus an index
row and a cross-link from the Playwright testing reference.
@schloerke
schloerke enabled auto-merge (squash) September 11, 2026 20:41
@schloerke
schloerke merged commit 95784c5 into main Sep 11, 2026
175 checks passed
@schloerke
schloerke deleted the schloerke/skill-test-server-docs branch September 11, 2026 20:45
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.

1 participant