Bind all network helpers

[alcuadrado]

Previously this would fail

const {networkHelpers: {mine}} = await network.connect();

[changeset-bot]

🦋 Changeset detected

Latest commit: 76ab773

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 21 packages

Name	Type
@nomicfoundation/hardhat-network-helpers	Patch
@nomicfoundation/hardhat-utils	Patch
hardhat	Patch
@nomicfoundation/hardhat-errors	Patch
@nomicfoundation/hardhat-ethers-chai-matchers	Patch
@nomicfoundation/hardhat-ignition	Patch
@nomicfoundation/ignition-core	Patch
@nomicfoundation/hardhat-ignition-ethers	Patch
@nomicfoundation/ignition-ui	Patch
@nomicfoundation/hardhat-ignition-viem	Patch
@nomicfoundation/hardhat-keystore	Patch
@nomicfoundation/hardhat-mocha	Patch
@nomicfoundation/hardhat-node-test-reporter	Patch
@nomicfoundation/hardhat-node-test-runner	Patch
@nomicfoundation/hardhat-test-utils	Patch
@nomicfoundation/hardhat-typechain	Patch
@nomicfoundation/hardhat-toolbox-mocha-ethers	Patch
@nomicfoundation/hardhat-verify	Patch
@nomicfoundation/hardhat-viem	Patch
@nomicfoundation/hardhat-viem-assertions	Patch
@nomicfoundation/hardhat-zod-utils	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[Copilot]

Pull Request Overview

This PR introduces a bindAllMethods utility function to solve a limitation where network helper methods would fail when destructured or assigned to independent variables. The fix ensures that methods from NetworkHelpers and Time classes maintain their this context when called independently.

• Adds a new bindAllMethods utility function that binds all methods of an object to preserve this context
• Updates NetworkHelpers and Time classes to use the binding utility in their constructors
• Adds comprehensive test coverage for the new utility function and verifies the fix works for destructured methods

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
v-next/hardhat-utils/src/lang.ts	Implements the bindAllMethods utility function with proper method filtering
v-next/hardhat-utils/test/lang.ts	Adds comprehensive test coverage for the new utility function
v-next/hardhat-network-helpers/src/internal/network-helpers/network-helpers.ts	Applies method binding to NetworkHelpers class
v-next/hardhat-network-helpers/src/internal/network-helpers/time/time.ts	Applies method binding to Time class
v-next/hardhat-network-helpers/test/index.ts	Adds integration test verifying destructured methods work correctly
.changeset/wild-rivers-float.md	Documents the addition of bindAllMethods utility
.changeset/gentle-planets-travel.md	Documents the network helpers enhancement

[Copilot]

The condition should also check if the property is writable to avoid attempting to bind read-only methods. Consider adding Object.getOwnPropertyDescriptor(obj, key)?.writable !== false to the condition.

[Copilot]

This code will fail with a TypeError if obj is null or has a null prototype (like objects created with Object.create(null)). Consider adding a null check: Object.getPrototypeOf(obj) && Object.getOwnPropertyNames(Object.getPrototypeOf(obj)).

Suggested change

	const keys = [
	...Object.getOwnPropertyNames(Object.getPrototypeOf(obj)),
	const proto = Object.getPrototypeOf(obj);
	const keys = [
	...(proto ? Object.getOwnPropertyNames(proto) : []),