Dev Tools: Source map decoder by barnjamin · Pull Request #590 · algorand/js-algorand-sdk

barnjamin · 2022-06-21T16:18:31Z

Implements a class to accept and decode a source map.

Also adds the option to request source map from the client from the /v2/teal/compile endpoint

Co-authored-by: Michael Diamant <michaeldiamant@users.noreply.github.com>

* properly set trace maxWidth (#593) * fix: safe intDecoding (#599) * fix: safe intDecoding * test: parse number in edge case * refactor: remove optional chaining for node12 * Remove code that relies on node's path module (#598) * Remove code that relies on node's path module * Replace url-parse with built in WHATWG URL API * Removed path-browserify fallback from webpack config * Removed path-browserify and url-parse from npm dependencies * Removed references to `path-browserify` in FAQ.md * bump version * Github-Actions: Adding PR title and label checks (#600) * Remove unused/unmaintained templates (#607) * Dev Tools: Source map decoder (#590) * adding source map decoder * Enhancement: Upgrade typedoc and plugins (#605) * Update ts-node, typescript, and typedoc to latest * docs: tealSign (#610) * bump version and add to changelog * update README.md for new version * API: Support attaching signatures to standard and multisig transactions (#595) * Add attach signature method to transaction class * Add multisig external signature methods * Fix failing multisig test * Add signature length checks * Add method to create an unsigned multisig transaction blob * Rename multisig create methods and use unencoded transaction * Refactor `createMultisigTransactionWithSignature` to use `createMultisigTransaction` method * Fix algosdk createMultisigTransaction export * Use MultisigMetadata without pks in new create method * These types should be consolidated in the future, and addrs seems like a better convention to use long-term * More descriptive test suite name * AVM: Consolidate TEAL and AVM versions (#609) * Testing: Use Dev mode network for cucumber tests (#614) * Send zero txn to itself * Refactor block advance functions * Revise steps to allow for rekeying transient accounts * Try to reduce flaky tests * Move constant into step * Add artificial sleep instead of sending blank txns when mimicking wait for block API * Reduce flaky tests * Remove unnecessary use of v2 algod client (#616) * Remove unnecessary use of v2 algod client * Add missing await for async function calls (#617) * Rename rekey tag in makefile * Revert testing branch back to master Co-authored-by: Michael Diamant <michaeldiamant@users.noreply.github.com> * Revert package lock from develop Co-authored-by: Joe Polny <50534337+joe-p@users.noreply.github.com> Co-authored-by: AlgoDoggo <93348148+AlgoDoggo@users.noreply.github.com> Co-authored-by: Bryan Dela Cruz <bdshi13@outlook.com> Co-authored-by: Lucky Baar <lucky.baar@algorand.com> Co-authored-by: Jack <87339414+algojack@users.noreply.github.com> Co-authored-by: Eric Warehime <eric.warehime@gmail.com> Co-authored-by: Ben Guidarelli <ben.guidarelli@gmail.com> Co-authored-by: Fionna Chan <fionnacst@gmail.com> Co-authored-by: Jack Smith <jack.smith@algorand.com> Co-authored-by: Jacob Daitzman <jdtzmn@gmail.com> Co-authored-by: Michael Diamant <michaeldiamant@users.noreply.github.com>

* Add boxes endpoint with path tests * Add lookup box by name api for indexer * Merge develop branch into feature/box-storage (#620) * properly set trace maxWidth (#593) * fix: safe intDecoding (#599) * fix: safe intDecoding * test: parse number in edge case * refactor: remove optional chaining for node12 * Remove code that relies on node's path module (#598) * Remove code that relies on node's path module * Replace url-parse with built in WHATWG URL API * Removed path-browserify fallback from webpack config * Removed path-browserify and url-parse from npm dependencies * Removed references to `path-browserify` in FAQ.md * bump version * Github-Actions: Adding PR title and label checks (#600) * Remove unused/unmaintained templates (#607) * Dev Tools: Source map decoder (#590) * adding source map decoder * Enhancement: Upgrade typedoc and plugins (#605) * Update ts-node, typescript, and typedoc to latest * docs: tealSign (#610) * bump version and add to changelog * update README.md for new version * API: Support attaching signatures to standard and multisig transactions (#595) * Add attach signature method to transaction class * Add multisig external signature methods * Fix failing multisig test * Add signature length checks * Add method to create an unsigned multisig transaction blob * Rename multisig create methods and use unencoded transaction * Refactor `createMultisigTransactionWithSignature` to use `createMultisigTransaction` method * Fix algosdk createMultisigTransaction export * Use MultisigMetadata without pks in new create method * These types should be consolidated in the future, and addrs seems like a better convention to use long-term * More descriptive test suite name * AVM: Consolidate TEAL and AVM versions (#609) * Testing: Use Dev mode network for cucumber tests (#614) * Send zero txn to itself * Refactor block advance functions * Revise steps to allow for rekeying transient accounts * Try to reduce flaky tests * Move constant into step * Add artificial sleep instead of sending blank txns when mimicking wait for block API * Reduce flaky tests * Remove unnecessary use of v2 algod client (#616) * Remove unnecessary use of v2 algod client * Add missing await for async function calls (#617) * Rename rekey tag in makefile * Revert testing branch back to master Co-authored-by: Michael Diamant <michaeldiamant@users.noreply.github.com> * Revert package lock from develop Co-authored-by: Joe Polny <50534337+joe-p@users.noreply.github.com> Co-authored-by: AlgoDoggo <93348148+AlgoDoggo@users.noreply.github.com> Co-authored-by: Bryan Dela Cruz <bdshi13@outlook.com> Co-authored-by: Lucky Baar <lucky.baar@algorand.com> Co-authored-by: Jack <87339414+algojack@users.noreply.github.com> Co-authored-by: Eric Warehime <eric.warehime@gmail.com> Co-authored-by: Ben Guidarelli <ben.guidarelli@gmail.com> Co-authored-by: Fionna Chan <fionnacst@gmail.com> Co-authored-by: Jack Smith <jack.smith@algorand.com> Co-authored-by: Jacob Daitzman <jdtzmn@gmail.com> Co-authored-by: Michael Diamant <michaeldiamant@users.noreply.github.com> * Revert package lock * Fix comments in box related APIs * Add next token to indexer box API test * rm unused step * Update src/client/v2/algod/getApplicationBoxByName.ts * Update .test-env Co-authored-by: Joe Polny <50534337+joe-p@users.noreply.github.com> Co-authored-by: AlgoDoggo <93348148+AlgoDoggo@users.noreply.github.com> Co-authored-by: Bryan Dela Cruz <bdshi13@outlook.com> Co-authored-by: Lucky Baar <lucky.baar@algorand.com> Co-authored-by: Jack <87339414+algojack@users.noreply.github.com> Co-authored-by: Eric Warehime <eric.warehime@gmail.com> Co-authored-by: Ben Guidarelli <ben.guidarelli@gmail.com> Co-authored-by: Fionna Chan <fionnacst@gmail.com> Co-authored-by: Jack Smith <jack.smith@algorand.com> Co-authored-by: Jacob Daitzman <jdtzmn@gmail.com> Co-authored-by: Michael Diamant <michaeldiamant@users.noreply.github.com> Co-authored-by: Hang Su <hang.su@algorand.com> Co-authored-by: Hang Su <87964331+ahangsu@users.noreply.github.com>

adding source map decoder

9ac12bf

jasonpaulos reviewed Jun 21, 2022

View reviewed changes

Comment thread src/logic/sourcemap.ts Outdated

barnjamin added 2 commits June 22, 2022 08:23

replace implemented funcs with vlq dep

2344ed6

tweak sourcemap to accept obj init

033972c

barnjamin mentioned this pull request Jun 24, 2022

SDKs: Support sourcemap decoding algorand/algorand-sdk-testing#198

Closed

barnjamin added 3 commits June 24, 2022 10:57

add sourcemmap query param

12e19f2

implement tests

d57e035

fix makefile

cfeb9a1

barnjamin marked this pull request as ready for review June 24, 2022 16:13

barnjamin requested a review from jasonpaulos June 24, 2022 16:14

barnjamin added 2 commits June 24, 2022 12:30

use new tests

5392e25

Remove undefined test

97b5716

barnjamin requested a review from algochoi June 24, 2022 17:16

revert docker shell script

626a246

barnjamin mentioned this pull request Jun 25, 2022

Add sourcemap query param to compile #597

Closed

barnjamin added 3 commits June 28, 2022 09:20

adding backwards compat for mapping and version check

de180d9

update with correct mapping

5567bdb

merge develop

84a9706

barnjamin closed this Jul 6, 2022

barnjamin reopened this Jul 6, 2022

barnjamin added the New Feature label Jul 6, 2022

barnjamin closed this Jul 6, 2022

barnjamin reopened this Jul 6, 2022

barnjamin changed the title ~~adding source map decoder~~ Dev Tools: Source map decoder Jul 6, 2022

barnjamin closed this Jul 6, 2022

barnjamin reopened this Jul 6, 2022

barnjamin added 4 commits July 11, 2022 08:18

tweak logic, add comments about undefined lineDelta

2bc3ee6

Merge branch 'develop' into src-map-decoder

9b6df7e

maybe?

d8c22e7

re-up package-lock

7034948

barnjamin force-pushed the src-map-decoder branch from a361a5d to 7034948 Compare July 11, 2022 13:17

michaeldiamant reviewed Jul 12, 2022

View reviewed changes

Comment thread src/logic/sourcemap.ts Outdated

michaeldiamant reviewed Jul 12, 2022

View reviewed changes

Comment thread Makefile

barnjamin added 3 commits July 12, 2022 11:23

adding new tests

c2a3421

adding source map tag to integration tests

fde8bcb

update to master branch of sdk testing repo

b7c1631

michaeldiamant reviewed Jul 14, 2022

View reviewed changes

Comment thread src/logic/sourcemap.ts Outdated

fix conditional

5e94b7b

michaeldiamant reviewed Jul 15, 2022

View reviewed changes

Comment thread tests/cucumber/steps/steps.js Outdated

michaeldiamant approved these changes Jul 15, 2022

View reviewed changes

barnjamin and others added 3 commits July 15, 2022 09:02

Update tests/cucumber/steps/steps.js

d708c35

Co-authored-by: Michael Diamant <michaeldiamant@users.noreply.github.com>

prettier?

8489632

prettier

2d17775

barnjamin merged commit 4266235 into develop Jul 15, 2022

barnjamin deleted the src-map-decoder branch July 15, 2022 17:29

algojack mentioned this pull request Jul 21, 2022

FOR REVIEW ONLY - Release v1.19.0 #613

Merged

michaeldiamant mentioned this pull request Aug 15, 2022

New Feature: Implemented disassemble method #622

Closed

jasonpaulos mentioned this pull request Nov 2, 2022

CI: Remove unneeded dependency & update package-lock #684

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Dev Tools: Source map decoder#590

Dev Tools: Source map decoder#590
barnjamin merged 23 commits into
developfrom
src-map-decoder

barnjamin commented Jun 21, 2022 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

barnjamin commented Jun 21, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

barnjamin commented Jun 21, 2022 •

edited

Loading