Skip to content

Commit bf66243

Browse files
committed
chore: add prepare script to package.json and clarify operation type in transaction creation
1 parent b55b464 commit bf66243

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

safe-transaction/package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@
44
"description": "GitHub Action for proposing Safe transactions",
55
"main": "dist/index.js",
66
"scripts": {
7-
"build": "ncc build src/index.js -o dist"
7+
"build": "ncc build src/index.js -o dist",
8+
"prepare": "npm run build"
89
},
910
"dependencies": {
1011
"@actions/core": "^1.10.1",

safe-transaction/src/index.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@ async function run() {
1515
const chainId = BigInt(process.env.INPUT_CHAIN_ID || "42161");
1616
const transactionValue = process.env.INPUT_TRANSACTION_VALUE || "0";
1717
const transactionData = process.env.INPUT_TRANSACTION_DATA || "0x";
18-
// Hardcoded to 0 (Call operation) - DelegateCall (1) is not supported for security reasons
19-
const operation = 0;
2018

2119
core.info(`🚀 Starting Safe transaction creation...`);
2220
core.info(`📍 Safe Address: ${safeAddress}`);
@@ -44,7 +42,7 @@ async function run() {
4442
to: transactionTargetAddress,
4543
value: transactionValue,
4644
data: transactionData,
47-
operation: parseInt(operation),
45+
operation: OperationType.Call, // Hardcoded Call operation - DelegateCall is not supported for security reasons
4846
};
4947

5048
core.info("📝 Creating Safe transaction...");

0 commit comments

Comments
 (0)