Skip to content

feat(desktop): add Show Me tool - #885

Open
myu404 wants to merge 2 commits into
tableau:feature/desktopfrom
myu404:codex/add-show-me-tools
Open

feat(desktop): add Show Me tool#885
myu404 wants to merge 2 commits into
tableau:feature/desktopfrom
myu404:codex/add-show-me-tools

Conversation

@myu404

@myu404 myu404 commented Sep 4, 2026

Copy link
Copy Markdown

Decision

This establishes a rule we now keep: worksheet Show Me actions use a dedicated typed Desktop tool instead of the generic command bridge.

Description

  • Add show-me with the exact Show Me type values and optional datasource and field-selection inputs.
  • Resolve worksheet names or stable IDs and send the action through the External Client API Operation pipeline.
  • Hide the tool from Desktop API versions earlier than 0.2.11.

Motivation and Context

Agents need a first-class way to invoke the worksheet Show Me route after the Desktop-side route became available.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

How Has This Been Tested?

Targeted Desktop tool, transport, registry, schema-budget, and version-gate tests.

Verification

  • npx vitest run src/tools/desktop/api/showMe.test.ts src/server.desktop.test.ts src/tools/toolName.test.ts src/server.desktop.schema-equivalence.test.ts

Related Issues

  • None.

Checklist

  • I have updated the version in the package.json file by using npm run version. For example,
    use npm run version:patch for a patch version bump.
  • I have made any necessary changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have documented any breaking changes in the PR description. For example, renaming a config
    environment variable or changing its default value.

Contributor Agreement

By submitting this pull request, I confirm that:

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