Merge pull request #2202 from PR0M3TH3AN/chore/code-health/remove-unused-promises-app-init-18262315909809877684

chore: remove dead code and unused promises in Application.init
This commit is contained in:
thePR0M3TH3AN authored and GitHub committed 2026-02-07 20:07:29 -05:00
commit cc2e29c185
1 file changed
+24 -15
+24 -15
View File
@@ -1079,10 +1079,20 @@ class Application {
}
}
async _initSessionActor() {
if (typeof nostrClient.onSessionActorChange === "function") {
nostrClient.onSessionActorChange((detail) => {
this._handleSessionActorReady(detail).catch((error) => {
accessControl
.refresh()
.then(() => {
if (
accessControl.lastError &&
accessControl.lastError?.code === "nostr-unavailable"
) {
devLogger.warn(
"[app.init()] Access control refresh should not run before nostrClient.init()",
accessControl.lastError
);
}
})
.catch((error) => {
devLogger.warn(
"[app.init()] Failed to process session actor change:",
error,
@@ -1091,17 +1101,16 @@ class Application {
});
}
await this._syncSessionActorBlacklist("post-refresh");
}
async _initNostr() {
// Initialize the pool early to unblock bootstrapTrustedSeeds,
// but do NOT await the full connection process here.
try {
await nostrClient.ensurePool();
} catch (poolError) {
devLogger.warn("[app.init()] Pool ensure failed:", poolError);
}
if (this.profileController) {
Promise.resolve()
.then(() => this.profileController.refreshAdminPaneState())
.catch((error) => {
devLogger.warn(
"Failed to update admin pane after connecting to Nostr:",
error
);
});
}
// Kick off relay connection in the background.
nostrClient.init().catch((err) => {