Feat: app metadata (#570)

* chore: wg.Add() before go (#559)

* chore: code cleanup (#558)

* Update README.md

Add command to make the install script executable in the instructions.

* feat: add migration card in wallet screen (#555)

* feat: add migration card in wallet screen

* chore: transfer funds if channel exists

* chore: typo

* chore: improve copy

* chore: add transfer funds button

* fix: remove duplicate icon

---------

Co-authored-by: Roland Bewick <roland.bewick@gmail.com>

* chore: run extra workflows on PR (#563)

* chore: run extra workflows on PR

* fix: pull request workflows

* build(deps): bump github.com/nbd-wtf/go-nostr from 0.34.5 to 0.34.10 (#551)

Bumps [github.com/nbd-wtf/go-nostr](https://github.com/nbd-wtf/go-nostr) from 0.34.5 to 0.34.10.
- [Commits](nbd-wtf/go-nostr@v0.34.5...v0.34.10)

---
updated-dependencies:
- dependency-name: github.com/nbd-wtf/go-nostr
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump gopkg.in/DataDog/dd-trace-go.v1 from 1.66.0 to 1.67.0 (#552)

Bumps gopkg.in/DataDog/dd-trace-go.v1 from 1.66.0 to 1.67.0.

---
updated-dependencies:
- dependency-name: gopkg.in/DataDog/dd-trace-go.v1
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix: do not render 0 on withdraw onchain funds page when reserve balance is 0 (#549)

* chore: update fly default kill timeout to ensure app gracefully shuts down (#547)

* feat: app metadata

* fix(wails): parse max length in log output endpoint (#568)

* fix: alby account image and name on transaction list

* fix: vertically center transaction item content

* fix: use slashed zero on wallet balance

* chore: align app image with app name in transaction list

* fix: ensure auth with correct user if refresh token expires (#572)

* fix: do not allow editing name of Alby Account connection

* fix: change "Create Wallet" to "Create Subaccount"

* fix: subaccount copy

* chore: update friends and family app icon

* fix: update app store images (#574)

* fix: update app store images

* chore: update paper scissors hodl and zapplanner icons

* chore: update lume and habla news icons

* chore: resize and compress zapplanner and rps logos

---------

Co-authored-by: Roland Bewick <roland.bewick@gmail.com>

* feat: add new auto channel flow that works with existing channels (#556)

* feat: add new auto channel flow that works with existing channels

* chore: remove "first" from auto channel copy

* feat: new transaction list item design

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Matjaž Lipuš <matjazl@gmail.com>
Co-authored-by: BtcPins <pardus79@gmail.com>
Co-authored-by: Michael Bumann <hello@michaelbumann.com>
Co-authored-by: Adithya Vardhan <imadithyavardhan@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

.github/workflows/http.yml

@@ -1,6 +1,10 @@
 name: HTTP build - Linux and MacOS
 on:
   push:
+    branches:
+      - master
+  pull_request:
+    types: [opened, synchronize]
   workflow_call:
     inputs:
       build-release:

.github/workflows/wails.yml

@@ -2,6 +2,10 @@ name: Wails build - all platforms
 
 on:
   push:
+    branches:
+      - master
+  pull_request:
+    types: [opened, synchronize]
 
   workflow_call:
     inputs:

alby/alby_oauth_service.go

@@ -439,6 +439,7 @@ func (svc *albyOAuthService) LinkAccount(ctx context.Context, lnClient lnclient.
 		nil,
 		scopes,
 		false,
+		nil,
 	)
 
 	if err != nil {

api/api.go

@@ -75,7 +75,8 @@ func (api *api) CreateApp(createAppRequest *CreateAppRequest) (*CreateAppRespons
 		createAppRequest.BudgetRenewal,
 		expiresAt,
 		createAppRequest.Scopes,
-		createAppRequest.Isolated)
+		createAppRequest.Isolated,
+		createAppRequest.Metadata)
 
 	if err != nil {
 		return nil, err
@@ -220,6 +221,16 @@ func (api *api) GetApp(dbApp *db.App) *App {
 	maxAmount := uint64(paySpecificPermission.MaxAmountSat)
 	budgetUsage = queries.GetBudgetUsageSat(api.db, &paySpecificPermission)
 
+	var metadata Metadata
+	if dbApp.Metadata != nil {
+		jsonErr := json.Unmarshal(dbApp.Metadata, &metadata)
+		if jsonErr != nil {
+			logger.Logger.WithError(jsonErr).WithFields(logrus.Fields{
+				"app_id": dbApp.ID,
+			}).Error("Failed to deserialize app metadata")
+		}
+	}
+
 	response := App{
 		ID:            dbApp.ID,
 		Name:          dbApp.Name,
@@ -233,6 +244,7 @@ func (api *api) GetApp(dbApp *db.App) *App {
 		BudgetUsage:   budgetUsage,
 		BudgetRenewal: paySpecificPermission.BudgetRenewal,
 		Isolated:      dbApp.Isolated,
+		Metadata:      metadata,
 	}
 
 	if dbApp.Isolated {
@@ -300,6 +312,17 @@ func (api *api) ListApps() ([]App, error) {
 			apiApp.LastEventAt = &lastEvent.CreatedAt
 		}
 
+		var metadata Metadata
+		if dbApp.Metadata != nil {
+			jsonErr := json.Unmarshal(dbApp.Metadata, &metadata)
+			if jsonErr != nil {
+				logger.Logger.WithError(jsonErr).WithFields(logrus.Fields{
+					"app_id": dbApp.ID,
+				}).Error("Failed to deserialize app metadata")
+			}
+			apiApp.Metadata = metadata
+		}
+
 		apiApps = append(apiApps, apiApp)
 	}
 	return apiApps, nil

api/models.go

@@ -70,6 +70,7 @@ type App struct {
 	BudgetRenewal string     `json:"budgetRenewal"`
 	Isolated      bool       `json:"isolated"`
 	Balance       uint64     `json:"balance"`
+	Metadata      Metadata   `json:"metadata,omitempty"`
 }
 
 type ListAppsResponse struct {
@@ -93,6 +94,7 @@ type CreateAppRequest struct {
 	Scopes        []string `json:"scopes"`
 	ReturnTo      string   `json:"returnTo"`
 	Isolated      bool     `json:"isolated"`
+	Metadata      Metadata `json:"metadata,omitempty"`
 }
 
 type StartRequest struct {

cmd/http/main.go

@@ -17,7 +17,7 @@ import (
 )
 
 func main() {
-	log.Info("NWC Starting in HTTP mode")
+	log.Info("AlbyHub Starting in HTTP mode")
 
 	// Create a channel to receive OS signals.
 	osSignalChannel := make(chan os.Signal, 1)

db/db_service.go

@@ -2,6 +2,7 @@ package db
 
 import (
 	"encoding/hex"
+	"encoding/json"
 	"errors"
 	"fmt"
 	"slices"
@@ -11,6 +12,7 @@ import (
 	"github.com/getAlby/hub/events"
 	"github.com/getAlby/hub/logger"
 	"github.com/nbd-wtf/go-nostr"
+	"gorm.io/datatypes"
 	"gorm.io/gorm"
 )
 
@@ -26,7 +28,7 @@ func NewDBService(db *gorm.DB, eventPublisher events.EventPublisher) *dbService
 	}
 }
 
-func (svc *dbService) CreateApp(name string, pubkey string, maxAmountSat uint64, budgetRenewal string, expiresAt *time.Time, scopes []string, isolated bool) (*App, string, error) {
+func (svc *dbService) CreateApp(name string, pubkey string, maxAmountSat uint64, budgetRenewal string, expiresAt *time.Time, scopes []string, isolated bool, metadata map[string]interface{}) (*App, string, error) {
 	if isolated && (slices.Contains(scopes, constants.SIGN_MESSAGE_SCOPE)) {
 		// cannot sign messages because the isolated app is a custodial subaccount
 		return nil, "", errors.New("isolated app cannot have sign_message scope")
@@ -47,7 +49,17 @@ func (svc *dbService) CreateApp(name string, pubkey string, maxAmountSat uint64,
 		}
 	}
 
-	app := App{Name: name, NostrPubkey: pairingPublicKey, Isolated: isolated}
+	var metadataBytes []byte
+	if metadata != nil {
+		var err error
+		metadataBytes, err = json.Marshal(metadata)
+		if err != nil {
+			logger.Logger.WithError(err).Error("Failed to serialize metadata")
+			return nil, "", err
+		}
+	}
+
+	app := App{Name: name, NostrPubkey: pairingPublicKey, Isolated: isolated, Metadata: datatypes.JSON(metadataBytes)}
 
 	err := svc.db.Transaction(func(tx *gorm.DB) error {
 		err := tx.Save(&app).Error

db/migrations/202408291715_app_metadata.go

@@ -0,0 +1,25 @@
+package migrations
+
+import (
+	_ "embed"
+
+	"github.com/go-gormigrate/gormigrate/v2"
+	"gorm.io/gorm"
+)
+
+var _202408291715_app_metadata = &gormigrate.Migration{
+	ID: "202408291715_app_metadata",
+	Migrate: func(tx *gorm.DB) error {
+
+		if err := tx.Exec(`
+	ALTER TABLE apps ADD COLUMN metadata JSON;
+`).Error; err != nil {
+			return err
+		}
+
+		return nil
+	},
+	Rollback: func(tx *gorm.DB) error {
+		return nil
+	},
+}

db/migrations/migrate.go

@@ -21,6 +21,7 @@ func Migrate(gormDB *gorm.DB) error {
 		_202407262257_remove_invalid_scopes,
 		_202408061737_add_boostagrams_and_use_json,
 		_202408191242_transaction_failure_reason,
+		_202408291715_app_metadata,
 	})
 
 	return m.Migrate()

db/models.go

@@ -23,6 +23,7 @@ type App struct {
 	CreatedAt   time.Time
 	UpdatedAt   time.Time
 	Isolated    bool
+	Metadata    datatypes.JSON
 }
 
 type AppPermission struct {
@@ -86,7 +87,7 @@ type Transaction struct {
 }
 
 type DBService interface {
-	CreateApp(name string, pubkey string, maxAmountSat uint64, budgetRenewal string, expiresAt *time.Time, scopes []string, isolated bool) (*App, string, error)
+	CreateApp(name string, pubkey string, maxAmountSat uint64, budgetRenewal string, expiresAt *time.Time, scopes []string, isolated bool, metadata map[string]interface{}) (*App, string, error)
 }
 
 const (

fly.toml

@@ -6,6 +6,7 @@
 app = 'nwc'
 primary_region = 'lax'
 swap_size_mb = 2048
+kill_timeout = 120
 
 [build]
   image = 'ghcr.io/getalby/hub:latest'

frontend/src/components/AppAvatar.tsx

@@ -1,27 +1,48 @@
+import { suggestedApps } from "src/components/SuggestedAppData";
+import UserAvatar from "src/components/UserAvatar";
 import { cn } from "src/lib/utils";
+import { App } from "src/types";
 
 type Props = {
-  appName: string;
+  app: App;
   className?: string;
 };
 
-export default function AppAvatar({ appName, className }: Props) {
+export default function AppAvatar({ app, className }: Props) {
+  if (app.name === "getalby.com") {
+    return <UserAvatar className={className} />;
+  }
+  const appStoreApp = app?.metadata?.app_store_app_id
+    ? suggestedApps.find(
+        (suggestedApp) => suggestedApp.id === app.metadata?.app_store_app_id
+      )
+    : undefined;
+  const image = appStoreApp?.logo;
+
   const gradient =
-    appName
+    app.name
       .split("")
       .map((c) => c.charCodeAt(0))
       .reduce((a, b) => a + b, 0) % 10;
   return (
     <div
       className={cn(
-        "rounded-lg border relative",
-        `avatar-gradient-${gradient}`,
+        "rounded-lg border relative overflow-hidden",
+        !image && `avatar-gradient-${gradient}`,
         className
       )}
     >
-      <span className="absolute top-1/2 left-1/2 transform -translate-x-1/2 -translate-y-1/2 text-white text-xl font-medium capitalize">
-        {appName.charAt(0)}
-      </span>
+      {image && (
+        <img
+          src={image}
+          className={cn("absolute w-full h-full rounded-lg", className)}
+        />
+      )}
+      {!image && (
+        <span className="absolute top-1/2 left-1/2 transform -translate-x-1/2 -translate-y-1/2 text-white text-xl font-medium capitalize">
+          {app.name.charAt(0)}
+        </span>
+      )}
     </div>
   );
 }

frontend/src/components/SuggestedAppData.tsx

@@ -9,9 +9,9 @@ import nostur from "src/assets/suggested-apps/nostur.png";
 import paperScissorsHodl from "src/assets/suggested-apps/paper-scissors-hodl.png";
 import primal from "src/assets/suggested-apps/primal.png";
 import snort from "src/assets/suggested-apps/snort.png";
-import stackernews from "src/assets/suggested-apps/stackernews.png";
+import stackernews from "src/assets/suggested-apps/stacker-news.png";
 import uncleJim from "src/assets/suggested-apps/uncle-jim.png";
-import wavelake from "src/assets/suggested-apps/wavelake.png";
+import wavlake from "src/assets/suggested-apps/wavlake.png";
 import wherostr from "src/assets/suggested-apps/wherostr.png";
 import yakihonne from "src/assets/suggested-apps/yakihonne.png";
 import zapstream from "src/assets/suggested-apps/zap-stream.png";
@@ -85,7 +85,7 @@ export const suggestedApps: SuggestedApp[] = [
     title: "Wavlake",
     description: "Creators platform",
     webLink: "https://www.wavlake.com/",
-    logo: wavelake,
+    logo: wavlake,
   },
   {
     id: "snort",

frontend/src/components/SuggestedApps.tsx

@@ -1,4 +1,4 @@
-import { Globe } from "lucide-react";
+import { ExternalLinkIcon, Globe } from "lucide-react";
 import { Link } from "react-router-dom";
 import ExternalLink from "src/components/ExternalLink";
 import { AppleIcon } from "src/components/icons/Apple";
@@ -84,7 +84,7 @@ function InternalAppCard({ id, title, description, logo }: SuggestedApp) {
       <CardFooter className="flex flex-row justify-end">
         <Link to={`/internal-apps/${id}`}>
           <Button variant="outline">
-            <NostrWalletConnectIcon className="w-4 h-4 mr-2" />
+            <ExternalLinkIcon className="w-4 h-4 mr-2" />
             Open
           </Button>
         </Link>