⚙ D17072 [ecash-agora] Update linter settings for monorepo consistency

bytesofman created this revision.Nov 4 2024, 17:49

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 4 2024, 17:49

back out the no-use-before-undefined changes, remove ignore-next-line since rule is patched

Harbormaster completed remote builds in B30986: Diff 50645.Nov 4 2024, 17:52

bytesofman edited the test plan for this revision. (Show Details)Nov 4 2024, 17:52

Build Bitcoin ABC Diffs / Diff Testing (cashtab-tests) failed.

Tail of the build log:

   Compiling log v0.4.21
   Compiling cc v1.0.92
   Compiling thiserror v1.0.58
   Compiling wasm-bindgen v0.2.92
   Compiling cfg-if v1.0.0
   Compiling abc-rust-lint v0.1.0 (/work/chronik/abc-rust-lint)
   Compiling generic-array v0.14.7
   Compiling quote v1.0.36
   Compiling syn v2.0.72
   Compiling ecash-secp256k1-sys v0.10.0 (/work/modules/ecash-secp256k1/ecash-secp256k1-sys)
   Compiling block-buffer v0.10.4
   Compiling crypto-common v0.1.6
   Compiling digest v0.10.7
   Compiling sha2 v0.10.8
   Compiling ripemd v0.1.3
   Compiling wasm-bindgen-backend v0.2.92
   Compiling ecash-secp256k1 v0.30.0 (/work/modules/ecash-secp256k1)
   Compiling thiserror-impl v1.0.58
   Compiling wasm-bindgen-macro-support v0.2.92
   Compiling wasm-bindgen-macro v0.2.92
   Compiling ecash-lib-wasm v0.1.0 (/work/modules/ecash-lib-wasm)
    Finished release-wasm [optimized] target(s) in 5.14s
/work/modules/ecash-lib /work/modules/ecash-lib-wasm /work/modules/ecash-script /work/modules/chronik-client /work/modules/mock-chronik-client /work/modules/ecashaddrjs /work/abc-ci-builds/cashtab-tests

added 363 packages, and audited 366 packages in 2s

60 packages are looking for funding
  run `npm fund` for details

1 moderate severity vulnerability

To address all issues, run:
  npm audit fix

Run `npm audit` for details.

> ecash-lib@0.2.1 build
> tsc && tsc -p ./tsconfig.build.json && cp -r ./src/ffi ./dist

Installing ecash-agora dependencies...
/work/modules/ecash-agora /work/modules/ecash-lib /work/modules/ecash-lib-wasm /work/modules/ecash-script /work/modules/chronik-client /work/modules/mock-chronik-client /work/modules/ecashaddrjs /work/abc-ci-builds/cashtab-tests

added 364 packages, and audited 367 packages in 1s

60 packages are looking for funding
  run `npm fund` for details

1 moderate severity vulnerability

To address all issues, run:
  npm audit fix

Run `npm audit` for details.

> ecash-agora@0.1.1 build
> tsc && tsc -p ./tsconfig.build.json

tests/partial.locktime.test.ts(87,9): error TS2552: Cannot find name 'makerScriptHex'. Did you mean 'makerScript'?
tests/partial.locktime.test.ts(92,9): error TS2552: Cannot find name 'takerScriptHex'. Did you mean 'takerScript'?
Build cashtab-tests failed with exit code 2

Harbormaster completed remote builds in B30987: Diff 50646.Nov 4 2024, 17:53

Build Bitcoin ABC Diffs / Diff Testing (cashtab-tests) failed.

Tail of the build log:

   Compiling once_cell v1.19.0
   Compiling log v0.4.21
   Compiling wasm-bindgen v0.2.92
   Compiling thiserror v1.0.58
   Compiling cfg-if v1.0.0
   Compiling abc-rust-lint v0.1.0 (/work/chronik/abc-rust-lint)
   Compiling generic-array v0.14.7
   Compiling quote v1.0.36
   Compiling syn v2.0.72
   Compiling ecash-secp256k1-sys v0.10.0 (/work/modules/ecash-secp256k1/ecash-secp256k1-sys)
   Compiling crypto-common v0.1.6
   Compiling block-buffer v0.10.4
   Compiling digest v0.10.7
   Compiling ripemd v0.1.3
   Compiling sha2 v0.10.8
   Compiling wasm-bindgen-backend v0.2.92
   Compiling thiserror-impl v1.0.58
   Compiling ecash-secp256k1 v0.30.0 (/work/modules/ecash-secp256k1)
   Compiling wasm-bindgen-macro-support v0.2.92
   Compiling wasm-bindgen-macro v0.2.92
   Compiling ecash-lib-wasm v0.1.0 (/work/modules/ecash-lib-wasm)
    Finished release-wasm [optimized] target(s) in 4.75s
/work/modules/ecash-lib /work/modules/ecash-lib-wasm /work/modules/ecash-script /work/modules/chronik-client /work/modules/mock-chronik-client /work/modules/ecashaddrjs /work/abc-ci-builds/cashtab-tests

added 363 packages, and audited 366 packages in 2s

60 packages are looking for funding
  run `npm fund` for details

1 moderate severity vulnerability

To address all issues, run:
  npm audit fix

Run `npm audit` for details.

> ecash-lib@0.2.1 build
> tsc && tsc -p ./tsconfig.build.json && cp -r ./src/ffi ./dist

Installing ecash-agora dependencies...
/work/modules/ecash-agora /work/modules/ecash-lib /work/modules/ecash-lib-wasm /work/modules/ecash-script /work/modules/chronik-client /work/modules/mock-chronik-client /work/modules/ecashaddrjs /work/abc-ci-builds/cashtab-tests

added 364 packages, and audited 367 packages in 1s

60 packages are looking for funding
  run `npm fund` for details

1 moderate severity vulnerability

To address all issues, run:
  npm audit fix

Run `npm audit` for details.

> ecash-agora@0.1.1 build
> tsc && tsc -p ./tsconfig.build.json

tests/partial.locktime.test.ts(87,9): error TS2552: Cannot find name 'makerScriptHex'. Did you mean 'makerScript'?
tests/partial.locktime.test.ts(92,9): error TS2552: Cannot find name 'takerScriptHex'. Did you mean 'takerScript'?
Build cashtab-tests failed with exit code 2

Build Bitcoin ABC Diffs / Diff Testing (ecash-agora-integration-tests) failed.

Failed tests logs:

====== AgoraPartial enforcedLockTime: "before all" hook for "AgoraPartial enforcedLockTime".AgoraPartial enforcedLockTime "before all" hook for "AgoraPartial enforcedLockTime" ======
ReferenceError: makerScriptHex is not defined
    at Context.<anonymous> (tests/partial.locktime.test.ts:87:9)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Each failure log is accessible here:
AgoraPartial enforcedLockTime: "before all" hook for "AgoraPartial enforcedLockTime".AgoraPartial enforcedLockTime "before all" hook for "AgoraPartial enforcedLockTime"

Build Bitcoin ABC Diffs / Diff Testing (ecash-agora-integration-tests) failed.

Failed tests logs:

====== AgoraPartial enforcedLockTime: "before all" hook for "AgoraPartial enforcedLockTime".AgoraPartial enforcedLockTime "before all" hook for "AgoraPartial enforcedLockTime" ======
ReferenceError: makerScriptHex is not defined
    at Context.<anonymous> (tests/partial.locktime.test.ts:87:9)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Each failure log is accessible here:
AgoraPartial enforcedLockTime: "before all" hook for "AgoraPartial enforcedLockTime".AgoraPartial enforcedLockTime "before all" hook for "AgoraPartial enforcedLockTime"

remove unused calcs

Harbormaster completed remote builds in B30988: Diff 50647.Nov 4 2024, 18:18

bytesofman published this revision for review.Nov 4 2024, 19:01

emack requested changes to this revision.Nov 5 2024, 04:18

emack added a subscriber: emack.

emack added inline comments.

modules/ecash-agora/tests/partial.locktime.test.ts
22 ↗	(On Diff #50647)

This revision now requires changes to proceed.Nov 5 2024, 04:18

fix lingering eslint error

Harbormaster completed remote builds in B30997: Diff 50658.Nov 5 2024, 04:48

emack accepted this revision.Nov 5 2024, 06:23

This revision is now accepted and ready to land.Nov 5 2024, 06:23

Closed by commit rABC753bbb9ff679: [ecash-agora] Update linter settings for monorepo consistency (authored by bytesofman). · Explain WhyNov 5 2024, 17:45

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC753bbb9ff679: [ecash-agora] Update linter settings for monorepo consistency.

[ecash-agora] Update linter settings for monorepo consistency
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 50671

modules/ecash-agora/.eslintrc.js

modules/ecash-agora/README.md

modules/ecash-agora/eslint.config.js

modules/ecash-agora/package-lock.json

modules/ecash-agora/package.json

modules/ecash-agora/src/agora.ts

modules/ecash-agora/tests/partial.locktime.test.ts

modules/ecash-agora/tests/partial.slp.bigsats.test.ts

[ecash-agora] Update linter settings for monorepo consistencyClosedPublicActions