Skip to content

Migration/pre-typescriptv6#276

Merged
BlobMaster41 merged 9 commits intomainfrom
migration/typescriptv6
Apr 1, 2026
Merged

Migration/pre-typescriptv6#276
BlobMaster41 merged 9 commits intomainfrom
migration/typescriptv6

Conversation

@BlobMaster41
Copy link
Copy Markdown
Contributor

@BlobMaster41 BlobMaster41 commented Mar 24, 2026

Description

Disable and reorder an ESLint rule in eslint.config.js (explicitly turn off no-useless-assignment with a comment and adjust related rule ordering). Update package.json dependency versions: bump @btc-vision/transaction, TypeScript to 6.0.2, typescript-eslint to 8.57.2, eslint to 10.1.0, gulp-eslint-new, @vitest/coverage-v8 and vitest; remove the older typescript-eslint packages from dependencies and add an overrides block to ensure typescript-eslint uses the workspace TypeScript. These changes align linting and dev tooling with the newer TypeScript and package versions.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Performance improvement
  • Consensus change (changes that affect state calculation or validation)
  • Refactoring (no functional changes)
  • Documentation update
  • CI/CD changes
  • Dependencies update

Checklist

Build & Tests

  • npm install completes without errors
  • npm run build completes without errors
  • npm test passes all tests

Code Quality

  • Code follows the project's coding standards
  • No new compiler warnings introduced
  • Error handling is appropriate
  • Logging is appropriate for debugging and monitoring

Documentation

  • Code comments added for complex logic
  • Public APIs are documented
  • README updated (if applicable)

Security

  • No sensitive data (keys, credentials) committed
  • No new security vulnerabilities introduced
  • RPC endpoints properly authenticated
  • Input validation in place for external data

OP_NET Node Specific

  • Changes are compatible with existing network state
  • Consensus logic changes are documented and tested
  • State transitions are deterministic
  • WASM VM execution is reproducible across nodes
  • P2P protocol changes are backward-compatible (or migration planned)
  • Database schema changes include migration path
  • Epoch finality and PoC/PoW logic unchanged (or documented if changed)

Testing

Consensus Impact

Related Issues


By submitting this PR, I confirm that my contribution is made under the terms of the project's license.

Introduce a maxLength based on OPNetConsensus.consensus.EPOCH.GRAFFITI_LENGTH minus 4 and use it when validating and reading the graffiti field. This ensures the graffiti bytes read do not exceed the protocol limit and replaces the previous bytesLeft-based check/read which could allow out-of-spec lengths.
Disable and reorder an ESLint rule in eslint.config.js (explicitly turn off no-useless-assignment with a comment and adjust related rule ordering). Update package.json dependency versions: bump @btc-vision/transaction, TypeScript to 6.0.2, typescript-eslint to 8.57.2, eslint to 10.1.0, gulp-eslint-new, @vitest/coverage-v8 and vitest; remove the older typescript-eslint packages from dependencies and add an overrides block to ensure typescript-eslint uses the workspace TypeScript. These changes align linting and dev tooling with the newer TypeScript and package versions.
@BlobMaster41 BlobMaster41 added enhancement New feature or request dependencies Pull requests that update a dependency file labels Mar 24, 2026
@BlobMaster41 BlobMaster41 changed the title Migration/typescriptv6 Migration/pre-typescriptv6 Apr 1, 2026
@BlobMaster41 BlobMaster41 merged commit 337c6df into main Apr 1, 2026
7 checks passed
@BlobMaster41 BlobMaster41 deleted the migration/typescriptv6 branch April 1, 2026 03:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant