Skip to content

Commit 052b634

Browse files
committed
chore(fix) missing sign return value
1 parent 9b57773 commit 052b634

File tree

9 files changed

+47
-39
lines changed

9 files changed

+47
-39
lines changed

packages/ndk/example/bunkers_example.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,9 @@ Future<void> main() async {
2222
tags: [],
2323
);
2424

25-
await ndk.accounts.sign(event);
25+
final signedEvent = await ndk.accounts.sign(event);
2626
log('Event signed successfully!');
27-
log('Event ID: ${event.id}');
27+
log('Event ID: ${signedEvent.id}');
2828
} catch (e) {
2929
log('Error: $e');
3030
}

packages/ndk/example/nostrconnect_example.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,9 @@ Future<void> main() async {
2828
tags: [],
2929
);
3030

31-
await ndk.accounts.sign(event);
31+
final signedEvent = await ndk.accounts.sign(event);
3232
log('Event signed successfully!');
33-
log('Event ID: ${event.id}');
33+
log('Event ID: ${signedEvent.id}');
3434
} catch (e) {
3535
log('Error: $e');
3636
}

packages/ndk/lib/data_layer/repositories/signers/nip46_event_signer.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,9 +105,9 @@ class Nip46EventSigner implements EventSigner {
105105
content: encryptedRequest!,
106106
);
107107

108-
await localEventSigner.sign(requestEvent);
108+
final signedEvent = await localEventSigner.sign(requestEvent);
109109
final broadcastRes = broadcast.broadcast(
110-
nostrEvent: requestEvent,
110+
nostrEvent: signedEvent,
111111
specificRelays: connection.relays,
112112
);
113113
await broadcastRes.broadcastDoneFuture;

packages/ndk/lib/domain_layer/usecases/bunkers/bunkers.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,9 +75,9 @@ class Bunkers {
7575
content: encryptedRequest!,
7676
);
7777

78-
await localEventSigner.sign(requestEvent);
78+
final signedEvent = await localEventSigner.sign(requestEvent);
7979
final broadcastRes =
80-
_broadcast.broadcast(nostrEvent: requestEvent, specificRelays: relays);
80+
_broadcast.broadcast(nostrEvent: signedEvent, specificRelays: relays);
8181
await broadcastRes.broadcastDoneFuture;
8282

8383
final subscription = _requests.subscription(

packages/ndk/test/signers/nip46_event_signer_test.dart

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,10 +62,10 @@ void main() {
6262
content: 'Hello, world!',
6363
);
6464

65-
await signer.sign(event);
65+
final signedEvent = await signer.sign(event);
6666

67-
expect(event.id, isNotNull);
68-
expect(event.sig, isNotNull);
67+
expect(signedEvent.id, isNotNull);
68+
expect(signedEvent.sig, isNotNull);
6969
});
7070

7171
test('getPublicKey should throw when not cached', () {

packages/ndk/test/usecases/lists/lists_test.dart

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,11 @@ void main() async {
4848
Nip01Event event0 = await bookmarkListKey0.toEvent(signer0);
4949
Nip01Event event1 = await favoriteRelaysKey1.toEvent(signer1);
5050

51-
await signer0.sign(event0);
52-
await signer1.sign(event1);
51+
final signedEvent0 = await signer0.sign(event0);
52+
final signedEvent1 = await signer1.sign(event1);
5353

54-
await relay0.startServer(textNotes: {key0: event0, key1: event1});
54+
await relay0
55+
.startServer(textNotes: {key0: signedEvent0, key1: signedEvent1});
5556

5657
final cache = MemCacheManager();
5758
final NdkConfig config = NdkConfig(
@@ -354,12 +355,12 @@ void main() async {
354355
final oldEvent = await oldSet.toEvent(signer1);
355356
final newEvent = await newSet.toEvent(signer1);
356357

357-
await signer1.sign(oldEvent);
358-
await signer1.sign(newEvent);
358+
final singedOldEvent = await signer1.sign(oldEvent);
359+
final signedNewEvent = await signer1.sign(newEvent);
359360

360361
// Save both to cache
361-
await ndk.config.cache.saveEvent(oldEvent);
362-
await ndk.config.cache.saveEvent(newEvent);
362+
await ndk.config.cache.saveEvent(singedOldEvent);
363+
await ndk.config.cache.saveEvent(signedNewEvent);
363364

364365
// Fetch the set - should return the newer one
365366
final fetchedSet = await ndk.lists.getSetByName(
@@ -428,10 +429,10 @@ void main() async {
428429
content: nip04Content,
429430
);
430431

431-
await signer1.sign(nip04Event);
432+
final signedNip04Event = await signer1.sign(nip04Event);
432433

433434
// Parse the event back to set - should handle NIP-04 format
434-
final parsedSet = await Nip51Set.fromEvent(nip04Event, signer1);
435+
final parsedSet = await Nip51Set.fromEvent(signedNip04Event, signer1);
435436

436437
expect(parsedSet, isNotNull);
437438
expect(parsedSet!.name, "nip04-test-set");

packages/nip07_event_signer/test/nip07_event_signer_test.dart

Lines changed: 18 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -335,13 +335,19 @@ void main() {
335335
createdAt: 1234567890,
336336
);
337337

338-
await nip07Signer.sign(event);
338+
final signedEvent = await nip07Signer.sign(event);
339339

340-
expect(event.id, isNotNull);
341-
expect(event.id.length, equals(64)); // SHA-256 hash is 64 hex characters
342-
expect(RegExp(r'^[a-f0-9]{64}$').hasMatch(event.id), isTrue);
343-
expect(event.sig, isNotNull);
344-
expect(event.sig!.length, equals(64)); // Signature is 64 hex characters
340+
expect(signedEvent.id, isNotNull);
341+
expect(
342+
signedEvent.id.length,
343+
equals(64),
344+
); // SHA-256 hash is 64 hex characters
345+
expect(RegExp(r'^[a-f0-9]{64}$').hasMatch(signedEvent.id), isTrue);
346+
expect(signedEvent.sig, isNotNull);
347+
expect(
348+
signedEvent.sig!.length,
349+
equals(64),
350+
); // Signature is 64 hex characters
345351
});
346352

347353
test('encrypt using NIP-04', () async {
@@ -602,14 +608,14 @@ void main() {
602608
content: 'GM',
603609
);
604610

605-
await nip07Signer.sign(event);
611+
final signedEvent = await nip07Signer.sign(event);
606612

607613
// Verify event was signed
608-
expect(event.id, isNotNull);
609-
expect(event.sig, isNotNull);
610-
expect(event.id.length, equals(64));
611-
expect(RegExp(r'^[a-f0-9]{64}$').hasMatch(event.id), isTrue);
612-
expect(event.sig!.length, equals(64));
614+
expect(signedEvent.id, isNotNull);
615+
expect(signedEvent.sig, isNotNull);
616+
expect(signedEvent.id.length, equals(64));
617+
expect(RegExp(r'^[a-f0-9]{64}$').hasMatch(signedEvent.id), isTrue);
618+
expect(signedEvent.sig!.length, equals(64));
613619
});
614620
});
615621
});

packages/nip07_event_signer/test/test_nip07_event_signer/lib/main.dart

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -133,10 +133,10 @@ Future<bool> testSign(Nip07EventSigner nip07Signer) async {
133133
content: "GM",
134134
);
135135

136-
await nip07Signer.sign(event);
136+
final signedEvent = await nip07Signer.sign(event);
137137

138-
return await Bip340EventVerifier().verify(event) &&
139-
event.kind == 1 &&
140-
event.pubKey == pubKey &&
141-
event.content == "GM";
138+
return await Bip340EventVerifier().verify(signedEvent) &&
139+
signedEvent.kind == 1 &&
140+
signedEvent.pubKey == pubKey &&
141+
signedEvent.content == "GM";
142142
}

packages/sample-app/lib/verifiers_performance.dart

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,10 +43,11 @@ class _VerifiersPerformancePageState extends State<VerifiersPerformancePage> {
4343
);
4444
});
4545

46+
final List<Nip01Event> signedList = [];
4647
for (final event in list) {
47-
await widget.ndk.accounts.sign(event);
48+
signedList.add(await widget.ndk.accounts.sign(event));
4849
}
49-
return list;
50+
return signedList;
5051
}
5152

5253
_verifyEventsWaiting({required EventVerifier verifier}) async {

0 commit comments

Comments
 (0)