Cleaner useApi hook, remove method name hack

[david-crespo]

Closes #209

Slight change to the calling API for useApiData:

- const { data } = useApiData(api.apiProjectInstancesGet, { projectName })
+ const { data } = useApi('apiProjectInstancesGet', { projectName })

No difference in type safety. data still gets the right type. But we get rid of two different hacks:

• not minifying function names, which made the bundle bigger for no reason
• pre-binding all the methods on api to api, which made me want to cry

Plus we don't have to import or call api either because the hook wraps it!

Next

One quirk right now is that for empty params objects, you need to pass an empty object, which feels silly.

const { data: projects } = useApiData(api, 'apiProjectsGet', {})

On the other hand there won't like be many of those because we'll be passing the optional page param pretty much all the time.

export interface ApiProjectsGetRequest {
  limit?: number
  pageToken?: string
  sortBy?: ApiNameOrIdSortMode
}

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/oxidecomputer/console-ui-storybook/7GNsbrqkfBcfvpevZ25zK5FTYQqA
✅ Preview: https://console-ui-storybook-git-fix-api-data-type-oxidecomputer.vercel.app

[github-actions]

Preview will be deployed at https://console-git-fix-api-data-type.internal.oxide.computer

[david-crespo]

good riddance. blech