Skip to content

Comments

Upgrade nostr-java to 2.0.0#9

Merged
tcheeric merged 4 commits intodevelopfrom
feature/nostr-java-2-0
Feb 24, 2026
Merged

Upgrade nostr-java to 2.0.0#9
tcheeric merged 4 commits intodevelopfrom
feature/nostr-java-2-0

Conversation

@tcheeric
Copy link
Contributor

Description

Migrate the entire project from nostr-java 1.2.0 to 2.0.0, adapting to the new module structure and API changes. Bumps project version from 0.1.1 to 0.2.0.

Related Issue

N/A

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Test coverage improvement

Checklist

  • My code follows the project's code style
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Testing

Built successfully with mvn clean install -DskipTests.

18 files changed, 81 insertions(+), 70 deletions(-)

Key changes

Dependency migration (nostr-java 1.2.0 → 2.0.0):

  • nostr-java-apinostr-java-core
  • nostr-java-cryptonostr-java-core
  • nostr-java-idnostr-java-identity
  • nostr-java-encryptionnostr-java-identity
  • Removed nostr-java-base (merged into core)

Code adaptation:

  • Replaced PubKeyTag with GenericTag in NsecBunkerAdminClient and RelayContainerIntegrationTest

Version bump:

  • All 15 modules bumped from 0.1.1 → 0.2.0
  • Created CHANGELOG.md

Additional Notes

Tagged as v0.2.0.

🤖 Generated with Claude Code

tcheeric and others added 3 commits February 24, 2026 21:55
Migrate artifact names to match nostr-java 2.0 module structure:
- nostr-java-api → nostr-java-core
- nostr-java-crypto → nostr-java-core
- nostr-java-id → nostr-java-identity
- nostr-java-encryption → nostr-java-identity
- Remove nostr-java-base (merged into core)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace PubKeyTag with GenericTag for tag handling in
NsecBunkerAdminClient and RelayContainerIntegrationTest,
aligning with the simplified tag API in nostr-java 2.0.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Projects updated:
- nsecbunker-java (parent): 0.1.1 → 0.2.0 (minor)
- All 14 submodules: 0.1.1 → 0.2.0 (minor)

Changes include:
- Upgraded nostr-java from 1.2.0 to 2.0.0
- Migrated to new nostr-java 2.0 module structure
- Adapted code to nostr-java 2.0 API (GenericTag)
- Created CHANGELOG.md

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Deleted unused `.github/pull_request_template.md` and `.github/workflows/release.yml` files to clean up the repository.
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR upgrades the nostr-java dependency from version 1.2.0 to 2.0.0, adapting to the new module structure and API changes introduced in the newer version. The project version is bumped from 0.1.1 to 0.2.0 to reflect this breaking dependency change.

Changes:

  • Upgraded nostr-java dependency from 1.2.0 to 2.0.0 across all modules
  • Migrated to nostr-java 2.0 module structure (consolidated artifacts)
  • Replaced deprecated PubKeyTag API with GenericTag in admin client and integration tests

Reviewed changes

Copilot reviewed 20 out of 20 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pom.xml Updated nostr-java version to 2.0.0, consolidated dependency artifacts from 6 to 3 modules
nsecbunker-admin/pom.xml Updated parent version to 0.2.0, migrated to nostr-java-identity and nostr-java-core
nsecbunker-admin/src/main/java/xyz/tcheeric/nsecbunker/admin/NsecBunkerAdminClient.java Replaced PubKeyTag with GenericTag.of() for tag creation and serialization
nsecbunker-account/pom.xml Updated parent version to 0.2.0
nsecbunker-client/pom.xml Updated parent version to 0.2.0, added nostr-java-core and nostr-java-identity dependencies
nsecbunker-connection/pom.xml Updated parent version to 0.2.0, changed dependency from nostr-java-api to nostr-java-event
nsecbunker-core/pom.xml Updated parent version to 0.2.0, removed nostr-java-api dependency
nsecbunker-monitoring/pom.xml Updated parent version to 0.2.0
nsecbunker-protocol/pom.xml Updated parent version to 0.2.0, consolidated nostr-java-crypto and nostr-java-encryption into nostr-java-identity
nsecbunker-spring-boot-starter/pom.xml Updated parent version to 0.2.0
nsecbunker-tests/pom.xml Updated parent version to 0.2.0
nsecbunker-tests/nsecbunker-chaos/pom.xml Updated parent version to 0.2.0
nsecbunker-tests/nsecbunker-e2e/pom.xml Updated parent version to 0.2.0, migrated to nostr-java-identity and nostr-java-core
nsecbunker-tests/nsecbunker-it/pom.xml Updated parent version to 0.2.0
nsecbunker-tests/nsecbunker-it/src/test/java/xyz/tcheeric/nsecbunker/it/RelayContainerIntegrationTest.java Replaced PubKeyTag with GenericTag for event tag serialization in test
nsecbunker-tests/nsecbunker-perf/pom.xml Updated parent version to 0.2.0, migrated to nostr-java-identity and nostr-java-core
nsecbunker-tests/nsecbunker-security/pom.xml Updated parent version to 0.2.0
CHANGELOG.md Added entry for version 0.2.0 documenting the nostr-java upgrade and API changes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tcheeric tcheeric merged commit d72c621 into develop Feb 24, 2026
2 of 4 checks passed
@tcheeric tcheeric deleted the feature/nostr-java-2-0 branch February 24, 2026 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant