feat: add ability to remove work without slash and ejection

[JannikSt]

No description provided.

[manveerxyz]

just one comment - otherwise looks good!

[Copilot]

Pull Request Overview

Adds soft-slash functionality to remove work units without full ejection, updating interfaces, implementations, and tests to support a new removeWork API.

• Extend IWorkValidation, IComputePool, and IRewardsDistributor with soft-invalidation/remove APIs
• Implement softInvalidateWork in the validator and wire through ComputePool, PrimeNetwork, and the ring-buffer RewardsDistributorWorkSubmission
• Add comprehensive tests for removeWork across single/multiple buckets, zero/excess removal, and multi-node scenarios

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
test/RewardsDistributorWorkSubmission.t.sol	Added new tests for removeWork edge cases and multi-node behavior
src/interfaces/IWorkValidation.sol	Added softInvalidateWork signature
src/interfaces/IComputePool.sol	Exposed softInvalidateWork in pool interface
src/interfaces/IRewardsDistributor.sol	Introduced removeWork API for soft-slash
src/SyntheticDataWorkValidator.sol	Implemented softInvalidateWork and WorkRemoved event
src/ComputePool.sol	Delegates softInvalidateWork to validator and calls removeWork
src/PrimeNetwork.sol	Added passthrough softInvalidateWork
src/RewardsDistributorWorkSubmission.sol	Added removeWork implementation in ring-buffer distributor
src/RewardsDistributorFixed.sol	Stubbed removeWork (no-op) in fixed distributor
src/RewardsDistributor.sol	Stubbed removeWork (no-op) in base distributor

Comments suppressed due to low confidence (2)

src/SyntheticDataWorkValidator.sol:11

• [nitpick] The event name WorkRemoved may be confused with the hard invalidation case; consider renaming it to WorkSoftInvalidated or similar for clarity.

event WorkRemoved(uint256 poolId, address provider, address nodeId, bytes32 workKey, uint256 workUnits);

src/interfaces/IRewardsDistributor.sol:24

• Add NatSpec comments describing the behavior, permission requirements, and effects of removeWork to keep the interface documentation up to date.

function removeWork(address node, uint256 workUnits) external;