Skip to content

feat: Migrate to hey-api#204

Merged
hhvrc merged 8 commits into
developfrom
feat/hey-api
May 19, 2026
Merged

feat: Migrate to hey-api#204
hhvrc merged 8 commits into
developfrom
feat/hey-api

Conversation

@hhvrc
Copy link
Copy Markdown
Contributor

@hhvrc hhvrc commented May 19, 2026

No description provided.

@hhvrc hhvrc self-assigned this May 19, 2026
Copilot AI review requested due to automatic review settings May 19, 2026 08:22
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

hhvrc added 2 commits May 19, 2026 13:32
# Conflicts:
#	package.json
#	pnpm-lock.yaml
#	src/lib/components/Table/ColumnUtils.ts
#	src/routes/(app)/hubs/[hubId=guid]/update/+page.svelte
#	src/routes/(app)/settings/api-tokens/dialog-token-create.svelte
#	src/routes/(app)/shares/public/dialog-publicshare-create.svelte
#	src/routes/(meta)/sitemap.xml/+server.ts
#	src/routes/terminal/SerialTerminal.svelte
@hhvrc hhvrc requested a review from Copilot May 19, 2026 12:01
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

- Fix RateLimitError.name ('ResponseError' → 'RateLimitError'); widen
  base ResponseError.name to string so the subclass override type-checks
- Remove unreachable null guards for non-nullable envelope data fields
  (LcgResponse.data, StatsResponse.data); keep guards where data is
  genuinely nullable (DeviceWithShockersResponse, OwnerShockerResponse)
- Simplify nullable-data guards to drop unused message interpolation
- Restore DateTimePicker in ExpirationPicker (replaces bare datetime-local
  input); convert ZonedDateTime → Temporal.Instant inline without the
  removed instantFromDate helper
- Extend Date→Temporal.Instant regex in regenerate-api.js to cover
  Date[] and Array<Date> patterns (add [ and > to lookahead set)
- Fix '$lib/api/index.js' import path → '$lib/api' in shockers/logs page
- Restore missing blank lines after import blocks in comparers and
  backend-metadata-state
@hhvrc hhvrc merged commit d8ace40 into develop May 19, 2026
11 checks passed
@hhvrc hhvrc deleted the feat/hey-api branch May 19, 2026 13:06
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.

2 participants