Skip to content

fix: replace Record<string, any> with Record<string, unknown> in GraphQL hooks#1288

Open
Mayank Joshi (Lothnic) wants to merge 1 commit into
browseros-ai:mainfrom
Lothnic:fix/graphql-type-safety
Open

fix: replace Record<string, any> with Record<string, unknown> in GraphQL hooks#1288
Mayank Joshi (Lothnic) wants to merge 1 commit into
browseros-ai:mainfrom
Lothnic:fix/graphql-type-safety

Conversation

@Lothnic

@Lothnic Mayank Joshi (Lothnic) commented Jun 19, 2026

Copy link
Copy Markdown

Description:

  • Remove stale TODOs from mediaUrls.ts (URLs already present)
  • Rename ambiguous type params T/V to TResult/TVariables
  • Replace Record<string, any> | undefined with Record<string, unknown> = Record<string, never>
  • Remove no-longer-needed biome-ignore suppressions

CLA Signature

I have read the CLA Document and I hereby sign the CLA

…hQL hooks

- Remove stale TODOs from mediaUrls.ts (URLs already present)
- Rename ambiguous type params T/V to TResult/TVariables
- Replace Record<string, any> | undefined with Record<string, unknown> = Record<string, never>
- Remove no-longer-needed biome-ignore suppressions
@greptile-apps

greptile-apps Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

PR author is not in the allowed authors list.

@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA. Thank you!
Posted by the CLA Assistant Lite bot.

@Lothnic Mayank Joshi (Lothnic) changed the title fix: replace Record<string, any> with Record<string, unknown> in Grap… fix: replace Record<string, any> with Record<string, unknown> in GraphQL hooks Jun 19, 2026
@Lothnic

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

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