Skip to content

Commit 7c316b0

Browse files
shrey150claude
andcommitted
Remove unused destructured vars without eslint-disable
Use delete instead of underscore-prefixed destructuring to exclude properties from rest spreads, avoiding no-unused-vars lint errors. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent db2ef63 commit 7c316b0

2 files changed

Lines changed: 4 additions & 5 deletions

File tree

packages/core/lib/v3/providerConfig.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -141,8 +141,8 @@ export function getProviderConstructorOptions(
141141
return undefined;
142142
}
143143

144-
// eslint-disable-next-line @typescript-eslint/no-unused-vars
145-
const { providerConfig, provider: _provider, ...rest } = normalizedOptions;
144+
const { providerConfig, ...rest } = normalizedOptions;
145+
delete (rest as Record<string, unknown>).provider;
146146

147147
if (providerConfig?.provider === "bedrock") {
148148
return {

packages/server-v3/src/lib/InMemorySessionStore.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -209,9 +209,8 @@ export class InMemorySessionStore implements SessionStore {
209209
typeof ctx.modelConfig?.modelName === "string"
210210
? ctx.modelConfig.modelName
211211
: undefined;
212-
// eslint-disable-next-line @typescript-eslint/no-unused-vars
213-
const { modelName: _requestModelName, ...requestModelClientOptions } =
214-
ctx.modelConfig ?? {};
212+
const { ...requestModelClientOptions } = ctx.modelConfig ?? {};
213+
delete (requestModelClientOptions as Record<string, unknown>).modelName;
215214
const modelClientOptions = {
216215
...(params.modelClientOptions ?? {}),
217216
...requestModelClientOptions,

0 commit comments

Comments
 (0)