Skip to content

Commit 3f33dc4

Browse files
committed
Remove borrow injection from translated runtimes
1 parent 5dcd625 commit 3f33dc4

16 files changed

Lines changed: 9 additions & 626 deletions

go/modcdp/client/ModCDPClient.go

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,6 @@ type RouterConfig = types.ModCDPRouterConfig
7878
type ExtensionInjector = injector.ExtensionInjector
7979
type DiscoverExtensionInjector = injector.DiscoverExtensionInjector
8080
type BBExtensionInjector = injector.BBExtensionInjector
81-
type BorrowExtensionInjector = injector.BorrowExtensionInjector
8281
type CLIExtensionInjector = injector.CLIExtensionInjector
8382
type CDPExtensionInjector = injector.CDPExtensionInjector
8483
type UpstreamMode = transportpkg.UpstreamMode
@@ -96,7 +95,6 @@ var NewBBBrowserLauncher = launcher.NewBBBrowserLauncher
9695
var NewNoneBrowserLauncher = launcher.NewNoneBrowserLauncher
9796
var NewDiscoverExtensionInjector = injector.NewDiscoverExtensionInjector
9897
var NewBBExtensionInjector = injector.NewBBExtensionInjector
99-
var NewBorrowExtensionInjector = injector.NewBorrowExtensionInjector
10098
var NewCLIExtensionInjector = injector.NewCLIExtensionInjector
10199
var NewCDPExtensionInjector = injector.NewCDPExtensionInjector
102100
var NewWSUpstreamTransport = transportpkg.NewWSUpstreamTransport
@@ -1132,10 +1130,6 @@ func (c *ModCDPClient) extensionInjectorsForConfig() ([]extensionInjector, *Exte
11321130
injector := NewBBExtensionInjector(InjectorConfig{})
11331131
return []extensionInjector{&injector}, &injector.ExtensionInjector
11341132
}
1135-
if c.Config.Injector.InjectorMode == "borrow" {
1136-
injector := NewBorrowExtensionInjector(InjectorConfig{})
1137-
return []extensionInjector{&injector}, &injector.ExtensionInjector
1138-
}
11391133
if c.Config.Injector.InjectorMode == "discover" {
11401134
injector := NewDiscoverExtensionInjector(InjectorConfig{})
11411135
return []extensionInjector{&injector}, &injector.ExtensionInjector
@@ -1152,7 +1146,7 @@ func isKnownUpstreamMode(mode string) bool {
11521146
}
11531147

11541148
func isKnownExtensionMode(mode string) bool {
1155-
return mode == "cli" || mode == "cdp" || mode == "bb" || mode == "borrow" || mode == "discover" || mode == "none"
1149+
return mode == "cli" || mode == "cdp" || mode == "bb" || mode == "discover" || mode == "none"
11561150
}
11571151

11581152
func (c *ModCDPClient) baseInjectorConfig(send SendCDP) InjectorConfig {
@@ -1165,7 +1159,6 @@ func (c *ModCDPClient) baseInjectorConfig(send SendCDP) InjectorConfig {
11651159
InjectorCDPExtensionID: c.Config.Injector.InjectorCDPExtensionID,
11661160
InjectorBBExtensionPath: c.Config.Injector.InjectorBBExtensionPath,
11671161
InjectorBBExtensionID: c.Config.Injector.InjectorBBExtensionID,
1168-
InjectorBorrowExtensionPath: c.Config.Injector.InjectorBorrowExtensionPath,
11691162
InjectorDiscoverExtensionPath: c.Config.Injector.InjectorDiscoverExtensionPath,
11701163
InjectorServiceWorkerExtensionID: c.Config.Injector.InjectorServiceWorkerExtensionID,
11711164
InjectorServiceWorkerURLIncludes: c.Config.Injector.InjectorServiceWorkerURLIncludes,

go/modcdp/injector/BorrowExtensionInjector.go

Lines changed: 0 additions & 227 deletions
This file was deleted.

go/modcdp/injector/BorrowExtensionInjector_test.go

Lines changed: 0 additions & 67 deletions
This file was deleted.

go/modcdp/injector/ExtensionInjector.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -94,9 +94,6 @@ func (i *ExtensionInjector) Update(config InjectorConfig) *ExtensionInjector {
9494
if config.InjectorBBExtensionID != "" {
9595
i.Config.InjectorBBExtensionID = config.InjectorBBExtensionID
9696
}
97-
if config.InjectorBorrowExtensionPath != "" {
98-
i.Config.InjectorBorrowExtensionPath = config.InjectorBorrowExtensionPath
99-
}
10097
if config.InjectorDiscoverExtensionPath != "" {
10198
i.Config.InjectorDiscoverExtensionPath = config.InjectorDiscoverExtensionPath
10299
}

go/modcdp/modcdp.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ type ExtensionInjector = injector.ExtensionInjector
3535
type PreparedExtension = injector.PreparedExtension
3636
type DiscoverExtensionInjector = injector.DiscoverExtensionInjector
3737
type BBExtensionInjector = injector.BBExtensionInjector
38-
type BorrowExtensionInjector = injector.BorrowExtensionInjector
3938
type CLIExtensionInjector = injector.CLIExtensionInjector
4039
type CDPExtensionInjector = injector.CDPExtensionInjector
4140
type UpstreamMode = transport.UpstreamMode
@@ -97,7 +96,6 @@ var ResolveCdpWebSocketUrl = launcher.WebsocketURLFor
9796
var NewExtensionInjector = injector.NewExtensionInjector
9897
var NewDiscoverExtensionInjector = injector.NewDiscoverExtensionInjector
9998
var NewBBExtensionInjector = injector.NewBBExtensionInjector
100-
var NewBorrowExtensionInjector = injector.NewBorrowExtensionInjector
10199
var NewCLIExtensionInjector = injector.NewCLIExtensionInjector
102100
var NewCDPExtensionInjector = injector.NewCDPExtensionInjector
103101
var DefaultModCDPExtensionPath = injector.DefaultModCDPExtensionPath

go/modcdp/modcdp_test.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,9 @@ func TestRootExportsConcreteLaunchersInjectorsAndTransports(t *testing.T) {
2424
extensionInjector := NewExtensionInjector(InjectorConfig{})
2525
discoveredInjector := NewDiscoverExtensionInjector(InjectorConfig{})
2626
bbInjector := NewBBExtensionInjector(InjectorConfig{})
27-
borrowInjector := NewBorrowExtensionInjector(InjectorConfig{})
2827
localLaunchInjector := NewCLIExtensionInjector(InjectorConfig{})
2928
loadUnpackedInjector := NewCDPExtensionInjector(InjectorConfig{})
30-
_ = []any{extensionInjector, discoveredInjector, bbInjector, borrowInjector, localLaunchInjector, loadUnpackedInjector}
29+
_ = []any{extensionInjector, discoveredInjector, bbInjector, localLaunchInjector, loadUnpackedInjector}
3130

3231
if NewUpstreamTransport(UpstreamTransportConfig{}).Config.UpstreamMode != string(UpstreamModeWS) {
3332
t.Fatal("NewUpstreamTransport did not hydrate default ws mode")

go/modcdp/types/types.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,6 @@ type InjectorConfig struct {
6868
InjectorCDPExtensionID string `json:"injector_cdp_extension_id,omitempty"`
6969
InjectorBBExtensionPath string `json:"injector_bb_extension_path,omitempty"`
7070
InjectorBBExtensionID string `json:"injector_bb_extension_id,omitempty"`
71-
InjectorBorrowExtensionPath string `json:"injector_borrow_extension_path,omitempty"`
7271
InjectorDiscoverExtensionPath string `json:"injector_discover_extension_path,omitempty"`
7372
InjectorServiceWorkerExtensionID string `json:"injector_service_worker_extension_id,omitempty"`
7473
InjectorServiceWorkerURLIncludes []string `json:"injector_service_worker_url_includes,omitempty"`

js/src/injector/BorrowExtensionInjector.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
1-
// MODCDP_TRANSLATE: KEEP THIS FILE TRANSLATED ACROSS TYPESCRIPT, PYTHON, AND GO.
2-
// Keep all shapes, signatures, behavior, and tests 1:1 in sync with:
3-
// - ./python/modcdp/injector/BorrowExtensionInjector.py
4-
// - ./go/modcdp/injector/BorrowExtensionInjector.go
1+
// MODCDP_TS_ONLY: DO NOT TRANSLATE THIS FILE TO OTHER LANGUAGES.
2+
// Reason: not needed by Stagehand.
53
import fs from "node:fs";
64
import path from "node:path";
75
import * as Runtime from "../types/generated/zod/Runtime.js";

0 commit comments

Comments
 (0)