Conversation
| * @returns Sheet shares data | ||
| */ | ||
| async getSheetShares( | ||
| sheetId: string, |
There was a problem hiding this comment.
my assumption is that most users will not
- switch a sheet to Grid
- click file -> properties
- copy "Sheet ID"
to ask the AI
There is an undocumented public API I added a couple years ago to the API get sheet endpoint by the "DirectId"
Could you create another tool for this...maybe use similar logic as the toolsearch_in_sheet_by_url
https://github.com/smartsheet-platform/smar-mcp/blob/main/src/tools/smartsheet-search-tools.ts#L86
Could also be an opportunity to dry up the implementation
There was a problem hiding this comment.
may be more related to adding a tool for get_sheet_shares_by_url
https://github.com/smartsheet-platform/smar-mcp/pull/35/files#diff-64f0e3433efa66c925b105d3a627ac1f453a855b6013e1c13988d82b3ee459d2R471
There was a problem hiding this comment.
Pushed - I left this as a separate tool for convenience. It's possible we may be able to dry it up further but I don't think keeping these separate (especially given this is undocumented) is a bad thing.
There was a problem hiding this comment.
There is an undocumented public API I added a couple years ago to the API get sheet endpoint by the "DirectId"
Is there a way we can document it? Jim could help us get it in?
Adds
getSheetShares()method tosmartsheet-sheet-api.ts.Adds corresponding tool in
smartsheet-sheet-tools.tsto fetch all sheet shares.Also adds
build-windowstopackage.json.