fix: ignore datasetorder burned TEE frameworks tags from deal tag computation

[PierreJeanjacquot]

No description provided.

[github-actions]

🐳 Hadolint Dockerfile Lint Results

Click to expand detailed results

Dockerfile:17 DL3016 warning: Pin versions in npm. Instead of `npm install <package>` use `npm install <package>@<version>`

[Copilot]

Pull request overview

This PR implements a fix to ignore TEE framework tags from dataset orders when computing deal tags, aligning the SDK with blockchain behavior where dataset TEE framework tags are "burned" (ignored).

Changes:

• Added stripTeeFrameworkFromTag utility function to clear TEE framework bits (bits 1, 2, 3) from tags
• Updated tag computation logic to strip TEE framework bits from dataset tags before summing with app and request tags
• Refactored tag encoding/decoding functions to use native BigInt instead of BN.js for cleaner, more modern code

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
src/common/utils/utils.js	Added stripTeeFrameworkFromTag function and refactored tag utilities to use BigInt
src/common/market/order.js	Updated tag validation logic to strip TEE frameworks from dataset tags
src/lib/IExecOrderModule.js	Applied dataset tag stripping in order matching preflight checks
src/cli/cmd/iexec-order.js	Applied dataset tag stripping in CLI order fill command
src/cli/cmd/iexec-app.js	Applied dataset tag stripping when computing minimum required tags
test/lib/unit/utils.test.js	Added unit test for stripTeeFrameworkFromTag function
test/lib/e2e/IExecOrderModule.test.js	Added e2e test demonstrating burned tag behavior and reformatted existing tests

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.