util: strip v prefix from version number in versioninfo generator

.github/workflows/build-docker.yml

@@ -3,6 +3,8 @@ on:
   push:
     tags:
       - v*
+    branches:
+      - master
   workflow_dispatch:
 
 env:
@@ -37,6 +39,11 @@ jobs:
         uses: docker/metadata-action@v5
         with:
           images: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}
+          tags: |
+            type=schedule
+            type=ref,event=branch
+            type=ref,event=tag
+            type=raw,value=bleeding,enable=${{ github.ref_name == 'master' }}
 
       - name: Build and push Docker image
         uses: docker/build-push-action@v6

.github/workflows/build.yml

@@ -12,11 +12,11 @@ jobs:
     runs-on: macos-13
     steps:
     - name: Checkout
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
     - name: Setup Go
-      uses: actions/setup-go@v4
+      uses: actions/setup-go@v6
       with:
-        go-version: '^1.23'
+        go-version-file: 'go.mod'
     - name: Build arm64 gui
       uses: ./.github/actions/prepare
       with:
@@ -47,11 +47,11 @@ jobs:
     runs-on: windows-2022
     steps:
     - name: Checkout
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
     - name: Setup Go
-      uses: actions/setup-go@v4
+      uses: actions/setup-go@v6
       with:
-        go-version: '^1.23'
+        go-version-file: 'go.mod'
     - name: Build amd64 gui
       uses: ./.github/actions/prepare
       with:
@@ -71,11 +71,11 @@ jobs:
     runs-on: ubuntu-22.04
     steps:
     - name: Checkout
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
     - name: Setup Go
-      uses: actions/setup-go@v4
+      uses: actions/setup-go@v6
       with:
-        go-version: '^1.23'
+        go-version-file: 'go.mod'
     - name: Setup Zig
       uses: mlugg/setup-zig@v1
       with:

.github/workflows/test.yml

@@ -6,14 +6,17 @@ jobs:
   test:
     runs-on: ubuntu-latest
 
+    env:
+      YARR_POSTGRES_TEST_IMAGE: postgres:17-alpine
+
     steps:
     - name: Checkout code
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
 
     - name: Set up Go
-      uses: actions/setup-go@v4
+      uses: actions/setup-go@v6
       with:
-        go-version: '^1.18'
+        go-version-file: 'go.mod'
 
     - name: Run tests
       run: make test

cmd/yarr/main.go

@@ -73,7 +73,7 @@ func main() {
 	flag.Parse()
 
 	if ver {
-		fmt.Printf("v%s (%s)\n", Version, GitHash)
+		fmt.Printf("%s (%s)\n", Version, GitHash)
 		return
 	}
 

doc/changelog.md

@@ -1,10 +1,14 @@
 # upcoming
 
+- (new) initial PostgreSQL support
+- (new) i18n: English, Chinese, French, German, Japanese, Portuguese, Russian, Spanish
 - (fix) articles not resetting immediately after feed/filter selection (thank to @scratchmex for the report)
 - (fix) crash on empty article list with article is selected (thanks to @rksvc)
 - (fix) invalid article title in RSS feeds with media containing titles (thanks to @bwwu-git for the report)
 - (fix) missing image enclosures in certain RSS feeds (thanks to @palinek for the report)
 - (fix) parsing namespaced legacy RSS feeds (thanks to @f100024)
+- (fix) marking feeds read in Fever API (thanks to @weskoop)
+- (etc) systray improvements for macOS
 
 # v2.6 (2025-11-24)
 

etc/windows_versioninfo.sh

@@ -51,8 +51,9 @@ while [[ $# -gt 0 ]]; do
   esac
 done
 
-# Replace dots with commas for version_comma
-version_comma="${version//./,}"
+# Strip leading 'v' and replace dots with commas for version_comma
+version_num="${version#v}"
+version_comma="${version_num//./,}"
 
 # Use a here document for the template with ENDFILE delimiter
 cat <<ENDFILE > "$outfile"

go.mod

@@ -1,8 +1,6 @@
 module github.com/nkanaev/yarr
 
-go 1.23.0
-
-toolchain go1.23.5
+go 1.25.0
 
 require (
 	fyne.io/systray v1.12.0

makefile

@@ -1,4 +1,4 @@
-VERSION=2.6
+VERSION=$(shell git describe --exact-match --tags HEAD 2>/dev/null || echo bleeding)
 GITHASH=$(shell git rev-parse --short=8 HEAD)
 
 GO_TAGS    = sqlite_foreign_keys sqlite_json sqlite_fts5

src/storage/tests/item_test.go

@@ -291,14 +291,12 @@ func TestListItemsPaginated(t *testing.T) {
 	})
 }
 
-func TestMarkItemsRead(t *testing.T) {
-	// NOTE: starred items must not be marked as read
+func TestMarkAllItemsRead(t *testing.T) {
 	var read model.ItemStatus = model.READ
-
-	dbtest(t, func(t *testing.T, db1 storage.Storage) {
-		testItemsSetup(db1)
-		db1.MarkItemsRead(model.MarkFilter{})
-		have := getItemGuids(db1.ListItems(model.ItemFilter{Status: &read}, 10, false, false))
+	dbtest(t, func(t *testing.T, db storage.Storage) {
+		testItemsSetup(db)
+		db.MarkItemsRead(model.MarkFilter{})
+		have := getItemGuids(db.ListItems(model.ItemFilter{Status: &read}, 10, false, false))
 		want := []string{
 			"item111", "item112", "item121", "item122",
 			"item211", "item011", "item012",
@@ -309,11 +307,14 @@ func TestMarkItemsRead(t *testing.T) {
 			t.Fail()
 		}
 	})
+}
 
-	dbtest(t, func(t *testing.T, db2 storage.Storage) {
-		scope2 := testItemsSetup(db2)
-		db2.MarkItemsRead(model.MarkFilter{FolderID: &scope2.folder1.Id})
-		have := getItemGuids(db2.ListItems(model.ItemFilter{Status: &read}, 10, false, false))
+func TestMarkItemsReadByFolder(t *testing.T) {
+	var read model.ItemStatus = model.READ
+	dbtest(t, func(t *testing.T, db storage.Storage) {
+		scope := testItemsSetup(db)
+		db.MarkItemsRead(model.MarkFilter{FolderID: &scope.folder1.Id})
+		have := getItemGuids(db.ListItems(model.ItemFilter{Status: &read}, 10, false, false))
 		want := []string{
 			"item111", "item112", "item121", "item122",
 			"item211", "item012",
@@ -324,11 +325,14 @@ func TestMarkItemsRead(t *testing.T) {
 			t.Fail()
 		}
 	})
+}
 
-	dbtest(t, func(t *testing.T, db3 storage.Storage) {
-		scope3 := testItemsSetup(db3)
-		db3.MarkItemsRead(model.MarkFilter{FeedID: &scope3.feed11.Id})
-		have := getItemGuids(db3.ListItems(model.ItemFilter{Status: &read}, 10, false, false))
+func TestMarkItemsReadByFeed(t *testing.T) {
+	var read model.ItemStatus = model.READ
+	dbtest(t, func(t *testing.T, db storage.Storage) {
+		scope := testItemsSetup(db)
+		db.MarkItemsRead(model.MarkFilter{FeedID: &scope.feed11.Id})
+		have := getItemGuids(db.ListItems(model.ItemFilter{Status: &read}, 10, false, false))
 		want := []string{
 			"item111", "item112", "item122",
 			"item211", "item012",

src/storage/tests/storage_test.go

@@ -1,30 +1,32 @@
 package tests
 
 import (
-	"crypto/rand"
+	"context"
+	"crypto/sha256"
 	"database/sql"
-	"encoding/hex"
 	"fmt"
-	"net/url"
 	"os"
+	"os/exec"
+	"strings"
 	"testing"
+	"time"
 
 	_ "github.com/lib/pq"
 	"github.com/nkanaev/yarr/src/storage"
 )
 
 func dbtest(t *testing.T, testcase func(t *testing.T, db storage.Storage)) {
+	t.Parallel()
 	testurls := map[string]string{
 		"sqlite": ":memory:",
 	}
 
-	if pgUrl := os.Getenv("YARR_POSTGRES_TEST_URL"); pgUrl != "" {
-		dburl, cleanup, err := createPostgresDB(pgUrl)
-		if err != nil {
-			t.Fatalf("failed to create postgres test database: %v", err)
-		}
+	if pgImage := os.Getenv("YARR_POSTGRES_TEST_IMAGE"); pgImage != "" {
+		dburl, cleanup := startPostgresContainer(t, pgImage)
 		t.Cleanup(cleanup)
 		testurls["postgres"] = dburl
+	} else if !testing.Short() {
+		t.Fatalf("YARR_POSTGRES_TEST_IMAGE not set; use -short to skip docker tests")
 	}
 
 	for testname, url := range testurls {
@@ -38,49 +40,72 @@ func dbtest(t *testing.T, testcase func(t *testing.T, db storage.Storage)) {
 	}
 }
 
-func createPostgresDB(pgUrl string) (string, func(), error) {
-	u, err := url.Parse(pgUrl)
+func startPostgresContainer(t *testing.T, image string) (string, func()) {
+	// database credentials
+	dbUser := "testuser"
+	dbPass := "password"
+	dbName := "yarrtest"
+
+	// generate unique container name
+	testHash := sha256.Sum256([]byte(t.Name()))
+	containerName := fmt.Sprintf("yarr-test-pg-%x-%d", testHash[:8], time.Now().UnixNano())
+
+	cmd := exec.Command(
+		"docker", "run", "-d", "--rm",
+		"--name", containerName,
+		"-p", "0:5432",
+		"-e", "POSTGRES_USER="+dbUser,
+		"-e", "POSTGRES_PASSWORD="+dbPass,
+		"-e", "POSTGRES_DB="+dbName,
+		image,
+	)
+	out, err := cmd.CombinedOutput()
 	if err != nil {
-		return "", nil, err
+		t.Fatalf("failed to start postgres container: %v\n%s", err, string(out))
 	}
 
-	u.Path = "/postgres"
-	adminConnStr := u.String()
-
-	adminDB, err := sql.Open("postgres", adminConnStr)
+	// retrieve the host port assigned by docker
+	portCmd := exec.Command("docker", "port", containerName, "5432/tcp")
+	portOut, err := portCmd.Output()
 	if err != nil {
-		return "", nil, fmt.Errorf("admin connect: %w", err)
+		t.Fatalf("failed to get container port: %v", err)
 	}
+	parts := strings.Split(strings.TrimSpace(string(portOut)), ":")
+	dbPort := parts[len(parts)-1]
 
-	b := make([]byte, 4)
-	if _, err := rand.Read(b); err != nil {
-		adminDB.Close()
-		return "", nil, fmt.Errorf("generate suffix: %w", err)
-	}
+	// build connection string
+	pgUrl := fmt.Sprintf(
+		"postgres://%s:%s@localhost:%s/%s?sslmode=disable",
+		dbUser,
+		dbPass,
+		dbPort,
+		dbName,
+	)
 
-	testDBName := "yarr_test_" + hex.EncodeToString(b)
-
-	if _, err := adminDB.Exec(fmt.Sprintf(`CREATE DATABASE "%s"`, testDBName)); err != nil {
-		adminDB.Close()
-		return "", nil, fmt.Errorf("create database: %w", err)
-	}
-	adminDB.Close()
-
-	u.Path = "/" + testDBName
-	testURL := u.String()
-
-	cleanup := func() {
-		dropDB, err := sql.Open("postgres", adminConnStr)
+	// wait up to 15 seconds for the container to accept connections
+	deadline := time.Now().Add(15 * time.Second)
+	for time.Now().Before(deadline) {
+		db, err := sql.Open("postgres", pgUrl)
 		if err != nil {
-			return
+			continue
 		}
-		defer dropDB.Close()
-		dropDB.Exec(fmt.Sprintf(
-			`SELECT pg_terminate_backend(pid) FROM pg_stat_activity WHERE datname = '%s' AND pid <> pg_backend_pid()`,
-			testDBName,
-		))
-		dropDB.Exec(fmt.Sprintf(`DROP DATABASE IF EXISTS "%s"`, testDBName))
+		ctx, cancel := context.WithTimeout(context.Background(), 200*time.Millisecond)
+		err = db.PingContext(ctx)
+		cancel()
+		db.Close()
+		if err == nil {
+			goto ready
 		}
+		time.Sleep(200 * time.Millisecond)
+	}
+	t.Fatalf("timed out waiting for postgres container to be ready")
 
-	return testURL, cleanup, nil
+ready:
+	// return connection url and a cleanup function that stops the container
+	return pgUrl, func() {
+		stop := exec.Command("docker", "stop", containerName)
+		if err := stop.Run(); err != nil {
+			t.Logf("failed to stop container %s: %v", containerName, err)
+		}
+	}
 }