Remove FullStackNetworkProvider and BitcoinRpcNetworkProvider

Author: rkalis

AGENTS.md

@@ -71,7 +71,7 @@ The compiler uses the **visitor pattern** throughout — AST nodes accept traver
 
 - `Contract` — loads compiled artifacts, instantiates with constructor args
 - `TransactionBuilder` — builds and signs Bitcoin Cash transactions
-- Network providers: `ElectrumNetworkProvider`, `BitcoinRpcNetworkProvider`, `MockNetworkProvider`, `FullStackNetworkProvider`
+- Network providers: `ElectrumNetworkProvider`, `MockNetworkProvider`
 - `src/libauth-template/` — integrates with `@bitauth/libauth` for transaction evaluation and debugging
 
 ### Utils (`@cashscript/utils`)

packages/cashscript/package.json

@@ -44,12 +44,10 @@
     "@bitauth/libauth": "^3.1.0-next.8",
     "@cashscript/utils": "^0.13.0-next.6",
     "@electrum-cash/network": "^4.1.3",
-    "@mr-zwets/bchn-api-wrapper": "^1.0.1",
     "fflate": "^0.8.2",
     "semver": "^7.7.2"
   },
   "devDependencies": {
-    "@psf/bch-js": "^6.8.0",
     "@types/semver": "^7.5.8",
     "@vitest/coverage-v8": "^4.0.15",
     "eslint": "^8.54.0",

packages/cashscript/src/external-types/@psf/bch-js.d.ts

@@ -15,9 +15,7 @@ export * from './Errors.js';
 export * from './network/errors.js';
 export {
   type NetworkProvider,
-  BitcoinRpcNetworkProvider,
   ElectrumNetworkProvider,
-  FullStackNetworkProvider,
   MockNetworkProvider,
 } from './network/index.js';
 export { randomUtxo, randomToken, randomNFT } from './utils.js';

packages/cashscript/src/index.ts

@@ -1,5 +1,3 @@
 export type { default as NetworkProvider } from './NetworkProvider.js';
-export { default as BitcoinRpcNetworkProvider } from './BitcoinRpcNetworkProvider.js';
 export { default as ElectrumNetworkProvider } from './ElectrumNetworkProvider.js';
-export { default as FullStackNetworkProvider } from './FullStackNetworkProvider.js';
 export { default as MockNetworkProvider } from './MockNetworkProvider.js';

packages/cashscript/src/network/BitcoinRpcNetworkProvider.ts

@@ -87,6 +87,18 @@ If you are using a custom network provider, you will need to update the code for
   getUtxosForLockingBytecode(lockingBytecode: Uint8Array | string): Promise<Utxo[]>;
 ```
 
+### FullStackNetworkProvider & BitcoinRpcNetworkProvider
+
+The `FullStackNetworkProvider` and `BitcoinRpcNetworkProvider` have been removed from the SDK. If you were using these providers, you will need to update your code to use a different network provider.
+
+```ts
+// Before: FullStackNetworkProvider
+const provider = new FullStackNetworkProvider('mainnet', bchjs);
+
+// After: ElectrumNetworkProvider
+const provider = new ElectrumNetworkProvider('mainnet');
+```
+
 ## v0.11 to v0.12
 
 There are several breaking changes to the SDK in this release.

packages/cashscript/src/network/FullStackNetworkProvider.ts

@@ -27,6 +27,7 @@ This release contains several breaking changes, please refer to the [migration n
 - :sparkles: Add `getUtxosForLockingBytecode()` method to `ElectrumNetworkProvider` class and `MockNetworkProvider` interface.
 - :sparkles: In the `MockNetworkProvider`, `addUtxo()` now also allows UTXOs to be added by locking bytecode.
 - :sparkles: Add `gatherBchUtxos()` and `gatherFungibleTokenUtxos()` functions to the SDK for gathering UTXOs.
+- :boom: **BREAKING**: Remove `BitcoinRpcNetworkProvider` and `FullStackNetworkProvider` from the SDK.
 - :hammer_and_wrench: **BREAKING**: Rename `addressType` option on `Contract` constructor to `contractType`.
 - :hammer_and_wrench: **BREAKING**: Remove undocumented `redeemScript` property from `Contract` class.
 - :hammer_and_wrench: **BREAKING**: Remove undocumented `buildLibauthTransaction()` method from `TransactionBuilder` class.