A decentralized bitcoin exchange network
This report summarizes scanner evidence for bisq-network/bisq, a Java project with 5,152 GitHub stars. SynthScan examined 461,888 lines of code across 3105 source files, recording 112 pattern matches distributed across 10 syntactic categories. The legacy adjusted pattern signal is 0.5 (Low pattern signal); it is not an authorship estimate.
Use the findings to ask whether maintainers can explain intent, verify dependencies, simplify unnecessary scaffolding, and focus review effort. Risk Payload v1 adds ownership, supply-chain, over-engineering, and review-effort vectors when available. These are prompts for human verification, not probabilities, security findings, or measured maintenance costs.
Use each vector to guide manual verification of the underlying findings.
Payload schema: 1.0.0 · Engine: 2.0.0 · Scoring: heuristic-density-v1 · Scan date: 2026-10-31.
Can maintainers explain intent, assumptions, and trade-offs? 103 finding(s).
Are imports declared and package/API names verified? 9 finding(s).
Are wrappers, guards, and repeated structures necessary? 8 finding(s).
Which changes need focused human review? 112 finding(s).
This chart tracks the legacy adjusted pattern score across scan runs. Changes can reflect source changes, scan coverage, or detector/scoring versions; they do not measure AI adoption or prove that a team refactored code. Compare runs only when their scoring versions are compatible. The dashed line tracks the raw pattern-hit count.
Groups findings by configured severity weight. Severity is not a confidence estimate, security rating, or proof that a package is missing. Verify imports against the project's manifests and inspect each finding in its local design context.
This horizontal bar chart shows where configured pattern scores accumulate across directories. Treat the bars as locations to inspect, not as evidence of authorship or code quality. Use the file-level findings to decide whether a directory needs a focused ownership, dependency, complexity, or review check.
The scanner identified 112 distinct pattern matches across 10 syntactic categories. Each entry below represents a discrete location in the source code where the engine recorded a detector match to inspect in context. Expand any category row to inspect the individual file paths, line numbers, code snippets, and the lexical context (CODE, COMMENT, or STRING) in which each match was detected.
Reading the findings table: The Severity column indicates the configured finding weight (CRITICAL / HIGH / MEDIUM / LOW). The Context column identifies whether the match occurred inside executable code, an inline comment, or a string literal — legacy comment-context matches receive a ×1.5 score weight. The ⚡ bolt icon marks clustered matches: three or more patterns within a 10-line window, each receiving an additional ×1.5 density multiplier under the legacy scoring model.
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| CRITICAL | …ain/java/bisq/daemon/grpc/GrpcErrorMessageHandler.java | 137 | return bisq.core.offer.availability.AvailabilityResult.fromProto(proto).getDescription(); | CODE |
| CRITICAL | …/src/test/java/bisq/apitest/dao/ProposalPhaseTest.java | 110 | @org.junit.jupiter.params.provider.ValueSource(strings = { | CODE |
| CRITICAL | …est/java/bisq/apitest/method/wallet/BtcWalletTest.java | 104 | bisq.core.api.model.BtcBalanceInfo.valueOf( | CODE |
| CRITICAL | …st/java/bisq/apitest/method/wallet/WalletTestUtil.java | 16 | bisq.core.api.model.BtcBalanceInfo.valueOf(1000000000, | CODE |
| CRITICAL | …st/java/bisq/apitest/method/wallet/WalletTestUtil.java | 47 | return bisq.core.api.model.BsqBalanceInfo.valueOf(availableBalance, | CODE |
| CRITICAL | …main/java/bisq/network/p2p/storage/P2PDataStorage.java | 1241 | .map(e -> new Tuple2<>(org.bitcoinj.core.Utils.HEX.encode(get32ByteHashAsByteArray(e.getProtectedSto | CODE |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | …/dao/burningman/BurningManPresentationServiceTest.java | 41 | // Target myShare: 0.75 | COMMENT |
| LOW | …/dao/burningman/BurningManPresentationServiceTest.java | 81 | myTargetShare = 0; | COMMENT |
| LOW | …/dao/burningman/BurningManPresentationServiceTest.java | 101 | // Others: 99 | COMMENT |
| LOW | …rc/main/java/bisq/core/btc/TxFeeEstimationService.java | 41 | */ | COMMENT |
| LOW | …rc/main/java/bisq/core/btc/TxFeeEstimationService.java | 161 | // We start with the initialEstimatedTxVsize for a tx with 1 input (175) vbytes and get from BitcoinJ a tx back whic | COMMENT |
| LOW | …re/src/main/java/bisq/core/btc/setup/WalletConfig.java | 321 | } else if (chainFileExists) { | COMMENT |
| LOW | …c/main/java/bisq/core/btc/wallet/BtcWalletService.java | 301 | // Blind vote tx | COMMENT |
| LOW | …c/main/java/bisq/core/btc/wallet/BtcWalletService.java | 421 | // Vote reveal tx | COMMENT |
| LOW | …c/main/java/bisq/core/btc/wallet/BtcWalletService.java | 441 | COMMENT | |
| LOW | …c/main/java/bisq/core/btc/wallet/BtcWalletService.java | 481 | // preparedBsqTx has following structure: | COMMENT |
| LOW | …main/java/bisq/core/btc/wallet/TradeWalletService.java | 221 | // preparedBsqTx has following structure: | COMMENT |
| LOW | …rc/main/java/bisq/core/offer/bisq_v1/OfferPayload.java | 61 | // Instance fields | COMMENT |
| LOW | …/main/java/bisq/core/offer/bisq_v1/TakeOfferModel.java | 141 | @Override | COMMENT |
| LOW | …va/bisq/core/trade/protocol/bisq_v1/BuyerProtocol.java | 61 | // the trade without any resend attempts (buyer stuck at "Please send confirmation | COMMENT |
| LOW | …ler/SellerSendsDepositTxAndDelayedPayoutTxMessage.java | 101 | COMMENT | |
| LOW | …main/java/bisq/core/payment/payload/PaymentMethod.java | 321 | COMMENT | |
| LOW | …n/java/bisq/core/dao/burningman/BurnTargetService.java | 181 | .filter(reimbursementModel -> reimbursementModel.getHeight() <= chainHeight) | COMMENT |
| LOW | …src/main/java/bisq/core/dao/state/DaoStateService.java | 821 | COMMENT | |
| LOW | …e/src/main/java/bisq/core/dao/state/GenesisTxInfo.java | 81 | @Getter | COMMENT |
| LOW | …/java/bisq/core/dao/state/DaoStateSnapshotService.java | 181 | // and we got called back from daoStateMonitoringService once the hashes are created from peers data. | COMMENT |
| LOW | …e/dao/governance/blindvote/MyBlindVoteListService.java | 401 | return bsqWalletService.signTxAndVerifyNoDustOutputs(txWithBtcFee); | COMMENT |
| LOW | …java/bisq/core/dao/governance/period/CycleService.java | 101 | return blockHeight >= cycle.getHeightOfFirstBlock() && | COMMENT |
| LOW | …rc/main/java/bisq/core/dao/governance/param/Param.java | 121 | // We re-purpose it as value for BTC fee revenue per cycle. This can be added as oracle data by DAO voting. | COMMENT |
| LOW | …/core/dao/governance/votereveal/VoteRevealService.java | 161 | /////////////////////////////////////////////////////////////////////////////////////////// | COMMENT |
| LOW | …q/core/dao/node/messages/NewBlockBroadcastMessage.java | 81 | COMMENT | |
| LOW | …ain/java/bisq/core/dao/node/parser/TxOutputParser.java | 301 | case ASSET_LISTING_FEE: | COMMENT |
| LOW | …e/src/main/java/bisq/core/provider/fee/FeeService.java | 161 | } | COMMENT |
| LOW | core/src/main/java/bisq/core/api/CoreOffersService.java | 361 | String upperCaseCurrencyCode = currencyCode.toUpperCase(); | COMMENT |
| LOW | core/src/main/java/bisq/core/api/CoreHelpService.java | 81 | // out.println(coreHelpService.getMethodHelp("lockwallet")); | COMMENT |
| LOW | …/java/bisq/core/account/sign/SignedWitnessService.java | 661 | if (set.isEmpty()) { | COMMENT |
| LOW | …in/java/bisq/desktop/util/validation/SbpValidator.java | 21 | COMMENT | |
| LOW | …ain/java/bisq/desktop/components/BalanceTextField.java | 81 | /////////////////////////////////////////////////////////////////////////////////////////// | COMMENT |
| LOW | …main/java/bisq/desktop/components/chart/ChartView.java | 721 | // Chart update after data change | COMMENT |
| LOW | …p/main/offer/bisq_v1/takeoffer/TakeOfferDataModel.java | 221 | COMMENT | |
| LOW | …p/main/offer/bisq_v1/takeoffer/TakeOfferDataModel.java | 361 | // Another edge case not handled correctly is: If there are many small inputs and user add a large input later the | COMMENT |
| LOW | …mon/src/test/java/bisq/common/crypto/EquihashTest.java | 141 | public void solutionCountPerNonceStats() { | COMMENT |
| LOW | apitest/docker/dao-compose.compat.yml | 1 | # Compatibility override for dao-compose.yml. Points one peer (bob) at the | COMMENT |
| LOW | apitest/docker/run-e2e-tests-compat.sh | 1 | #!/usr/bin/env bash | COMMENT |
| LOW | apitest/docker/run-e2e-tests.sh | 1 | #!/usr/bin/env bash | COMMENT |
| LOW | apitest/docker/run-e2e-tests.sh | 21 | # READY_TIMEOUT_SECS per-daemon readiness wait (default: 300) | COMMENT |
| LOW | apitest/docker/run-e2e-tests.sh | 81 | LOGS_DIR="${SCRIPT_DIR}/logs" | COMMENT |
| LOW | apitest/docker/bitcoind-entrypoint.sh | 1 | #!/bin/sh | COMMENT |
| LOW | apitest/scripts/trade-xmr-simulation.sh | 1 | #! /bin/bash | COMMENT |
| LOW | apitest/scripts/trade-xmr-simulation.sh | 21 | # | COMMENT |
| LOW | apitest/scripts/mainnet-test.sh | 1 | #!/usr/bin/env bats | COMMENT |
| LOW | apitest/scripts/trade-simulation.sh | 1 | #! /bin/bash | COMMENT |
| LOW | apitest/scripts/trade-simulation.sh | 21 | # - Only regtest btc can be bought or sold with the test payment account. | COMMENT |
| LOW | apitest/scripts/limit-order-simulation.sh | 1 | #! /bin/bash | COMMENT |
| LOW | apitest/scripts/limit-order-simulation.sh | 21 | # | COMMENT |
| LOW | apitest/scripts/bsqswap-simulation.sh | 1 | #! /bin/bash | COMMENT |
| LOW | apitest/scripts/bsqswap-simulation.sh | 21 | # | COMMENT |
| LOW | apitest/scripts/rolling-offer-simulation.sh | 1 | #! /bin/bash | COMMENT |
| LOW | apitest/scripts/rolling-offer-simulation.sh | 21 | # These should be run using the apitest harness. From the root project dir, run: | COMMENT |
| LOW | …itest/src/main/java/bisq/apitest/linux/BitcoinCli.java | 81 | if (isError()) | COMMENT |
| LOW | …/test/java/bisq/network/http/UrlSafetyCheckerTest.java | 201 | // P2P, an explorer URL pulled from preferences, etc. Goals tested below: | COMMENT |
| LOW | p2p/src/test/java/bisq/network/p2p/DummySeedNode.java | 61 | private final String defaultUserDataDir; | COMMENT |
| LOW | p2p/src/main/java/bisq/network/http/HttpClientImpl.java | 361 | // Register socket factories that pass the unresolved hostname through to | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| MEDIUM | …a/bisq/core/account/sign/SignedWitnessServiceTest.java | 845 | // sw1 also signs sw3 (not supported yet but a possible addition for a more robust system) | COMMENT |
| MEDIUM | apitest/docker/run-e2e-tests-compat.sh | 76 | # 1. Current-tree install dists + dao-setup (master harness + 3 peers). | COMMENT |
| MEDIUM | apitest/docker/run-e2e-tests-compat.sh | 108 | # 4. Release peer image (inherits master harness, swaps app binaries). | COMMENT |
| MEDIUM⚡ | apitest/scripts/trade-xmr-simulation.sh | 15 | # These should be run using the apitest harness. From the root project dir, run: | COMMENT |
| MEDIUM | apitest/scripts/trade-simulation.sh | 18 | # These should be run using the apitest harness. From the root project dir, run: | COMMENT |
| MEDIUM⚡ | apitest/scripts/limit-order-simulation.sh | 17 | # These should be run using the apitest harness. From the root project dir, run: | COMMENT |
| MEDIUM | apitest/scripts/bsqswap-simulation.sh | 15 | # These should be run using the apitest harness. From the root project dir, run: | COMMENT |
| MEDIUM⚡ | apitest/scripts/rolling-offer-simulation.sh | 21 | # These should be run using the apitest harness. From the root project dir, run: | COMMENT |
| MEDIUM | .github/workflows/e2e-tests-compat.yml | 54 | # Master (current branch) install dists + dao-setup — the stack harness and | COMMENT |
| MEDIUM | .github/workflows/e2e-tests-compat.yml | 125 | # Release peer image: inherits the master harness, swaps app binaries. | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| MEDIUM | cli/package/create-cli-dist.sh | 26 | # Create a clean directory to hold the tarball's content. | COMMENT |
| MEDIUM | daemon/package/create-daemon-dist.sh | 26 | # Create a clean directory to hold the tarball's content. | COMMENT |
| MEDIUM | apitest/docker/bitcoind-entrypoint.sh | 71 | # Create a descriptor wallet "testwallet" after RPC comes up. The legacy | COMMENT |
| MEDIUM⚡ | apitest/scripts/trade-xmr-simulation.sh | 28 | # Create a buy/xmr offer to buy 0.125 btc at an xmr fixed-price of 0.05 btc, using an xmr payment account: | COMMENT |
| MEDIUM⚡ | apitest/scripts/trade-xmr-simulation.sh | 32 | # Create a sell/xmr offer to sell 0.125 btc at at an xmr mkt-price-margin of 0%, using using an xmr payment account: | COMMENT |
| MEDIUM⚡ | apitest/scripts/limit-order-simulation.sh | 32 | # Create a sell/eur offer to sell 0.125 btc at a fixed-price of 38,000 euros, using a France face to face | COMMENT |
| MEDIUM⚡ | apitest/scripts/rolling-offer-simulation.sh | 36 | # Create a buy/usd offer to sell 0.1 btc at 2% above market price, using a US face to face payment account: | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | apitest/docker/run-e2e-tests-compat.sh | 12 | # Usage: | COMMENT |
| LOW | apitest/docker/run-e2e-tests.sh | 10 | # Usage: | COMMENT |
| LOW⚡ | apitest/scripts/trade-xmr-simulation.sh | 18 | # Usage: | COMMENT |
| LOW | apitest/scripts/mainnet-test.sh | 13 | # Usage: | COMMENT |
| LOW | apitest/scripts/trade-simulation.sh | 23 | # Usage: | COMMENT |
| LOW⚡ | apitest/scripts/limit-order-simulation.sh | 22 | # Usage: | COMMENT |
| LOW | apitest/scripts/bsqswap-simulation.sh | 18 | # Usage: | COMMENT |
| LOW⚡ | apitest/scripts/rolling-offer-simulation.sh | 26 | # Usage: | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | desktop/src/main/java/bisq/desktop/app/BisqApp.java | 134 | // NOTE: This method is not called on the JavaFX Application Thread. | COMMENT |
| LOW | docs/release-checklist.md | 19 | ## Step 1: Update the version number | COMMENT |
| LOW | docs/release-checklist.md | 49 | ## Step 2: Update the bundled data files | COMMENT |
| LOW | docs/release-checklist.md | 286 | ## Step 3: Release notes | COMMENT |
| LOW | docs/release-checklist.md | 309 | ## Step 4: Build | COMMENT |
| LOW | docs/release-checklist.md | 382 | ## Step 5: Sign | COMMENT |
| LOW | docs/release-checklist.md | 430 | ## Step 6: Publish | COMMENT |
| LOW | docs/release-checklist.md | 475 | ## Step 7: After the release | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | …/network/p2p/inventory/GetInventoryRequestHandler.java | 63 | public class GetInventoryRequestHandler implements MessageListener { | CODE |
| LOW | …de/full/network/GetAccountingBlocksRequestHandler.java | 49 | class GetAccountingBlocksRequestHandler { | CODE |
| LOW | …/blindvote/network/RepublishGovernanceDataHandler.java | 58 | public final class RepublishGovernanceDataHandler { | CODE |
| LOW | …ore/dao/node/full/network/GetBlocksRequestHandler.java | 53 | class GetBlocksRequestHandler { | CODE |
| LOW | …ork/p2p/peers/peerexchange/GetPeersRequestHandler.java | 45 | class GetPeersRequestHandler { | CODE |
| LOW | …bisq/network/p2p/peers/getdata/RequestDataManager.java | 61 | public class RequestDataManager implements MessageListener, ConnectionListener, PeerManager.Listener { | CODE |
| LOW | …bisq/network/p2p/peers/getdata/RequestDataHandler.java | 59 | class RequestDataHandler implements MessageListener { | CODE |
| LOW | …q/network/p2p/peers/getdata/GetDataRequestHandler.java | 44 | public class GetDataRequestHandler { | CODE |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | …re/src/main/java/bisq/core/offer/OpenOfferManager.java | 1310 | // we need to check if in the meantime the offer has been removed | COMMENT |
| LOW | …/src/main/java/bisq/core/dao/node/parser/TxParser.java | 226 | // We need to check if any tempTxOutput is available and if so and the OpReturn data is invalid we | COMMENT |
| LOW | …ore/crypto/ed25519/arithmetic/Ed25519FieldElement.java | 941 | // Step 1: | COMMENT |
| LOW | …ore/crypto/ed25519/arithmetic/Ed25519FieldElement.java | 955 | // Step 2: | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | restapi/src/main/resources/doc/v1/swagger-ui.js | 1 | !function(e,t){"object"==typeof exports&&"object"==typeof module?module.exports=t():"function"==typeof define&&define.am | CODE |
| LOW | …src/main/resources/doc/v1/swagger-ui-es-bundle-core.js | 2 | import*as e from"react-immutable-pure-component";import*as t from"remarkable/linkify";import*as r from"dompurify";import | CODE |
| LOW | …/main/resources/doc/v1/swagger-ui-standalone-preset.js | 2 | !function(t,e){"object"==typeof exports&&"object"==typeof module?module.exports=e():"function"==typeof define&&define.am | CODE |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| MEDIUM | apitest/docker/entrypoint.sh | 96 | # which only works for local-process scaffolding, not docker bridge). | COMMENT |