Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1578 nakamoto remove anchor mode #1654

Merged
merged 1 commit into from
Jun 10, 2024
Merged

Conversation

janniks
Copy link
Collaborator

@janniks janniks commented Mar 11, 2024

This PR was published to npm with the version 6.13.0
e.g. npm install @stacks/[email protected] --save-exact

  • remove anchor mode from builder methods, default to .Any -- this byte will be ignored by nodes in >= Nakamoto

@janniks janniks linked an issue Mar 11, 2024 that may be closed by this pull request
Copy link

vercel bot commented Mar 11, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
stacksjs-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2024 3:29pm

Copy link

codecov bot commented Mar 11, 2024

Codecov Report

Attention: Patch coverage is 75.51020% with 36 lines in your changes missing coverage. Please review.

Project coverage is 66.44%. Comparing base (922e037) to head (17dee7b).
Report is 60 commits behind head on next.

Current head 17dee7b differs from pull request most recent head ca3756a

Please upload reports for the commit ca3756a to get more accurate results.

Files Patch % Lines
packages/stacking/src/index.ts 48.83% 17 Missing and 5 partials ⚠️
packages/common/src/buffer.ts 41.66% 7 Missing ⚠️
packages/stacking/src/utils.ts 86.66% 3 Missing and 1 partial ⚠️
packages/transactions/src/payload.ts 94.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             next    #1654      +/-   ##
==========================================
+ Coverage   66.37%   66.44%   +0.06%     
==========================================
  Files         119      119              
  Lines        8717     8800      +83     
  Branches     1920     1958      +38     
==========================================
+ Hits         5786     5847      +61     
+ Misses       2806     2709      -97     
- Partials      125      244     +119     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@MicaiahReid MicaiahReid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's one small suggestion, but looks good to me!

packages/stacking/src/utils.ts Outdated Show resolved Hide resolved
@janniks janniks force-pushed the 1578-nakamoto-remove-anchor-mode branch from 17dee7b to 8e80adc Compare May 13, 2024 11:32
@janniks janniks force-pushed the 1578-nakamoto-remove-anchor-mode branch from 8e80adc to f58b754 Compare May 13, 2024 13:01
@janniks janniks changed the base branch from main to remove-legacy-triplesec-support May 13, 2024 13:05
@janniks janniks force-pushed the 1578-nakamoto-remove-anchor-mode branch from f58b754 to 512cb36 Compare May 28, 2024 02:47
@janniks janniks force-pushed the 1578-nakamoto-remove-anchor-mode branch from 512cb36 to 3a1bdd1 Compare May 28, 2024 02:52
@janniks janniks force-pushed the remove-legacy-triplesec-support branch from 6ec9048 to 4721d1a Compare May 28, 2024 20:23
@janniks janniks force-pushed the 1578-nakamoto-remove-anchor-mode branch from 3a1bdd1 to 45e12d1 Compare May 28, 2024 20:23
@janniks janniks force-pushed the remove-legacy-triplesec-support branch from 4721d1a to 0eb233b Compare June 10, 2024 12:38
@janniks janniks force-pushed the 1578-nakamoto-remove-anchor-mode branch from 45e12d1 to 04b96a7 Compare June 10, 2024 12:39
@janniks janniks force-pushed the remove-legacy-triplesec-support branch from 0eb233b to 94b2f5d Compare June 10, 2024 13:48
@janniks janniks force-pushed the 1578-nakamoto-remove-anchor-mode branch from 04b96a7 to 22752ff Compare June 10, 2024 13:48
@janniks janniks force-pushed the remove-legacy-triplesec-support branch from 94b2f5d to 3e09560 Compare June 10, 2024 15:07
@janniks janniks force-pushed the 1578-nakamoto-remove-anchor-mode branch 2 times, most recently from 00d4bfe to cbf6ad1 Compare June 10, 2024 15:14
@janniks janniks force-pushed the remove-legacy-triplesec-support branch from 3e09560 to 5750984 Compare June 10, 2024 15:14
@janniks janniks force-pushed the 1578-nakamoto-remove-anchor-mode branch from cbf6ad1 to d798285 Compare June 10, 2024 15:17
@janniks janniks force-pushed the remove-legacy-triplesec-support branch 2 times, most recently from fdcb76d to e6f4c12 Compare June 10, 2024 15:20
@janniks janniks force-pushed the 1578-nakamoto-remove-anchor-mode branch from d798285 to 1884150 Compare June 10, 2024 15:20
Base automatically changed from remove-legacy-triplesec-support to next June 10, 2024 15:22
@janniks janniks force-pushed the 1578-nakamoto-remove-anchor-mode branch from 1884150 to ca3756a Compare June 10, 2024 15:23
@janniks janniks merged commit 2bec7ae into next Jun 10, 2024
7 of 8 checks passed
@janniks janniks deleted the 1578-nakamoto-remove-anchor-mode branch June 10, 2024 15:27
janniks added a commit that referenced this pull request Oct 25, 2024
* fix!: consolidate poxaddress return types

BREAKING CHANGE:
Convert `version` of PoXAddress from array to number.

* feat: cleanup common files

* refactor: update to correct import path

* feat!: update to new network object (#1623)

* feat: split network into object and api opts

* fix: update imports

---------

Co-authored-by: janniks <[email protected]>

* feat: add new api package

* fix: update remaining packages for api (#1625)

Co-authored-by: janniks <[email protected]>

* refactor: update other packages to new paradigm

* test: fix remaining packages

* chore: add jest config

* chore: resolve circular dependency

* build: update commitlint to allow longer content

* refactor!: change clarity types to be human readable

BREAKING CHANGE:
The `ClarityType` was changed to a readable string, from the previous Number type. This makes it easier to read and construct types when debugging.

* test: add clarity wire format tests

* docs: add migration docs

* docs: pr update

Co-authored-by: Hugo C <[email protected]>

* docs: pr update

Co-authored-by: Hugo C <[email protected]>

* docs: enumerate nets

* feat!: remove wrapped private key type

BREAKING CHANGE:
Remove wrapped `StacksPrivateKey` type in favor of simple type alias for string/Uint8Array.

* test: update bns tests

* chore: update cli (#1656)

Co-authored-by: janniks <[email protected]>

* feat!: Make serialize methods return hex-encoded strings, instead of bytes

* docs: update migration guide

* fix!: Remove legacy CLI methods

* refactor: decrease circular deps in tx package

* build: fix typedoc build

* ci: update pr version detetction

* fix!: Disable legacy triplesec support for mnemonic encryption

BREAKING CHANGE:
Support for encrypting/decrypting mnemonics using `triplesec` has been removed.

* feat: Remove optional anchor mode from transaction options (#1654)

Co-authored-by: janniks <[email protected]>

* ci: update canary publishing

* build: update lerna

* fix: Add `.bootAddress` to network objects

* chore: disable rule correctly

* chore: reduce guessing tx version

* refactor: add addressVersion to network object

* refactor: consolidate address types

* fix: Allow network name in more helper functions

* refactor: move types

* fix: Improve address helper functions

* refactor!: Rename message types

* refactor!: Make StacksTransaction.serialize() return a hex-string instead of bytes

* refactor!: Rename more types to use "Wire" suffix

* refactor!: Remove BN.js compatability

* feat!: Update post-condition representation to human readable types

* refactor: move internal wire types to reduce circular dependencies

* refactor: add internal package for reusing test and other helpers

* feat: Add postConditionToHex helper

* fix: Allow .appendOrigin to be used with unwrapped public key

* feat!: Rename networking/fetch methods to include `fetch` prefix

* fix: Consolidate `is` prefixed boolean methods

* chore: add comment

* feat: Add STX unit helper methods

* feat: Add `Address` namespace

* feat: Add origin post-condition principals

* refactor: minor updates

* fix!: Remove signed option for intToBigInt

* fix!: Move more helper functions to take network parameter

* feat: Reduce wrapping of types for public key appending

* refactor: Remove Serialize prefix from enum keys

* refactor!: Remove wrapper type for message signatures

* fix!: Compressing private key should ensure string format

* refactor: Fix build breaking refactor changes

* fix: Generate compressed private keys by default

* refactor: Add renamed `randomSeedPhrase` method

* refactor: Rename api to client and undo previous StacksNodeAPI changes

* refactor: Continue client rename

* test: Update to succeed tests

* ci: update canary releases

* fix: Remove legacy methods

* chore: update comments

* fix: Fix string parsing in structured signature decode

* chore: Fix example in comment

* refactor: Undo removal

* fix!: Switch to a network and client param solution

* chore: fix imports

* docs: Update migration file

* refactor: Minor refactoring to match common theme

* refactor: Rename make methods

* refactor: Deprecate wallet config

* refactor: Remove unneeded deserialize overloads

* refactor!: Update StacksTransaction constructor to options object

* refactor!: Rename StacksTransaction to StacksTransactionWire

* fix: Update post conditions input in transactions

* chore: add forgotten rename

* docs: Update migration guide

* refactor: Change getStxAddress to use network parameter

* refactor!: Update Cl.serialize to return string instead of bytes

---------

Co-authored-by: janniks <[email protected]>
Co-authored-by: Hugo C <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[Nakamoto] Remove anchor-mode
2 participants