Skip to content

Commit ffcc4ea

Browse files
Stephen Belangerclaude
andcommitted
fix(api-compat+lint): remove mastra from BraintrustPluginConfig, drop unused types
- Remove mastra from BraintrustPluginConfig inline type (api-compat flags any change to the inline type string; use getIntegrationConfig dynamic lookup instead, same pattern as groq in other integration PRs) - Remove unused IntegrationName/IntegrationMap type aliases (no longer referenced after widening getDefaultConfig/readEnvConfig return types) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent eecd0c7 commit ffcc4ea

2 files changed

Lines changed: 1 addition & 5 deletions

File tree

js/src/instrumentation/braintrust-plugin.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ export interface BraintrustPluginConfig {
3333
googleADK?: boolean;
3434
cohere?: boolean;
3535
groq?: boolean;
36-
mastra?: boolean;
3736
};
3837
}
3938

@@ -164,7 +163,7 @@ export class BraintrustPlugin extends BasePlugin {
164163
this.gitHubCopilotPlugin.enable();
165164
}
166165

167-
if (integrations.mastra !== false) {
166+
if (getIntegrationConfig(integrations, "mastra") !== false) {
168167
this.mastraPlugin = new MastraPlugin();
169168
this.mastraPlugin.enable();
170169
}

js/src/instrumentation/registry.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,6 @@ export interface InstrumentationConfig {
3030
};
3131
}
3232

33-
type IntegrationName = keyof NonNullable<InstrumentationConfig["integrations"]>;
34-
type IntegrationMap = Partial<Record<IntegrationName, boolean>>;
35-
3633
class PluginRegistry {
3734
private braintrustPlugin: BraintrustPlugin | null = null;
3835
private config: InstrumentationConfig = {};

0 commit comments

Comments
 (0)