Skip to content

Conversation

@billfienberg
Copy link
Contributor

@billfienberg billfienberg commented Jan 7, 2026

Background

Adds a new admin.app.tools.data extension target that enables apps to provide tool results for the admin tools functionality.

Solution

Introduces admin.app.tools.data extension target using RunnableExtension<StandardApi, undefined>.

Checklist

  • I have 🎩'd these changes
  • I have updated relevant documentation

References

@billfienberg billfienberg force-pushed the bf/add-admin-search-data-target branch from 5806d06 to cbc93e1 Compare January 7, 2026 17:31
@billfienberg billfienberg changed the base branch from unstable to 2025-10 January 7, 2026 17:33
@billfienberg billfienberg mentioned this pull request Jan 7, 2026
2 tasks
@Shopify Shopify deleted a comment from kbav Jan 7, 2026
@Shopify Shopify deleted a comment from kbav Jan 8, 2026
@kbav
Copy link
Contributor

kbav commented Jan 8, 2026

/snapit

1 similar comment
@billfienberg
Copy link
Contributor Author

/snapit

@billfienberg
Copy link
Contributor Author

/snapit Status Update

Tested /snapit with OIDC authentication. The snapit action changes are working (PR Shopify/snapit#52), but npm publish failed with:

npm notice Access token expired or revoked. Please try logging in again.
npm error 404 Not Found - PUT https://registry.npmjs.org/@shopify%2fui-extensions

Root cause: The @shopify/ui-extensions package needs to be configured as a Trusted Publisher on npmjs.com.

Action needed: Someone with npm admin access needs to:

  1. Go to https://www.npmjs.com/package/@shopify/ui-extensions/access
  2. Add a Trusted Publisher for the Shopify/ui-extensions repo
  3. Configure it to trust the deploy.yml workflow

Once that's done, /snapit should work with OIDC authentication.

Related:

- Rename SearchDataApi to ToolsDataApi
- Rename SearchDataOutput to ToolsDataOutput
- Rename api/search-data directory to api/tools-data
- Update extension target name
@billfienberg billfienberg force-pushed the bf/add-admin-search-data-target branch from 270c8ee to 301c379 Compare January 12, 2026 17:42
@billfienberg
Copy link
Contributor Author

/snapit

@shopify-github-actions-access
Copy link
Contributor

🫰✨ Thanks @billfienberg! Your snapshot has been published to npm.

Test the snapshot by updating your package.json with the newly published version:

"@shopify/ui-extensions": "0.0.0-snapshot-20260112174403"

@billfienberg billfienberg changed the title Add admin.app.search.data extension target Add admin.app.tools.data extension target Jan 13, 2026
billfienberg and others added 3 commits January 13, 2026 14:11
Co-Authored-By: Claude Opus 4.5 <[email protected]>
Remove ToolsDataOutput and ToolResult types as the extension
target now returns undefined per issue requirements.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Remove custom ToolsDataApi in favor of StandardApi.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@billfienberg billfienberg marked this pull request as ready for review January 13, 2026 20:21
@billfienberg
Copy link
Contributor Author

/snapit

@shopify-github-actions-access
Copy link
Contributor

🫰✨ Thanks @billfienberg! Your snapshot has been published to npm.

Test the snapshot by updating your package.json with the newly published version:

"@shopify/ui-extensions": "0.0.0-snapshot-20260113202545"

@billfienberg billfienberg merged commit 2112c03 into 2025-10 Jan 14, 2026
7 checks passed
@billfienberg billfienberg deleted the bf/add-admin-search-data-target branch January 14, 2026 16:15
kbav added a commit that referenced this pull request Jan 16, 2026
Port of #3706 from 2025-10 branch.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
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.

3 participants